From: grygorii.strashko@ti•com (Grygorii Strashko)
To: linux-arm-kernel@lists•infradead.org
Subject: [linux-next][regression] [PATCH] percpu: add preemption checks to __this_cpu ops
Date: Tue, 18 Mar 2014 17:48:15 +0200 [thread overview]
Message-ID: <53286ABF.3040408@ti.com> (raw)
In-Reply-To: <alpine.DEB.2.10.1403180922050.17965@nuc>
On 03/18/2014 04:37 PM, Christoph Lameter wrote:
> On Tue, 18 Mar 2014, Grygorii Strashko wrote:
>
>> diff --git a/lib/smp_processor_id.c b/lib/smp_processor_id.c
>> index a270dce..73a2004 100644
>> --- a/lib/smp_processor_id.c
>> +++ b/lib/smp_processor_id.c
>> @@ -58,9 +58,6 @@ EXPORT_SYMBOL(debug_smp_processor_id);
>>
>> notrace void __this_cpu_preempt_check(const char *op)
>> {
>> - char text[40];
>> -
>> - snprintf(text, sizeof(text), "__this_cpu_%s()", op);
>> - check_preemption_disabled(text);
>> + check_preemption_disabled(op);
>> }
>> EXPORT_SYMBOL(__this_cpu_preempt_check);
>
> So it looks like there is an early this cpu operation in a context that
> cannot handle snprintf. But the checks in check_preemption_disabled()
> avoid processing there so that works.
Just to be sure that I've described time point of issue correctly:
[ 2.161746] dwc3 2690000.dwc3: failed to initialize core
[ 2.167255] dwc3: probe of 2690000.dwc3 failed with error -38
[ 2.259687] Freeing unused kernel memory: 280K (c0678000 - c06be000)
Please press Enter to activate this console.
^^^ system stall here
Any way, I can boot and console works fine with your change :)
Thanks.
Tested-by: Grygorii Strashko <grygorii.strashko@ti•com>
>
> We could fix this by moving the string concatenation operation into
> the check function.
>
> Index: linux/lib/smp_processor_id.c
> ===================================================================
> --- linux.orig/lib/smp_processor_id.c 2014-03-18 09:36:31.330450525 -0500
> +++ linux/lib/smp_processor_id.c 2014-03-18 09:36:37.822315534 -0500
> @@ -7,7 +7,8 @@
> #include <linux/kallsyms.h>
> #include <linux/sched.h>
>
> -notrace static unsigned int check_preemption_disabled(char *what)
> +notrace static unsigned int check_preemption_disabled(const char *what1,
> + const char *what2)
> {
> int this_cpu = raw_smp_processor_id();
>
> @@ -38,8 +39,8 @@
> if (!printk_ratelimit())
> goto out_enable;
>
> - printk(KERN_ERR "BUG: using %s in preemptible [%08x] code: %s/%d\n",
> - what, preempt_count() - 1, current->comm, current->pid);
> + printk(KERN_ERR "BUG: using %s%s() in preemptible [%08x] code: %s/%d\n",
> + what1, what2, preempt_count() - 1, current->comm, current->pid);
>
> print_symbol("caller is %s\n", (long)__builtin_return_address(0));
> dump_stack();
> @@ -52,15 +53,12 @@
>
> notrace unsigned int debug_smp_processor_id(void)
> {
> - return check_preemption_disabled("smp_processor_id()");
> + return check_preemption_disabled("smp_processor_id","");
> }
> EXPORT_SYMBOL(debug_smp_processor_id);
>
> notrace void __this_cpu_preempt_check(const char *op)
> {
> - char text[40];
> -
> - snprintf(text, sizeof(text), "__this_cpu_%s()", op);
> - check_preemption_disabled(text);
> + check_preemption_disabled("__this_cpu_", op);
> }
> EXPORT_SYMBOL(__this_cpu_preempt_check);
>
next prev parent reply other threads:[~2014-03-18 15:48 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-18 15:01 [linux-next][regression] [PATCH] percpu: add preemption checks to __this_cpu ops Grygorii Strashko
2014-03-18 14:37 ` Christoph Lameter
2014-03-18 15:48 ` Grygorii Strashko [this message]
2014-03-18 15:54 ` Christoph Lameter
2014-03-18 21:37 ` Andrew Morton
2014-03-19 11:18 ` Grygorii Strashko
2014-03-20 14:06 ` Christoph Lameter
[not found] ` <534C182C.8040206@ti.com>
2014-04-14 17:12 ` [3.15-rc1] a huge number of warnings produced by xhci & Christoph Lameter
2014-04-15 10:00 ` Grygorii Strashko
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=53286ABF.3040408@ti.com \
--to=grygorii.strashko@ti$(echo .)com \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox