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  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]
Date:	Wed, 29 Aug 2012 10:02:37 +0200 (CEST)
From:	Jesper Dangaard Brouer <>
To:	Patrick McHardy <>
Cc:	Pablo Neira Ayuso <>,
	Netfilter Developers <>,
	netdev <>
Subject: Re: [PATCH 02/19] Cleaning up the IPv6 MTU checking in the IPVS xmit
 code, by using a common helper function __mtu_check_toobig_v6().

Just a little nitpick.

The original title/subj was:
  "ipvs: IPv6 MTU checking cleanup and bugfix"

And the curr/used title/subj were part of the commit text.

On Tue, 28 Aug 2012, Patrick McHardy wrote:

> From: Jesper Dangaard Brouer <>
  Cleaning up the IPv6 MTU checking in the IPVS xmit code, by using a
  common helper function __mtu_check_toobig_v6().

> The MTU check for tunnel mode can also use this helper as
> ntohs(old_iph->payload_len) + sizeof(struct ipv6hdr) is qual to
> skb->len.  And the 'mtu' variable have been adjusted before
> calling helper.
> Notice, this also fixes a bug, as the the MTU check in ip_vs_dr_xmit_v6()
> were missing a check for skb_is_gso().
> This bug e.g. caused issues for KVM IPVS setups, where different
> Segmentation Offloading techniques are utilized, between guests,
> via the virtio driver.  This resulted in very bad performance,
> due to the ICMPv6 "too big" messages didn't affect the sender.
> Signed-off-by: Jesper Dangaard Brouer <>
> Signed-off-by: Patrick McHardy <>
> ---
> net/netfilter/ipvs/ip_vs_xmit.c |   21 +++++++++++++++------
> 1 files changed, 15 insertions(+), 6 deletions(-)


   Jesper Brouer

MSc. Master of Computer Science
Dept. of Computer Science, University of Copenhagen
Author of
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to
More majordomo info at

Powered by blists - more mailing lists