<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Oct 26, 2021 at 9:03 PM Jakub Kicinski <<a href="mailto:kuba@kernel.org">kuba@kernel.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tue, 26 Oct 2021 20:19:53 +0530 Kumar Thangavel wrote:<br>
> Update NC-SI command handler (both standard and OEM) to take into<br>
> account of payload paddings in allocating skb (in case of payload<br>
> size is not 32-bit aligned).<br>
> <br>
> The checksum field follows payload field, without taking payload<br>
> padding into account can cause checksum being truncated, leading to<br>
> dropped packets.<br>
> <br>
> Signed-off-by: Kumar Thangavel <<a href="mailto:kumarthangavel.hcl@gmail.com" target="_blank">kumarthangavel.hcl@gmail.com</a>><br>
> Acked-by: Samuel Mendoza-Jonas <<a href="mailto:sam@mendozajonas.com" target="_blank">sam@mendozajonas.com</a>><br>
> Reviewed-by: Paul Menzel <<a href="mailto:pmenzel@molgen.mpg.de" target="_blank">pmenzel@molgen.mpg.de</a>><br>
<br>
You need to CC netdev@ if you want this applied to networking trees.<br>
<br>
> diff --git a/net/ncsi/ncsi-cmd.c b/net/ncsi/ncsi-cmd.c<br>
> index ba9ae482141b..179e39105072 100644<br>
> --- a/net/ncsi/ncsi-cmd.c<br>
> +++ b/net/ncsi/ncsi-cmd.c<br>
> @@ -18,6 +18,8 @@<br>
>  #include "internal.h"<br>
>  #include "ncsi-pkt.h"<br>
>  <br>
> +#define MAX(X, Y) (((X) > (Y)) ? (X) : (Y))<br>
<br>
Please use one of the existing max() macroes in the kernel.<br></blockquote><div><br></div><div>Ack.  I will use existing macros.</div><div><br></div><div>Thanks,</div><div>Kumar.</div></div></div>