From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751007AbdE3Byd (ORCPT ); Mon, 29 May 2017 21:54:33 -0400 Received: from smtprelay0186.hostedemail.com ([216.40.44.186]:43855 "EHLO smtprelay.hostedemail.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1750897AbdE3Byc (ORCPT ); Mon, 29 May 2017 21:54:32 -0400 X-Session-Marker: 6A6F6540706572636865732E636F6D X-Spam-Summary: 2,0,0,,d41d8cd98f00b204,joe@perches.com,:::::::::::,RULES_HIT:2:41:69:334:355:379:541:599:800:960:973:988:989:1260:1277:1311:1313:1314:1345:1359:1373:1431:1437:1515:1516:1518:1535:1593:1594:1605:1606:1730:1747:1777:1792:2194:2199:2393:2553:2559:2562:2828:3138:3139:3140:3141:3142:3622:3865:3867:3868:3870:3871:4117:4321:5007:6119:8603:8957:9592:10004:10848:10967:11026:11232:11473:11658:11783:11914:12043:12291:12296:12438:12555:12683:12740:12895:13160:13229:13439:13894:14110:14659:21060:21063:21080:21433:21451:21611:21627:30025:30029:30054:30060:30062:30070:30090:30091,0,RBL:none,CacheIP:none,Bayesian:0.5,0.5,0.5,Netcheck:none,DomainCache:0,MSF:not bulk,SPF:,MSBL:0,DNSBL:none,Custom_rules:0:0:0,LFtime:2,LUA_SUMMARY:none X-HE-Tag: toe97_27004b62c1345 X-Filterd-Recvd-Size: 6883 Message-ID: <1496109266.2618.5.camel@perches.com> Subject: Re: linux-next: build failure after merge of the rcu tree From: Joe Perches To: Stephen Rothwell , "Paul E. McKenney" Cc: Linux-Next Mailing List , Linux Kernel Mailing List , Mike Frysinger , Steven Miao Date: Mon, 29 May 2017 18:54:26 -0700 In-Reply-To: <20170530114050.2b702cc1@canb.auug.org.au> References: <20170529160209.5230c05b@canb.auug.org.au> <20170529211505.GP3956@linux.vnet.ibm.com> <20170530114050.2b702cc1@canb.auug.org.au> Content-Type: text/plain; charset="ISO-8859-1" X-Mailer: Evolution 3.22.6-1ubuntu1 Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 2017-05-30 at 11:40 +1000, Stephen Rothwell wrote: > Hi Paul, > > > On Mon, 29 May 2017 14:15:05 -0700 "Paul E. McKenney" wrote: > > > > Anyone see any other options? My preferred option would be removing pr_fmt and adding a couple new macros. ---  arch/blackfin/kernel/module.c | 39 +++++++++++++++++++++------------------  1 file changed, 21 insertions(+), 18 deletions(-) diff --git a/arch/blackfin/kernel/module.c b/arch/blackfin/kernel/module.c index 0188c933b155..e43aec7eb8d3 100644 --- a/arch/blackfin/kernel/module.c +++ b/arch/blackfin/kernel/module.c @@ -4,8 +4,6 @@   * Licensed under the GPL-2 or later   */   -#define pr_fmt(fmt) "module %s: " fmt, mod->name -  #include  #include  #include @@ -16,6 +14,11 @@  #include  #include   +#define mod_err(mod, fmt, ...) \ + pr_err("module %s: " fmt, (mod)->name, ##__VA_ARGS__) +#define mod_debug(mod, ...) \ + pr_debug("module %s: " fmt, (mod)->name, ##__VA_ARGS__) +  /* Transfer the section to the L1 memory */  int  module_frob_arch_sections(Elf_Ehdr *hdr, Elf_Shdr *sechdrs, @@ -44,7 +47,7 @@ module_frob_arch_sections(Elf_Ehdr *hdr, Elf_Shdr *sechdrs,   dest = l1_inst_sram_alloc(s->sh_size);   mod->arch.text_l1 = dest;   if (dest == NULL) { - pr_err("L1 inst memory allocation failed\n"); + mod_err(mod, "L1 inst memory allocation failed\n");   return -1;   }   dma_memcpy(dest, (void *)s->sh_addr, s->sh_size); @@ -56,7 +59,7 @@ module_frob_arch_sections(Elf_Ehdr *hdr, Elf_Shdr *sechdrs,   dest = l1_data_sram_alloc(s->sh_size);   mod->arch.data_a_l1 = dest;   if (dest == NULL) { - pr_err("L1 data memory allocation failed\n"); + mod_err(mod, "L1 data memory allocation failed\n");   return -1;   }   memcpy(dest, (void *)s->sh_addr, s->sh_size); @@ -68,7 +71,7 @@ module_frob_arch_sections(Elf_Ehdr *hdr, Elf_Shdr *sechdrs,   dest = l1_data_sram_zalloc(s->sh_size);   mod->arch.bss_a_l1 = dest;   if (dest == NULL) { - pr_err("L1 data memory allocation failed\n"); + mod_err(mod, "L1 data memory allocation failed\n");   return -1;   }   @@ -77,7 +80,7 @@ module_frob_arch_sections(Elf_Ehdr *hdr, Elf_Shdr *sechdrs,   dest = l1_data_B_sram_alloc(s->sh_size);   mod->arch.data_b_l1 = dest;   if (dest == NULL) { - pr_err("L1 data memory allocation failed\n"); + mod_err(mod, "L1 data memory allocation failed\n");   return -1;   }   memcpy(dest, (void *)s->sh_addr, s->sh_size); @@ -87,7 +90,7 @@ module_frob_arch_sections(Elf_Ehdr *hdr, Elf_Shdr *sechdrs,   dest = l1_data_B_sram_alloc(s->sh_size);   mod->arch.bss_b_l1 = dest;   if (dest == NULL) { - pr_err("L1 data memory allocation failed\n"); + mod_err(mod, "L1 data memory allocation failed\n");   return -1;   }   memset(dest, 0, s->sh_size); @@ -99,7 +102,7 @@ module_frob_arch_sections(Elf_Ehdr *hdr, Elf_Shdr *sechdrs,   dest = l2_sram_alloc(s->sh_size);   mod->arch.text_l2 = dest;   if (dest == NULL) { - pr_err("L2 SRAM allocation failed\n"); + mod_err(mod, "L2 SRAM allocation failed\n");   return -1;   }   memcpy(dest, (void *)s->sh_addr, s->sh_size); @@ -111,7 +114,7 @@ module_frob_arch_sections(Elf_Ehdr *hdr, Elf_Shdr *sechdrs,   dest = l2_sram_alloc(s->sh_size);   mod->arch.data_l2 = dest;   if (dest == NULL) { - pr_err("L2 SRAM allocation failed\n"); + mod_err(mod, "L2 SRAM allocation failed\n");   return -1;   }   memcpy(dest, (void *)s->sh_addr, s->sh_size); @@ -123,7 +126,7 @@ module_frob_arch_sections(Elf_Ehdr *hdr, Elf_Shdr *sechdrs,   dest = l2_sram_zalloc(s->sh_size);   mod->arch.bss_l2 = dest;   if (dest == NULL) { - pr_err("L2 SRAM allocation failed\n"); + mod_err(mod, "L2 SRAM allocation failed\n");   return -1;   }   @@ -157,8 +160,8 @@ apply_relocate_add(Elf_Shdr *sechdrs, const char *strtab,   Elf32_Sym *sym;   unsigned long location, value, size;   - pr_debug("applying relocate section %u to %u\n", - relsec, sechdrs[relsec].sh_info); + mod_debug(mod, "applying relocate section %u to %u\n", +   relsec, sechdrs[relsec].sh_info);     for (i = 0; i < sechdrs[relsec].sh_size / sizeof(*rel); i++) {   /* This is where to make the change */ @@ -174,14 +177,14 @@ apply_relocate_add(Elf_Shdr *sechdrs, const char *strtab,    #ifdef CONFIG_SMP   if (location >= COREB_L1_DATA_A_START) { - pr_err("cannot relocate in L1: %u (SMP kernel)\n", + mod_err(mod, "cannot relocate in L1: %u (SMP kernel)\n",   ELF32_R_TYPE(rel[i].r_info));   return -ENOEXEC;   }  #endif   - pr_debug("location is %lx, value is %lx type is %d\n", - location, value, ELF32_R_TYPE(rel[i].r_info)); + mod_debug(mod, "location is %lx, value is %lx type is %d\n", +   location, value, ELF32_R_TYPE(rel[i].r_info));     switch (ELF32_R_TYPE(rel[i].r_info)) {   @@ -200,12 +203,12 @@ apply_relocate_add(Elf_Shdr *sechdrs, const char *strtab,   case R_BFIN_PCREL12_JUMP:   case R_BFIN_PCREL12_JUMP_S:   case R_BFIN_PCREL10: - pr_err("unsupported relocation: %u (no -mlong-calls?)\n", + mod_err(mod, "unsupported relocation: %u (no -mlong-calls?)\n",   ELF32_R_TYPE(rel[i].r_info));   return -ENOEXEC;     default: - pr_err("unknown relocation: %u\n", + mod_err(mod, "unknown relocation: %u\n",   ELF32_R_TYPE(rel[i].r_info));   return -ENOEXEC;   } @@ -222,7 +225,7 @@ apply_relocate_add(Elf_Shdr *sechdrs, const char *strtab,   isram_memcpy((void *)location, &value, size);   break;   default: - pr_err("invalid relocation for %#lx\n", location); + mod_err(mod, "invalid relocation for %#lx\n", location);   return -ENOEXEC;   }   }