mbox series

[0/8] Random fixes + modinfo --modname

Message ID 20220216075533.185693-1-lucas.demarchi@intel.com (mailing list archive)
Headers show
Series Random fixes + modinfo --modname | expand

Message

Lucas De Marchi Feb. 16, 2022, 7:55 a.m. UTC
First commits are some random fixes and refactors while preparing the
code base for a new API: kmod_module_new_from_name_lookup(). This new
function is used by modinfo to allow handling the argument as a module
name instead of alias or file. Main motivation is to allow to bypass
aliases set by kernel or user configuration that would otherwise forbid
to get the info from a particular module. Although for modules it's
possible to just find the correspondent file, this is not possible
for builtin modules.

Lucas De Marchi (8):
  test-initstate: Check for negative value on error
  libkmod-module: Fix return code for kmod_module_new_from_lookup()
  gitignore: ignore gtk-doc.make
  libkmod: Add helper function to iterate lookup options
  libkmod: Update docs about indexes order
  libkmod: Add lookup from module name
  modinfo: Update help message with "modulename"
  modinfo: Allow to force arg as module name

 .gitignore                 |   1 +
 libkmod/libkmod-module.c   | 131 +++++++++++++++++++++++++------------
 libkmod/libkmod.h          |   3 +
 libkmod/libkmod.sym        |   1 +
 testsuite/test-initstate.c |   2 +-
 tools/modinfo.c            |  32 ++++++++-
 6 files changed, 125 insertions(+), 45 deletions(-)

Comments

Luis Chamberlain Feb. 17, 2022, 8:11 a.m. UTC | #1
On Tue, Feb 15, 2022 at 11:55:25PM -0800, Lucas De Marchi wrote:
> First commits are some random fixes and refactors while preparing the
> code base for a new API: kmod_module_new_from_name_lookup(). This new
> function is used by modinfo to allow handling the argument as a module
> name instead of alias or file. Main motivation is to allow to bypass
> aliases set by kernel or user configuration that would otherwise forbid
> to get the info from a particular module. Although for modules it's
> possible to just find the correspondent file, this is not possible
> for builtin modules.

For 1-7:

Reviewed-by: Luis Chamberlain <mcgrof@kernel.org>

  Luis