public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* Question regarding Qdisc.stats
@ 2002-08-20 11:44 Ravikiran G Thirumalai
  2002-09-15  5:46 ` Werner Almesberger
  0 siblings, 1 reply; 2+ messages in thread
From: Ravikiran G Thirumalai @ 2002-08-20 11:44 UTC (permalink / raw)
  To: linux-kernel

Where do the Qdisc.stats show up? in /proc? (Assuming CONFIG_NET_SCHED is
not defined) I can see that it is copied to the skb from 
qdisc_copy_stats with 'TCA_STATS' attribute, but where is it being 
reported? or what is it being used for ?? Specifically, does 
Qdisc.stats.drops get reported anywere?

While I am at it, I see that qdisc->enqueue routines kfree the skb with the
dev->queue_lock held (when the enqueue routine drops the packet -- 
NET_XMIT_DROP)  from dev_queue_xmit. kfree_skb can be done outside the 
lock.... right? I can make a patch for this if I am not missing something.....

TIA
Kiran

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Question regarding Qdisc.stats
  2002-08-20 11:44 Question regarding Qdisc.stats Ravikiran G Thirumalai
@ 2002-09-15  5:46 ` Werner Almesberger
  0 siblings, 0 replies; 2+ messages in thread
From: Werner Almesberger @ 2002-09-15  5:46 UTC (permalink / raw)
  To: Ravikiran G Thirumalai; +Cc: linux-kernel

Ravikiran G Thirumalai wrote:
> Where do the Qdisc.stats show up? in /proc?

No, only on (rt)netlink.

> (Assuming CONFIG_NET_SCHED is not defined)

In this case, the statistics are not usefully accessible. The
tx_dropped you get with ifconfig (or such) comes from the device
level, not the traffic control subsystem.

> qdisc_copy_stats with 'TCA_STATS' attribute, but where is it being 
> reported? or what is it being used for ?? Specifically, does 
> Qdisc.stats.drops get reported anywere?

Yep, e.g. in iproute2/tc/tc_qdisc.c:print_tcstats

> While I am at it, I see that qdisc->enqueue routines kfree the skb with the
> dev->queue_lock held (when the enqueue routine drops the packet -- 
> NET_XMIT_DROP)  from dev_queue_xmit. kfree_skb can be done outside the 
> lock.... right? I can make a patch for this if I am not missing something.....

That certainly sounds good to me.

- Werner

-- 
  _________________________________________________________________________
 / Werner Almesberger, Buenos Aires, Argentina         wa@almesberger.net /
/_http://www.almesberger.net/____________________________________________/

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2002-09-16  1:09 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2002-08-20 11:44 Question regarding Qdisc.stats Ravikiran G Thirumalai
2002-09-15  5:46 ` Werner Almesberger

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox