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
