All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Andreas Färber" <afaerber@suse.de>
To: Alexander Graf <agraf@suse.de>
Cc: qemu-devel@nongnu.org, "Peter Maydell" <peter.maydell@linaro.org>,
	"Anthony Liguori" <aliguori@us.ibm.com>,
	"Riku Voipio" <riku.voipio@iki.fi>,
	"Igor Mitsyanko" <i.mitsyanko@samsung.com>,
	"Mark Langsdorf" <mark.langsdorf@calxeda.com>,
	"Evgeny Voevodin" <e.voevodin@samsung.com>,
	"Marcelo Tosatti" <mtosatti@redhat.com>,
	"Gleb Natapov" <gleb@redhat.com>,
	"Luiz Capitulino" <lcapitulino@redhat.com>,
	"Peter Crosthwaite" <peter.crosthwaite@petalogix.com>,
	"Andreas Färber" <andreas.faerber@web.de>,
	qemu-ppc <qemu-ppc@nongnu.org>,
	"Paul Brook" <paul@codesourcery.com>,
	"David Gibson" <david@gibson.dropbear.id.au>,
	kvm <kvm@vger.kernel.org>,
	"Edgar E. Iglesias" <edgar.iglesias@gmail.com>,
	"Maksim Kozlov" <m.kozlov@samsung.com>,
	"Aurelien Jarno" <aurelien@aurel32.net>,
	"Dmitry Solodkiy" <d.solodkiy@samsung.com>
Subject: Re: [Qemu-devel] [PATCH qom-cpu v3 05/14] cpu: Make first_cpu and next_cpu CPUState
Date: Mon, 01 Jul 2013 19:37:05 +0200	[thread overview]
Message-ID: <51D1BE41.40901@suse.de> (raw)
In-Reply-To: <1372268143-3894-6-git-send-email-afaerber@suse.de>

Am 26.06.2013 19:35, schrieb Andreas Färber:
> Move next_cpu from CPU_COMMON to CPUState.
> Move first_cpu variable to qom/cpu.h.
> 
> gdbstub needs to use CPUState::env_ptr for now.
> cpu_copy() no longer needs to save and restore cpu_next.
> 
> Acked-by: Paolo Bonzini <pbonzini@redhat.com>
> [AF: Rebased, simplified cpu_copy()]
> Signed-off-by: Andreas Färber <afaerber@suse.de>

Squashing the following additional conversion after rebase.

Andreas

