[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <557EDF61.8010709@cumulusnetworks.com>
Date: Mon, 15 Jun 2015 07:21:21 -0700
From: roopa <roopa@...ulusnetworks.com>
To: sfeldma@...il.com
CC: netdev@...r.kernel.org, jiri@...nulli.us,
simon.horman@...ronome.com, ronen.arad@...el.com,
john.r.fastabend@...el.com, andrew@...n.ch, f.fainelli@...il.com,
linux@...ck-us.net, davidch@...adcom.com,
stephen@...workplumber.org
Subject: Re: [RFC PATCH net-next 1/4] net: don't reforward packets already
forwarded by offload device
On 6/13/15, 11:04 AM, sfeldma@...il.com wrote:
> diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h
> index 6f5f71f..181b08f 100644
> --- a/include/linux/netdevice.h
> +++ b/include/linux/netdevice.h
> @@ -1444,6 +1444,8 @@ enum netdev_priv_flags {
> *
> * @xps_maps: XXX: need comments on this one
> *
> + * @fwd_mark: Offload device fwding mark
> + *
> * @trans_start: Time (in jiffies) of last Tx
> * @watchdog_timeo: Represents the timeout that is used by
> * the watchdog ( see dev_watchdog() )
> @@ -1681,6 +1683,10 @@ struct net_device {
> struct xps_dev_maps __rcu *xps_maps;
> #endif
>
> +#ifdef CONFIG_NET_SWITCHDEV
> + u32 fwd_mark;
> +#endif
> +
>
scott, Just calling out a few other use-cases to make sure this flag can
be used in a generic way in the future:
- avoid updating acl counters in the kernel when hardware acl counters
have already accounted for this packet
- ability to conditionally clear the mark by other protocols in the
kernel before xmit: one use case is the bridge driver allowing software
forwarding of some protocol packets when hardware forwarding of these is
turned off eg igmp (As I understand, This is possible with your patch
series if we need to in the future)
PS: Maybe the name should be generic enough to suite other use-cases in
the future (i like the name i had hw_offloaded better ;).
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists