diff mbox series

[v2] modprobe: add --show-exports

Message ID 20181112154918.3137-1-yauheni.kaliuta@redhat.com (mailing list archive)
State New, archived
Headers show
Series [v2] modprobe: add --show-exports | expand

Commit Message

Yauheni Kaliuta Nov. 12, 2018, 3:49 p.m. UTC
modprobe has --show-modversions switch, which dumps symbols with
their modversion crcs from the __versions sections.

At the moment the section contains information for the dependency
symbols only, while exported symbols add to symtab entries with
__crc_ prefix (the format may differ, see 1e48901166ef libkmod-elf:
resolve CRC if module is built with MODULE_REL_CRCS).

The patch makes it to show exported symbols as well.

The function is basically cut'n'paste of show_modversions(),
but 'version' family replaced with 'symbol' one.

Signed-off-by: Yauheni Kaliuta <yauheni.kaliuta@redhat.com>
---
 tools/modprobe.c | 38 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 38 insertions(+)

Comments

Lucas De Marchi Nov. 12, 2018, 11:15 p.m. UTC | #1
On Mon, Nov 12, 2018 at 7:48 AM Yauheni Kaliuta
<yauheni.kaliuta@redhat.com> wrote:
>
> modprobe has --show-modversions switch, which dumps symbols with
> their modversion crcs from the __versions sections.
>
> At the moment the section contains information for the dependency
> symbols only, while exported symbols add to symtab entries with
> __crc_ prefix (the format may differ, see 1e48901166ef libkmod-elf:
> resolve CRC if module is built with MODULE_REL_CRCS).
>
> The patch makes it to show exported symbols as well.
>
> The function is basically cut'n'paste of show_modversions(),
> but 'version' family replaced with 'symbol' one.
>
> Signed-off-by: Yauheni Kaliuta <yauheni.kaliuta@redhat.com>
> ---
>  tools/modprobe.c | 38 ++++++++++++++++++++++++++++++++++++++
>  1 file changed, 38 insertions(+)
>
> diff --git a/tools/modprobe.c b/tools/modprobe.c
> index 43605ccaf0f0..ca161d09940c 100644
> --- a/tools/modprobe.c
> +++ b/tools/modprobe.c
> @@ -76,6 +76,8 @@ static const struct option cmdopts[] = {
>         {"show-config", no_argument, 0, 'c'},
>         {"show-modversions", no_argument, 0, 4},
>         {"dump-modversions", no_argument, 0, 4},
> +       {"show-exports", no_argument, 0, 6},
> +       {"dump-exports", no_argument, 0, 6},
>
>         {"dry-run", no_argument, 0, 'n'},
>         {"show", no_argument, 0, 'n'},
> @@ -124,6 +126,8 @@ static void help(void)
>                 "\t-c, --show-config           Same as --showconfig\n"
>                 "\t    --show-modversions      Dump module symbol version and exit\n"
>                 "\t    --dump-modversions      Same as --show-modversions\n"
> +               "\t    --show-exports          Dump module exported symbol versions and exit\n"
> +               "\t    --dump-exports          Same as --show-exports\n"

all the others are show-*. The dump-* and show* should be considered
aliases for compatibility. So I don't think we need that on a new one.

Lucas De Marchi

>                 "\n"
>                 "General Options:\n"
>                 "\t-n, --dry-run               Do not execute operations, just print out\n"
> @@ -232,6 +236,34 @@ static int show_modversions(struct kmod_ctx *ctx, const char *filename)
>         return 0;
>  }
>
> +static int show_exports(struct kmod_ctx *ctx, const char *filename)
> +{
> +       struct kmod_list *l, *list = NULL;
> +       struct kmod_module *mod;
> +       int err = kmod_module_new_from_path(ctx, filename, &mod);
> +       if (err < 0) {
> +               LOG("Module %s not found.\n", filename);
> +               return err;
> +       }
> +
> +       err = kmod_module_get_symbols(mod, &list);
> +       if (err < 0) {
> +               LOG("could not get symbols of %s: %s\n",
> +                       filename, strerror(-err));
> +               kmod_module_unref(mod);
> +               return err;
> +       }
> +
> +       kmod_list_foreach(l, list) {
> +               const char *symbol = kmod_module_symbol_get_symbol(l);
> +               uint64_t crc = kmod_module_symbol_get_crc(l);
> +               printf("0x%08"PRIx64"\t%s\n", crc, symbol);
> +       }
> +       kmod_module_symbols_free_list(list);
> +       kmod_module_unref(mod);
> +       return 0;
> +}
> +
>  static int command_do(struct kmod_module *module, const char *type,
>                                 const char *command, const char *cmdline_opts)
>  {
> @@ -727,6 +759,7 @@ static int do_modprobe(int argc, char **orig_argv)
>         int do_remove = 0;
>         int do_show_config = 0;
>         int do_show_modversions = 0;
> +       int do_show_exports = 0;
>         int err;
>
>         argv = prepend_options_from_env(&argc, orig_argv);
> @@ -783,6 +816,9 @@ static int do_modprobe(int argc, char **orig_argv)
>                 case 4:
>                         do_show_modversions = 1;
>                         break;
> +               case 6:
> +                       do_show_exports = 1;
> +                       break;
>                 case 'n':
>                         dry_run = 1;
>                         break;
> @@ -886,6 +922,8 @@ static int do_modprobe(int argc, char **orig_argv)
>                 err = show_config(ctx);
>         else if (do_show_modversions)
>                 err = show_modversions(ctx, args[0]);
> +       else if (do_show_exports)
> +               err = show_exports(ctx, args[0]);
>         else if (do_remove)
>                 err = rmmod_all(ctx, args, nargs);
>         else if (use_all)
> --
> 2.19.1
>
diff mbox series

Patch

diff --git a/tools/modprobe.c b/tools/modprobe.c
index 43605ccaf0f0..ca161d09940c 100644
--- a/tools/modprobe.c
+++ b/tools/modprobe.c
@@ -76,6 +76,8 @@  static const struct option cmdopts[] = {
 	{"show-config", no_argument, 0, 'c'},
 	{"show-modversions", no_argument, 0, 4},
 	{"dump-modversions", no_argument, 0, 4},
+	{"show-exports", no_argument, 0, 6},
+	{"dump-exports", no_argument, 0, 6},
 
 	{"dry-run", no_argument, 0, 'n'},
 	{"show", no_argument, 0, 'n'},
@@ -124,6 +126,8 @@  static void help(void)
 		"\t-c, --show-config           Same as --showconfig\n"
 		"\t    --show-modversions      Dump module symbol version and exit\n"
 		"\t    --dump-modversions      Same as --show-modversions\n"
+		"\t    --show-exports          Dump module exported symbol versions and exit\n"
+		"\t    --dump-exports          Same as --show-exports\n"
 		"\n"
 		"General Options:\n"
 		"\t-n, --dry-run               Do not execute operations, just print out\n"
@@ -232,6 +236,34 @@  static int show_modversions(struct kmod_ctx *ctx, const char *filename)
 	return 0;
 }
 
+static int show_exports(struct kmod_ctx *ctx, const char *filename)
+{
+	struct kmod_list *l, *list = NULL;
+	struct kmod_module *mod;
+	int err = kmod_module_new_from_path(ctx, filename, &mod);
+	if (err < 0) {
+		LOG("Module %s not found.\n", filename);
+		return err;
+	}
+
+	err = kmod_module_get_symbols(mod, &list);
+	if (err < 0) {
+		LOG("could not get symbols of %s: %s\n",
+			filename, strerror(-err));
+		kmod_module_unref(mod);
+		return err;
+	}
+
+	kmod_list_foreach(l, list) {
+		const char *symbol = kmod_module_symbol_get_symbol(l);
+		uint64_t crc = kmod_module_symbol_get_crc(l);
+		printf("0x%08"PRIx64"\t%s\n", crc, symbol);
+	}
+	kmod_module_symbols_free_list(list);
+	kmod_module_unref(mod);
+	return 0;
+}
+
 static int command_do(struct kmod_module *module, const char *type,
 				const char *command, const char *cmdline_opts)
 {
@@ -727,6 +759,7 @@  static int do_modprobe(int argc, char **orig_argv)
 	int do_remove = 0;
 	int do_show_config = 0;
 	int do_show_modversions = 0;
+	int do_show_exports = 0;
 	int err;
 
 	argv = prepend_options_from_env(&argc, orig_argv);
@@ -783,6 +816,9 @@  static int do_modprobe(int argc, char **orig_argv)
 		case 4:
 			do_show_modversions = 1;
 			break;
+		case 6:
+			do_show_exports = 1;
+			break;
 		case 'n':
 			dry_run = 1;
 			break;
@@ -886,6 +922,8 @@  static int do_modprobe(int argc, char **orig_argv)
 		err = show_config(ctx);
 	else if (do_show_modversions)
 		err = show_modversions(ctx, args[0]);
+	else if (do_show_exports)
+		err = show_exports(ctx, args[0]);
 	else if (do_remove)
 		err = rmmod_all(ctx, args, nargs);
 	else if (use_all)