[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20191205032503.GA3981@debian>
Date: Thu, 5 Dec 2019 08:55:07 +0530
From: Bhaskar Chowdhury <unixbhaskar@...il.com>
To: dsterba@...e.cz, Masahiro Yamada <masahiroy@...nel.org>,
Michal Marek <michal.lkml@...kovi.net>,
Randy Dunlap <rdunlap@...radead.org>,
Linux Kbuild mailing list <linux-kbuild@...r.kernel.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/2] Enlist running kernel modules information
On 16:07 Wed 04 Dec 2019, David Sterba wrote:
>On Wed, Dec 04, 2019 at 12:10:25PM +0900, Masahiro Yamada wrote:
>> On Tue, Dec 3, 2019 at 6:49 PM Bhaskar Chowdhury <unixbhaskar@...il.com> wrote:
>> > +awk '{print $1}' "/proc/modules" | xargs modinfo | awk '/^(filename|desc|depends)/'
>>
>> I want to see a good reason (e.g. useful for other developers) for upstreaming.
>> This script looks like your custom script, which you can maintain locally.
>
>I think the verbosity should be added to either lsmod or modinfo, not
>some script in kernel git.
lsmod and modinfo already are pretty verbose and the one liner is using
one of them to cut thing out of it....can you give it another look???
Moreover,this is sort and precise and can be parsed by other scripts.
The whole point behind this is to give the developers a convenient point
without going through all the rigorous details.
Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)
Powered by blists - more mailing lists