Re: [Qemu-devel] [PATCH v4 0/9] VMXNET3 paravirtual NIC device implementation

2012-03-21 Thread Gerhard Wiesinger
On 20.03.2012 09:00, Dmitry Fleytman wrote: Hello, Gerhard I've tested telnet connections on Knoppix running on QEMU-KVM with patch V5. Everything works fine on my setup. What is your network setup? How do you connect tap1 interface to the outer world? Hello Dmitry , Did you also test with ta

Re: [Qemu-devel] [PATCH v4 2/7] RTC: Update the RTC clock only when reading it

2012-03-21 Thread Zhang, Yang Z
> -Original Message- > From: Stefano Stabellini [mailto:stefano.stabell...@eu.citrix.com] > Sent: Tuesday, March 20, 2012 10:16 PM > To: Zhang, Yang Z > Cc: qemu-devel@nongnu.org; Paolo Bonzini; aligu...@us.ibm.com; > k...@vger.kernel.org > Subject: Re: [Qemu-devel] [PATCH v4 2/7] RTC: Upda

Re: [Qemu-devel] [PATCH v4 4/7] RTC: Set internal millisecond register to 500ms when reset divider

2012-03-21 Thread Zhang, Yang Z
> -Original Message- > From: Stefano Stabellini [mailto:stefano.stabell...@eu.citrix.com] > Sent: Wednesday, March 21, 2012 1:39 AM > To: Zhang, Yang Z > Cc: qemu-devel@nongnu.org; Paolo Bonzini; aligu...@us.ibm.com; > k...@vger.kernel.org > Subject: Re: [Qemu-devel] [PATCH v4 4/7] RTC: Set

Re: [Qemu-devel] [PATCH v4 5/7] RTC:Add RTC update-ended interrupt support

2012-03-21 Thread Paolo Bonzini
Il 20/03/2012 19:35, Stefano Stabellini ha scritto: > This is the function that is used to figure out whether we need the > timers or not, the condition seems to be: > > (Not (REG_C_UF | REG_C_AF)) And (Not (REG_B_SET)) > > Shouldn't actually check for UIE being enabled? No, you need to set UF i

Re: [Qemu-devel] [PATCH 1/4] add MIPS DSP helpers define

2012-03-21 Thread Stefan Weil
Am 12.03.2012 09:32, schrieb Jia Liu: This patch is the helper define of MIPS ASE DSP. Signed-off-by: Jia Liu --- target-mips/helper.h | 152 ++ 1 files changed, 152 insertions(+), 0 deletions(-) diff --git a/target-mips/helper.h b/target-mips/

Re: [Qemu-devel] [PATCH 2/4] add MIPS DSP helpers implement

2012-03-21 Thread Stefan Weil
Am 12.03.2012 09:32, schrieb Jia Liu: This patch is the helper implementation of MIPS ASE DSP. Signed-off-by: Jia Liu --- target-mips/op_helper.c | 3936 +++ 1 files changed, 3936 insertions(+), 0 deletions(-) diff --git a/target-mips/op_helper.c b

Re: [Qemu-devel] [PATCH 3/4] add MIPS DSP translation

2012-03-21 Thread Stefan Weil
Please see my inline comments. Regards, Stefan Weil Am 12.03.2012 09:32, schrieb Jia Liu: This patch is the translation of MIPS ASE DSP. Signed-off-by: Jia Liu --- target-mips/translate.c | 1114 +-- 1 files changed, 1088 insertions(+), 26 deleti

Re: [Qemu-devel] [PATCH 4/4] add MIPS DSP testcase

2012-03-21 Thread Stefan Weil
Am 12.03.2012 09:32, schrieb Jia Liu: This patch is the testcases of MIPS ASE DSP. Signed-off-by: Jia Liu --- tests/tcg/mips/mips32-dsp/Makefile | 133 tests/tcg/mips/mips32-dsp/absq_s_ph.c | 28 + tests/tcg/mips/mips32-dsp/absq_s_w.c | 35 ++ tests/tcg/mips/mips3

Re: [Qemu-devel] qemu prompt comes up instead of all the kernel stuff

2012-03-21 Thread Mulyadi Santosa
Hi.. On Tue, Mar 20, 2012 at 18:17, Krishna Pavan wrote: > > Hi, > I have tried a kernel to be loaded. > I get QEMU prompt and not any kernel text. > Please tell me why it is happening so. Here's the screenshot. care to tell us the exact command line you used to execute Qemu? And what OS did you

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-21 Thread Gleb Natapov
On Wed, Mar 21, 2012 at 08:56:03AM +0800, Wen Congyang wrote: > At 03/20/2012 11:45 PM, Gleb Natapov Wrote: > > On Tue, Mar 20, 2012 at 05:59:16PM +0800, Wen Congyang wrote: > >> At 03/19/2012 03:33 PM, Wen Congyang Wrote: > >>> At 03/08/2012 03:57 PM, Wen Congyang Wrote: > We can know the gue

Re: [Qemu-devel] [PATCH 10/12] trace: [tracetool] Automatically establish available backends and formats

2012-03-21 Thread Stefan Hajnoczi
2012/3/20 Lluís Vilanova : > Stefan Hajnoczi writes: > >> 2012/3/20 Lluís Vilanova : >>> Stefan Hajnoczi writes: >>> 2012/3/13 Lluís Vilanova : > Adds decorators to establish which backend and/or format each routine is > meant > to process. > > With this, tables enumeratin

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-21 Thread Wen Congyang
At 03/21/2012 05:11 PM, Gleb Natapov Wrote: > On Wed, Mar 21, 2012 at 08:56:03AM +0800, Wen Congyang wrote: >> At 03/20/2012 11:45 PM, Gleb Natapov Wrote: >>> On Tue, Mar 20, 2012 at 05:59:16PM +0800, Wen Congyang wrote: At 03/19/2012 03:33 PM, Wen Congyang Wrote: > At 03/08/2012 03:57 PM,

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-21 Thread Gleb Natapov
On Wed, Mar 21, 2012 at 05:35:49PM +0800, Wen Congyang wrote: > At 03/21/2012 05:11 PM, Gleb Natapov Wrote: > > On Wed, Mar 21, 2012 at 08:56:03AM +0800, Wen Congyang wrote: > >> At 03/20/2012 11:45 PM, Gleb Natapov Wrote: > >>> On Tue, Mar 20, 2012 at 05:59:16PM +0800, Wen Congyang wrote: > A

