| To: | Thomas Graf <tgraf@xxxxxxx> |
|---|---|
| Subject: | Re: RFC/PATCH capture qdisc requeue event in stats |
| From: | jamal <hadi@xxxxxxxxxx> |
| Date: | 28 Sep 2004 22:54:24 -0400 |
| Cc: | "David S. Miller" <davem@xxxxxxxxxxxxx>, netdev@xxxxxxxxxxx, shemminger@xxxxxxxx |
| In-reply-to: | <20040929003656.GX31616@xxxxxxxxxxxxxx> |
| Organization: | jamalopolous |
| References: | <1093799632.1073.410.camel@xxxxxxxxxxxxxxxx> <20040830144033.2265a6e6.davem@xxxxxxxxxx> <1093904088.1043.12.camel@xxxxxxxxxxxxxxxx> <20040830154430.769d1d59.davem@xxxxxxxxxx> <1093906592.1037.32.camel@xxxxxxxxxxxxxxxx> <20040830160052.548c4846.davem@xxxxxxxxxx> <1093916592.1037.51.camel@xxxxxxxxxxxxxxxx> <20040830191716.0d002f91.davem@xxxxxxxxxx> <1093919823.1043.80.camel@xxxxxxxxxxxxxxxx> <20040830212910.78047bcd.davem@xxxxxxxxxxxxx> <20040929003656.GX31616@xxxxxxxxxxxxxx> |
| Reply-to: | hadi@xxxxxxxxxx |
| Sender: | netdev-bounce@xxxxxxxxxxx |
On Tue, 2004-09-28 at 20:36, Thomas Graf wrote: > * David S. Miller <20040830212910.78047bcd.davem@xxxxxxxxxxxxx> 2004-08-30 > 21:29 > > Look, let's get real about this topic. We can't be breaking shit > > like this all the time. We're nearly letting it happen a lot > > lately. > > > > These data structures are user visible APIs, they are just like > > system call data structures, and if we cannot modify > > them without potentially breaking some existing application we > > cannot make that change. > > Why not do it by using nested TLVs?: > > TCA_STATS2 [ > TCA_STAT_BYTES > TCA_STAT_PACKETS > TCA_STAT_DROPS > ... > ] Refer to my earlier email; i think this is a noble approach. Lets do it on gnet stats though so we can make it more accessible. I think your granularity maybe too thin: bytes,packets, drops may need to be in the same TLV. > This way we can add as many new stats as we want without > even thinking about backward compatibility in the future. > This would also allow to implement dynamic size statistics > or introduce TCA_STAT_*_64 if ever needed. Yep. do you have cycles to run with that patch? cheers, jamal |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH] Improve behaviour of Netlink Sockets, jamal |
|---|---|
| Next by Date: | Re: [PATCH] Improve behaviour of Netlink Sockets, jamal |
| Previous by Thread: | Re: RFC/PATCH capture qdisc requeue event in stats, Thomas Graf |
| Next by Thread: | Re: RFC/PATCH capture qdisc requeue event in stats, Thomas Graf |
| Indexes: | [Date] [Thread] [Top] [All Lists] |