From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.8 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_PASS,URIBL_BLOCKED,USER_AGENT_NEOMUTT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 05C92C04EB8 for ; Thu, 6 Dec 2018 14:10:10 +0000 (UTC) Received: from lists.ozlabs.org (lists.ozlabs.org [203.11.71.2]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 7A31320868 for ; Thu, 6 Dec 2018 14:10:09 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=linaro.org header.i=@linaro.org header.b="UmuU97v0" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 7A31320868 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linaro.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Received: from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3]) by lists.ozlabs.org (Postfix) with ESMTP id 439ctl3cQ3zDr4h for ; Fri, 7 Dec 2018 01:10:07 +1100 (AEDT) Authentication-Results: lists.ozlabs.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: lists.ozlabs.org; dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=linaro.org header.i=@linaro.org header.b="UmuU97v0"; dkim-atps=neutral Authentication-Results: lists.ozlabs.org; spf=pass (mailfrom) smtp.mailfrom=linaro.org (client-ip=2a00:1450:4864:20::344; helo=mail-wm1-x344.google.com; envelope-from=daniel.thompson@linaro.org; receiver=) Authentication-Results: lists.ozlabs.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: lists.ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=linaro.org header.i=@linaro.org header.b="UmuU97v0"; dkim-atps=neutral Received: from mail-wm1-x344.google.com (mail-wm1-x344.google.com [IPv6:2a00:1450:4864:20::344]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 439cqc04mpzDr3Q for ; Fri, 7 Dec 2018 01:07:23 +1100 (AEDT) Received: by mail-wm1-x344.google.com with SMTP id y1so1100443wmi.3 for ; Thu, 06 Dec 2018 06:07:23 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=mrLQohKt3Zd4tuJsYkErScQ9F7yT3CKumXJ2/9XoJ7E=; b=UmuU97v0dECMyw4EJIt9lW+d20XfcZXGQFmnW7vIWPnCOUf+sGziZH+3SB8t0Z99lG tj9b9a9Hcc5hWs4EDLZ0jJOdmNNGzmf6wkrC1O2uJwR9IOWsMu/V6uXaKm0nzfI5o+2N 5NhRHBpZy8N763cn9zx9YNMXEdDgJKssT6tQg= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=mrLQohKt3Zd4tuJsYkErScQ9F7yT3CKumXJ2/9XoJ7E=; b=iCk82xmZtdqMm3RuvqYo9ji9vppLTnvQO/eQ0Bpi/I0lT3Dy05lQ7qEctEWMFPU2gt Z3PB65cNwXqbv6XU02PtafaqpC719ZSHhkPDRvIJOu+51ZDGgbIm19EJ+UWb1uKTaAEc Z8WIRfW47e13zvLhjElWrmHXc+/lO/ymYcur+xeJ6AhWxqM6i4aGQfaKi1nuAM92uaEe V+hGD9H5x9H1PhHBArIGjN/D/5TbvN+KmzTT/aU4eGoa+O4IX0KSSRiY3yvK84HsRidA DdfYSiILG2KJw8oJ1+ij4eG7I4oMHXv54/jmWQk+j1j8JnXerlkwx9eMp5hkDt5ZR7Wq OMww== X-Gm-Message-State: AA+aEWYirre6gc7aPAmUdt82JaUnvrnGeVQvkLOgrwqGMJEElukgnKDX NEATYbKf/SQFsdUrzYWAkkPk9Q== X-Google-Smtp-Source: AFSGD/VuBwY3hV6UF2pPBo1+kokfEApcL4yyEGJdnr2177AM/Ki8Dt+pW3gUt8MYRwavhA45czNNgg== X-Received: by 2002:a1c:7306:: with SMTP id d6mr9347766wmb.98.1544105240991; Thu, 06 Dec 2018 06:07:20 -0800 (PST) Received: from holly.lan (cpc141214-aztw34-2-0-cust773.18-1.cable.virginm.net. [86.9.19.6]) by smtp.gmail.com with ESMTPSA id l15sm663737wrw.4.2018.12.06.06.07.18 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 06 Dec 2018 06:07:19 -0800 (PST) Date: Thu, 6 Dec 2018 14:07:16 +0000 From: Daniel Thompson To: Christophe Leroy Subject: Re: [PATCH 2/2] kgdb/treewide: constify struct kgdb_arch arch_kgdb_ops Message-ID: <20181206140716.apapnpmdk3nrn7i3@holly.lan> References: <75bbcdd1e9277d66ebb06e349dda304bd01ce761.1543957194.git.christophe.leroy@c-s.fr> <6b1a19ffa0da02cff9b82b866ba31d57478501ce.1543957194.git.christophe.leroy@c-s.fr> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <6b1a19ffa0da02cff9b82b866ba31d57478501ce.1543957194.git.christophe.leroy@c-s.fr> User-Agent: NeoMutt/20180716 X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Rich Felker , "Gustavo A. R. Silva" , Will Deacon , linux-kernel@vger.kernel.org, Paul Mackerras , sparclinux@vger.kernel.org, linux-hexagon@vger.kernel.org, Yoshinori Sato , linux-sh@vger.kernel.org, x86@kernel.org, Russell King , Ingo Molnar , Catalin Marinas , James Hogan , linux-snps-arc@lists.infradead.org, uclinux-h8-devel@lists.sourceforge.jp, linux-mips@vger.kernel.org, Borislav Petkov , nios2-dev@lists.rocketboards.org, Thomas Gleixner , linux-arm-kernel@lists.infradead.org, Michal Simek , Vineet Gupta , Randy Dunlap , Douglas Anderson , Ralf Baechle , Richard Kuo , Paul Burton , Jason Wessel , kgdb-bugreport@lists.sourceforge.net, Ley Foon Tan , linuxppc-dev@lists.ozlabs.org, "David S. Miller" Errors-To: linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Sender: "Linuxppc-dev" On Wed, Dec 05, 2018 at 04:41:11AM +0000, Christophe Leroy wrote: > checkpatch.pl reports the following: > > WARNING: struct kgdb_arch should normally be const > #28: FILE: arch/mips/kernel/kgdb.c:397: > +struct kgdb_arch arch_kgdb_ops = { > > This report makes sense, as all other ops struct, this > one should also be const. This patch does the change. > > Signed-off-by: Christophe Leroy Acked-by: Daniel Thompson Similar to https://patchwork.kernel.org/patch/10701129/ I would be more comfortable to see a resend with the relevant arch maintainers explicitly called out with a Cc: entry here. > --- > arch/arc/kernel/kgdb.c | 2 +- > arch/arm/kernel/kgdb.c | 2 +- > arch/arm64/kernel/kgdb.c | 2 +- > arch/h8300/kernel/kgdb.c | 2 +- > arch/hexagon/kernel/kgdb.c | 2 +- > arch/microblaze/kernel/kgdb.c | 2 +- > arch/mips/kernel/kgdb.c | 2 +- > arch/nios2/kernel/kgdb.c | 2 +- > arch/powerpc/kernel/kgdb.c | 2 +- > arch/sh/kernel/kgdb.c | 2 +- > arch/sparc/kernel/kgdb_32.c | 2 +- > arch/sparc/kernel/kgdb_64.c | 2 +- > arch/x86/kernel/kgdb.c | 2 +- > include/linux/kgdb.h | 2 +- > 14 files changed, 14 insertions(+), 14 deletions(-) > > diff --git a/arch/arc/kernel/kgdb.c b/arch/arc/kernel/kgdb.c > index 9a3c34af2ae8..bfd04b442e36 100644 > --- a/arch/arc/kernel/kgdb.c > +++ b/arch/arc/kernel/kgdb.c > @@ -204,7 +204,7 @@ void kgdb_roundup_cpus(unsigned long flags) > local_irq_disable(); > } > > -struct kgdb_arch arch_kgdb_ops = { > +const struct kgdb_arch arch_kgdb_ops = { > /* breakpoint instruction: TRAP_S 0x3 */ > #ifdef CONFIG_CPU_BIG_ENDIAN > .gdb_bpt_instr = {0x78, 0x7e}, > diff --git a/arch/arm/kernel/kgdb.c b/arch/arm/kernel/kgdb.c > index caa0dbe3dc61..21a6d5958955 100644 > --- a/arch/arm/kernel/kgdb.c > +++ b/arch/arm/kernel/kgdb.c > @@ -274,7 +274,7 @@ int kgdb_arch_remove_breakpoint(struct kgdb_bkpt *bpt) > * and we handle the normal undef case within the do_undefinstr > * handler. > */ > -struct kgdb_arch arch_kgdb_ops = { > +const struct kgdb_arch arch_kgdb_ops = { > #ifndef __ARMEB__ > .gdb_bpt_instr = {0xfe, 0xde, 0xff, 0xe7} > #else /* ! __ARMEB__ */ > diff --git a/arch/arm64/kernel/kgdb.c b/arch/arm64/kernel/kgdb.c > index a20de58061a8..fe1d1f935b90 100644 > --- a/arch/arm64/kernel/kgdb.c > +++ b/arch/arm64/kernel/kgdb.c > @@ -357,7 +357,7 @@ void kgdb_arch_exit(void) > unregister_die_notifier(&kgdb_notifier); > } > > -struct kgdb_arch arch_kgdb_ops; > +const struct kgdb_arch arch_kgdb_ops; > > int kgdb_arch_set_breakpoint(struct kgdb_bkpt *bpt) > { > diff --git a/arch/h8300/kernel/kgdb.c b/arch/h8300/kernel/kgdb.c > index 1a1d30cb0609..602e478afbd5 100644 > --- a/arch/h8300/kernel/kgdb.c > +++ b/arch/h8300/kernel/kgdb.c > @@ -129,7 +129,7 @@ void kgdb_arch_exit(void) > /* Nothing to do */ > } > > -struct kgdb_arch arch_kgdb_ops = { > +const struct kgdb_arch arch_kgdb_ops = { > /* Breakpoint instruction: trapa #2 */ > .gdb_bpt_instr = { 0x57, 0x20 }, > }; > diff --git a/arch/hexagon/kernel/kgdb.c b/arch/hexagon/kernel/kgdb.c > index 16c24b22d0b2..f1924d483e78 100644 > --- a/arch/hexagon/kernel/kgdb.c > +++ b/arch/hexagon/kernel/kgdb.c > @@ -83,7 +83,7 @@ struct dbg_reg_def_t dbg_reg_def[DBG_MAX_REG_NUM] = { > { "syscall_nr", GDB_SIZEOF_REG, offsetof(struct pt_regs, syscall_nr)}, > }; > > -struct kgdb_arch arch_kgdb_ops = { > +const struct kgdb_arch arch_kgdb_ops = { > /* trap0(#0xDB) 0x0cdb0054 */ > .gdb_bpt_instr = {0x54, 0x00, 0xdb, 0x0c}, > }; > diff --git a/arch/microblaze/kernel/kgdb.c b/arch/microblaze/kernel/kgdb.c > index 6366f69d118e..130cd0f064ce 100644 > --- a/arch/microblaze/kernel/kgdb.c > +++ b/arch/microblaze/kernel/kgdb.c > @@ -143,7 +143,7 @@ void kgdb_arch_exit(void) > /* > * Global data > */ > -struct kgdb_arch arch_kgdb_ops = { > +const struct kgdb_arch arch_kgdb_ops = { > #ifdef __MICROBLAZEEL__ > .gdb_bpt_instr = {0x18, 0x00, 0x0c, 0xba}, /* brki r16, 0x18 */ > #else > diff --git a/arch/mips/kernel/kgdb.c b/arch/mips/kernel/kgdb.c > index 31eff1bec577..edfdc2ec2d16 100644 > --- a/arch/mips/kernel/kgdb.c > +++ b/arch/mips/kernel/kgdb.c > @@ -394,7 +394,7 @@ int kgdb_arch_handle_exception(int vector, int signo, int err_code, > return -1; > } > > -struct kgdb_arch arch_kgdb_ops = { > +const struct kgdb_arch arch_kgdb_ops = { > #ifdef CONFIG_CPU_BIG_ENDIAN > .gdb_bpt_instr = { spec_op << 2, 0x00, 0x00, break_op }, > #else > diff --git a/arch/nios2/kernel/kgdb.c b/arch/nios2/kernel/kgdb.c > index 117859122d1c..37b25f844a2d 100644 > --- a/arch/nios2/kernel/kgdb.c > +++ b/arch/nios2/kernel/kgdb.c > @@ -165,7 +165,7 @@ void kgdb_arch_exit(void) > /* Nothing to do */ > } > > -struct kgdb_arch arch_kgdb_ops = { > +const struct kgdb_arch arch_kgdb_ops = { > /* Breakpoint instruction: trap 30 */ > .gdb_bpt_instr = { 0xba, 0x6f, 0x3b, 0x00 }, > }; > diff --git a/arch/powerpc/kernel/kgdb.c b/arch/powerpc/kernel/kgdb.c > index 59c578f865aa..bdb588b1d8fb 100644 > --- a/arch/powerpc/kernel/kgdb.c > +++ b/arch/powerpc/kernel/kgdb.c > @@ -477,7 +477,7 @@ int kgdb_arch_remove_breakpoint(struct kgdb_bkpt *bpt) > /* > * Global data > */ > -struct kgdb_arch arch_kgdb_ops; > +const struct kgdb_arch arch_kgdb_ops; > > static int kgdb_not_implemented(struct pt_regs *regs) > { > diff --git a/arch/sh/kernel/kgdb.c b/arch/sh/kernel/kgdb.c > index 4f04c6638a4d..a24c48446e98 100644 > --- a/arch/sh/kernel/kgdb.c > +++ b/arch/sh/kernel/kgdb.c > @@ -382,7 +382,7 @@ void kgdb_arch_exit(void) > unregister_die_notifier(&kgdb_notifier); > } > > -struct kgdb_arch arch_kgdb_ops = { > +const struct kgdb_arch arch_kgdb_ops = { > /* Breakpoint instruction: trapa #0x3c */ > #ifdef CONFIG_CPU_LITTLE_ENDIAN > .gdb_bpt_instr = { 0x3c, 0xc3 }, > diff --git a/arch/sparc/kernel/kgdb_32.c b/arch/sparc/kernel/kgdb_32.c > index 639c8e54530a..7580775a14b9 100644 > --- a/arch/sparc/kernel/kgdb_32.c > +++ b/arch/sparc/kernel/kgdb_32.c > @@ -166,7 +166,7 @@ void kgdb_arch_set_pc(struct pt_regs *regs, unsigned long ip) > regs->npc = regs->pc + 4; > } > > -struct kgdb_arch arch_kgdb_ops = { > +const struct kgdb_arch arch_kgdb_ops = { > /* Breakpoint instruction: ta 0x7d */ > .gdb_bpt_instr = { 0x91, 0xd0, 0x20, 0x7d }, > }; > diff --git a/arch/sparc/kernel/kgdb_64.c b/arch/sparc/kernel/kgdb_64.c > index a68bbddbdba4..5d6c2d287e85 100644 > --- a/arch/sparc/kernel/kgdb_64.c > +++ b/arch/sparc/kernel/kgdb_64.c > @@ -195,7 +195,7 @@ void kgdb_arch_set_pc(struct pt_regs *regs, unsigned long ip) > regs->tnpc = regs->tpc + 4; > } > > -struct kgdb_arch arch_kgdb_ops = { > +const struct kgdb_arch arch_kgdb_ops = { > /* Breakpoint instruction: ta 0x72 */ > .gdb_bpt_instr = { 0x91, 0xd0, 0x20, 0x72 }, > }; > diff --git a/arch/x86/kernel/kgdb.c b/arch/x86/kernel/kgdb.c > index 8e36f249646e..e7effc02f13c 100644 > --- a/arch/x86/kernel/kgdb.c > +++ b/arch/x86/kernel/kgdb.c > @@ -804,7 +804,7 @@ int kgdb_arch_remove_breakpoint(struct kgdb_bkpt *bpt) > (char *)bpt->saved_instr, BREAK_INSTR_SIZE); > } > > -struct kgdb_arch arch_kgdb_ops = { > +const struct kgdb_arch arch_kgdb_ops = { > /* Breakpoint instruction: */ > .gdb_bpt_instr = { 0xcc }, > .flags = KGDB_HW_BREAKPOINT, > diff --git a/include/linux/kgdb.h b/include/linux/kgdb.h > index e465bb15912d..3bf313311cca 100644 > --- a/include/linux/kgdb.h > +++ b/include/linux/kgdb.h > @@ -281,7 +281,7 @@ struct kgdb_io { > int is_console; > }; > > -extern struct kgdb_arch arch_kgdb_ops; > +extern const struct kgdb_arch arch_kgdb_ops; > > extern unsigned long kgdb_arch_pc(int exception, struct pt_regs *regs); > > -- > 2.13.3 >