Message ID | 20171013195813.25321-1-xiyou.wangcong@gmail.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
Series | [net-next] net_sched: fix a compile warning in act_ife | expand |
Cong Wang <xiyou.wangcong@gmail.com> writes: > Apparently ife_meta_id2name() is only called when > CONFIG_MODULES is defined. > > This fixes: > > net/sched/act_ife.c:251:20: warning: ‘ife_meta_id2name’ defined but not used [-Wunused-function] > static const char *ife_meta_id2name(u32 metaid) > ^~~~~~~~~~~~~~~~ Fair enough, thanks Cong! > Fixes: d3f24ba895f0 ("net sched actions: fix module auto-loading") > Cc: Roman Mashak <mrv@mojatatu.com> > Signed-off-by: Cong Wang <xiyou.wangcong@gmail.com> > --- > net/sched/act_ife.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/net/sched/act_ife.c b/net/sched/act_ife.c > index 252ee7d8c731..3007cb1310ea 100644 > --- a/net/sched/act_ife.c > +++ b/net/sched/act_ife.c > @@ -248,6 +248,7 @@ static int ife_validate_metatype(struct tcf_meta_ops *ops, void *val, int len) > return ret; > } > > +#ifdef CONFIG_MODULES > static const char *ife_meta_id2name(u32 metaid) > { > switch (metaid) { > @@ -261,6 +262,7 @@ static const char *ife_meta_id2name(u32 metaid) > return "unknown"; > } > } > +#endif > > /* called when adding new meta information > * under ife->tcf_lock for existing action
From: Cong Wang <xiyou.wangcong@gmail.com> Date: Fri, 13 Oct 2017 12:58:13 -0700 > Apparently ife_meta_id2name() is only called when > CONFIG_MODULES is defined. > > This fixes: > > net/sched/act_ife.c:251:20: warning: ‘ife_meta_id2name’ defined but not used [-Wunused-function] > static const char *ife_meta_id2name(u32 metaid) > ^~~~~~~~~~~~~~~~ > > Fixes: d3f24ba895f0 ("net sched actions: fix module auto-loading") > Cc: Roman Mashak <mrv@mojatatu.com> > Signed-off-by: Cong Wang <xiyou.wangcong@gmail.com> Applied.
diff --git a/net/sched/act_ife.c b/net/sched/act_ife.c index 252ee7d8c731..3007cb1310ea 100644 --- a/net/sched/act_ife.c +++ b/net/sched/act_ife.c @@ -248,6 +248,7 @@ static int ife_validate_metatype(struct tcf_meta_ops *ops, void *val, int len) return ret; } +#ifdef CONFIG_MODULES static const char *ife_meta_id2name(u32 metaid) { switch (metaid) { @@ -261,6 +262,7 @@ static const char *ife_meta_id2name(u32 metaid) return "unknown"; } } +#endif /* called when adding new meta information * under ife->tcf_lock for existing action
Apparently ife_meta_id2name() is only called when CONFIG_MODULES is defined. This fixes: net/sched/act_ife.c:251:20: warning: ‘ife_meta_id2name’ defined but not used [-Wunused-function] static const char *ife_meta_id2name(u32 metaid) ^~~~~~~~~~~~~~~~ Fixes: d3f24ba895f0 ("net sched actions: fix module auto-loading") Cc: Roman Mashak <mrv@mojatatu.com> Signed-off-by: Cong Wang <xiyou.wangcong@gmail.com> --- net/sched/act_ife.c | 2 ++ 1 file changed, 2 insertions(+)