Re: [ANNOUNCE] iproute2-2.6.23-rc3

2007-09-01 Thread Stephen Hemminger
On Sat, 01 Sep 2007 09:12:41 +0200
Patrick McHardy <[EMAIL PROTECTED]> wrote:

> Pavel Emelyanov wrote:
> > Patrick McHardy wrote:
> >> On Fri, 31 Aug 2007, Pavel Emelyanov wrote:
> >>
>  Why does this add a new ip subcommand instead and uses genetlink
>  instead of the rtnl_link API, which was introduced exactly for
>  this stuff?
> 
> >>> Hm... It does not, it just adds a parser for veth commands that
> >>> generic layer isn't aware of.
> >>
> >> It does, you have "ip veth add", "ip veth del", ...
> >>
> >> Didn't you already sent a patch that used "ip link add ..."?
> > 
> > The patch I sent last used "ip link add" and "ip link del".
> > The link_veth.c file only parses all the extra arguments.
> 
> 
> Thats what I thought. I seems Stephen applied an old version of
> the patch that still uses genetlink.
> 

Send me a new one, and I'll revert the old one.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [ANNOUNCE] iproute2-2.6.23-rc3

2007-09-01 Thread Patrick McHardy

Pavel Emelyanov wrote:

Patrick McHardy wrote:

On Fri, 31 Aug 2007, Pavel Emelyanov wrote:


Why does this add a new ip subcommand instead and uses genetlink
instead of the rtnl_link API, which was introduced exactly for
this stuff?


Hm... It does not, it just adds a parser for veth commands that
generic layer isn't aware of.


It does, you have "ip veth add", "ip veth del", ...

Didn't you already sent a patch that used "ip link add ..."?


The patch I sent last used "ip link add" and "ip link del".
The link_veth.c file only parses all the extra arguments.



Thats what I thought. I seems Stephen applied an old version of
the patch that still uses genetlink.

-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [ANNOUNCE] iproute2-2.6.23-rc3

2007-08-31 Thread Pavel Emelyanov
Patrick McHardy wrote:
> On Fri, 31 Aug 2007, Pavel Emelyanov wrote:
> 
>> Patrick McHardy wrote:
>>> On Wed, 22 Aug 2007, Stephen Hemminger wrote:
>>>
 There have been a lot of changes for 2.6.23, so here is a test release
 of iproute2 that should capture all the submitted patches

 Pavel Emelianov (1):
  Make ip utility veth driver aware
>>>
>>> Why does this add a new ip subcommand instead and uses genetlink
>>> instead of the rtnl_link API, which was introduced exactly for
>>> this stuff?
>>>
>> Hm... It does not, it just adds a parser for veth commands that
>> generic layer isn't aware of.
> 
> 
> It does, you have "ip veth add", "ip veth del", ...
> 
> Didn't you already sent a patch that used "ip link add ..."?

The patch I sent last used "ip link add" and "ip link del".
The link_veth.c file only parses all the extra arguments.

> In any case this should be replaced by proper use of the rtnl_link
> stuff.
> 

-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [ANNOUNCE] iproute2-2.6.23-rc3

2007-08-31 Thread Patrick McHardy

On Fri, 31 Aug 2007, Pavel Emelyanov wrote:


Patrick McHardy wrote:

On Wed, 22 Aug 2007, Stephen Hemminger wrote:


There have been a lot of changes for 2.6.23, so here is a test release
of iproute2 that should capture all the submitted patches

Pavel Emelianov (1):
 Make ip utility veth driver aware



Why does this add a new ip subcommand instead and uses genetlink
instead of the rtnl_link API, which was introduced exactly for
this stuff?



Hm... It does not, it just adds a parser for veth commands that
generic layer isn't aware of.



It does, you have "ip veth add", "ip veth del", ...

Didn't you already sent a patch that used "ip link add ..."?

In any case this should be replaced by proper use of the rtnl_link
stuff.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [ANNOUNCE] iproute2-2.6.23-rc3

2007-08-31 Thread Pavel Emelyanov
Patrick McHardy wrote:
> On Wed, 22 Aug 2007, Stephen Hemminger wrote:
> 
>> There have been a lot of changes for 2.6.23, so here is a test release
>> of iproute2 that should capture all the submitted patches
>>
>> Pavel Emelianov (1):
>>  Make ip utility veth driver aware
> 
> 
> Why does this add a new ip subcommand instead and uses genetlink
> instead of the rtnl_link API, which was introduced exactly for
> this stuff?
> 

