On 6/13/15, 11:04 AM, sfel...@gmail.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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to