Re: [Qemu-devel] (PAM stuff) reset doesn't work on OVMF + SeaBIOS CSM

2013-02-14 Thread Michael Tokarev
15.02.2013 07:43, Kevin O'Connor wrote: > On Fri, Feb 15, 2013 at 04:10:59AM +0100, Laszlo Ersek wrote: >> On 02/15/13 02:22, Kevin O'Connor wrote: >>> On Thu, Feb 14, 2013 at 08:16:02PM -0500, Kevin O'Connor wrote: >>> By chance, are you using an older version of kvm? There was a bug in >>> kvm t

[Qemu-devel] [PATCH v1 3/5] cadence_gem: fix interrupt events

2013-02-14 Thread Peter Crosthwaite
Bits in the ISR were continually mirroring their corresponding TX/RX SR bits. This is incorrect. The ISR bits are only every set at the time their corresponding event occurs. Signed-off-by: Peter Crosthwaite --- hw/cadence_gem.c | 27 --- 1 files changed, 4 insertions(

[Qemu-devel] [PATCH v1 4/5] cadence_gem: Dont reset rx desc pointer on rx_en

2013-02-14 Thread Peter Crosthwaite
This doesnt happen in the real hardware. The Zynq TRM explicitly states that this bit has no effect on the rx descriptor pointer ("The receive queue pointer register is unaffected"). Signed-off-by: Peter Crosthwaite --- hw/cadence_gem.c |4 1 files changed, 0 insertions(+), 4 deletions

[Qemu-devel] [PATCH v1 5/5] cadence_gem: Add debug msgs for rx desc movement

2013-02-14 Thread Peter Crosthwaite
Add some helpful messages that show the rx descriptor pointer moving as packets are rxed. Signed-off-by: Peter Crosthwaite --- hw/cadence_gem.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/hw/cadence_gem.c b/hw/cadence_gem.c index 61f1801..de7d15a 100644 --- a/hw/ca

[Qemu-devel] [PATCH v1 2/5] cadence_gem: factor out can_rx() logic replication

2013-02-14 Thread Peter Crosthwaite
The gem_receive() function replicates the logic for whether or not the device can rx. Just call the actual gem_can_receive() function in place. Signed-off-by: Peter Crosthwaite --- hw/cadence_gem.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/cadence_gem.c b/hw/c

[Qemu-devel] [PATCH v1 1/5] cadence_gem: Flush queued packets

2013-02-14 Thread Peter Crosthwaite
The device needs to check for queued RX packets when the RX path is re-enabled. Signed-off-by: Peter Crosthwaite --- hw/cadence_gem.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/hw/cadence_gem.c b/hw/cadence_gem.c index ab86c17..e6032ea 100644 --- a/hw/cadence_gem

[Qemu-devel] [PATCH v1 0/5] Zynq GEM updates

2013-02-14 Thread Peter Crosthwaite
Misc fixes and cleanups to the Zynq GEM device model. Patch 1 fixes a flow control issue talking to the network. Patches 2 and 5 are trivial. Patches 3 & 4 correct inaccuracies in the device model. Peter Crosthwaite (5): cadence_gem: Flush queued packets cadence_gem: factor out can_rx() logi

Re: [Qemu-devel] [RFC PATCH 02/10] Fix errors and warnings while compiling with c++ compilier

2013-02-14 Thread Tomoki Sekiyama
On 2013/02/14 21:27, Luiz Capitulino wrote: > On Thu, 14 Feb 2013 15:10:36 +0900 > Tomoki Sekiyama wrote: > >> Rename 'class' member in class_info of PciDeviceInfo to 'dev_class', and >> add some casts to avoid errors from c++ compiler. > > [...] > >> # >> # @class_info.desc: #optional a stri

Re: [Qemu-devel] [RFC PATCH 03/10] qemu-ga: Add an configure option to specify path to Windows VSS SDK

2013-02-14 Thread Tomoki Sekiyama
Hi Paolo, On 2013/02/14 23:36, Paolo Bonzini wrote: > Il 14/02/2013 07:10, Tomoki Sekiyama ha scritto: >> To enable VSS support in qemu-ga for Windows, header files included in >> VSS SDK is required. >> The VSS support is enabled when the option like below: >> ./configure --with-vss-sdk="/pass/

Re: [Qemu-devel] [RFC PATCH 03/10] qemu-ga: Add an configure option to specify path to Windows VSS SDK

2013-02-14 Thread Tomoki Sekiyama
On 2013/02/15 9:47, mdroth wrote: [...] > > Hi Tomoki, Hi, > Did you happen to run into this issue compiling the test program? > > [mdroth@vm qemu-build]$ ls ~/w/vsssdk/inc/win2003/ > vdslun.hvsbackup.hvscoordint.idl vsmgmt.idl vsprov.idl > vss.idlvsswprv.idl > vdslun.idl vscoordin

Re: [Qemu-devel] [RFC PATCH 04/10] qemu-ga: Add Windows VSS provider to quiesce applications on fsfreeze

2013-02-14 Thread Tomoki Sekiyama
Hi Paolo, thanks for your review. On 2013/02/14 22:22, Paolo Bonzini wrote:> Il 14/02/2013 07:10, Tomoki Sekiyama ha scritto: >> diff --git a/qga/vss-win32-provider/Makefile b/qga/vss-win32-provider/Makefile >> new file mode 100644 >> index 000..1f213f2 >> --- /dev/null >> +++ b/qga/vss-win32-

Re: [Qemu-devel] (PAM stuff) reset doesn't work on OVMF + SeaBIOS CSM

2013-02-14 Thread Kevin O'Connor
On Fri, Feb 15, 2013 at 04:10:59AM +0100, Laszlo Ersek wrote: > On 02/15/13 02:22, Kevin O'Connor wrote: > > On Thu, Feb 14, 2013 at 08:16:02PM -0500, Kevin O'Connor wrote: > > By chance, are you using an older version of kvm? There was a bug in > > kvm that caused changes to memory mapped at 0xe0

Re: [Qemu-devel] (PAM stuff) reset doesn't work on OVMF + SeaBIOS CSM

2013-02-14 Thread Laszlo Ersek
On 02/15/13 02:22, Kevin O'Connor wrote: > On Thu, Feb 14, 2013 at 08:16:02PM -0500, Kevin O'Connor wrote: >> On Fri, Feb 15, 2013 at 12:01:38AM +0100, Laszlo Ersek wrote: >>> On 02/14/13 21:55, David Woodhouse wrote: >>> Thanks for testing this, btw. Are you looking at suspend/resume too? :)

