netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Julian Anastasov <ja@ssi.bg>
To: Hannes Frederic Sowa <hannes@stressinduktion.org>
Cc: "Simon Horman" <horms@verge.net.au>,
	"YOSHIFUJI Hideaki / 吉藤英明" <yoshfuji@linux-ipv6.org>,
	lvs-devel@vger.kernel.org, netdev@vger.kernel.org,
	"Mark Brooks" <mark@loadbalancer.org>
Subject: Re: [RFC net-next] ipv6: Use destination address determined by IPVS
Date: Thu, 17 Oct 2013 01:50:38 +0300 (EEST)	[thread overview]
Message-ID: <alpine.LFD.2.03.1310170115320.1917@ssi.bg> (raw)
In-Reply-To: <20131016215953.GD18135@order.stressinduktion.org>


	Hello,

On Wed, 16 Oct 2013, Hannes Frederic Sowa wrote:

> To have ip6_dst_check working, there must to be a valid link from
> the rt6_info to the fib6_node. Otherwise we cannot check the serial
> number. As I currently see we also need a link from the fib6_node down
> to the dst entry for resource management. Thus we would have to insert
> the special dst-entry with RTF_GATEWAY and non-null rt6i_gateway back
> into the fib and have it globally visible. This could have unforseen
> side effects. We still cache all dst entries in the fib. One think I
> foresee as a possible problem is the automatic aggregation of ECMP routes,
> too.

	I'm still not sure what is needed. Looking at
ip6_pol_route(), I see that everything should just work: for
routes without RTF_GATEWAY flag we return cloned route
from rt6_alloc_cow() with valid rt6i_gateway. I still
didn't tested the problem myself, so I'll stop with the
comments before that.

> We should provide something similar to what IPv4 does with the
> KNOWN_NH flag. I guess my idea with exchanging rt6i_dst as nexthop would
> solve this without too much hassle but this would have to be checked by
> implementing it.
> 
> I don't think that storing a changed nexthop in the ipv6 cb is that nice and
> maintainable.

	Agreed. I'm not going to test that change. I'll test
latest net-next without any changes to see where exactly is
the failure because the IPv6 routing seems capable for what
we need. I have problems with my setup, so I'll need some
days. As Simon is also testing the problem, he can find the
reason before me.

Regards

--
Julian Anastasov <ja@ssi.bg>

  reply	other threads:[~2013-10-16 22:50 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-16  0:02 [RFC net-next] ipv6: Use destination address determined by IPVS Simon Horman
2013-10-16  0:13 ` Eric Dumazet
2013-10-16  0:28   ` Simon Horman
2013-10-16  0:39     ` Eric Dumazet
2013-10-16  2:13       ` Simon Horman
2013-10-16  2:40         ` Eric Dumazet
2013-10-16  4:26           ` Simon Horman
2013-10-16  0:53 ` Hannes Frederic Sowa
2013-10-16  2:14   ` Simon Horman
2013-10-16  7:27 ` Julian Anastasov
2013-10-16 14:32   ` Hannes Frederic Sowa
2013-10-16 20:22     ` Julian Anastasov
2013-10-16 21:59       ` Hannes Frederic Sowa
2013-10-16 22:50         ` Julian Anastasov [this message]
2013-10-17 23:03 ` Julian Anastasov
2013-10-18  2:10   ` Hannes Frederic Sowa
2013-10-18  6:33     ` Julian Anastasov
2013-10-18 16:33       ` Hannes Frederic Sowa
2013-10-19 16:37     ` Julian Anastasov
2013-10-19 18:34       ` Hannes Frederic Sowa
2013-10-19 22:36         ` Hannes Frederic Sowa
2013-10-19 22:39           ` Hannes Frederic Sowa
2013-10-20  7:11           ` Julian Anastasov
2013-10-20  7:33             ` Hannes Frederic Sowa
2013-10-20  8:33               ` Julian Anastasov
2013-10-20 12:27               ` Julian Anastasov
2013-10-20  6:39         ` Julian Anastasov
2013-10-20  6:47           ` Hannes Frederic Sowa

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=alpine.LFD.2.03.1310170115320.1917@ssi.bg \
    --to=ja@ssi.bg \
    --cc=hannes@stressinduktion.org \
    --cc=horms@verge.net.au \
    --cc=lvs-devel@vger.kernel.org \
    --cc=mark@loadbalancer.org \
    --cc=netdev@vger.kernel.org \
    --cc=yoshfuji@linux-ipv6.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).