All of lore.kernel.org
 help / color / mirror / Atom feed
From: Patrick McHardy <kaber@trash.net>
To: Pierre Chifflier <chifflier@wzdftpd.net>
Cc: netfilter-devel@vger.kernel.org
Subject: Re: [RFH] bridge: add new target NFQUEUE for ebtables
Date: Thu, 17 Feb 2011 11:47:57 +0100	[thread overview]
Message-ID: <4D5CFCDD.10007@trash.net> (raw)
In-Reply-To: <4D5C01E5.8000302@wzdftpd.net>

Am 16.02.2011 17:57, schrieb Pierre Chifflier:
> Hi,
> 
> Thanks for your reply Patrick.
> So I did the following:
> - rebased on today's nf-next-2.6
> - apply only the first patch (which makes afinfo optional)
> - revert all other patches
> - apply the recent fix on nf_iterate since it was the cause of my oops
> 
> I patched ebtables to use xt_NFQUEUE (using a struct xt_NFQ_info_v1 with
> arguments queuenum 1 and queues_total 1), and removed any other change.
> 
> When I add a rule with the NFQUEUE target with ebtables, I almost
> immediately get a panic (full backtrace later in this mail).
> 
> What is weird is that I got a NULL skb  in ebt_in_hook (frame 2) while
> the skb was not NULL earlier - like if it was stolen by some hook. Any
> idea on what could cause that ?

The backtrace doesn't seem to be fully accurate. Please also post
the full oops output corresponding to the backtrace.

Two more questions:

- is the bridge device in promiscous mode?
- do you have IGMP snooping enabled?

  reply	other threads:[~2011-02-17 10:47 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-24 20:55 ebtables_nfqueue: missing structure afinfo Pierre Chifflier
2011-01-25 10:34 ` Patrick McHardy
2011-02-02 19:22   ` Pierre Chifflier
2011-02-02 22:59     ` Patrick McHardy
2011-02-03 14:32       ` WIP/RFC: add new module ebt_NFQUEUE for ebtables Pierre Chifflier
2011-02-03 20:19         ` Bart De Schuymer
2011-02-04 10:05         ` Pierre Chifflier
2011-02-04 13:07           ` Patrick McHardy
2011-02-04 13:20             ` Pierre Chifflier
2011-02-04 13:21               ` Patrick McHardy
2011-02-03 14:32       ` [PATCH 1/3] Make the afinfo structure optional in nf_queue and nf_reinject Pierre Chifflier
2011-02-04 13:27         ` Patrick McHardy
2011-02-04 14:15           ` Pierre Chifflier
2011-02-03 14:32       ` [PATCH 2/3] bridge: add support for the EBT_QUEUE target Pierre Chifflier
2011-02-03 14:32       ` [PATCH 3/3] bridge: add new target NFQUEUE for ebtables Pierre Chifflier
2011-02-04 13:25         ` Patrick McHardy
2011-02-04 13:40           ` Pierre Chifflier
2011-02-04 13:41             ` Patrick McHardy
     [not found]               ` <4D5104C4.3010105@edenwall.com>
     [not found]                 ` <4D59C047.5050404@trash.net>
2011-02-16 16:57                   ` [RFH] " Pierre Chifflier
2011-02-17 10:47                     ` Patrick McHardy [this message]
2011-02-17 13:37                       ` Pierre Chifflier
2011-02-18 13:42                         ` Patrick McHardy

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=4D5CFCDD.10007@trash.net \
    --to=kaber@trash.net \
    --cc=chifflier@wzdftpd.net \
    --cc=netfilter-devel@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.