LVS
lvs-devel
Google
 
Web LinuxVirtualServer.org

Re: [PATCH] ipvs: make "no destination available" message more informati

To: Patrick Schaaf <netdev@xxxxxx>
Subject: Re: [PATCH] ipvs: make "no destination available" message more informative
Cc: lvs-devel@xxxxxxxxxxxxxxx
From: Simon Horman <horms@xxxxxxxxxxxx>
Date: Wed, 16 Feb 2011 14:52:46 +0900
On Wed, Feb 16, 2011 at 06:12:47AM +0100, Patrick Schaaf wrote:
> Hi Simon,
> 
> On Wed, 2011-02-16 at 07:31 +0900, Simon Horman wrote: 
> > > Thanks, this looks good to me.
> > > I will apply it with the minor modification noted below.
> >
> > I have slightly modified the IPv4/IPv6 logic in ip_vs_scheduler_err()
> > and tested the code with both IPv4 and IPv6. It seems to work well :-)
> >
> > I also did some indentation reformatting of ip_vs_scheduler_err()
> > while I was there.
> 
> Thank you for taking your time and looking at it. I'm fine with your
> changes. Signoff follows.

Thanks.

> The first hunk of your patch deletes an empty line in ip_vs.h that is
> unrelated to the other changes. Fine with me too, just thought I'd note
> it.

Ooops, I'll remove that hunk.
--
To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

<Prev in Thread] Current Thread [Next in Thread>