Re: [PATCH] cfq: get rid of cfqq hash

[Jens Axboe]

On Tue, Apr 24 2007, Vasily Tarasov wrote:
> @@ -1806,7 +1765,11 @@ static int cfq_may_queue(request_queue_t
>  	 * so just lookup a possibly existing queue, or return 'may queue'
>  	 * if that fails
>  	 */
> -	cfqq = cfq_find_cfq_hash(cfqd, key, tsk->ioprio);
> +	cic = cfq_get_io_context_noalloc(cfqd, tsk);
> +	if (!cic)
> +		return ELV_MQUEUE_MAY; 
> +
> +	cfqq = cic->cfqq[rw & REQ_RW_SYNC];
>  	if (cfqq) {
>  		cfq_init_prio_data(cfqq);
>  		cfq_prio_boost(cfqq);

Ahem, how well did you test this patch?

-- 
Jens Axboe

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/