Date: Mon, 28 Nov 2016 14:50:55 -0500 From: Jessica Yu <jeyu@...hat.com> To: AKASHI Takahiro <takahiro.akashi@...aro.org>, Andreas Ziegler <andreas.ziegler@....de>, Rusty Russell <rusty@...tcorp.com.au>, Kees Cook <keescook@...omium.org>, kernel-hardening@...ts.openwall.com, linux-kernel <linux-kernel@...r.kernel.org> Subject: Re: module: extend 'rodata=off' boot cmdline parameter to module mappings +++ AKASHI Takahiro [28/11/16 18:18 +0900]: >On Mon, Nov 28, 2016 at 08:59:40AM +0100, Andreas Ziegler wrote: >> Hi Akashi, >> >> your patch "module: extend 'rodata=off' boot cmdline parameter to module >> mappings" showed up in linux-next today, and I noticed a small error in it. >> >> The first modified #ifdef is fine, the second one, however, has a spelling >> mistake in it: the CONFIG_ variable should be CONFIG_DEBUG_SET_MODULE_RONX >> instead of CONFIG_SET_MODULE_RONX (note the missing DEBUG). > >Thank you for pointing this out. >(I didn't notice it because DEBUG_RODATA is always on on arm64.) > >Rusty, Jessica, should I submit v4 to correct it? Andreas, Akashi - The typo has been fixed with Arnd's patch: http://email@example.com Thanks for finding the issue! Jessica
Powered by blists - more mailing lists
Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.