From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44065) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1brPtK-0004QR-Vy for qemu-devel@nongnu.org; Tue, 04 Oct 2016 09:36:21 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1brPtF-000415-SS for qemu-devel@nongnu.org; Tue, 04 Oct 2016 09:36:18 -0400 Received: from mx1.redhat.com ([209.132.183.28]:33832) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1brPtF-00040R-LY for qemu-devel@nongnu.org; Tue, 04 Oct 2016 09:36:13 -0400 From: "Daniel P. Berrange" Date: Tue, 4 Oct 2016 14:35:41 +0100 Message-Id: <1475588159-30598-3-git-send-email-berrange@redhat.com> In-Reply-To: <1475588159-30598-1-git-send-email-berrange@redhat.com> References: <1475588159-30598-1-git-send-email-berrange@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: [Qemu-devel] [PATCH v6 02/20] trace: remove double-underscore in event name List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: =?UTF-8?q?Llu=C3=ADs=20Vilanova?= , Stefan Hajnoczi , "Daniel P. Berrange" If there is a double underscore in an event name, at least systemtap 3.0 will turn that into a single underscore. This in turn breaks the generated QEMU code that calls this event. The generated-tracers.h file calls a macro called QEMU_SPAPR_VSCSI__PROCESS_LOGIN, but the corresponding generated-tracers-dtrace.h file defines a macro called QEMU_SPAPR_VSCSI_PROCESS_LOGIN. It is unclear if this removal of double underscores is intended behaviour of systemtap or not. So rather than try to change the tracetool to generate the systemtap macro with the double underscore squished, it is simpler to just avoid this usage in the first place. Reviewed-by: Llu=C3=ADs Vilanova Reviewed-by: Stefan Hajnoczi Signed-off-by: Daniel P. Berrange --- hw/scsi/spapr_vscsi.c | 2 +- hw/scsi/trace-events | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/scsi/spapr_vscsi.c b/hw/scsi/spapr_vscsi.c index d8a2296..6090a20 100644 --- a/hw/scsi/spapr_vscsi.c +++ b/hw/scsi/spapr_vscsi.c @@ -658,7 +658,7 @@ static void vscsi_process_login(VSCSIState *s, vscsi_= req *req) struct srp_login_rsp *rsp =3D &iu->srp.login_rsp; uint64_t tag =3D iu->srp.rsp.tag; =20 - trace_spapr_vscsi__process_login(); + trace_spapr_vscsi_process_login(); =20 /* TODO handle case that requested size is wrong and * buffer format is wrong diff --git a/hw/scsi/trace-events b/hw/scsi/trace-events index d1995b8..4a2e5d6 100644 --- a/hw/scsi/trace-events +++ b/hw/scsi/trace-events @@ -225,7 +225,7 @@ spapr_vscsi_command_complete_sense_data2(unsigned s8,= unsigned s9, unsigned s10, spapr_vscsi_command_complete_status(uint32_t status) "Command complete e= rr=3D%"PRIu32 spapr_vscsi_save_request(uint32_t qtag, unsigned desc, unsigned offset) = "saving tag=3D%"PRIu32", current desc#%u, offset=3D0x%x" spapr_vscsi_load_request(uint32_t qtag, unsigned desc, unsigned offset) = "restoring tag=3D%"PRIu32", current desc#%u, offset=3D0x%x" -spapr_vscsi__process_login(void) "Got login, sending response !" +spapr_vscsi_process_login(void) "Got login, sending response !" spapr_vscsi_queue_cmd_no_drive(uint64_t lun) "Command for lun %08" PRIx6= 4 " with no drive" spapr_vscsi_queue_cmd(uint32_t qtag, unsigned cdb, const char *cmd, int = lun, int ret) "Queued command tag 0x%"PRIx32" CMD 0x%x=3D%s LUN %d ret: %= d" spapr_vscsi_do_crq(unsigned c0, unsigned c1) "crq: %02x %02x ..." --=20 2.7.4