Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1422853AbbEON1y (ORCPT ); Fri, 15 May 2015 09:27:54 -0400 Received: from e38.co.us.ibm.com ([32.97.110.159]:59484 "EHLO e38.co.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1161059AbbEON1w (ORCPT ); Fri, 15 May 2015 09:27:52 -0400 From: "Shreyas B. Prabhu" To: akpm@linux-foundation.org, rostedt@goodmis.org Cc: linux-kernel@vger.kernel.org, preeti@linux.vnet.ibm.com, paulmck@linux.vnet.ibm.com, mingo@redhat.com, "Shreyas B. Prabhu" Subject: [PATCH linux-next] tracing/mm: Use raw_smp_processor_id() instead of smp_processor_id() Date: Fri, 15 May 2015 18:57:41 +0530 Message-Id: <1431696461-9683-1-git-send-email-shreyas@linux.vnet.ibm.com> X-Mailer: git-send-email 1.9.3 X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 15051513-0029-0000-0000-000009CD6CA5 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3538 Lines: 92 trace_mm_page_pcpu_drain, trace_kmem_cache_free, trace_mm_page_free can be potentially called from an offlined cpu. Since trace points use RCU and RCU should not be used from offlined cpus, we have checks to filter out such calls. But these checks use smp_processor_id() and since these trace calls can happen from preemptable sections, this throws a warning when running with CONFIG_DEBUG_PREEMPT set. Now consider task gets migrated after calling smp_processor_id() - From an online cpu to another online cpu - No impact - From an online cpu to an offline cpu - Should never happen - From an offline cpu to an online cpu - Once a cpu has been offlined it returns to cpu_idle_loop, discovers its offline and calls arch_cpu_idle_dead. All this happens with preemption disabled. So this scenario too should never happen. Thus running with PREEMPT set has no impact on the condition. So use raw_smp_processor_id() so that the warnings are suppressed. Signed-off-by: Shreyas B. Prabhu --- include/trace/events/kmem.h | 34 +++++++++++++++++++++++++++++++--- 1 file changed, 31 insertions(+), 3 deletions(-) diff --git a/include/trace/events/kmem.h b/include/trace/events/kmem.h index 6cd975f..f7554fd 100644 --- a/include/trace/events/kmem.h +++ b/include/trace/events/kmem.h @@ -146,7 +146,16 @@ DEFINE_EVENT_CONDITION(kmem_free, kmem_cache_free, TP_ARGS(call_site, ptr), - TP_CONDITION(cpu_online(smp_processor_id())) + /* + * This trace can be potentially called from an offlined cpu. + * Since trace points use RCU and RCU should not be used from + * offline cpus, filter such calls out. + * While this trace can be called from a preemptable section, + * it has no impact on the condition since tasks can migrate + * only from online cpus to other online cpus. Thus its safe + * to use raw_smp_processor_id. + */ + TP_CONDITION(cpu_online(raw_smp_processor_id())) ); TRACE_EVENT_CONDITION(mm_page_free, @@ -155,7 +164,17 @@ TRACE_EVENT_CONDITION(mm_page_free, TP_ARGS(page, order), - TP_CONDITION(cpu_online(smp_processor_id())), + + /* + * This trace can be potentially called from an offlined cpu. + * Since trace points use RCU and RCU should not be used from + * offline cpus, filter such calls out. + * While this trace can be called from a preemptable section, + * it has no impact on the condition since tasks can migrate + * only from online cpus to other online cpus. Thus its safe + * to use raw_smp_processor_id. + */ + TP_CONDITION(cpu_online(raw_smp_processor_id())), TP_STRUCT__entry( __field( unsigned long, pfn ) @@ -263,7 +282,16 @@ TRACE_EVENT_CONDITION(mm_page_pcpu_drain, TP_ARGS(page, order, migratetype), - TP_CONDITION(cpu_online(smp_processor_id())), + /* + * This trace can be potentially called from an offlined cpu. + * Since trace points use RCU and RCU should not be used from + * offline cpus, filter such calls out. + * While this trace can be called from a preemptable section, + * it has no impact on the condition since tasks can migrate + * only from online cpus to other online cpus. Thus its safe + * to use raw_smp_processor_id. + */ + TP_CONDITION(cpu_online(raw_smp_processor_id())), TP_STRUCT__entry( __field( unsigned long, pfn ) -- 1.9.3 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/