LVS
lvs-devel
Google
 
Web LinuxVirtualServer.org

Re: [bug] FWMARKs and persistence in IPVS: The Use of Unions

To: Jan Engelhardt <jengelh@xxxxxxxxxx>
Subject: Re: [bug] FWMARKs and persistence in IPVS: The Use of Unions
Cc: netfilter-devel <netfilter-devel@xxxxxxxxxxxxxxx>, lvs-devel@xxxxxxxxxxxxxxx, Fabien Duchêne <fabien.duchene@xxxxxxxxxxxxxxxxxxxx>, Joseph Mack NA3T <jmack@xxxxxxxx>, Julius Volz <julius.volz@xxxxxxxxx>
From: Simon Horman <horms@xxxxxxxxxxxx>
Date: Tue, 28 Apr 2009 19:23:55 +1000
On Tue, Apr 28, 2009 at 11:07:40AM +0200, Jan Engelhardt wrote:
> 
> On Tuesday 2009-04-28 10:15, Simon Horman wrote:
> >
> >It seems to me that it should be easy enough to fix by changing
> >fwmark in ip_vs_sched_persist() from:
> >
> >union nf_inet_addr fwmark = {
> >     .all = { 0, 0, 0, htonl(svc->fwmark) }
> >};
> >
> >to:
> >
> >union nf_inet_addr fwmark = {
> >     .all = { htonl(svc->fwmark), 0, 0, 0 }
> >};
> >
> >Assuming that this would result in fwmark->ip being set to
> >htonl(svc->fwmark), which is relevant if svc->af is AF_INET - that is,
> >for IPv4.[...]
> >An alternate idea would be to change the af value used for fwmarks,
> >but this seems to be even less clean than the current (slightly broken)
> >technique of using nf_inet_addr for IPv4 or IPv6 addresses, or fwmarks.
> 
> If you use ->all, then using NFPROTO_UNSPEC as af
> seems to me like a good match.

That seems reasonable, though ip_vs_ct_in_get() would still
need to use the real af for the cp->af == af and
ip_vs_addr_equal(af, s_addr, &cp->caddr) portinos of the check.

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

--
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>