Re: [PATCH net v3 2/4] net: add rx_nohandler stat counter

From: Eric Dumazet
Date: Mon Feb 08 2016 - 17:57:52 EST


On Mon, 2016-02-08 at 11:38 -0800, Stephen Hemminger wrote:

> The iproute2 command can be fixed, but adding dependency on size of response
> gets gross fast. Imagine when 4 more fields get added, this doesn't scale well.

Really ? I see no problem at all doing the proper tests.

>
> Also, the definition of userspace ABI is that structures can't change.
> There are many other utilities that are not visible that may get broken.
> Traditionally Linux has guaranteed that programs will continue to work
> no matter how they were coded.

Stephen, we have been doing that for years.

Whole point of TLV is that it allows us to add new fields at the end of
the structures.

Yes, some buggy programs might need a fix.

Look at iproute2, you were the one adding in 2004 code to cope with
various tcp_info sizes.

So 12 years later, you cannot say it does not work anymore.