From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:53349)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <stefanha@redhat.com>) id 1ba2yc-0002Rh-A1
	for qemu-devel@nongnu.org; Wed, 17 Aug 2016 11:41:59 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <stefanha@redhat.com>) id 1ba2yX-0004by-B9
	for qemu-devel@nongnu.org; Wed, 17 Aug 2016 11:41:57 -0400
Received: from mx1.redhat.com ([209.132.183.28]:55290)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <stefanha@redhat.com>) id 1ba2yX-0004bu-5G
	for qemu-devel@nongnu.org; Wed, 17 Aug 2016 11:41:53 -0400
Date: Wed, 17 Aug 2016 16:41:48 +0100
From: Stefan Hajnoczi <stefanha@redhat.com>
Message-ID: <20160817154148.GD9207@stefanha-x1.localdomain>
References: <147101601558.15723.5286973980439276800.stgit@fimbulvetr.bsc.es>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha1;
	protocol="application/pgp-signature"; boundary="d9ADC0YsG2v16Js0"
Content-Disposition: inline
In-Reply-To: <147101601558.15723.5286973980439276800.stgit@fimbulvetr.bsc.es>
Subject: Re: [Qemu-devel] [PATCH v4 0/2] trace: Simplify late initialization
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel/>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: =?iso-8859-1?Q?Llu=EDs?= Vilanova <vilanova@ac.upc.edu>
Cc: qemu-devel@nongnu.org, Paolo Bonzini <pbonzini@redhat.com>, "'Daniel P. Berrange'" <berrange@redhat.com>


--d9ADC0YsG2v16Js0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Fri, Aug 12, 2016 at 05:33:35PM +0200, Llu=EDs Vilanova wrote:
> Removes the need for 'trace_events_dstate_init' and does a little cleanup=
 in how
> state values are modified (to avoid implicit conversions from bool).
>=20
> Changes in v2
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>=20
> * Fix late-init state value [Daniel P. Berrange].
>=20
> Changes in v3
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>=20
> * Avoid implicit conversions from bool to integers (second patch) [Daniel
>   P. Berrange].
>=20
> Changes in v4
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>=20
> * Correctly implement idempotent state changes.
> * Clarify when state changes are idempotent [Daniel P. Berrange].
>=20
> Signed-off-by: Llu=EDs Vilanova <vilanova@ac.upc.edu>
> ---
>=20
> Llu=EDs Vilanova (2):
>       trace: Remove 'trace_events_dstate_init'
>       trace: Avoid implicit bool->integer conversions
>=20
>=20
>  stubs/trace-control.c  |   22 ++++++++++++++++++++--
>  trace/control-target.c |   34 ++++++++++++++++++++++++++++++++--
>  trace/control.c        |   23 ++++++++++-------------
>  trace/control.h        |    3 +++
>  trace/event-internal.h |    1 +
>  5 files changed, 66 insertions(+), 17 deletions(-)
>=20
>=20
> To: qemu-devel@nongnu.org
> Cc: Stefan Hajnoczi <stefanha@redhat.com>
> Cc: 'Daniel P. Berrange' <berrange@redhat.com>
> Cc: Paolo Bonzini <pbonzini@redhat.com>

I will wait for Daniel Berrange to review before merging since he had
comments on previous revisions.

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>

--d9ADC0YsG2v16Js0
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQEcBAEBAgAGBQJXtIW8AAoJEJykq7OBq3PIzloH/2ZIwWSA5A/53g2+ugWVE0wJ
jDB4lWorj01p+nTTqEFKfDahRMDaokdsXsK4LC6YuZK44rq1/wtJNYOHnMkMhW5p
tkcFZ6iF4VzeVoTDNltxyIEjWRmOwvZNnZUclqnEJW2Zdt7nhTur8z4n3JPXnVl0
C5O/zBF/366+EeXvL+2DVN59gDP6j8GQRK1j5Y24huQfVj/KAGa9kSA8EvE4/kLC
DIMCt9zyLgsyHQwBww+ogSnfG6fC4zyPDUrOGF+rS4f9L9d4fyV0Xe+mVddWMkpK
f7G5DqkYx1eOq4qPAimoEIK6clKzCpLnGbh7XzvyhsrOe8GsyW6xuaBesLR4uJQ=
=tAOh
-----END PGP SIGNATURE-----

--d9ADC0YsG2v16Js0--