Re: [Qemu-devel] [PATCH v3 0/3] TPM NVRAM persistent storage

2013-06-14 Thread Stefan Berger
On 06/14/2013 11:38 AM, Anthony Liguori wrote: Corey Bryant writes: On 06/14/2013 10:01 AM, Anthony Liguori wrote: Corey Bryant writes: This patch series provides persistent storage support that a TPM can use to store NVRAM data. It uses QEMU's block driver to store data on a drive image.

Re: [Qemu-devel] [PATCH v3 0/3] TPM NVRAM persistent storage

2013-06-14 Thread Corey Bryant
On 06/14/2013 11:38 AM, Anthony Liguori wrote: Corey Bryant writes: On 06/14/2013 10:01 AM, Anthony Liguori wrote: Corey Bryant writes: This patch series provides persistent storage support that a TPM can use to store NVRAM data. It uses QEMU's block driver to store data on a drive imag

Re: [Qemu-devel] [PATCH v3 0/3] TPM NVRAM persistent storage

2013-06-14 Thread Anthony Liguori
Corey Bryant writes: > On 06/14/2013 11:38 AM, Anthony Liguori wrote: >> Corey Bryant writes: >> >>> On 06/14/2013 10:01 AM, Anthony Liguori wrote: Corey Bryant writes: > This patch series provides persistent storage support that a TPM > can use to store NVRAM data. It uses Q

Re: [Qemu-devel] [Qemu-ppc] [PATCH] target-ppc: Change default machine for 64-bit

2013-06-14 Thread Andreas Färber
Am 13.06.2013 21:02, schrieb Anthony Liguori: > Scott Wood writes: > >> On 06/13/2013 01:31:48 PM, Anthony Liguori wrote: >>> I'll note that we have a CONFIG_PSERIES today. I'd suggest getting >>> rid >>> of it first before making pseries the default. >> >> Why? > > I believe it's there only

Re: [Qemu-devel] [PATCH v3 0/3] TPM NVRAM persistent storage

2013-06-14 Thread Corey Bryant
On 06/14/2013 11:56 AM, Anthony Liguori wrote: Corey Bryant writes: On 06/14/2013 11:38 AM, Anthony Liguori wrote: Corey Bryant writes: On 06/14/2013 10:01 AM, Anthony Liguori wrote: Corey Bryant writes: This patch series provides persistent storage support that a TPM can use to stor

[Qemu-devel] [PATCH 0/2] Support ramdisks with U-Boot header

2013-06-14 Thread Soren Brinkmann
For loading Linux kernels, QEMU already is able to recognize and load them when they feature a U-Boot header. This patch series targets to extend this support to ramdisks. Furthermore the ARM support code is changed to use the new functionality. Regards, Sören Soren Brinkmann (2):

[Qemu-devel] [PATCH 2/2] hw/arm: Use 'load_ramdisk()' for loading ramdisk

2013-06-14 Thread Soren Brinkmann
The load_ramdisk function takes over loading traditional ramdisks images and does also load ramdisks with u-boot header. Signed-off-by: Soren Brinkmann --- hw/arm/boot.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/hw/arm/boot.c b/hw/arm/boot.c index f310f73..ab2b2

[Qemu-devel] [PATCH 1/2] hw/loader: Support ramdisk with u-boot header

2013-06-14 Thread Soren Brinkmann
Introduce 'load_ramdisk()' which can load "normal" ramdisks and ramdisks with a u-boot header. To enable this and leverage synergies 'load_uimage()' is refactored to accomodate this additional use case. Signed-off-by: Soren Brinkmann --- hw/core/loader.c| 86 +

Re: [Qemu-devel] [RFC] sanitize memory on system reset

2013-06-14 Thread H. Peter Anvin
Only on a real 286. At least since 486 the legacy switch has been INIT, not RESET. Alexander Graf wrote: > > >Am 14.06.2013 um 08:56 schrieb Christian Borntraeger >: > >> On 13/06/13 13:56, Anthony Liguori wrote: >>> Markus Armbruster writes: >>> Peter Lieven writes: > On 13.

Re: [Qemu-devel] [PATCH 1/2] hw/loader: Support ramdisk with u-boot header

2013-06-14 Thread Peter Maydell
On 14 June 2013 17:36, Soren Brinkmann wrote: > Introduce 'load_ramdisk()' which can load "normal" ramdisks and ramdisks > with a u-boot header. > To enable this and leverage synergies 'load_uimage()' is refactored to > accomodate this additional use case. Hi; thanks for this patch; some review c

Re: [Qemu-devel] [PATCH] arm/boot: Free dtb blob memory after use

