* [Qemu-devel] [PATCH v3] qemu crashes when a negative number used for 'maxcpus'
@ 2017-08-28 13:53 Seeteena Thoufeek
2017-08-28 14:27 ` Fam Zheng
2017-08-28 14:40 ` Bharata B Rao
0 siblings, 2 replies; 3+ messages in thread
From: Seeteena Thoufeek @ 2017-08-28 13:53 UTC (permalink / raw)
To: qemu-devel, qemu-ppc, famz, pbonzini; +Cc: s1seetee
---Steps to Reproduce---
When passed a negative number to 'maxcpus' parameter, Qemu aborts
with a core dump.
Run the following command with maxcpus argument as negative number
ppc64-softmmu/qemu-system-ppc64 --nographic -vga none -machine
pseries,accel=kvm,kvm-type=HV -m size=200g -device virtio-blk-pci,
drive=rootdisk -drive file=/home/images/pegas-1.0-ppc64le.qcow2,
if=none,cache=none,id=rootdisk,format=qcow2 -monitor telnet
:127.0.0.1:1234,server,nowait -net nic,model=virtio -net
user -redir tcp:2000::22 -device nec-usb-xhci -smp 8,cores=1,
threads=1,maxcpus=-12
(process:12149): GLib-ERROR **: gmem.c:130: failed to allocate
18446744073709550568 bytes
Trace/breakpoint trap
Reported-by: R.Nageswara Sastry <rnsastry@linux.vnet.ibm.com>
Signed-off-by: Seeteena Thoufeek <s1seetee@linux.vnet.ibm.com>
Reviewed-by: Bharata B Rao <bharata@linux.vnet.ibm.com>
---
vl.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/vl.c b/vl.c
index 8e247cc..fb45b6d 100644
--- a/vl.c
+++ b/vl.c
@@ -1244,7 +1244,10 @@ static void smp_parse(QemuOpts *opts)
}
max_cpus = qemu_opt_get_number(opts, "maxcpus", cpus);
-
+ if (max_cpus <= 0) {
+ error_report("Invalid max_cpus : %d", max_cpus);
+ exit(1);
+ }
if (max_cpus < cpus) {
error_report("maxcpus must be equal to or greater than smp");
exit(1);
--
1.8.3.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH v3] qemu crashes when a negative number used for 'maxcpus'
2017-08-28 13:53 [Qemu-devel] [PATCH v3] qemu crashes when a negative number used for 'maxcpus' Seeteena Thoufeek
@ 2017-08-28 14:27 ` Fam Zheng
2017-08-28 14:40 ` Bharata B Rao
1 sibling, 0 replies; 3+ messages in thread
From: Fam Zheng @ 2017-08-28 14:27 UTC (permalink / raw)
To: Seeteena Thoufeek; +Cc: qemu-devel, qemu-ppc, pbonzini
Hi Seeteena, thanks for sending the patch and fixing the coding style!
I suggest to change subject to "vl: exit if maxcpus is negative". The subject of
a patch email is going to be the summary of the commit message when applied,
therefore it should be worded to summarize the change. Commonly there is a
"subsystem prefix" followed by a colon, like "vl:", "net:". While there is no
hard rule, the most suitable prefix can be guessed by inspecting the log of the
changed files. In the case of this patch: "git log vl.c".
A more detailed guide on submitting "perfect" QEMU patches can be found here,
in case for your future reference:
https://wiki.qemu.org/Contribute/SubmitAPatch
On Mon, 08/28 19:23, Seeteena Thoufeek wrote:
> ---Steps to Reproduce---
>
> When passed a negative number to 'maxcpus' parameter, Qemu aborts
> with a core dump.
>
> Run the following command with maxcpus argument as negative number
>
> ppc64-softmmu/qemu-system-ppc64 --nographic -vga none -machine
> pseries,accel=kvm,kvm-type=HV -m size=200g -device virtio-blk-pci,
> drive=rootdisk -drive file=/home/images/pegas-1.0-ppc64le.qcow2,
> if=none,cache=none,id=rootdisk,format=qcow2 -monitor telnet
> :127.0.0.1:1234,server,nowait -net nic,model=virtio -net
> user -redir tcp:2000::22 -device nec-usb-xhci -smp 8,cores=1,
> threads=1,maxcpus=-12
>
> (process:12149): GLib-ERROR **: gmem.c:130: failed to allocate
> 18446744073709550568 bytes
>
> Trace/breakpoint trap
>
> Reported-by: R.Nageswara Sastry <rnsastry@linux.vnet.ibm.com>
> Signed-off-by: Seeteena Thoufeek <s1seetee@linux.vnet.ibm.com>
> Reviewed-by: Bharata B Rao <bharata@linux.vnet.ibm.com>
> ---
Would be good if there is a comment below a "---" line on how current revision
differs from the previous one, like:
---
v3: Fix coding style pointed out by patchew.
> vl.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/vl.c b/vl.c
> index 8e247cc..fb45b6d 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -1244,7 +1244,10 @@ static void smp_parse(QemuOpts *opts)
> }
>
> max_cpus = qemu_opt_get_number(opts, "maxcpus", cpus);
> -
> + if (max_cpus <= 0) {
> + error_report("Invalid max_cpus : %d", max_cpus);
The space before ":" can be dropped, I think.
> + exit(1);
> + }
> if (max_cpus < cpus) {
> error_report("maxcpus must be equal to or greater than smp");
> exit(1);
> --
> 1.8.3.1
>
Fam
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH v3] qemu crashes when a negative number used for 'maxcpus'
2017-08-28 13:53 [Qemu-devel] [PATCH v3] qemu crashes when a negative number used for 'maxcpus' Seeteena Thoufeek
2017-08-28 14:27 ` Fam Zheng
@ 2017-08-28 14:40 ` Bharata B Rao
1 sibling, 0 replies; 3+ messages in thread
From: Bharata B Rao @ 2017-08-28 14:40 UTC (permalink / raw)
To: Seeteena Thoufeek; +Cc: qemu-devel, qemu-ppc, Fam Zheng, Paolo Bonzini
Seeteena,
On Mon, Aug 28, 2017 at 7:23 PM, Seeteena Thoufeek <
s1seetee@linux.vnet.ibm.com> wrote:
> ---Steps to Reproduce---
>
> When passed a negative number to 'maxcpus' parameter, Qemu aborts
> with a core dump.
>
> Run the following command with maxcpus argument as negative number
>
> ppc64-softmmu/qemu-system-ppc64 --nographic -vga none -machine
> pseries,accel=kvm,kvm-type=HV -m size=200g -device virtio-blk-pci,
> drive=rootdisk -drive file=/home/images/pegas-1.0-ppc64le.qcow2,
> if=none,cache=none,id=rootdisk,format=qcow2 -monitor telnet
> :127.0.0.1:1234,server,nowait -net nic,model=virtio -net
> user -redir tcp:2000::22 -device nec-usb-xhci -smp 8,cores=1,
> threads=1,maxcpus=-12
>
> (process:12149): GLib-ERROR **: gmem.c:130: failed to allocate
> 18446744073709550568 bytes
>
> Trace/breakpoint trap
>
> Reported-by: R.Nageswara Sastry <rnsastry@linux.vnet.ibm.com>
> Signed-off-by: Seeteena Thoufeek <s1seetee@linux.vnet.ibm.com>
> Reviewed-by: Bharata B Rao <bharata@linux.vnet.ibm.com>
>
In the bugzilla, I was only suggesting to post the fix upstream and it
doesn't mean a Reviewed-by. You might want to remove this in your next
version.
Regards,
Bharata.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-08-28 14:40 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-28 13:53 [Qemu-devel] [PATCH v3] qemu crashes when a negative number used for 'maxcpus' Seeteena Thoufeek
2017-08-28 14:27 ` Fam Zheng
2017-08-28 14:40 ` Bharata B Rao
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.