[f2fs-dev] [PATCH v2] f2fs: split free nid list

2016-10-12 Thread Chao Yu
During free nid allocation, in order to do preallocation, we will tag free nid entry as allocated one and still leave it in free nid list, for other allocators who want to grab free nids, it needs to traverse the free nid list for lookup. It becomes overhead in scenario of allocating free nid inten

Re: [f2fs-dev] [PATCH 3/7] f2fs: rename free nid cache operation

2016-10-12 Thread Chao Yu
Hi Jaegeuk, On 2016/10/12 1:19, Jaegeuk Kim wrote: > Hi Chao, > > On Tue, Oct 11, 2016 at 10:31:32PM +0800, Chao Yu wrote: >> From: Chao Yu >> >> Rename free nid cache operation for readability, no functionality change. > > Well, I don't think this can be a *cache*, since there is no cache-rela

Re: [f2fs-dev] [PATCH 1/8] f2fs: clear nlink if fail to add_link

2016-10-12 Thread Chao Yu
Hi Jaegeuk, On 2016/10/12 6:19, Jaegeuk Kim wrote: > Hi Chao, > > On Tue, Oct 11, 2016 at 10:56:59PM +0800, Chao Yu wrote: >> From: Chao Yu >> >> We don't need to keep incomplete created inode in cache, so if we fail to >> add link into directory during new inode creation, it's better to set >>

[f2fs-dev] [RFC PATCH 1/2] f2fs: rename f2fs_update_time

2016-10-12 Thread Chao Yu
From: Chao Yu For readability, no functionality change. Signed-off-by: Chao Yu --- fs/f2fs/checkpoint.c | 2 +- fs/f2fs/data.c | 2 +- fs/f2fs/dir.c| 6 +++--- fs/f2fs/f2fs.h | 2 +- fs/f2fs/file.c | 24 fs/f2fs/super.c | 4 ++-- fs/

[f2fs-dev] [RFC PATCH 2/2] f2fs: fix allocation failure

2016-10-12 Thread Chao Yu
From: Chao Yu tests/generic/251 of fstest reports a f2fs bug in below message: [ cut here ] invalid opcode: [#1] PREEMPT SMP CPU: 1 PID: 109 Comm: kworker/u8:2 Tainted: GW O4.8.0-rc4+ #22 Hardware name: innotek GmbH VirtualBox/VirtualBox, BIOS VirtualBox

Re: [f2fs-dev] [PATCH 1/8] f2fs: clear nlink if fail to add_link

2016-10-12 Thread Jaegeuk Kim
On Wed, Oct 12, 2016 at 11:24:28PM +0800, Chao Yu wrote: > Hi Jaegeuk, > > On 2016/10/12 6:19, Jaegeuk Kim wrote: > > Hi Chao, > > > > On Tue, Oct 11, 2016 at 10:56:59PM +0800, Chao Yu wrote: > >> From: Chao Yu > >> > >> We don't need to keep incomplete created inode in cache, so if we fail to >

Re: [f2fs-dev] [PATCH 3/7] f2fs: rename free nid cache operation

2016-10-12 Thread Jaegeuk Kim
On Wed, Oct 12, 2016 at 11:14:42PM +0800, Chao Yu wrote: > Hi Jaegeuk, > > On 2016/10/12 1:19, Jaegeuk Kim wrote: > > Hi Chao, > > > > On Tue, Oct 11, 2016 at 10:31:32PM +0800, Chao Yu wrote: > >> From: Chao Yu > >> > >> Rename free nid cache operation for readability, no functionality change. >

[f2fs-dev] [PATCH] f2fs: fix wrong sum_page pointer in f2fs_gc

2016-10-12 Thread Jaegeuk Kim
This patch fixes using a wrong pointer for sum_page in f2fs_gc. Signed-off-by: Jaegeuk Kim --- fs/f2fs/gc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c index e48142f..9c18917 100644 --- a/fs/f2fs/gc.c +++ b/fs/f2fs/gc.c @@ -854,16 +8

Re: [f2fs-dev] [PATCH] fscrypto: make XTS tweak initialization endian-independent

2016-10-12 Thread Theodore Ts'o
On Fri, Sep 30, 2016 at 7:58 PM, Eric Biggers wrote: > The XTS tweak (or IV) was initialized differently on little endian and > big endian systems. Because the ciphertext depends on the XTS tweak, it > was not possible to use an encrypted filesystem created by a little > endian system on a big en