From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757607AbYJWRZj (ORCPT ); Thu, 23 Oct 2008 13:25:39 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753581AbYJWRZP (ORCPT ); Thu, 23 Oct 2008 13:25:15 -0400 Received: from qb-out-0506.google.com ([72.14.204.228]:20444 "EHLO qb-out-0506.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752494AbYJWRZM (ORCPT ); Thu, 23 Oct 2008 13:25:12 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:to:subject:cc:in-reply-to:mime-version :content-type:content-transfer-encoding:content-disposition :references; b=wcXG1UyzsC+VVa5xHxLqBV5NK0f7hF6J6WF7W+Uc1tLX5AHwrxnedQiOmjZoAbASKR KXS2yVTxWWyN5QU5kPt6MoOQwVOzsEqwQPDUaB9eDiwe8lU7f32d63VboFOQv5TWs3K8 m5gL8jWN21mTYbE8OcVCRQzBChoKPpK/uqNSQ= Message-ID: Date: Thu, 23 Oct 2008 19:25:10 +0200 From: "=?ISO-8859-1?Q?Fr=E9d=E9ric_Weisbecker?=" To: "Steven Rostedt" Subject: Re: [PATCH] tracing: fix a build error on alpha and m68k Cc: "Ingo Molnar" , "Alexey Dobriyan" , "Geert Uytterhoeven" , "Stephen Rothwell" , linux-next@vger.kernel.org, linux-kernel@vger.kernel.org In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <4900B414.7090002@gmail.com> <20081023163433.GA3160@x200.localdomain> <20081023163850.GB16591@elte.hu> <20081023165504.GB25697@elte.hu> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 2008/10/23 Steven Rostedt : > > On Thu, 23 Oct 2008, Fr?d?ric Weisbecker wrote: >> >> But to let the core trace.c to be built for other types of tracing, >> the best thing I see is to put some ifdef CONFIG_TRACE_IRQFLAGS on the >> tracing_cpumask functions and structure, and actually on the creation >> of this file. > > Is there a reason that you use "raw_local_irq_disable" instead of > local_irq_disable? I'm not using anything :) This is about the tracing_cpumask debugfs file implemented in trace.c It's tracing_cpumask_write function uses the raw_local functions, I guess because its locking doesn't need to be traced. And the problem here is that some archs don't implement raw_local functions and so trace.c doesn't compile with alpha or m68k. But actually this file is not needed for them.