Re: [PATCH] modpost: fix ALL_INIT_DATA_SECTIONS

From: Jan Beulich
Date: Tue Mar 27 2012 - 02:46:05 EST


>>> On 26.03.12 at 22:41, Michal Marek <mmarek@xxxxxxx> wrote:
> On Thu, Mar 08, 2012 at 09:41:25AM +0000, Jan Beulich wrote:
>> This was lacking a comma between two supposed to be separate strings.
>>
>> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
>>
>> ---
>> scripts/mod/modpost.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> --- 3.3-rc6/scripts/mod/modpost.c
>> +++ 3.3-rc6-modpost-all-init-data/scripts/mod/modpost.c
>> @@ -849,7 +849,7 @@ static void check_section(const char *mo
>>
>> #define ALL_INIT_DATA_SECTIONS \
>> ".init.setup$", ".init.rodata$", \
>> - ".devinit.rodata$", ".cpuinit.rodata$", ".meminit.rodata$" \
>> + ".devinit.rodata$", ".cpuinit.rodata$", ".meminit.rodata$", \
>
> Applied to kbuild.git#kbuild with a Cc: stable. BTW, was it causing actual
> false positives to be reported?

No, I just noticed this in the context of a bigger (PCI related) patch
that so far I didn't get any reaction on from you (intending to resend
at some point).

Jan

--
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/