netdev - Re: Re: [PATCH net-next v3] octeontx2-pf: Add support for page pool

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date: Fri, 19 May 2023 15:20:52 +0200
From: Simon Horman <simon.horman@...igine.com>
To: Ratheesh Kannoth <rkannoth@...vell.com>
Cc: "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Sunil Kovvuri Goutham <sgoutham@...vell.com>,
	"davem@...emloft.net" <davem@...emloft.net>,
	"edumazet@...gle.com" <edumazet@...gle.com>,
	"kuba@...nel.org" <kuba@...nel.org>,
	"pabeni@...hat.com" <pabeni@...hat.com>,
	"linyunsheng@...wei.com" <linyunsheng@...wei.com>,
	Subbaraya Sundeep Bhatta <sbhatta@...vell.com>,
	Geethasowjanya Akula <gakula@...vell.com>,
	Srujana Challa <schalla@...vell.com>,
	Hariprasad Kelam <hkelam@...vell.com>
Subject: Re: Re: [PATCH net-next v3] octeontx2-pf: Add support for page pool

On Fri, May 19, 2023 at 10:21:44AM +0000, Ratheesh Kannoth wrote:
> 
> > -----Original Message-----
> > From: Simon Horman <simon.horman@...igine.com>
> > Sent: Friday, May 19, 2023 3:33 PM
> > To: Ratheesh Kannoth <rkannoth@...vell.com>
> > Cc: netdev@...r.kernel.org; linux-kernel@...r.kernel.org; Sunil Kovvuri
> > Goutham <sgoutham@...vell.com>; davem@...emloft.net;
> > edumazet@...gle.com; kuba@...nel.org; pabeni@...hat.com;
> > linyunsheng@...wei.com; Subbaraya Sundeep Bhatta
> > <sbhatta@...vell.com>; Geethasowjanya Akula <gakula@...vell.com>;
> > Srujana Challa <schalla@...vell.com>; Hariprasad Kelam
> > <hkelam@...vell.com>
> > Subject: [EXT] Re: [PATCH net-next v3] octeontx2-pf: Add support for page
> > pool
> > 
> > External Email
> > 
> > ----------------------------------------------------------------------
> > On Fri, May 19, 2023 at 12:43:52PM +0530, Ratheesh Kannoth wrote:
> > > Page pool for each rx queue enhance rx side performance by reclaiming
> > > buffers back to each queue specific pool. DMA mapping is done only for
> > > first allocation of buffers.
> > > As subsequent buffers allocation avoid DMA mapping, it results in
> > > performance improvement.
> > >
> > > Image        |  Performance
> > > ------------ | ------------
> > > Vannila      |   3Mpps
> > >              |
> > > with this    |   42Mpps
> > > change	     |
> > > ---------------------------
> > >
> > > Signed-off-by: Ratheesh Kannoth <rkannoth@...vell.com>
> > 
> > ...
> > 
> > > @@ -1205,10 +1226,28 @@ void otx2_sq_free_sqbs(struct otx2_nic *pfvf)
> > >  	}
> > >  }
> > >
> > > +void otx2_free_bufs(struct otx2_nic *pfvf, struct otx2_pool *pool,
> > > +		    u64 iova, int size)
> > > +{
> > > +	u64 pa = otx2_iova_to_phys(pfvf->iommu_domain, iova);
> > > +	struct page *page = virt_to_head_page(phys_to_virt(pa));
> > 
> > nit: please arrange local variables in networking code in reverse xmas tree
> >      order - longest line to shortest.
> Variable "pa" is used in second line.  Are you suggesting to defer assignment later; and only declare variables here in reverse xmas tree style ? 

Yes, that is my suggestion.