* [RFC 4/5] implement metadata_ra in btrfs
@ 2010-12-13 7:22 Shaohua Li
2010-12-14 1:06 ` Yan, Zheng
0 siblings, 1 reply; 3+ messages in thread
From: Shaohua Li @ 2010-12-13 7:22 UTC (permalink / raw)
To: linux-btrfs@vger.kernel.org, linux-fsdevel@vger.kernel.org
Cc: Chris Mason, Christoph Hellwig, Arjan van de Ven, Andrew Morton
Implementation btrfs .metadata_readahead. In btrfs, all metadata pages are in a
special btree_inode. We do readahead in it.
Signed-off-by: Shaohua Li <shaohua.li@intel.com>
---
fs/btrfs/disk-io.c | 10 ++++++++++
fs/btrfs/super.c | 13 +++++++++++++
mm/readahead.c | 1 +
3 files changed, 24 insertions(+)
Index: linux/fs/btrfs/disk-io.c
===================================================================
--- linux.orig/fs/btrfs/disk-io.c 2010-12-07 13:32:24.000000000 +0800
+++ linux/fs/btrfs/disk-io.c 2010-12-07 13:33:08.000000000 +0800
@@ -776,6 +776,15 @@ static int btree_readpage(struct file *f
return extent_read_full_page(tree, page, btree_get_extent);
}
+static int btree_readpages(struct file *file, struct address_space *mapping,
+ struct list_head *pages, unsigned nr_pages)
+{
+ struct extent_io_tree *tree;
+ tree = &BTRFS_I(mapping->host)->io_tree;
+ return extent_readpages(tree, mapping, pages, nr_pages,
+ btree_get_extent);
+}
+
static int btree_releasepage(struct page *page, gfp_t gfp_flags)
{
struct extent_io_tree *tree;
@@ -819,6 +828,7 @@ static void btree_invalidatepage(struct
static const struct address_space_operations btree_aops = {
.readpage = btree_readpage,
+ .readpages = btree_readpages,
.writepage = btree_writepage,
.writepages = btree_writepages,
.releasepage = btree_releasepage,
Index: linux/fs/btrfs/super.c
===================================================================
--- linux.orig/fs/btrfs/super.c 2010-12-07 13:32:24.000000000 +0800
+++ linux/fs/btrfs/super.c 2010-12-07 13:33:08.000000000 +0800
@@ -892,6 +892,18 @@ out:
return -ENOENT;
}
+static int btrfs_metadata_readahead(struct super_block *sb, loff_t offset,
+ ssize_t size)
+{
+ struct btrfs_root *tree_root = btrfs_sb(sb);
+ struct inode *btree_inode = tree_root->fs_info->btree_inode;
+ struct address_space *mapping = btree_inode->i_mapping;
+
+ force_page_cache_readahead(mapping, NULL, offset >> PAGE_CACHE_SHIFT,
+ size >> PAGE_CACHE_SHIFT);
+ return 0;
+}
+
static const struct super_operations btrfs_super_ops = {
.drop_inode = btrfs_drop_inode,
.evict_inode = btrfs_evict_inode,
@@ -907,6 +919,7 @@ static const struct super_operations btr
.freeze_fs = btrfs_freeze,
.unfreeze_fs = btrfs_unfreeze,
.metadata_incore = btrfs_metadata_incore,
+ .metadata_readahead = btrfs_metadata_readahead,
};
static const struct file_operations btrfs_ctl_fops = {
Index: linux/mm/readahead.c
===================================================================
--- linux.orig/mm/readahead.c 2010-12-07 13:32:24.000000000 +0800
+++ linux/mm/readahead.c 2010-12-07 13:33:08.000000000 +0800
@@ -228,6 +228,7 @@ int force_page_cache_readahead(struct ad
}
return ret;
}
+EXPORT_SYMBOL_GPL(force_page_cache_readahead);
/*
* Given a desired number of PAGE_CACHE_SIZE readahead pages, return a
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [RFC 4/5] implement metadata_ra in btrfs
2010-12-13 7:22 [RFC 4/5] implement metadata_ra in btrfs Shaohua Li
@ 2010-12-14 1:06 ` Yan, Zheng
2010-12-14 1:14 ` Shaohua Li
0 siblings, 1 reply; 3+ messages in thread
From: Yan, Zheng @ 2010-12-14 1:06 UTC (permalink / raw)
To: Shaohua Li
Cc: linux-btrfs@vger.kernel.org, linux-fsdevel@vger.kernel.org,
Chris Mason, Christoph Hellwig, Arjan van de Ven, Andrew Morton
On Mon, Dec 13, 2010 at 3:22 PM, Shaohua Li <shaohua.li@intel.com> wrote:
> Implementation btrfs .metadata_readahead. In btrfs, all metadata pages are in a
> special btree_inode. We do readahead in it.
>
> Signed-off-by: Shaohua Li <shaohua.li@intel.com>
>
> ---
> fs/btrfs/disk-io.c | 10 ++++++++++
> fs/btrfs/super.c | 13 +++++++++++++
> mm/readahead.c | 1 +
> 3 files changed, 24 insertions(+)
>
> Index: linux/fs/btrfs/disk-io.c
> ===================================================================
> --- linux.orig/fs/btrfs/disk-io.c 2010-12-07 13:32:24.000000000 +0800
> +++ linux/fs/btrfs/disk-io.c 2010-12-07 13:33:08.000000000 +0800
> @@ -776,6 +776,15 @@ static int btree_readpage(struct file *f
> return extent_read_full_page(tree, page, btree_get_extent);
> }
>
> +static int btree_readpages(struct file *file, struct address_space *mapping,
> + struct list_head *pages, unsigned nr_pages)
> +{
> + struct extent_io_tree *tree;
> + tree = &BTRFS_I(mapping->host)->io_tree;
> + return extent_readpages(tree, mapping, pages, nr_pages,
> + btree_get_extent);
> +}
> +
> static int btree_releasepage(struct page *page, gfp_t gfp_flags)
> {
> struct extent_io_tree *tree;
> @@ -819,6 +828,7 @@ static void btree_invalidatepage(struct
>
> static const struct address_space_operations btree_aops = {
> .readpage = btree_readpage,
> + .readpages = btree_readpages,
> .writepage = btree_writepage,
> .writepages = btree_writepages,
> .releasepage = btree_releasepage,
> Index: linux/fs/btrfs/super.c
> ===================================================================
> --- linux.orig/fs/btrfs/super.c 2010-12-07 13:32:24.000000000 +0800
> +++ linux/fs/btrfs/super.c 2010-12-07 13:33:08.000000000 +0800
> @@ -892,6 +892,18 @@ out:
> return -ENOENT;
> }
>
> +static int btrfs_metadata_readahead(struct super_block *sb, loff_t offset,
> + ssize_t size)
> +{
> + struct btrfs_root *tree_root = btrfs_sb(sb);
> + struct inode *btree_inode = tree_root->fs_info->btree_inode;
> + struct address_space *mapping = btree_inode->i_mapping;
> +
> + force_page_cache_readahead(mapping, NULL, offset >> PAGE_CACHE_SHIFT,
> + size >> PAGE_CACHE_SHIFT);
> + return 0;
> +}
> +
> static const struct super_operations btrfs_super_ops = {
> .drop_inode = btrfs_drop_inode,
> .evict_inode = btrfs_evict_inode,
> @@ -907,6 +919,7 @@ static const struct super_operations btr
> .freeze_fs = btrfs_freeze,
> .unfreeze_fs = btrfs_unfreeze,
> .metadata_incore = btrfs_metadata_incore,
> + .metadata_readahead = btrfs_metadata_readahead,
> };
>
> static const struct file_operations btrfs_ctl_fops = {
> Index: linux/mm/readahead.c
> ===================================================================
> --- linux.orig/mm/readahead.c 2010-12-07 13:32:24.000000000 +0800
> +++ linux/mm/readahead.c 2010-12-07 13:33:08.000000000 +0800
> @@ -228,6 +228,7 @@ int force_page_cache_readahead(struct ad
> }
> return ret;
> }
> +EXPORT_SYMBOL_GPL(force_page_cache_readahead);
>
> /*
> * Given a desired number of PAGE_CACHE_SIZE readahead pages, return a
>
>
btrfs will crash If the read-ahead range falls into unallocated chunk.
need code to check validity of the user input.
Yan, Zheng
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [RFC 4/5] implement metadata_ra in btrfs
2010-12-14 1:06 ` Yan, Zheng
@ 2010-12-14 1:14 ` Shaohua Li
0 siblings, 0 replies; 3+ messages in thread
From: Shaohua Li @ 2010-12-14 1:14 UTC (permalink / raw)
To: Yan, Zheng
Cc: linux-btrfs@vger.kernel.org, linux-fsdevel@vger.kernel.org,
Chris Mason, Christoph Hellwig, Arjan van de Ven, Andrew Morton
On Tue, 2010-12-14 at 09:06 +0800, Yan, Zheng wrote:
> On Mon, Dec 13, 2010 at 3:22 PM, Shaohua Li <shaohua.li@intel.com> wrote:
> > Implementation btrfs .metadata_readahead. In btrfs, all metadata pages are in a
> > special btree_inode. We do readahead in it.
> >
> > Signed-off-by: Shaohua Li <shaohua.li@intel.com>
> >
> > ---
> > fs/btrfs/disk-io.c | 10 ++++++++++
> > fs/btrfs/super.c | 13 +++++++++++++
> > mm/readahead.c | 1 +
> > 3 files changed, 24 insertions(+)
> >
> > Index: linux/fs/btrfs/disk-io.c
> > ===================================================================
> > --- linux.orig/fs/btrfs/disk-io.c 2010-12-07 13:32:24.000000000 +0800
> > +++ linux/fs/btrfs/disk-io.c 2010-12-07 13:33:08.000000000 +0800
> > @@ -776,6 +776,15 @@ static int btree_readpage(struct file *f
> > return extent_read_full_page(tree, page, btree_get_extent);
> > }
> >
> > +static int btree_readpages(struct file *file, struct address_space *mapping,
> > + struct list_head *pages, unsigned nr_pages)
> > +{
> > + struct extent_io_tree *tree;
> > + tree = &BTRFS_I(mapping->host)->io_tree;
> > + return extent_readpages(tree, mapping, pages, nr_pages,
> > + btree_get_extent);
> > +}
> > +
> > static int btree_releasepage(struct page *page, gfp_t gfp_flags)
> > {
> > struct extent_io_tree *tree;
> > @@ -819,6 +828,7 @@ static void btree_invalidatepage(struct
> >
> > static const struct address_space_operations btree_aops = {
> > .readpage = btree_readpage,
> > + .readpages = btree_readpages,
> > .writepage = btree_writepage,
> > .writepages = btree_writepages,
> > .releasepage = btree_releasepage,
> > Index: linux/fs/btrfs/super.c
> > ===================================================================
> > --- linux.orig/fs/btrfs/super.c 2010-12-07 13:32:24.000000000 +0800
> > +++ linux/fs/btrfs/super.c 2010-12-07 13:33:08.000000000 +0800
> > @@ -892,6 +892,18 @@ out:
> > return -ENOENT;
> > }
> >
> > +static int btrfs_metadata_readahead(struct super_block *sb, loff_t offset,
> > + ssize_t size)
> > +{
> > + struct btrfs_root *tree_root = btrfs_sb(sb);
> > + struct inode *btree_inode = tree_root->fs_info->btree_inode;
> > + struct address_space *mapping = btree_inode->i_mapping;
> > +
> > + force_page_cache_readahead(mapping, NULL, offset >> PAGE_CACHE_SHIFT,
> > + size >> PAGE_CACHE_SHIFT);
> > + return 0;
> > +}
> > +
> > static const struct super_operations btrfs_super_ops = {
> > .drop_inode = btrfs_drop_inode,
> > .evict_inode = btrfs_evict_inode,
> > @@ -907,6 +919,7 @@ static const struct super_operations btr
> > .freeze_fs = btrfs_freeze,
> > .unfreeze_fs = btrfs_unfreeze,
> > .metadata_incore = btrfs_metadata_incore,
> > + .metadata_readahead = btrfs_metadata_readahead,
> > };
> >
> > static const struct file_operations btrfs_ctl_fops = {
> > Index: linux/mm/readahead.c
> > ===================================================================
> > --- linux.orig/mm/readahead.c 2010-12-07 13:32:24.000000000 +0800
> > +++ linux/mm/readahead.c 2010-12-07 13:33:08.000000000 +0800
> > @@ -228,6 +228,7 @@ int force_page_cache_readahead(struct ad
> > }
> > return ret;
> > }
> > +EXPORT_SYMBOL_GPL(force_page_cache_readahead);
> >
> > /*
> > * Given a desired number of PAGE_CACHE_SIZE readahead pages, return a
> >
> >
>
> btrfs will crash If the read-ahead range falls into unallocated chunk.
> need code to check validity of the user input.
didn't hit this, maybe I'm lucky. I'll double check this.
Thanks,
Shaohua
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-12-14 1:14 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-12-13 7:22 [RFC 4/5] implement metadata_ra in btrfs Shaohua Li
2010-12-14 1:06 ` Yan, Zheng
2010-12-14 1:14 ` Shaohua Li
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).