2013-06-14 Thread Andreas Färber
Am 14.06.2013 13:27, schrieb Peter Maydell: > The dtb blob returned by load_device_tree() is in memory allocated > with g_malloc(). Free it accordingly once we have copied its > contents into the guest memory. To make this easy, we need also to > clean up the error handling in load_dtb() so that we

Re: [Qemu-devel] [PATCH 1/2] hw/loader: Support ramdisk with u-boot header

2013-06-14 Thread Sören Brinkmann
Hi Peter, On Fri, Jun 14, 2013 at 05:56:31PM +0100, Peter Maydell wrote: > On 14 June 2013 17:36, Soren Brinkmann wrote: > > Introduce 'load_ramdisk()' which can load "normal" ramdisks and ramdisks > > with a u-boot header. > > To enable this and leverage synergies 'load_uimage()' is refactored t

Re: [Qemu-devel] [PATCH] arm/boot: Free dtb blob memory after use

2013-06-14 Thread Peter Maydell
On 14 June 2013 18:14, Andreas Färber wrote: > Am 14.06.2013 13:27, schrieb Peter Maydell: >> The dtb blob returned by load_device_tree() is in memory allocated >> with g_malloc(). Free it accordingly once we have copied its >> contents into the guest memory. To make this easy, we need also to >>

Re: [Qemu-devel] [PATCH 1/2] hw/loader: Support ramdisk with u-boot header

2013-06-14 Thread Peter Maydell
On 14 June 2013 18:14, Sören Brinkmann wrote: > On Fri, Jun 14, 2013 at 05:56:31PM +0100, Peter Maydell wrote: >> If we're providing separate functions for kernel and >> ramdisk loading shouldn't they check that the uimage >> has the appropriate type? > I'm not sure I understand what you mean here

Re: [Qemu-devel] [PATCH 1/2] hw/loader: Support ramdisk with u-boot header

2013-06-14 Thread Sören Brinkmann
On Fri, Jun 14, 2013 at 06:36:06PM +0100, Peter Maydell wrote: > On 14 June 2013 18:14, Sören Brinkmann wrote: > > On Fri, Jun 14, 2013 at 05:56:31PM +0100, Peter Maydell wrote: > >> If we're providing separate functions for kernel and > >> ramdisk loading shouldn't they check that the uimage > >>

Re: [Qemu-devel] [PULL 21/26] ide-test: Add FLUSH CACHE test case

2013-06-14 Thread Peter Maydell
On 7 June 2013 12:58, Stefan Hajnoczi wrote: > From: Kevin Wolf > > This checks in particular that BSY is set while the flush request is in > flight. This test doesn't seem to pass. At commit bd07684aac (where the test landed): cam-vm-266:precise:qemu$ (cd build/x86; QTEST_QEMU_BINARY=i386-soft

Re: [Qemu-devel] [PATCH] make screendump an async command

2013-06-14 Thread Alon Levy
> Hi, > > > Note: due to QAPI not generating async commands yet I had to remove the > > schema screendump definition. > > Hmm, that will break libvirt I suspect. Guess this one has to wait > until QAPI gained proper async command support. It doesn't break anything. I've tested, and here is the

Re: [Qemu-devel] [PATCH] make screendump an async command

2013-06-14 Thread Alon Levy
> > Hi, > > > > > Note: due to QAPI not generating async commands yet I had to remove the > > > schema screendump definition. > > > > Hmm, that will break libvirt I suspect. Guess this one has to wait > > until QAPI gained proper async command support. > > It doesn't break anything. I've tested

Re: [Qemu-devel] [PULL 21/26] ide-test: Add FLUSH CACHE test case

2013-06-14 Thread Peter Maydell
On 14 June 2013 18:54, Peter Maydell wrote: > On 7 June 2013 12:58, Stefan Hajnoczi wrote: >> From: Kevin Wolf >> >> This checks in particular that BSY is set while the flush request is in >> flight. > > This test doesn't seem to pass. > ERROR:/home/petmay01/linaro/qemu-from-laptop/qemu/tests/i

[Qemu-devel] [PATCH] vl: fix compilation without SDL

2013-06-14 Thread Vincent Stehlé
Fix the following error, which happens when CONFIG_SDL is not defined: vl.c: In function ‘main’: vl.c:3528:17: error: ‘no_frame’ undeclared (first use in this function) vl.c:3528:17: note: each undeclared identifier is reported only once for each function it appears in Protect code accessi

Re: [Qemu-devel] [PULL 21/26] ide-test: Add FLUSH CACHE test case

