From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiri Pirko Subject: Re: [patch net-next 2/6] sched: move tcf_proto_destroy and tcf_destroy_chain helpers into cls_api Date: Tue, 14 Feb 2017 18:26:42 +0100 Message-ID: <20170214172642.GA21867@nanopsycho> References: <1486647540-14656-1-git-send-email-jiri@resnulli.us> <1486647540-14656-3-git-send-email-jiri@resnulli.us> <20170214172501.GA4182@bistromath.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: netdev@vger.kernel.org, davem@davemloft.net, jhs@mojatatu.com, mlxsw@mellanox.com To: Sabrina Dubroca Return-path: Received: from mail-wr0-f193.google.com ([209.85.128.193]:35503 "EHLO mail-wr0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754394AbdBNR0p (ORCPT ); Tue, 14 Feb 2017 12:26:45 -0500 Received: by mail-wr0-f193.google.com with SMTP id o16so28455588wra.2 for ; Tue, 14 Feb 2017 09:26:45 -0800 (PST) Content-Disposition: inline In-Reply-To: <20170214172501.GA4182@bistromath.localdomain> Sender: netdev-owner@vger.kernel.org List-ID: Tue, Feb 14, 2017 at 06:25:01PM CET, sd@queasysnail.net wrote: >Hi Jiri, > >2017-02-09, 14:38:56 +0100, Jiri Pirko wrote: >> From: Jiri Pirko >> >> Creation is done in this file, move destruction to be at the same place. >> >> Signed-off-by: Jiri Pirko >> --- >> include/net/pkt_cls.h | 2 ++ >> include/net/sch_generic.h | 2 -- >> net/sched/cls_api.c | 21 +++++++++++++++++++++ >> net/sched/sch_api.c | 22 ---------------------- >> net/sched/sch_atm.c | 1 + >> net/sched/sch_cbq.c | 1 + >> net/sched/sch_choke.c | 1 + >> net/sched/sch_dsmark.c | 1 + >> net/sched/sch_fq_codel.c | 1 + >> net/sched/sch_htb.c | 1 + >> net/sched/sch_ingress.c | 1 + >> net/sched/sch_multiq.c | 2 +- >> net/sched/sch_prio.c | 2 +- >> net/sched/sch_sfb.c | 1 + >> net/sched/sch_sfq.c | 1 + >> 15 files changed, 34 insertions(+), 26 deletions(-) > >This broke the build for !NET_CLS: > >net/built-in.o: In function `fq_codel_destroy': >/home/sab/linux/net-next/net/sched/sch_fq_codel.c:468: undefined reference to `tcf_destroy_chain' Will fix. Thanks.