Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751270AbdFBMH1 (ORCPT ); Fri, 2 Jun 2017 08:07:27 -0400 Received: from mail-wm0-f67.google.com ([74.125.82.67]:32889 "EHLO mail-wm0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751147AbdFBMH0 (ORCPT ); Fri, 2 Jun 2017 08:07:26 -0400 From: Corentin Labbe To: jeyu@redhat.com, rusty@rustcorp.com.au Cc: linux-kernel@vger.kernel.org, Corentin Labbe Subject: [PATCH] kernel/module.c: fix warning about unused nowarn variable Date: Fri, 2 Jun 2017 14:05:11 +0200 Message-Id: <20170602120511.23112-1-clabbe.montjoie@gmail.com> X-Mailer: git-send-email 2.13.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1597 Lines: 59 This patch fix the following warning: kernel/module.c: In function 'add_usage_links': kernel/module.c:1653:6: warning: variable 'nowarn' set but not used [-Wunused-but-set-variable] int nowarn; Signed-off-by: Corentin Labbe --- kernel/module.c | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/kernel/module.c b/kernel/module.c index e3e9dbba6a5b..aed902bcb4aa 100644 --- a/kernel/module.c +++ b/kernel/module.c @@ -1663,19 +1663,20 @@ static inline void remove_notes_attrs(struct module *mod) } #endif /* CONFIG_KALLSYMS */ -static void add_usage_links(struct module *mod) +static int add_usage_links(struct module *mod) { + int ret = 0; #ifdef CONFIG_MODULE_UNLOAD struct module_use *use; - int nowarn; mutex_lock(&module_mutex); list_for_each_entry(use, &mod->target_list, target_list) { - nowarn = sysfs_create_link(use->target->holders_dir, + ret = sysfs_create_link(use->target->holders_dir, &mod->mkobj.kobj, mod->name); } mutex_unlock(&module_mutex); #endif + return ret; } static void del_usage_links(struct module *mod) @@ -1798,13 +1799,17 @@ static int mod_sysfs_setup(struct module *mod, if (err) goto out_unreg_param; - add_usage_links(mod); + err = add_usage_links(mod); + if (err) + goto out_unreg_usage_links; add_sect_attrs(mod, info); add_notes_attrs(mod, info); kobject_uevent(&mod->mkobj.kobj, KOBJ_ADD); return 0; +out_unreg_usage_links: + module_remove_modinfo_attrs(mod); out_unreg_param: module_param_sysfs_remove(mod); out_unreg_holders: -- 2.13.0