New flags for Fibre Channel over Ethernet (FCoE)

Hi,
I am working on the Fibre Channel over Ethernet (FCoE) at open-fcoe.org and would like to add the following new define/feature flogs in netdev for FCoE.

The first patch adds the protocol type as ETH_P_FCOE for FCoE protocol and the second one adds offload feature flags for FCoE in netdev, which is based on the earlier post: http://marc.info/?l=linux-netdev&m=122228979113312&w=2.

Thanks.

yi
---

Chris Leech (1):
net: define feature flags for FCoE offloads

Yi Zou (1):
net: add ETH_P_FCOE for Fibre Channel over Ethernet (FCoE)

 include/linux/if_ether.h  |    1 +
 include/linux/netdevice.h |    3 +++
 include/linux/skbuff.h    |    2 ++
 3 files changed, 6 insertions(+), 0 deletions(-)

Signature : Yi Zou <[email protected]>

---

 include/linux/if_ether.h |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

index 723a1c5..e225b41 100644
@@ -77,6 +77,7 @@
 #define ETH_P_PAE	0x888E		/* Port Access Entity (IEEE 802.1X) */
 #define ETH_P_AOE	0x88A2		/* ATA over Ethernet		*/
 #define ETH_P_TIPC	0x88CA		/* TIPC 			*/
+#define ETH_P_FCOE      0x8906          /* Fibre Channel over Ethernet  */
 
 /*
  *	Non DIX types. Won't clash for 1500 types.

trivial -- but the patch format makes it pretty obvious -- it would probably be nice to use tabs to align things on the FCOE line the same way the other lines in the file do, rather than using spaces.

*/

Ethernet

Thanks, will fix this and resubmit.

yi

From: Chris Leech <[email protected]>

---

 include/linux/netdevice.h |    3 +++
 include/linux/skbuff.h    |    2 ++
 2 files changed, 5 insertions(+), 0 deletions(-)

index 9cfd20b..ef65927 100644
@@ -522,6 +522,8 @@ struct net_device
 #define NETIF_F_NETNS_LOCAL	8192	/* Does not change network namespaces */
 #define NETIF_F_LRO		32768	/* large receive offload */
 
+#define NETIF_F_FCOE_CRC	(1 << 24) /* FCoE CRC32 */
+
 	/* Segmentation offload features */
 #define NETIF_F_GSO_SHIFT	16
 #define NETIF_F_GSO_MASK	0xffff0000

@@ -530,6 +532,7 @@ struct net_device
 #define NETIF_F_GSO_ROBUST	(SKB_GSO_DODGY << NETIF_F_GSO_SHIFT)
 #define NETIF_F_TSO_ECN		(SKB_GSO_TCP_ECN << NETIF_F_GSO_SHIFT)
 #define NETIF_F_TSO6		(SKB_GSO_TCPV6 << NETIF_F_GSO_SHIFT)
+#define NETIF_F_GSO_FCOE	(SKB_GSO_FCOE << NETIF_F_GSO_SHIFT)
 
 	/* List of features with software fallbacks. */
 #define NETIF_F_GSO_SOFTWARE	(NETIF_F_TSO | NETIF_F_TSO_ECN | NETIF_F_TSO6)

index 720b688..be60560 100644
@@ -188,6 +188,8 @@ enum {
 	SKB_GSO_TCP_ECN = 1 << 3,
 
 	SKB_GSO_TCPV6 = 1 << 4,
+
+	SKB_GSO_FCOE = 1 << 5,
 };
 
 #if BITS_PER_LONG > 32

I apologize for this patch as one patch to claim the a few bits back from SKB_GSO_MASK was missing here in the original series. I shall resubmit the complete series to have all patches included, plus fixing the style issue in the first patch.

Thanks.

yi