Message ID | 201010191141.22496.hans.schillstrom@ericsson.com |
---|---|
State | Not Applicable, archived |
Delegated to: | David Miller |
Headers | show |
Hi Simon I'm sorry it's not a bug it works perfectly OK. (I didn't see that it was a pointer, It was another bug in my test code...) so forget about the last mail On Tuesday 19 October 2010 11:41:21 Hans Schillstrom wrote: > Hi Simon, > There is a show stopper for IPv6 in this patch. > ip_vs_conn_fill_param() doesn't handle IPv6. > > On Monday 04 October 2010 15:46:28 Simon Horman wrote: > > Signed-off-by: Simon Horman <horms@verge.net.au> > > Acked-by: Julian Anastasov <ja@ssi.bg> > > > > --- > > > > The motivation for this is to allow persistence engine modules to > > fill in the parameters. -- Regards Hans Schillstrom <hans.schillstrom@ericsson.com> -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tue, Oct 19, 2010 at 11:56:43AM +0200, Hans Schillstrom wrote: > Hi Simon > I'm sorry it's not a bug it works perfectly OK. > (I didn't see that it was a pointer, It was another bug in my test code...) > so forget about the last mail No problem, thanks for looking over things :-) -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
--- a/include/net/ip_vs.h +++ b/include/net/ip_vs.h @@ -670,9 +670,19 @@ static inline void ip_vs_conn_fill_param(int af, int protocol, { p->af = af; p->protocol = protocol; - p->caddr = caddr; p->cport = cport; - p->vaddr = vaddr; +#ifdef CONFIG_IP_VS_IPV6 + if (af == AF_INET6) { + ipv6_addr_copy(&p->caddr, caddr); + ipv6_addr_copy(&p->caddr, caddr); + } + else +#else + } + p->caddr = caddr; + p->vaddr = vaddr; + } +#endif p->vport = vport; p->pe = NULL; p->pe_data = NULL;