From: "Philippe Mathieu-Daudé" <philippe.mathieu.daude@gmail.com>
To: Paolo Bonzini <pbonzini@redhat.com>, qemu-devel@nongnu.org
Cc: hreitz@redhat.com, qemu-block@nongnu.org, stefanha@redhat.com,
sguelton@redhat.com
Subject: Re: [PATCH 05/35] coroutine: small code cleanup in qemu_co_rwlock_wrlock
Date: Thu, 10 Mar 2022 15:10:28 +0100 [thread overview]
Message-ID: <c25b1d81-77a5-d8b4-b092-21e09b81603a@gmail.com> (raw)
In-Reply-To: <20220310124413.1102441-6-pbonzini@redhat.com>
On 10/3/22 13:43, Paolo Bonzini wrote:
> qemu_co_rwlock_wrlock stores the current coroutine in a local variable,
> use it instead of calling qemu_coroutine_self() again.
>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
> util/qemu-coroutine-lock.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
next prev parent reply other threads:[~2022-03-10 14:18 UTC|newest]
Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-03-10 12:43 [PATCH experiment 00/35] stackless coroutine backend Paolo Bonzini
2022-03-10 12:43 ` [PATCH 01/35] coroutine: add missing coroutine_fn annotations for CoRwlock functions Paolo Bonzini
2022-03-10 12:43 ` [PATCH 02/35] coroutine: qemu_coroutine_get_aio_context is not a coroutine_fn Paolo Bonzini
2022-03-10 12:43 ` [PATCH 03/35] coroutine: introduce QemuCoLockable Paolo Bonzini
2022-03-10 12:43 ` [PATCH 04/35] coroutine: introduce coroutine_only_fn Paolo Bonzini
2022-03-10 12:43 ` [PATCH 05/35] coroutine: small code cleanup in qemu_co_rwlock_wrlock Paolo Bonzini
2022-03-10 14:10 ` Philippe Mathieu-Daudé [this message]
2022-03-10 12:43 ` [PATCH 06/35] disable some code Paolo Bonzini
2022-03-10 12:43 ` [PATCH 07/35] coroutine: introduce the "stackless coroutine" backend Paolo Bonzini
2022-03-10 12:43 ` [PATCH 08/35] /basic/lifecycle Paolo Bonzini
2022-03-10 12:43 ` [PATCH 09/35] convert qemu-coroutine-sleep.c to stackless coroutines Paolo Bonzini
2022-03-10 12:43 ` [PATCH 10/35] enable tail call optimization of qemu_co_mutex_lock Paolo Bonzini
2022-03-10 12:43 ` [PATCH 11/35] convert CoMutex to stackless coroutines Paolo Bonzini
2022-03-10 12:43 ` [PATCH 12/35] define magic macros for " Paolo Bonzini
2022-03-10 12:43 ` [PATCH 13/35] /basic/yield Paolo Bonzini
2022-03-10 12:43 ` [PATCH 14/35] /basic/nesting Paolo Bonzini
2022-03-10 12:43 ` [PATCH 15/35] /basic/self Paolo Bonzini
2022-03-10 12:43 ` [PATCH 16/35] /basic/entered Paolo Bonzini
2022-03-10 12:43 ` [PATCH 17/35] /basic/in_coroutine Paolo Bonzini
2022-03-10 12:43 ` [PATCH 18/35] /basic/order Paolo Bonzini
2022-03-10 12:43 ` [PATCH 19/35] /perf/lifecycle Paolo Bonzini
2022-03-10 12:43 ` [PATCH 20/35] /perf/nesting Paolo Bonzini
2022-03-10 12:43 ` [PATCH 21/35] /perf/yield Paolo Bonzini
2022-03-10 12:44 ` [PATCH 22/35] /perf/function-call Paolo Bonzini
2022-03-10 12:44 ` [PATCH 23/35] /perf/cost Paolo Bonzini
2022-03-10 12:44 ` [PATCH 24/35] /basic/no-dangling-access Paolo Bonzini
2022-03-10 12:44 ` [PATCH 25/35] /locking/co-mutex Paolo Bonzini
2022-03-10 12:44 ` [PATCH 26/35] convert qemu_co_mutex_lock_slowpath to magic macros Paolo Bonzini
2022-03-10 12:44 ` [PATCH 27/35] /locking/co-mutex/lockable Paolo Bonzini
2022-03-10 12:44 ` [PATCH 28/35] qemu_co_rwlock_maybe_wake_one Paolo Bonzini
2022-03-10 12:44 ` [PATCH 29/35] qemu_co_rwlock_rdlock Paolo Bonzini
2022-03-10 12:44 ` [PATCH 30/35] qemu_co_rwlock_unlock Paolo Bonzini
2022-03-10 12:44 ` [PATCH 31/35] qemu_co_rwlock_downgrade Paolo Bonzini
2022-03-10 12:44 ` [PATCH 32/35] qemu_co_rwlock_wrlock Paolo Bonzini
2022-03-10 12:44 ` [PATCH 33/35] qemu_co_rwlock_upgrade Paolo Bonzini
2022-03-10 12:44 ` [PATCH 34/35] /locking/co-rwlock/upgrade Paolo Bonzini
2022-03-10 12:44 ` [PATCH 35/35] /locking/co-rwlock/downgrade Paolo Bonzini
2022-03-10 17:42 ` [PATCH experiment 00/35] stackless coroutine backend Stefan Hajnoczi
2022-03-10 20:14 ` Paolo Bonzini
2022-03-11 9:27 ` Stefan Hajnoczi
2022-03-11 12:04 ` Paolo Bonzini
2022-03-11 12:17 ` Daniel P. Berrangé
2022-03-13 15:18 ` Paolo Bonzini
2022-03-14 13:43 ` Stefan Hajnoczi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=c25b1d81-77a5-d8b4-b092-21e09b81603a@gmail.com \
--to=philippe.mathieu.daude@gmail.com \
--cc=hreitz@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=sguelton@redhat.com \
--cc=stefanha@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).