Re: [Qemu-devel] [RFC PATCH 03/10] qemu-ga: Add an configure option to specify path to Windows VSS SDK

2013-02-14 Thread mdroth
On Thu, Feb 14, 2013 at 03:10:39PM +0900, Tomoki Sekiyama wrote: > To enable VSS support in qemu-ga for Windows, header files included in > VSS SDK is required. > The VSS support is enabled when the option like below: > ./configure --with-vss-sdk="/pass/to/VSS SDK" > > VSS SDK is available from:

Re: [Qemu-devel] [PATCH] Fix conversion between 12 hours and 24 hours modes.

2013-02-14 Thread Antoine Mathys
First, the ds1338 code was in a poor state and never handled the 12 hour clock correctly. My first patch failed to fully fix the problem so I had to write a second one, but at no point did Peter or I introduce a regression, quite the opposite. Second, I don't know where you got the idea that I

Re: [Qemu-devel] [edk2] [SeaBIOS] (PAM stuff) reset doesn't work on OVMF + SeaBIOS CSM

2013-02-14 Thread Andrew Fish
On Feb 14, 2013, at 2:09 PM, "H. Peter Anvin" wrote: > On 02/14/2013 01:27 PM, David Woodhouse wrote: >> >> So it *is* jumping to 0xfff0 but the memory at that location isn't >> what we expect? Do the PAM registers affect *that* too, or only the >> region from 0xc-0xf? Surely the co

Re: [Qemu-devel] [seabios][PATCH 1/2] move all acpi-table related definitions to acpi.h

2013-02-14 Thread li guang
在 2013-02-06三的 23:15 -0500,Kevin O'Connor写道: > On Mon, Feb 04, 2013 at 10:27:59AM +0800, liguang wrote: > > Signed-off-by: liguang > > Thanks. Some comments. > > [...] > > --- a/src/acpi.h > > +++ b/src/acpi.h > [...] > > +#include "acpi-dsdt.hex" > > Moving the acpi structure defines to the h

Re: [Qemu-devel] (PAM stuff) reset doesn't work on OVMF + SeaBIOS CSM

2013-02-14 Thread Kevin O'Connor
On Thu, Feb 14, 2013 at 08:16:02PM -0500, Kevin O'Connor wrote: > On Fri, Feb 15, 2013 at 12:01:38AM +0100, Laszlo Ersek wrote: > > On 02/14/13 21:55, David Woodhouse wrote: > > > > > Thanks for testing this, btw. Are you looking at suspend/resume too? :) > > > > Entering S3 seemed OK (except the

Re: [Qemu-devel] [seabios][PATCH 2/2] acpi: change numa data format from fw_cfg interface

2013-02-14 Thread li guang
Sorry for late reply 在 2013-02-06三的 23:18 -0500,Kevin O'Connor写道: > On Mon, Feb 04, 2013 at 10:28:00AM +0800, liguang wrote: > > the old numa format got form fw_cfg is: > > number of nodes > > node id of cpu (array) > > node memory size (array) > > > > now, format it like array of: > > apci_map,

Re: [Qemu-devel] [PATCH for-1.4 v2 4/6] qemu-option: Disable two helpful messages that got broken recently

2013-02-14 Thread Laszlo Ersek
On 02/15/13 01:20, Laszlo Ersek wrote: > On 02/14/13 17:36, Luiz Capitulino wrote: >> On Thu, 14 Feb 2013 14:31:50 +0100 >> Markus Armbruster wrote: >>> chardev-add: the schema defines an object type for each backend >>> (ChardevFile, ChardevSocket, ...), and collects them together in >>> discrim

Re: [Qemu-devel] (PAM stuff) reset doesn't work on OVMF + SeaBIOS CSM

2013-02-14 Thread Kevin O'Connor
On Fri, Feb 15, 2013 at 12:01:38AM +0100, Laszlo Ersek wrote: > On 02/14/13 21:55, David Woodhouse wrote: > > > Thanks for testing this, btw. Are you looking at suspend/resume too? :) > > Entering S3 seemed OK (except the screen was not cleared; using Cirrus). > I woke up the guest with > > # vi

Re: [Qemu-devel] [PATCH moxie 3/5] Moxie target code

2013-02-14 Thread Richard Henderson
On 02/14/2013 03:19 PM, Richard Henderson wrote: >> > +tcg_gen_brcond_i32(TCG_COND_EQ, REG(a), REG(b), label_equal); >> > + >> > +#define CMPTEST(t,CODE) \ >> > +{ \ >> > +

Re: [Qemu-devel] [PATCH for-1.4 07/19] target-sparc: Fix debug output for DEBUG_MMU

2013-02-14 Thread Andreas Färber
Am 14.02.2013 22:42, schrieb Blue Swirl: > On Thu, Feb 14, 2013 at 3:46 PM, Andreas Färber wrote: >> Am 27.01.2013 14:32, schrieb Andreas Färber: >>> Signed-off-by: Andreas Färber >>> --- >>> target-sparc/ldst_helper.c |2 +- >>> 1 Datei geändert, 1 Zeile hinzugefügt(+), 1 Zeile entfernt(-)

Re: [Qemu-devel] [PATCH for-1.4 v2 4/6] qemu-option: Disable two helpful messages that got broken recently

