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:	Sat, 08 Jun 2013 13:58:57 -0700
From:	Graeme Smecher <gsmecher@...eespeedlogic.com>
To:	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] microblaze: Don't mark arch_kgdb_ops as const.

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.

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