2013-06-14 Thread Anthony Liguori
Peter Maydell writes: > On 14 June 2013 18:54, Peter Maydell wrote: >> On 7 June 2013 12:58, Stefan Hajnoczi wrote: >>> From: Kevin Wolf >>> >>> This checks in particular that BSY is set while the flush request is in >>> flight. >> >> This test doesn't seem to pass. > >> ERROR:/home/petmay01/l

Re: [Qemu-devel] [PATCH] make screendump an async command

2013-06-14 Thread Anthony Liguori
Alon Levy writes: > This fixes the broken screendump behavior for qxl devices in native mode > since 81fb6f1504fb9ef71f2382f44af34756668296e8. > > Note: due to QAPI not generating async commands yet I had to remove the > schema screendump definition. > > Related RHBZ: 973374 > This patch is not e

Re: [Qemu-devel] [PATCH v3 0/3] TPM NVRAM persistent storage

2013-06-14 Thread Anthony Liguori
Corey Bryant writes: > On 06/14/2013 11:56 AM, Anthony Liguori wrote: >> Corey Bryant writes: >> >>> On 06/14/2013 11:38 AM, Anthony Liguori wrote: Corey Bryant writes: > On 06/14/2013 10:01 AM, Anthony Liguori wrote: >> Corey Bryant writes: >> >>> This patch series p

Re: [Qemu-devel] [PATCH] vl: fix compilation without SDL

2013-06-14 Thread Stefan Weil
Am 14.06.2013 20:12, schrieb Vincent Stehlé: > Fix the following error, which happens when CONFIG_SDL is not defined: > > vl.c: In function ‘main’: > vl.c:3528:17: error: ‘no_frame’ undeclared (first use in this function) > vl.c:3528:17: note: each undeclared identifier is reported only once

[Qemu-devel] [PATCH] Revert "Unbreak -no-quit for GTK, validate SDL options"

2013-06-14 Thread Michael Tokarev
This reverts commit 047d4e151dd462915786a4fddc12f774d0028af5. The commit in question introduced old/legacy options (-no-quit and some more) for the new interface type (gtk), -- on the second thought I think we shouldn't do that, instead, we should use the display-specific options (like -display sd

[Qemu-devel] [PATCH v9 13/14] rdma: fix mlock() freezes and accounting

2013-06-14 Thread mrhines
From: "Michael R. Hines" This patch is contained to migration-rdma.c and fixes the problems experienced by others when the x-rdma-pin-all feature appeared to freeze the VM. By moving this operation out of the connection setup time and instead moving it to ram_save_setup() code, we no longer execu

[Qemu-devel] [PATCH v9 00/14] rdma: migration support

2013-06-14 Thread mrhines
From: "Michael R. Hines" Changes since v8: For very large virtual machines, pinning can take a long time. While this does not affect the migration's *actual* time itself, it is still important for the user to know what's going on and to know what component of the total time is act

[Qemu-devel] [PATCH v9 02/14] rdma: introduce qemu_update_position()

2013-06-14 Thread mrhines
From: "Michael R. Hines" RDMA writes happen asynchronously, and thus the performance accounting also needs to be able to occur asynchronously. This allows anybody to call into savevm.c to update both f->pos as well as into arch_init.c to update the acct_info structure with up-to-date values when

Re: [Qemu-devel] RDMA: please pull and re-test freezing fixes

2013-06-14 Thread Michael R. Hines
Chegu, I sent a V9 to the mailing list: The version goes even further, by explicitly timing the pinning latency and pushing the value out to QMP so the user clearly knows which component of total migration time is consumed by pinning. If you're satisfied, I'd appreciate if I could add your Revi

[Qemu-devel] [PATCH v9 07/14] rdma: introduce ram_handle_compressed()

2013-06-14 Thread mrhines
From: "Michael R. Hines" This gives RDMA shared access to madvise() on the destination side when an entire chunk is found to be zero. Reviewed-by: Paolo Bonzini Signed-off-by: Michael R. Hines --- arch_init.c | 24 include/migration/migration.h |

[Qemu-devel] [PATCH v9 12/14] rdma: send pc.ram

2013-06-14 Thread mrhines
From: "Michael R. Hines" This takes advantages of the previous patches: 1. use the new QEMUFileOps hook 'save_page' 2. call out to the right accessor methods to invoke the iteration hooks defined in QEMUFileOps Reviewed-by: Paolo Bonzini Signed-off-by: Michael R. Hines --- arch_init.c |

[Qemu-devel] [PATCH v9 14/14] rdma: add pin-all accounting timestamp to QMP statistics

