Re: [Qemu-devel] [PATCH v2 01/12] ARM: Export cpu_env

2013-06-28 Thread Andreas Färber
Am 28.06.2013 16:20, schrieb Peter Maydell: > On 30 April 2013 13:00, Andreas Färber wrote: >> Am 30.04.2013 13:54, schrieb Alexander Graf: >>> Am 30.04.2013 um 13:42 schrieb Andreas Färber : >>> Am 30.04.2013 08:36, schrieb John Rigby: > From: Alexander Graf > diff --git a/target-ar

Re: [Qemu-devel] [PATCH v2 01/12] ARM: Export cpu_env

2013-06-28 Thread Andreas Färber
Am 28.06.2013 16:35, schrieb Peter Maydell: > On 28 June 2013 15:28, Andreas Färber wrote: >> I don't mind that cpu_env change getting committed as interim solution, >> so far I did not come up with a better patch - we'd need to split out >> host parts from tcg/tcg.h first, for which I did not fin

Re: [Qemu-devel] [PATCH v2 01/12] ARM: Export cpu_env

2013-06-28 Thread Peter Maydell
On 28 June 2013 15:28, Andreas Färber wrote: > I don't mind that cpu_env change getting committed as interim solution, > so far I did not come up with a better patch - we'd need to split out > host parts from tcg/tcg.h first, for which I did not find time yet. Interim solution on the path to wher

Re: [Qemu-devel] [PATCH v2 01/12] ARM: Export cpu_env

2013-06-28 Thread Alexander Graf
On 28.06.2013, at 16:20, Peter Maydell wrote: > On 30 April 2013 13:00, Andreas Färber wrote: >> Am 30.04.2013 13:54, schrieb Alexander Graf: >>> Am 30.04.2013 um 13:42 schrieb Andreas Färber : >>> Am 30.04.2013 08:36, schrieb John Rigby: > From: Alexander Graf > diff --git a/targ

Re: [Qemu-devel] [PATCH v2 01/12] ARM: Export cpu_env

2013-06-28 Thread Peter Maydell
On 30 April 2013 13:00, Andreas Färber wrote: > Am 30.04.2013 13:54, schrieb Alexander Graf: >> Am 30.04.2013 um 13:42 schrieb Andreas Färber : >> >>> Am 30.04.2013 08:36, schrieb John Rigby: From: Alexander Graf diff --git a/target-arm/translate.c b/target-arm/translate.c index 67

Re: [Qemu-devel] [PATCH v2 01/12] ARM: Export cpu_env

2013-05-01 Thread Richard Henderson
On 2013-04-30 13:00, Andreas Färber wrote: Not sure if you understood my point? Exposing cpu_env from target-arm would seem to torpedo our efforts to link target-arm and, e.g., target-microblaze together, since they all have cpu_env. I don't really care if it's qom/cpu.c or tcg/shared.c or renami

Re: [Qemu-devel] [PATCH v2 01/12] ARM: Export cpu_env

2013-04-30 Thread Alexander Graf
On 04/30/2013 02:00 PM, Andreas Färber wrote: Am 30.04.2013 13:54, schrieb Alexander Graf: Am 30.04.2013 um 13:42 schrieb Andreas Färber: Am 30.04.2013 08:36, schrieb John Rigby: From: Alexander Graf The cpu_env tcg variable will be used by both the AArch32 and AArch64 handling code. Unstati

Re: [Qemu-devel] [PATCH v2 01/12] ARM: Export cpu_env

2013-04-30 Thread Andreas Färber
Am 30.04.2013 13:54, schrieb Alexander Graf: > > Am 30.04.2013 um 13:42 schrieb Andreas Färber : > >> Am 30.04.2013 08:36, schrieb John Rigby: >>> From: Alexander Graf >>> >>> The cpu_env tcg variable will be used by both the AArch32 and AArch64 >>> handling code. Unstaticify it, so that both si

Re: [Qemu-devel] [PATCH v2 01/12] ARM: Export cpu_env

2013-04-30 Thread Alexander Graf
Am 30.04.2013 um 13:42 schrieb Andreas Färber : > Am 30.04.2013 08:36, schrieb John Rigby: >> From: Alexander Graf >> >> The cpu_env tcg variable will be used by both the AArch32 and AArch64 >> handling code. Unstaticify it, so that both sides can make use of it. >> >> Signed-off-by: Alexande

Re: [Qemu-devel] [PATCH v2 01/12] ARM: Export cpu_env

2013-04-30 Thread Andreas Färber
Am 30.04.2013 08:36, schrieb John Rigby: > From: Alexander Graf > > The cpu_env tcg variable will be used by both the AArch32 and AArch64 > handling code. Unstaticify it, so that both sides can make use of it. > > Signed-off-by: Alexander Graf John, when you submit patches, you need to add you

[Qemu-devel] [PATCH v2 01/12] ARM: Export cpu_env

2013-04-29 Thread John Rigby
From: Alexander Graf The cpu_env tcg variable will be used by both the AArch32 and AArch64 handling code. Unstaticify it, so that both sides can make use of it. Signed-off-by: Alexander Graf --- target-arm/translate.c |2 +- target-arm/translate.h |2 ++ 2 files changed, 3 insertions(+