Re: [PATCH v5 2/4] net/bnx2x: fix warnings about rte_memcpy lengths

2023-01-09 Thread David Marchand
Hello bnx2x maintainers, On Wed, Dec 28, 2022 at 6:37 PM Morten Brørup wrote: > > > From: Stephen Hemminger [mailto:step...@networkplumber.org] > > Sent: Wednesday, 28 December 2022 18.03 > > > > On Wed, 28 Dec 2022 17:38:56 +0100 > > Morten Brørup wrote: > > > > > From: Stanisław Kardach [mail

RE: [PATCH v5 2/4] net/bnx2x: fix warnings about rte_memcpy lengths

2022-12-28 Thread Morten Brørup
> From: Stephen Hemminger [mailto:step...@networkplumber.org] > Sent: Wednesday, 28 December 2022 18.03 > > On Wed, 28 Dec 2022 17:38:56 +0100 > Morten Brørup wrote: > > > From: Stanisław Kardach [mailto:k...@semihalf.com] > > Sent: Wednesday, 28 December 2022 17.14 > > > On Wed, Dec 28, 2022, 1

Re: [PATCH v5 2/4] net/bnx2x: fix warnings about rte_memcpy lengths

2022-12-28 Thread Stephen Hemminger
On Wed, 28 Dec 2022 17:38:56 +0100 Morten Brørup wrote: > From: Stanisław Kardach [mailto:k...@semihalf.com] > Sent: Wednesday, 28 December 2022 17.14 > > On Wed, Dec 28, 2022, 16:10 Morten Brørup > > wrote: > > > Bugfix: The vlan in the bulletin does not contain a VLAN header, only the > >

RE: [PATCH v5 2/4] net/bnx2x: fix warnings about rte_memcpy lengths

2022-12-28 Thread Morten Brørup
From: Stanisław Kardach [mailto:k...@semihalf.com] Sent: Wednesday, 28 December 2022 17.14 > On Wed, Dec 28, 2022, 16:10 Morten Brørup wrote: > > Bugfix: The vlan in the bulletin does not contain a VLAN header, only the > > VLAN ID, so only copy 2 byte, not 4. The target structure has padding > >

Re: [PATCH v5 2/4] net/bnx2x: fix warnings about rte_memcpy lengths

2022-12-28 Thread Stanisław Kardach
On Wed, Dec 28, 2022, 16:10 Morten Brørup wrote: > Bugfix: The vlan in the bulletin does not contain a VLAN header, only the > VLAN ID, so only copy 2 byte, not 4. The target structure has padding > after the field, so copying 2 byte too many is effectively harmless. > It is a small nitpick but w