On Sat, Feb 12, 2022 at 09:43:35PM -1000, Lucas De Marchi wrote:
> The modules.builtin.alias.bin is way larger than the
> modules.builtin.bin. On a normal "distro kernel":
>
> 21k modules.builtin.alias.bin
> 11k modules.builtin.bin
>
> >From the kernel we get both modules.builtin and modules.builtin.modinfo.
> depmod generates modules.builtin.bin and modules.builtin.alias.bin
> from them respectively. modules.bultin is not going away: it's not
> deprecated by the new index added. So, let's just stop duplicating the
> information inside modules.builtin.alias.bin and just use the other
> index.
The modules.builtin contains only module names. The modules.builtin.modinfo
contains full info about builtin modules including names.
I thought that if there is complete information about the modules, then
reading the index with only the names does not make sense. And only in the
absence of modules.builtin.modinfo, you need to fallback to the index
with the names.
> ---
> libkmod/libkmod-module.c | 12 +++++++-----
> 1 file changed, 7 insertions(+), 5 deletions(-)
>
> diff --git a/libkmod/libkmod-module.c b/libkmod/libkmod-module.c
> index 6f7747c..6423339 100644
> --- a/libkmod/libkmod-module.c
> +++ b/libkmod/libkmod-module.c
> @@ -576,13 +576,15 @@ KMOD_EXPORT int kmod_module_new_from_lookup(struct kmod_ctx *ctx,
> err = kmod_lookup_alias_from_aliases_file(ctx, alias, list);
> CHECK_ERR_AND_FINISH(err, fail, list, finish);
>
> + DBG(ctx, "lookup modules.builtin %s\n", alias);
> + err = kmod_lookup_alias_from_builtin_file(ctx, alias, list);
> + CHECK_ERR_AND_FINISH(err, fail, list, finish);
> +
> DBG(ctx, "lookup modules.builtin.modinfo %s\n", alias);
> err = kmod_lookup_alias_from_kernel_builtin_file(ctx, alias, list);
> - if (err == -ENOSYS) {
> - /* Optional index missing, try the old one */
> - DBG(ctx, "lookup modules.builtin %s\n", alias);
> - err = kmod_lookup_alias_from_builtin_file(ctx, alias, list);
> - }
> + /* Optional index missing, ignore */
> + if (err == -ENOSYS)
> + err = 0;
> CHECK_ERR_AND_FINISH(err, fail, list, finish);
Yep. Looks good for me.
--
Rgrds, legion
