NAK/cmnt: [SRU][F:linux-bluefield][PATCH V1 1/3] net: openvswitch: Be liberal in tcp conntrack.

[Bodong Wang]

On 2/18/2022 10:56 AM, Kleber Souza wrote:
> Hi Bodong,
>
> On 18.02.22 16:46, Bodong Wang wrote:
>> Hi Kleber,
>>
>> Could you be more specific for the misalignment? I can only tell Paul 
>> added his own signoff which is expected as he did the backport.
>
> OK, that's more clear to me now what process you are following. Thanks 
> for the
> clarification.
>
> When we state that we are cherry-picking or backporting a patch from 
> upstream/mainline,
> we assume that the patch was taken from that particular tree and the 
> provenance block
> should be kept intact before the "(cherry picked|backported from ...") 
> line.
>
> So if Paul Blakey did the cherry-pick/backport, the provenance block 
> would
> look like:
>
> Suggested-by: Florian Westphal <fw at strlen.de>
> Signed-off-by: Numan Siddique <nusiddiq at redhat.com>
> Acked-by: Florian Westphal <fw at strlen.de>
> Link: 
> https://lore.kernel.org/r/20201116130126.3065077-1-nusiddiq@redhat.com
> Signed-off-by: Jakub Kicinski <kuba at kernel.org>
> (cherry picked from commit e2ef5203c817a60bfb591343ffd851b6537370ff)
> Signed-off-by: Paul Blakey <paulb at nvidia.com>
> Signed-off-by: Bodong Wang <bodong at nvidia.com>
>
> Please note that when someone is part of the work of 
> cherry-picking/backporting/
> submitting a patch to an Ubuntu kernel, the signed-off-by should be 
> added at the bottom
> even if the person is the original author of that patch.
>
> For patch 2/3 ("net/sched: act_ct: Fill offloading tuple iifidx"), 
> this would be:
>
> Signed-off-by: Oz Shlomo <ozsh at nvidia.com>
>
> Signed-off-by: Paul Blakey <paulb at nvidia.com>
>
> Signed-off-by: David S. Miller <davem at davemloft.net>
>
> (cherry picked from commit 9795ded7f924b6486e54976619b1b094fcc1969d)
> Signed-off-by: Paul Blakey <paulb at nvidia.com>
> Signed-off-by: Bodong Wang <bodong at nvidia.com>
>
> Note that in the patch submitted with this thread it was missing the 
> s-o-b of the
> upstream maintainer (David Miller).
>
> I hope this helps and sorry for the nitpicking, this is needed to keep 
> the
> complete history and origin of the patch.
>
> Thanks,
> Kleber
>
Fixed and sent V2, please review.