netdev - RE: [PATCH 0/8] net: Phase out hybrid PCI devres API

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]

Message-ID: <CO1PR11MB5089309753FBA1EDE6C2D663D6BD2@CO1PR11MB5089.namprd11.prod.outlook.com>
Date: Wed, 16 Apr 2025 17:10:38 +0000
From: "Keller, Jacob E" <jacob.e.keller@...el.com>
To: Philipp Stanner <phasta@...nel.org>, Sunil Goutham <sgoutham@...vell.com>,
	Andrew Lunn <andrew+netdev@...n.ch>, "David S. Miller" <davem@...emloft.net>,
	"Dumazet, Eric" <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>,
	Paolo Abeni <pabeni@...hat.com>, Geetha sowjanya <gakula@...vell.com>,
	Subbaraya Sundeep <sbhatta@...vell.com>, hariprasad <hkelam@...vell.com>,
	Bharat Bhushan <bbhushan2@...vell.com>, Taras Chornyi
	<taras.chornyi@...ision.eu>, Daniele Venzano <venza@...wnhat.org>, "Heiner
 Kallweit" <hkallweit1@...il.com>, Russell King <linux@...linux.org.uk>,
	Thomas Gleixner <tglx@...utronix.de>, Helge Deller <deller@....de>, "Ingo
 Molnar" <mingo@...nel.org>, Simon Horman <horms@...nel.org>, Al Viro
	<viro@...iv.linux.org.uk>, Sabrina Dubroca <sd@...asysnail.net>
CC: "linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>, "netdev@...r.kernel.org"
	<netdev@...r.kernel.org>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>, "linux-parisc@...r.kernel.org"
	<linux-parisc@...r.kernel.org>
Subject: RE: [PATCH 0/8] net: Phase out hybrid PCI devres API



> -----Original Message-----
> From: Philipp Stanner <phasta@...nel.org>
> Sent: Wednesday, April 16, 2025 9:44 AM
> To: Sunil Goutham <sgoutham@...vell.com>; Andrew Lunn
> <andrew+netdev@...n.ch>; David S. Miller <davem@...emloft.net>; Dumazet,
> Eric <edumazet@...gle.com>; Jakub Kicinski <kuba@...nel.org>; Paolo Abeni
> <pabeni@...hat.com>; Geetha sowjanya <gakula@...vell.com>; Subbaraya
> Sundeep <sbhatta@...vell.com>; hariprasad <hkelam@...vell.com>; Bharat
> Bhushan <bbhushan2@...vell.com>; Taras Chornyi
> <taras.chornyi@...ision.eu>; Daniele Venzano <venza@...wnhat.org>; Heiner
> Kallweit <hkallweit1@...il.com>; Russell King <linux@...linux.org.uk>; Thomas
> Gleixner <tglx@...utronix.de>; Philipp Stanner <phasta@...nel.org>; Helge Deller
> <deller@....de>; Ingo Molnar <mingo@...nel.org>; Simon Horman
> <horms@...nel.org>; Al Viro <viro@...iv.linux.org.uk>; Sabrina Dubroca
> <sd@...asysnail.net>; Keller, Jacob E <jacob.e.keller@...el.com>
> Cc: linux-arm-kernel@...ts.infradead.org; netdev@...r.kernel.org; linux-
> kernel@...r.kernel.org; linux-parisc@...r.kernel.org
> Subject: [PATCH 0/8] net: Phase out hybrid PCI devres API
> 
> Fixes a number of minor issues with the usage of the PCI API in net.
> Notbaly, it replaces calls to the sometimes-managed
> pci_request_regions() to the always-managed pcim_request_all_regions(),
> enabling us to remove that hybrid functionality from PCI.
> 

Removing the problematic "sometimes" behavior is good.

Reviewed-by: Jacob Keller <jacob.e.keller@...el.com>