From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ntl@pobox.com>
Received: from sceptre.pobox.com (sceptre.pobox.com [207.106.133.20])
	by ozlabs.org (Postfix) with ESMTP id F3F14DDF11
	for <linuxppc-dev@ozlabs.org>; Wed, 12 Sep 2007 00:51:53 +1000 (EST)
Date: Tue, 11 Sep 2007 09:51:31 -0500
From: Nathan Lynch <ntl@pobox.com>
To: Joachim Fenkes <fenkes@de.ibm.com>
Subject: Re: [PATCH 08/12] IB/ehca: Replace get_paca()->paca_index by the more
	portable smp_processor_id()
Message-ID: <20070911145131.GN32388@localdomain>
References: <200709111518.26276.fenkes@de.ibm.com>
	<200709111533.14333.fenkes@de.ibm.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
In-Reply-To: <200709111533.14333.fenkes@de.ibm.com>
Cc: LKML <linux-kernel@vger.kernel.org>, OF-EWG <ewg@lists.openfabrics.org>,
	LinuxPPC-Dev <linuxppc-dev@ozlabs.org>,
	Christoph Raisch <raisch@de.ibm.com>,
	OF-General <general@lists.openfabrics.org>,
	Stefan Roscher <stefan.roscher@de.ibm.com>
List-Id: Linux on PowerPC Developers Mail List <linuxppc-dev.ozlabs.org>
List-Unsubscribe: <https://ozlabs.org/mailman/listinfo/linuxppc-dev>,
	<mailto:linuxppc-dev-request@ozlabs.org?subject=unsubscribe>
List-Archive: <http://ozlabs.org/pipermail/linuxppc-dev>
List-Post: <mailto:linuxppc-dev@ozlabs.org>
List-Help: <mailto:linuxppc-dev-request@ozlabs.org?subject=help>
List-Subscribe: <https://ozlabs.org/mailman/listinfo/linuxppc-dev>,
	<mailto:linuxppc-dev-request@ozlabs.org?subject=subscribe>

Hi,

Joachim Fenkes wrote:
> Signed-off-by: Joachim Fenkes <fenkes@de.ibm.com>
> ---
>  drivers/infiniband/hw/ehca/ehca_tools.h |   14 +++++++-------
>  1 files changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/infiniband/hw/ehca/ehca_tools.h b/drivers/infiniband/hw/ehca/ehca_tools.h
> index f9b264b..863f972 100644
> --- a/drivers/infiniband/hw/ehca/ehca_tools.h
> +++ b/drivers/infiniband/hw/ehca/ehca_tools.h
> @@ -73,37 +73,37 @@ extern int ehca_debug_level;
>  		if (unlikely(ehca_debug_level)) \
>  			dev_printk(KERN_DEBUG, (ib_dev)->dma_device, \
>  				   "PU%04x EHCA_DBG:%s " format "\n", \
> -				   get_paca()->paca_index, __FUNCTION__, \
> +				   smp_processor_id(), __FUNCTION__, \
>  				   ## arg); \
>  	} while (0)
>  
>  #define ehca_info(ib_dev, format, arg...) \
>  	dev_info((ib_dev)->dma_device, "PU%04x EHCA_INFO:%s " format "\n", \
> -		 get_paca()->paca_index, __FUNCTION__, ## arg)
> +		 smp_processor_id(), __FUNCTION__, ## arg)
>  
>  #define ehca_warn(ib_dev, format, arg...) \
>  	dev_warn((ib_dev)->dma_device, "PU%04x EHCA_WARN:%s " format "\n", \
> -		 get_paca()->paca_index, __FUNCTION__, ## arg)
> +		 smp_processor_id(), __FUNCTION__, ## arg)
>  
>  #define ehca_err(ib_dev, format, arg...) \
>  	dev_err((ib_dev)->dma_device, "PU%04x EHCA_ERR:%s " format "\n", \
> -		get_paca()->paca_index, __FUNCTION__, ## arg)
> +		smp_processor_id(), __FUNCTION__, ## arg)

I think I see these macros used in preemptible code (e.g. ehca_probe),
where smp_processor_id() will print a warning when
CONFIG_DEBUG_PREEMPT=y.  Probably better to use raw_smp_processor_id.