LVS
lvs-devel
Google
 
Web LinuxVirtualServer.org

Re: [PATCH 2/3] IPVS: make friends with nf_conntrack

To: Hannes Eder <heder@xxxxxxxxxx>
Subject: Re: [PATCH 2/3] IPVS: make friends with nf_conntrack
Cc: lvs-devel@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxxxxxx, netfilter-devel@xxxxxxxxxxxxxxx, Fabien Duchêne <mad_fab@xxxxxxxxx>, Jan Engelhardt <jengelh@xxxxxxxxxx>, Jean-Luc Fortemaison <jl.fortemaison@xxxxxxxxxxxx>, Julius Volz <julius.volz@xxxxxxxxx>, Laurent Grawet <laurent.grawet@xxxxxxxxxxxx>, Patrick McHardy <kaber@xxxxxxxxx>, Simon Horman <horms@xxxxxxxxxxxx>, Wensong Zhang <wensong@xxxxxxxxxxxx>
From: Julian Anastasov <ja@xxxxxx>
Date: Thu, 3 Sep 2009 22:50:58 +0300 (EEST)
        Hello,

On Wed, 2 Sep 2009, Hannes Eder wrote:

> Update the nf_conntrack tuple in reply direction, as we will see
> traffic from the real server (RIP) to the client (CIP).  Once this is
> done we can use netfilters SNAT in POSTROUTING, especially with
> xt_ipvs, to do source NAT, e.g.:
> 
> % iptables -t nat -A POSTROUTING -m ipvs --vaddr 192.168.100.30/32 --vport 
> 8080 \
> > -j SNAT --to-source 192.168.10.10
> 
> Signed-off-by: Hannes Eder <heder@xxxxxxxxxx>
> ---

        The following changes in ip_vs_core.c may be break normal
ip_vs_ftp users. Somehow you decided that this POST_ROUTING code is not
needed and deleted it. This code should be present by default.

        From http://www.ssi.bg/~ja/LVS.txt:

===
        Now after  many changes in  latest kernels  I'm not sure
        what  happens if  netfilter sees  IPVS traffic  in POST_ROUTING.
        Such  change require  testing of  ip_vs_ftp in  both passive and
        active  LVS-NAT mode,  with different length  of IP address:port
        representation  in FTP  commands, to check  if resulting packets
        survive double NAT when payload size is changed.  It is the best
        test  for  IPVS to  see  if netfilter  additionally  changes FTP
        packets  leading to  wrong payload.     
===

        So, you have to check the ip_vs_ftp case because double
NAT for IPs and Ports usually works but double changing of SEQs
and payload may be not.

        You can also check NFCT for IPVS (http://www.ssi.bg/~ja/nfct/)
for using netfilter functions and structures (ip_vs_nfct.c)

        most recent rediff:
http://www.ssi.bg/~ja/nfct/ipvs-nfct-2.6.28-1.diff

> diff --git a/net/netfilter/ipvs/ip_vs_core.c b/net/netfilter/ipvs/ip_vs_core.c
> index b227750..27bd002 100644
> --- a/net/netfilter/ipvs/ip_vs_core.c
> +++ b/net/netfilter/ipvs/ip_vs_core.c
> @@ -521,26 +521,6 @@ int ip_vs_leave(struct ip_vs_service *svc, struct 
> sk_buff *skb,
>       return NF_DROP;
>  }
>  
> -
> -/*
> - *      It is hooked before NF_IP_PRI_NAT_SRC at the NF_INET_POST_ROUTING
> - *      chain, and is used for VS/NAT.
> - *      It detects packets for VS/NAT connections and sends the packets
> - *      immediately. This can avoid that iptable_nat mangles the packets
> - *      for VS/NAT.
> - */
> -static unsigned int ip_vs_post_routing(unsigned int hooknum,
> -                                    struct sk_buff *skb,
> -                                    const struct net_device *in,
> -                                    const struct net_device *out,
> -                                    int (*okfn)(struct sk_buff *))
> -{
> -     if (!skb->ipvs_property)
> -             return NF_ACCEPT;
> -     /* The packet was sent from IPVS, exit this chain */
> -     return NF_STOP;
> -}
> -
>  __sum16 ip_vs_checksum_complete(struct sk_buff *skb, int offset)
>  {
>       return csum_fold(skb_checksum(skb, offset, skb->len - offset, 0));
> @@ -1431,14 +1411,6 @@ static struct nf_hook_ops ip_vs_ops[] __read_mostly = {
>               .hooknum        = NF_INET_FORWARD,
>               .priority       = 99,
>       },
> -     /* Before the netfilter connection tracking, exit from POST_ROUTING */
> -     {
> -             .hook           = ip_vs_post_routing,
> -             .owner          = THIS_MODULE,
> -             .pf             = PF_INET,
> -             .hooknum        = NF_INET_POST_ROUTING,
> -             .priority       = NF_IP_PRI_NAT_SRC-1,
> -     },
>  #ifdef CONFIG_IP_VS_IPV6
>       /* After packet filtering, forward packet through VS/DR, VS/TUN,
>        * or VS/NAT(change destination), so that filtering rules can be
> @@ -1467,14 +1439,6 @@ static struct nf_hook_ops ip_vs_ops[] __read_mostly = {
>               .hooknum        = NF_INET_FORWARD,
>               .priority       = 99,
>       },
> -     /* Before the netfilter connection tracking, exit from POST_ROUTING */
> -     {
> -             .hook           = ip_vs_post_routing,
> -             .owner          = THIS_MODULE,
> -             .pf             = PF_INET6,
> -             .hooknum        = NF_INET_POST_ROUTING,
> -             .priority       = NF_IP6_PRI_NAT_SRC-1,
> -     },
>  #endif
>  };

Regards

--
Julian Anastasov <ja@xxxxxx>
--
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>