From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jiri Pirko <jiri@resnulli.us>
Subject: Re: [patch net-next v10 02/13] net: sched: introduce shared filter
 blocks infrastructure
Date: Wed, 17 Jan 2018 09:59:50 +0100
Message-ID: <20180117085950.GA2114@nanopsycho>
References: <20180116153316.4751-1-jiri@resnulli.us>
 <20180116153316.4751-3-jiri@resnulli.us>
 <CAM_iQpV8Z_-8Tj0TJTi5QOhjtdngx0JtgArhGiCx_Se2v9CRGA@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: Linux Kernel Network Developers <netdev@vger.kernel.org>,
        David Miller <davem@davemloft.net>,
        Jamal Hadi Salim <jhs@mojatatu.com>, mlxsw@mellanox.com,
        andrew@lunn.ch,
        Vivien Didelot <vivien.didelot@savoirfairelinux.com>,
        Florian Fainelli <f.fainelli@gmail.com>,
        michael.chan@broadcom.com, ganeshgr@chelsio.com,
        Saeed Mahameed <saeedm@mellanox.com>, matanb@mellanox.com,
        leonro@mellanox.com, Ido Schimmel <idosch@mellanox.com>,
        Jakub Kicinski <jakub.kicinski@netronome.com>,
        Simon Horman <simon.horman@netronome.com>,
        Pieter Jansen van Vuuren
        <pieter.jansenvanvuuren@netronome.com>, john.hurley@netronome.com,
        Alexander Duyck <alexander.h.duyck@intel.com>,
        Or Gerlitz <ogerlitz@mellanox.com>,
        John Fastabend <john.fastabend@gmail.com>,
        Daniel Borkmann <daniel@iogearbo
To: Cong Wang <xiyou.wangcong@gmail.com>
Return-path: <netdev-owner@vger.kernel.org>
Received: from mail-wm0-f65.google.com ([74.125.82.65]:38321 "EHLO
        mail-wm0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1750861AbeAQI7w (ORCPT
        <rfc822;netdev@vger.kernel.org>); Wed, 17 Jan 2018 03:59:52 -0500
Received: by mail-wm0-f65.google.com with SMTP id 141so13936028wme.3
        for <netdev@vger.kernel.org>; Wed, 17 Jan 2018 00:59:52 -0800 (PST)
Content-Disposition: inline
In-Reply-To: <CAM_iQpV8Z_-8Tj0TJTi5QOhjtdngx0JtgArhGiCx_Se2v9CRGA@mail.gmail.com>
Sender: netdev-owner@vger.kernel.org
List-ID: <netdev.vger.kernel.org>

Wed, Jan 17, 2018 at 01:03:15AM CET, xiyou.wangcong@gmail.com wrote:
>On Tue, Jan 16, 2018 at 7:33 AM, Jiri Pirko <jiri@resnulli.us> wrote:
>>  static int __init tc_filter_init(void)
>>  {
>> +       int err;
>> +
>>         tc_filter_wq = alloc_ordered_workqueue("tc_filter_workqueue", 0);
>>         if (!tc_filter_wq)
>>                 return -ENOMEM;
>>
>> +       err = register_pernet_subsys(&tcf_net_ops);
>> +       if (err)
>> +               return err;
>
>Need to destroy the above workqueue on error.

Will fix. Thanks.