diff options
| author | Zhou Jie <zhoujie2011@cn.fujitsu.com> | 2016-04-29 13:12:01 +0800 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2016-05-12 15:22:08 +0200 |
| commit | ed79f37d9b6f1d3543afe5103bbe9dab43484b66 (patch) | |
| tree | 4b60625a8d6e408996b8f75325807723ac1b3ef8 /block/sheepdog.c | |
| parent | 547cb1574e29082aa194ec65c43c8e08ac1f9aa8 (diff) | |
| download | qemu-ed79f37d9b6f1d3543afe5103bbe9dab43484b66.zip qemu-ed79f37d9b6f1d3543afe5103bbe9dab43484b66.tar.gz qemu-ed79f37d9b6f1d3543afe5103bbe9dab43484b66.tar.bz2 | |
block: always compile-check debug prints
Files with conditional debug statements should ensure that the printf is
always compiled. This prevents bitrot of the format string of the debug
statement. And switch debug output to stderr.
Signed-off-by: Zhou Jie <zhoujie2011@cn.fujitsu.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'block/sheepdog.c')
| -rw-r--r-- | block/sheepdog.c | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/block/sheepdog.c b/block/sheepdog.c index 33e0a33..9023686 100644 --- a/block/sheepdog.c +++ b/block/sheepdog.c @@ -294,13 +294,16 @@ static inline size_t count_data_objs(const struct SheepdogInode *inode) #undef DPRINTF #ifdef DEBUG_SDOG -#define DPRINTF(fmt, args...) \ - do { \ - fprintf(stdout, "%s %d: " fmt, __func__, __LINE__, ##args); \ - } while (0) +#define DEBUG_SDOG_PRINT 1 #else -#define DPRINTF(fmt, args...) +#define DEBUG_SDOG_PRINT 0 #endif +#define DPRINTF(fmt, args...) \ + do { \ + if (DEBUG_SDOG_PRINT) { \ + fprintf(stderr, "%s %d: " fmt, __func__, __LINE__, ##args); \ + } \ + } while (0) typedef struct SheepdogAIOCB SheepdogAIOCB; |