[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-2.6 04/10] s390x/cpu: Tolerate max_cpus
From: |
Cornelia Huck |
Subject: |
Re: [Qemu-devel] [PATCH for-2.6 04/10] s390x/cpu: Tolerate max_cpus |
Date: |
Thu, 10 Mar 2016 10:52:55 +0100 |
On Wed, 9 Mar 2016 22:55:30 +0100
Thomas Huth <address@hidden> wrote:
> While you're at it, it might be better to use g_new0 here instead
> (see e.g. https://patchwork.ozlabs.org/patch/517377/ for a description
> why this is better).
Using g_new0 sounds sensible; but rather than respinning again, I'll
just put the following on top:
From 0a11840959f61861672858ce961fc34b71efa3be Mon Sep 17 00:00:00 2001
From: Cornelia Huck <address@hidden>
Date: Thu, 10 Mar 2016 10:19:46 +0100
Subject: [PATCH] s390x/cpu: use g_new0
Let's use g_new0 to allocate cpu_states.
Suggested-by: Thomas Huth <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>
---
hw/s390x/s390-virtio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/s390x/s390-virtio.c b/hw/s390x/s390-virtio.c
index 4ea9040..7c6e281 100644
--- a/hw/s390x/s390-virtio.c
+++ b/hw/s390x/s390-virtio.c
@@ -103,7 +103,7 @@ void s390_init_cpus(MachineState *machine)
machine->cpu_model = "host";
}
- cpu_states = g_malloc0(sizeof(S390CPU *) * max_cpus);
+ cpu_states = g_new0(S390CPU *, max_cpus);
for (i = 0; i < max_cpus; i++) {
name = g_strdup_printf("cpu[%i]", i);
--
2.7.2
- [Qemu-devel] [PATCH for-2.6 05/10] s390x/cpu: Add CPU property links, (continued)
- [Qemu-devel] [PATCH for-2.6 05/10] s390x/cpu: Add CPU property links, Cornelia Huck, 2016/03/09
- [Qemu-devel] [PATCH for-2.6 03/10] s390x/cpu: Get rid of side effects when creating a vcpu, Cornelia Huck, 2016/03/09
- [Qemu-devel] [PATCH for-2.6 02/10] s390x/cpu: Set initial CPU state in common routine, Cornelia Huck, 2016/03/09
- [Qemu-devel] [PATCH for-2.6 08/10] pc-bios/s390-ccw: fix old bug in ptr increment, Cornelia Huck, 2016/03/09
- [Qemu-devel] [PATCH for-2.6 07/10] s390x/cpu: Allow hotplug of CPUs, Cornelia Huck, 2016/03/09
- [Qemu-devel] [PATCH for-2.6 09/10] s390x: Introduce machine definition macros, Cornelia Huck, 2016/03/09
- [Qemu-devel] [PATCH for-2.6 06/10] s390x/cpu: Add error handling to cpu creation, Cornelia Huck, 2016/03/09
- [Qemu-devel] [PATCH for-2.6 01/10] s390x/cpu: Cleanup init in preparation for hotplug, Cornelia Huck, 2016/03/09
- [Qemu-devel] [PATCH for-2.6 04/10] s390x/cpu: Tolerate max_cpus, Cornelia Huck, 2016/03/09
- [Qemu-devel] [PATCH for-2.6 10/10] s390x: Introduce S390MachineClass, Cornelia Huck, 2016/03/09