Re: [PATCH] kallsyms: reduce size a little on 64-bit

From: Masahiro Yamada <yamada.masahiro@socionext.com>
To: Jan Beulich <JBeulich@suse.com>
Cc: Linux Kbuild mailing list <linux-kbuild@vger.kernel.org>,
	Michal Marek <michal.lkml@markovi.net>
Subject: Re: [PATCH] kallsyms: reduce size a little on 64-bit
Date: Mon, 3 Sep 2018 02:12:25 +0900	[thread overview]
Message-ID: <CAK7LNAQZw88d08wEn=S7mVgEz86djuig6MYMgT_hWwBU5jzrCA@mail.gmail.com> (raw)
In-Reply-To: <5B8563A702000078001E2A82@prv1-mh.provo.novell.com>

Hi.

2018-08-29 0:00 GMT+09:00 Jan Beulich <JBeulich@suse.com>:
> Both kallsyms_num_syms and kallsyms_markers[] don't really need to use
> unsigned long as their (base) types; unsigned int fully suffices.
>
> Signed-off-by: Jan Beulich <jbeulich@suse.com>
> ---

> --- 4.19-rc1/scripts/kallsyms.c
> +++ 4.19-rc1-kallsyms-reduce-size/scripts/kallsyms.c
> @@ -182,6 +182,8 @@ static int symbol_in_range(struct sym_en
>         for (i = 0; i < entries; ++i) {
>                 ar = &ranges[i];
>
> +               if (!ar->end)
> +                       continue;
>                 if (s->addr >= ar->start && s->addr <= ar->end)
>                         return 1;
>         }

What is this hunk doing?

At least, I cannot understand it from the patch description.

-- 
Best Regards
Masahiro Yamada