diff --git a/hw/ppc/e500.c b/hw/ppc/e500.c
index 38f7990..ff86154 100644
--- a/hw/ppc/e500.c
+++ b/hw/ppc/e500.c
@@ -500,7 +500,6 @@ static DeviceState
*ppce500_init_mpic_kvm(PPCE500Params *params,
                                           qemu_irq **irqs)
 {
     DeviceState *dev;
-    CPUPPCState *env;
     CPUState *cs;
     int r;

@@ -512,9 +511,7 @@ static DeviceState
*ppce500_init_mpic_kvm(PPCE500Params *params,
         return NULL;
     }

-    for (env = first_cpu; env != NULL; env = env->next_cpu) {
-        cs = ENV_GET_CPU(env);
-
+    for (cs = first_cpu; cs != NULL; cs = cs->next_cpu) {
         if (kvm_openpic_connect_vcpu(dev, cs)) {


-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

WARNING: multiple messages have this Message-ID (diff)
From: "Andreas Färber" <afaerber@suse.de>
To: Alexander Graf <agraf@suse.de>
Cc: "Peter Maydell" <peter.maydell@linaro.org>,
	"Anthony Liguori" <aliguori@us.ibm.com>,
	"Marcelo Tosatti" <mtosatti@redhat.com>,
	"Igor Mitsyanko" <i.mitsyanko@samsung.com>,
	"Evgeny Voevodin" <e.voevodin@samsung.com>,
	"Mark Langsdorf" <mark.langsdorf@calxeda.com>,
	"Riku Voipio" <riku.voipio@iki.fi>,
	qemu-devel@nongnu.org, "Gleb Natapov" <gleb@redhat.com>,
	"Luiz Capitulino" <lcapitulino@redhat.com>,
	"Peter Crosthwaite" <peter.crosthwaite@petalogix.com>,
	"Dmitry Solodkiy" <d.solodkiy@samsung.com>,
	"Andreas Färber" <andreas.faerber@web.de>,
	qemu-ppc <qemu-ppc@nongnu.org>,
	"Paul Brook" <paul@codesourcery.com>, kvm <kvm@vger.kernel.org>,
	"Edgar E. Iglesias" <edgar.iglesias@gmail.com>,
	"Maksim Kozlov" <m.kozlov@samsung.com>,
	"Aurelien Jarno" <aurelien@aurel32.net>,
	"David Gibson" <david@gibson.dropbear.id.au>
Subject: Re: [Qemu-devel] [PATCH qom-cpu v3 05/14] cpu: Make first_cpu and next_cpu CPUState
Date: Mon, 01 Jul 2013 19:37:05 +0200	[thread overview]
Message-ID: <51D1BE41.40901@suse.de> (raw)
In-Reply-To: <1372268143-3894-6-git-send-email-afaerber@suse.de>

Am 26.06.2013 19:35, schrieb Andreas Färber:
> Move next_cpu from CPU_COMMON to CPUState.
> Move first_cpu variable to qom/cpu.h.
> 
> gdbstub needs to use CPUState::env_ptr for now.
> cpu_copy() no longer needs to save and restore cpu_next.
> 
> Acked-by: Paolo Bonzini <pbonzini@redhat.com>
> [AF: Rebased, simplified cpu_copy()]
> Signed-off-by: Andreas Färber <afaerber@suse.de>

Squashing the following additional conversion after rebase.

Andreas

diff --git a/hw/ppc/e500.c b/hw/ppc/e500.c
index 38f7990..ff86154 100644
--- a/hw/ppc/e500.c
+++ b/hw/ppc/e500.c
@@ -500,7 +500,6 @@ static DeviceState
*ppce500_init_mpic_kvm(PPCE500Params *params,
                                           qemu_irq **irqs)
 {
     DeviceState *dev;
-    CPUPPCState *env;
     CPUState *cs;
     int r;

@@ -512,9 +511,7 @@ static DeviceState
*ppce500_init_mpic_kvm(PPCE500Params *params,
         return NULL;
     }

-    for (env = first_cpu; env != NULL; env = env->next_cpu) {
-        cs = ENV_GET_CPU(env);
-
+    for (cs = first_cpu; cs != NULL; cs = cs->next_cpu) {
         if (kvm_openpic_connect_vcpu(dev, cs)) {


-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

  reply	other threads:[~2013-07-01 17:37 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-26 17:35 [Qemu-devel] [PATCH qom-cpu v3 00/14] QOM CPUState, part 10: CPU loops Andreas Färber
2013-06-26 17:35 ` [PATCH qom-cpu v3 01/14] kvm: Free current_cpu identifier Andreas Färber
2013-06-26 17:35   ` [Qemu-devel] " Andreas Färber
2013-06-26 17:35 ` [Qemu-devel] [PATCH qom-cpu v3 02/14] cpu: Replace cpu_single_env with CPUState current_cpu Andreas Färber
2013-06-26 17:35 ` [PATCH qom-cpu v3 03/14] kvm: Change kvm_remove_all_breakpoints() argument to CPUState Andreas Färber
2013-06-26 17:35   ` [Qemu-devel] " Andreas Färber
2013-06-26 17:35 ` [Qemu-devel] [PATCH qom-cpu v3 04/14] linux-user: Clean up do_syscall() Coding Style for TARGET_NR_exit Andreas Färber
2013-06-26 17:35 ` [PATCH qom-cpu v3 05/14] cpu: Make first_cpu and next_cpu CPUState Andreas Färber
2013-06-26 17:35   ` [Qemu-devel] " Andreas Färber
2013-07-01 17:37   ` Andreas Färber [this message]
2013-07-01 17:37     ` Andreas Färber
2013-06-26 17:35 ` [Qemu-devel] [PATCH qom-cpu v3 06/14] linux-user: Change thread_env to CPUState Andreas Färber
2013-06-26 17:35 ` [Qemu-devel] [PATCH qom-cpu v3 07/14] bsd-user: " Andreas Färber
2013-06-26 17:35 ` [Qemu-devel] [PATCH qom-cpu v3 08/14] intc/arm_gic: Build arm_gic only once Andreas Färber
2013-06-26 17:35 ` [Qemu-devel] [PATCH qom-cpu v3 09/14] intc/openpic: Build openpic " Andreas Färber
2013-06-26 17:35 ` [Qemu-devel] [PATCH qom-cpu v3 10/14] timer/arm_mptimer: Build arm_mptimer " Andreas Färber
2013-06-26 17:35 ` [Qemu-devel] [PATCH qom-cpu v3 11/14] target-ppc: Don't overuse ENV_GET_CPU() Andreas Färber
2013-06-26 17:35 ` [PATCH qom-cpu v3 12/14] target-s390x: " Andreas Färber
2013-06-26 17:35   ` [Qemu-devel] " Andreas Färber
2013-06-26 17:35 ` [PATCH qom-cpu v3 13/14] target-s390x: Change handle_{hypercall,diag}() argument to S390CPU Andreas Färber
2013-06-26 17:35   ` [Qemu-devel] [PATCH qom-cpu v3 13/14] target-s390x: Change handle_{hypercall, diag}() " Andreas Färber
2013-06-26 17:35 ` [Qemu-devel] [PATCH qom-cpu v3 14/14] target-i386: Don't overuse CPUArchState Andreas Färber
2013-07-02  0:31 ` [Qemu-devel] [PATCH qom-cpu v3 00/14] QOM CPUState, part 10: CPU loops Andreas Färber

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=51D1BE41.40901@suse.de \
    --to=afaerber@suse.de \
    --cc=agraf@suse.de \
    --cc=aliguori@us.ibm.com \
    --cc=andreas.faerber@web.de \
    --cc=aurelien@aurel32.net \
    --cc=d.solodkiy@samsung.com \
    --cc=david@gibson.dropbear.id.au \
    --cc=e.voevodin@samsung.com \
    --cc=edgar.iglesias@gmail.com \
    --cc=gleb@redhat.com \
    --cc=i.mitsyanko@samsung.com \
    --cc=kvm@vger.kernel.org \
    --cc=lcapitulino@redhat.com \
    --cc=m.kozlov@samsung.com \
    --cc=mark.langsdorf@calxeda.com \
    --cc=mtosatti@redhat.com \
    --cc=paul@codesourcery.com \
    --cc=peter.crosthwaite@petalogix.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.org \
    --cc=riku.voipio@iki.fi \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.