Re: [PATCH 2/6] flow, tcp_splice: Prefer 'sidei' for variables referring to side index

On Wed, 17 Jul 2024 15:50:32 +1000
David Gibson <david(a)gibson.dropbear.id.au> wrote:

...
  On Wed, Jul 17, 2024 at 03:28:05PM +1000, David Gibson
wrote:
  On Wed, Jul 17, 2024 at 07:20:17AM +0200, Stefano
Brivio wrote:  
  On Wed, 17 Jul 2024 14:52:19 +1000
 David Gibson <david(a)gibson.dropbear.id.au> wrote:
   
  @@ -164,17 +164,17 @@ struct flow_common {
  
  /**
   * struct flow_sidx - ID for one side of a specific flow
 - * @side:	Side referenced (0 or 1)
 + * @side:	Index of side referenced (0 or 1)
   * @flow:	Index of flow referenced   
 
 Nit (I can fix this up on merge, and I didn't finish reviewing yet):
 the comment to the struct should also use sidei/flowi.   
 
 If this is the only issue, go ahead and fix on merge.  If there are
 any more nits I'll make a new spin.   
 
 Actually, never mind.  I discovered I hadn't rebased onto the latest
 and there's a conflict, so I'll respin anyway. 
It's trivial, just a bit of fuzz on 1/6, no need as far as I'm
concerned.

-- 
Stefano