Re: [PATCH] nvme: add comment to not use kobj_to_dev()
From: Christoph Hellwig <hch@lst.de>
Date: 2021-06-21 16:20:31
From: Christoph Hellwig <hch@lst.de>
Date: 2021-06-21 16:20:31
On Mon, Jun 21, 2021 at 09:17:43AM -0700, Keith Busch wrote:
On Mon, Jun 21, 2021 at 07:32:51AM +0200, Christoph Hellwig wrote:quoted
On Sun, Jun 20, 2021 at 08:09:40PM -0700, Chaitanya Kulkarni wrote:quoted
Add a comment in nvme_ns_id_attrs_are_visible() and nvme_dev_attrs_are_visible() not to replace container_of() with kobj_to_dev(). Since use of kobj_to_dev() makes code harder to read. Suggested-by: Keith Busch <kbusch@kernel.org> Signed-off-by: Chaitanya Kulkarni <redacted>Err, no. This code is completely obvious as is, and if people keep sending these stupid patches we need to start blacklisting them.People will continue to send these patches as long as scripts/coccinelle/api/kobj_to_dev.cocci keeps warning about it.
Maybe we need to fix that instead? _______________________________________________ Linux-nvme mailing list Linux-nvme@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-nvme