lkml.org 
[lkml]   [2006]   [May]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [rfc][patch] remove racy sync_page?
Linus Torvalds wrote:

> And I suspect your objection to unplugging is not really about unplugging
> itself. It's literally about the fact that we use the same page lock for
> IO and for the ->mapping thing, isn't it?

Nearly, but not quite that far: it's that we sync_page in lock_page.

I don't think using the single lock for both is too bad (in many
ways they are related eg. you don't want the page to be truncated
while IO is in progress).

>
> IOW, you don't actually dislike plugging itself, you dislike it due to the
> effects of a totally unrelated locking issue, namely that we use the same
> lock for two totally independent things. If the ->mapping thing were to
> use a PG_map_lock that didn't affect plugging one way or the other, you
> wouldn't have any issues with unplugging, would you?
>
> And I think _that_ is what really gets us to the problem.

No I don't dislike plugging at all ;) Just this tangle as you say.

--
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-05-31 18:29    [W:0.118 / U:0.084 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site