Re: [RFC] [PATCH] Don't run __qdisc_run() on a stopped TX queue
From: Jarek Poplawski <hidden>
Date: 2009-07-28 09:02:35
From: Jarek Poplawski <hidden>
Date: 2009-07-28 09:02:35
On Tue, Jul 28, 2009 at 04:44:51PM +0800, Herbert Xu wrote:
On Tue, Jul 28, 2009 at 08:37:13AM +0000, Jarek Poplawski wrote:quoted
Hmm.. would something IMHO_more_readable/less_fixing like: (qdisc->flags & TCQ_F_DEFAULT) be too much? (Btw, IIRC there was an idea long time ago to treat all queues equally.)Well the fact that as it stands the only true multiqueue qdisc is the default is a mere coincidence. There is no fundamental reason why non-default qdiscs cannot be made multiqueue aware, well, for some of them anyway. For example, sfq can naturally be enhanced as a multiqueue qdisc just like the default qdisc. So I don't think we want to prejudice what a future non-default qdisc may support.
If it's only about the name I'm OK with: (qdisc->flags & TCQ_F_DEFAULT_WITHOUT_PREJUDICE_MAYBE_NON_DEFAULT_IN_THE_FUTURE) or any other, too ;-) Cheers, Jarek P.