[PATCH 09/10] xfs: nowait aio support

2017-06-06 Thread Goldwyn Rodrigues
From: Goldwyn Rodrigues If IOCB_NOWAIT is set, bail if the i_rwsem is not lockable immediately. IF IOMAP_NOWAIT is set, return EAGAIN in xfs_file_iomap_begin if it needs allocation either due to file extension, writing to a hole, or COW or waiting for other DIOs to finish.

[PATCH 09/10] xfs: nowait aio support

2017-06-04 Thread Goldwyn Rodrigues
From: Goldwyn Rodrigues If IOCB_NOWAIT is set, bail if the i_rwsem is not lockable immediately. IF IOMAP_NOWAIT is set, return EAGAIN in xfs_file_iomap_begin if it needs allocation either due to file extension, writing to a hole, or COW or waiting for other DIOs to finish.

Re: [PATCH 09/10] xfs: nowait aio support

2017-05-31 Thread Jan Kara
On Tue 30-05-17 11:13:29, Goldwyn Rodrigues wrote: > > Btw, can you write a small blurb up for the man page to document these > > Ñ•emantics in man-page like language? > > > > Yes, but which man page would it belong to? > Should it be a subsection of errors in io_getevents/io_submit. We don't >

Re: [PATCH 09/10] xfs: nowait aio support

2017-05-30 Thread Goldwyn Rodrigues
On 05/29/2017 03:33 AM, Christoph Hellwig wrote: > On Sun, May 28, 2017 at 09:38:26PM -0500, Goldwyn Rodrigues wrote: >> >> >> On 05/28/2017 04:31 AM, Christoph Hellwig wrote: >>> Despite my previous reviewed-by tag this will need another fix: >>> >>> xfs_file_iomap_begin needs to return EAGAIN

Re: [PATCH 09/10] xfs: nowait aio support

2017-05-29 Thread Christoph Hellwig
On Sun, May 28, 2017 at 09:38:26PM -0500, Goldwyn Rodrigues wrote: > > > On 05/28/2017 04:31 AM, Christoph Hellwig wrote: > > Despite my previous reviewed-by tag this will need another fix: > > > > xfs_file_iomap_begin needs to return EAGAIN if we don't have the extent > > list in memoery

Re: [PATCH 09/10] xfs: nowait aio support

2017-05-29 Thread Jan Kara
On Sun 28-05-17 21:38:26, Goldwyn Rodrigues wrote: > On 05/28/2017 04:31 AM, Christoph Hellwig wrote: > > Despite my previous reviewed-by tag this will need another fix: > > > > xfs_file_iomap_begin needs to return EAGAIN if we don't have the extent > > list in memoery already. E.g. something

Re: [PATCH 09/10] xfs: nowait aio support

2017-05-28 Thread Goldwyn Rodrigues
On 05/28/2017 04:31 AM, Christoph Hellwig wrote: > Despite my previous reviewed-by tag this will need another fix: > > xfs_file_iomap_begin needs to return EAGAIN if we don't have the extent > list in memoery already. E.g. something like this: > > if ((flags & IOMAP_NOWAIT) &&

Re: [PATCH 09/10] xfs: nowait aio support

2017-05-28 Thread Christoph Hellwig
Despite my previous reviewed-by tag this will need another fix: xfs_file_iomap_begin needs to return EAGAIN if we don't have the extent list in memoery already. E.g. something like this: if ((flags & IOMAP_NOWAIT) && !(ip->i_d.if_flags & XFS_IFEXTENTS)) { error =

Re: [PATCH 09/10] xfs: nowait aio support

2017-05-24 Thread Darrick J. Wong
On Wed, May 24, 2017 at 11:41:49AM -0500, Goldwyn Rodrigues wrote: > From: Goldwyn Rodrigues > > If IOCB_NOWAIT is set, bail if the i_rwsem is not lockable > immediately. > > IF IOMAP_NOWAIT is set, return EAGAIN in xfs_file_iomap_begin > if it needs allocation either due to

[PATCH 09/10] xfs: nowait aio support

2017-05-24 Thread Goldwyn Rodrigues
From: Goldwyn Rodrigues If IOCB_NOWAIT is set, bail if the i_rwsem is not lockable immediately. IF IOMAP_NOWAIT is set, return EAGAIN in xfs_file_iomap_begin if it needs allocation either due to file extension, writing to a hole, or COW or waiting for other DIOs to finish.

[PATCH 09/10] xfs: nowait aio support

2017-05-11 Thread Goldwyn Rodrigues
From: Goldwyn Rodrigues If IOCB_NOWAIT is set, bail if the i_rwsem is not lockable immediately. IF IOMAP_NOWAIT is set, return EAGAIN in xfs_file_iomap_begin if it needs allocation either due to file extension, writing to a hole, or COW or waiting for other DIOs to finish.