Hm... It does not, it just adds a parser for veth commands that
generic layer isn't aware of.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [ANNOUNCE] iproute2-2.6.23-rc3

2007-08-31 Thread Patrick McHardy

On Wed, 22 Aug 2007, Stephen Hemminger wrote:


There have been a lot of changes for 2.6.23, so here is a test release
of iproute2 that should capture all the submitted patches

Pavel Emelianov (1):
 Make ip utility veth driver aware



Why does this add a new ip subcommand instead and uses genetlink
instead of the rtnl_link API, which was introduced exactly for
this stuff?
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [ANNOUNCE] iproute2-2.6.23-rc3

2007-08-31 Thread Patrick McHardy

On Wed, 22 Aug 2007, Stephen Hemminger wrote:


There have been a lot of changes for 2.6.23, so here is a test release
of iproute2 that should capture all the submitted patches

Patrick McHardy (6):
 TC action parsing bug fix
 Bug fix tc action drop
 IPROUTE2: RTNETLINK nested attributes
 Use FRA_* attributes for routing rules
 iplink: use netlink for link configuration
 Fix meta ematch usage of 0 values



I just noticed the iplink_vlan stuff is missing. Do you want me
to resend?

-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [ANNOUNCE] iproute2-2.6.23-rc3

2007-08-27 Thread Jarek Poplawski
On Fri, Aug 24, 2007 at 12:26:28PM -0700, Stephen Hemminger wrote:
> On Fri, 24 Aug 2007 12:10:44 +0200
> Jarek Poplawski <[EMAIL PROTECTED]> wrote:
> 
> > On 22-08-2007 20:08, Stephen Hemminger wrote:
> > > There have been a lot of changes for 2.6.23, so here is a test release
> > > of iproute2 that should capture all the submitted patches
> > > 
> > > 
> > > http://developer.osdl.org/shemminger/iproute2/download/iproute2-2.6.23-rc3.tar.gz
> > 
> > But... isn't it forged, btw?!
> 
> No, I just didn't sign a temporary testing version.  A final version
> will be out after 2.6.23

So, I'm calmer now... On the other hand with kernel testing versions
there seems to be more afraid of?

Jarek P.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [ANNOUNCE] iproute2-2.6.23-rc3

2007-08-24 Thread Stephen Hemminger
On Fri, 24 Aug 2007 12:10:44 +0200
Jarek Poplawski <[EMAIL PROTECTED]> wrote:

> On 22-08-2007 20:08, Stephen Hemminger wrote:
> > There have been a lot of changes for 2.6.23, so here is a test release
> > of iproute2 that should capture all the submitted patches
> > 
> > 
> > http://developer.osdl.org/shemminger/iproute2/download/iproute2-2.6.23-rc3.tar.gz
> 
> But... isn't it forged, btw?!

No, I just didn't sign a temporary testing version.  A final version
will be out after 2.6.23

-- 
Stephen Hemminger <[EMAIL PROTECTED]>
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [ANNOUNCE] iproute2-2.6.23-rc3

2007-08-24 Thread Jarek Poplawski
On 22-08-2007 20:08, Stephen Hemminger wrote:
> There have been a lot of changes for 2.6.23, so here is a test release
> of iproute2 that should capture all the submitted patches
> 
> 
> http://developer.osdl.org/shemminger/iproute2/download/iproute2-2.6.23-rc3.tar.gz

But... isn't it forged, btw?!

Cheers,
Jarek P.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[ANNOUNCE] iproute2-2.6.23-rc3

2007-08-22 Thread Stephen Hemminger
There have been a lot of changes for 2.6.23, so here is a test release
of iproute2 that should capture all the submitted patches


http://developer.osdl.org/shemminger/iproute2/download/iproute2-2.6.23-rc3.tar.gz

Johannes Berg (1):
  show multicast groups

PJ Waskiewicz (1):
  iproute2: sch_rr support in tc

Patrick McHardy (6):
  TC action parsing bug fix
  Bug fix tc action drop
  IPROUTE2: RTNETLINK nested attributes
  Use FRA_* attributes for routing rules
  iplink: use netlink for link configuration
  Fix meta ematch usage of 0 values

Pavel Emelianov (1):
  Make ip utility veth driver aware

Sridhar Samudrala (1):
  Fix bug  in display of ipv6 cloned/cached routes

Stephen Hemminger (3):
  Fix ss to handle partial records.
  sanitized headers update to 2.6.23-rc3
  Fix m_ipt build

-- 
Stephen Hemminger <[EMAIL PROTECTED]>
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html