From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiri Pirko Subject: Re: [RFC net-next] net: sch_clsact: add support for global per-netns classifier mode Date: Wed, 6 Sep 2017 09:24:13 +0200 Message-ID: <20170906072413.GC2523@nanopsycho> References: <1504615701-20912-1-git-send-email-nikolay@cumulusnetworks.com> <59AF291E.90508@iogearbox.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Daniel Borkmann , Cong Wang , Nikolay Aleksandrov , Linux Kernel Network Developers , David Ahern , Jamal Hadi Salim To: Roopa Prabhu Return-path: Received: from mail-wr0-f195.google.com ([209.85.128.195]:35841 "EHLO mail-wr0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750819AbdIFHYQ (ORCPT ); Wed, 6 Sep 2017 03:24:16 -0400 Received: by mail-wr0-f195.google.com with SMTP id g50so203941wra.3 for ; Wed, 06 Sep 2017 00:24:16 -0700 (PDT) Content-Disposition: inline In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: Wed, Sep 06, 2017 at 06:04:17AM CEST, roopa@cumulusnetworks.com wrote: >On Tue, Sep 5, 2017 at 3:45 PM, Daniel Borkmann wrote: >> On 09/06/2017 12:01 AM, Roopa Prabhu wrote: >>> >>> On Tue, Sep 5, 2017 at 11:18 AM, Cong Wang >>> wrote: >>>> >>>> On Tue, Sep 5, 2017 at 5:48 AM, Nikolay Aleksandrov >>>> wrote: >>>>> >>>>> Hi all, >>>>> This RFC adds a new mode for clsact which designates a device's egress >>>>> classifier as global per netns. The packets that are not classified for >>>>> a particular device will be classified using the global classifier. >>>>> We have needed a global classifier for some time now for various >>>>> purposes and setting the single bridge or loopback/vrf device as the >> >> >> Can you elaborate a bit more on the ... "we have needed a global >> classifier for some time now for various purposes". > >Most of our acl's are global or use a wildcard. eg iptables supports >global rules without an dev. We do end up having hundreds of netdevs. >Another use case for the future is use of tc for policy based routing >which requires global rules. That is not how TC works. There are devices, qdiscs, blocks, chains. The global approach does not fit. The block sharing gets you what you need, without need for any ugly hack.