lkml.org 
[lkml]   [2004]   [Jan]   [1]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
DateThu, 1 Jan 2004 16:24:27 -0800
FromAndrew Morton <>
SubjectRe: ext2 on a CD-RW
Peter Osterlund <petero2@telia.com> 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@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: 2005-03-22 12:59    [from the cache]
©2003-2008