From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755459AbbCRJHF (ORCPT ); Wed, 18 Mar 2015 05:07:05 -0400 Received: from userp1040.oracle.com ([156.151.31.81]:21168 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753351AbbCRJHA (ORCPT ); Wed, 18 Mar 2015 05:07:00 -0400 Date: Wed, 18 Mar 2015 10:08:47 +0100 From: Quentin Casasnovas To: Quentin Casasnovas , Rusty Russell , Michal Marek Cc: lkml , Oleg Nesterov , Borislav Petkov , Linus Torvalds Subject: Re: [PATCH 4/7] modpost: factorize symbol pretty print in get_pretty_name(). Message-ID: <20150318090846.GI19131@chrystal.uk.oracle.com> References: <1426596002-26128-1-git-send-email-quentin.casasnovas@oracle.com> <1426596002-26128-5-git-send-email-quentin.casasnovas@oracle.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1426596002-26128-5-git-send-email-quentin.casasnovas@oracle.com> User-Agent: Mutt/1.5.22 (2013-10-16) X-Source-IP: aserv0022.oracle.com [141.146.126.234] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Adding Rusty and Michal to CC. On Tue, Mar 17, 2015 at 01:39:59PM +0100, Quentin Casasnovas wrote: > Signed-off-by: Quentin Casasnovas > --- > scripts/mod/modpost.c | 23 ++++++++++++----------- > 1 file changed, 12 insertions(+), 11 deletions(-) > > diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c > index 0f48f8b..c69681e 100644 > --- a/scripts/mod/modpost.c > +++ b/scripts/mod/modpost.c > @@ -1270,6 +1270,15 @@ static void print_section_list(const char * const list[20]) > fprintf(stderr, "\n"); > } > > +static inline void get_pretty_name(int is_func, const char** name, const char** name_p) > +{ > + switch (is_func) { > + case 0: *name = "variable"; *name_p = ""; break; > + case 1: *name = "function"; *name_p = "()"; break; > + default: *name = "(unknown reference)"; *name_p = ""; break; > + } > +} > + > /* > * Print a warning about a section mismatch. > * Try to find symbols near it so user can find it. > @@ -1289,21 +1298,13 @@ static void report_sec_mismatch(const char *modname, > char *prl_from; > char *prl_to; > > - switch (from_is_func) { > - case 0: from = "variable"; from_p = ""; break; > - case 1: from = "function"; from_p = "()"; break; > - default: from = "(unknown reference)"; from_p = ""; break; > - } > - switch (to_is_func) { > - case 0: to = "variable"; to_p = ""; break; > - case 1: to = "function"; to_p = "()"; break; > - default: to = "(unknown reference)"; to_p = ""; break; > - } > - > sec_mismatch_count++; > if (!sec_mismatch_verbose) > return; > > + get_pretty_name(from_is_func, &from, &from_p); > + get_pretty_name(to_is_func, &to, &to_p); > + > warn("%s(%s+0x%llx): Section mismatch in reference from the %s %s%s " > "to the %s %s:%s%s\n", > modname, fromsec, fromaddr, from, fromsym, from_p, to, tosec, > -- > 2.0.5 >