Re: [PATCH v3] ntb: Remove error checking for debugfs_create_dir()

From: Dan Carpenter
Date: Fri Jul 14 2023 - 10:08:35 EST


On Fri, Jul 14, 2023 at 02:44:11PM +0200, Markus Elfring wrote:
> > > It is expected that most callers should _ignore_ the errors
> > > return by debugfs_create_dir() in tool_setup_dbgfs()
>
> > The patch itself is correct for sure:
> > Reviewed-by: Serge Semin <fancer.lancer@xxxxxxxxx>
>
> How does such information fit to the Linux development requirement
> for imperative change descriptions?
>

Markus, you already replied to this thread.

Greg, HCH, Matthew Wilcox and Krzysztof Kozlowski have all asked you to
stop. Those are respected kernel maintainers. You should listen to
them.

If you see a bug, that's useful. But if you're just nitpicking the
commit message, that's not useful. We have explained this many times as
clearly as we know how.

> See also:
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.5-rc1#n94
>
>
> How do you think about to add the tag “Fixes” because of the deletion
> of an inappropriate error detection?

No. It's not a bug fix so a Fixes tag is innappropriate.

regards,
dan carpenter