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

From: Serge Semin
Date: Fri Jul 14 2023 - 10:03:37 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?
>
> 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

Well, the patch author does follow that rule in the subject. Regarding the
patch log body it is impossible to use the "imperative mood" always in the
entire text. In this case the explanation is pretty much clear and doesn't
use the patterns like "This patch", or any personal pronouns so IMO it's
fine to have the log as is.

>
>
> How do you think about to add the tag “Fixes” because of the deletion
> of an inappropriate error detection?

It doesn't fix any real bug. So there is no need in the tag.

-Serge(y)

>
> Regards,
> Markus