LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [f2fs-dev][RFC PATCH 04/10] f2fs: introduce universal lookup/update interface for extent cache
@ 2015-01-12 7:12 Chao Yu
2015-01-23 0:36 ` Jaegeuk Kim
0 siblings, 1 reply; 3+ messages in thread
From: Chao Yu @ 2015-01-12 7:12 UTC (permalink / raw)
To: Jaegeuk Kim, Changman Lee; +Cc: linux-f2fs-devel, linux-kernel
In this patch, we do these jobs:
1. rename {check,update}_extent_cache to f2fs_{lookup,update}_extent_info;
2. introduce universal lookup/update interface of extent cache:
f2fs_{lookup,update}_extent_cache including above two real functions, then
export them to function callers.
So after above cleanup, we can add new rb-tree based extent cache into exported
interfaces.
Signed-off-by: Chao Yu <chao2.yu@samsung.com>
---
fs/f2fs/data.c | 66 +++++++++++++++++++++++++++++++++---------------------
fs/f2fs/f2fs.h | 2 +-
fs/f2fs/file.c | 2 +-
fs/f2fs/inline.c | 2 +-
fs/f2fs/recovery.c | 2 +-
5 files changed, 44 insertions(+), 30 deletions(-)
diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
index 5a3e489..4f5b871e 100644
--- a/fs/f2fs/data.c
+++ b/fs/f2fs/data.c
@@ -270,20 +270,20 @@ static void f2fs_map_bh(struct inode *inode, pgoff_t pgofs,
bh_result->b_size = UINT_MAX;
}
-static int check_extent_cache(struct inode *inode, pgoff_t pgofs,
- struct extent_info *ei)
+static bool f2fs_lookup_extent_info(struct inode *inode, pgoff_t pgofs,
+ struct extent_info *ei)
{
struct f2fs_inode_info *fi = F2FS_I(inode);
pgoff_t start_fofs, end_fofs;
block_t start_blkaddr;
if (is_inode_flag_set(fi, FI_NO_EXTENT))
- return 0;
+ return false;
read_lock(&fi->ext_lock);
if (fi->ext.len == 0) {
read_unlock(&fi->ext_lock);
- return 0;
+ return false;
}
stat_inc_total_hit(inode->i_sb);
@@ -296,29 +296,22 @@ static int check_extent_cache(struct inode *inode, pgoff_t pgofs,
*ei = fi->ext;
stat_inc_read_hit(inode->i_sb);
read_unlock(&fi->ext_lock);
- return 1;
+ return true;
}
read_unlock(&fi->ext_lock);
- return 0;
+ return false;
}
-void update_extent_cache(struct dnode_of_data *dn)
+static bool f2fs_update_extent_info(struct inode *inode, pgoff_t fofs,
+ block_t blkaddr)
{
- struct f2fs_inode_info *fi = F2FS_I(dn->inode);
- pgoff_t fofs, start_fofs, end_fofs;
+ struct f2fs_inode_info *fi = F2FS_I(inode);
+ pgoff_t start_fofs, end_fofs;
block_t start_blkaddr, end_blkaddr;
int need_update = true;
- f2fs_bug_on(F2FS_I_SB(dn->inode), dn->data_blkaddr == NEW_ADDR);
-
- /* Update the page address in the parent node */
- __set_data_blkaddr(dn);
-
if (is_inode_flag_set(fi, FI_NO_EXTENT))
- return;
-
- fofs = start_bidx_of_node(ofs_of_node(dn->node_page), fi) +
- dn->ofs_in_node;
+ return false;
write_lock(&fi->ext_lock);
@@ -333,16 +326,16 @@ void update_extent_cache(struct dnode_of_data *dn)
/* Initial extent */
if (fi->ext.len == 0) {
- if (dn->data_blkaddr != NULL_ADDR) {
+ if (blkaddr != NULL_ADDR) {
fi->ext.fofs = fofs;
- fi->ext.blk = dn->data_blkaddr;
+ fi->ext.blk = blkaddr;
fi->ext.len = 1;
}
goto end_update;
}
/* Front merge */
- if (fofs == start_fofs - 1 && dn->data_blkaddr == start_blkaddr - 1) {
+ if (fofs == start_fofs - 1 && blkaddr == start_blkaddr - 1) {
fi->ext.fofs--;
fi->ext.blk--;
fi->ext.len++;
@@ -350,7 +343,7 @@ void update_extent_cache(struct dnode_of_data *dn)
}
/* Back merge */
- if (fofs == end_fofs + 1 && dn->data_blkaddr == end_blkaddr + 1) {
+ if (fofs == end_fofs + 1 && blkaddr == end_blkaddr + 1) {
fi->ext.len++;
goto end_update;
}
@@ -377,9 +370,30 @@ void update_extent_cache(struct dnode_of_data *dn)
}
end_update:
write_unlock(&fi->ext_lock);
- if (need_update)
+ return need_update;
+}
+
+static bool f2fs_lookup_extent_cache(struct inode *inode, pgoff_t pgofs,
+ struct extent_info *ei)
+{
+ return f2fs_lookup_extent_info(inode, pgofs, ei);
+}
+
+void f2fs_update_extent_cache(struct dnode_of_data *dn)
+{
+ struct f2fs_inode_info *fi = F2FS_I(dn->inode);
+ pgoff_t fofs;
+
+ f2fs_bug_on(F2FS_I_SB(dn->inode), dn->data_blkaddr == NEW_ADDR);
+
+ /* Update the page address in the parent node */
+ __set_data_blkaddr(dn);
+
+ fofs = start_bidx_of_node(ofs_of_node(dn->node_page), fi) +
+ dn->ofs_in_node;
+
+ if (f2fs_update_extent_info(dn->inode, fofs, dn->data_blkaddr))
sync_inode_page(dn);
- return;
}
struct page *find_data_page(struct inode *inode, pgoff_t index, bool sync)
@@ -625,7 +639,7 @@ static int __get_data_block(struct inode *inode, sector_t iblock,
/* Get the page offset from the block offset(iblock) */
pgofs = (pgoff_t)(iblock >> (PAGE_CACHE_SHIFT - blkbits));
- if (check_extent_cache(inode, pgofs, &ei)) {
+ if (f2fs_lookup_extent_cache(inode, pgofs, &ei)) {
f2fs_map_bh(inode, pgofs, &ei, bh_result);
goto out;
}
@@ -792,7 +806,7 @@ int do_write_data_page(struct page *page, struct f2fs_io_info *fio)
set_inode_flag(F2FS_I(inode), FI_UPDATE_WRITE);
} else {
write_data_page(page, &dn, fio);
- update_extent_cache(&dn);
+ f2fs_update_extent_cache(&dn);
set_inode_flag(F2FS_I(inode), FI_APPEND_WRITE);
}
out_writepage:
diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h
index a74ad85..eb9ffae 100644
--- a/fs/f2fs/f2fs.h
+++ b/fs/f2fs/f2fs.h
@@ -1475,7 +1475,7 @@ void f2fs_submit_page_mbio(struct f2fs_sb_info *, struct page *,
struct f2fs_io_info *);
int reserve_new_block(struct dnode_of_data *);
int f2fs_reserve_block(struct dnode_of_data *, pgoff_t);
-void update_extent_cache(struct dnode_of_data *);
+void f2fs_update_extent_cache(struct dnode_of_data *);
struct page *find_data_page(struct inode *, pgoff_t, bool);
struct page *get_lock_data_page(struct inode *, pgoff_t);
struct page *get_new_data_page(struct inode *, struct page *, pgoff_t, bool);
diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c
index 5df3367..0dcd9fb 100644
--- a/fs/f2fs/file.c
+++ b/fs/f2fs/file.c
@@ -430,7 +430,7 @@ int truncate_data_blocks_range(struct dnode_of_data *dn, int count)
continue;
dn->data_blkaddr = NULL_ADDR;
- update_extent_cache(dn);
+ f2fs_update_extent_cache(dn);
invalidate_blocks(sbi, blkaddr);
nr_free++;
}
diff --git a/fs/f2fs/inline.c b/fs/f2fs/inline.c
index fa2aa2f..9f6c745 100644
--- a/fs/f2fs/inline.c
+++ b/fs/f2fs/inline.c
@@ -116,7 +116,7 @@ no_update:
set_page_writeback(page);
fio.blk_addr = dn->data_blkaddr;
write_data_page(page, dn, &fio);
- update_extent_cache(dn);
+ f2fs_update_extent_cache(dn);
f2fs_wait_on_page_writeback(page, DATA);
if (dirty)
inode_dec_dirty_pages(dn->inode);
diff --git a/fs/f2fs/recovery.c b/fs/f2fs/recovery.c
index c4211a5..a7adbdb 100644
--- a/fs/f2fs/recovery.c
+++ b/fs/f2fs/recovery.c
@@ -397,7 +397,7 @@ static int do_recover_data(struct f2fs_sb_info *sbi, struct inode *inode,
/* write dummy data page */
recover_data_page(sbi, NULL, &sum, src, dest);
dn.data_blkaddr = dest;
- update_extent_cache(&dn);
+ f2fs_update_extent_cache(&dn);
recovered++;
}
dn.ofs_in_node++;
--
2.2.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [f2fs-dev][RFC PATCH 04/10] f2fs: introduce universal lookup/update interface for extent cache
2015-01-12 7:12 [f2fs-dev][RFC PATCH 04/10] f2fs: introduce universal lookup/update interface for extent cache Chao Yu
@ 2015-01-23 0:36 ` Jaegeuk Kim
2015-01-23 1:55 ` Chao Yu
0 siblings, 1 reply; 3+ messages in thread
From: Jaegeuk Kim @ 2015-01-23 0:36 UTC (permalink / raw)
To: Chao Yu; +Cc: Changman Lee, linux-f2fs-devel, linux-kernel
Hi Chao,
On Mon, Jan 12, 2015 at 03:12:13PM +0800, Chao Yu wrote:
> In this patch, we do these jobs:
> 1. rename {check,update}_extent_cache to f2fs_{lookup,update}_extent_info;
> 2. introduce universal lookup/update interface of extent cache:
> f2fs_{lookup,update}_extent_cache including above two real functions, then
> export them to function callers.
>
> So after above cleanup, we can add new rb-tree based extent cache into exported
> interfaces.
>
> Signed-off-by: Chao Yu <chao2.yu@samsung.com>
> ---
> fs/f2fs/data.c | 66 +++++++++++++++++++++++++++++++++---------------------
> fs/f2fs/f2fs.h | 2 +-
> fs/f2fs/file.c | 2 +-
> fs/f2fs/inline.c | 2 +-
> fs/f2fs/recovery.c | 2 +-
> 5 files changed, 44 insertions(+), 30 deletions(-)
>
> diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
> index 5a3e489..4f5b871e 100644
> --- a/fs/f2fs/data.c
> +++ b/fs/f2fs/data.c
> @@ -270,20 +270,20 @@ static void f2fs_map_bh(struct inode *inode, pgoff_t pgofs,
> bh_result->b_size = UINT_MAX;
> }
>
> -static int check_extent_cache(struct inode *inode, pgoff_t pgofs,
> - struct extent_info *ei)
> +static bool f2fs_lookup_extent_info(struct inode *inode, pgoff_t pgofs,
> + struct extent_info *ei)
This is a static function, so I think it would be good to remove "f2fs_".
> {
> struct f2fs_inode_info *fi = F2FS_I(inode);
> pgoff_t start_fofs, end_fofs;
> block_t start_blkaddr;
>
> if (is_inode_flag_set(fi, FI_NO_EXTENT))
> - return 0;
> + return false;
>
> read_lock(&fi->ext_lock);
> if (fi->ext.len == 0) {
> read_unlock(&fi->ext_lock);
> - return 0;
> + return false;
> }
>
> stat_inc_total_hit(inode->i_sb);
> @@ -296,29 +296,22 @@ static int check_extent_cache(struct inode *inode, pgoff_t pgofs,
> *ei = fi->ext;
> stat_inc_read_hit(inode->i_sb);
> read_unlock(&fi->ext_lock);
> - return 1;
> + return true;
> }
> read_unlock(&fi->ext_lock);
> - return 0;
> + return false;
> }
>
> -void update_extent_cache(struct dnode_of_data *dn)
> +static bool f2fs_update_extent_info(struct inode *inode, pgoff_t fofs,
> + block_t blkaddr)
Ditto.
Thanks,
> {
> - struct f2fs_inode_info *fi = F2FS_I(dn->inode);
> - pgoff_t fofs, start_fofs, end_fofs;
> + struct f2fs_inode_info *fi = F2FS_I(inode);
> + pgoff_t start_fofs, end_fofs;
> block_t start_blkaddr, end_blkaddr;
> int need_update = true;
>
> - f2fs_bug_on(F2FS_I_SB(dn->inode), dn->data_blkaddr == NEW_ADDR);
> -
> - /* Update the page address in the parent node */
> - __set_data_blkaddr(dn);
> -
> if (is_inode_flag_set(fi, FI_NO_EXTENT))
> - return;
> -
> - fofs = start_bidx_of_node(ofs_of_node(dn->node_page), fi) +
> - dn->ofs_in_node;
> + return false;
>
> write_lock(&fi->ext_lock);
>
> @@ -333,16 +326,16 @@ void update_extent_cache(struct dnode_of_data *dn)
>
> /* Initial extent */
> if (fi->ext.len == 0) {
> - if (dn->data_blkaddr != NULL_ADDR) {
> + if (blkaddr != NULL_ADDR) {
> fi->ext.fofs = fofs;
> - fi->ext.blk = dn->data_blkaddr;
> + fi->ext.blk = blkaddr;
> fi->ext.len = 1;
> }
> goto end_update;
> }
>
> /* Front merge */
> - if (fofs == start_fofs - 1 && dn->data_blkaddr == start_blkaddr - 1) {
> + if (fofs == start_fofs - 1 && blkaddr == start_blkaddr - 1) {
> fi->ext.fofs--;
> fi->ext.blk--;
> fi->ext.len++;
> @@ -350,7 +343,7 @@ void update_extent_cache(struct dnode_of_data *dn)
> }
>
> /* Back merge */
> - if (fofs == end_fofs + 1 && dn->data_blkaddr == end_blkaddr + 1) {
> + if (fofs == end_fofs + 1 && blkaddr == end_blkaddr + 1) {
> fi->ext.len++;
> goto end_update;
> }
> @@ -377,9 +370,30 @@ void update_extent_cache(struct dnode_of_data *dn)
> }
> end_update:
> write_unlock(&fi->ext_lock);
> - if (need_update)
> + return need_update;
> +}
> +
> +static bool f2fs_lookup_extent_cache(struct inode *inode, pgoff_t pgofs,
> + struct extent_info *ei)
> +{
> + return f2fs_lookup_extent_info(inode, pgofs, ei);
> +}
> +
> +void f2fs_update_extent_cache(struct dnode_of_data *dn)
> +{
> + struct f2fs_inode_info *fi = F2FS_I(dn->inode);
> + pgoff_t fofs;
> +
> + f2fs_bug_on(F2FS_I_SB(dn->inode), dn->data_blkaddr == NEW_ADDR);
> +
> + /* Update the page address in the parent node */
> + __set_data_blkaddr(dn);
> +
> + fofs = start_bidx_of_node(ofs_of_node(dn->node_page), fi) +
> + dn->ofs_in_node;
> +
> + if (f2fs_update_extent_info(dn->inode, fofs, dn->data_blkaddr))
> sync_inode_page(dn);
> - return;
> }
>
> struct page *find_data_page(struct inode *inode, pgoff_t index, bool sync)
> @@ -625,7 +639,7 @@ static int __get_data_block(struct inode *inode, sector_t iblock,
> /* Get the page offset from the block offset(iblock) */
> pgofs = (pgoff_t)(iblock >> (PAGE_CACHE_SHIFT - blkbits));
>
> - if (check_extent_cache(inode, pgofs, &ei)) {
> + if (f2fs_lookup_extent_cache(inode, pgofs, &ei)) {
> f2fs_map_bh(inode, pgofs, &ei, bh_result);
> goto out;
> }
> @@ -792,7 +806,7 @@ int do_write_data_page(struct page *page, struct f2fs_io_info *fio)
> set_inode_flag(F2FS_I(inode), FI_UPDATE_WRITE);
> } else {
> write_data_page(page, &dn, fio);
> - update_extent_cache(&dn);
> + f2fs_update_extent_cache(&dn);
> set_inode_flag(F2FS_I(inode), FI_APPEND_WRITE);
> }
> out_writepage:
> diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h
> index a74ad85..eb9ffae 100644
> --- a/fs/f2fs/f2fs.h
> +++ b/fs/f2fs/f2fs.h
> @@ -1475,7 +1475,7 @@ void f2fs_submit_page_mbio(struct f2fs_sb_info *, struct page *,
> struct f2fs_io_info *);
> int reserve_new_block(struct dnode_of_data *);
> int f2fs_reserve_block(struct dnode_of_data *, pgoff_t);
> -void update_extent_cache(struct dnode_of_data *);
> +void f2fs_update_extent_cache(struct dnode_of_data *);
> struct page *find_data_page(struct inode *, pgoff_t, bool);
> struct page *get_lock_data_page(struct inode *, pgoff_t);
> struct page *get_new_data_page(struct inode *, struct page *, pgoff_t, bool);
> diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c
> index 5df3367..0dcd9fb 100644
> --- a/fs/f2fs/file.c
> +++ b/fs/f2fs/file.c
> @@ -430,7 +430,7 @@ int truncate_data_blocks_range(struct dnode_of_data *dn, int count)
> continue;
>
> dn->data_blkaddr = NULL_ADDR;
> - update_extent_cache(dn);
> + f2fs_update_extent_cache(dn);
> invalidate_blocks(sbi, blkaddr);
> nr_free++;
> }
> diff --git a/fs/f2fs/inline.c b/fs/f2fs/inline.c
> index fa2aa2f..9f6c745 100644
> --- a/fs/f2fs/inline.c
> +++ b/fs/f2fs/inline.c
> @@ -116,7 +116,7 @@ no_update:
> set_page_writeback(page);
> fio.blk_addr = dn->data_blkaddr;
> write_data_page(page, dn, &fio);
> - update_extent_cache(dn);
> + f2fs_update_extent_cache(dn);
> f2fs_wait_on_page_writeback(page, DATA);
> if (dirty)
> inode_dec_dirty_pages(dn->inode);
> diff --git a/fs/f2fs/recovery.c b/fs/f2fs/recovery.c
> index c4211a5..a7adbdb 100644
> --- a/fs/f2fs/recovery.c
> +++ b/fs/f2fs/recovery.c
> @@ -397,7 +397,7 @@ static int do_recover_data(struct f2fs_sb_info *sbi, struct inode *inode,
> /* write dummy data page */
> recover_data_page(sbi, NULL, &sum, src, dest);
> dn.data_blkaddr = dest;
> - update_extent_cache(&dn);
> + f2fs_update_extent_cache(&dn);
> recovered++;
> }
> dn.ofs_in_node++;
> --
> 2.2.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* RE: [f2fs-dev][RFC PATCH 04/10] f2fs: introduce universal lookup/update interface for extent cache
2015-01-23 0:36 ` Jaegeuk Kim
@ 2015-01-23 1:55 ` Chao Yu
0 siblings, 0 replies; 3+ messages in thread
From: Chao Yu @ 2015-01-23 1:55 UTC (permalink / raw)
To: 'Jaegeuk Kim'
Cc: 'Changman Lee', linux-f2fs-devel, linux-kernel
Hi Jaegeuk,
> -----Original Message-----
> From: Jaegeuk Kim [mailto:jaegeuk@kernel.org]
> Sent: Friday, January 23, 2015 8:37 AM
> To: Chao Yu
> Cc: Changman Lee; linux-f2fs-devel@lists.sourceforge.net; linux-kernel@vger.kernel.org
> Subject: Re: [f2fs-dev][RFC PATCH 04/10] f2fs: introduce universal lookup/update interface for
> extent cache
>
> Hi Chao,
>
> On Mon, Jan 12, 2015 at 03:12:13PM +0800, Chao Yu wrote:
> > In this patch, we do these jobs:
> > 1. rename {check,update}_extent_cache to f2fs_{lookup,update}_extent_info;
> > 2. introduce universal lookup/update interface of extent cache:
> > f2fs_{lookup,update}_extent_cache including above two real functions, then
> > export them to function callers.
> >
> > So after above cleanup, we can add new rb-tree based extent cache into exported
> > interfaces.
> >
> > Signed-off-by: Chao Yu <chao2.yu@samsung.com>
> > ---
> > fs/f2fs/data.c | 66 +++++++++++++++++++++++++++++++++---------------------
> > fs/f2fs/f2fs.h | 2 +-
> > fs/f2fs/file.c | 2 +-
> > fs/f2fs/inline.c | 2 +-
> > fs/f2fs/recovery.c | 2 +-
> > 5 files changed, 44 insertions(+), 30 deletions(-)
> >
> > diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
> > index 5a3e489..4f5b871e 100644
> > --- a/fs/f2fs/data.c
> > +++ b/fs/f2fs/data.c
> > @@ -270,20 +270,20 @@ static void f2fs_map_bh(struct inode *inode, pgoff_t pgofs,
> > bh_result->b_size = UINT_MAX;
> > }
> >
> > -static int check_extent_cache(struct inode *inode, pgoff_t pgofs,
> > - struct extent_info *ei)
> > +static bool f2fs_lookup_extent_info(struct inode *inode, pgoff_t pgofs,
> > + struct extent_info *ei)
>
> This is a static function, so I think it would be good to remove "f2fs_".
OK, I will fix this.
>
> > {
> > struct f2fs_inode_info *fi = F2FS_I(inode);
> > pgoff_t start_fofs, end_fofs;
> > block_t start_blkaddr;
> >
> > if (is_inode_flag_set(fi, FI_NO_EXTENT))
> > - return 0;
> > + return false;
> >
> > read_lock(&fi->ext_lock);
> > if (fi->ext.len == 0) {
> > read_unlock(&fi->ext_lock);
> > - return 0;
> > + return false;
> > }
> >
> > stat_inc_total_hit(inode->i_sb);
> > @@ -296,29 +296,22 @@ static int check_extent_cache(struct inode *inode, pgoff_t pgofs,
> > *ei = fi->ext;
> > stat_inc_read_hit(inode->i_sb);
> > read_unlock(&fi->ext_lock);
> > - return 1;
> > + return true;
> > }
> > read_unlock(&fi->ext_lock);
> > - return 0;
> > + return false;
> > }
> >
> > -void update_extent_cache(struct dnode_of_data *dn)
> > +static bool f2fs_update_extent_info(struct inode *inode, pgoff_t fofs,
> > + block_t blkaddr)
>
> Ditto.
I see.
Thanks,
Yu
>
> Thanks,
>
> > {
> > - struct f2fs_inode_info *fi = F2FS_I(dn->inode);
> > - pgoff_t fofs, start_fofs, end_fofs;
> > + struct f2fs_inode_info *fi = F2FS_I(inode);
> > + pgoff_t start_fofs, end_fofs;
> > block_t start_blkaddr, end_blkaddr;
> > int need_update = true;
> >
> > - f2fs_bug_on(F2FS_I_SB(dn->inode), dn->data_blkaddr == NEW_ADDR);
> > -
> > - /* Update the page address in the parent node */
> > - __set_data_blkaddr(dn);
> > -
> > if (is_inode_flag_set(fi, FI_NO_EXTENT))
> > - return;
> > -
> > - fofs = start_bidx_of_node(ofs_of_node(dn->node_page), fi) +
> > - dn->ofs_in_node;
> > + return false;
> >
> > write_lock(&fi->ext_lock);
> >
> > @@ -333,16 +326,16 @@ void update_extent_cache(struct dnode_of_data *dn)
> >
> > /* Initial extent */
> > if (fi->ext.len == 0) {
> > - if (dn->data_blkaddr != NULL_ADDR) {
> > + if (blkaddr != NULL_ADDR) {
> > fi->ext.fofs = fofs;
> > - fi->ext.blk = dn->data_blkaddr;
> > + fi->ext.blk = blkaddr;
> > fi->ext.len = 1;
> > }
> > goto end_update;
> > }
> >
> > /* Front merge */
> > - if (fofs == start_fofs - 1 && dn->data_blkaddr == start_blkaddr - 1) {
> > + if (fofs == start_fofs - 1 && blkaddr == start_blkaddr - 1) {
> > fi->ext.fofs--;
> > fi->ext.blk--;
> > fi->ext.len++;
> > @@ -350,7 +343,7 @@ void update_extent_cache(struct dnode_of_data *dn)
> > }
> >
> > /* Back merge */
> > - if (fofs == end_fofs + 1 && dn->data_blkaddr == end_blkaddr + 1) {
> > + if (fofs == end_fofs + 1 && blkaddr == end_blkaddr + 1) {
> > fi->ext.len++;
> > goto end_update;
> > }
> > @@ -377,9 +370,30 @@ void update_extent_cache(struct dnode_of_data *dn)
> > }
> > end_update:
> > write_unlock(&fi->ext_lock);
> > - if (need_update)
> > + return need_update;
> > +}
> > +
> > +static bool f2fs_lookup_extent_cache(struct inode *inode, pgoff_t pgofs,
> > + struct extent_info *ei)
> > +{
> > + return f2fs_lookup_extent_info(inode, pgofs, ei);
> > +}
> > +
> > +void f2fs_update_extent_cache(struct dnode_of_data *dn)
> > +{
> > + struct f2fs_inode_info *fi = F2FS_I(dn->inode);
> > + pgoff_t fofs;
> > +
> > + f2fs_bug_on(F2FS_I_SB(dn->inode), dn->data_blkaddr == NEW_ADDR);
> > +
> > + /* Update the page address in the parent node */
> > + __set_data_blkaddr(dn);
> > +
> > + fofs = start_bidx_of_node(ofs_of_node(dn->node_page), fi) +
> > + dn->ofs_in_node;
> > +
> > + if (f2fs_update_extent_info(dn->inode, fofs, dn->data_blkaddr))
> > sync_inode_page(dn);
> > - return;
> > }
> >
> > struct page *find_data_page(struct inode *inode, pgoff_t index, bool sync)
> > @@ -625,7 +639,7 @@ static int __get_data_block(struct inode *inode, sector_t iblock,
> > /* Get the page offset from the block offset(iblock) */
> > pgofs = (pgoff_t)(iblock >> (PAGE_CACHE_SHIFT - blkbits));
> >
> > - if (check_extent_cache(inode, pgofs, &ei)) {
> > + if (f2fs_lookup_extent_cache(inode, pgofs, &ei)) {
> > f2fs_map_bh(inode, pgofs, &ei, bh_result);
> > goto out;
> > }
> > @@ -792,7 +806,7 @@ int do_write_data_page(struct page *page, struct f2fs_io_info *fio)
> > set_inode_flag(F2FS_I(inode), FI_UPDATE_WRITE);
> > } else {
> > write_data_page(page, &dn, fio);
> > - update_extent_cache(&dn);
> > + f2fs_update_extent_cache(&dn);
> > set_inode_flag(F2FS_I(inode), FI_APPEND_WRITE);
> > }
> > out_writepage:
> > diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h
> > index a74ad85..eb9ffae 100644
> > --- a/fs/f2fs/f2fs.h
> > +++ b/fs/f2fs/f2fs.h
> > @@ -1475,7 +1475,7 @@ void f2fs_submit_page_mbio(struct f2fs_sb_info *, struct page *,
> > struct f2fs_io_info *);
> > int reserve_new_block(struct dnode_of_data *);
> > int f2fs_reserve_block(struct dnode_of_data *, pgoff_t);
> > -void update_extent_cache(struct dnode_of_data *);
> > +void f2fs_update_extent_cache(struct dnode_of_data *);
> > struct page *find_data_page(struct inode *, pgoff_t, bool);
> > struct page *get_lock_data_page(struct inode *, pgoff_t);
> > struct page *get_new_data_page(struct inode *, struct page *, pgoff_t, bool);
> > diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c
> > index 5df3367..0dcd9fb 100644
> > --- a/fs/f2fs/file.c
> > +++ b/fs/f2fs/file.c
> > @@ -430,7 +430,7 @@ int truncate_data_blocks_range(struct dnode_of_data *dn, int count)
> > continue;
> >
> > dn->data_blkaddr = NULL_ADDR;
> > - update_extent_cache(dn);
> > + f2fs_update_extent_cache(dn);
> > invalidate_blocks(sbi, blkaddr);
> > nr_free++;
> > }
> > diff --git a/fs/f2fs/inline.c b/fs/f2fs/inline.c
> > index fa2aa2f..9f6c745 100644
> > --- a/fs/f2fs/inline.c
> > +++ b/fs/f2fs/inline.c
> > @@ -116,7 +116,7 @@ no_update:
> > set_page_writeback(page);
> > fio.blk_addr = dn->data_blkaddr;
> > write_data_page(page, dn, &fio);
> > - update_extent_cache(dn);
> > + f2fs_update_extent_cache(dn);
> > f2fs_wait_on_page_writeback(page, DATA);
> > if (dirty)
> > inode_dec_dirty_pages(dn->inode);
> > diff --git a/fs/f2fs/recovery.c b/fs/f2fs/recovery.c
> > index c4211a5..a7adbdb 100644
> > --- a/fs/f2fs/recovery.c
> > +++ b/fs/f2fs/recovery.c
> > @@ -397,7 +397,7 @@ static int do_recover_data(struct f2fs_sb_info *sbi, struct inode *inode,
> > /* write dummy data page */
> > recover_data_page(sbi, NULL, &sum, src, dest);
> > dn.data_blkaddr = dest;
> > - update_extent_cache(&dn);
> > + f2fs_update_extent_cache(&dn);
> > recovered++;
> > }
> > dn.ofs_in_node++;
> > --
> > 2.2.1
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-01-23 1:56 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-12 7:12 [f2fs-dev][RFC PATCH 04/10] f2fs: introduce universal lookup/update interface for extent cache Chao Yu
2015-01-23 0:36 ` Jaegeuk Kim
2015-01-23 1:55 ` Chao Yu
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).