Re: [Qemu-devel] [PATCH] fix incorrect bracket in tracetool

2012-03-21 Thread Stefan Hajnoczi
On Tue, Mar 20, 2012 at 5:11 PM, Lee Essen wrote: > On 20/03/2012 16:59, Stefan Hajnoczi wrote: >> >> On Mon, Mar 19, 2012 at 1:35 PM, Lee Essen >>  wrote: >>> >>> On 19 Mar 2012, at 12:32, Andreas Färber wrote: >>> Am 19.03.2012 13:05, schrieb Lee Essen: In my original (way-too-long) pa

[Qemu-devel] Thoughts around dtrace linking...

2012-03-21 Thread Lee Essen
Hi, I've been trying to find a sensible way to solve the Solaris/Illumos dtrace requirement to pass all the objs to the dtrace command so that the resultant object file contains all the symbols needed to properly link the relevant binary. The easiest way to do this is just prior to linking t

Re: [Qemu-devel] [PATCH v2] tracetool dtrace disabled-events fix

2012-03-21 Thread Stefan Hajnoczi
On Tue, Mar 20, 2012 at 05:02:40PM +, Lee Essen wrote: > If there are "disabled" entries in the trace-events file then > linetod_nop() is called if the backend is dtrace, it's currently > not present. Also equivalent fix for stap. > > Signed-off-by: Lee Essen > > -- Thanks, applied to the tr

Re: [Qemu-devel] [PATCH] Remove PCI class code from virtio balloon device

2012-03-21 Thread Stefan Hajnoczi
On Tue, Mar 20, 2012 at 09:19:47PM +1100, David Gibson wrote: > On Tue, Mar 20, 2012 at 09:54:20AM +, Stefan Hajnoczi wrote: > > On Tue, Mar 20, 2012 at 12:42 AM, David Gibson > > wrote: > > > On Mon, Mar 19, 2012 at 11:33:10AM +, Stefan Hajnoczi wrote: > > >> On Mon, Mar 19, 2012 at 03:59

Re: [Qemu-devel] [PATCH] Remove PCI class code from virtio balloon device

2012-03-21 Thread Stefan Hajnoczi
Hi, It seems your Mail-Followup-To: header causes my client to drop you from the To: list. On Wed, Mar 21, 2012 at 11:26 AM, Stefan Hajnoczi wrote: > On Tue, Mar 20, 2012 at 09:19:47PM +1100, David Gibson wrote: >> On Tue, Mar 20, 2012 at 09:54:20AM +, Stefan Hajnoczi wrote: >> > On Tue, Mar

Re: [Qemu-devel] [PATCH 1/3] test: remove qemu-ga reference

2012-03-21 Thread Stefan Hajnoczi
On Tue, Mar 20, 2012 at 12:18 AM, Michael Roth wrote: > This was added by mistake a while back. > > Signed-off-by: Michael Roth > --- >  tests/Makefile |    2 +- >  1 files changed, 1 insertions(+), 1 deletions(-) After applying your series and rebuilding, I get the following error. I checked th

Re: [Qemu-devel] [PATCH V2 1/3] exynos4210: add Exynos4210 i2c implementation

2012-03-21 Thread Peter Maydell
On 15 March 2012 07:35, Igor Mitsyanko wrote: > Create 9 exynos4210 i2c interfaces. > > Signed-off-by: Igor Mitsyanko Mostly this looks OK but I still find the i2c slave stuff odd -- should the controller really register itself as a slave on its own bus? Doesn't this mean that the controller can

Re: [Qemu-devel] [PATCH v3] Man page: Add -global description

2012-03-21 Thread Markus Armbruster
Miroslav Rezanina writes: > There's only TODO information in qemu man page for -global option. This is a > basic description of this option with simple example. > > Signed-off-by: Miroslav Rezanina > > Patch: > -- > diff --git a/qemu-options.hx b/qemu-options.hx > index daefce3..db8be37 100644

Re: [Qemu-devel] [PATCH] Fix typo in i400FX chipset init code

2012-03-21 Thread Markus Armbruster
Alexey Korolev writes: >> Hi, >> >> There is a typo in i440FX init code. This is causing problems when >> somebody wants to access 64bit PCI range. >> >> >> Signed-off-by: Alexey Korolev >> --- >> >> hw/piix_pci.c |2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --

Re: [Qemu-devel] [PATCH] Fix typo in i400FX chipset init code

2012-03-21 Thread Andreas Färber
Hi, Am 21.03.2012 06:26, schrieb Alexey Korolev: >> Hi, >> >> There is a typo in i440FX init code. This is causing problems when >> somebody wants to access 64bit PCI range. >> >> >> Signed-off-by: Alexey Korolev >> --- >> >> hw/piix_pci.c |2 +- >> 1 files changed, 1 insertions(+), 1 deleti

[Qemu-devel] [PATCH] fix '-cpu ?' Segfault

2012-03-21 Thread Eduardo Habkost
Fix stupid copy&paste mistake at commit ecf40beae7dcbb057d4f115207f9d8276832a774: I moved code around but kept "optarg" on the cpu_list() call. Reported-by: Jiri Denemark Signed-off-by: Eduardo Habkost --- vl.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/vl.c b/vl.

