Re: [f2fs-dev] [GIT PULL] f2fs for 4.8

2016-07-27 Thread Jaegeuk Kim
On Wed, Jul 27, 2016 at 01:41:01AM -0700, Christoph Hellwig wrote: > > - ioctl to move a range of data between files > > Please run this through the linux-api list first. We had so many > problem with random ioctls on random file systems in the past. Also > I know Samsung had an xfs (and maybe

Re: [f2fs-dev] [GIT PULL] f2fs for 4.8

2016-07-27 Thread Jaegeuk Kim
On Wed, Jul 27, 2016 at 11:46:11AM -0700, Linus Torvalds wrote: > On Wed, Jul 27, 2016 at 11:29 AM, Jens Axboe wrote: > > > > Looks OK to me, though I think you could have dropped the ->bi_rw > > assignment in f2fs_submit_page_bio(): > > > > bio->bi_rw = fio->op_flags; > >

Re: [f2fs-dev] [GIT PULL] f2fs for 4.8

2016-07-27 Thread Christoph Hellwig
> - ioctl to move a range of data between files Please run this through the linux-api list first. We had so many problem with random ioctls on random file systems in the past. Also I know Samsung had an xfs (and maybe ext4?) implementation of this functionality in the past, so it would be