Re: [RFC PATCH] kernel/kallsyms.c: only show legal kernel symbol

From: Russell King - ARM Linux
Date: Wed Oct 30 2013 - 19:09:52 EST


On Mon, Oct 28, 2013 at 04:20:19PM +1030, Rusty Russell wrote:
> Ming Lei <tom.leiming@xxxxxxxxx> writes:
> > On Mon, 28 Oct 2013 13:44:30 +1030
> > Rusty Russell <rusty@xxxxxxxxxxxxxxx> wrote:
> >
> >> Ming Lei <tom.leiming@xxxxxxxxx> writes:
> >>
> >> I don't know... It would be your job, as the person making the change,
> >> to find all the users of kallsyms and prove that.
> >>
> >> This is why it is easier not to include incorrect values in the kernel's
> >> kallsyms in the first place.
> >
> > OK, thanks for your comment, and I figured out one way to do it in
> > scripts/kallsyms.c, could you comment on below patch?
>
> Looks great! Seems like we spent more time arguing than it took you to
> code that up.
>
> Acked-by: Rusty Russell <rusty@xxxxxxxxxxxxxxx>
>
> Russell, this seems logical for you to take along with the changes which
> caused the problem?

The changes are already in mainline since a long time (back in July/August
time). Am I the right person to take stuff for scripts/ ? Isn't that
more kbuild territory?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/