mbox series

[v2,0/4] block: Convert from DPRINTF() macro to trace event

Message ID 20181213162727.17438-1-lvivier@redhat.com
Headers show
Series block: Convert from DPRINTF() macro to trace event | expand

Message

Laurent Vivier Dec. 13, 2018, 4:27 p.m. UTC
Convert all the remaining uses of DPRINTF() in the directory block.

Compiled for all target but only tested with "make check"

v2: replace "sd_" prefix by "sheepdog_"
    remove the dot at the end of the text strings
    use %zu with size_t
    change an uint64_t to int64_t to match % PRIi64

Laurent Vivier (4):
  block/ssh: Convert from DPRINTF() macro to trace events
  block/curl: Convert from DPRINTF() macro to trace events
  block/file-posix: Convert from DPRINTF() macro to trace events
  block/sheepdog: Convert from DPRINTF() macro to trace events

 block/curl.c       | 29 ++++++++--------------------
 block/file-posix.c | 25 ++++++------------------
 block/sheepdog.c   | 47 +++++++++++++++++-----------------------------
 block/ssh.c        | 46 +++++++++++++++++----------------------------
 block/trace-events | 47 ++++++++++++++++++++++++++++++++++++++++++++++
 5 files changed, 95 insertions(+), 99 deletions(-)

Comments

Max Reitz Jan. 14, 2019, 12:14 p.m. UTC | #1
On 13.12.18 17:27, Laurent Vivier wrote:
> Convert all the remaining uses of DPRINTF() in the directory block.
> 
> Compiled for all target but only tested with "make check"
> 
> v2: replace "sd_" prefix by "sheepdog_"
>     remove the dot at the end of the text strings
>     use %zu with size_t
>     change an uint64_t to int64_t to match % PRIi64
> 
> Laurent Vivier (4):
>   block/ssh: Convert from DPRINTF() macro to trace events
>   block/curl: Convert from DPRINTF() macro to trace events
>   block/file-posix: Convert from DPRINTF() macro to trace events
>   block/sheepdog: Convert from DPRINTF() macro to trace events

Applied to my block branch, with three fixes (Philippe asked me to do
one, so what are two more O:-)):
- Patch 1: Fixed ssh_{read,write}_return's @ret sign
- Patch 3: Moved the pointer asterisk in file_xfs_discard from type to name
- Patch 4: Fixed sheepdog_snapshot_create_inode's format string to use
PRIx32 instead of %x for uint32_ts

(https://git.xanclic.moe/XanClic/qemu/commits/branch/block)

Thanks!

Max
Philippe Mathieu-Daudé Jan. 14, 2019, 12:26 p.m. UTC | #2
On 1/14/19 1:14 PM, Max Reitz wrote:
> On 13.12.18 17:27, Laurent Vivier wrote:
>> Convert all the remaining uses of DPRINTF() in the directory block.
>>
>> Compiled for all target but only tested with "make check"
>>
>> v2: replace "sd_" prefix by "sheepdog_"
>>     remove the dot at the end of the text strings
>>     use %zu with size_t
>>     change an uint64_t to int64_t to match % PRIi64
>>
>> Laurent Vivier (4):
>>   block/ssh: Convert from DPRINTF() macro to trace events
>>   block/curl: Convert from DPRINTF() macro to trace events
>>   block/file-posix: Convert from DPRINTF() macro to trace events
>>   block/sheepdog: Convert from DPRINTF() macro to trace events
> 
> Applied to my block branch, with three fixes (Philippe asked me to do
> one, so what are two more O:-)):
> - Patch 1: Fixed ssh_{read,write}_return's @ret sign
> - Patch 3: Moved the pointer asterisk in file_xfs_discard from type to name
> - Patch 4: Fixed sheepdog_snapshot_create_inode's format string to use
> PRIx32 instead of %x for uint32_ts

Thank you Max!

> 
> (https://git.xanclic.moe/XanClic/qemu/commits/branch/block)
> 
> Thanks!
> 
> Max
>