Thread (55 messages) 55 messages, 2 authors, 2018-03-20

Re: [PATCH 07/36] aio: add delayed cancel support

flat view

From: Christoph Hellwig <hch@lst.de>
Date: 2018-03-20 15:20:22
Also in: linux-fsdevel, lkml, netdev

On Mon, Mar 19, 2018 at 08:19:57PM -0700, Darrick J. Wong wrote:
On Mon, Mar 05, 2018 at 01:27:14PM -0800, Christoph Hellwig wrote:
quoted
The upcoming aio poll support would like to be able to complete the
iocb inline from the cancellation context, but that would cause
a lock order reversal.  Add support for optionally moving the cancelation
outside the context lock to avoid this reversal.
I started to wonder which lock order reversal the commit message refers
to?

I think the reason for adding delayed cancellations is that we want to
be able to call io_cancel -> kiocb_cancel -> aio_poll_cancel ->
aio_complete without double locking ctx_lock?
It is.  I've updated the commit message.

--
To unsubscribe, send a message with 'unsubscribe linux-aio' in
the body to majordomo@kvack.org.  For more info on Linux AIO,
see: http://www.kvack.org/aio/
Don't email: <a href=mailto:"aart@kvack.org">aart@kvack.org</a>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help