[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/8] fw_cfg: Replace debug prints by tracepoints
From: |
Peter Lieven |
Subject: |
Re: [Qemu-devel] [PATCH 1/8] fw_cfg: Replace debug prints by tracepoints |
Date: |
Thu, 24 Jan 2013 11:43:33 +0100 |
Am 24.01.2013 um 11:32 schrieb Markus Armbruster <address@hidden>:
> Peter Lieven <address@hidden> writes:
>
>> Latest git doesn't compile with this patch:
>>
>> hw/fw_cfg.c: In function 'fw_cfg_write':
>> hw/fw_cfg.c:207:5: error: implicit declaration of function
>> trace_fw_cfg_write' [-Werror=implicit-function-declaration]
>> hw/fw_cfg.c:207:5: error: nested extern declaration of
>> trace_fw_cfg_write' [-Werror=nested-externs]
>> hw/fw_cfg.c: In function 'fw_cfg_select':
>> hw/fw_cfg.c:232:5: error: implicit declaration of function
>> trace_fw_cfg_select' [-Werror=implicit-function-declaration]
>> hw/fw_cfg.c:232:5: error: nested extern declaration of
>> trace_fw_cfg_select' [-Werror=nested-externs]
>> hw/fw_cfg.c: In function 'fw_cfg_read':
>> hw/fw_cfg.c:247:5: error: implicit declaration of function
>> trace_fw_cfg_read' [-Werror=implicit-function-declaration]
>> hw/fw_cfg.c:247:5: error: nested extern declaration of
>> trace_fw_cfg_read' [-Werror=nested-externs]
>> hw/fw_cfg.c: In function 'fw_cfg_add_file':
>> hw/fw_cfg.c:460:13: error: implicit declaration of function
>> trace_fw_cfg_add_file_dupe' [-Werror=implicit-function-declaration]
>> hw/fw_cfg.c:460:13: error: nested extern declaration of
>> trace_fw_cfg_add_file_dupe' [-Werror=nested-externs]
>> hw/fw_cfg.c:467:5: error: implicit declaration of function
>> trace_fw_cfg_add_file' [-Werror=implicit-function-declaration]
>> hw/fw_cfg.c:467:5: error: nested extern declaration of
>> trace_fw_cfg_add_file' [-Werror=nested-externs]
>> cc1: all warnings being treated as errors
>> make: *** [hw/fw_cfg.o] Error 1
>
> Compiles fine for me.
>
> What happens if you blow away your build tree and start over?
>
> In my experience, rebuilding of trace-related stuff is rather fragile...
If I make a fresh clone of the qmu git repository it works.
Peter
- [Qemu-devel] [PATCH 0/8] Fixes and cleanups around fw_cfg, Markus Armbruster, 2013/01/16
- [Qemu-devel] [PATCH 3/8] fw_cfg: New fw_cfg_add_string(), Markus Armbruster, 2013/01/16
- [Qemu-devel] [PATCH 4/8] pc: Fix unchecked strdup() by switching to fw_cfg_add_string(), Markus Armbruster, 2013/01/16
- [Qemu-devel] [PATCH 7/8] fw_cfg: Use void *, size_t instead of uint8_t *, uint32_t for blobs, Markus Armbruster, 2013/01/16
- [Qemu-devel] [PATCH 1/8] fw_cfg: Replace debug prints by tracepoints, Markus Armbruster, 2013/01/16
- Re: [Qemu-devel] [PATCH 1/8] fw_cfg: Replace debug prints by tracepoints,
Peter Lieven <=
- Re: [Qemu-devel] [PATCH 1/8] fw_cfg: Replace debug prints by tracepoints, Markus Armbruster, 2013/01/24
[Qemu-devel] [PATCH 8/8] vl: Use size_t for sizes in get_boot_devices_list(), Markus Armbruster, 2013/01/16
[Qemu-devel] [PATCH 2/8] fw_cfg: Dumb down fw_cfg_add_*() not to return success / failure, Markus Armbruster, 2013/01/16
[Qemu-devel] [PATCH 5/8] sun4: Fix unchecked strdup() by switching to fw_cfg_add_string(), Markus Armbruster, 2013/01/16
[Qemu-devel] [PATCH 6/8] pc: Clean up bochs_bios_init()'s (non-)use of sizeof, Markus Armbruster, 2013/01/16
Re: [Qemu-devel] [PATCH 0/8] Fixes and cleanups around fw_cfg, Gleb Natapov, 2013/01/17