Re: [PATCH] [2/3] Use libnl instead of iproute/AF_PACKET (nm_system_device_replace_default_ipv4_route)
- From: Dan Williams <dcbw redhat com>
- To: Benoit Boissinot <bboissin+networkmanager gmail com>
- Cc: Thomas Graf <tgraf redhat com>, networkmanager-list gnome org
- Subject: Re: [PATCH] [2/3] Use libnl instead of iproute/AF_PACKET (nm_system_device_replace_default_ipv4_route)
- Date: Wed, 23 Apr 2008 13:55:00 -0400
On Wed, 2008-04-23 at 19:26 +0200, Benoit Boissinot wrote:
> On Wed, Apr 23, 2008 at 07:01:14PM +0200, Benoit Boissinot wrote:
> > On Wed, Apr 23, 2008 at 12:23:45PM -0400, Dan Williams wrote:
> > > probably need to poke Thomas about this one, before I start
> > > instrumenting iproute2 to dump the netlink packets it sends, doing the
> > > same for libnl, and see what's different.
> >
> > I'm on it, I can reproduce it (and I've already dumped libnl vs iproute).
>
> got it, we have to set the scope to RT_SCOPE_UNIVERSE, there's an XXX in
> libnl:
> /* XXX Change to UNIVERSE if gw || nexthops */
>
> so I should add rtnl_route_set_scope(route, RT_SCOPE_UNIVERSE); to all
> patches that add a route with a gateway.
>
> Should I resend them ? (or preferably fix libnl first ?)
Lets not block on libnl though, just fix up and resend patch #2. Both
#1 and #3 work fine for me. We can poke thomas to fix libnl in
parallel, or we can come up with patches for it ourselves and push them
to him.
Thanks a lot!
Dan
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]