2013-02-14 Thread Laszlo Ersek
Hi, sorry for the late answer. I can only address the netdev_add / opts-visitor stuff now. On 02/14/13 17:36, Luiz Capitulino wrote: > On Thu, 14 Feb 2013 14:31:50 +0100 > Markus Armbruster wrote: >> Luiz Capitulino writes: >>> On Thu, 14 Feb 2013 10:45:22 +0100 >>> Markus Armbruster wrote: >

Re: [Qemu-devel] [Qemu-ppc] [PATCH] pseries: Implements h_read hcall

2013-02-14 Thread Alexander Graf
On 13.02.2013, at 06:21, David Gibson wrote: > On Tue, Feb 12, 2013 at 11:07:10PM +0100, Alexander Graf wrote: >> >> On 07.02.2013, at 12:28, Erlon Cruz wrote: >> >>> From: Erlon Cruz >>> >>> This h_call is useful for DLPAR in future amongst other things. Given an >>> index >>> it fetches th

Re: [Qemu-devel] [Qemu-ppc] [PATCH] pseries: Implements h_read hcall

2013-02-14 Thread David Gibson
On Thu, Feb 14, 2013 at 10:44:34AM -0200, Erlon Cruz wrote: > On Sun, Feb 10, 2013 at 10:10 PM, David Gibson > wrote: > > > > On Thu, Feb 07, 2013 at 09:28:20AM -0200, Erlon Cruz wrote: > > > From: Erlon Cruz > > > > > > This h_call is useful for DLPAR in future amongst other things. Given an >

Re: [Qemu-devel] [edk2] (PAM stuff) reset doesn't work on OVMF + SeaBIOS CSM

2013-02-14 Thread David Woodhouse
On Fri, 2013-02-15 at 00:01 +0100, Laszlo Ersek wrote: > > Entering S3 seemed OK (except the screen was not cleared; using > Cirrus). > I woke up the guest with > > # virsh qemu-monitor-command fw-ovmf.g-f18xfce2012121716.e-rhel63 \ > --hmp --cmd system_wakeup > > Trailing portion of the l

Re: [Qemu-devel] [PATCH moxie 3/5] Moxie target code

2013-02-14 Thread Richard Henderson
On 02/13/2013 02:26 PM, Anthony Green wrote: > +typedef struct CPUMoxieState { > + > + > + uint32_t flags; /* general execution flags */ > + uint32_t gregs[16]; /* general registers */ > + uint32_t sregs[256]; /* special registers */ > + uint32_t pc; > + uint32

Re: [Qemu-devel] (PAM stuff) reset doesn't work on OVMF + SeaBIOS CSM

2013-02-14 Thread Laszlo Ersek
On 02/14/13 21:55, David Woodhouse wrote: > Thanks for testing this, btw. Are you looking at suspend/resume too? :) Entering S3 seemed OK (except the screen was not cleared; using Cirrus). I woke up the guest with # virsh qemu-monitor-command fw-ovmf.g-f18xfce2012121716.e-rhel63 \ --hmp --

Re: [Qemu-devel] [PATCH v2] Move File operations to qemu-file.c

2013-02-14 Thread Anthony Liguori
Blue Swirl writes: > On Thu, Feb 14, 2013 at 2:34 AM, Anthony Liguori > wrote: >> Joel Schopp writes: >> > +if(popen_file == NULL) { Please make a preparatory patch which adds missing spaces between 'if' statements and '('. >>> >>> I'll do a preparatory style cleanup pat

Re: [Qemu-devel] [SeaBIOS] [edk2] (PAM stuff) reset doesn't work on OVMF + SeaBIOS CSM

2013-02-14 Thread H. Peter Anvin
On 02/14/2013 01:27 PM, David Woodhouse wrote: So it *is* jumping to 0xfff0 but the memory at that location isn't what we expect? Do the PAM registers affect *that* too, or only the region from 0xc-0xf? Surely the contents at 4GiB-δ should be unchanged by *anything* we do with the PA

Re: [Qemu-devel] [SeaBIOS] [edk2] (PAM stuff) reset doesn't work on OVMF + SeaBIOS CSM

2013-02-14 Thread Laszlo Ersek
On 02/14/13 22:27, David Woodhouse wrote: > On Thu, 2013-02-14 at 22:14 +0100, Laszlo Ersek wrote: >> On 02/14/13 21:54, H. Peter Anvin wrote: >>> On 02/14/2013 12:41 PM, Laszlo Ersek wrote: ). cpu_reset() [target-i386/helper.c] sets CS:IP to f000:fff0, which is the exact address of.

Re: [Qemu-devel] [PATCH moxie 4/5] Add sample moxie system

2013-02-14 Thread Anthony Green
Thanks to you and everybody else for the detailed feedback. I really appreciate the time you put into this. I've cleaned up the formatting and licensing, and will address the other comments when I repost my patches in the new few days. Thanks, AG On Thu, Feb 14, 2013 at 7:08 AM, Andreas Färber

Re: [Qemu-devel] (PAM stuff) reset doesn't work on OVMF + SeaBIOS CSM

2013-02-14 Thread David Woodhouse
On Thu, 2013-02-14 at 21:41 +0100, Laszlo Ersek wrote: > I noticed that under OVMF + SeaBIOS CSM + your related patches for both, > reset requested by the guest doesn't work as expected. The behavior is > an infinite loop, with the following debug fragment repeated by the > CSM-ized SeaBIOS: > >

Re: [Qemu-devel] [RFC] Disassembler options going forward

2013-02-14 Thread Anthony Liguori
Peter Maydell writes: > On 14 February 2013 21:10, Richard Henderson wrote: >> The only options I can see going forward are >> >> 1) Provide a configure time option to link to the "system" libopcodes. >> 2) Use someone else's (bsd licensed?) disassember. >> 3) Rearrange relevant translators s

Re: [Qemu-devel] (PAM stuff) reset doesn't work on OVMF + SeaBIOS CSM

2013-02-14 Thread Laszlo Ersek
On 02/14/13 21:55, David Woodhouse wrote: > So, if real hardware would reset the PAMs on reset and avoid the need > for SeaBIOS to do so, I think we should be doing the same in qemu too. That's what I couldn't figure out from the i440FX spec, but I believe one could argue that "reset" should in f

