]> www.infradead.org Git - users/dwmw2/linux.git/commitdiff
modpost: rename variables in handle_moddevtable()
authorMasahiro Yamada <masahiroy@kernel.org>
Tue, 19 Nov 2024 23:56:51 +0000 (08:56 +0900)
committerMasahiro Yamada <masahiroy@kernel.org>
Wed, 27 Nov 2024 23:46:02 +0000 (08:46 +0900)
This commit renames the variables in handle_moddevtable() as follows:

    name       -> type
    namelen    -> typelen
    identifier -> name

These changes align with the definition in include/linux/module.h:

  extern typeof(name) __mod_##type##__##name##_device_table

Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
scripts/mod/file2alias.c

index 541e6a3f95bc287778af686edd49dfe65cf1842b..038b2ab79e1175e1ac872b1cc43618d67d9290cc 100644 (file)
@@ -1503,8 +1503,8 @@ void handle_moddevtable(struct module *mod, struct elf_info *info,
 {
        void *symval;
        char *zeros = NULL;
-       const char *name, *identifier;
-       unsigned int namelen;
+       const char *type, *name;
+       size_t typelen;
 
        /* We're looking for a section relative symbol */
        if (!sym->st_shndx || get_secindex(info, sym) >= info->num_sections)
@@ -1514,19 +1514,19 @@ void handle_moddevtable(struct module *mod, struct elf_info *info,
        if (ELF_ST_TYPE(sym->st_info) != STT_OBJECT)
                return;
 
-       /* All our symbols are of form __mod_<name>__<identifier>_device_table. */
+       /* All our symbols are of form __mod_<type>__<name>_device_table. */
        if (!strstarts(symname, "__mod_"))
                return;
-       name = symname + strlen("__mod_");
-       namelen = strlen(name);
-       if (namelen < strlen("_device_table"))
+       type = symname + strlen("__mod_");
+       typelen = strlen(type);
+       if (typelen < strlen("_device_table"))
                return;
-       if (strcmp(name + namelen - strlen("_device_table"), "_device_table"))
+       if (strcmp(type + typelen - strlen("_device_table"), "_device_table"))
                return;
-       identifier = strstr(name, "__");
-       if (!identifier)
+       name = strstr(type, "__");
+       if (!name)
                return;
-       namelen = identifier - name;
+       typelen = name - type;
 
        /* Handle all-NULL symbols allocated into .bss */
        if (info->sechdrs[get_secindex(info, sym)].sh_type & SHT_NOBITS) {
@@ -1539,7 +1539,7 @@ void handle_moddevtable(struct module *mod, struct elf_info *info,
        for (int i = 0; i < ARRAY_SIZE(devtable); i++) {
                const struct devtable *p = &devtable[i];
 
-               if (sym_is(name, namelen, p->device_id)) {
+               if (sym_is(type, typelen, p->device_id)) {
                        do_table(symval, sym->st_size, p->id_size,
                                 p->device_id, p->do_entry, mod);
                        break;