All of lore.kernel.org
 help / color / mirror / Atom feed
From: Patrick McHardy <kaber@trash.net>
To: "richardvoigt@gmail.com" <richardvoigt@gmail.com>
Cc: bridge@linuxfoundation.org, Stephen Hemminger <shemminger@vyatta.com>
Subject: Re: [Bridge] 802.1q packets
Date: Tue, 01 Jul 2008 17:10:08 +0200	[thread overview]
Message-ID: <486A48D0.8040102@trash.net> (raw)
In-Reply-To: <2e59e6970806301542t405646das78baa26f0cda8f6e@mail.gmail.com>

richardvoigt@gmail.com wrote:
> On Mon, Jun 30, 2008 at 5:07 PM, Fulvio Ricciardi <
> fulvio.ricciardi@zeroshell.net> wrote:
> 
>>> That mostly rules out other devices in the path as the
>>> cause of the problem.  There's just one chance of a
>>> netfilter interaction that I can think of: netfilter may
>>> cause fragments to be recombined, without netfilter the
>>> fragments could be bridged.  Are you running the ping
>>> command from the bridge itself, or across the bridge? (I
>>> presume across the bridge because you are discussing the
>>> FORWARD chain only)
>> I ping across the bridge. If instead a ping from the bridge
>> itself, all works right.
>>
>>> Do the large ping requests show up in the iptables
>>> counters?
>> Yes, in any case (either ping -s 1472 and ping -s 1473) the
>> packets are counted in the FORWARD chain.
>>
>>> What happens if you set no fragmentation when you run
>>> ping?
>> it's the same
> 
> 
> Just to verify, you mean that with no fragmentation, large pings go through
> if and only if bridge-nf-call-iptables is disabled?


Just FYI for all affected, I'm looking into this. One
problem is that only packets with skb->protocol == ETH_P_IP
are refragmented, but not ETH_P_8021Q. That change alone
doesn't fix it though, still trying to track it down.


  reply	other threads:[~2008-07-01 15:10 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-06-30 17:07 [Bridge] 802.1q packets Fulvio Ricciardi
2008-06-30 22:42 ` richardvoigt
2008-07-01 15:10   ` Patrick McHardy [this message]
2009-03-17 14:58     ` Saikiran Madugula
2009-03-17 16:26       ` Saikiran Madugula
  -- strict thread matches above, loose matches on Subject: below --
2008-07-22 17:40 Fulvio Ricciardi
2008-07-22 19:35 ` Adam Osuchowski
2008-07-22 21:59   ` Stephen Hemminger
2008-07-23  3:51     ` richardvoigt
2008-09-20 20:47     ` Adam Osuchowski
2008-06-30 16:58 Fulvio Ricciardi
2008-06-30  5:53 Leigh Sharpe
2008-06-28 21:17 Fulvio Ricciardi
2008-06-29  0:08 ` richardvoigt
2008-06-28 19:50 Fulvio Ricciardi
2008-06-28 21:06 ` richardvoigt
2008-06-28 12:07 Fulvio Ricciardi
2008-06-28 18:31 ` Stephen Hemminger
2008-06-28  4:56 Fulvio Ricciardi
2008-07-22 11:09 ` Adam Osuchowski

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=486A48D0.8040102@trash.net \
    --to=kaber@trash.net \
    --cc=bridge@linuxfoundation.org \
    --cc=richardvoigt@gmail.com \
    --cc=shemminger@vyatta.com \
    /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.