Re: [Qemu-devel] [PATCH] Fix typo in i400FX chipset init code

2012-03-21 Thread Andreas Färber
Am 21.03.2012 13:28, schrieb Markus Armbruster: > Alexey Korolev writes: > >>> Hi, >>> >>> There is a typo in i440FX init code. This is causing problems when >>> somebody wants to access 64bit PCI range. >>> >>> >>> Signed-off-by: Alexey Korolev >>> --- >>> >>> hw/piix_pci.c |2 +- >>> 1 fi

[Qemu-devel] [PATCH uq/master] kvm: Drop redundant kvm_enabled from cpu_thread_is_idle

2012-03-21 Thread Jan Kiszka
This is now implied by kvm_irqchip_in_kernel. Signed-off-by: Jan Kiszka --- cpus.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/cpus.c b/cpus.c index 17b055f..cf732b4 100644 --- a/cpus.c +++ b/cpus.c @@ -421,8 +421,7 @@ static bool cpu_thread_is_idle(CPUState *env)

[Qemu-devel] [PATCH uq/master] kvm: Drop unused kvm_pit_in_kernel

2012-03-21 Thread Jan Kiszka
This is now implied by kvm_irqchip_in_kernel. Signed-off-by: Jan Kiszka --- kvm-all.c |6 -- kvm-stub.c |6 -- kvm.h |2 -- 3 files changed, 0 insertions(+), 14 deletions(-) diff --git a/kvm-all.c b/kvm-all.c index 21c7dd2..7f8c188 100644 --- a/kvm-all.c +++ b/kvm-all.

Re: [Qemu-devel] [PATCH 3/3] configure: add --confsuffix option

2012-03-21 Thread Paolo Bonzini
Il 20/03/2012 21:15, Eduardo Habkost ha scritto: > This way we can choose a configuration file path different from > ${sysconfdir}/PACKAGE. > > This also changes the configure output to show the full configuration > dir path (including $confsuffix), instead of just $sysconfdir. Can you please app

Re: [Qemu-devel] [PATCH 1/3] Makefile: use $(confdir) instead of hardcoding $(sysconfdir)/qemu

2012-03-21 Thread Paolo Bonzini
Il 20/03/2012 21:15, Eduardo Habkost ha scritto: > Signed-off-by: Eduardo Habkost > --- > Makefile |4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/Makefile b/Makefile > index 1bc3cb0..9d583c4 100644 > --- a/Makefile > +++ b/Makefile > @@ -279,8 +279,8 @@ ifdef CO

Re: [Qemu-devel] [PATCH 2/3] qemu-options.hx: refer to confdir instead of sysconfdir on docs

2012-03-21 Thread Paolo Bonzini
Il 20/03/2012 21:15, Eduardo Habkost ha scritto: > The current docs are wrong: ${sysconfdir} is (by default) /etc, > ${confdir} is (by default) /etc/qemu, that's where the config files are > stored. > > Signed-off-by: Eduardo Habkost > --- > qemu-options.hx |4 ++-- > 1 files changed, 2 inse

[Qemu-devel] [PATCH v4] Man page: Add -global description

2012-03-21 Thread Miroslav Rezanina
There's only TODO information in qemu man page for -global option. This is a basic description of this option with simple example. Signed-off-by: Miroslav Rezanina v4: - break long line v3: - add use case description - use prop instead of property v2: - Use better value in example Patch

Re: [Qemu-devel] [PATCH] Fix typo in i400FX chipset init code

2012-03-21 Thread Michael S. Tsirkin
On Wed, Feb 29, 2012 at 02:35:14PM +1300, Alexey Korolev wrote: > Hi, > > There is a typo in i440FX init code. This is causing problems when > somebody wants to access 64bit PCI range. > > > Signed-off-by: Alexey Korolev I've fixed the commit message and applied. How does one trigger the probl

Re: [Qemu-devel] Thoughts around dtrace linking...

2012-03-21 Thread Andreas Färber
Hi, Am 21.03.2012 11:45, schrieb Lee Essen: > I've been trying to find a sensible way to solve the Solaris/Illumos > dtrace requirement to pass all the objs to the dtrace command so that > the resultant object file contains all the symbols needed to properly > link the relevant binary. > > The ea

Re: [Qemu-devel] [PATCH] Fix typo in i400FX chipset init code

2012-03-21 Thread Michael S. Tsirkin
On Wed, Mar 21, 2012 at 01:32:44PM +0100, Andreas Färber wrote: > Hi, > > Am 21.03.2012 06:26, schrieb Alexey Korolev: > >> Hi, > >> > >> There is a typo in i440FX init code. This is causing problems when > >> somebody wants to access 64bit PCI range. > >> > >> > >> Signed-off-by: Alexey Korolev

Re: [Qemu-devel] [PATCH] Remove PCI class code from virtio balloon device

2012-03-21 Thread Michael S. Tsirkin
On Wed, Mar 21, 2012 at 11:26:15AM +, Stefan Hajnoczi wrote: > On Tue, Mar 20, 2012 at 09:19:47PM +1100, David Gibson wrote: > > On Tue, Mar 20, 2012 at 09:54:20AM +, Stefan Hajnoczi wrote: > > > On Tue, Mar 20, 2012 at 12:42 AM, David Gibson > > > wrote: > > > > On Mon, Mar 19, 2012 at 11

Re: [Qemu-devel] [PATCH V2 1/3] exynos4210: add Exynos4210 i2c implementation

2012-03-21 Thread Peter Maydell
On 21 March 2012 13:07, Igor Mitsyanko wrote: > On 03/21/2012 03:55 PM, Peter Maydell wrote: >> I suspect that what's happening here is that the hardware >> lets you put the i2c controller into slave mode so some >> other device on the bus can be a master. But QEMU's >> i2c bus abstraction doesn't

Re: [Qemu-devel] [PATCH] fix '-cpu ?' Segfault

