| To: | jamal <hadi@xxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 5/5] PKT_SCHED: Qdisc are not supposed to dump TCA_STATS themselves |
| From: | Thomas Graf <tgraf@xxxxxxx> |
| Date: | Sat, 9 Oct 2004 18:56:25 +0200 |
| Cc: | "David S. Miller" <davem@xxxxxxxxxxxxx>, netdev@xxxxxxxxxxx |
| In-reply-to: | <1097340107.1051.178.camel@xxxxxxxxxxxxxxxx> |
| References: | <20041007010146.GA18621@xxxxxxxxxxxxxx> <20041007010819.GF18621@xxxxxxxxxxxxxx> <1097336911.1051.149.camel@xxxxxxxxxxxxxxxx> <20041009162602.GA21977@xxxxxxxxxxxxxx> <1097340107.1051.178.camel@xxxxxxxxxxxxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
> The action code change is trivial since the dump_stats exists. Located > in cls_api.c in call to copy_stats. > My recomendation is to to not even bother using old API since this is > new code. Go ahead and rip it out and just send TC_STAT2 _only_ > Sorry, dont have the energy to do it right now. OK, will do, doesn't take too long I guess. > Youve heard of the TheLinuxWAY(tm) I hope ;-> Otherwise know as > cutnpaste. One has bug others follow ;-> I know, I was more amazed by how long a slab corruption can exist without anyone noticing ;) > BTW, you would be submitting the iproute2 patches as well, correct ? > i.e you are not just using your tool to do the testing? Sure, once the kernel part is done. I use iproute2 to check if the backward compatibility is working and my tool in debug mode printing out the hierarchy of the TLVs to see if the new stuff is all right. Adding support to iproute will not take longer than 1-2 hours. |
| Previous by Date: | Re: [PATCH 5/5] PKT_SCHED: Qdisc are not supposed to dump TCA_STATS themselves, jamal |
|---|---|
| Next by Date: | Re: [PATCH] pktgen: sprintf arg type warning, Sam Leffler |
| Previous by Thread: | Re: [PATCH 5/5] PKT_SCHED: Qdisc are not supposed to dump TCA_STATS themselves, jamal |
| Next by Thread: | Re: [PATCH 0/5] PKT_SCHED: Convert Qdiscs to use generic network statistics/estimator, David S. Miller |
| Indexes: | [Date] [Thread] [Top] [All Lists] |