Re: [PATCH] Fix disk statistic reporting to include all disks

From: Bill Wilson (
Date: Wed Aug 30 2000 - 12:18:07 EST

On 30-Aug-2000 Jens Axboe wrote:
> Right now the code in my tree looks something like this:
> for (i = 0; i < MAX_BLKDEV; i++) {
> request_queue_t *q = blk_get_queue(MKDEV(i, 0));
> if (!q)
> continue;
> if (!active)
> continue;
> /* print stats */
> }
> So it's not really that bad. I suppose things could be made quicker
> by just browsing a list of active queues, but I'm not sure it's worth
> the trouble (but hey, proove me wrong). Note that linking blk_dev
> does not make much sense, since not all queues are available from
> there. Drivers are free to use their own queue, even several if they
> want.

Agreed, this looks far better than the 16 x 16 table looping
that was going on before (and not getting all disks). My last question,
when you print stats will the format still be (%u,%u):(%u,%u,%u,%u)?
And now that disk_index() is going away, will the real minor number be

Bill Wilson <>

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
Please read the FAQ at

This archive was generated by hypermail 2b29 : Thu Aug 31 2000 - 21:00:25 EST