Re: [PATCH net-next 1/2] bnxt_en: Fix W=1 warning in bnxt_dcb.c from fortify memcpy()

["Gustavo A. R. Silva" <gustavoars@kernel.org>]

On Mon, Aug 07, 2023 at 07:57:19AM -0700, Michael Chan wrote:
> Fix the following warning:
> 
> inlined from ‘bnxt_hwrm_queue_cos2bw_qcfg’ at drivers/net/ethernet/broadcom/bnxt/bnxt_dcb.c:165:3,
> ./include/linux/fortify-string.h:592:4: error: call to ‘__read_overflow2_field’ declared with attribute warning: detected read beyond size of field (2nd parameter); maybe use struct_group()? [-Werror]
>     __read_overflow2_field(q_size_field, size);
>     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 
> Modify the FW interface defintion of struct hwrm_queue_cos2bw_qcfg_output
> to use an array of sub struct for the queue1 to queue7 fields.  Note that
> the layout of the queue0 fields are different and these are not part of
> the array.  This makes the code much cleaner by removing the pointer
> arithmetic for memcpy().

Thanks for fixing this. :)

> 
> Link: https://lore.kernel.org/netdev/20230727190726.1859515-2-kuba@kernel.org/
> Reviewed-by: Andy Gospodarek <andrew.gospodarek@broadcom.com>
> Signed-off-by: Michael Chan <michael.chan@broadcom.com>

Reviewed-by: Gustavo A. R. Silva <gustavoars@kernel.org>

--
Gustavo