modpost: merge bad_tosec=ALL_EXIT_SECTIONS entries in sectioncheck table
authorMasahiro Yamada <masahiroy@kernel.org>
Sun, 21 May 2023 16:04:22 +0000 (01:04 +0900)
committerMasahiro Yamada <masahiroy@kernel.org>
Sun, 28 May 2023 11:39:52 +0000 (20:39 +0900)
There is no distinction between TEXT_TO_ANY_EXIT and DATA_TO_ANY_EXIT.
Just merge them.

Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
Reviewed-by: Nick Desaulniers <ndesaulniers@google.com>
scripts/mod/modpost.c

index 21417a4..cc4cf40 100644 (file)
@@ -859,8 +859,7 @@ static const char *const optim_symbols[] = { "*.constprop.*", NULL };
 enum mismatch {
        TEXT_TO_ANY_INIT,
        DATA_TO_ANY_INIT,
-       TEXT_TO_ANY_EXIT,
-       DATA_TO_ANY_EXIT,
+       TEXTDATA_TO_ANY_EXIT,
        XXXINIT_TO_SOME_INIT,
        XXXEXIT_TO_SOME_EXIT,
        ANY_INIT_TO_ANY_EXIT,
@@ -904,14 +903,9 @@ static const struct sectioncheck sectioncheck[] = {
        .mismatch = DATA_TO_ANY_INIT,
 },
 {
-       .fromsec = { TEXT_SECTIONS, NULL },
-       .bad_tosec = { ALL_EXIT_SECTIONS, NULL },
-       .mismatch = TEXT_TO_ANY_EXIT,
-},
-{
-       .fromsec = { DATA_SECTIONS, NULL },
+       .fromsec = { TEXT_SECTIONS, DATA_SECTIONS, NULL },
        .bad_tosec = { ALL_EXIT_SECTIONS, NULL },
-       .mismatch = DATA_TO_ANY_EXIT,
+       .mismatch = TEXTDATA_TO_ANY_EXIT,
 },
 /* Do not reference init code/data from meminit code/data */
 {
@@ -1203,8 +1197,7 @@ static void default_mismatch_handler(const char *modname, struct elf_info *elf,
        switch (mismatch->mismatch) {
        case TEXT_TO_ANY_INIT:
        case DATA_TO_ANY_INIT:
-       case TEXT_TO_ANY_EXIT:
-       case DATA_TO_ANY_EXIT:
+       case TEXTDATA_TO_ANY_EXIT:
        case XXXINIT_TO_SOME_INIT:
        case XXXEXIT_TO_SOME_EXIT:
        case ANY_INIT_TO_ANY_EXIT: