netdev - Re: [PATCH v1 net] page_pool: Cap queue size to 32k.

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] [thread-next>] [day] [month] [year] [list]

Date: Mon, 14 Aug 2023 09:54:56 +0200
From: Johannes Berg <johannes@...solutions.net>
To: Ratheesh Kannoth <rkannoth@...vell.com>, netdev@...r.kernel.org, 
	linux-kernel@...r.kernel.org
Cc: davem@...emloft.net, edumazet@...gle.com, kuba@...nel.org,
 pabeni@...hat.com,  ast@...nel.org, daniel@...earbox.net, hawk@...nel.org,
 john.fastabend@...il.com,  jiawenwu@...stnetic.com,
 mengyuanlou@...-swift.com, yang.lee@...ux.alibaba.com,  error27@...il.com,
 linyunsheng@...wei.com, linux-hyperv@...r.kernel.org,  kys@...rosoft.com,
 haiyangz@...rosoft.com, wei.liu@...nel.org, decui@...rosoft.com, 
 longli@...rosoft.com, shradhagupta@...ux.microsoft.com, 
 linux-hwmon@...r.kernel.org, michael.chan@...adcom.com,
 richardcochran@...il.com,  jdelvare@...e.com, linux@...ck-us.net,
 yisen.zhuang@...wei.com,  salil.mehta@...wei.com,
 linux-arm-kernel@...ts.infradead.org,  linux-mediatek@...ts.infradead.org,
 nbd@....name, john@...ozen.org,  sean.wang@...iatek.com,
 Mark-MC.Lee@...iatek.com, lorenzo@...nel.org,  matthias.bgg@...il.com,
 angelogioacchino.delregno@...labora.com,  linux@...linux.org.uk,
 linux-rdma@...r.kernel.org, saeedm@...dia.com,  leon@...nel.org,
 gerhard@...leder-embedded.com, maciej.fijalkowski@...el.com, 
 alexanderduyck@...com, wei.fang@....com, shenwei.wang@....com, 
 xiaoning.wang@....com, linux-imx@....com, lgirdwood@...il.com,
 broonie@...nel.org,  jaswinder.singh@...aro.org,
 ilias.apalodimas@...aro.org,  UNGLinuxDriver@...rochip.com,
 horatiu.vultur@...rochip.com,  linux-omap@...r.kernel.org,
 grygorii.strashko@...com, simon.horman@...igine.com, 
 vladimir.oltean@....com, aleksander.lobakin@...el.com, 
 linux-stm32@...md-mailman.stormreply.com, alexandre.torgue@...s.st.com, 
 joabreu@...opsys.com, mcoquelin.stm32@...il.com, p.zabel@...gutronix.de, 
 thomas.petazzoni@...tlin.com, mw@...ihalf.com, sgoutham@...vell.com, 
 gakula@...vell.com, sbhatta@...vell.com, hkelam@...vell.com, 
 xen-devel@...ts.xenproject.org, jgross@...e.com, sstabellini@...nel.org, 
 oleksandr_tyshchenko@...m.com, linux-wireless@...r.kernel.org, 
 ryder.lee@...iatek.com, shayne.chen@...iatek.com, kvalo@...nel.org, 
 andrii@...nel.org, martin.lau@...ux.dev, song@...nel.org,
 yonghong.song@...ux.dev,  kpsingh@...nel.org, sdf@...gle.com,
 haoluo@...gle.com, jolsa@...nel.org
Subject: Re: [PATCH v1 net] page_pool: Cap queue size to 32k.

On Mon, 2023-08-14 at 11:34 +0530, Ratheesh Kannoth wrote:
> Clamp to 32k instead of returning error.
> 
> Please find discussion at
> https://lore.kernel.org/lkml/
> CY4PR1801MB1911E15D518A77535F6E51E2D308A@...PR1801MB1911.
> namprd18.prod.outlook.com/T/
> 

I'm not the one who's going to apply this, but honestly, I don't think
that will work as a commit message for such a change ...

Sure, link to it by all means, but also summarize it and make sense of
it for the commit message?

johannes