[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/1] unbreak dtrace tracing due to double _ in r
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 1/1] unbreak dtrace tracing due to double _ in rdma names |
Date: |
Mon, 16 Feb 2015 18:06:24 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 |
On 16/02/2015 17:58, Dr. David Alan Gilbert (git) wrote:
> From: "Dr. David Alan Gilbert" <address@hidden>
>
> It looks like the dtrace trace code gets upset if you have trace names
> with __ in, which the migration/rdma.c code does.
>
> Rename the functions and the associated traces.
>
> Fixes: 733252deb8b7d37beacda5976c2769e18642b2fa
>
> Signed-off-by: Dr. David Alan Gilbert <address@hidden>
> Reported-by: Andreas Färber <address@hidden>
> ---
> migration/rdma.c | 24 +++++++++++-------------
> trace-events | 4 ++--
> 2 files changed, 13 insertions(+), 15 deletions(-)
>
> diff --git a/migration/rdma.c b/migration/rdma.c
> index fc351ea..e7d77f6 100644
> --- a/migration/rdma.c
> +++ b/migration/rdma.c
> @@ -510,7 +510,7 @@ static inline uint8_t *ram_chunk_end(const RDMALocalBlock
> *rdma_ram_block,
> return result;
> }
>
> -static int __qemu_rdma_add_block(RDMAContext *rdma, void *host_addr,
> +static int rdma_add_block(RDMAContext *rdma, void *host_addr,
> ram_addr_t block_offset, uint64_t length)
> {
> RDMALocalBlocks *local = &rdma->local_ram_blocks;
> @@ -551,13 +551,12 @@ static int __qemu_rdma_add_block(RDMAContext *rdma,
> void *host_addr,
>
> g_hash_table_insert(rdma->blockmap, (void *) block_offset, block);
>
> - trace___qemu_rdma_add_block(local->nb_blocks,
> - (uint64_t) block->local_host_addr, block->offset,
> - block->length,
> - (uint64_t) (block->local_host_addr +
> block->length),
> - BITS_TO_LONGS(block->nb_chunks) *
> - sizeof(unsigned long) * 8,
> - block->nb_chunks);
> + trace_rdma_add_block(local->nb_blocks, (uint64_t) block->local_host_addr,
> + block->offset, block->length,
> + (uint64_t) (block->local_host_addr + block->length),
> + BITS_TO_LONGS(block->nb_chunks) *
> + sizeof(unsigned long) * 8,
> + block->nb_chunks);
>
> local->nb_blocks++;
>
> @@ -572,7 +571,7 @@ static int __qemu_rdma_add_block(RDMAContext *rdma, void
> *host_addr,
> static void qemu_rdma_init_one_block(void *host_addr,
> ram_addr_t block_offset, ram_addr_t length, void *opaque)
> {
> - __qemu_rdma_add_block(opaque, host_addr, block_offset, length);
> + rdma_add_block(opaque, host_addr, block_offset, length);
> }
>
> /*
> @@ -595,7 +594,7 @@ static int qemu_rdma_init_ram_blocks(RDMAContext *rdma)
> return 0;
> }
>
> -static int __qemu_rdma_delete_block(RDMAContext *rdma, ram_addr_t
> block_offset)
> +static int rdma_delete_block(RDMAContext *rdma, ram_addr_t block_offset)
> {
> RDMALocalBlocks *local = &rdma->local_ram_blocks;
> RDMALocalBlock *block = g_hash_table_lookup(rdma->blockmap,
> @@ -657,7 +656,7 @@ static int __qemu_rdma_delete_block(RDMAContext *rdma,
> ram_addr_t block_offset)
> local->block = NULL;
> }
>
> - trace___qemu_rdma_delete_block(local->nb_blocks,
> + trace_rdma_delete_block(local->nb_blocks,
> (uint64_t)block->local_host_addr,
> block->offset, block->length,
> (uint64_t)(block->local_host_addr +
> block->length),
> @@ -2190,8 +2189,7 @@ static void qemu_rdma_cleanup(RDMAContext *rdma)
>
> if (rdma->local_ram_blocks.block) {
> while (rdma->local_ram_blocks.nb_blocks) {
> - __qemu_rdma_delete_block(rdma,
> - rdma->local_ram_blocks.block->offset);
> + rdma_delete_block(rdma, rdma->local_ram_blocks.block->offset);
> }
> }
>
> diff --git a/trace-events b/trace-events
> index 57f357f..378505f 100644
> --- a/trace-events
> +++ b/trace-events
> @@ -1337,8 +1337,6 @@ migrate_pending(uint64_t size, uint64_t max) "pending
> size %" PRIu64 " max %" PR
> migrate_transferred(uint64_t tranferred, uint64_t time_spent, double
> bandwidth, uint64_t size) "transferred %" PRIu64 " time_spent %" PRIu64 "
> bandwidth %g max_size %" PRId64
>
> # migration/rdma.c
> -__qemu_rdma_add_block(int block, uint64_t addr, uint64_t offset, uint64_t
> len, uint64_t end, uint64_t bits, int chunks) "Added Block: %d, addr: %"
> PRIu64 ", offset: %" PRIu64 " length: %" PRIu64 " end: %" PRIu64 " bits %"
> PRIu64 " chunks %d"
> -__qemu_rdma_delete_block(int block, uint64_t addr, uint64_t offset, uint64_t
> len, uint64_t end, uint64_t bits, int chunks) "Deleted Block: %d, addr: %"
> PRIu64 ", offset: %" PRIu64 " length: %" PRIu64 " end: %" PRIu64 " bits %"
> PRIu64 " chunks %d"
> qemu_dma_accept_incoming_migration(void) ""
> qemu_dma_accept_incoming_migration_accepted(void) ""
> qemu_rdma_accept_pin_state(bool pin) "%d"
> @@ -1392,6 +1390,8 @@ qemu_rdma_write_one_recvregres(int mykey, int theirkey,
> uint64_t chunk) "Receive
> qemu_rdma_write_one_sendreg(uint64_t chunk, int len, int index, int64_t
> offset) "Sending registration request chunk %" PRIu64 " for %d bytes, index:
> %d, offset: %" PRId64
> qemu_rdma_write_one_top(uint64_t chunks, uint64_t size) "Writing %" PRIu64 "
> chunks, (%" PRIu64 " MB)"
> qemu_rdma_write_one_zero(uint64_t chunk, int len, int index, int64_t offset)
> "Entire chunk is zero, sending compress: %" PRIu64 " for %d bytes, index: %d,
> offset: %" PRId64
> +rdma_add_block(int block, uint64_t addr, uint64_t offset, uint64_t len,
> uint64_t end, uint64_t bits, int chunks) "Added Block: %d, addr: %" PRIu64 ",
> offset: %" PRIu64 " length: %" PRIu64 " end: %" PRIu64 " bits %" PRIu64 "
> chunks %d"
> +rdma_delete_block(int block, uint64_t addr, uint64_t offset, uint64_t len,
> uint64_t end, uint64_t bits, int chunks) "Deleted Block: %d, addr: %" PRIu64
> ", offset: %" PRIu64 " length: %" PRIu64 " end: %" PRIu64 " bits %" PRIu64 "
> chunks %d"
> rdma_start_incoming_migration(void) ""
> rdma_start_incoming_migration_after_dest_init(void) ""
> rdma_start_incoming_migration_after_rdma_listen(void) ""
>
Good thing even if it weren't for the broken tracing.
Reviewed-by: Paolo Bonzini <address@hidden>