All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jiri Slaby <jslaby@suse.cz>
To: Florian Westphal <fw@strlen.de>,
	stable@vger.kernel.org, ML netdev <netdev@vger.kernel.org>
Subject: Re: Linux 3.4.92 breaks forwarding with netfilter enabled
Date: Wed, 11 Jun 2014 09:07:01 +0200	[thread overview]
Message-ID: <53980015.5010409@suse.cz> (raw)
In-Reply-To: <20140610220325.GG1970@breakpoint.cc>

On 06/11/2014 12:03 AM, Florian Westphal wrote:
> [ stable specific regression.  I wanted to CC stable@ on my reply below
>   but I fat-fingered the address... ]
> 
>> On Tue, Jun 10, 2014 at 12:43 PM, Egerváry Gergely <gergely@egervary.hu> wrote:
>>> Hi,
>>>
>>> we have just upgraded our systems from 3.4.91 (longterm) to 3.4.92.
>>> Since then we are experiencing dozens of MTU-related network timeout
>>> issues. Reverting back to 3.4.91 fixes all of these problems.
>>>
>>> Both kernel versions are built from vanilla sources with the same
>>> .config. These can be highly reproduced over VPN tunnels or even over
>>> simple ethernet connections when using DNAT. (TCP port forward) It
>>> looks like MTU path discovery is somehow affected.
>>>
>>> ICMP is not filtered in our network. (For testing, we flushed all
>>> iptables rules, set all policies to ACCEPT, and problem still exists.)
>>> We do not use any special sysctl settings. Was there any changes
>>> related to packet forwarding or MTU discovery?
> 
> These were in earlier releases than 3.4.92.
> 
>> some combination of old kernel and backport of ("net: ipv4:
>> ip_forward: fix inverted local_df test") ?
>> just guessing
> 
> Excellent guess.  Yes, this is the culprit.
> 
> Quoting that patch changelog:
> 
> 'This wasn't noticed earlier [..] because netfilter ip defrag did not set local_df
> until couple of days ago.'
> 
> -stable lacks commit 895162b1101b3ea5db08ca6822ae9672717efec0, so
> netfilter never sets ->local_df.  Thus all defragmented packets
> that go over the mtu are refused to be forwarded.
> 
> Gergely, please either revert
> 
> commit bd91cb56f951a7b0da8c3098ea9cd56854ece66c
> Author: Florian Westphal <fw@strlen.de>
> net: ipv4: ip_forward: fix inverted local_df test
> [ Upstream commit ca6c5d4ad216d5942ae544bbf02503041bd802aa ]
> 
> Or, alternatively, also apply upstream commit 895162b1101b3ea5db08ca6822ae9672717efec0
> Author: Florian Westphal <fw@strlen.de>
> Date:   Fri May 2 15:32:16 2014 +0200
> netfilter: ipv4: defrag: set local_df flag on defragmented skb
> 
> on top of your 3.4.92 kernel.
> 
> (The latter might be a better option since it will also fix the
>  long-standing issue where netfilter sends bogus frag-needed message
>  under very rare circumstances).

I believe, the same holds for 3.12, right? (I have just applied "defrag:
set local_df flag on defragmented skb".)

thanks,
-- 
js
suse labs

       reply	other threads:[~2014-06-11  7:07 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20140610220325.GG1970@breakpoint.cc>
2014-06-11  7:07 ` Jiri Slaby [this message]
2014-06-11  8:10   ` Linux 3.4.92 breaks forwarding with netfilter enabled Florian Westphal
2014-06-12 17:25     ` Guillaume Morin
2014-06-12 17:54       ` Greg KH

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=53980015.5010409@suse.cz \
    --to=jslaby@suse.cz \
    --cc=fw@strlen.de \
    --cc=netdev@vger.kernel.org \
    --cc=stable@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.