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
| ||
|
Message-ID: <20231004163929.70977d05@kernel.org> Date: Wed, 4 Oct 2023 16:39:29 -0700 From: Jakub Kicinski <kuba@...nel.org> To: Moshe Shemesh <moshe@...dia.com> Cc: "David S. Miller" <davem@...emloft.net>, <netdev@...r.kernel.org>, Jiri Pirko <jiri@...dia.com>, <linux-kernel@...r.kernel.org> Subject: Re: [PATCH net] devlink: Hold devlink lock on health reporter dump get On Sun, 1 Oct 2023 18:19:40 +0300 Moshe Shemesh wrote: > Devlink health dump get callback should take devlink lock as any other > devlink callback. Add devlink lock to the callback and to any call for > devlink_health_do_dump(). > > As devlink lock is added to any callback of dump, the reporter dump_lock > is now redundant and can be removed. I love the change but it needs more info in the commit message :) 1. what exact / example but real problem are you solving? 2. some words of reassurance that you checked all the drivers and the locking change should be safe (none of them take instance locks in reporter callbacks etc) -- pw-bot: cr
Powered by blists - more mailing lists