Hi Jaegeuk,
Sorry for the delay.
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Friday, May 01, 2015 2:04 AM
To: Chao Yu
Cc: Changman Lee; linux-f2fs-de...@lists.sourceforge.net;
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 1/3] f2fs: introduce
Introduce a generic function replace_block base on recover_data_page,
and export it. So with it we can operate file's meta data which is in
CP/SSA area when we invoke fallocate with FALLOC_FL_COLLAPSE_RANGE
flag.
Signed-off-by: Chao Yu chao2...@samsung.com
---
v3:
* rename replace_block
and ext4.
4) Collaspe range does not work beyond i_size.
This patch implements fallocate's FALLOC_FL_COLLAPSE_RANGE for f2fs.
Signed-off-by: Chao Yu chao2...@samsung.com
---
v2:
* rebase to last git repository of f2fs (20150430)
* remove replace_block() introduction.
v3:
* rebase to last git
a hole of len bytes at offset.
This patch implements fallocate's FALLOC_FL_ZERO_RANGE for f2fs.
Signed-off-by: Chao Yu chao2...@samsung.com
---
v2:
* rebase to last git repository of f2fs (20150430)
v3:
* rebase to last git repository of f2fs (20150506)
fs/f2fs/file.c | 106
This patch fixes memory leak issue in error path of f2fs_fname_setup_filename().
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/crypto_fname.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/fs/f2fs/crypto_fname.c b/fs/f2fs/crypto_fname.c
index e3a1bdc
memory space which is pointed by other invalid field in union
structure for each ctx.
Then the error occurs, let's fix it with this patch.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/crypto.c | 8
1 file changed, 8 deletions(-)
diff --git a/fs/f2fs/crypto.c b/fs/f2fs/crypto.c
Add missed f2fs_balance_fs to reserve space for -tmpfile.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/namei.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/fs/f2fs/namei.c b/fs/f2fs/namei.c
index bed0cb0..38a783e 100644
--- a/fs/f2fs/namei.c
+++ b/fs/f2fs/namei.c
@@ -517,6
register_shrinker() in ubifs_init() can fail due to fail to call kzalloc.
This patch fixes to check the return value of register_shrinker, otherwise
our shrinker may be unregistered after ubifs initialized successfully.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/ubifs/super.c | 5
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Tuesday, May 19, 2015 1:51 PM
To: Chao Yu
Cc: Changman Lee; linux-f2fs-de...@lists.sourceforge.net;
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/2] f2fs: support RENAME_WHITEOUT
Hi Chao
This patch tries to clean up code as part code of f2fs_read_end_io
and mpage_end_io are the same, so it's better to merge and reuse them.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/data.c | 35 +--
1 file changed, 5 insertions(+), 30 deletions
, we just try to follow the way that xfs/ext4 use.
Signed-off-by: Chao Yu chao2...@samsung.com
---
v2:
* fix to release reference of whiteout inode after being linkated.
* add f2fs_balance_fs for -tmpfile().
fs/f2fs/namei.c | 153
1 file
Add a help function update_meta_page() to update meta page with specified
buffer.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/checkpoint.c | 26 ++
fs/f2fs/f2fs.h | 1 +
fs/f2fs/segment.c| 26 --
3 files changed, 23
, we just try to follow the way that xfs/ext4 use.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/namei.c | 140
1 file changed, 90 insertions(+), 50 deletions(-)
diff --git a/fs/f2fs/namei.c b/fs/f2fs/namei.c
index 16b74da..bed0cb0
Now page cache of meta inode is used by garbage collection for encrypted page,
it may contain random data, so we should zero it before issuing discard.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/segment.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/f2fs
As Rafael suggested, docking station driver is maintained along with the
ACPI core nowadays, so it's better to remove this entry.
Signed-off-by: Chao Yu chao2...@samsung.com
---
MAINTAINERS | 6 --
1 file changed, 6 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index a36be4e..4d5c09d
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Thursday, April 16, 2015 3:31 PM
To: linux-kernel@vger.kernel.org; linux-fsde...@vger.kernel.org;
linux-f2fs-de...@lists.sourceforge.net
Cc: Jaegeuk Kim
Subject: [f2fs-dev] [PATCH 1/3] f2fs: flush
and ext4.
4) Collaspe range does not work beyond i_size.
This patch implements fallocate's FALLOC_FL_COLLAPSE_RANGE for f2fs.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/f2fs.h| 2 +
fs/f2fs/file.c| 130 +-
fs/f2fs/segment.c
safer, so that we can avoid potential bug in its caller,
especially for ocfs2.
Let's back port the patch to f2fs.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/acl.c | 46 --
1 file changed, 20 insertions(+), 26 deletions(-)
diff --git a/fs
Export is_valid_blkaddr() and use it to replace some codes for readability.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/checkpoint.c | 3 +--
fs/f2fs/f2fs.h | 1 +
fs/f2fs/recovery.c | 6 +++---
3 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/fs/f2fs
has_fsynced_inode() has no other caller out of node.c, make it static.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/f2fs.h | 1 -
fs/f2fs/node.c | 2 +-
2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h
index 5cf7e9f..9fd730a 100644
--- a/fs
) where x i_size will not trim all blocks past i_size.
This method is used in xfs, and then ext4/btrfs will follow the rule.
This patch fixes to follow the new rule for f2fs.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/file.c | 8
1 file changed, 4 insertions(+), 4
) where x i_size will not trim all blocks past i_size.
This method is used in xfs, and then ext4/btrfs will follow the rule.
This patch fixes to follow the new rule for f2fs.
Signed-off-by: Chao Yu chao2...@samsung.com
---
v2:
* adjust code style pointed out by Jaegeuk Kim.
fs/f2fs/file.c
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Tuesday, June 09, 2015 2:07 AM
To: Chao Yu
Cc: linux-f2fs-de...@lists.sourceforge.net; linux-kernel@vger.kernel.org
Subject: Re: [PATCH] f2fs: do not trim preallocated blocks when truncating
after i_size
In recovery procedure for superblock, we try to write data of valid
superblock into invalid one for recovery, work should be finished here,
but then still we will write the valid one with its original data.
This operation is not needed. Let's skip doing this unnecessary work.
Signed-off-by: Chao
.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/super.c | 20 +---
1 file changed, 9 insertions(+), 11 deletions(-)
diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c
index fbbf785..a06b0b4 100644
--- a/fs/f2fs/super.c
+++ b/fs/f2fs/super.c
@@ -258,6 +258,7 @@ static void
Hi Jaegeuk,
-Original Message-
From: Chao Yu [mailto:chao2...@samsung.com]
Sent: Tuesday, June 09, 2015 10:28 AM
To: Jaegeuk Kim; Changman Lee
Cc: linux-kernel@vger.kernel.org; linux-f2fs-de...@lists.sourceforge.net
Subject: [f2fs-dev] [PATCH v2] f2fs: do not trim preallocated
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Friday, May 29, 2015 10:29 AM
To: Chao Yu
Cc: linux-kernel@vger.kernel.org; linux-f2fs-de...@lists.sourceforge.net
Subject: Re: [f2fs-dev] [PATCH 1/3] f2fs crypto: check context consistent
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Thursday, May 28, 2015 3:10 AM
To: Chao Yu
Cc: linux-kernel@vger.kernel.org; linux-fsde...@vger.kernel.org;
linux-f2fs-de...@lists.sourceforge.net
Subject: Re: [f2fs-dev] [PATCH 2/3] f2fs crypto: use
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Wednesday, May 20, 2015 8:43 AM
To: linux-kernel@vger.kernel.org; linux-fsde...@vger.kernel.org;
linux-f2fs-de...@lists.sourceforge.net
Cc: Jaegeuk Kim
Subject: [f2fs-dev] [PATCH 3/4] f2fs crypto:
patch.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/f2fs.h | 4 ++--
fs/f2fs/file.c | 12 ++--
fs/f2fs/recovery.c | 9 ++---
fs/f2fs/segment.c | 17 -
4 files changed, 22 insertions(+), 20 deletions(-)
diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h
we can easily create
a hole for inserting without removing original data.
This patch introduces f2fs_insert_range() to support FALLOC_FL_INSERT_RANGE.
Signed-off-by: Chao Yu chao2...@samsung.com
Signed-off-by: Yuan Zhong yuan.mark.zh...@samsung.com
---
fs/f2fs/file.c | 102
Hi all,
-Original Message-
From: Chao Yu [mailto:chao2...@samsung.com]
Sent: Monday, May 25, 2015 6:08 PM
To: Jaegeuk Kim; Changman Lee
Cc: linux-kernel@vger.kernel.org; linux-f2fs-de...@lists.sourceforge.net
Subject: [f2fs-dev] [PATCH 2/3] f2fs crypto: check dir entry just
entries
This patch registers shrinking nat_cache entries.
Signed-off-by: Jaegeuk Kim jaeg...@kernel.org
Reviewed-by: Chao Yu chao2...@samsung.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Wednesday, July 01, 2015 9:33 AM
To: Chao Yu; Chao Yu
Cc: linux-kernel@vger.kernel.org; linux-f2fs-de...@lists.sourceforge.net;
linux-kernel@vger.kernel.org; linux-f2fs-de...@lists.sourceforge.net
))
seq_puts(seq, ,extent_cache);
+else
+ seq_puts(seq, ,noextent_cache);
Signed-off-by: Jaegeuk Kim jaeg...@kernel.org
The other looks good to me, so please add:
Reviewed-by: Chao Yu chao2...@samsung.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel
From e5c6600d01c4462c4e1ee0c70ec1d9319862077d Mon Sep 17 00:00:00 2001
From: Chao Yu chao2...@samsung.com
Date: Thu, 2 Jul 2015 18:52:46 +0800
Subject: [PATCH] f2fs: reduce lock overhead of extent node releasing
Open and close critical section for each extent node when traversing rb-tree
results
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Wednesday, July 01, 2015 9:26 AM
To: linux-kernel@vger.kernel.org; linux-fsde...@vger.kernel.org;
linux-f2fs-de...@lists.sourceforge.net; linux-kernel@vger.kernel.org;
linux-fsde...@vger.kernel.org;
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Tuesday, June 30, 2015 2:40 AM
To: linux-kernel@vger.kernel.org; linux-fsde...@vger.kernel.org;
linux-f2fs-de...@lists.sourceforge.net
Cc: Jaegeuk Kim
Subject: [f2fs-dev] [PATCH 12/12] f2fs: use
entries
This patch registers shrinking extent_caches.
Signed-off-by: Jaegeuk Kim jaeg...@kernel.org
Reviewed-by: Chao Yu chao2...@samsung.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Tuesday, June 30, 2015 2:40 AM
To: linux-kernel@vger.kernel.org; linux-fsde...@vger.kernel.org;
linux-f2fs-de...@lists.sourceforge.net
Cc: Jaegeuk Kim
Subject: [f2fs-dev] [PATCH 08/12] f2fs: introduce a shrinker
of relying on extent_tree, we can simply check the cached one in
extent
tree accordingly.
Signed-off-by: Jaegeuk Kim jaeg...@kernel.org
Reviewed-by: Chao Yu chao2...@samsung.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Wednesday, July 01, 2015 9:26 AM
To: Chao Yu; Chao Yu
Cc: linux-kernel@vger.kernel.org; linux-f2fs-de...@lists.sourceforge.net;
linux-kernel@vger.kernel.org; linux-f2fs-de...@lists.sourceforge.net
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Saturday, July 04, 2015 1:16 PM
To: Chao Yu
Cc: linux-fsde...@vger.kernel.org; linux-kernel@vger.kernel.org;
linux-f2fs-de...@lists.sourceforge.net
Subject: Re: [f2fs-dev] [PATCH 12/12] f2fs: use
.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/checkpoint.c | 3 ++-
fs/f2fs/f2fs.h | 3 ++-
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/fs/f2fs/checkpoint.c b/fs/f2fs/checkpoint.c
index b70bbe1..de7a0d6 100644
--- a/fs/f2fs/checkpoint.c
+++ b/fs/f2fs/checkpoint.c
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Saturday, July 04, 2015 2:29 PM
To: Chao Yu
Cc: linux-fsde...@vger.kernel.org; linux-kernel@vger.kernel.org;
linux-f2fs-de...@lists.sourceforge.net
Subject: Re: [f2fs-dev] [PATCH 2/2] f2fs: shrink unreferenced
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Saturday, July 04, 2015 3:04 PM
To: Chao Yu
Cc: linux-kernel@vger.kernel.org; linux-f2fs-de...@lists.sourceforge.net
Subject: Re: [f2fs-dev] [PATCH] f2fs: reduce lock overhead of extent node
releasing
On Thu
this change, it will reduce the probability we set the
wrong cold file, also it helps a little for fs_mark's performance
on f2fs.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/namei.c | 9 -
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/fs/f2fs/namei.c b/fs/f2fs/namei.c
0m0.072s
sys 0m10.760s
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/data.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
index 6cf75d3..37a2632 100644
--- a/fs/f2fs/data.c
+++ b/fs/f2fs/data.c
@@ -1703,11 +1703,10 @@ static int
This patch adds missed trace file in maintainer-ship of f2fs,
so it completes the description of files maintained in f2fs,
and also it allows people to find correct mailing list by using
get_maintainer.pl when only patching the trace file of f2fs.
Signed-off-by: Chao Yu chao2...@samsung.com
-Original Message-
From: Chao Yu [mailto:yuchaoch...@hotmail.com]
Sent: Saturday, July 04, 2015 2:30 PM
To: 'Jaegeuk Kim'
Cc: linux-fsde...@vger.kernel.org; linux-kernel@vger.kernel.org;
linux-f2fs-de...@lists.sourceforge.net
Subject: Re: [f2fs-dev] [PATCH 12/12] f2fs: use
efficiently.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/data.c | 113 ++---
fs/f2fs/f2fs.h | 6 +++
2 files changed, 73 insertions(+), 46 deletions(-)
diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
index e90522a..e96916a 100644
This patch fixes to add a judgement condition to verify type of the
target inode before empty directory verification.
So wrong verification could be avoided for non-directory inode.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/crypto_policy.c | 2 +-
1 file changed, 1 insertion(+), 1
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Friday, May 22, 2015 8:40 AM
To: linux-kernel@vger.kernel.org; linux-fsde...@vger.kernel.org;
linux-f2fs-de...@lists.sourceforge.net
Cc: Jaegeuk Kim
Subject: [f2fs-dev] [PATCH 2/3] f2fs crypto: use
This patch add XATTR_CREATE flag in setxattr when setting encryption
context for inode. Without this flag the context could be set more than
once, this should never happen. So, fix it.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/crypto_policy.c | 4 ++--
1 file changed, 2 insertions
This patch tries to clean up code because part code of f2fs_read_end_io
and mpage_end_io are the same, so it's better to merge and reuse them.
Signed-off-by: Chao Yu chao2...@samsung.com
---
v2:
* do not merge 'f2fs_end_io_crypto_work' and 'return' to indicate
f2fs_end_io_crypto_work
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Wednesday, May 20, 2015 1:49 AM
To: Chao Yu
Cc: Changman Lee; linux-f2fs-de...@lists.sourceforge.net;
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/2] f2fs: avoid duplicated code by reusing
For exchange rename, we should check context consistent of encryption
between new_dir and old_inode or old_dir and new_inode. Otherwise
inheritance of parent's encryption context will be broken.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/namei.c | 9 +
1 file changed, 9
Ping.
Add Cc Richard Weinberger.
-Original Message-
From: linux-mtd [mailto:linux-mtd-boun...@lists.infradead.org] On Behalf Of
Chao Yu
Sent: Monday, May 18, 2015 4:43 PM
To: dedeki...@gmail.com; adrian.hun...@intel.com
Cc: linux-...@lists.infradead.org; linux-kernel
On Aug 8, 2015, at 11:12 AM, Nicholas Krause xerofo...@gmail.com wrote:
This fixes concurrent access issues in the function recover_dentry
by locking and locking another the block labeled retry with the
functions f2fs_lock_op/f2fs_unlock_op on a pointer of the
superblock used by the passed
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
This patch splits __insert_extent_tree_ret into __try_merge_extent_node
__insert_extent_tree for code readability.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/extent_cache.c | 49 +
1 file changed, 29 insertions(+), 20 deletions
)
Inner Struct Count:
tree count, node count.
Before:
Extent Hit Ratio: 0 / 2
Extent Tree Count: 3
Extent Node Count: 2
Patched:
Exten Cacache:
- Hit Count: L1-1:4871 L1-2:2074 L2:208
- Hit Ratio: 1% (7153 / 550751)
- Inner Struct Count: tree: 26560, node: 11824
Signed-off-by: Chao Yu
This patch adds to stat the hit count of largest/cached node for showing
in debugfs.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/debug.c| 9 +++--
fs/f2fs/extent_cache.c | 14 +-
fs/f2fs/f2fs.h | 8 +++-
3 files changed, 23 insertions(+), 8
of
get_data() nor check mapping info of buffer_head, result in returning
the random block address.
After fixing the issue, our result shows correctly:
file_pos start_blk end_blkblks
0 0 0 256
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs
:
- Hit Count: L1-1:2048 L1-2:0 L2:0
- Hit Ratio: 33% (2048 / 6144)
- Inner Struct Count: tree: 1, node: 961
This patch fixes to lookup neighbors of target node for further
merging.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/extent_cache.c | 34
When converting inline dentry, we will zero out target dentry page before
duplicating data of inline dentry into target page, it become overhead
since inline dentry size is not small.
So this patch tries to remove unneeded initializing in the space of target
dentry page.
Signed-off-by: Chao Yu
.
This patch removes these dead codes, besides, rename __insert_extent_tree
to __init_extent_tree for readability.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/extent_cache.c | 82 --
1 file changed, 6 insertions(+), 76 deletions(-)
diff
This patch introduce a new helper f2fs_update_extent_tree_range
which can update extent nodes in extent tree in batches.
Now, we use the function to invalidate blocks in batches instead of
invalidating them one by one when truncating blocks.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs
Add annotation to let us know more clearly about space utilization
information of regular dentry and inline dentry.
Signed-off-by: Chao Yu chao2...@samsung.com
---
include/linux/f2fs_fs.h | 16 +---
1 file changed, 13 insertions(+), 3 deletions(-)
diff --git a/include/linux
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Saturday, August 15, 2015 7:09 AM
To: linux-kernel@vger.kernel.org; linux-fsde...@vger.kernel.org;
linux-f2fs-de...@lists.sourceforge.net
Cc: Jaegeuk Kim
Subject: [f2fs-dev] [PATCH 1/2] f2fs: handle
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Friday, August 21, 2015 1:48 AM
To: Chao Yu
Cc: linux-f2fs-de...@lists.sourceforge.net; linux-kernel@vger.kernel.org
Subject: Re: [PATCH 9/9] f2fs: update extent tree in batches
Hi Chao,
On Wed
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Thursday, August 20, 2015 11:35 PM
To: Chao Yu
Cc: linux-kernel@vger.kernel.org; linux-fsde...@vger.kernel.org;
linux-f2fs-de...@lists.sourceforge.net
Subject: Re: [f2fs-dev] [PATCH 5/5] f2fs: check
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Friday, August 21, 2015 1:17 AM
To: Chao Yu
Cc: linux-f2fs-de...@lists.sourceforge.net; linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2/9] f2fs: avoid unneeded initializing when converting
inline
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Thursday, August 20, 2015 11:57 PM
To: Chao Yu
Cc: linux-kernel@vger.kernel.org; linux-fsde...@vger.kernel.org;
linux-f2fs-de...@lists.sourceforge.net
Subject: Re: [f2fs-dev] [PATCH 1/2] f2fs: handle failed bio
On Aug 21, 2015, at 8:48 PM, Chao Yu chao2...@samsung.com wrote:
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Thursday, August 20, 2015 11:35 PM
To: Chao Yu
Cc: linux-kernel@vger.kernel.org; linux-fsde...@vger.kernel.org;
linux-f2fs-de
aggressively
If we can reuse nids as many as possible, we can mitigate producing obsolete
node pages in the page cache.
Signed-off-by: Jaegeuk Kim jaeg...@kernel.org
Reviewed-by: Chao Yu chao2...@samsung.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body
Reviewed-by: Chao Yu chao2...@samsung.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
collecting already moved
node blocks
If node blocks were already moved, we don't need to move them again.
Signed-off-by: Jaegeuk Kim jaeg...@kernel.org
Reviewed-by: Chao Yu chao2...@samsung.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Tuesday, August 18, 2015 4:46 PM
To: linux-kernel@vger.kernel.org; linux-fsde...@vger.kernel.org;
linux-f2fs-de...@lists.sourceforge.net
Cc: Jaegeuk Kim
Subject: [f2fs-dev] [PATCH 5/5] f2fs: check
for insert_inode_locked failure
We should not call unlock_new_inode when insert_inode_locked failed.
Signed-off-by: Jaegeuk Kim jaeg...@kernel.org
Reviewed-by: Chao Yu chao2...@samsung.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Monday, August 24, 2015 12:54 PM
To: Chao Yu
Cc: linux-kernel@vger.kernel.org; linux-fsde...@vger.kernel.org;
linux-f2fs-de...@lists.sourceforge.net
Subject: Re: [f2fs-dev] [PATCH 1/2] f2fs: handle
When converting inline dentry, we will zero out target dentry page before
duplicating data of inline dentry into target page, it become overhead
since inline dentry size is not small.
So this patch tries to remove unneeded initializing in the space of target
dentry page.
Signed-off-by: Chao Yu
Hi Jaegeuk,
-Original Message-
From: Chao Yu [mailto:yuchaoch...@hotmail.com]
Sent: Friday, August 21, 2015 11:00 PM
To: Chao Yu
Cc: Jaegeuk Kim; linux-fsde...@vger.kernel.org; linux-kernel@vger.kernel.org;
linux-f2fs-de...@lists.sourceforge.net
Subject: Re: [f2fs-dev] [PATCH 5/5
This patch fixes to return error number of f2fs_truncate, so that we
can handle the error correctly in callers.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/f2fs.h | 2 +-
fs/f2fs/file.c | 26 +-
2 files changed, 18 insertions(+), 10 deletions(-)
diff --git
list if we fail to truncate
inode, so that we can obtain a second chance to release it in orphan
recovery flow.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/f2fs.h | 2 +-
fs/f2fs/inode.c | 53 ++---
fs/f2fs/node.c | 14
for
update_inode_page
Previously, update_inode_page is not called under f2fs_lock_op.
Instead we should call with f2fs_write_inode.
Signed-off-by: Jaegeuk Kim jaeg...@kernel.org
Reviewed-by: Chao Yu chao2...@samsung.com
--
To unsubscribe from this list: send the line unsubscribe linux
information
The write_checkpoint can update stat information, so we should destroy the
stat
structure after it.
Signed-off-by: Jaegeuk Kim jaeg...@kernel.org
Reviewed-by: Chao Yu chao2...@samsung.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body
wrong block address calculation
for a split extent
This patch fixes wrong calculation on block address field when an extent is
split.
Oh, that's a terrible mistake, sorry about it.
Thank you for fixing this.
Signed-off-by: Jaegeuk Kim jaeg...@kernel.org
Reviewed-by: Chao Yu chao2
for various
fallocate
For newly added fallocate types, it should convert inline_data before handling
block swapping.
Signed-off-by: Jaegeuk Kim jaeg...@kernel.org
Reviewed-by: Chao Yu chao2...@samsung.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body
checking finally
This patch relocates cached_en not only to be covered by spin_lock, but also
to set once after checking out completely.
Signed-off-by: Jaegeuk Kim jaeg...@kernel.org
Reviewed-by: Chao Yu chao2...@samsung.com
--
To unsubscribe from this list: send the line unsubscribe linux
inode immediately
Before iput is called, the inode number used by a bad inode can be reassigned
to other new inode, resulting in any abnormal behaviors on the new inode.
This should not happen for the new inode.
Nice catch!
Reviewed-by: Chao Yu chao2...@samsung.com
--
To unsubscribe from
. f2fs_evict_inode; on-disk extent is X
4. open update the extent X with Y
5. sync; trigger checkpoint
6. power-cut
after power-on, f2fs should serve extent Y, but we have an on-disk extent X.
This causes a failure on xfstests/311.
Signed-off-by: Jaegeuk Kim jaeg...@kernel.org
Reviewed-by: Chao Yu
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Tuesday, June 30, 2015 2:39 AM
To: linux-kernel@vger.kernel.org; linux-fsde...@vger.kernel.org;
linux-f2fs-de...@lists.sourceforge.net
Cc: Jaegeuk Kim
Subject: [f2fs-dev] [PATCH 04/12] f2fs: remove
that the basic implementation was copied from fs/btrfs/shrinker.c
Great! Good to see it's being implemented in f2fs.
Signed-off-by: Jaegeuk Kim jaeg...@kernel.org
Reviewed-by: Chao Yu chao2...@samsung.com
[snip]
@@ -1406,6 +1425,9 @@ static int __init init_f2fs_fs(void)
err
This patch moves extent cache related code from data.c into extent_cache.c
since extent cache is independent feature, and its code is not relate to
others in data.c, it's better for us to maintain it in separated place.
Certainly there is no functionality change.
Signed-off-by: Chao Yu chao2
fs readonly and
and stop doing checkpoint.
Signed-off-by: Chao Yu chao2...@samsung.com
---
fs/f2fs/checkpoint.c | 8
1 file changed, 8 insertions(+)
diff --git a/fs/f2fs/checkpoint.c b/fs/f2fs/checkpoint.c
index 84e0dfe..c311176 100644
--- a/fs/f2fs/checkpoint.c
+++ b/fs/f2fs/checkpoint.c
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Tuesday, July 28, 2015 11:57 PM
To: Chao Yu
Cc: linux-kernel@vger.kernel.org; linux-fsde...@vger.kernel.org;
linux-f2fs-de...@lists.sourceforge.net
Subject: Re: [f2fs-dev] [PATCH 2/2] f2fs: handle
Hi Jaegeuk,
-Original Message-
From: Jaegeuk Kim [mailto:jaeg...@kernel.org]
Sent: Sunday, July 26, 2015 8:21 AM
To: linux-kernel@vger.kernel.org; linux-fsde...@vger.kernel.org;
linux-f2fs-de...@lists.sourceforge.net
Cc: Jaegeuk Kim
Subject: [f2fs-dev] [PATCH 2/2] f2fs: handle
[8149deea] __percpu_counter_add+0x1a/0x90
RSP 880295143ac8
CR2: 00a8
---[ end trace 5132449a58ed93a3 ]---
note: gcc[10356] exited with preempt_count 2
Signed-off-by: Jaegeuk Kim jaeg...@kernel.org
Good catch!
Reviewed-by: Chao Yu chao2...@samsung.com
BTW, is this commit
This patch introduces __count_free_nids/try_to_free_nids and registers
them in slab shrinker for shrinking under memory pressure.
Signed-off-by: Chao Yu chao2...@samsung.com
---
v2:
* fix wrong bug_on in try_to_free_nids.
fs/f2fs/f2fs.h | 1 +
fs/f2fs/node.c | 28
501 - 600 of 5204 matches
Mail list logo