All of lore.kernel.org
 help / color / mirror / Atom feed
From: Simon Horman <horms@verge.net.au>
To: Hans Schillstrom <hans.schillstrom@ericsson.com>
Cc: Julian Anastasov <ja@ssi.bg>,
	LVS-Devel <lvs-devel@vger.kernel.org>,
	Daniel Lezcano <daniel.lezcano@free.fr>
Subject: Re: [*v4 PATCH 0/3] IPVS: Backup Adding Ipv6 and Persistence support
Date: Tue, 16 Nov 2010 15:44:39 +0900	[thread overview]
Message-ID: <20101116064436.GA19642@verge.net.au> (raw)
In-Reply-To: <201011151931.26789.hans.schillstrom@ericsson.com>

On Mon, Nov 15, 2010 at 07:31:26PM +0100, Hans Schillstrom wrote:
> This patch series adds/(updates) the following functionality
> in the synchronization between master and backup daemons.
> 
>  - IPv6
>  - Persistence Engine
>  - Firewall marks transferred
>  - Time-outs transferred.
>  - Flag field increased to 32 bits.
> 
> Note:
> It depends on Patch 1-9 in staging branch.
> "IPVS: Backup, Prepare for transferring firewall marks (fwmark) to the backup daemon."
> to ...
> "IPVS: skb defrag in L7 helpers"

Hi Hans,

unfortuantely I am a bit confused about which tree your patches apply to.

I have sent a pull request to Patrick just now and as part of that
I have updated the master, for-patrick, and stable branches such
that they are currently all the same thing. This means stable has
been purged and then re-added. In the process I dropped the following
patches:

* IPVS: ip_vs_pe.c, use strncmp to be safe.

  Julian didn't seem very happy with this change.
  He suggested using pe->name_len to be safe.

* IPVS: Prepare for transferring firewall marks (fwmark) to the backup daemon.

  I believe you have revised this idea

* IPVS: Backup, Adding structs for new sync format

  I believe this one has changed too.

I notice that Julian made some comments too.  When you are ready could you
please post a fresh patch series, that includes all required patches and
applies cleanly on top of the master branch of lvs-next-2.6?


  parent reply	other threads:[~2010-11-16  6:44 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-15 18:31 [*v4 PATCH 0/3] IPVS: Backup Adding Ipv6 and Persistence support Hans Schillstrom
2010-11-15 22:29 ` Julian Anastasov
2010-11-16 14:51   ` Hans Schillstrom
2010-11-16 21:15     ` Julian Anastasov
2010-11-16  6:44 ` Simon Horman [this message]
2010-11-16  7:22   ` Hans Schillstrom
2010-11-16  8:08     ` 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=20101116064436.GA19642@verge.net.au \
    --to=horms@verge.net.au \
    --cc=daniel.lezcano@free.fr \
    --cc=hans.schillstrom@ericsson.com \
    --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.