2012-03-21 Thread Stefan Hajnoczi
On Wed, Mar 21, 2012 at 09:33:40AM -0300, Eduardo Habkost wrote: > Fix stupid copy&paste mistake at commit > ecf40beae7dcbb057d4f115207f9d8276832a774: I moved code around but kept > "optarg" on the cpu_list() call. > > Reported-by: Jiri Denemark > Signed-off-by: Eduardo Habkost > --- > vl.c |

Re: [Qemu-devel] [PATCH] Remove PCI class code from virtio balloon device

2012-03-21 Thread David Gibson
On Wed, Mar 21, 2012 at 11:28:47AM +, Stefan Hajnoczi wrote: > Hi, > It seems your Mail-Followup-To: header causes my client to drop you > from the To: list. Not mine, it's added by the list AFAICT. And it's frickin' annoying. -- David Gibson| I'll have my music baroque,

Re: [Qemu-devel] [PATCH 2/3] qemu-options.hx: refer to confdir instead of sysconfdir on docs

2012-03-21 Thread Eduardo Habkost
On Wed, Mar 21, 2012 at 01:43:58PM +0100, Paolo Bonzini wrote: > Il 20/03/2012 21:15, Eduardo Habkost ha scritto: > > The current docs are wrong: ${sysconfdir} is (by default) /etc, > > ${confdir} is (by default) /etc/qemu, that's where the config files are > > stored. > > > > Signed-off-by: Eduar

Re: [Qemu-devel] [PATCH uq/master] kvm: Drop redundant kvm_enabled from cpu_thread_is_idle

2012-03-21 Thread Avi Kivity
On 03/21/2012 02:36 PM, Jan Kiszka wrote: > This is now implied by kvm_irqchip_in_kernel. > > Applied, thanks. -- error compiling committee.c: too many arguments to function

Re: [Qemu-devel] [PATCH uq/master] kvm: Drop unused kvm_pit_in_kernel

2012-03-21 Thread Avi Kivity
On 03/21/2012 02:36 PM, Jan Kiszka wrote: > This is now implied by kvm_irqchip_in_kernel. > > So we can't have -no-kvm-pit? No huge loss, but unexpected. -- error compiling committee.c: too many arguments to function

Re: [Qemu-devel] Thoughts around dtrace linking...

2012-03-21 Thread Lee Essen
On 21/03/2012 13:01, Andreas Färber wrote: Hi, Am 21.03.2012 11:45, schrieb Lee Essen: I've been trying to find a sensible way to solve the Solaris/Illumos dtrace requirement to pass all the objs to the dtrace command so that the resultant object file contains all the symbols needed to properly

Re: [Qemu-devel] [PATCH uq/master] kvm: Drop unused kvm_pit_in_kernel

2012-03-21 Thread Jan Kiszka
On 2012-03-21 14:36, Avi Kivity wrote: > On 03/21/2012 02:36 PM, Jan Kiszka wrote: >> This is now implied by kvm_irqchip_in_kernel. >> >> > > So we can't have -no-kvm-pit? > > No huge loss, but unexpected. See e81dda195556e72f8cd294998296c1051aab30a8. Jan -- Siemens AG, Corporate Technology,

Re: [Qemu-devel] Can VMX provide real mode support?

2012-03-21 Thread Jan Kiszka
On 2012-03-21 13:38, GaoYi wrote: > Hi Jan, > > Since the newest Intel-VT supports the guest OS under the real mode, > which was already supported in AMD-V, can the VMX in the latest KVM support > that case? Yes, both with our without that "unrestricted guest" support (as Intel called it),

Re: [Qemu-devel] [PATCH uq/master] kvm: Drop unused kvm_pit_in_kernel

2012-03-21 Thread Gleb Natapov
On Wed, Mar 21, 2012 at 02:39:47PM +0100, Jan Kiszka wrote: > On 2012-03-21 14:36, Avi Kivity wrote: > > On 03/21/2012 02:36 PM, Jan Kiszka wrote: > >> This is now implied by kvm_irqchip_in_kernel. > >> > >> > > > > So we can't have -no-kvm-pit? > > > > No huge loss, but unexpected. > > See e81d

Re: [Qemu-devel] Can VMX provide real mode support?

2012-03-21 Thread Avi Kivity
On 03/21/2012 03:40 PM, Jan Kiszka wrote: > On 2012-03-21 13:38, GaoYi wrote: > > Hi Jan, > > > > Since the newest Intel-VT supports the guest OS under the real mode, > > which was already supported in AMD-V, can the VMX in the latest KVM support > > that case? > > Yes, both with our without

Re: [Qemu-devel] [PATCH uq/master] kvm: Drop unused kvm_pit_in_kernel

2012-03-21 Thread Jan Kiszka
On 2012-03-21 14:41, Gleb Natapov wrote: > On Wed, Mar 21, 2012 at 02:39:47PM +0100, Jan Kiszka wrote: >> On 2012-03-21 14:36, Avi Kivity wrote: >>> On 03/21/2012 02:36 PM, Jan Kiszka wrote: This is now implied by kvm_irqchip_in_kernel. >>> >>> So we can't have -no-kvm-pit? >>> >>> N

Re: [Qemu-devel] [PATCH 10/12] trace: [tracetool] Automatically establish available backends and formats

2012-03-21 Thread Lluís Vilanova
Stefan Hajnoczi writes: [...] >> Maybe this would work nice for everybody: >> >> tracetool.py                  # main program (just parse cmdline opts and >> call tracetool module) >> tracetool/__init__.py         # common boilerplate code (e.g., event parsing >> and call dispatching) >> traceto

Re: [Qemu-devel] [PATCH V2 2/3] exynos4210: add exynos4210 GPIO implementation

