[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20190814080307.GA2911@google.com>
Date: Wed, 14 Aug 2019 09:03:07 +0100
From: Matthias Maennich <maennich@...gle.com>
To: Himanshu Jha <himanshujha199640@...il.com>
Cc: linux-kernel@...r.kernel.org, maco@...roid.com,
kstewart@...uxfoundation.org, linux-m68k@...r.kernel.org,
oneukum@...e.com,
Mauro Carvalho Chehab <mchehab+samsung@...nel.org>,
usb-storage@...ts.one-eyed-alien.net, hpa@...or.com,
joel@...lfernandes.org, sam@...nborg.org, cocci@...teme.lip6.fr,
linux-arch@...r.kernel.org, linux-scsi@...r.kernel.org,
x86@...nel.org, lucas.de.marchi@...il.com, mingo@...hat.com,
geert@...ux-m68k.org, stern@...land.harvard.edu,
kernel-team@...roid.com, sspatil@...gle.com, arnd@...db.de,
linux-kbuild@...r.kernel.org,
Nicolas Palix <nicolas.palix@...g.fr>, jeyu@...nel.org,
Jonathan Cameron <Jonathan.Cameron@...wei.com>,
tglx@...utronix.de, michal.lkml@...kovi.net,
gregkh@...uxfoundation.org, linux-usb@...r.kernel.org,
Nicolas Ferre <nicolas.ferre@...rochip.com>, maco@...gle.com,
pombredanne@...b.com, "David S. Miller" <davem@...emloft.net>,
linux-modules@...r.kernel.org
Subject: Re: [Cocci] [PATCH v2 08/10] scripts: Coccinelle script for
namespace dependencies.
On Wed, Aug 14, 2019 at 12:06:11PM +0530, Himanshu Jha wrote:
>On Tue, Aug 13, 2019 at 01:17:05PM +0100, Matthias Maennich wrote:
>> A script that uses the '<module>.ns_deps' file generated by modpost to
>> automatically add the required symbol namespace dependencies to each
>> module.
>>
>> Usage:
>> 1) Move some symbols to a namespace with EXPORT_SYMBOL_NS() or define
>> DEFAULT_SYMBOL_NAMESPACE
>> 2) Run 'make' (or 'make modules') and get warnings about modules not
>> importing that namespace.
>> 3) Run 'make nsdeps' to automatically add required import statements
>> to said modules.
>>
>> This makes it easer for subsystem maintainers to introduce and maintain
>> symbol namespaces into their codebase.
>>
>> Co-developed-by: Martijn Coenen <maco@...roid.com>
>> Signed-off-by: Martijn Coenen <maco@...roid.com>
>> Signed-off-by: Matthias Maennich <maennich@...gle.com>
>> ---
>
>[]
>
>> MAINTAINERS | 5 ++
>> Makefile | 12 +++++
>> scripts/Makefile.modpost | 4 +-
>> scripts/coccinelle/misc/add_namespace.cocci | 23 +++++++++
>> scripts/nsdeps | 54 +++++++++++++++++++++
>> 5 files changed, 97 insertions(+), 1 deletion(-)
>> create mode 100644 scripts/coccinelle/misc/add_namespace.cocci
>> create mode 100644 scripts/nsdeps
>
>[]
>
>> +if [ "$SPATCH_VERSION_NUM" -lt "$SPATCH_REQ_VERSION_NUM" ] ; then
>> + echo 'spatch needs to be version 1.06 or higher'
>
>Nitpick: 1.0.6
>
Good catch! Thanks!
Actually it needs to be even 1.0.4, so I will just use
$SPATCH_REQ_VERSION from above.
>> + exit 1
>> +fi
>
>
>--
>Himanshu Jha
>Undergraduate Student
>Department of Electronics & Communication
>Guru Tegh Bahadur Institute of Technology
Powered by blists - more mailing lists