lkml.org 
[lkml]   [2005]   [Oct]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH][noop-iosched] don't reuse a freed request
On Mon, Oct 31 2005, Tejun Heo wrote:
> >For now, we should add the former.
>
> Yeap, also verified oops doesn't happen with the following patch.

Good

> I'll soon post a patch to convert noop such that it does proper
> dispatching. BTW, while I was looking at the code, I found something

Sounds good.

> else, in elv_former/latter_request functions, if the iosched doesn't
> supply the callbacks, it uses rq->queue_list.prev/next implicitly
> (without this, this noop bug wouldn't have been triggered). I think
> this code is not necessary anymore. What do you think?

Well that should still work for noop, if it has its own internal
queueing list. But I would be quite fine with removing that implicit
next/prev support and simply require io scheds to supply these functions
always if they require rq-to-rq merging. Noop is the only one that
relied on this in the past, now seems a good time to clean that up as
well.

Besides, as you note, with merging disallowed on the ->queue_head, they
don't work as expected anymore.

--
Jens Axboe

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

\
 
 \ /
  Last update: 2005-10-31 10:13    [W:0.621 / U:0.084 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site