[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAHk-=wgT6wuBB=pK2bGNQbioNCdJHT8Jnvs+4oT25jX1JFyhgg@mail.gmail.com>
Date: Fri, 28 Jan 2022 10:00:29 +0200
From: Linus Torvalds <torvalds@...ux-foundation.org>
To: kernel test robot <oliver.sang@...el.com>
Cc: Luis Chamberlain <mcgrof@...nel.org>, Jan Kara <jack@...e.cz>,
Al Viro <viro@...iv.linux.org.uk>,
Amir Goldstein <amir73il@...il.com>,
Andy Shevchenko <andriy.shevchenko@...ux.intel.com>,
Antti Palosaari <crope@....fi>, Arnd Bergmann <arnd@...db.de>,
Benjamin Herrenschmidt <benh@...nel.crashing.org>,
Benjamin LaHaise <bcrl@...ck.org>,
Clemens Ladisch <clemens@...isch.de>,
David Airlie <airlied@...ux.ie>,
"Eric W. Biederman" <ebiederm@...ssion.com>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Iurii Zaikin <yzaikin@...gle.com>,
Jani Nikula <jani.nikula@...ux.intel.com>,
Joel Becker <jlbec@...lplan.org>,
Joonas Lahtinen <joonas.lahtinen@...ux.intel.com>,
Joseph Qi <joseph.qi@...ux.alibaba.com>,
Julia Lawall <julia.lawall@...ia.fr>,
Kees Cook <keescook@...omium.org>,
Lukas Middendorf <kernel@...force.de>,
Mark Fasheh <mark@...heh.com>, Paul Turner <pjt@...gle.com>,
Peter Zijlstra <peterz@...radead.org>,
Petr Mladek <pmladek@...e.com>,
Phillip Potter <phil@...lpotter.co.uk>,
Qing Wang <wangqing@...o.com>,
Rodrigo Vivi <rodrigo.vivi@...el.com>,
Sebastian Reichel <sre@...nel.org>,
Sergey Senozhatsky <senozhatsky@...omium.org>,
Stephen Kitt <steve@....org>,
Tetsuo Handa <penguin-kernel@...ove.sakura.ne.jp>,
Xiaoming Ni <nixiaoming@...wei.com>,
Douglas Gilbert <dgilbert@...erlog.com>,
"James E.J. Bottomley" <jejb@...ux.ibm.com>,
Jani Nikula <jani.nikula@...el.com>,
John Ogness <john.ogness@...utronix.de>,
"Martin K. Petersen" <martin.petersen@...cle.com>,
"Rafael J. Wysocki" <rafael@...nel.org>,
Steven Rostedt <rostedt@...dmis.org>,
Suren Baghdasaryan <surenb@...gle.com>,
"Theodore Ts'o" <tytso@....edu>,
Andrew Morton <akpm@...ux-foundation.org>,
LKML <linux-kernel@...r.kernel.org>, lkp@...ts.01.org,
kernel test robot <lkp@...el.com>
Subject: Re: [ocfs2] c42ff46f97: sysctl_table_check_failed
On Fri, Jan 28, 2022 at 8:53 AM kernel test robot <oliver.sang@...el.com> wrote:
>
> commit: 46e33fd45a52bf03769906e64d8a8a1ab317777d ("ocfs2: simplify subdirectory
> registration with register_sysctl()")
Well, it's apparently commit c42ff46f97c1 ("ocfs2: simplify
subdirectory registration with register_sysctl()") in mainline now.
What worries me a bit is that the commit was auto-generated, and when
reading the commit message it reads as if it wasn't supposed to cause
any semantic changes at all.
Is the cause of this that 'nm' is supposed to be a directory, and
register_sysctl() doesn't handle directories?
I don't know this code at all, should it have been simplified even
further with something (TOTALLY UNTESTED) like the attached?
Linus
View attachment "patch.diff" of type "text/x-patch" (923 bytes)
Powered by blists - more mailing lists