lkml.org 
[lkml]   [2015]   [Sep]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/3] f2fs: introduce __try_update_largest_extent
Hi Chao,

On Fri, Sep 18, 2015 at 04:53:07PM +0800, Chao Yu wrote:
> This patch adds a new helper __try_update_largest_extent for cleanup.
>
> Signed-off-by: Chao Yu <chao2.yu@samsung.com>
> ---
> fs/f2fs/extent_cache.c | 14 +++++---------
> fs/f2fs/f2fs.h | 7 +++++++
> 2 files changed, 12 insertions(+), 9 deletions(-)
>
> diff --git a/fs/f2fs/extent_cache.c b/fs/f2fs/extent_cache.c
> index c9d1cfd..31d5b48 100644
> --- a/fs/f2fs/extent_cache.c
> +++ b/fs/f2fs/extent_cache.c
> @@ -351,8 +351,7 @@ static struct extent_node *__try_merge_extent_node(struct f2fs_sb_info *sbi,
> }
>
> if (en) {
> - if (en->ei.len > et->largest.len)
> - et->largest = en->ei;
> + __try_update_largest_extent(&et->largest, &en->ei);
> et->cached_en = en;
> }
> return en;
> @@ -389,8 +388,7 @@ do_insert:
> if (!en)
> return NULL;
>
> - if (en->ei.len > et->largest.len)
> - et->largest = en->ei;
> + __try_update_largest_extent(&et->largest, &en->ei);
> et->cached_en = en;
> return en;
> }
> @@ -476,12 +474,10 @@ static unsigned int f2fs_update_extent_tree_range(struct inode *inode,
> : NULL;
> }
>
> - if (parts) {
> - if (en->ei.len > et->largest.len)
> - et->largest = en->ei;
> - } else {
> + if (parts)
> + __try_update_largest_extent(&et->largest, &en->ei);
> + else
> __detach_extent_node(sbi, et, en);
> - }
>
> /*
> * if original extent is split into zero or two parts, extent
> diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h
> index dbceb35..3342619 100644
> --- a/fs/f2fs/f2fs.h
> +++ b/fs/f2fs/f2fs.h
> @@ -498,6 +498,13 @@ static inline bool __is_front_mergeable(struct extent_info *cur,
> return __is_extent_mergeable(cur, front);
> }
>
> +static inline void __try_update_largest_extent(struct extent_info *largest,
> + struct extent_info *cur)

How about __try_update_largest_extent(et, en) ?

> +{
> + if (cur->len > largest->len)
> + *largest = *cur;
> +}
> +
> struct f2fs_nm_info {
> block_t nat_blkaddr; /* base disk address of NAT */
> nid_t max_nid; /* maximum possible node ids */
> --
> 2.5.2


\
 
 \ /
  Last update: 2015-09-21 21:01    [W:0.049 / U:0.900 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site