diff options
author | Chris Mason <chris.mason@oracle.com> | 2008-10-30 13:22:14 -0400 |
---|---|---|
committer | Chris Mason <chris.mason@oracle.com> | 2008-10-30 13:22:14 -0400 |
commit | cfbc246eaae2a1089911016094b74b3055e8a906 (patch) | |
tree | c450f02cd605b38a578778dacd9c8768ce041789 /fs/btrfs/compression.c | |
parent | 87ef2bb46bfc4be0b40799e68115cbe28d80a1bd (diff) |
Btrfs: walk compressed pages based on the nr_pages count instead of bytes
The byte walk counting was awkward and error prone. This uses the
number of pages sent the higher layer to build bios.
Signed-off-by: Chris Mason <chris.mason@oracle.com>
Diffstat (limited to 'fs/btrfs/compression.c')
-rw-r--r-- | fs/btrfs/compression.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/fs/btrfs/compression.c b/fs/btrfs/compression.c index c5470367ca5c..9adaa79adad9 100644 --- a/fs/btrfs/compression.c +++ b/fs/btrfs/compression.c | |||
@@ -296,7 +296,7 @@ int btrfs_submit_compressed_write(struct inode *inode, u64 start, | |||
296 | 296 | ||
297 | /* create and submit bios for the compressed pages */ | 297 | /* create and submit bios for the compressed pages */ |
298 | bytes_left = compressed_len; | 298 | bytes_left = compressed_len; |
299 | while(bytes_left > 0) { | 299 | for (page_index = 0; page_index < cb->nr_pages; page_index++) { |
300 | page = compressed_pages[page_index]; | 300 | page = compressed_pages[page_index]; |
301 | page->mapping = inode->i_mapping; | 301 | page->mapping = inode->i_mapping; |
302 | if (bio->bi_size) | 302 | if (bio->bi_size) |
@@ -324,7 +324,10 @@ int btrfs_submit_compressed_write(struct inode *inode, u64 start, | |||
324 | bio->bi_end_io = end_compressed_bio_write; | 324 | bio->bi_end_io = end_compressed_bio_write; |
325 | bio_add_page(bio, page, PAGE_CACHE_SIZE, 0); | 325 | bio_add_page(bio, page, PAGE_CACHE_SIZE, 0); |
326 | } | 326 | } |
327 | page_index++; | 327 | if (bytes_left < PAGE_CACHE_SIZE) { |
328 | printk("bytes left %lu compress len %lu nr %lu\n", | ||
329 | bytes_left, cb->compressed_len, cb->nr_pages); | ||
330 | } | ||
328 | bytes_left -= PAGE_CACHE_SIZE; | 331 | bytes_left -= PAGE_CACHE_SIZE; |
329 | first_byte += PAGE_CACHE_SIZE; | 332 | first_byte += PAGE_CACHE_SIZE; |
330 | } | 333 | } |