diff options
| author | Masahiro Yamada <masahiroy@kernel.org> | 2024-07-20 12:30:12 +0200 |
|---|---|---|
| committer | Masahiro Yamada <masahiroy@kernel.org> | 2024-07-21 18:24:22 +0200 |
| commit | bde6fb37bbc3a41eb63b1e4d06e2ff217d4a2f18 (patch) | |
| tree | e988ae57ab57cd191d8ae040408bf4635bd66cca /scripts/kallsyms.c | |
| parent | kbuild: add script and target to generate pacman package (diff) | |
| download | linux-bde6fb37bbc3a41eb63b1e4d06e2ff217d4a2f18.tar.xz linux-bde6fb37bbc3a41eb63b1e4d06e2ff217d4a2f18.zip | |
kallsyms: avoid repeated calculation of array size for markers
Introduce the markers_cnt variable for readability.
No functional change intended.
Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
Diffstat (limited to 'scripts/kallsyms.c')
| -rw-r--r-- | scripts/kallsyms.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/scripts/kallsyms.c b/scripts/kallsyms.c index 55a423519f2e..f0ea8c922dc8 100644 --- a/scripts/kallsyms.c +++ b/scripts/kallsyms.c @@ -393,7 +393,7 @@ static void write_src(void) { unsigned int i, k, off; unsigned int best_idx[256]; - unsigned int *markers; + unsigned int *markers, markers_cnt; char buf[KSYM_NAME_LEN]; printf("#include <asm/bitsperlong.h>\n"); @@ -413,7 +413,8 @@ static void write_src(void) /* table of offset markers, that give the offset in the compressed stream * every 256 symbols */ - markers = malloc(sizeof(unsigned int) * ((table_cnt + 255) / 256)); + markers_cnt = (table_cnt + 255) / 256; + markers = malloc(sizeof(*markers) * markers_cnt); if (!markers) { fprintf(stderr, "kallsyms failure: " "unable to allocate required memory\n"); @@ -469,7 +470,7 @@ static void write_src(void) } output_label("kallsyms_markers"); - for (i = 0; i < ((table_cnt + 255) >> 8); i++) + for (i = 0; i < markers_cnt; i++) printf("\t.long\t%u\n", markers[i]); printf("\n"); |