linux-kernel - Re: [PATCH net-next] Driver: Vmxnet3: Copy TCP header to mapped frame for IPv6 packets

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <54F24ACC.7020108@cogentembedded.com>
Date:	Sun, 01 Mar 2015 02:10:04 +0300
From:	Sergei Shtylyov <sergei.shtylyov@...entembedded.com>
To:	Shrikrishna Khare <skhare@...are.com>, sbhatewara@...are.com,
	pv-drivers@...are.com, netdev@...r.kernel.org,
	linux-kernel@...r.kernel.org
CC:	Amitabha Banerjee <banerjeea@...are.com>
Subject: Re: [PATCH net-next] Driver: Vmxnet3: Copy TCP header to mapped frame
 for IPv6 packets

Hello.

On 02/28/2015 10:58 PM, Shrikrishna Khare wrote:

> Allows for packet parsing to be done by the fast path. This performance
> optimization already exists for IPv4. Add similar logic for IPv6.

> Signed-off-by: Amitabha Banerjee <banerjeea@...are.com>
> Signed-off-by: Shrikrishna Khare <skhare@...are.com>
> ---
>   drivers/net/vmxnet3/vmxnet3_drv.c | 26 ++++++++++++++++----------
>   drivers/net/vmxnet3/vmxnet3_int.h |  5 +++--
>   2 files changed, 19 insertions(+), 12 deletions(-)

> diff --git a/drivers/net/vmxnet3/vmxnet3_drv.c b/drivers/net/vmxnet3/vmxnet3_drv.c
> index 294214c..9216e6a 100644
> --- a/drivers/net/vmxnet3/vmxnet3_drv.c
> +++ b/drivers/net/vmxnet3/vmxnet3_drv.c
[...]
> @@ -831,16 +832,20 @@ vmxnet3_parse_and_copy_hdr(struct sk_buff *skb, struct vmxnet3_tx_queue *tq,
>   			if (ctx->ipv4) {
>   				const struct iphdr *iph = ip_hdr(skb);
>
> -				if (iph->protocol == IPPROTO_TCP)
> -					ctx->l4_hdr_size = tcp_hdrlen(skb);
> -				else if (iph->protocol == IPPROTO_UDP)
> -					ctx->l4_hdr_size = sizeof(struct udphdr);
> -				else
> -					ctx->l4_hdr_size = 0;
> -			} else {
> -				/* for simplicity, don't copy L4 headers */
> -				ctx->l4_hdr_size = 0;
> +				protocol = iph->protocol;
> +			} else if (ctx->ipv6) {
> +				const struct ipv6hdr *ipv6h = ipv6_hdr(skb);
> +
> +				protocol = ipv6h->nexthdr;
>   			}
> +
> +			if (protocol == IPPROTO_TCP)
> +				ctx->l4_hdr_size = tcp_hdrlen(skb);
> +			else if (protocol == IPPROTO_UDP)
> +				ctx->l4_hdr_size = sizeof(struct udphdr);
> +			else
> +				ctx->l4_hdr_size = 0;
> +

    I think the above is asking to be a 'switch (protocol)' instead.

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/