2012-03-21 Thread Igor Mitsyanko
On 03/20/2012 05:27 PM, Peter Maydell wrote: On 15 March 2012 07:35, Igor Mitsyanko wrote: Now that we have GPIO emulation for exynos4210 SoC we can use it to properly hook up IRQ line to lan9215 controller on SMDK board. +#elif EXYNOS4210_GPIO_DEBUG == 1 +#define DPRINT_L1(fmt, args...)

[Qemu-devel] Can VMX provide real mode support?

2012-03-21 Thread GaoYi
Hi Jan, Since the newest Intel-VT supports the guest OS under the real mode, which was already supported in AMD-V, can the VMX in the latest KVM support that case? Thanks, Yi

Re: [Qemu-devel] [PATCH V2 1/3] exynos4210: add Exynos4210 i2c implementation

2012-03-21 Thread Igor Mitsyanko
On 03/21/2012 03:55 PM, Peter Maydell wrote: On 15 March 2012 07:35, Igor Mitsyanko wrote: Create 9 exynos4210 i2c interfaces. Signed-off-by: Igor Mitsyanko Mostly this looks OK but I still find the i2c slave stuff odd -- should the controller really register itself as a slave on its own bus

Re: [Qemu-devel] [PATCH V2 1/3] exynos4210: add Exynos4210 i2c implementation

2012-03-21 Thread Igor Mitsyanko
On 03/21/2012 05:09 PM, Peter Maydell wrote: On 21 March 2012 13:07, Igor Mitsyanko wrote: On 03/21/2012 03:55 PM, Peter Maydell wrote: I suspect that what's happening here is that the hardware lets you put the i2c controller into slave mode so some other device on the bus can be a master. But

Re: [Qemu-devel] [PATCH V2 1/3] exynos4210: add Exynos4210 i2c implementation

2012-03-21 Thread Peter Maydell
On 21 March 2012 14:18, Igor Mitsyanko wrote: > Do you mean we shouldn't register EXYNOS4_I2C_SLAVE at all so some > hypothetical bus master wouldn't even find EXYNOS4_I2C_SLAVE on a bus? > Maybe the best solution is to make exynos4210_i2c_slave_send() and > exynos4210_i2c_slave_recv() always retu

[Qemu-devel] Can VMX support real mode?

2012-03-21 Thread GaoYi
Hi Avi, According to your reply if the CPU supports the real mode, the KVM can support the real mode guest and there is still work to be done for CPU without real mode support (i.e., software emulation)? Thanks again, Yi

Re: [Qemu-devel] [PATCH 3/3] configure: add --confsuffix option

2012-03-21 Thread Eduardo Habkost
On Wed, Mar 21, 2012 at 01:39:39PM +0100, Paolo Bonzini wrote: > Il 20/03/2012 21:15, Eduardo Habkost ha scritto: > > This way we can choose a configuration file path different from > > ${sysconfdir}/PACKAGE. > > > > This also changes the configure output to show the full configuration > > dir pat

Re: [Qemu-devel] [PATCH] Remove PCI class code from virtio balloon device

