Re: [PATCH 2/8] nowait aio: Introduce RWF_NOWAIT

2017-05-11 Thread Christoph Hellwig
On Tue, May 09, 2017 at 07:22:13AM -0500, Goldwyn Rodrigues wrote: > From: Goldwyn Rodrigues > > This flag informs kernel to bail out if an AIO request will block > for reasons such as file allocations, or a writeback triggered, > or would block while allocating requests while

[PATCH 2/8] nowait aio: Introduce RWF_NOWAIT

2017-05-09 Thread Goldwyn Rodrigues
From: Goldwyn Rodrigues This flag informs kernel to bail out if an AIO request will block for reasons such as file allocations, or a writeback triggered, or would block while allocating requests while performing direct I/O. Unfortunately, aio_flags is not checked for

Re: [PATCH 2/8] nowait aio: Introduce RWF_NOWAIT

2017-04-19 Thread Jan Kara
On Wed 19-04-17 05:30:24, Goldwyn Rodrigues wrote: > > > On 04/19/2017 01:39 AM, Christoph Hellwig wrote: > > > >> @@ -1593,6 +1593,11 @@ static int io_submit_one(struct kioctx *ctx, struct > >> iocb __user *user_iocb, > >>} > >> > >>req->common.ki_flags |=

Re: [PATCH 2/8] nowait aio: Introduce RWF_NOWAIT

2017-04-19 Thread Goldwyn Rodrigues
On 04/19/2017 01:39 AM, Christoph Hellwig wrote: > >> @@ -1593,6 +1593,11 @@ static int io_submit_one(struct kioctx *ctx, struct >> iocb __user *user_iocb, >> } >> >> req->common.ki_flags |= iocb_rw_flags(iocb->aio_rw_flags); >> +if ((req->common.ki_flags & IOCB_NOWAIT) && >> +

Re: [PATCH 2/8] nowait aio: Introduce RWF_NOWAIT

2017-04-19 Thread Christoph Hellwig
> } > > - > /* prevent overflows */ Weird whitespace change. > @@ -1593,6 +1593,11 @@ static int io_submit_one(struct kioctx *ctx, struct > iocb __user *user_iocb, > } > > req->common.ki_flags |= iocb_rw_flags(iocb->aio_rw_flags); > + if ((req->common.ki_flags &

[PATCH 2/8] nowait aio: Introduce RWF_NOWAIT

2017-04-14 Thread Goldwyn Rodrigues
From: Goldwyn Rodrigues This flag informs kernel to bail out if an AIO request will block for reasons such as file allocations, or a writeback triggered, or would block while allocating requests while performing direct I/O. Unfortunately, aio_flags is not checked for