On 25.01.24 06:06, Philippe Mathieu-Daudé wrote:
Hi,
On 25/1/24 03:33, Xiaoyao Li wrote:
Use __func__ to avoid hard-coded function name.
Signed-off-by: Xiaoyao Li <[email protected]>
Reviewed-by: David Hildenbrand <[email protected]>
---
system/physmem.c | 38 +++++++++++++++++---------------------
1 file changed, 17 insertions(+), 21 deletions(-)
if ((start + length) <= rb->max_length) {
bool need_madvise, need_fallocate;
if (!QEMU_IS_ALIGNED(length, rb->page_size)) {
- error_report("ram_block_discard_range: Unaligned length: %zx",
- length);
+ error_report("%s: Unaligned length: %zx", __func__, length);
Pre-existing, should we enforce 0x prefix for %*x format?
Likely yes, but then, most of these errors here are more similar to
assertions that we never ever see unless we have real bugs in QEMU. So I
wouldn't lose sleep over it.
--
Cheers,
David / dhildenb
