All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pablo Neira Ayuso <pablo@netfilter.org>
To: Simon Horman <horms@verge.net.au>
Cc: Chris J Arges <chris.j.arges@canonical.com>,
	Julian Anastasov <ja@ssi.bg>,
	lvs-devel@vger.kernel.org
Subject: Re: [PATCH net] ipvs: fix ipv6 hook registration for local replies
Date: Thu, 25 Sep 2014 18:57:57 +0200	[thread overview]
Message-ID: <20140925165757.GA13339@salvia> (raw)
In-Reply-To: <20140925002216.GB20466@verge.net.au>

On Thu, Sep 25, 2014 at 09:22:16AM +0900, Simon Horman wrote:
> [CC Pablo]
> 
> On Wed, Sep 24, 2014 at 09:55:11AM -0500, Chris J Arges wrote:
> > On 08/27/2014 08:55 PM, Simon Horman wrote:
> > > On Fri, Aug 22, 2014 at 05:53:41PM +0300, Julian Anastasov wrote:
> > >> commit fc604767613b6d2036cdc35b660bc39451040a47
> > >> ("ipvs: changes for local real server") from 2.6.37
> > >> introduced DNAT support to local real server but the
> > >> IPv6 LOCAL_OUT handler ip_vs_local_reply6() is
> > >> registered incorrectly as IPv4 hook causing any outgoing
> > >> IPv4 traffic to be dropped depending on the IP header values.
> > >>
> > >> Chris tracked down the problem to CONFIG_IP_VS_IPV6=y
> > >> Bug report: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1349768
> > >>
> > >> Reported-by: Chris J Arges <chris.j.arges@canonical.com>
> > >> Tested-by: Chris J Arges <chris.j.arges@canonical.com>
> > >> Signed-off-by: Julian Anastasov <ja@ssi.bg>
> > > 
> > > Thanks, and sorry for the delay.
> > > I have queued this up in the ipvs tree as a fix for v3.17.
> > > I also plan to ask for it to be considered for -stable.
> > > 
> > Julian,
> > Has this been queued up for netdev stable? Or should I send an email?
> > Thanks,
> 
> Pablo I'm not sure that this has been queued up for stable
> (possibly due to some error on my part). What is the best
> way forwards?

Just sent a batch with pending Netfilter/IPVS to -stable. The patch
authors has been Cc'ed. I didn't Cc'ed the mailing list to not
generate more load on redudant patches, but I can do it if you prefer
so.

Let me know if we don't get any feedback from -stable soon to get this
patches in. Thanks.

  reply	other threads:[~2014-09-25 16:57 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-22 14:53 [PATCH net] ipvs: fix ipv6 hook registration for local replies Julian Anastasov
2014-08-28  1:55 ` Simon Horman
2014-09-24 14:55   ` Chris J Arges
2014-09-25  0:22     ` Simon Horman
2014-09-25 16:57       ` Pablo Neira Ayuso [this message]
2014-09-30  5:13         ` Simon Horman

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=20140925165757.GA13339@salvia \
    --to=pablo@netfilter.org \
    --cc=chris.j.arges@canonical.com \
    --cc=horms@verge.net.au \
    --cc=ja@ssi.bg \
    --cc=lvs-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.