[Qemu-devel] [PATCH V22 0/7] QEMU Trusted Platform Module (TPM) integration

2013-02-14 Thread Stefan Berger
From: root The following series of patches adds TPM (Trusted Platform Module) support to QEMU. An emulator for the TIS (TPM Interface Spec) interface is added that provides the basis for accessing a 'backend' implementing the actual TPM functionality. The TIS emulator serves as a 'frontend' enabl

Re: [Qemu-devel] [RFC] Disassembler options going forward

2013-02-14 Thread Blue Swirl
On Thu, Feb 14, 2013 at 9:10 PM, Richard Henderson wrote: > On 02/13/2013 06:28 PM, Anthony Liguori wrote: >> QEMU is GPLv2 only so we can't take GPLv3 code. We're stuck on binutils >> code that predates the v3 relicense. > > Ok, this is something that's going to bite us more and more. > > We nee

[Qemu-devel] [PATCH V22 1/7] Support for TPM command line options

2013-02-14 Thread Stefan Berger
This patch adds support for TPM command line options. The command line options supported here are ./qemu-... -tpmdev passthrough,path=,id= -device tpm-tis,tpmdev= and ./qemu-... -tpmdev help where the latter works similar to -soundhw ? and shows a list of available TPM backends (for

[Qemu-devel] [PATCH V22 3/7] Add a debug register

2013-02-14 Thread Stefan Berger
This patch uses the possibility to add a vendor-specific register and adds a debug register useful for dumping the TIS's internal state. This register is only active in a debug build (#define DEBUG_TIS). Signed-off-by: Stefan Berger Reviewed-by: Corey Bryant --- tpm/tpm_tis.c | 70 +

[Qemu-devel] [PATCH V22 5/7] Add a TPM Passthrough backend driver implementation

2013-02-14 Thread Stefan Berger
>From Andreas Niederl's original posting with adaptations where necessary: This patch is based of off version 9 of Stefan Berger's patch series "QEMU Trusted Platform Module (TPM) integration" and adds a new backend driver for it. This patch adds a passthrough backend driver for passing command

[Qemu-devel] [PATCH V22 7/7] Introduce --enable-tpm-passthrough configure option

2013-02-14 Thread Stefan Berger
Signed-off-by: Stefan Berger Reviewed-by: Corey Bryant --- configure | 17 + 1 file changed, 17 insertions(+) diff --git a/configure b/configure index b7359aa..b4f0a82 100755 --- a/configure +++ b/configure @@ -227,6 +227,7 @@ seccomp="" glusterfs="" virtio_blk_data_plane=""

[Qemu-devel] [PATCH V22 6/7] Add support for cancelling of a TPM command

2013-02-14 Thread Stefan Berger
This patch adds support for cancelling an executing TPM command. In Linux for example a user can cancel a command through the TPM's sysfs 'cancel' entry using echo "1" > /sysfs/class/misc/tpm0/device/cancel This patch propagates the cancellation of a command inside a VM to the host TPM's sysfs en

Re: [Qemu-devel] [RFC PATCH v2 13/23] qcow2: handle_copied(): Implement non-zero host_offset

2013-02-14 Thread Blue Swirl
On Thu, Feb 14, 2013 at 9:40 AM, Kevin Wolf wrote: > Am 13.02.2013 22:17, schrieb Blue Swirl: >> On Wed, Feb 13, 2013 at 1:22 PM, Kevin Wolf wrote: >>> Look only for clusters that start at a given physical offset. >>> >>> Signed-off-by: Kevin Wolf >>> --- >>> block/qcow2-cluster.c | 26 ++

[Qemu-devel] [PATCH V22 2/7] Add TPM (frontend) hardware interface (TPM TIS) to QEMU

2013-02-14 Thread Stefan Berger
This patch adds the main code of the TPM frontend driver, the TPM TIS interface, to QEMU. The code is largely based on the previous implementation for Xen but has been significantly extended to meet the standard's requirements, such as the support for changing of localities and all the functionalit

[Qemu-devel] [PATCH V22 4/7] Build the TPM frontend code

2013-02-14 Thread Stefan Berger
Build the TPM frontend code that has been added so far. Signed-off-by: Stefan Berger Reviewed-by: Corey Bryant --- configure | 11 +++ tpm/Makefile.objs | 1 + 2 files changed, 12 insertions(+) diff --git a/configure b/configure index 8789324..b7359aa 100755 --- a/configure ++

Re: [Qemu-devel] [PATCH for-1.4 07/19] target-sparc: Fix debug output for DEBUG_MMU

2013-02-14 Thread Blue Swirl
On Thu, Feb 14, 2013 at 3:46 PM, Andreas Färber wrote: > Blue, > > Am 27.01.2013 14:32, schrieb Andreas Färber: >> Signed-off-by: Andreas Färber >> --- >> target-sparc/ldst_helper.c |2 +- >> 1 Datei geändert, 1 Zeile hinzugefügt(+), 1 Zeile entfernt(-) >> >> diff --git a/target-sparc/ldst_h

Re: [Qemu-devel] [RFC] Disassembler options going forward

2013-02-14 Thread Anthony Liguori
Richard Henderson writes: > On 02/13/2013 06:28 PM, Anthony Liguori wrote: >> QEMU is GPLv2 only so we can't take GPLv3 code. We're stuck on binutils >> code that predates the v3 relicense. > > Ok, this is something that's going to bite us more and more. > > We need *some* solution that allows u

Re: [Qemu-devel] [RFC] Disassembler options going forward

2013-02-14 Thread Peter Maydell
On 14 February 2013 21:10, Richard Henderson wrote: > The only options I can see going forward are > > 1) Provide a configure time option to link to the "system" libopcodes. > 2) Use someone else's (bsd licensed?) disassember. > 3) Rearrange relevant translators so that they can disassemble and

Re: [Qemu-devel] [PATCH v2] Move File operations to qemu-file.c

