From patchwork Fri Feb 1 22:33:58 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Joe Perches X-Patchwork-Id: 2083241 Return-Path: X-Original-To: patchwork-linux-rdma@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork2.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork2.kernel.org (Postfix) with ESMTP id D6FA5DF264 for ; Fri, 1 Feb 2013 22:34:04 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757895Ab3BAWeD (ORCPT ); Fri, 1 Feb 2013 17:34:03 -0500 Received: from perches-mx.perches.com ([206.117.179.246]:55588 "EHLO labridge.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1757885Ab3BAWeC (ORCPT ); Fri, 1 Feb 2013 17:34:02 -0500 Received: from [174.254.66.22] (account joe@perches.com HELO [192.168.42.183]) by labridge.com (CommuniGate Pro SMTP 5.0.14) with ESMTPA id 20585860; Fri, 01 Feb 2013 14:34:01 -0800 Message-ID: <1359758038.17807.18.camel@joe-AO722> Subject: Re: [PATCH] tile: revert pr_info to printk in From: Joe Perches To: Chris Metcalf Cc: linux-kernel@vger.kernel.org, David Dillow , Roland Dreier , Sean Hefty , Hal Rosenstock , linux-rdma@vger.kernel.org Date: Fri, 01 Feb 2013 14:33:58 -0800 In-Reply-To: <201302012043.r11KhFXi024365@farm-0021.internal.tilera.com> References: <201302012043.r11KhFXi024365@farm-0021.internal.tilera.com> X-Mailer: Evolution 3.6.2-0ubuntu0.1 Mime-Version: 1.0 Sender: linux-rdma-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-rdma@vger.kernel.org On Fri, 2013-02-01 at 12:34 -0500, Chris Metcalf wrote: > Using pr_info in a header exposes us to potential trouble from > subsystems that define pr_fmt. This change fixes: > > In file included from include/linux/scatterlist.h:10, > from include/scsi/scsi.h:12, > from drivers/infiniband/ulp/srp/ib_srp.c:46: > arch/tile/include/asm/io.h: In function ‘ioport_map’: > arch/tile/include/asm/io.h:296: error: expected ‘)’ before ‘PFX’ Interesting. > diff --git a/arch/tile/include/asm/io.h b/arch/tile/include/asm/io.h [] > @@ -292,7 +292,7 @@ static inline long ioport_panic(void) > > static inline void __iomem *ioport_map(unsigned long port, unsigned int len) > { > - pr_info("ioport_map: mapping IO resources is unsupported on tile.\n"); > + printk("ioport_map: mapping IO resources is unsupported on tile.\n"); It'd be nicer to add an appropriate KERN_ here. My preference would be to change ib_srp.c (and ib_srpt) like: Acked-by: Chris Metcalf --- drivers/infiniband/ulp/srp/ib_srp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/drivers/infiniband/ulp/srp/ib_srp.c b/drivers/infiniband/ulp/srp/ib_srp.c index d5088ce..59bf409 100644 --- a/drivers/infiniband/ulp/srp/ib_srp.c +++ b/drivers/infiniband/ulp/srp/ib_srp.c @@ -30,7 +30,7 @@ * SOFTWARE. */ -#define pr_fmt(fmt) PFX fmt +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt #include #include