* (unknown),
@ 2012-10-25 18:29 Joseph Gasparakis
2012-10-25 18:29 ` [RFC PATCH 1/2] net: Add support for hardware-offloaded encapsulation Joseph Gasparakis
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Joseph Gasparakis @ 2012-10-25 18:29 UTC (permalink / raw)
To: davem, shemminger, chrisw; +Cc: Joseph Gasparakis, netdev
Subject: [RFC PATCH 0/2] Add support for hardware-offloaded encapsulation
This is an RFC and not intended to get merged at the moment.
The series contains updates to add in the NIC Rx and Tx checksumming support
for encapsulated packets.
The sk_buff needs to somehow have information of the inner packet, and adding
three fields for the inner mac, network and transport headers was the prefered
approach.
Not adding these fields would mean that the drivers would need to parse the
sk_buff data in hot-path, having a negative impact in the performance.
Adding in sk_buff a pointer to the skbuff of the inner packet made sense, but
would be a complicated change as assumptions needed to be made with regards to
helper functions such as skb_clone() skb_copy(). Also code for the existing
encapsulation protocols (such as VXLAN and IP GRE) had to be reworked, so the
decision was to have the simple approach of adding these three fields.
^ permalink raw reply [flat|nested] 8+ messages in thread* [RFC PATCH 1/2] net: Add support for hardware-offloaded encapsulation 2012-10-25 18:29 (unknown), Joseph Gasparakis @ 2012-10-25 18:29 ` Joseph Gasparakis 2012-10-25 21:16 ` Stephen Hemminger 2012-10-25 18:29 ` [RFC PATCH 2/2] tunneling: capture inner headers during encapsulation Joseph Gasparakis 2012-10-25 18:38 ` (unknown) David Miller 2 siblings, 1 reply; 8+ messages in thread From: Joseph Gasparakis @ 2012-10-25 18:29 UTC (permalink / raw) To: davem, shemminger, chrisw Cc: Joseph Gasparakis, netdev, Peter P Waskiewicz Jr This patch adds suport in the kernel for offloading in the NIC Tx and Rx checksuming for encapsulated packets (such as VXLAN and IP GRE) Signed-off-by: Joseph Gasparakis <joseph.gasparakis@intel.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com> --- Documentation/networking/netdev-features.txt | 10 +++ include/linux/if_ether.h | 5 ++ include/linux/ip.h | 5 ++ include/linux/netdev_features.h | 3 + include/linux/skbuff.h | 114 +++++++++++++++++++++++++++ include/linux/udp.h | 5 ++ net/core/ethtool.c | 2 + 7 files changed, 144 insertions(+) diff --git a/Documentation/networking/netdev-features.txt b/Documentation/networking/netdev-features.txt index 4164f5c..82695c0 100644 --- a/Documentation/networking/netdev-features.txt +++ b/Documentation/networking/netdev-features.txt @@ -165,3 +165,13 @@ This requests that the NIC receive all possible frames, including errored frames (such as bad FCS, etc). This can be helpful when sniffing a link with bad packets on it. Some NICs may receive more packets if also put into normal PROMISC mdoe. + +* tx-enc-checksum-offload + +This feature implies that the NIC will be able to calculate the Tx checksums +for both inner and outer packets in the case of vxlan and ipgre encapsulation. + +* rx-enc-checksum-offload + +This feature implies that the NIC will be able to verify the Rx checksums +for both inner and outer packets in the case of vxlan and ipgre encapsulation. diff --git a/include/linux/if_ether.h b/include/linux/if_ether.h index 167ce5b..b6ebb4b5 100644 --- a/include/linux/if_ether.h +++ b/include/linux/if_ether.h @@ -139,6 +139,11 @@ static inline struct ethhdr *eth_hdr(const struct sk_buff *skb) return (struct ethhdr *)skb_mac_header(skb); } +static inline struct ethhdr *eth_inner_hdr(const struct sk_buff *skb) +{ + return (struct ethhdr *)skb_inner_mac_header(skb); +} + int eth_header_parse(const struct sk_buff *skb, unsigned char *haddr); int mac_pton(const char *s, u8 *mac); diff --git a/include/linux/ip.h b/include/linux/ip.h index bd0a2a8..e94f000 100644 --- a/include/linux/ip.h +++ b/include/linux/ip.h @@ -112,6 +112,11 @@ static inline struct iphdr *ip_hdr(const struct sk_buff *skb) return (struct iphdr *)skb_network_header(skb); } +static inline struct iphdr *ip_inner_hdr(const struct sk_buff *skb) +{ + return (struct iphdr *)skb_inner_network_header(skb); +} + static inline struct iphdr *ipip_hdr(const struct sk_buff *skb) { return (struct iphdr *)skb_transport_header(skb); diff --git a/include/linux/netdev_features.h b/include/linux/netdev_features.h index 5ac3212..6dd59a5 100644 --- a/include/linux/netdev_features.h +++ b/include/linux/netdev_features.h @@ -19,6 +19,7 @@ enum { NETIF_F_IP_CSUM_BIT, /* Can checksum TCP/UDP over IPv4. */ __UNUSED_NETIF_F_1, NETIF_F_HW_CSUM_BIT, /* Can checksum all the packets. */ + NETIF_F_HW_CSUM_ENC_BIT, /* Can checksum all inner headers */ NETIF_F_IPV6_CSUM_BIT, /* Can checksum TCP/UDP over IPV6 */ NETIF_F_HIGHDMA_BIT, /* Can DMA to high memory. */ NETIF_F_FRAGLIST_BIT, /* Scatter/gather IO. */ @@ -52,6 +53,8 @@ enum { NETIF_F_NTUPLE_BIT, /* N-tuple filters supported */ NETIF_F_RXHASH_BIT, /* Receive hashing offload */ NETIF_F_RXCSUM_BIT, /* Receive checksumming offload */ + NETIF_F_RXCSUM_ENC_BIT, /* Receive checksuming offload */ + /* for encapsulation */ NETIF_F_NOCACHE_COPY_BIT, /* Use no-cache copyfromuser */ NETIF_F_LOOPBACK_BIT, /* Enable loopback */ NETIF_F_RXFCS_BIT, /* Append FCS to skb pkt data */ diff --git a/include/linux/skbuff.h b/include/linux/skbuff.h index b33a3a1..aaa17a8 100644 --- a/include/linux/skbuff.h +++ b/include/linux/skbuff.h @@ -377,6 +377,9 @@ typedef unsigned char *sk_buff_data_t; * @transport_header: Transport layer header * @network_header: Network layer header * @mac_header: Link layer header + * @inner_transport_header: Inner transport layer header (encapsulation) + * @inner_network_header: Network layer header (encapsulation) + * @inner_mac_header: Link layer header (encapsulation) * @tail: Tail pointer * @end: End pointer * @head: Head of buffer @@ -487,6 +490,9 @@ struct sk_buff { sk_buff_data_t transport_header; sk_buff_data_t network_header; sk_buff_data_t mac_header; + sk_buff_data_t inner_transport_header; + sk_buff_data_t inner_network_header; + sk_buff_data_t inner_mac_header; /* These elements must be at the end, see alloc_skb() for details. */ sk_buff_data_t tail; sk_buff_data_t end; @@ -1441,6 +1447,63 @@ static inline void skb_reset_mac_len(struct sk_buff *skb) } #ifdef NET_SKBUFF_DATA_USES_OFFSET +static inline unsigned char *skb_inner_transport_header(const struct sk_buff + *skb) +{ + return skb->head + skb->inner_transport_header; +} + +static inline void skb_reset_inner_transport_header(struct sk_buff *skb) +{ + skb->inner_transport_header = skb->data - skb->head; +} + +static inline void skb_set_inner_transport_header(struct sk_buff *skb, + const int offset) +{ + skb_reset_inner_transport_header(skb); + skb->inner_transport_header += offset; +} + +static inline unsigned char *skb_inner_network_header(const struct sk_buff *skb) +{ + return skb->head + skb->inner_network_header; +} + +static inline void skb_reset_inner_network_header(struct sk_buff *skb) +{ + skb->inner_network_header = skb->data - skb->head; +} + +static inline void skb_set_inner_network_header(struct sk_buff *skb, + const int offset) +{ + skb_reset_inner_network_header(skb); + skb->inner_network_header += offset; +} + +static inline unsigned char *skb_inner_mac_header(const struct sk_buff *skb) +{ + return skb->head + skb->inner_mac_header; +} + +static inline int skb_inner_mac_header_was_set(const struct sk_buff *skb) +{ + return skb->inner_mac_header != ~0U; +} + +static inline void skb_reset_inner_mac_header(struct sk_buff *skb) +{ + skb->inner_mac_header = skb->data - skb->head; +} + +static inline void skb_set_inner_mac_header(struct sk_buff *skb, + const int offset) +{ + skb_reset_inner_mac_header(skb); + skb->inner_mac_header += offset; +} + static inline unsigned char *skb_transport_header(const struct sk_buff *skb) { return skb->head + skb->transport_header; @@ -1496,7 +1559,58 @@ static inline void skb_set_mac_header(struct sk_buff *skb, const int offset) } #else /* NET_SKBUFF_DATA_USES_OFFSET */ +static inline unsigned char *skb_inner_transport_header(const struct sk_buff + *skb) +{ + return skb->inner_transport_header; +} + +static inline void skb_reset_inner_transport_header(struct sk_buff *skb) +{ + skb->inner_transport_header = skb->data; +} + +static inline void skb_set_inner_transport_header(struct sk_buff *skb, + const int offset) +{ + skb->inner_transport_header = skb->data + offset; +} + +static inline unsigned char *skb_inner_network_header(const struct sk_buff *skb) +{ + return skb->inner_network_header; +} + +static inline void skb_reset_inner_network_header(struct sk_buff *skb) +{ + skb->inner_network_header = skb->data; +} + +static inline void skb_set_inner_network_header(struct sk_buff *skb, + const int offset) +{ + skb->inner_network_header = skb->data + offset; +} + +static inline unsigned char *skb_inner_mac_header(const struct sk_buff *skb) +{ + return skb->inner_mac_header; +} + +static inline int skb_inner_mac_header_was_set(const struct sk_buff *skb) +{ + return skb->inner_mac_header != NULL; +} +static inline void skb_reset_mac_header(struct sk_buff *skb) +{ + skb->inner_mac_header = skb->data; +} + +static inline void skb_set_mac_header(struct sk_buff *skb, const int offset) +{ + skb->inner_mac_header = skb->data + offset; +} static inline unsigned char *skb_transport_header(const struct sk_buff *skb) { return skb->transport_header; diff --git a/include/linux/udp.h b/include/linux/udp.h index 03f72a2..e3294fd 100644 --- a/include/linux/udp.h +++ b/include/linux/udp.h @@ -45,6 +45,11 @@ static inline struct udphdr *udp_hdr(const struct sk_buff *skb) return (struct udphdr *)skb_transport_header(skb); } +static inline struct udphdr *udp_inner_hdr(const struct sk_buff *skb) +{ + return (struct udphdr *)skb_inner_transport_header(skb); +} + #define UDP_HTABLE_SIZE_MIN (CONFIG_BASE_SMALL ? 128 : 256) static inline int udp_hashfn(struct net *net, unsigned num, unsigned mask) diff --git a/net/core/ethtool.c b/net/core/ethtool.c index 4d64cc2..11f928d 100644 --- a/net/core/ethtool.c +++ b/net/core/ethtool.c @@ -58,6 +58,7 @@ static const char netdev_features_strings[NETDEV_FEATURE_COUNT][ETH_GSTRING_LEN] [NETIF_F_IP_CSUM_BIT] = "tx-checksum-ipv4", [NETIF_F_HW_CSUM_BIT] = "tx-checksum-ip-generic", [NETIF_F_IPV6_CSUM_BIT] = "tx-checksum-ipv6", + [NETIF_F_HW_CSUM_ENC_BIT] = "tx-checksum-enc-offload", [NETIF_F_HIGHDMA_BIT] = "highdma", [NETIF_F_FRAGLIST_BIT] = "tx-scatter-gather-fraglist", [NETIF_F_HW_VLAN_TX_BIT] = "tx-vlan-hw-insert", @@ -84,6 +85,7 @@ static const char netdev_features_strings[NETDEV_FEATURE_COUNT][ETH_GSTRING_LEN] [NETIF_F_NTUPLE_BIT] = "rx-ntuple-filter", [NETIF_F_RXHASH_BIT] = "rx-hashing", [NETIF_F_RXCSUM_BIT] = "rx-checksum", + [NETIF_F_RXCSUM_ENC_BIT] = "rx-enc-checksum-offload", [NETIF_F_NOCACHE_COPY_BIT] = "tx-nocache-copy", [NETIF_F_LOOPBACK_BIT] = "loopback", [NETIF_F_RXFCS_BIT] = "rx-fcs", -- 1.7.11.7 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [RFC PATCH 1/2] net: Add support for hardware-offloaded encapsulation 2012-10-25 18:29 ` [RFC PATCH 1/2] net: Add support for hardware-offloaded encapsulation Joseph Gasparakis @ 2012-10-25 21:16 ` Stephen Hemminger 2012-10-25 22:20 ` Ben Hutchings 0 siblings, 1 reply; 8+ messages in thread From: Stephen Hemminger @ 2012-10-25 21:16 UTC (permalink / raw) To: Joseph Gasparakis; +Cc: davem, chrisw, netdev, Peter P Waskiewicz Jr On Thu, 25 Oct 2012 11:29:12 -0700 Joseph Gasparakis <joseph.gasparakis@intel.com> wrote: > @@ -19,6 +19,7 @@ enum { > NETIF_F_IP_CSUM_BIT, /* Can checksum TCP/UDP over IPv4. */ > __UNUSED_NETIF_F_1, > NETIF_F_HW_CSUM_BIT, /* Can checksum all the packets. */ > + NETIF_F_HW_CSUM_ENC_BIT, /* Can checksum all inner headers */ > NETIF_F_IPV6_CSUM_BIT, /* Can checksum TCP/UDP over IPV6 */ > NETIF_F_HIGHDMA_BIT, /* Can DMA to high memory. */ > NETIF_F_FRAGLIST_BIT, /* Scatter/gather IO. */ > @@ -52,6 +53,8 @@ enum { > NETIF_F_NTUPLE_BIT, /* N-tuple filters supported */ > NETIF_F_RXHASH_BIT, /* Receive hashing offload */ > NETIF_F_RXCSUM_BIT, /* Receive checksumming offload */ > + NETIF_F_RXCSUM_ENC_BIT, /* Receive checksuming offload */ > + /* for encapsulation */ > NETIF_F_NOCACHE_COPY_BIT, /* Use no-cache copyfromuser */ > NETIF_F_LOOPBACK_BIT, /* Enable loopback */ > NETIF_F_RXFCS_BIT, /* Append FCS to skb pkt data */ Add new features at the end, or reuse __UNUSED_ bits to avoid any issues with binary compatibility. I don't think these bits are in any userspace API, maybe ethtool? ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [RFC PATCH 1/2] net: Add support for hardware-offloaded encapsulation 2012-10-25 21:16 ` Stephen Hemminger @ 2012-10-25 22:20 ` Ben Hutchings 2012-10-26 19:54 ` Gasparakis, Joseph 0 siblings, 1 reply; 8+ messages in thread From: Ben Hutchings @ 2012-10-25 22:20 UTC (permalink / raw) To: Stephen Hemminger Cc: Joseph Gasparakis, davem, chrisw, netdev, Peter P Waskiewicz Jr On Thu, 2012-10-25 at 14:16 -0700, Stephen Hemminger wrote: > On Thu, 25 Oct 2012 11:29:12 -0700 > Joseph Gasparakis <joseph.gasparakis@intel.com> wrote: > > > @@ -19,6 +19,7 @@ enum { > > NETIF_F_IP_CSUM_BIT, /* Can checksum TCP/UDP over IPv4. */ > > __UNUSED_NETIF_F_1, > > NETIF_F_HW_CSUM_BIT, /* Can checksum all the packets. */ > > + NETIF_F_HW_CSUM_ENC_BIT, /* Can checksum all inner headers */ > > NETIF_F_IPV6_CSUM_BIT, /* Can checksum TCP/UDP over IPV6 */ > > NETIF_F_HIGHDMA_BIT, /* Can DMA to high memory. */ > > NETIF_F_FRAGLIST_BIT, /* Scatter/gather IO. */ > > @@ -52,6 +53,8 @@ enum { > > NETIF_F_NTUPLE_BIT, /* N-tuple filters supported */ > > NETIF_F_RXHASH_BIT, /* Receive hashing offload */ > > NETIF_F_RXCSUM_BIT, /* Receive checksumming offload */ > > + NETIF_F_RXCSUM_ENC_BIT, /* Receive checksuming offload */ > > + /* for encapsulation */ > > NETIF_F_NOCACHE_COPY_BIT, /* Use no-cache copyfromuser */ > > NETIF_F_LOOPBACK_BIT, /* Enable loopback */ > > NETIF_F_RXFCS_BIT, /* Append FCS to skb pkt data */ > > Add new features at the end, or reuse __UNUSED_ bits to avoid any > issues with binary compatibility. I don't think these bits are in any > userspace API, maybe ethtool? There should be no binary compatibility issue here as feature bits are meant to be looked up by name at run-time. Ben. -- Ben Hutchings, Staff Engineer, Solarflare Not speaking for my employer; that's the marketing department's job. They asked us to note that Solarflare product names are trademarked. ^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [RFC PATCH 1/2] net: Add support for hardware-offloaded encapsulation 2012-10-25 22:20 ` Ben Hutchings @ 2012-10-26 19:54 ` Gasparakis, Joseph 2012-10-26 20:21 ` Stephen Hemminger 0 siblings, 1 reply; 8+ messages in thread From: Gasparakis, Joseph @ 2012-10-26 19:54 UTC (permalink / raw) To: Ben Hutchings, Stephen Hemminger Cc: davem@davemloft.net, chrisw@sous-sol.org, netdev@vger.kernel.org, Waskiewicz Jr, Peter P Stephen, are you happy with Ben's comment? If that is the case I would prefer to keep the patch as is as I believe it makes sense to group the features that way. -----Original Message----- From: Ben Hutchings [mailto:bhutchings@solarflare.com] Sent: Thursday, October 25, 2012 3:21 PM To: Stephen Hemminger Cc: Gasparakis, Joseph; davem@davemloft.net; chrisw@sous-sol.org; netdev@vger.kernel.org; Waskiewicz Jr, Peter P Subject: Re: [RFC PATCH 1/2] net: Add support for hardware-offloaded encapsulation On Thu, 2012-10-25 at 14:16 -0700, Stephen Hemminger wrote: > On Thu, 25 Oct 2012 11:29:12 -0700 > Joseph Gasparakis <joseph.gasparakis@intel.com> wrote: > > > @@ -19,6 +19,7 @@ enum { > > NETIF_F_IP_CSUM_BIT, /* Can checksum TCP/UDP over IPv4. */ > > __UNUSED_NETIF_F_1, > > NETIF_F_HW_CSUM_BIT, /* Can checksum all the packets. */ > > + NETIF_F_HW_CSUM_ENC_BIT, /* Can checksum all inner headers */ > > NETIF_F_IPV6_CSUM_BIT, /* Can checksum TCP/UDP over IPV6 */ > > NETIF_F_HIGHDMA_BIT, /* Can DMA to high memory. */ > > NETIF_F_FRAGLIST_BIT, /* Scatter/gather IO. */ > > @@ -52,6 +53,8 @@ enum { > > NETIF_F_NTUPLE_BIT, /* N-tuple filters supported */ > > NETIF_F_RXHASH_BIT, /* Receive hashing offload */ > > NETIF_F_RXCSUM_BIT, /* Receive checksumming offload */ > > + NETIF_F_RXCSUM_ENC_BIT, /* Receive checksuming offload */ > > + /* for encapsulation */ > > NETIF_F_NOCACHE_COPY_BIT, /* Use no-cache copyfromuser */ > > NETIF_F_LOOPBACK_BIT, /* Enable loopback */ > > NETIF_F_RXFCS_BIT, /* Append FCS to skb pkt data */ > > Add new features at the end, or reuse __UNUSED_ bits to avoid any > issues with binary compatibility. I don't think these bits are in any > userspace API, maybe ethtool? There should be no binary compatibility issue here as feature bits are meant to be looked up by name at run-time. Ben. -- Ben Hutchings, Staff Engineer, Solarflare Not speaking for my employer; that's the marketing department's job. They asked us to note that Solarflare product names are trademarked. ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [RFC PATCH 1/2] net: Add support for hardware-offloaded encapsulation 2012-10-26 19:54 ` Gasparakis, Joseph @ 2012-10-26 20:21 ` Stephen Hemminger 0 siblings, 0 replies; 8+ messages in thread From: Stephen Hemminger @ 2012-10-26 20:21 UTC (permalink / raw) To: Gasparakis, Joseph Cc: Ben Hutchings, davem@davemloft.net, chrisw@sous-sol.org, netdev@vger.kernel.org, Waskiewicz Jr, Peter P On Fri, 26 Oct 2012 19:54:12 +0000 "Gasparakis, Joseph" <joseph.gasparakis@intel.com> wrote: > Stephen, are you happy with Ben's comment? > If that is the case I would prefer to keep the patch as is as I believe it makes sense to group the features that way. No problem. As long as it's safe which Ben said. ^ permalink raw reply [flat|nested] 8+ messages in thread
* [RFC PATCH 2/2] tunneling: capture inner headers during encapsulation 2012-10-25 18:29 (unknown), Joseph Gasparakis 2012-10-25 18:29 ` [RFC PATCH 1/2] net: Add support for hardware-offloaded encapsulation Joseph Gasparakis @ 2012-10-25 18:29 ` Joseph Gasparakis 2012-10-25 18:38 ` (unknown) David Miller 2 siblings, 0 replies; 8+ messages in thread From: Joseph Gasparakis @ 2012-10-25 18:29 UTC (permalink / raw) To: davem, shemminger, chrisw; +Cc: Joseph Gasparakis, netdev Populating the inner header pointers of skb for vxlan and ipgre --- drivers/net/vxlan.c | 4 ++++ net/ipv4/ip_gre.c | 4 ++++ 2 files changed, 8 insertions(+) diff --git a/drivers/net/vxlan.c b/drivers/net/vxlan.c index 51de9ed..7cd1e5a 100644 --- a/drivers/net/vxlan.c +++ b/drivers/net/vxlan.c @@ -647,6 +647,10 @@ static netdev_tx_t vxlan_xmit(struct sk_buff *skb, struct net_device *dev) __u8 tos, ttl; int err; + skb->inner_mac_header = skb->mac_header; + skb->inner_network_header = skb->network_header; + skb->inner_transport_header = skb->transport_header; + /* Need space for new headers (invalidates iph ptr) */ if (skb_cow_head(skb, VXLAN_HEADROOM)) goto drop; diff --git a/net/ipv4/ip_gre.c b/net/ipv4/ip_gre.c index 7240f8e..b20ff8f 100644 --- a/net/ipv4/ip_gre.c +++ b/net/ipv4/ip_gre.c @@ -902,6 +902,10 @@ static netdev_tx_t ipgre_tunnel_xmit(struct sk_buff *skb, struct net_device *dev tunnel->err_count = 0; } + skb->inner_mac_header = skb->mac_header; + skb->inner_network_header = skb->network_header; + skb->inner_transport_header = skb->transport_header; + max_headroom = LL_RESERVED_SPACE(tdev) + gre_hlen + rt->dst.header_len; if (skb_headroom(skb) < max_headroom || skb_shared(skb)|| -- 1.7.11.7 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* (unknown) 2012-10-25 18:29 (unknown), Joseph Gasparakis 2012-10-25 18:29 ` [RFC PATCH 1/2] net: Add support for hardware-offloaded encapsulation Joseph Gasparakis 2012-10-25 18:29 ` [RFC PATCH 2/2] tunneling: capture inner headers during encapsulation Joseph Gasparakis @ 2012-10-25 18:38 ` David Miller 2 siblings, 0 replies; 8+ messages in thread From: David Miller @ 2012-10-25 18:38 UTC (permalink / raw) To: joseph.gasparakis; +Cc: shemminger, chrisw, netdev From: Joseph Gasparakis <joseph.gasparakis@intel.com> Date: Thu, 25 Oct 2012 11:29:11 -0700 > Subject: [RFC PATCH 0/2] Add support for hardware-offloaded encapsulation You messed up the subject line, putting it into the body of your email instead of amongst the headers where it belongs. A lot of mail systems are going to reject this, and with the real subject line being empty, a lot of people are going to simply delete it since the subject line is non-informative. Please repost this series with this fixed. ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2012-10-26 20:21 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-10-25 18:29 (unknown), Joseph Gasparakis 2012-10-25 18:29 ` [RFC PATCH 1/2] net: Add support for hardware-offloaded encapsulation Joseph Gasparakis 2012-10-25 21:16 ` Stephen Hemminger 2012-10-25 22:20 ` Ben Hutchings 2012-10-26 19:54 ` Gasparakis, Joseph 2012-10-26 20:21 ` Stephen Hemminger 2012-10-25 18:29 ` [RFC PATCH 2/2] tunneling: capture inner headers during encapsulation Joseph Gasparakis 2012-10-25 18:38 ` (unknown) David Miller
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).