2013-06-14 Thread mrhines
From: "Michael R. Hines" For very large virtual machines, pinning can take a long time. While this does not affect the migration's *actual* time itself, it is still important for the user to know what's going on and to know what component of the total time is actual taken up by pinning. For exam

[Qemu-devel] [PATCH v9 10/14] rdma: introduce capability x-rdma-pin-all

2013-06-14 Thread mrhines
From: "Michael R. Hines" This capability allows you to disable dynamic chunk registration for better throughput on high-performance links. For example, using an 8GB RAM virtual machine with all 8GB of memory in active use and the VM itself is completely idle using a 40 gbps infiniband link: 1.

[Qemu-devel] [PATCH v9 01/14] rdma: add documentation

2013-06-14 Thread mrhines
From: "Michael R. Hines" docs/rdma.txt contains full documentation, wiki links, github url and contact information. Reviewed-by: Paolo Bonzini Signed-off-by: Michael R. Hines --- docs/rdma.txt | 404 + 1 file changed, 404 insertions(+)

[Qemu-devel] [PATCH v9 05/14] rdma: introduce qemu_file_mode_is_not_valid()

2013-06-14 Thread mrhines
From: "Michael R. Hines" QEMUFileRDMA also has read and write modes. This function is now shared to reduce code duplication. Reviewed-by: Paolo Bonzini Signed-off-by: Michael R. Hines --- include/migration/qemu-file.h |1 + savevm.c | 20 +--- 2 file

[Qemu-devel] [PATCH v9 03/14] rdma: export yield_until_fd_readable()

2013-06-14 Thread mrhines
From: "Michael R. Hines" The RDMA event channel can be made non-blocking just like a TCP socket. Exporting this function allows us to yield so that the QEMU monitor remains available. Reviewed-by: Paolo Bonzini Signed-off-by: Michael R. Hines --- include/block/coroutine.h |6 ++ qemu-

[Qemu-devel] [PATCH v9 09/14] rdma: new QEMUFileOps hooks

2013-06-14 Thread mrhines
From: "Michael R. Hines" These are the prototypes and implementation of new hooks that RDMA takes advantage of to perform dynamic page registration. An optional hook is also introduced for a custom function to be able to override the default save_page function. Also included are the prototypes

[Qemu-devel] [PATCH v9 08/14] rdma: introduce qemu_ram_foreach_block()

2013-06-14 Thread mrhines
From: "Michael R. Hines" This is used during RDMA initialization in order to transmit a description of all the RAM blocks to the peer for later dynamic chunk registration purposes. Reviewed-by: Paolo Bonzini Signed-off-by: Michael R. Hines --- exec.c|9 + inclu

[Qemu-devel] [PATCH v9 06/14] rdma: export qemu_fflush()

2013-06-14 Thread mrhines
From: "Michael R. Hines" RDMA uses this to flush the control channel before sending its own message to handle page registrations. Reviewed-by: Paolo Bonzini Signed-off-by: Michael R. Hines --- include/migration/qemu-file.h |1 + savevm.c |2 +- 2 files changed, 2

[Qemu-devel] [PATCH v9 04/14] rdma: export throughput w/ MigrationStats QMP

2013-06-14 Thread mrhines
From: "Michael R. Hines" This exposes throughput (in megabits/sec) through QMP. Reviewed-by: Paolo Bonzini Signed-off-by: Michael R. Hines --- hmp.c |2 ++ include/migration/migration.h |1 + migration.c |6 ++ qapi-schema.json

Re: [Qemu-devel] [Qemu-ppc] [PATCH] target-ppc: Change default machine for 64-bit

2013-06-14 Thread Benjamin Herrenschmidt
On Fri, 2013-06-14 at 16:28 +0200, Andreas Färber wrote: > > I would still love to see -M mac99 emulate a proper U2 or U1 based > system for 32bit. And I would also love to see a real U4 based > emulation model come up for qemu-system-ppc64. But I doubt I'll have > time to work on either :). Unino

Re: [Qemu-devel] [PATCH v2] kvm/openpic: in-kernel mpic support

