Messages in this thread | | | From | Ratheesh Kannoth <> | Subject | RE: [EXT] Re: [PATCH v1 net] page_pool: Cap queue size to 32k. | Date | Mon, 14 Aug 2023 08:55:14 +0000 |
| |
> From: Jesper Dangaard Brouer <hawk@kernel.org> > Subject: Re: [EXT] Re: [PATCH v1 net] page_pool: Cap queue size to 32k.
> I agree with Johannes, this commit message is too thin. ACK.
> It makes sense to give a summary of the discussion, because it show us > (page_pool maintainers) what you concluded for the discussion. Got it. Thanks.
> Further more, you also send another patch: > - "[PATCH net-next] page_pool: Set page pool size" Okay.
> - > https://urldefense.proofpoint.com/v2/url?u=https- > 3A__lore.kernel.org_all_20230809021920.913324-2D1-2Drkannoth- > 40marvell.com_&d=DwICaQ&c=nKjWec2b6R0mOyPaz7xtfQ&r=aekcsyBCH00 > _LewrEDcQBzsRw8KCpUR0vZb_auTHk4M&m=uvV_vt_cNyQItTD90jF1LdKovP > 7j7FYtnr7I38__nYY6wHtFHSozYoRSSvCI14nh&s=vGgt2ccGdiRTEhj3MoGVx- > EXHmB03v6I3UIIY1fEb24&e= > > That patch solves the issue for your driver marvell/octeontx2 and I like than > change. Okay.
> Why did you conclude that PP core should also change? I could not answer Jacub's question at https://lore.kernel.org/netdev/20230810024422.1781312-1-rkannoth@marvell.com/T/
> (p.s. Cc/To list have gotten excessive with 89 recipients) I added maintainters of all files which used page_pool_init().
| |