* [Qemu-devel] [PATCH v2] vl: Print maxmem in hex format for error message
@ 2014-09-16 7:45 zhanghailiang
2014-09-16 9:12 ` Igor Mammedov
0 siblings, 1 reply; 3+ messages in thread
From: zhanghailiang @ 2014-09-16 7:45 UTC (permalink / raw)
To: qemu-trivial; +Cc: zhanghailiang, luonengjun, qemu-devel, peter.huangpeng
In error message, maxmem is printed in Dec but ram_size in Hex.
It is better to print them in same format.
Also use error_report instead of fprintf.
Signed-off-by: zhanghailiang <zhang.zhanghailiang@huawei.com>
---
v2:
- Remove redundant "qemu:" in error message, for error_report already prepend
a variation of "qemu" (comment of Eric Blake)
---
vl.c | 22 +++++++++++-----------
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/vl.c b/vl.c
index 5db0d08..71a831b 100644
--- a/vl.c
+++ b/vl.c
@@ -3358,34 +3358,34 @@ int main(int argc, char **argv, char **envp)
sz = qemu_opt_get_size(opts, "maxmem", 0);
if (sz < ram_size) {
- fprintf(stderr, "qemu: invalid -m option value: maxmem "
- "(%" PRIu64 ") <= initial memory ("
- RAM_ADDR_FMT ")\n", sz, ram_size);
+ error_report("invalid -m option value: maxmem "
+ "(0x%" PRIx64 ") <= initial memory (0x"
+ RAM_ADDR_FMT ")", sz, ram_size);
exit(EXIT_FAILURE);
}
slots = qemu_opt_get_number(opts, "slots", 0);
if ((sz > ram_size) && !slots) {
- fprintf(stderr, "qemu: invalid -m option value: maxmem "
- "(%" PRIu64 ") more than initial memory ("
+ error_report("invalid -m option value: maxmem "
+ "(0x%" PRIx64 ") more than initial memory (0x"
RAM_ADDR_FMT ") but no hotplug slots where "
- "specified\n", sz, ram_size);
+ "specified", sz, ram_size);
exit(EXIT_FAILURE);
}
if ((sz <= ram_size) && slots) {
- fprintf(stderr, "qemu: invalid -m option value: %"
+ error_report("invalid -m option value: %"
PRIu64 " hotplug slots where specified but "
- "maxmem (%" PRIu64 ") <= initial memory ("
- RAM_ADDR_FMT ")\n", slots, sz, ram_size);
+ "maxmem (0x%" PRIx64 ") <= initial memory (0x"
+ RAM_ADDR_FMT ")", slots, sz, ram_size);
exit(EXIT_FAILURE);
}
maxram_size = sz;
ram_slots = slots;
} else if ((!maxmem_str && slots_str) ||
(maxmem_str && !slots_str)) {
- fprintf(stderr, "qemu: invalid -m option value: missing "
- "'%s' option\n", slots_str ? "maxmem" : "slots");
+ error_report("invalid -m option value: missing "
+ "'%s' option", slots_str ? "maxmem" : "slots");
exit(EXIT_FAILURE);
}
break;
--
1.7.12.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH v2] vl: Print maxmem in hex format for error message
2014-09-16 7:45 [Qemu-devel] [PATCH v2] vl: Print maxmem in hex format for error message zhanghailiang
@ 2014-09-16 9:12 ` Igor Mammedov
2014-09-16 9:23 ` zhanghailiang
0 siblings, 1 reply; 3+ messages in thread
From: Igor Mammedov @ 2014-09-16 9:12 UTC (permalink / raw)
To: zhanghailiang; +Cc: qemu-trivial, luonengjun, qemu-devel, peter.huangpeng
On Tue, 16 Sep 2014 15:45:32 +0800
zhanghailiang <zhang.zhanghailiang@huawei.com> wrote:
> In error message, maxmem is printed in Dec but ram_size in Hex.
> It is better to print them in same format.
> Also use error_report instead of fprintf.
>
> Signed-off-by: zhanghailiang <zhang.zhanghailiang@huawei.com>
> ---
> v2:
> - Remove redundant "qemu:" in error message, for error_report already prepend
> a variation of "qemu" (comment of Eric Blake)
> ---
> vl.c | 22 +++++++++++-----------
> 1 file changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/vl.c b/vl.c
> index 5db0d08..71a831b 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -3358,34 +3358,34 @@ int main(int argc, char **argv, char **envp)
>
> sz = qemu_opt_get_size(opts, "maxmem", 0);
> if (sz < ram_size) {
> - fprintf(stderr, "qemu: invalid -m option value: maxmem "
> - "(%" PRIu64 ") <= initial memory ("
> - RAM_ADDR_FMT ")\n", sz, ram_size);
> + error_report("invalid -m option value: maxmem "
> + "(0x%" PRIx64 ") <= initial memory (0x"
> + RAM_ADDR_FMT ")", sz, ram_size);
> exit(EXIT_FAILURE);
> }
>
> slots = qemu_opt_get_number(opts, "slots", 0);
> if ((sz > ram_size) && !slots) {
> - fprintf(stderr, "qemu: invalid -m option value: maxmem "
> - "(%" PRIu64 ") more than initial memory ("
> + error_report("invalid -m option value: maxmem "
> + "(0x%" PRIx64 ") more than initial memory (0x"
> RAM_ADDR_FMT ") but no hotplug slots where "
> - "specified\n", sz, ram_size);
> + "specified", sz, ram_size);
> exit(EXIT_FAILURE);
> }
>
> if ((sz <= ram_size) && slots) {
> - fprintf(stderr, "qemu: invalid -m option value: %"
> + error_report("invalid -m option value: %"
> PRIu64 " hotplug slots where specified but "
> - "maxmem (%" PRIu64 ") <= initial memory ("
> - RAM_ADDR_FMT ")\n", slots, sz, ram_size);
> + "maxmem (0x%" PRIx64 ") <= initial memory (0x"
> + RAM_ADDR_FMT ")", slots, sz, ram_size);
> exit(EXIT_FAILURE);
> }
> maxram_size = sz;
> ram_slots = slots;
> } else if ((!maxmem_str && slots_str) ||
> (maxmem_str && !slots_str)) {
> - fprintf(stderr, "qemu: invalid -m option value: missing "
> - "'%s' option\n", slots_str ? "maxmem" : "slots");
> + error_report("invalid -m option value: missing "
indent error
> + "'%s' option", slots_str ? "maxmem" : "slots");
> exit(EXIT_FAILURE);
> }
> break;
with above fixed,
Reviewed-By: Igor Mammedov <imammedo@redhat.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH v2] vl: Print maxmem in hex format for error message
2014-09-16 9:12 ` Igor Mammedov
@ 2014-09-16 9:23 ` zhanghailiang
0 siblings, 0 replies; 3+ messages in thread
From: zhanghailiang @ 2014-09-16 9:23 UTC (permalink / raw)
To: Igor Mammedov; +Cc: qemu-trivial, luonengjun, qemu-devel, peter.huangpeng
On 2014/9/16 17:12, Igor Mammedov wrote:
> On Tue, 16 Sep 2014 15:45:32 +0800
> zhanghailiang<zhang.zhanghailiang@huawei.com> wrote:
>
>> In error message, maxmem is printed in Dec but ram_size in Hex.
>> It is better to print them in same format.
>> Also use error_report instead of fprintf.
>>
>> Signed-off-by: zhanghailiang<zhang.zhanghailiang@huawei.com>
>> ---
>> v2:
>> - Remove redundant "qemu:" in error message, for error_report already prepend
>> a variation of "qemu" (comment of Eric Blake)
>> ---
>> vl.c | 22 +++++++++++-----------
>> 1 file changed, 11 insertions(+), 11 deletions(-)
>>
>> diff --git a/vl.c b/vl.c
>> index 5db0d08..71a831b 100644
>> --- a/vl.c
>> +++ b/vl.c
>> @@ -3358,34 +3358,34 @@ int main(int argc, char **argv, char **envp)
>>
>> sz = qemu_opt_get_size(opts, "maxmem", 0);
>> if (sz< ram_size) {
>> - fprintf(stderr, "qemu: invalid -m option value: maxmem "
>> - "(%" PRIu64 ")<= initial memory ("
>> - RAM_ADDR_FMT ")\n", sz, ram_size);
>> + error_report("invalid -m option value: maxmem "
>> + "(0x%" PRIx64 ")<= initial memory (0x"
>> + RAM_ADDR_FMT ")", sz, ram_size);
>> exit(EXIT_FAILURE);
>> }
>>
>> slots = qemu_opt_get_number(opts, "slots", 0);
>> if ((sz> ram_size)&& !slots) {
>> - fprintf(stderr, "qemu: invalid -m option value: maxmem "
>> - "(%" PRIu64 ") more than initial memory ("
>> + error_report("invalid -m option value: maxmem "
>> + "(0x%" PRIx64 ") more than initial memory (0x"
>> RAM_ADDR_FMT ") but no hotplug slots where "
>> - "specified\n", sz, ram_size);
>> + "specified", sz, ram_size);
>> exit(EXIT_FAILURE);
>> }
>>
>> if ((sz<= ram_size)&& slots) {
>> - fprintf(stderr, "qemu: invalid -m option value: %"
>> + error_report("invalid -m option value: %"
>> PRIu64 " hotplug slots where specified but "
>> - "maxmem (%" PRIu64 ")<= initial memory ("
>> - RAM_ADDR_FMT ")\n", slots, sz, ram_size);
>> + "maxmem (0x%" PRIx64 ")<= initial memory (0x"
>> + RAM_ADDR_FMT ")", slots, sz, ram_size);
>> exit(EXIT_FAILURE);
>> }
>> maxram_size = sz;
>> ram_slots = slots;
>> } else if ((!maxmem_str&& slots_str) ||
>> (maxmem_str&& !slots_str)) {
>> - fprintf(stderr, "qemu: invalid -m option value: missing "
>> - "'%s' option\n", slots_str ? "maxmem" : "slots");
>> + error_report("invalid -m option value: missing "
> indent error
>
>> + "'%s' option", slots_str ? "maxmem" : "slots");
>> exit(EXIT_FAILURE);
>> }
>> break;
>
> with above fixed,
>
Will fix it, Thanks;)
> Reviewed-By: Igor Mammedov<imammedo@redhat.com>
>
> .
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-09-16 9:24 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-09-16 7:45 [Qemu-devel] [PATCH v2] vl: Print maxmem in hex format for error message zhanghailiang
2014-09-16 9:12 ` Igor Mammedov
2014-09-16 9:23 ` zhanghailiang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).