From mboxrd@z Thu Jan  1 00:00:00 1970
From: Simon Horman <simon.horman@netronome.com>
Subject: Re: [patch net-next 1/9] net_sch: red: Add offload ability to RED
 qdisc
Date: Tue, 7 Nov 2017 10:39:45 +0900
Message-ID: <20171107013943.qqac47yodas2n7qo@netronome.com>
References: <20171106062349.4873-1-jiri@resnulli.us>
 <20171106062349.4873-2-jiri@resnulli.us>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: netdev@vger.kernel.org, davem@davemloft.net, nogahf@mellanox.com,
        jhs@mojatatu.com, xiyou.wangcong@gmail.com, mlxsw@mellanox.com,
        andrew@lunn.ch, vivien.didelot@savoirfairelinux.com,
        f.fainelli@gmail.com, michael.chan@broadcom.com,
        ganeshgr@chelsio.com, saeedm@mellanox.com, matanb@mellanox.com,
        leonro@mellanox.com, idosch@mellanox.com,
        jakub.kicinski@netronome.com, pieter.jansenvanvuuren@netronome.com,
        john.hurley@netronome.com, alexander.h.duyck@intel.com,
        ogerlitz@mellanox.com, john.fastabend@gmail.com
To: Jiri Pirko <jiri@resnulli.us>
Return-path: <netdev-owner@vger.kernel.org>
Received: from mail-wm0-f44.google.com ([74.125.82.44]:45041 "EHLO
        mail-wm0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1751910AbdKGBj7 (ORCPT
        <rfc822;netdev@vger.kernel.org>); Mon, 6 Nov 2017 20:39:59 -0500
Received: by mail-wm0-f44.google.com with SMTP id n74so12596619wmi.1
        for <netdev@vger.kernel.org>; Mon, 06 Nov 2017 17:39:59 -0800 (PST)
Content-Disposition: inline
In-Reply-To: <20171106062349.4873-2-jiri@resnulli.us>
Sender: netdev-owner@vger.kernel.org
List-ID: <netdev.vger.kernel.org>

On Mon, Nov 06, 2017 at 07:23:41AM +0100, Jiri Pirko wrote:
> From: Nogah Frankel <nogahf@mellanox.com>
> 
> Add the ability to offload RED qdisc by using ndo_setup_tc.
> There are four commands for RED offloading:
> * TC_RED_SET: handles set and change.
> * TC_RED_DESTROY: handle qdisc destroy.
> * TC_RED_STATS: update the qdiscs counters (given as reference)
> * TC_RED_XSTAT: returns red xstats.
> 
> Whether RED is being offloaded is being determined every time dump action
> is being called because parent change of this qdisc could change its
> offload state but doesn't require any RED function to be called.
> 
> Signed-off-by: Nogah Frankel <nogahf@mellanox.com>
> Signed-off-by: Jiri Pirko <jiri@mellanox.com>

Reviewed-by: Simon Horman <simon.horman@netronome.com>