Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

2016-01-05 Thread Jaegeuk Kim
Hi, ... > > > > > > > > Nope, what I'm talking about is *correctness* of our > > > > > > > > ->fiemap interface, but you're trying to avoid it by saying > > > > > > > > "support more > > > > > > > cases, > > > > > > > > it's an improvement". That doesn't make any sense to me, since > > > > > > >

Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

2016-01-05 Thread Chao Yu
> -Original Message- > From: Jaegeuk Kim [mailto:jaeg...@kernel.org] > Sent: Tuesday, January 05, 2016 11:30 AM > To: Chao Yu > Cc: 'Fan Li'; linux-f2fs-devel@lists.sourceforge.net > Subject: Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding

Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

2016-01-04 Thread Fan Li
> -Original Message- > From: Jaegeuk Kim [mailto:jaeg...@kernel.org] > Sent: Tuesday, January 05, 2016 11:08 AM > To: Fan Li; ""@jaegeuk-2.local > Cc: 'Chao Yu'; linux-f2fs-devel@lists.sourceforge.net > Subject: Re: [f2fs-dev] [PATCH 2/3] f2f

Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

2016-01-04 Thread Chao Yu
> -Original Message- > From: Fan Li [mailto:fanofcode...@samsung.com] > Sent: Monday, January 04, 2016 6:55 PM > To: 'Chao Yu'; 'Jaegeuk Kim' > Cc: linux-f2fs-devel@lists.sourceforge.net > Subject: RE: [f2fs-dev] [PATCH 2/3] f2f

Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

2016-01-04 Thread Jaegeuk Kim
Hi, ... > > > > Nope, what I'm talking about is *correctness* of our ->fiemap > > > > interface, but you're trying > > to avoid it by saying "support more > > > cases, > > > > it's an improvement". That doesn't make any sense to me, since > > > > correctness issue still not > > be fixed. > > >

Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

2016-01-04 Thread Chao Yu
Hi all, > -Original Message- > From: Jaegeuk Kim [mailto:jaeg...@kernel.org] > Sent: Monday, January 04, 2016 2:56 PM > To: Fan Li > Cc: 'Chao Yu'; linux-f2fs-devel@lists.sourceforge.net > Subject: Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents aft

Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

2016-01-04 Thread Jaegeuk Kim
Hi, > > > > > > > > On 12/30/15 5:17 PM, Fan Li wrote: > > > > > > > > > f2fs allows preallocation beyond isize, but f2fs_fiemap > > > > > > > > > only look up extents within isize. Therefore add this support. > > > > > > > > > > > > > > > > > > Note: It's possible that there are holes after isize

Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

2016-01-04 Thread Fan Li
> -Original Message- > From: Jaegeuk Kim [mailto:jaeg...@kernel.org] > Sent: Tuesday, January 05, 2016 7:14 AM > To: Fan Li > Cc: 'Chao Yu'; linux-f2fs-devel@lists.sourceforge.net > Subject: Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

2016-01-04 Thread Jaegeuk Kim
gt; > > From: Chao Yu [mailto:chao2...@samsung.com] > > > > Sent: Thursday, December 31, 2015 2:34 PM > > > > To: 'Fan Li'; 'Jaegeuk Kim' > > > > Cc: linux-f2fs-devel@lists.sourceforge.net > > > > Subject: RE: [f2fs-dev] [PATCH

Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

2016-01-04 Thread Fan Li
> -Original Message- > From: Jaegeuk Kim [mailto:jaeg...@kernel.org] > Sent: Monday, January 04, 2016 2:56 PM > To: Fan Li > Cc: 'Chao Yu'; linux-f2fs-devel@lists.sourceforge.net > Subject: Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

2016-01-04 Thread Fan Li
> -Original Message- > From: Chao Yu [mailto:chao2...@samsung.com] > Sent: Monday, January 04, 2016 6:00 PM > To: 'Fan Li'; 'Jaegeuk Kim' > Cc: linux-f2fs-devel@lists.sourceforge.net > Subject: RE: [f2fs-dev] [PATCH 2/3] f2fs: support finding ex

Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

2016-01-04 Thread Chao Yu
> -Original Message- > From: Fan Li [mailto:fanofcode...@samsung.com] > Sent: Monday, January 04, 2016 1:57 PM > To: 'Chao Yu'; 'Jaegeuk Kim' > Cc: linux-f2fs-devel@lists.sourceforge.net > Subject: RE: [f2fs-dev] [PATCH 2/3] f2f

Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

2016-01-03 Thread Jaegeuk Kim
-f2fs-devel@lists.sourceforge.net > > Subject: RE: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after > > isize > > > > > -Original Message- > > > From: Fan Li [mailto:fanofcode...@samsung.com] > > > Sent: Thursday, December 31, 2015 11:37

Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

2016-01-03 Thread Fan Li
> -Original Message- > From: Chao Yu [mailto:chao2...@samsung.com] > Sent: Thursday, December 31, 2015 2:34 PM > To: 'Fan Li'; 'Jaegeuk Kim' > Cc: linux-f2fs-devel@lists.sourceforge.net > Subject: RE: [f2fs-dev] [PATCH 2/3] f2fs: support finding ex

Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

2015-12-30 Thread Chao Yu
> -Original Message- > From: Fan Li [mailto:fanofcode...@samsung.com] > Sent: Thursday, December 31, 2015 11:37 AM > To: 'Chao Yu'; 'Jaegeuk Kim' > Cc: linux-f2fs-devel@lists.sourceforge.net > Subject: Re: [f2fs-dev] [PATCH 2/3] f2f

Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

2015-12-30 Thread Fan Li
> -Original Message- > From: Chao Yu [mailto:c...@kernel.org] > Sent: Wednesday, December 30, 2015 9:28 PM > To: Fan Li; 'Jaegeuk Kim' > Cc: linux-f2fs-devel@lists.sourceforge.net > Subject: Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

Re: [f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

2015-12-30 Thread Chao Yu
Hi, On 12/30/15 5:17 PM, Fan Li wrote: > f2fs allows preallocation beyond isize, but f2fs_fiemap only look up > extents within isize. Therefore add this support. > > Note: It's possible that there are holes after isize, for example, > fallocate multiple discontinuous extents after isize with >

[f2fs-dev] [PATCH 2/3] f2fs: support finding extents after isize

2015-12-30 Thread Fan Li
f2fs allows preallocation beyond isize, but f2fs_fiemap only look up extents within isize. Therefore add this support. Note: It's possible that there are holes after isize, for example, fallocate multiple discontinuous extents after isize with FALLOC_FL_KEEP_SIZE set. Since I can tell no differe