[PATCH v3] Add payload to be 32-bit aligned to fix dropped packets

Kumar Thangavel kumarthangavel.hcl at gmail.com
Wed Oct 27 03:38:48 AEDT 2021


On Tue, Oct 26, 2021 at 9:03 PM Jakub Kicinski <kuba at kernel.org> wrote:

> On Tue, 26 Oct 2021 20:19:53 +0530 Kumar Thangavel wrote:
> > Update NC-SI command handler (both standard and OEM) to take into
> > account of payload paddings in allocating skb (in case of payload
> > size is not 32-bit aligned).
> >
> > The checksum field follows payload field, without taking payload
> > padding into account can cause checksum being truncated, leading to
> > dropped packets.
> >
> > Signed-off-by: Kumar Thangavel <kumarthangavel.hcl at gmail.com>
> > Acked-by: Samuel Mendoza-Jonas <sam at mendozajonas.com>
> > Reviewed-by: Paul Menzel <pmenzel at molgen.mpg.de>
>
> You need to CC netdev@ if you want this applied to networking trees.
>
> > diff --git a/net/ncsi/ncsi-cmd.c b/net/ncsi/ncsi-cmd.c
> > index ba9ae482141b..179e39105072 100644
> > --- a/net/ncsi/ncsi-cmd.c
> > +++ b/net/ncsi/ncsi-cmd.c
> > @@ -18,6 +18,8 @@
> >  #include "internal.h"
> >  #include "ncsi-pkt.h"
> >
> > +#define MAX(X, Y) (((X) > (Y)) ? (X) : (Y))
>
> Please use one of the existing max() macroes in the kernel.
>

Ack.  I will use existing macros.

Thanks,
Kumar.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ozlabs.org/pipermail/openbmc/attachments/20211026/d80af9dd/attachment.htm>


More information about the openbmc mailing list