2013-06-14 Thread Scott Wood
On 06/14/2013 09:59:18 AM, Andreas Färber wrote: Am 13.06.2013 19:32, schrieb Scott Wood: > On 06/13/2013 06:01:49 AM, Andreas Färber wrote: >> Am 12.06.2013 22:32, schrieb Scott Wood: >> > +typedef struct KVMOpenPICState { >> > +SysBusDevice busdev; >> >> SysBusDevice parent_obj; please! >>

[Qemu-devel] [PATCH v2 0/2] Support ramdisks with U-Boot header

2013-06-14 Thread Soren Brinkmann
Updated with Peter's comments. The detailed changelog can be found in the following emails with the patches. Sören Soren Brinkmann (2): hw/loader: Support ramdisk with u-boot header hw/arm: Use 'load_ramdisk()' for loading ramdisk hw/arm/boot.c | 8 ++--- hw/core/loader.c

[Qemu-devel] [PATCH v2 2/2] hw/arm: Use 'load_ramdisk()' for loading ramdisk

2013-06-14 Thread Soren Brinkmann
The load_ramdisk function takes over loading traditional ramdisks images and does also load ramdisks with u-boot header. Signed-off-by: Soren Brinkmann --- hw/arm/boot.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/hw/arm/boot.c b/hw/arm/boot.c index f310f73..ab2b2

[Qemu-devel] [PATCH v2 1/2] hw/loader: Support ramdisk with u-boot header

2013-06-14 Thread Soren Brinkmann
Introduce 'load_ramdisk()' which can load "normal" ramdisks and ramdisks with a u-boot header. To enable this and leverage synergies 'load_uimage()' is refactored to accomodate this additional use case. Signed-off-by: Soren Brinkmann --- v2: - document the new 'load_ramdisk()' function - print

Re: [Qemu-devel] [PATCH] make screendump an async command

2013-06-14 Thread Alon Levy
On Fri, 2013-06-14 at 13:21 -0500, Anthony Liguori wrote: > Alon Levy writes: > > > This fixes the broken screendump behavior for qxl devices in native mode > > since 81fb6f1504fb9ef71f2382f44af34756668296e8. > > > > Note: due to QAPI not generating async commands yet I had to remove the > > sche

Re: [Qemu-devel] [SeaBIOS] solaris x86 in qemu? [bisected]

2013-06-14 Thread Kevin O'Connor
On Fri, Jun 14, 2013 at 06:14:00PM +0400, Michael Tokarev wrote: > 14.06.2013 16:36, Gerd Hoffmann wrote: > > Hi, > > > >> Hmm. Speaking of the splitting. Does the current bios include the > >> the tables which were split into separate files? > > > > Yes, they are in out/ too after building s

[Qemu-devel] [0/3] Accumulated ppc patches

2013-06-14 Thread David Gibson
Hi Alex, I see that you're now back from your holiday. Here, rather belatedly, is my series of ppc related patches collected while acting ppc maintainer. As you can see there aren't many at all. I'm not sure if that's because I wasn't looking in the right places for people sending patches, or th

[Qemu-devel] [PATCH 1/3] target-ppc: Change default machine for 64-bit

2013-06-14 Thread David Gibson
Currently, for qemu-system-ppc64, the default machine type is 'mac99'. The mac99 machine is not being actively maintained, and represents a bizarre hybrid of components that never actually existed as a real system. This patch changes the default machine to 'pseries', which is actively maintained an

[Qemu-devel] [PATCH 2/3] target-ppc kvm: save cr register

2013-06-14 Thread David Gibson
From: Alexey Kardashevskiy This adds a missing code to save CR (condition register) via kvm_arch_put_registers(). kvm_arch_get_registers() already has it. Signed-off-by: Alexey Kardashevskiy Signed-off-by: David Gibson --- target-ppc/kvm.c | 5 + 1 file changed, 5 insertions(+) diff --gi

[Qemu-devel] [PATCH 3/3] pseries: Update MAINTAINERS information

2013-06-14 Thread David Gibson
I'm no longer at IBM, and therefore no long actively working on the pseries (aka sPAPR) qemu machine type. This patch replaces my information in the MAINTAINERS file with that for Alexey Kardashevskiy, who is taking over maintainership of the pseries code. While we're at it, I've added some extra

Re: [Qemu-devel] [PATCH] spapr: add yet another maintainer

2013-06-14 Thread David Gibson
On Thu, Jun 13, 2013 at 02:10:43AM +0200, Alexander Graf wrote: > > On 13.06.2013, at 02:08, Alexey Kardashevskiy wrote: > > > On 06/13/2013 12:33 AM, Alexander Graf wrote: > >> > >> On 12.06.2013, at 16:27, Alexey Kardashevskiy wrote: > >> > >>> Signed-off-by: Alexey Kardashevskiy > >>> --- >

Re: [Qemu-devel] [PATCH] QEMUBH: make AioContext's bh re-entrant

2013-06-14 Thread Paolo Bonzini
Il 13/06/2013 22:55, Liu Ping Fan ha scritto: > BH will be used outside big lock, so introduce lock to protect it. > Note that the lock only affects the writer and bh's callback does > not take this extra lock. > > Signed-off-by: Liu Ping Fan > --- > async.c | 10 +- > includ

<    1   2   3