From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Thu, 02 Feb 2017 11:00:57 +0100 (CET) Received: from mailapp01.imgtec.com ([195.59.15.196]:30764 "EHLO mailapp01.imgtec.com" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S23993543AbdBBKAuAZann convert rfc822-to-8bit (ORCPT ); Thu, 2 Feb 2017 11:00:50 +0100 Received: from hhmail02.hh.imgtec.org (unknown [10.100.10.20]) by Forcepoint Email with ESMTPS id B8C2B864B93E0; Thu, 2 Feb 2017 10:00:38 +0000 (GMT) Received: from localhost (192.168.154.110) by hhmail02.hh.imgtec.org (10.100.10.21) with Microsoft SMTP Server (TLS) id 14.3.294.0; Thu, 2 Feb 2017 10:00:41 +0000 Date: Thu, 2 Feb 2017 10:00:41 +0000 From: James Hogan To: Joshua Kinard CC: Ralf Baechle , "Maciej W. Rozycki" , Linux/MIPS Subject: Re: [PATCH] MIPS: Replace printk calls in cpu-bugs64.c with pr_info/pr_cont Message-ID: <20170202100041.GA13058@jhogan-linux.le.imgtec.org> References: <29b7fc69-97f4-6a3a-0e65-2678f9c30cef@gentoo.org> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline Content-Transfer-Encoding: 8BIT In-Reply-To: <29b7fc69-97f4-6a3a-0e65-2678f9c30cef@gentoo.org> User-Agent: Mutt/1.5.24 (2015-08-30) X-Originating-IP: [192.168.154.110] Return-Path: X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0) X-Orcpt: rfc822;linux-mips@linux-mips.org Original-Recipient: rfc822;linux-mips@linux-mips.org X-archive-position: 56586 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: james.hogan@imgtec.com Precedence: bulk List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-Id: linux-mips X-List-ID: linux-mips List-subscribe: List-owner: List-post: List-archive: X-list: linux-mips Hi Joshua, On Thu, Feb 02, 2017 at 03:56:26AM -0500, Joshua Kinard wrote: > From: Joshua Kinard > > In arch/mips/kernel/cpu-bugs64.c, replace initial printk's in three > bug-checking functions with pr_info and replace several continuation > printk's with pr_info/pr_cont calls to avoid kernel log output like > this: > > [ 0.899065] Checking for the daddi bug... > [ 0.899098] no. > > This makes the output appear correctly: > > [ 0.898643] Checking for the daddi bug... no. > > Signed-off-by: Joshua Kinard A variation of this patch is already applied, but without the change of printk -> pr_info: https://patchwork.linux-mips.org/patch/14916/ https://git.linux-mips.org/cgit/ralf/upstream-sfr.git/commit/?id=35e7f7885e1b1b272a73c0de3227fc9a3e95a7e3 Cheers James > --- > arch/mips/kernel/cpu-bugs64.c | 30 +++++++++++++++--------------- > 1 file changed, 15 insertions(+), 15 deletions(-) > > diff --git a/arch/mips/kernel/cpu-bugs64.c b/arch/mips/kernel/cpu-bugs64.c > index a378e44688f5..d4483551a926 100644 > --- a/arch/mips/kernel/cpu-bugs64.c > +++ b/arch/mips/kernel/cpu-bugs64.c > @@ -122,7 +122,7 @@ static inline void check_mult_sh(void) > long v1[8], v2[8], w[8]; > int bug, fix, i; > > - printk("Checking for the multiply/shift bug... "); > + pr_info("Checking for the multiply/shift bug... "); > > /* > * Testing discovered false negatives for certain code offsets > @@ -148,11 +148,11 @@ static inline void check_mult_sh(void) > bug = 1; > > if (bug == 0) { > - printk("no.\n"); > + pr_cont("no.\n"); > return; > } > > - printk("yes, workaround... "); > + pr_cont("yes, workaround... "); > > fix = 1; > for (i = 0; i < 8; i++) > @@ -160,11 +160,11 @@ static inline void check_mult_sh(void) > fix = 0; > > if (fix == 1) { > - printk("yes.\n"); > + pr_cont("yes.\n"); > return; > } > > - printk("no.\n"); > + pr_cont("no.\n"); > panic(bug64hit, !R4000_WAR ? r4kwar : nowar); > } > > @@ -187,7 +187,7 @@ static inline void check_daddi(void) > void *handler; > long v, tmp; > > - printk("Checking for the daddi bug... "); > + pr_info("Checking for the daddi bug... "); > > local_irq_save(flags); > handler = set_except_vector(EXCCODE_OV, handle_daddi_ov); > @@ -218,11 +218,11 @@ static inline void check_daddi(void) > local_irq_restore(flags); > > if (daddi_ov) { > - printk("no.\n"); > + pr_cont("no.\n"); > return; > } > > - printk("yes, workaround... "); > + pr_cont("yes, workaround... "); > > local_irq_save(flags); > handler = set_except_vector(EXCCODE_OV, handle_daddi_ov); > @@ -236,11 +236,11 @@ static inline void check_daddi(void) > local_irq_restore(flags); > > if (daddi_ov) { > - printk("yes.\n"); > + pr_cont("yes.\n"); > return; > } > > - printk("no.\n"); > + pr_cont("no.\n"); > panic(bug64hit, !DADDI_WAR ? daddiwar : nowar); > } > > @@ -250,7 +250,7 @@ static inline void check_daddiu(void) > { > long v, w, tmp; > > - printk("Checking for the daddiu bug... "); > + pr_info("Checking for the daddiu bug... "); > > /* > * The following code leads to a wrong result of daddiu when > @@ -288,11 +288,11 @@ static inline void check_daddiu(void) > daddiu_bug = v != w; > > if (!daddiu_bug) { > - printk("no.\n"); > + pr_cont("no.\n"); > return; > } > > - printk("yes, workaround... "); > + pr_cont("yes, workaround... "); > > asm volatile( > "addiu %2, $0, %3\n\t" > @@ -304,11 +304,11 @@ static inline void check_daddiu(void) > : "I" (0xffffffffffffdb9aUL), "I" (0x1234)); > > if (v == w) { > - printk("yes.\n"); > + pr_cont("yes.\n"); > return; > } > > - printk("no.\n"); > + pr_cont("no.\n"); > panic(bug64hit, !DADDI_WAR ? daddiwar : nowar); > } > > >