Commit 8467fadc authored by Gal Pressman's avatar Gal Pressman Committed by David S. Miller
Browse files

net: gro: Fix a 'directive in macro's argument list' sparse warning 



Following the cited commit, sparse started complaining about:
../include/net/gro.h:58:1: warning: directive in macro's argument list
../include/net/gro.h:59:1: warning: directive in macro's argument list

Fix that by moving the defines out of the struct_group() macro.

Fixes: de5a1f3c ("net: gro: minor optimization for dev_gro_receive()")
Reviewed-by: default avatarMaxim Mikityanskiy <maximmi@nvidia.com>
Signed-off-by: default avatarGal Pressman <gal@nvidia.com>
Acked-by: default avatarAlexander Lobakin <alexandr.lobakin@intel.com>
Acked-by: default avatarPaolo Abeni <pabeni@redhat.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 129c77b5

include/net/gro.h

+3 −2
Original line number Diff line number Diff line
@@ -35,6 +35,9 @@ struct napi_gro_cb { 
	/* jiffies when first packet was created/queued */

	unsigned long age;



/* Used in napi_gro_cb::free */

#define NAPI_GRO_FREE             1

#define NAPI_GRO_FREE_STOLEN_HEAD 2

	/* portion of the cb set to zero at every gro iteration */

	struct_group(zeroed,
@@ -55,8 +58,6 @@ struct napi_gro_cb { 


		/* Free the skb? */

		u8	free:2;

#define NAPI_GRO_FREE		  1

#define NAPI_GRO_FREE_STOLEN_HEAD 2



		/* Used in foo-over-udp, set in udp[46]_gro_receive */

		u8	is_ipv6:1;