From mboxrd@z Thu Jan 1 00:00:00 1970 From: SF Markus Elfring Subject: Re: [PATCH 49/54] md/raid10: Replace printk() calls by the usage of higher level interfaces Date: Thu, 6 Oct 2016 19:20:53 +0200 Message-ID: <321cc4aa-679a-a2fe-be0b-96285b19e489@users.sourceforge.net> References: <566ABCD9.1060404@users.sourceforge.net> <786843ef-4b6f-eb04-7326-2f6f5b408826@users.sourceforge.net> <1475771616.1914.9.camel@perches.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1475771616.1914.9.camel@perches.com> Sender: linux-raid-owner@vger.kernel.org To: Joe Perches Cc: linux-raid@vger.kernel.org, Christoph Hellwig , Guoqing Jiang , Jens Axboe , Mike Christie , Neil Brown , Shaohua Li , Tomasz Majchrzak , LKML , kernel-janitors@vger.kernel.org, Julia Lawall List-Id: linux-raid.ids >> diff --git a/drivers/md/raid10.c b/drivers/md/raid10.c >> index 5119846..0f2cb20 100644 >> --- a/drivers/md/raid10.c >> +++ b/drivers/md/raid10.c >> @@ -18,6 +18,8 @@ >> * Software Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. >> */ >> >> +#define MY_LOG_PREFIX KBUILD_MODNAME ": " >> +#define pr_fmt(fmt) MY_LOG_PREFIX fmt > > Please just use > > #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt > > like the more than thousand other uses in the kernel. Thanks for your suggestion. I got the impression that the omission of a macro like "MY_LOG_PREFIX" would not really work for the suggested source code transformation so far. How should a multiline log message be achieved as it was constructed in the function "raid10_error" (for example)? https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/tree/drivers/md/raid10.c?id=c802e87fbe2d4dd58982d01b3c39bc5a781223aa#n1589 Regards, Markus