Re: [PATCH net-next 1/2] net: Keep sk->sk_forward_alloc as a proper size

[Simon Horman <simon.horman@corigine.com>]

On Tue, May 09, 2023 at 06:57:44AM +0000, Zhang, Cathy wrote:
> 
> 
> > -----Original Message-----
> > From: Jakub Kicinski <kuba@kernel.org>
> > Sent: Tuesday, May 9, 2023 10:06 AM
> > To: Zhang, Cathy <cathy.zhang@intel.com>
> > Cc: edumazet@google.com; davem@davemloft.net; pabeni@redhat.com;
> > Brandeburg, Jesse <jesse.brandeburg@intel.com>; Srinivas, Suresh
> > <suresh.srinivas@intel.com>; Chen, Tim C <tim.c.chen@intel.com>; You,
> > Lizhen <lizhen.you@intel.com>; eric.dumazet@gmail.com;
> > netdev@vger.kernel.org
> > Subject: Re: [PATCH net-next 1/2] net: Keep sk->sk_forward_alloc as a proper
> > size
> > 
> > On Sun,  7 May 2023 19:08:00 -0700 Cathy Zhang wrote:
> > > Fixes: 4890b686f408 ("net: keep sk->sk_forward_alloc as small as
> > > possible")
> > >
> > 
> > Ah, and for your future patches - no empty lines between trailers / tags,
> > please.
> 
> Sorry, I do not quite get your point here. Do you mean there should be no blanks between 'Fixes' line and 'Signed-off-by' line?

I'm not Jakub.
But, yes, I'm pretty sure that is what he means here.

> > 
> > > Signed-off-by: Cathy Zhang <cathy.zhang@intel.com>
> > > Signed-off-by: Lizhen You <lizhen.you@intel.com>
> > > Tested-by: Long Tao <tao.long@intel.com>
> > > Reviewed-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
> > > Reviewed-by: Tim Chen <tim.c.chen@linux.intel.com>
> > > Reviewed-by: Suresh Srinivas <suresh.srinivas@intel.com>
>