[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 00/26] block: fix coroutine_fn annotations
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH v2 00/26] block: fix coroutine_fn annotations |
Date: |
Mon, 9 May 2022 12:53:23 +0100 |
User-agent: |
Mutt/2.1.5 (2021-12-30) |
On Mon, May 09, 2022 at 12:29:53PM +0200, Paolo Bonzini wrote:
> This is the initial result of reviving Marc-André's series at
> https://patchew.org/QEMU/20170704220346.29244-1-marcandre.lureau@redhat.com/.
> A lot of the patches are similar to the ones that Marc-André wrote,
> but due to the changes in the code it was easier to redo them.
>
> For nbd, the patch is on top of "nbd: mark more coroutine_fns" that
> I sent a few days ago and that (AIUI) Eric has already queued; only
> one function was missing, much to my surprise.
>
> Apart from this, I also identified the following functions that
> can be called both in coroutine context and outside:
snip
> - qio_channel_readv_full_all_eof
> - qio_channel_writev_full_all
I'm trying to understand what criteria makes those two functions
liable for the annotation, but not others in the I/O code ?
What is the actual rule for when to apply 'coroutine_fn' annotation
to a function, and does it apply transitively to up and/or down the
call stack ?
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- [PATCH v2 18/26] quorum: add missing coroutine_fn annotations, (continued)
- [PATCH v2 18/26] quorum: add missing coroutine_fn annotations, Paolo Bonzini, 2022/05/09
- [PATCH v2 23/26] raw-format: add missing coroutine_fn annotations, Paolo Bonzini, 2022/05/09
- [PATCH v2 16/26] curl: add missing coroutine_fn annotations, Paolo Bonzini, 2022/05/09
- [PATCH v2 21/26] job: add missing coroutine_fn annotations, Paolo Bonzini, 2022/05/09
- [PATCH v2 26/26] test-coroutine: add missing coroutine_fn annotations, Paolo Bonzini, 2022/05/09
- [PATCH v2 25/26] migration: add missing coroutine_fn annotations, Paolo Bonzini, 2022/05/09
- [PATCH v2 24/26] 9p: add missing coroutine_fn annotations, Paolo Bonzini, 2022/05/09
- [PATCH v2 19/26] throttle: add missing coroutine_fn annotations, Paolo Bonzini, 2022/05/09
- [PATCH v2 20/26] vmdk: add missing coroutine_fn annotations, Paolo Bonzini, 2022/05/09
- [PATCH v2 22/26] coroutine-lock: add missing coroutine_fn annotations, Paolo Bonzini, 2022/05/09
- Re: [PATCH v2 00/26] block: fix coroutine_fn annotations,
Daniel P . Berrangé <=