David Chinner wrote:
Only if we leave the page in the page cache. If we toss the page, the time it takes to do the I/O for the page fault is enough for the direct I/o to complete. Sure it's not an absolute guarantee, but if you want an absolute guarantee:
So I guess you *could* relax it in theory... Anyway, don't take my pestering as advocacy for wanting XFS to do something more clever in such a corner case. I think you're quite right to be conservative and share codepaths between direct IO read and write. -- 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
- References:
- Re: [PATCH 1/2]: Fix BUG in cancel_dirty_pages on XFS
- From: Peter Zijlstra <[email protected]>
- Re: [PATCH 1/2]: Fix BUG in cancel_dirty_pages on XFS
- From: Nick Piggin <[email protected]>
- Re: [PATCH 1/2]: Fix BUG in cancel_dirty_pages on XFS
- From: David Chinner <[email protected]>
- Re: [PATCH 1/2]: Fix BUG in cancel_dirty_pages on XFS
- From: Nick Piggin <[email protected]>
- Re: [PATCH 1/2]: Fix BUG in cancel_dirty_pages on XFS
- From: David Chinner <[email protected]>
- Re: [PATCH 1/2]: Fix BUG in cancel_dirty_pages on XFS
- From: Nick Piggin <[email protected]>
- Re: [PATCH 1/2]: Fix BUG in cancel_dirty_pages on XFS
- From: David Chinner <[email protected]>
- Re: [PATCH 1/2]: Fix BUG in cancel_dirty_pages on XFS
- From: Nick Piggin <[email protected]>
- Re: [PATCH 1/2]: Fix BUG in cancel_dirty_pages on XFS
- From: David Chinner <[email protected]>
- Re: [PATCH 1/2]: Fix BUG in cancel_dirty_pages on XFS
- From: Nick Piggin <[email protected]>
- Re: [PATCH 1/2]: Fix BUG in cancel_dirty_pages on XFS
- From: David Chinner <[email protected]>
- Re: [PATCH 1/2]: Fix BUG in cancel_dirty_pages on XFS
- Prev by Date: Re: NCPFS and brittle connections
- Next by Date: Re: IO-APIC + timer doesn't work - 2.6.20-rc5 on abit KN9-Ultra b ios 1.6
- Previous by thread: Re: [PATCH 1/2]: Fix BUG in cancel_dirty_pages on XFS
- Next by thread: Re: [PATCH 1/2]: Fix BUG in cancel_dirty_pages on XFS
- Index(es):