Hi ----- Original Message ----- > > > On 05/07/2017 16:21, Marc-André Lureau wrote: > >> > >> They are, but it's an implementation detail. Why is this patch necessary? > > I didn't think this would be controversial :) well, the checks I added to > > clang verify function pointer share the coroutine attribute. > > > > The function themself are/need to be coroutine_fn (as they will call > > coroutine_fn too) > > It's not controversial, I would not have expected the functions to call > coroutine_fn. :) How do they do that? >
For example, null_co_readv() calls null_co_common() which calls co_aio_sleep_ns()