Re: [PATCH net-next] bnxt_en: Properly get address type of encapsulation IP headers

From: David Miller
Date: Mon Sep 03 2018 - 22:20:08 EST


From: YueHaibing <yuehaibing@xxxxxxxxxx>
Date: Sat, 1 Sep 2018 17:25:29 +0800

> gcc '-Wunused-but-set-variable' warning:
>
> drivers/net/ethernet/broadcom/bnxt/bnxt_tc.c: In function 'bnxt_tc_parse_flow':
> drivers/net/ethernet/broadcom/bnxt/bnxt_tc.c:186:6: warning:
> variable 'addr_type' set but not used [-Wunused-but-set-variable]
>
> As done elsewhere in TC/flower offload code, the address type of
> the encapsulation IP headers should be realized accroding to the
> addr_type field of the encapsulation control dissector key.
>
> Fixes: 8c95f773b4a3 ("bnxt_en: add support for Flower based vxlan encap/decap offload")
> Fixes: 2ae7408fedfe ("bnxt_en: bnxt: add TC flower filter offload support")
> Signed-off-by: YueHaibing <yuehaibing@xxxxxxxxxx>

Michael, please review.

And, if this fixes a bug, it should target 'net' not 'net-next'.