2012-03-21 Thread Anthony Liguori
On 03/21/2012 08:08 AM, Michael S. Tsirkin wrote: On Wed, Mar 21, 2012 at 11:26:15AM +, Stefan Hajnoczi wrote: On Tue, Mar 20, 2012 at 09:19:47PM +1100, David Gibson wrote: Looking at hw/pc_piix.c there are QEMUMachine types for each QEMU release. Legacy machine types (e.g. pc_machine_v0_14

Re: [Qemu-devel] Can VMX provide real mode support?

2012-03-21 Thread GaoYi
Hi, Thanks for your prompt response. So if the CPU supports the real mode, the KVM can support the real mode guest and there is still work to be done for CPU without real mode support, namely software emulation? Again 在 2012年3月21日 下午9:48,Avi Kivity 写道: > On 03/21/2012 03:40 PM, Jan Kiszka

Re: [Qemu-devel] [PATCH 1/3] test: remove qemu-ga reference

2012-03-21 Thread Michael Roth
On Wed, Mar 21, 2012 at 11:37:10AM +, Stefan Hajnoczi wrote: > On Tue, Mar 20, 2012 at 12:18 AM, Michael Roth > wrote: > > This was added by mistake a while back. > > > > Signed-off-by: Michael Roth > > --- > >  tests/Makefile |    2 +- > >  1 files changed, 1 insertions(+), 1 deletions(-) >

Re: [Qemu-devel] [PATCH] Remove PCI class code from virtio balloon device

2012-03-21 Thread Michael S. Tsirkin
On Wed, Mar 21, 2012 at 09:42:41AM -0500, Anthony Liguori wrote: > On 03/21/2012 08:08 AM, Michael S. Tsirkin wrote: > >On Wed, Mar 21, 2012 at 11:26:15AM +, Stefan Hajnoczi wrote: > >>On Tue, Mar 20, 2012 at 09:19:47PM +1100, David Gibson wrote: > >>Looking at hw/pc_piix.c there are QEMUMachin

[Qemu-devel] [PATCH v2] test: remove qemu-ga reference

2012-03-21 Thread Michael Roth
This was added by mistake a while back. Signed-off-by: Michael Roth --- Makefile |1 + tests/Makefile |2 +- 2 files changed, 2 insertions(+), 1 deletions(-) diff --git a/Makefile b/Makefile index 1bc3cb0..cab7c74 100644 --- a/Makefile +++ b/Makefile @@ -173,6 +173,7 @@ qemu-img-c

Re: [Qemu-devel] [PATCH 0/2] linux-user: Support prctl PR_GET/SET_NAME

2012-03-21 Thread Riku Voipio
Hi, looks like I'll be busy for the rest of the of the month :/ Riku On Tue, Mar 20, 2012 at 11:47:01AM +, Peter Maydell wrote: > Ping^3 (past the six-week mark now...) > > -- PMM > > On 8 March 2012 14:20, Peter Maydell wrote: > > Ping^2 ? > > > > -- PMM > > > > On 22 February 2012 22:55

Re: [Qemu-devel] [PATCH] Remove PCI class code from virtio balloon device

2012-03-21 Thread Anthony Liguori
On 03/21/2012 10:10 AM, Michael S. Tsirkin wrote: On Wed, Mar 21, 2012 at 09:42:41AM -0500, Anthony Liguori wrote: On 03/21/2012 08:08 AM, Michael S. Tsirkin wrote: On Wed, Mar 21, 2012 at 11:26:15AM +, Stefan Hajnoczi wrote: On Tue, Mar 20, 2012 at 09:19:47PM +1100, David Gibson wrote: Lo

[Qemu-devel] [PATCH V2 1/7] block: Add new BDRV_O_INCOMING flag to notice incoming live migration

2012-03-21 Thread Benoît Canet
>From original patch with Patchwork-id: 31110 by Stefan Hajnoczi "Add a flag to indicate that incoming migration is pending and care needs to be taken for data consistency. Block drivers should not modify the image file before incoming migration is complete since the migration source host is sti

[Qemu-devel] [PATCH V2 2/7] block: add a function to clear incoming live migration flags

2012-03-21 Thread Benoît Canet
This function will clear all BDRV_O_INCOMING flags. Signed-off-by: Benoit Canet --- block.c |9 + block.h |2 ++ 2 files changed, 11 insertions(+), 0 deletions(-) diff --git a/block.c b/block.c index b88ee90..45085e7 100644 --- a/block.c +++ b/block.c @@ -3584,6 +3584,15 @@ void

[Qemu-devel] [PATCH V2 3/7] blockdev: open images with BDRV_O_INCOMING on incoming live migration

2012-03-21 Thread Benoît Canet
Open images with BDRV_O_INCOMING in order to inform block drivers that an incoming live migration is coming. Signed-off-by: Benoit Canet --- blockdev.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/blockdev.c b/blockdev.c index 1a500b8..9b57133 100644 --- a/blockdev

[Qemu-devel] [PATCH V2 4/7] qed: add bdrv_invalidate_cache to be called after incoming live migration

2012-03-21 Thread Benoît Canet
The QED image is reopened to flush metadata and check consistency. Signed-off-by: Benoit Canet --- block/qed.c | 15 +++ block/qed.h |1 + 2 files changed, 16 insertions(+), 0 deletions(-) diff --git a/block/qed.c b/block/qed.c index a041d31..c47272c 100644 --- a/block/qed.c +

[Qemu-devel] [PATCH V2 7/7] qed: remove incoming live migration blocker

2012-03-21 Thread Benoît Canet
Signed-off-by: Benoit Canet --- block/qed.c |9 - block/qed.h |2 -- 2 files changed, 0 insertions(+), 11 deletions(-) diff --git a/block/qed.c b/block/qed.c index 4c04bc9..803a6c2 100644 --- a/block/qed.c +++ b/block/qed.c @@ -502,12 +502,6 @@ static int bdrv_qed_open(BlockDrive

[Qemu-devel] [PATCH v3 2/2] Change timedrift default value to slew

2012-03-21 Thread Crístian Viana
Windows 2008+ is very sensitive to missed ticks. The RTC is used by default as the time source. If time drift is not enabled, Windows is prone to blue screening. Signed-off-by: Crístian Viana --- hw/mc146818rtc.c |2 +- vl.c | 10 +- 2 files changed, 6 insertions(+), 6

[Qemu-devel] [PATCH v3 1/2] Force timedrift=none on previous machines

2012-03-21 Thread Crístian Viana
The current value for the -rtc timedrift option is none. This patch makes sure that the old machines configuration will work the same way even after that option changes its default value. Signed-off-by: Crístian Viana --- hw/pc_piix.c | 39 +++ 1 files chang

[Qemu-devel] [PATCH V2 6/7] qed: honor BDRV_O_INCOMING for incoming live migration

2012-03-21 Thread Benoît Canet
>From original commit with Patchwork-id: 31108 by Stefan Hajnoczi "The QED image format includes a file header bit to mark images dirty. QED normally checks dirty images on open and fixes inconsistent metadata. This is undesirable during live migration since the dirty bit may be set if the sourc

[Qemu-devel] [PATCH V2 0/7] Make QED with live migration safe

2012-03-21 Thread Benoît Canet
This is the second version of a patchset aiming at making the combined usage of QED and live migration safe. Since v1: -The block layer is not aware anymore of the migration state. (stefanha) -No bdrv_invalidate_cache renaming since the semantic do not change. (stefanha) -The qed bdrv_invalidate_

Re: [Qemu-devel] [PATCH] Remove PCI class code from virtio balloon device

2012-03-21 Thread Michael S. Tsirkin
On Wed, Mar 21, 2012 at 10:14:35AM -0500, Anthony Liguori wrote: > On 03/21/2012 10:10 AM, Michael S. Tsirkin wrote: > >On Wed, Mar 21, 2012 at 09:42:41AM -0500, Anthony Liguori wrote: > >>On 03/21/2012 08:08 AM, Michael S. Tsirkin wrote: > >>>On Wed, Mar 21, 2012 at 11:26:15AM +, Stefan Hajnoc

[Qemu-devel] [PATCH] ui/spice-display: use uintptr_t when casting qxl physical addresses

2012-03-21 Thread Alon Levy
The current intptr_t casts are a problem when the address's highest bit is 1, and it is cast to a intptr_t and then to uint64_t, such as at: surface.mem= (intptr_t)ssd->buf; This causes the sign bit to be extended which causes a wrong address to be passed on to spice, which then compl

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-21 Thread Gleb Natapov
On Wed, Mar 21, 2012 at 11:18:16AM -0500, Corey Minyard wrote: > > >Look at drivers/char/ipmi/ipmi_msghandler.c. It has code to send panic > >event over IMPI. The code is pretty complex. Of course if we a going to > >implement something more complex than simple hypercall for panic > >notification

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-21 Thread Avi Kivity
On 03/21/2012 06:18 PM, Corey Minyard wrote: > >> Look at drivers/char/ipmi/ipmi_msghandler.c. It has code to send panic >> event over IMPI. The code is pretty complex. Of course if we a going to >> implement something more complex than simple hypercall for panic >> notification we better do someth

Re: [Qemu-devel] [PATCH] Remove PCI class code from virtio balloon device

2012-03-21 Thread Anthony Liguori
On 03/21/2012 11:11 AM, Michael S. Tsirkin wrote: On Wed, Mar 21, 2012 at 10:14:35AM -0500, Anthony Liguori wrote: On 03/21/2012 10:10 AM, Michael S. Tsirkin wrote: On Wed, Mar 21, 2012 at 09:42:41AM -0500, Anthony Liguori wrote: On 03/21/2012 08:08 AM, Michael S. Tsirkin wrote: On Wed, Mar 2

Re: [Qemu-devel] [PATCH 24/36] vmstate: port arm cpu

2012-03-21 Thread Andreas Färber
Am 19.03.2012 23:57, schrieb Juan Quintela: > Use one subsection for each feature. This means that we don't need to > bump the version field each time that a new feature gets introduced. > > Introduce cpsr_vmstate field, as I am not sure if I can "use" > uncached_cpsr for saving state. > > Signe

Re: [Qemu-devel] [PATCH] Remove PCI class code from virtio balloon device

2012-03-21 Thread Anthony Liguori
On 03/21/2012 11:26 AM, Anthony Liguori wrote: On 03/21/2012 11:11 AM, Michael S. Tsirkin wrote: Frankly I don't see value in making it visible to the user, at all. We are just triggering windows reactivations without any user benefit. Why not return a fixed value there to avoid that? I don't

Re: [Qemu-devel] [PATCH v4 4/7] RTC: Set internal millisecond register to 500ms when reset divider

2012-03-21 Thread Stefano Stabellini
On Wed, 21 Mar 2012, Zhang, Yang Z wrote: > > > struct tm *tm = &s->current_tm; > > > -int64_t host_usec, guest_sec, guest_usec; > > > +int64_t host_usec, guest_sec, guest_usec, offset_usec, > > > old_guest_usec; > > > > > > host_usec = qemu_get_clock_ns(host_clock) / NS_PER_USEC

[Qemu-devel] [PATCH V2 5/7] migration: clear BDRV_O_INCOMING flags on end of incoming live migration

2012-03-21 Thread Benoît Canet
Signed-off-by: Benoît Canet --- migration.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/migration.c b/migration.c index 8c119ba..94f7839 100644 --- a/migration.c +++ b/migration.c @@ -91,6 +91,7 @@ void process_incoming_migration(QEMUFile *f) qemu_announce_self()

Re: [Qemu-devel] [PATCH 24/36] vmstate: port arm cpu

2012-03-21 Thread Peter Maydell
On 21 March 2012 16:29, Andreas Färber wrote: > Am 19.03.2012 23:57, schrieb Juan Quintela: >> Use one subsection for each feature.  This means that we don't need to >> bump the version field each time that a new feature gets introduced. >> >> Introduce cpsr_vmstate field, as I am not sure if I ca

Re: [Qemu-devel] [PATCH v4 5/7] RTC:Add RTC update-ended interrupt support

2012-03-21 Thread Stefano Stabellini
On Wed, 21 Mar 2012, Paolo Bonzini wrote: > Il 20/03/2012 19:35, Stefano Stabellini ha scritto: > > This is the function that is used to figure out whether we need the > > timers or not, the condition seems to be: > > > > (Not (REG_C_UF | REG_C_AF)) And (Not (REG_B_SET)) > > > > Shouldn't actuall

Re: [Qemu-devel] [PATCH] ui/spice-display: use uintptr_t when casting qxl physical addresses

2012-03-21 Thread Gerd Hoffmann
On 03/21/12 17:17, Alon Levy wrote: > This fixes running "-vga qxl -spice" with 32 bit compiled > qemu-system-i386. Patch added to spice patch queue. thanks, Gerd

[Qemu-devel] tracetool: cast const types to avoid compile time warnings

2012-03-21 Thread Lee Essen
On Solaris/Illumos dtrace will remove the "const" qualifier from any arguments when producing the header file, this results in hundreds of compile-time warnings. I have put together a patch to tracetool to cast any const argument appropriately to remove the warnings, but I don't know how it beha

Re: [Qemu-devel] [PATCH v4 00/36] VMState port of all cpus

2012-03-21 Thread Andreas Färber
Am 19.03.2012 23:57, schrieb Juan Quintela: > This repository contains all the changes: > > git://repo.or.cz/qemu/quintela.git vmstate-cpus-v4 > > [v4] > - rebase to top > - adapt to vmstate.h change > - adapt to CPUState -> CPU$archState rename > - integrate arm changes in the meantime > - add

Re: [Qemu-devel] [PATCH v2 2/2] Change timedrift default value to slew

2012-03-21 Thread Crístian Viana
On 21-03-2012 03:02, Paolo Bonzini wrote: > This piece of code from the previous if: > > if (!strcmp(value, "slew")) { > static GlobalProperty slew_lost_ticks[] = { > { > .driver = "mc146818rtc", > .property = "lost_tick

Re: [Qemu-devel] [PATCH v4] Man page: Add -global description

2012-03-21 Thread Markus Armbruster
Miroslav Rezanina writes: > There's only TODO information in qemu man page for -global option. This is a > basic description of this option with simple example. > > Signed-off-by: Miroslav Rezanina > > v4: > - break long line > > v3: > - add use case description > - use prop instead of prop

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-21 Thread Daniel P. Berrange
On Wed, Mar 21, 2012 at 06:25:16PM +0200, Avi Kivity wrote: > On 03/21/2012 06:18 PM, Corey Minyard wrote: > > > >> Look at drivers/char/ipmi/ipmi_msghandler.c. It has code to send panic > >> event over IMPI. The code is pretty complex. Of course if we a going to > >> implement something more compl

[Qemu-devel] [PATCH] uhci: stop queue filling when we find a in-flight td

2012-03-21 Thread Gerd Hoffmann
Not only QHs can form rings, but TDs too. With the new queuing/pipelining support we are following TD chains and can actually walk in circles. An assert() prevents us from entering an endless loop then. Fix is easy: Just stop queuing when we figure the TD we are about to queue up is in flight a

Re: [Qemu-devel] [PATCH v4 00/36] VMState port of all cpus

2012-03-21 Thread Juan Quintela
Andreas Färber wrote: > Am 19.03.2012 23:57, schrieb Juan Quintela: >> This repository contains all the changes: >> >> git://repo.or.cz/qemu/quintela.git vmstate-cpus-v4 >> >> [v4] >> - rebase to top >> - adapt to vmstate.h change >> - adapt to CPUState -> CPU$archState rename >> - integrate a

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-21 Thread Avi Kivity
On 03/21/2012 07:04 PM, Daniel P. Berrange wrote: > > > > In fact the feature can be implemented 100% host side by searching for a > > panic string signature in the console logs. > > You can even go one better and search for the panic string in the > guest memory directly, which is what virt-dmesg

Re: [Qemu-devel] [PATCH 4/6] fdc: Parametrize ISA base, IRQ and DMA

2012-03-21 Thread Markus Armbruster
Hervé Poussineau writes: > Keep the PC values as defaults but allow to override them for PReP. > > Signed-off-by: Hervé Poussineau > Cc: Markus Armbruster > Signed-off-by: Andreas Färber Reviewed-by: Markus Armbruster

Re: [Qemu-devel] [PATCH 08/36] vmstate: machine.c is only compiled for !CONFIG_USER_ONLY

2012-03-21 Thread Andreas Färber
Confirmed, in Makefile.target machine.o is confined to ifdef CONFIG_SOFTMMU. Am 19.03.2012 23:57, schrieb Juan Quintela: > Signed-off-by: Juan Quintela I verified that the right #endifs were removed and compile-tested it. Acked-by: Andreas Färber Andreas -- SUSE LINUX Products GmbH, Maxfeld

Re: [Qemu-devel] [PATCH] Remove PCI class code from virtio balloon device

2012-03-21 Thread Michael S. Tsirkin
On Wed, Mar 21, 2012 at 11:26:50AM -0500, Anthony Liguori wrote: > On 03/21/2012 11:11 AM, Michael S. Tsirkin wrote: > >On Wed, Mar 21, 2012 at 10:14:35AM -0500, Anthony Liguori wrote: > >>On 03/21/2012 10:10 AM, Michael S. Tsirkin wrote: > >>>On Wed, Mar 21, 2012 at 09:42:41AM -0500, Anthony Liguo

Re: [Qemu-devel] [PATCH] ui/spice-display: use uintptr_t when casting qxl physical addresses

2012-03-21 Thread Alon Levy
On Wed, Mar 21, 2012 at 05:42:25PM +0100, Gerd Hoffmann wrote: > On 03/21/12 17:17, Alon Levy wrote: > > This fixes running "-vga qxl -spice" with 32 bit compiled > > qemu-system-i386. > > Patch added to spice patch queue. So perhaps you can also ack those: http://patchwork.freedesktop.org/patch

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-21 Thread Jan Kiszka
On 2012-03-21 18:34, Avi Kivity wrote: > On 03/21/2012 07:04 PM, Daniel P. Berrange wrote: >>> >>> In fact the feature can be implemented 100% host side by searching for a >>> panic string signature in the console logs. >> >> You can even go one better and search for the panic string in the >> gues

Re: [Qemu-devel] [PATCH] Remove PCI class code from virtio balloon device

2012-03-21 Thread Michael S. Tsirkin
On Wed, Mar 21, 2012 at 11:33:21AM -0500, Anthony Liguori wrote: > On 03/21/2012 11:26 AM, Anthony Liguori wrote: > >On 03/21/2012 11:11 AM, Michael S. Tsirkin wrote: > >>Frankly I don't see value in making it visible to the user, > >>at all. We are just triggering windows reactivations > >>without

[Qemu-devel] [PATCH V9 7/8] Introduce apic-msidef.h

2012-03-21 Thread Anthony PERARD
This patch move the msi definition from apic.c to apic-msidef.h. So it can be used also by other .c files. Signed-off-by: Anthony PERARD Acked-by: Stefano Stabellini --- hw/apic-msidef.h | 30 ++ hw/apic.c| 11 +-- 2 files changed, 31 insertions(+

[Qemu-devel] [PATCH V9 3/8] Introduce HostPCIDevice to access a pci device on the host.

2012-03-21 Thread Anthony PERARD
Signed-off-by: Anthony PERARD Acked-by: Stefano Stabellini --- Makefile.target |3 + hw/host-pci-device.c | 278 ++ hw/host-pci-device.h | 75 ++ 3 files changed, 356 insertions(+), 0 deletions(-) create mode 100644 hw/host

[Qemu-devel] [PATCH V9 2/8] configure: Introduce --enable-xen-pci-passthrough.

2012-03-21 Thread Anthony PERARD
Signed-off-by: Anthony PERARD Acked-by: Stefano Stabellini --- configure | 25 + 1 files changed, 25 insertions(+), 0 deletions(-) diff --git a/configure b/configure index 8b4e3c1..f61bd48 100755 --- a/configure +++ b/configure @@ -136,6 +136,7 @@ vnc_png="" vnc_threa

  1   2   >