Re: [PATCH take 2] pkt_sched: Fix qdisc_watchdog() vs. dev_deactivate() race
From: Jarek Poplawski <hidden>
Date: 2008-09-15 06:08:04
From: Jarek Poplawski <hidden>
Date: 2008-09-15 06:08:04
On Sun, Sep 14, 2008 at 03:13:41PM -0700, Herbert Xu wrote:
On Sun, Sep 14, 2008 at 11:43:31PM +0200, Jarek Poplawski wrote:quoted
On the other hand, I wondered about a possibility of rehashing to other queues in some cases during requeuing, which would be impossible after such change.Why would you want to do that? Just because people have abused requeue in the psat doesn't mean that we need to support such abuses for perpetuity.
Well, it was only wondering, and probably you are right this is wrong. On the other hand, simple_tx_hash() choices are "probabilistic": user doesn't care if it goes through tx_queue #1 or #11. And here, in some cases, some tx_queues could be always full while other always empty, so some dynamic rehashing could be thought of, but I understand it's not trivial. Cheers, Jarek P.