Thomas Themel <themel@iwoars.net> wrote:
>
> it seems that device backed loopback is broken in the 2.6.0-test2 series.
doh.
We're currently setting PF_READAHEAD across a call into the page allocator.
We end up calling writepage() with PF_READAHEAD set and the block layer
aborts the writes, resulting in corrupted data.
It only seems to bite with loop-on-blockdev for some reason.
And add a warning in ll_rw_block() to catch any more occurrences.
drivers/block/ll_rw_blk.c | 8 +++++++-
mm/readahead.c | 22 +++++++++++-----------
2 files changed, 18 insertions(+), 12 deletions(-)
diff -puN mm/readahead.c~PF_READAHEAD-loop-fix mm/readahead.c
--- 25/mm/readahead.c~PF_READAHEAD-loop-fix 2003-08-06 16:59:29.000000000 -0700
+++ 25-akpm/mm/readahead.c 2003-08-06 16:59:29.000000000 -0700
@@ -202,9 +202,9 @@ out:
*
* Returns the number of pages which actually had IO started against them.
*/
-static inline int
+static int
__do_page_cache_readahead(struct address_space *mapping, struct file *filp,
- unsigned long offset, unsigned long nr_to_read)
+ unsigned long offset, unsigned long nr_to_read, int pf_readahead)
{
struct inode *inode = mapping->host;
struct page *page;
@@ -249,8 +249,11 @@ __do_page_cache_readahead(struct address
* uptodate then the caller will launch readpage again, and
* will then handle the error.
*/
- if (ret)
+ if (ret) {
+ current->flags |= pf_readahead;
read_pages(mapping, filp, &page_pool, ret);
+ current->flags &= ~pf_readahead;
+ }
BUG_ON(!list_empty(&page_pool));
out:
return ret;
@@ -275,8 +278,8 @@ int force_page_cache_readahead(struct ad
if (this_chunk > nr_to_read)
this_chunk = nr_to_read;
- err = __do_page_cache_readahead(mapping, filp,
- offset, this_chunk);
+ err = __do_page_cache_readahead(mapping, filp, offset,
+ this_chunk, 0);
if (err < 0) {
ret = err;
break;
@@ -300,12 +303,9 @@ int do_page_cache_readahead(struct addre
{
int ret = 0;
- if (!bdi_read_congested(mapping->backing_dev_info)) {
- current->flags |= PF_READAHEAD;
- ret = __do_page_cache_readahead(mapping, filp,
- offset, nr_to_read);
- current->flags &= ~PF_READAHEAD;
- }
+ if (!bdi_read_congested(mapping->backing_dev_info))
+ ret = __do_page_cache_readahead(mapping, filp, offset,
+ nr_to_read, PF_READAHEAD);
return ret;
}
diff -puN drivers/block/ll_rw_blk.c~PF_READAHEAD-loop-fix drivers/block/ll_rw_blk.c
--- 25/drivers/block/ll_rw_blk.c~PF_READAHEAD-loop-fix 2003-08-06 16:59:29.000000000 -0700
+++ 25-akpm/drivers/block/ll_rw_blk.c 2003-08-06 17:40:27.000000000 -0700
@@ -1847,7 +1847,13 @@ static int __make_request(request_queue_
barrier = test_bit(BIO_RW_BARRIER, &bio->bi_rw);
- ra = bio_flagged(bio, BIO_RW_AHEAD) || current->flags & PF_READAHEAD;
+ ra = bio_flagged(bio, BIO_RW_AHEAD);
+ if (current->flags & PF_READAHEAD) {
+ if (rw == WRITE)
+ WARN_ON(1);
+ else
+ ra = 1;
+ }
again:
insert_here = NULL;
_
-
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/
This archive was generated by hypermail 2b29 : Thu Aug 07 2003 - 22:00:36 EST