From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49276) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ehHiL-0005Cl-9p for qemu-devel@nongnu.org; Thu, 01 Feb 2018 11:27:54 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ehHiG-0001vk-Ro for qemu-devel@nongnu.org; Thu, 01 Feb 2018 11:27:53 -0500 Received: from mx1.redhat.com ([209.132.183.28]:55432) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1ehHiG-0001uD-68 for qemu-devel@nongnu.org; Thu, 01 Feb 2018 11:27:48 -0500 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 7543A28158 for ; Thu, 1 Feb 2018 16:27:47 +0000 (UTC) Date: Thu, 1 Feb 2018 16:27:41 +0000 From: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= Message-ID: <20180201162741.GV22810@redhat.com> Reply-To: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= References: <20180201162625.4276-1-stefanha@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20180201162625.4276-1-stefanha@redhat.com> Subject: Re: [Qemu-devel] [PATCH] trace: avoid SystemTap "char const" warnings List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: qemu-devel@nongnu.org On Thu, Feb 01, 2018 at 04:26:25PM +0000, Stefan Hajnoczi wrote: > SystemTap's dtrace(1) produces the following warning when it encounters > "char const" instead of "const char": > > Warning: /usr/bin/dtrace:trace-dtrace-root.dtrace:66: syntax error near: > probe flatview_destroy_rcu > > Warning: Proceeding as if --no-pyparsing was given. > > This is a limitation in current SystemTap releases. I have sent a patch > upstream to accept "char const" since it is valid C: > > https://sourceware.org/ml/systemtap/2018-q1/msg00017.html > > In QEMU we still wish to avoid warnings in the current SystemTap > release. It's simple enough to replace "char const" with "const char". > > I'm not changing the documentation or implementing checks to prevent > this from occurring again in the future. The next release of SystemTap > will hopefully resolve this issue. > > Cc: Daniel P. Berrange > Signed-off-by: Stefan Hajnoczi > --- > hw/block/trace-events | 4 ++-- > trace-events | 12 ++++++------ > 2 files changed, 8 insertions(+), 8 deletions(-) Reviewed-by: Daniel P. Berrange > > diff --git a/hw/block/trace-events b/hw/block/trace-events > index 5acd495207..6b9e733412 100644 > --- a/hw/block/trace-events > +++ b/hw/block/trace-events > @@ -17,7 +17,7 @@ nvme_irq_msix(uint32_t vector) "raising MSI-X IRQ vector %u" > nvme_irq_pin(void) "pulsing IRQ pin" > nvme_irq_masked(void) "IRQ is masked" > nvme_dma_read(uint64_t prp1, uint64_t prp2) "DMA read, prp1=0x%"PRIx64" prp2=0x%"PRIx64"" > -nvme_rw(char const *verb, uint32_t blk_count, uint64_t byte_count, uint64_t lba) "%s %"PRIu32" blocks (%"PRIu64" bytes) from LBA %"PRIu64"" > +nvme_rw(const char *verb, uint32_t blk_count, uint64_t byte_count, uint64_t lba) "%s %"PRIu32" blocks (%"PRIu64" bytes) from LBA %"PRIu64"" > nvme_create_sq(uint64_t addr, uint16_t sqid, uint16_t cqid, uint16_t qsize, uint16_t qflags) "create submission queue, addr=0x%"PRIx64", sqid=%"PRIu16", cqid=%"PRIu16", qsize=%"PRIu16", qflags=%"PRIu16"" > nvme_create_cq(uint64_t addr, uint16_t cqid, uint16_t vector, uint16_t size, uint16_t qflags, int ien) "create completion queue, addr=0x%"PRIx64", cqid=%"PRIu16", vector=%"PRIu16", qsize=%"PRIu16", qflags=%"PRIu16", ien=%d" > nvme_del_sq(uint16_t qid) "deleting submission queue sqid=%"PRIu16"" > @@ -25,7 +25,7 @@ nvme_del_cq(uint16_t cqid) "deleted completion queue, sqid=%"PRIu16"" > nvme_identify_ctrl(void) "identify controller" > nvme_identify_ns(uint16_t ns) "identify namespace, nsid=%"PRIu16"" > nvme_identify_nslist(uint16_t ns) "identify namespace list, nsid=%"PRIu16"" > -nvme_getfeat_vwcache(char const* result) "get feature volatile write cache, result=%s" > +nvme_getfeat_vwcache(const char* result) "get feature volatile write cache, result=%s" > nvme_getfeat_numq(int result) "get feature number of queues, result=%d" > nvme_setfeat_numq(int reqcq, int reqsq, int gotcq, int gotsq) "requested cq_count=%d sq_count=%d, responding with cq_count=%d sq_count=%d" > nvme_mmio_intm_set(uint64_t data, uint64_t new_mask) "wrote MMIO, interrupt mask set, data=0x%"PRIx64", new_mask=0x%"PRIx64"" > diff --git a/trace-events b/trace-events > index ec95e67089..89fcad0fd1 100644 > --- a/trace-events > +++ b/trace-events > @@ -73,13 +73,13 @@ flatview_destroy(FlatView *view, MemoryRegion *root) "%p (root %p)" > flatview_destroy_rcu(FlatView *view, MemoryRegion *root) "%p (root %p)" > > # gdbstub.c > -gdbstub_op_start(char const *device) "Starting gdbstub using device %s" > +gdbstub_op_start(const char *device) "Starting gdbstub using device %s" > gdbstub_op_exiting(uint8_t code) "notifying exit with code=0x%02x" > gdbstub_op_continue(void) "Continuing all CPUs" > gdbstub_op_continue_cpu(int cpu_index) "Continuing CPU %d" > gdbstub_op_stepping(int cpu_index) "Stepping CPU %d" > -gdbstub_op_extra_info(char const *info) "Thread extra info: %s" > -gdbstub_hit_watchpoint(char const *type, int cpu_gdb_index, uint64_t vaddr) "Watchpoint hit, type=\"%s\" cpu=%d, vaddr=0x%" PRIx64 "" > +gdbstub_op_extra_info(const char *info) "Thread extra info: %s" > +gdbstub_hit_watchpoint(const char *type, int cpu_gdb_index, uint64_t vaddr) "Watchpoint hit, type=\"%s\" cpu=%d, vaddr=0x%" PRIx64 "" > gdbstub_hit_internal_error(void) "RUN_STATE_INTERNAL_ERROR" > gdbstub_hit_break(void) "RUN_STATE_DEBUG" > gdbstub_hit_paused(void) "RUN_STATE_PAUSED" > @@ -87,9 +87,9 @@ gdbstub_hit_shutdown(void) "RUN_STATE_SHUTDOWN" > gdbstub_hit_io_error(void) "RUN_STATE_IO_ERROR" > gdbstub_hit_watchdog(void) "RUN_STATE_WATCHDOG" > gdbstub_hit_unknown(int state) "Unknown run state=0x%x" > -gdbstub_io_reply(char const *message) "Sent: %s" > -gdbstub_io_binaryreply(size_t ofs, char const *line) "0x%04zx: %s" > -gdbstub_io_command(char const *command) "Received: %s" > +gdbstub_io_reply(const char *message) "Sent: %s" > +gdbstub_io_binaryreply(size_t ofs, const char *line) "0x%04zx: %s" > +gdbstub_io_command(const char *command) "Received: %s" > gdbstub_io_got_ack(void) "Got ACK" > gdbstub_io_got_unexpected(uint8_t ch) "Got 0x%02x when expecting ACK/NACK" > gdbstub_err_got_nack(void) "Got NACK, retransmitting" > -- > 2.14.3 > Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|