2013-02-14 Thread Blue Swirl
On Thu, Feb 14, 2013 at 2:34 AM, Anthony Liguori wrote: > Joel Schopp writes: > +if(popen_file == NULL) { >>> >>> Please make a preparatory patch which adds missing spaces between 'if' >>> statements and '('. >> >> I'll do a preparatory style cleanup patch of existing code if it is >> de

Re: [Qemu-devel] [SeaBIOS] [edk2] (PAM stuff) reset doesn't work on OVMF + SeaBIOS CSM

2013-02-14 Thread David Woodhouse
On Thu, 2013-02-14 at 22:14 +0100, Laszlo Ersek wrote: > On 02/14/13 21:54, H. Peter Anvin wrote: > > On 02/14/2013 12:41 PM, Laszlo Ersek wrote: > >> > >> ). cpu_reset() [target-i386/helper.c] sets CS:IP to f000:fff0, which is > >> the exact address of... reset_vector() in SeaBIOS. > >> > > > > T

Re: [Qemu-devel] [PATCH] qemu:cpuid: speedup test by 3x times if grub2 is available

2013-02-14 Thread Lucas Meneghel Rodrigues
On 02/14/2013 05:57 PM, Eduardo Habkost wrote: On Thu, Feb 14, 2013 at 08:28:03PM +0100, Igor Mammedov wrote: On Thu, 14 Feb 2013 13:13:18 +0100 Paolo Bonzini wrote: Il 14/02/2013 12:18, Eduardo Habkost ha scritto: qemu boots from disk image 3 times faster than direct kernel load. That's su

[Qemu-devel] [Bug 1079080] Re: ARM instruction "srs" wrong behaviour

2013-02-14 Thread vcesson
** Attachment added: "test case image (similar image)" https://bugs.launchpad.net/qemu/+bug/1079080/+attachment/3528447/+files/serial2 -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1079080 Title:

[Qemu-devel] [Bug 1079080] Re: ARM instruction "srs" wrong behaviour

2013-02-14 Thread vcesson
** Attachment added: "test case dtb" https://bugs.launchpad.net/qemu/+bug/1079080/+attachment/3528448/+files/xilinx_zynq.dtb -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1079080 Title: ARM ins

Re: [Qemu-devel] [edk2] (PAM stuff) reset doesn't work on OVMF + SeaBIOS CSM

2013-02-14 Thread Laszlo Ersek
On 02/14/13 21:54, H. Peter Anvin wrote: > On 02/14/2013 12:41 PM, Laszlo Ersek wrote: >> >> ). cpu_reset() [target-i386/helper.c] sets CS:IP to f000:fff0, which is >> the exact address of... reset_vector() in SeaBIOS. >> > > This would be a bug, but it isn't quite true. > > If you look at x86_cp

[Qemu-devel] [RFC] Disassembler options going forward

2013-02-14 Thread Richard Henderson
On 02/13/2013 06:28 PM, Anthony Liguori wrote: > QEMU is GPLv2 only so we can't take GPLv3 code. We're stuck on binutils > code that predates the v3 relicense. Ok, this is something that's going to bite us more and more. We need *some* solution that allows us to disassemble current cpus. What we

Re: [Qemu-devel] [edk2] (PAM stuff) reset doesn't work on OVMF + SeaBIOS CSM

2013-02-14 Thread David Woodhouse
On Thu, 2013-02-14 at 12:54 -0800, H. Peter Anvin wrote: > This would be a bug, but it isn't quite true. > > If you look at x86_cpu_reset() you will note that it sets the code > segment base to 0x, not 0xf as one could expect from the > above. This is also true of a physical x86. > >

Re: [Qemu-devel] [PATCH v2] Add option to mlock qemu and guest memory

2013-02-14 Thread Satoru Moriya
On 02/14/2013 03:39 PM, Michael Tokarev wrote: > 15.02.2013 00:21, Satoru Moriya wrote: >> We have some plans to migrate legacy enterprise systems which require >> low latency (10 msec order) to kvm virtualized environment. In our >> usecase, the system runs with other untrusted guests and so loc

[Qemu-devel] [ANNOUNCE] QEMU 1.4.0-rc2 is now available

2013-02-14 Thread Anthony Liguori
Hi, On behalf of the QEMU Team, I'd like to announce the availability of the third release candidate for the QEMU 1.4 release. This release is meant for testing purposes and should not be used in a production environment. http://wiki.qemu.org/download/qemu-1.4.0-rc2.tar.bz2 You can help improve

Re: [Qemu-devel] (PAM stuff) reset doesn't work on OVMF + SeaBIOS CSM

2013-02-14 Thread David Woodhouse
On Thu, 2013-02-14 at 21:41 +0100, Laszlo Ersek wrote: > Can we dumb down ^W^W generalize this code? :) Or maybe should qemu > introduce a reset handler for PAMs? In the UEFI+CSM model, I believe the handling of PAM stuff is left *entirely* to the UEFI side and the CSM is supposed to be hardware-a

Re: [Qemu-devel] [edk2] (PAM stuff) reset doesn't work on OVMF + SeaBIOS CSM

2013-02-14 Thread H. Peter Anvin
On 02/14/2013 12:41 PM, Laszlo Ersek wrote: > > ). cpu_reset() [target-i386/helper.c] sets CS:IP to f000:fff0, which is > the exact address of... reset_vector() in SeaBIOS. > This would be a bug, but it isn't quite true. If you look at x86_cpu_reset() you will note that it sets the code segment

Re: [Qemu-devel] [PATCH for-1.4] e1000: unbreak the guest network migration to 1.3

2013-02-14 Thread Michael S. Tsirkin
On Thu, Feb 14, 2013 at 01:50:04PM -0600, Anthony Liguori wrote: > "Michael S. Tsirkin" writes: > > > QEMU 1.3 does not emulate the link auto negotiation, so if migrate to a > > 1.3 machine during link auto negotiation, the guest link will be set to > > down. > > Fix this by just disabling auto

