[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/35] WIP: coroutine: annotate coroutine with c
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH 01/35] WIP: coroutine: annotate coroutine with clang thread safety attributes |
Date: |
Wed, 5 Jul 2017 10:11:00 -0400 (EDT) |
Hi
----- Original Message -----
> On 05/07/2017 00:03, Marc-André Lureau wrote:
> > +typedef int TAA_ROLE coroutine_role;
> > +extern coroutine_role _coroutine_fn;
> > +
> > +static inline void co_role_acquire(coroutine_role R) TAA_ACQUIRE(R)
> > TAA_NO_ANALYSYS {}
> > +static inline void co_role_release(coroutine_role R) TAA_RELEASE(R)
> > TAA_NO_ANALYSYS {}
>
> Is the argument necessary, or could it be simply TAA_ACQUIRE(_coroutine_fn)?
Yes, it can be simplified.
thanks for the suggestion
- [Qemu-devel] [PATCH 00/35] RFC: coroutine annotations & clang check, Marc-André Lureau, 2017/07/04
- [Qemu-devel] [PATCH 01/35] WIP: coroutine: annotate coroutine with clang thread safety attributes, Marc-André Lureau, 2017/07/04
- [Qemu-devel] [PATCH 02/35] WIP: coroutine: manually tag the fast-paths, Marc-André Lureau, 2017/07/04
- [Qemu-devel] [PATCH 03/35] test-coroutine: fix coroutine attribute, Marc-André Lureau, 2017/07/04
- [Qemu-devel] [PATCH 04/35] coroutine: remove coroutine_fn from qemu_coroutine_self(), Marc-André Lureau, 2017/07/04
- [Qemu-devel] [PATCH 05/35] coroutine: remove coroutine_fn from qemu_co_queue_run_restart(), Marc-André Lureau, 2017/07/04