linux-kernel - Re: [PATCH] microblaze: Don't mark arch_kgdb

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Sun, 23 Jun 2013 14:16:51 -0700
From:	Graeme Smecher <gsmecher@...eespeedlogic.com>
To:	linux-kernel@...r.kernel.org
CC:	"microblaze-uclinux@...e.uq.edu.au" 
	<microblaze-uclinux@...e.uq.edu.au>, monstr@...str.eu
Subject: Re: [PATCH] microblaze: Don't mark arch_kgdb_ops as const.

Hi Michal,

On 08/06/13 01:58 PM, Graeme Smecher wrote:
> Hi all,
>
> On 08/06/13 09:52 AM, Graeme Smecher wrote:
>> Other architectures don't do it, and it conflicts with the extern'd 
>> definition
>> in include/linux/kgdb.h.
>>
>> Signed-off-by: Graeme Smecher<gsmecher@...eespeedlogic.com>
>> CC: Michal Simek<monstr@...str.eu>
>> CC:linux-kernel@...r.kernel.org
>> ---
>>   arch/microblaze/kernel/kgdb.c |    2 +-
>>   1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/arch/microblaze/kernel/kgdb.c 
>> b/arch/microblaze/kernel/kgdb.c
>> index 8adc9244..09a5e82 100644
>> --- a/arch/microblaze/kernel/kgdb.c
>> +++ b/arch/microblaze/kernel/kgdb.c
>> @@ -141,7 +141,7 @@ void kgdb_arch_exit(void)
>>   /*
>>    * Global data
>>    */
>> -const struct kgdb_arch arch_kgdb_ops = {
>> +struct kgdb_arch arch_kgdb_ops = {
>>   #ifdef __MICROBLAZEEL__
>>       .gdb_bpt_instr = {0x18, 0x00, 0x0c, 0xba}, /* brki r16, 0x18 */
>>   #else
>
> It's worth noting that this patch is flagged by checkpatch:
>
>    WARNING: struct kgdb_arch should normally be const
>    #25: FILE: arch/microblaze/kernel/kgdb.c:144:
>    +struct kgdb_arch arch_kgdb_ops = {
>
>    total: 0 errors, 1 warnings, 8 lines checked
>
>
> However, without it, my kernel fails to compile:
>
>       CC      arch/microblaze/kernel/kgdb.o
>    arch/microblaze/kernel/kgdb.c:144:24: error: conflicting type
>    qualifiers for 'arch_kgdb_ops'
>    include/linux/kgdb.h:284:26: note: previous declaration of
>    'arch_kgdb_ops' was here
>    make[1]: *** [arch/microblaze/kernel/kgdb.o] Error 1
>    make: *** [arch/microblaze/kernel] Error 2
>
>
> The checkpatch test was a response to this (rejected) patch series:
>
> http://lkml.indiana.edu/hypermail/linux/kernel/0912.1/02659.html
>
> I'm the lucky victim, here, since I'm using gcc 4.6.4 (which notices 
> the conflicting qualifiers), while other Microblaze users are still 
> working with gcc 4.1.2 (which didn't care.) All other architectures 
> (including mn10300, arc, and hexagon, which were accepted after the 
> checkpatch commit) ignore the checkpatch rule and declare 
> arch_kgdb_ops non-const.
>
> In any case, I wanted to point out (a) that the patch fails 
> checkpatch, (b) that I'm submitting it anyway, and (c) that I'm 
> utterly ambivalent about how the struct is declared, provided I can 
> compile working kernels on Microblaze.

Can you give me an update on including this patch in 
linux-2.6-microblaze? I realize the "const" was added as a result of a 
checkpatch.pl complaint (see 6bd55f0bbaebb79b39e147aa864401fd0c94db82), 
but it seems like a non-compiling kernel (even for trivial reasons) is 
worse than a noisy Perl script.

thanks,
Graeme
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/