Cong Wang <xiyou.wangc...@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 <m...@mojatatu.com> > Signed-off-by: Cong Wang <xiyou.wangc...@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