From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36836) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aYxRa-0008KD-2M for qemu-devel@nongnu.org; Thu, 25 Feb 2016 10:03:11 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aYxRW-0008Qt-8g for qemu-devel@nongnu.org; Thu, 25 Feb 2016 10:03:06 -0500 Received: from roura.ac.upc.edu ([147.83.33.10]:37962 helo=roura.ac.upc.es) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aYxRV-0008Qo-T6 for qemu-devel@nongnu.org; Thu, 25 Feb 2016 10:03:02 -0500 From: =?utf-8?b?TGx1w61z?= Vilanova Date: Thu, 25 Feb 2016 16:03:00 +0100 Message-Id: <145641258053.30097.3350702281012819654.stgit@localhost> In-Reply-To: <145641255678.30097.2919142707547689749.stgit@localhost> References: <145641255678.30097.2919142707547689749.stgit@localhost> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Subject: [Qemu-devel] [PATCH 3/6] [trivial] trace: Cosmetic changes on fast-path tracing List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Eduardo Habkost , Stefan Hajnoczi Signed-off-by: Llu=C3=ADs Vilanova --- trace/control-internal.h | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/trace/control-internal.h b/trace/control-internal.h index c78a45a..d1f99e3 100644 --- a/trace/control-internal.h +++ b/trace/control-internal.h @@ -58,14 +58,18 @@ static inline bool trace_event_get_state_static(Trace= Event *ev) return ev->sstate; } =20 -static inline bool trace_event_get_state_dynamic_by_id(int id) +static inline bool trace_event_get_state_dynamic_by_id(TraceEventID id) { + /* it's on fast path, avoid consistency checks (asserts) */ return unlikely(trace_events_enabled_count) && trace_events_dstate[i= d]; } =20 static inline bool trace_event_get_state_dynamic(TraceEvent *ev) { - int id =3D trace_event_get_id(ev); + TraceEventID id; + assert(ev !=3D NULL); + assert(trace_event_get_state_static(ev)); + id =3D trace_event_get_id(ev); return trace_event_get_state_dynamic_by_id(id); } =20