netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Simon Horman <horms@verge.net.au>
To: Julius Volz <juliusv@google.com>
Cc: lvs-devel@vger.kernel.org, netdev@vger.kernel.org,
	David Miller <davem@davemloft.net>,
	Patrick McHardy <kaber@trash.net>
Subject: Re: [PATCH] IPVS: Move IPVS to net/netfilter/ipvs
Date: Wed, 1 Oct 2008 16:07:26 +1000	[thread overview]
Message-ID: <20081001060725.GA28433@verge.net.au> (raw)
In-Reply-To: <20080922000748.GE1342@verge.net.au>

On Mon, Sep 22, 2008 at 10:07:50AM +1000, Simon Horman wrote:
> On Mon, Sep 22, 2008 at 09:53:56AM +1000, Simon Horman wrote:
> > On Fri, Sep 19, 2008 at 12:45:47PM +0200, Julius Volz wrote:
> > > Hi Horms,
> > > 
> > > I worked around the unrelated UML build problems for now and did the
> > > IPVS move from net/ipv4/ipvs to net/netfilter/ipvs, as discussed.
> > > Since the patch is huge (732K), I uploaded it rather than trying to
> > > include it in the mail:
> > > 
> > > http://www.mindbasket.com/ipvs/0001-IPVS-Move-IPVS-to-net-netfilter-ipvs.patch
> > > 
> > > The menuconfig option for IPVS is now also in the Netfilter submenu.
> > > Everything builds and runs ok here...
> > 
> > Hi Julius, this seems fine to me. Dave, Patrick, are you ok with this
> > change? If so I'll suck it into lvs-2.6 and send a pull request.
> 
> Here is the version of the patch that I have that applies
> against the lvs-next-2.6 branch of lvs-2.6, changeset
> "ipvs: Fix unused label warning" (8d5803bf6fbe5264000afc8c34bff08e8ecc023b)
> 
> http://www.vergenet.net/~horms/tmp/0001-IPVS-Move-IPVS-to-net-netfilter-ipvs.2.patch

Hi Patrick,

could you take a moment to look at this?
Unfortunately the patch is too long to post to the list
but it really is as simple as the description (below) claims.

I can handle the patch longness problem by pushing it into
lvs-2.6 and getting Dave to do a pull from there. But I'd
like to make sure that the move is ok with you first.

Thanks

-- 
Simon Horman
  VA Linux Systems Japan K.K., Sydney, Australia Satellite Office
  H: www.vergenet.net/~horms/             W: www.valinux.co.jp/en

From: Julius Volz <juliusv@google.com>

IPVS: Move IPVS to net/netfilter/ipvs

Since IPVS now has partial IPv6 support, this patch moves IPVS from
net/ipv4/ipvs to net/netfilter/ipvs. It's a result of:

$ git mv net/ipv4/ipvs net/netfilter

and adapting the relevant Kconfigs/Makefiles to the new path.

Signed-off-by: Julius Volz <juliusv@google.com>
Signed-off-by: Simon Horman <horms@verge.net.au>

[ diff omitted ]

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

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-09-19 10:45 [PATCH] IPVS: Move IPVS to net/netfilter/ipvs Julius Volz
2008-09-21 23:53 ` Simon Horman
2008-09-22  0:07   ` Simon Horman
2008-10-01  6:07     ` Simon Horman [this message]
2008-10-02  9:37       ` Patrick McHardy
2008-10-02  9:59         ` 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=20081001060725.GA28433@verge.net.au \
    --to=horms@verge.net.au \
    --cc=davem@davemloft.net \
    --cc=juliusv@google.com \
    --cc=kaber@trash.net \
    --cc=lvs-devel@vger.kernel.org \
    --cc=netdev@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).