Re: ext2 on a CD-RW

[Andrew Morton]

Peter Osterlund <petero2@xxxxxxxxx> wrote:
>
> If I create an ext2 filesystem
>  with 2kb blocksize, I hit a bug when I try to write some large files to
>  the filesystem. The problem is that the code in mpage_writepage() fails if
>  a page is mapped to disk across a packet boundary. In that case, the
>  bio_add_page() call at line 543 in mpage.c can fail even if the bio was
>  previously empty. The code then passes an empty bio to submit_bio(), which
>  triggers a bug at line 2303 in ll_rw_blk.c. This patch seems to fix the
>  problem.
> 
>  --- linux/fs/mpage.c.old	2004-01-02 00:26:19.000000000 +0100
>  +++ linux/fs/mpage.c	2004-01-02 00:26:50.000000000 +0100
>  @@ -541,6 +541,11 @@
>   
>   	length = first_unmapped << blkbits;
>   	if (bio_add_page(bio, page, length, 0) < length) {
>  +		if (!bio->bi_size) {
>  +			bio_put(bio);
>  +			bio = NULL;
>  +			goto confused;
>  +		}
>   		bio = mpage_bio_submit(WRITE, bio);
>   		goto alloc_new;
>   	}

Confused.  We initially have an empty BIO, and we run bio_add_page()
against it, adding one page.

How can that bio_add_page() fail to add the page?

Cold you describe the failure a little more please?

>  I noted that performance is quite bad with 2kb blocksize. It is a lot
>  faster with 4kb blocksize. (2kb blocksize with the udf filesystem is not
>  slow, only ext2 seems to have this problem.) Maybe the "confused" case
>  (which calls a_ops->writepage()) in mpage_writepage() isn't really meant
>  to be fast. Is there a better way to fix this problem?

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