Re: [PATCH 1/2] Enlist running kernel modules information

[Bhaskar Chowdhury]

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@xxxxxxxxx> 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.

Attachment: signature.asc
Description: PGP signature