diff mbox series

Bug#970871: "modinfo -F" always shows name for built-ins

Message ID X/e48mOg7WfVxRc0@bongo.bofh.it (mailing list archive)
State New
Headers show
Series Bug#970871: "modinfo -F" always shows name for built-ins | expand

Commit Message

Marco d'Itri Jan. 8, 2021, 1:44 a.m. UTC
The attached patch fixes this.

----- Forwarded message from Ben Hutchings <ben@decadent.org.uk> -----

From: Ben Hutchings <ben@decadent.org.uk>
To: Debian Bug Tracking System <submit@bugs.debian.org>
Subject: Bug#970871: "modinfo -F" always shows name for built-ins

Package: kmod
Version: 27+20200310-2
Severity: normal

Now that the kernel provides module information for potentially
modular code that's actually built-in, it's possible to query these
built-ins with "modinfo -F".  However, this doesn't work quite right:

$ modinfo -Flicense e1000e
GPL v2
$ modinfo -Flicense bitrev
name:           bitrev
GPL

Ben.

-- System Information:
Debian Release: bullseye/sid
  APT prefers unstable-debug
  APT policy: (500, 'unstable-debug'), (500, 'stable-updates'), (500, 'unstable'), (500, 'stable'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 5.8.0-1-amd64 (SMP w/2 CPU threads)
Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8), LANGUAGE not set
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

Versions of packages kmod depends on:
ii  libc6      2.31-3
ii  libkmod2   27+20200310-2
ii  liblzma5   5.2.4-1+b1
ii  libssl1.1  1.1.1g-1
ii  lsb-base   11.1.0

kmod recommends no packages.

kmod suggests no packages.

-- no debconf information

----- End forwarded message -----

Comments

Lucas De Marchi Jan. 8, 2021, 3:34 a.m. UTC | #1
On Thu, Jan 7, 2021 at 5:55 PM Marco d'Itri <md@linux.it> wrote:
>
> The attached patch fixes this.

who is the author?

Lucas De Marchi
Marco d'Itri Jan. 8, 2021, 4:13 a.m. UTC | #2
On Jan 08, Lucas De Marchi <lucas.de.marchi@gmail.com> wrote:

> On Thu, Jan 7, 2021 at 5:55 PM Marco d'Itri <md@linux.it> wrote:
> >
> > The attached patch fixes this.
> 
> who is the author?
I am.
Lucas De Marchi Jan. 11, 2021, 1:05 p.m. UTC | #3
On Thu, Jan 7, 2021 at 8:14 PM Marco d'Itri <md@linux.it> wrote:
>
> On Jan 08, Lucas De Marchi <lucas.de.marchi@gmail.com> wrote:
>
> > On Thu, Jan 7, 2021 at 5:55 PM Marco d'Itri <md@linux.it> wrote:
> > >
> > > The attached patch fixes this.
> >
> > who is the author?
> I am.

applied, thanks.

Lucas De Marchi

>
> --
> ciao,
> Marco
diff mbox series

Patch

--- a/tools/modinfo.c
+++ b/tools/modinfo.c
@@ -178,7 +178,11 @@  static int modinfo_do(struct kmod_module
 	is_builtin = (filename == NULL);
 
 	if (is_builtin) {
-		printf("%-16s%s%c", "name:", kmod_module_get_name(mod), separator);
+		if (field == NULL)
+			printf("%-16s%s%c", "name:",
+			       kmod_module_get_name(mod), separator);
+		else if (field != NULL && streq(field, "name"))
+			printf("%s%c", kmod_module_get_name(mod), separator);
 		filename = "(builtin)";
 	}