Re: [Qemu-devel] [PATCH v2] Add option to mlock qemu and guest memory

2013-02-14 Thread Michael Tokarev
15.02.2013 00:21, Satoru Moriya wrote: > We have some plans to migrate legacy enterprise systems which require > low latency (10 msec order) to kvm virtualized environment. In our > usecase, the system runs with other untrusted guests and so locking > memory which is used by the system is needed to

[Qemu-devel] (PAM stuff) reset doesn't work on OVMF + SeaBIOS CSM

2013-02-14 Thread Laszlo Ersek
Hi, I noticed that under OVMF + SeaBIOS CSM + your related patches for both, reset requested by the guest doesn't work as expected. The behavior is an infinite loop, with the following debug fragment repeated by the CSM-ized SeaBIOS: In resume (status=0) In 32bit resume Attempting a hard re

[Qemu-devel] [PATCH v2] Add option to mlock qemu and guest memory

2013-02-14 Thread Satoru Moriya
We have some plans to migrate legacy enterprise systems which require low latency (10 msec order) to kvm virtualized environment. In our usecase, the system runs with other untrusted guests and so locking memory which is used by the system is needed to avoid latency impacts from other guests' memor

Re: [Qemu-devel] [RFC PATCH RDMA support v2: 5/6] connection-setup code between client/server

2013-02-14 Thread Michael R. Hines
Orit (and anthony if you're not busy), I forgot to respond to this very important comment: On 02/13/2013 03:46 AM, Orit Wasserman wrote: Are you still using the tcp for transferring device state? If so you can call the tcp functions from the migration rdma code as a first step but I would pref

Re: [Qemu-devel] [PATCH] qemu:cpuid: speedup test by 3x times if grub2 is available

2013-02-14 Thread Eduardo Habkost
On Thu, Feb 14, 2013 at 08:28:03PM +0100, Igor Mammedov wrote: > On Thu, 14 Feb 2013 13:13:18 +0100 > Paolo Bonzini wrote: > > > Il 14/02/2013 12:18, Eduardo Habkost ha scritto: > > >> > qemu boots from disk image 3 times faster than direct kernel load. > > > That's surprising. Do you have any id

Re: [Qemu-devel] [RFC v5] target-i386: Slim conversion to X86CPU subclasses + KVM subclasses

2013-02-14 Thread Eduardo Habkost
On Thu, Feb 14, 2013 at 08:16:32PM +0100, Igor Mammedov wrote: > > [...] > > > > > + > > > > > +} > > > > > + > > > > > +static void x86_cpu_def_class_init(ObjectClass *oc, void *data) > > > > > +{ > > > > > +X86CPUClass *xcc = X86_CPU_CLASS(oc); > > > > > +x86_def_t *def = data; > > > > >

Re: [Qemu-devel] [PATCH for-1.4] e1000: unbreak the guest network migration to 1.3

2013-02-14 Thread Anthony Liguori
"Michael S. Tsirkin" writes: > QEMU 1.3 does not emulate the link auto negotiation, so if migrate to a > 1.3 machine during link auto negotiation, the guest link will be set to down. > Fix this by just disabling auto negotiation for 1.3. > > Signed-off-by: Michael S. Tsirkin > --- > hw/e1000.c

Re: [Qemu-devel] test images for musicpal and milkymist?

2013-02-14 Thread Michael Walle
Am Donnerstag 14 Februar 2013, 17:48:09 schrieb Peter Maydell: > Hi; I'm looking for test images (and command lines) for the musicpal > and milkymist targets, because I have a patchset which makes some > changes to them [I'm trying to get rid of sysbus_add_memory()]. > Does anybody have any to hand

Re: [Qemu-devel] test images for musicpal and milkymist?

2013-02-14 Thread Peter Maydell
On 14 February 2013 19:03, Michael Walle wrote: > there is an offical binary for the Milkymist SoC, which should work > out of the box. You can find it here: > > http://www.milkymist.org/updates/current/flickernoise > > start it with sth like this: > qemu-system-lm32 -M milkymist -serial stdio

Re: [Qemu-devel] [PATCH v2 00/10] Cleanup bitops vs host-utils

2013-02-14 Thread Richard Henderson
On 02/14/2013 01:43 AM, Peter Maydell wrote: > I was hoping we'd be able to get rid of host-utils.h instead, > since "Utility compute operations used by translated code" is > now a completely irrelevant categorisation... I suppose an 11th patch could move the code back to bitops.h, but I much pref

Re: [Qemu-devel] [PATCH] qemu:cpuid: speedup test by 3x times if grub2 is available

2013-02-14 Thread Igor Mammedov
On Thu, 14 Feb 2013 13:13:18 +0100 Paolo Bonzini wrote: > Il 14/02/2013 12:18, Eduardo Habkost ha scritto: > >> > qemu boots from disk image 3 times faster than direct kernel load. > > That's surprising. Do you have any idea why that happens? > > Because kernel load uses MMIO (from fw_cfg), whil

Re: [Qemu-devel] [PATCH v4 3/3] QEMU-AER: Qemu changes to support AER for VFIO-PCI devices

2013-02-14 Thread Alex Williamson
On Thu, 2013-02-14 at 04:41 -0600, Vijay Mohan Pandarathil wrote: > - Create eventfd per vfio device assigned to a guest and register an > event handler > > - This fd is passed to the vfio_pci driver through the SET_IRQ ioctl > > - When the device encounters an error,

Re: [Qemu-devel] [virt-tools-list] [PATCH] qemu:cpuid: speedup test by 3x times if grub2 is available

