netdev - Re: [PATCH v5 net-next 02/14] net: define gso types for IPx over IPv4 and IPv6

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 16 May 2016 11:31:08 -0700
From:	Alexander Duyck <alexander.duyck@...il.com>
To:	Tom Herbert <tom@...bertland.com>
Cc:	David Miller <davem@...emloft.net>,
	Netdev <netdev@...r.kernel.org>, Kernel Team <kernel-team@...com>
Subject: Re: [PATCH v5 net-next 02/14] net: define gso types for IPx over IPv4
 and IPv6

On Mon, May 16, 2016 at 11:28 AM, Tom Herbert <tom@...bertland.com> wrote:
> On Mon, May 16, 2016 at 11:13 AM, Alexander Duyck
> <alexander.duyck@...il.com> wrote:
>> On Mon, May 16, 2016 at 11:07 AM, Tom Herbert <tom@...bertland.com> wrote:
>>> On Mon, May 16, 2016 at 9:32 AM, Alexander Duyck
>>> <alexander.duyck@...il.com> wrote:
>>>> On Sun, May 15, 2016 at 4:42 PM, Tom Herbert <tom@...bertland.com> wrote:
>>>>> This patch defines two new GSO definitions SKB_GSO_IPXIP4 and
>>>>> SKB_GSO_IPXIP6 along with corresponding NETIF_F_GSO_IPXIP4 and
>>>>> NETIF_F_GSO_IPXIP6. These are used to described IP in IP
>>>>> tunnel and what the outer protocol is. The inner protocol
>>>>> can be deduced from other GSO types (e.g. SKB_GSO_TCPV4 and
>>>>> SKB_GSO_TCPV6). The GSO types of SKB_GSO_IPIP and SKB_GSO_SIT
>>>>> are removed (these are both instances of SKB_GSO_IPXIP4).
>>>>> SKB_GSO_IPXIP6 will be used when support for GSO with IP
>>>>> encapsulation over IPv6 is added.
>>>>>
>>>>> Signed-off-by: Tom Herbert <tom@...bertland.com>
>>>>> ---
>>>>>  drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c  |  5 ++---
>>>>>  drivers/net/ethernet/broadcom/bnxt/bnxt.c         |  4 ++--
>>>>>  drivers/net/ethernet/intel/i40e/i40e_main.c       |  3 +--
>>>>>  drivers/net/ethernet/intel/i40e/i40e_txrx.c       |  3 +--
>>>>>  drivers/net/ethernet/intel/i40evf/i40e_txrx.c     |  3 +--
>>>>>  drivers/net/ethernet/intel/i40evf/i40evf_main.c   |  3 +--
>>>>>  drivers/net/ethernet/intel/igb/igb_main.c         |  3 +--
>>>>>  drivers/net/ethernet/intel/ixgbe/ixgbe_main.c     |  3 +--
>>>>>  drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c |  3 +--
>>>>>  include/linux/netdev_features.h                   | 12 ++++++------
>>>>>  include/linux/netdevice.h                         |  4 ++--
>>>>>  include/linux/skbuff.h                            |  4 ++--
>>>>>  net/core/ethtool.c                                |  4 ++--
>>>>>  net/ipv4/af_inet.c                                |  2 +-
>>>>>  net/ipv4/ipip.c                                   |  2 +-
>>>>>  net/ipv6/ip6_offload.c                            |  4 ++--
>>>>>  net/ipv6/sit.c                                    |  4 ++--
>>>>>  net/netfilter/ipvs/ip_vs_xmit.c                   | 17 +++++++----------
>>>>>  18 files changed, 36 insertions(+), 47 deletions(-)
>>>>
>>>> It looks like you missed drivers/net/ethernet/intel/igb/netdev.c.  If
>>>> you don't get it then it will break the build.
>>>>
>>>
>>> I don't see file that in betdev branch, maybe it's new?
>>
>> Nope, it has been there for a while.  It got patched to support IPIP
>> and SIT tunnels in the same patch that updated igb/igb_main.c.
>>
>
> Looks like it's
>
> drivers/net/ethernet/intel/igbvf/netdev.c
>
>> I am also looking into the other patches now.  It looks like something
>> broke hardware offloads again as I am only getting .9 Mb/s for IPv6
>> based GRE tunnels with your patches applied.  I'm trying to bisect it
>> now.
>>
> What hardware are you using?

I'm using an Intel X710, it is an i40e based NIC.

- Alex