lkml.org 
[lkml]   [2006]   [Jun]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [rfc][patch] remove racy sync_page?
Nick Piggin wrote:
> Linus Torvalds wrote:
>
>>
>> On Thu, 1 Jun 2006, Nick Piggin wrote:
>>
>>> If this wasn't clear: I don't mean per-task plugs as in "the task
>>> explicitly plugs and unplugs the block device"[*]; I mean really
>>> per-task plugs.
>>
>>
>>
>> That would be insane. It would mean that you'd have to unplug whether you
>
>
> I don't think it is.
>
>> wanted to or not. Ie you've now made "sys_readahead()" impossible to
>> do well, and doing read-ahead across multiple files.
>
>
> Now it is you who are ignoring what I've been saying. I've been saying
> that I don't think your sys_readahead examples have had much to do with
> plugging:
>
> 1. If there are no other requests to seek to, plugging doesn't matter;
> 2. If there are other requests to seek to, the queue won't be plugged
> or will soon become unplugged anyway. So the current system isn't
> somehow going to do the right thing every time and be immune to
> seeking.

Lastly, an app can still issue multiple sys_readaheads and still get
seek protection -- it isn't like disks have suddenly got faster than
CPUs overnight.

as-iosched.c can handle the case where a request goes out to disk, and
later a new one comes in nearby (deadline IIRC isn't good at this
unfortunately, nor does it seek backwards)

OTOH, if you're using as-iosched.c, you explicitly get a good amount of
seek protection anyway.

--
SUSE Labs, Novell Inc.
Send instant messages to your online friends http://au.messenger.yahoo.com

-
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: 2006-06-02 04:42    [W:0.078 / U:0.752 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site