2013-02-14 Thread Richard W.M. Jones
On Thu, Feb 14, 2013 at 05:07:48PM -0200, Eduardo Habkost wrote: > On Thu, Feb 14, 2013 at 06:41:26PM +, Richard W.M. Jones wrote: > > On Thu, Feb 14, 2013 at 11:41:08AM -0200, Eduardo Habkost wrote: > > > (CCing virt-tools list in case they have any existing solutions or ideas > > > on how to

Re: [Qemu-devel] [RFC v5] target-i386: Slim conversion to X86CPU subclasses + KVM subclasses

2013-02-14 Thread Igor Mammedov
On Thu, 14 Feb 2013 13:52:59 -0200 Eduardo Habkost wrote: > On Thu, Feb 14, 2013 at 04:13:22PM +0100, Igor Mammedov wrote: > > On Thu, 14 Feb 2013 09:44:59 -0200 > > Eduardo Habkost wrote: > [...] > > > > +if (kvm_enabled()) { > > > > +typename = g_strdup_printf("%s-kvm-" TYPE_X86_CP

Re: [Qemu-devel] [virt-tools-list] [PATCH] qemu:cpuid: speedup test by 3x times if grub2 is available

2013-02-14 Thread Eduardo Habkost
On Thu, Feb 14, 2013 at 06:41:26PM +, Richard W.M. Jones wrote: > On Thu, Feb 14, 2013 at 11:41:08AM -0200, Eduardo Habkost wrote: > > (CCing virt-tools list in case they have any existing solutions or ideas > > on how to make this easier for everybody) > > supermin? http://people.redhat.com/

Re: [Qemu-devel] [PATCH v4 2/3] VFIO-AER: Vfio-pci driver changes for supporting AER

2013-02-14 Thread Alex Williamson
On Thu, 2013-02-14 at 04:41 -0600, Vijay Mohan Pandarathil wrote: > - New VFIO_SET_IRQ ioctl option to pass the eventfd that is signaled > when > an error occurs in the vfio_pci_device > > - Register pci_error_handler for the vfio_pci driver > > - When the device enco

Re: [Qemu-devel] 3 new x86 instructions

2013-02-14 Thread Richard Henderson
On 02/14/2013 05:38 AM, Torbjorn Granlund wrote: > I now updated x86-next in order to get support for adox/adcx. Hmm. I'm not quite sure what the state of that branch is atm. I know I started splitting out work into other branches as will be required by the eventual merge; perhaps I didn't put t

[Qemu-devel] [PATCH] linux-user: Fix layout of usage table to account for option text

2013-02-14 Thread Peter Maydell
The linux-user usage message attempts to line up the columns in its table by calculating the maximum width of any item in them. However for the 'Argument' column it was only accounting for the length of the option switch (eg "-d"), not the additional example text (eg "item[,...]"). This currently h

Re: [Qemu-devel] [virt-tools-list] [PATCH] qemu:cpuid: speedup test by 3x times if grub2 is available

2013-02-14 Thread Richard W.M. Jones
On Thu, Feb 14, 2013 at 06:41:26PM +, Richard W.M. Jones wrote: > On Thu, Feb 14, 2013 at 11:41:08AM -0200, Eduardo Habkost wrote: > > (CCing virt-tools list in case they have any existing solutions or ideas > > on how to make this easier for everybody) > > supermin? http://people.redhat.com/

Re: [Qemu-devel] [virt-tools-list] [PATCH] qemu:cpuid: speedup test by 3x times if grub2 is available

2013-02-14 Thread Richard W.M. Jones
On Thu, Feb 14, 2013 at 11:41:08AM -0200, Eduardo Habkost wrote: > (CCing virt-tools list in case they have any existing solutions or ideas > on how to make this easier for everybody) supermin? http://people.redhat.com/~rjones/supermin/ It already exists on Fedora, Debian & Ubuntu, although it m

Re: [Qemu-devel] VFIO: Not require to make VFIO_IOMMU_MAP_DMA for MMAPED PCI bars

2013-02-14 Thread Alex Williamson
On Thu, 2013-02-14 at 08:22 +, Bhushan Bharat-R65777 wrote: > Hi Alex Williamson, > > I am able (with some hacks :)) to directly assign the e1000 PCI device > to KVM guest using VFIO on Freescale device. One of the problem I am > facing is about the DMA mapping in IOMMU for PCI device BARs. On

Re: [Qemu-devel] [PATCH RFC] memory: drop _overlap variant

2013-02-14 Thread Avi Kivity
On Thu, Feb 14, 2013 at 8:12 PM, Michael S. Tsirkin wrote: >> >> Is there an actual real problem that needs fixing? > > Yes. Guests sometimes cause device BARs to temporary overlap > the APIC range during BAR sizing. It works fine on a physical > system but fails on KVM since pci has same priority

[Qemu-devel] RFC: handling "backend too fast" in virtio-net

2013-02-14 Thread Luigi Rizzo
virtio-style network devices (where the producer and consumer chase each other through a shared memory block) can enter into a bad operating regime when the consumer is too fast. I am hitting this case right now when virtio is used on top of the netmap/VALE backend that I posted a few weeks ago: w

Re: [Qemu-devel] [PATCH] qemu-log: default to stderr for logging output

2013-02-14 Thread Peter Maydell
On 13 February 2013 12:57, Markus Armbruster wrote: >> - "-d options activate log (default logfile=%s)\n" >> - "-D logfile override default logfile location\n" >> + "-d options activate log (default is to log to stderr)\n" >> + "-D logfile write logs

Re: [Qemu-devel] [PATCH RFC] memory: drop _overlap variant

2013-02-14 Thread Michael S. Tsirkin
On Thu, Feb 14, 2013 at 07:02:15PM +0200, Avi Kivity wrote: > On Thu, Feb 14, 2013 at 6:50 PM, Michael S. Tsirkin wrote: > >> > As you see, ioapic at 0xfec0 overlaps pci-hole. > >> > ioapic is guest programmable in theory - should use _overlap? > >> > pci-hole is not but can overlap with ioapi

Re: [Qemu-devel] [PATCH for-1.4] e1000: unbreak the guest network migration to 1.3

2013-02-14 Thread Paolo Bonzini
Il 14/02/2013 18:15, Michael S. Tsirkin ha scritto: > QEMU 1.3 does not emulate the link auto negotiation, so if migrate to a > 1.3 machine during link auto negotiation, the guest link will be set to down. > Fix this by just disabling auto negotiation for 1.3. > > Signed-off-by: Michael S. Tsirkin

Re: [Qemu-devel] [PATCH qom-cpu-next 0/6] QOM CPUState, part 8: CPU_COMMON continued

2013-02-14 Thread Andreas Färber
Am 01.02.2013 13:38, schrieb Andreas Färber: > Hello, > > This series moves more fields from CPU_COMMON / CPU*State to CPUState, > allowing access from target-independent code. > > The final patch in this series will help solve some issues (in particular > avoid a dependency on CPU_COMMON TLB ref

Re: [Qemu-devel] Google Summer of Code 2013 ideas wiki open

2013-02-14 Thread harryxiyou
On Thu, Feb 14, 2013 at 11:15 PM, Stefan Hajnoczi wrote: [...] > Hi Harry, Hi Stefan, > Thanks for your interest. You can begin thinking about ideas but > please keep in mind that we are still in the very early stages of GSoC > preparation. > > Google will publish the list of accepted organizati

[Qemu-devel] [PATCH for-1.4] e1000: unbreak the guest network migration to 1.3

2013-02-14 Thread Michael S. Tsirkin
QEMU 1.3 does not emulate the link auto negotiation, so if migrate to a 1.3 machine during link auto negotiation, the guest link will be set to down. Fix this by just disabling auto negotiation for 1.3. Signed-off-by: Michael S. Tsirkin --- hw/e1000.c | 25 + hw/pc_piix

Re: [Qemu-devel] [PATCH RFC] memory: drop _overlap variant

2013-02-14 Thread Avi Kivity
On Thu, Feb 14, 2013 at 6:50 PM, Michael S. Tsirkin wrote: >> > As you see, ioapic at 0xfec0 overlaps pci-hole. >> > ioapic is guest programmable in theory - should use _overlap? >> > pci-hole is not but can overlap with ioapic. >> > So also _overlap? >> >> It's a bug. The ioapic is in the pc

Re: [Qemu-devel] [PATCH RFC] memory: drop _overlap variant

2013-02-14 Thread Michael S. Tsirkin
On Thu, Feb 14, 2013 at 05:07:02PM +0200, Avi Kivity wrote: > On Thu, Feb 14, 2013 at 4:40 PM, Michael S. Tsirkin wrote: > > On Thu, Feb 14, 2013 at 04:14:39PM +0200, Avi Kivity wrote: > > > > But some parents are system created and shared by many devices so children > > for > > such have no idea

[Qemu-devel] test images for musicpal and milkymist?

2013-02-14 Thread Peter Maydell
Hi; I'm looking for test images (and command lines) for the musicpal and milkymist targets, because I have a patchset which makes some changes to them [I'm trying to get rid of sysbus_add_memory()]. Does anybody have any to hand? thanks -- PMM

Re: [Qemu-devel] [RFC PATCH v2 03/23] qcow2: Change handle_dependency to byte granularity

2013-02-14 Thread Kevin Wolf
On Thu, Feb 14, 2013 at 03:48:51PM +0100, Stefan Hajnoczi wrote: > On Wed, Feb 13, 2013 at 02:21:53PM +0100, Kevin Wolf wrote: > > Signed-off-by: Kevin Wolf > > --- > > block/qcow2-cluster.c | 40 > > block/qcow2.h | 11 +++ > > 2 files

Re: [Qemu-devel] [PATCH for-1.4 v2 4/6] qemu-option: Disable two helpful messages that got broken recently

2013-02-14 Thread Luiz Capitulino
On Thu, 14 Feb 2013 14:31:50 +0100 Markus Armbruster wrote: > [Some quoted material restored] > > Luiz Capitulino writes: > > > On Thu, 14 Feb 2013 10:45:22 +0100 > > Markus Armbruster wrote: > > > >> [Note cc: +Laszlo, +Anthony, -qemu-trivial] > >> > >> Luiz Capitulino writes: > >> > >> >

Re: [Qemu-devel] [PATCH v2 00/10] Cleanup bitops vs host-utils

2013-02-14 Thread Eric Blake
On 02/13/2013 06:47 PM, Richard Henderson wrote: > Version 1 merely tried to adjust bitops_flsl, here I instead eliminate > it all from bitops.h, and standardizes on the routines from host-utils.h. > > > r~ > > > Richard Henderson (10): > host-utils: Add host long specific aliases for clz, ct

[Qemu-devel] single step not working after hit a break point

2013-02-14 Thread Peter Cheung
Hi All I use the following code to insert a breakpoint in physical address 0×160CPUArchState *cpu = first_cpu; hwaddr addr; sscanf(command + 2, "%ld", &addr); int err = cpu_breakpoint_insert(cpu, addr, BP_GDB, NULL );qemu successfully hit the breakpoint and stop, then i try to single-step

Re: [Qemu-devel] [RFC v5] target-i386: Slim conversion to X86CPU subclasses + KVM subclasses

2013-02-14 Thread Eduardo Habkost
On Thu, Feb 14, 2013 at 04:13:22PM +0100, Igor Mammedov wrote: > On Thu, 14 Feb 2013 09:44:59 -0200 > Eduardo Habkost wrote: [...] > > > +if (kvm_enabled()) { > > > +typename = g_strdup_printf("%s-kvm-" TYPE_X86_CPU, name); > > > > * Could we use the "-tcg" suffix for the non-KVM mode

Re: [Qemu-devel] [PATCH for-1.4 07/19] target-sparc: Fix debug output for DEBUG_MMU

2013-02-14 Thread Andreas Färber
Blue, Am 27.01.2013 14:32, schrieb Andreas Färber: > Signed-off-by: Andreas Färber > --- > target-sparc/ldst_helper.c |2 +- > 1 Datei geändert, 1 Zeile hinzugefügt(+), 1 Zeile entfernt(-) > > diff --git a/target-sparc/ldst_helper.c b/target-sparc/ldst_helper.c > index cf1bddf..7decd66 1006

  1   2   >