[Qemu-devel] [PATCH 1/2 v5] add function DMA_set_return and delete bh_schedule in dma.c

2012-04-18 Thread Li Zhi Hui
add function DMA_set_return and delete bh_schedule in dma.c Signed-off-by: Li Zhi Hui --- hw/dma.c | 13 + hw/isa.h |1 + 2 files changed, 14 insertions(+), 0 deletions(-) diff --git a/hw/dma.c b/hw/dma.c index 0a9322d..55dc320 100644 --- a/hw/dma.c +++ b/hw/dma.c @@ -460,6 +4

Re: [Qemu-devel] [PATCH v9 05/10] Add MigrationParams structure

2012-04-18 Thread Orit Wasserman
On 04/18/2012 08:26 PM, Anthony Liguori wrote: > On 04/11/2012 01:49 PM, Orit Wasserman wrote: >> Signed-off-by: Isaku Yamahata > > If you're not going to SoB this patch, please make sure you set the > GIT_AUTHOR appropriately so that there's a From line for Isaku. Otherwise it > will mess up g

Re: [Qemu-devel] [PATCH v9 04/10] Add host_from_stream_offset_versioned function

2012-04-18 Thread Orit Wasserman
On 04/18/2012 05:42 PM, Juan Quintela wrote: > Orit Wasserman wrote: >> Signed-off-by: Orit Wasserman >> Signed-off-by: Benoit Hudzia >> Signed-off-by: Petter Svard >> Signed-off-by: Aidan Shribman >> --- >> arch_init.c | 26 +++--- >> 1 files changed, 23 insertions(+),

Re: [Qemu-devel] [PATCH v9 01/10] Add cache handling functions

2012-04-18 Thread Orit Wasserman
On 04/18/2012 08:19 PM, Anthony Liguori wrote: > On 04/11/2012 01:49 PM, Orit Wasserman wrote: >> Add LRU page cache mechanism. >> The page are accessed by their address. >> >> Signed-off-by: Orit Wasserman >> Signed-off-by: Benoit Hudzia >> Signed-off-by: Petter Svard >> Signed-off-by: Aidan Shrib

Re: [Qemu-devel] [PATCH v9 09/10] Add set_cachesize command

2012-04-18 Thread Orit Wasserman
On 04/18/2012 08:07 PM, Juan Quintela wrote: > Orit Wasserman wrote: >> Change XBZRLE cache size in bytes (the size should be a power of 2). >> If XBZRLE cache size is too small there will be many cache miss. > >> +.name = "migrate_set_cachesize", >> +.args_type = "value:o"

Re: [Qemu-devel] [PATCH v9 06/10] Add XBZRLE to ram_save_block and ram_save_live

2012-04-18 Thread Orit Wasserman
On 04/18/2012 07:54 PM, Juan Quintela wrote: > Orit Wasserman wrote: > >> @@ -104,6 +104,7 @@ const uint32_t arch_type = QEMU_ARCH; >> #define RAM_SAVE_FLAG_PAGE 0x08 >> #define RAM_SAVE_FLAG_EOS 0x10 >> #define RAM_SAVE_FLAG_CONTINUE 0x20 >> +#define RAM_SAVE_FLAG_XBZRLE 0x40 > > m

[Qemu-devel] [PATCH 2/2 v5] Replace bdrv_* to bdrv_aio_* functions in DMA mode in fdc.c.

2012-04-18 Thread Li Zhi Hui
Replace bdrv_* to bdrv_aio_* functions in DMA mode in fdc.c. Signed-off-by: Li Zhi Hui --- change log: v3: correct some mistakes about memory v4: change the return value of function fdctrl_transfer_handler v5: introduce a flag to avoid re-enter function fdctrl_transfer_handler hw/fdc.c | 277 +

Re: [Qemu-devel] [PATCH] add function DMA_set_return and delete bh_schedule in dma.c

2012-04-18 Thread Zhi Hui Li
On 2012年04月17日 16:46, Paolo Bonzini wrote: Il 16/04/2012 14:14, Stefan Hajnoczi ha scritto: Why are you removing the rearm behavior? I'm pretty sure other ISA devices rely on this otherwise the code wouldn't exist. Li is correct in that the code can be simplified a lot, but indeed his axe wen

[Qemu-devel] [PATCH 00/16] QEMU vhost-scsi support

2012-04-18 Thread zwu . kernel
From: Zhi Yong Wu The patchset was developed originally by Stefan about one year ago. I now rebase it to latest qemu.git/master and fixed some issues to make it work against tcm_vhost and virtio_scsi driver. But there are still some issues to fix. Let us make more effort later. Stefan Hajnocz

[Qemu-devel] [PATCH 14/16] vhost-scsi: enable vhost notifiers for multiple queues

2012-04-18 Thread zwu . kernel
From: Zhi Yong Wu Signed-off-by: Zhi Yong Wu --- hw/vhost-scsi.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/hw/vhost-scsi.c b/hw/vhost-scsi.c index 82e0a68..54e6216 100644 --- a/hw/vhost-scsi.c +++ b/hw/vhost-scsi.c @@ -64,6 +64,12 @@ int vhost_scsi_start(VHos

[Qemu-devel] [PATCH 06/16] virtio-scsi: Open and initialize /dev/vhost-scsi

2012-04-18 Thread zwu . kernel
From: Stefan Hajnoczi Begin adding vhost support by opening /dev/vhost-scsi. Signed-off-by: Stefan Hajnoczi Signed-off-by: Zhi Yong Wu --- hw/virtio-scsi.c | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/hw/virtio-scsi.c b/hw/virtio-scsi.c index ff5d351..5

[Qemu-devel] [PATCH 16/16] vhost-scsi: clear endpoint on stopped

2012-04-18 Thread zwu . kernel
From: Zhi Yong Wu Signed-off-by: Zhi Yong Wu --- hw/vhost-scsi.c | 11 ++- 1 files changed, 10 insertions(+), 1 deletions(-) diff --git a/hw/vhost-scsi.c b/hw/vhost-scsi.c index 62f80bb..3e3378a 100644 --- a/hw/vhost-scsi.c +++ b/hw/vhost-scsi.c @@ -84,7 +84,16 @@ int vhost_scsi_star

[Qemu-devel] [PATCH 12/16] virtio-scsi: use the vhost-scsi host device

2012-04-18 Thread zwu . kernel
From: Stefan Hajnoczi The vhost-scsi code was originally part of the virtio-scsi emulation. This code has since been factored out and should be used instead. Signed-off-by: Stefan Hajnoczi Signed-off-by: Zhi Yong Wu --- hw/virtio-pci.c |2 ++ hw/virtio-scsi.c | 10 +++--- hw/virtio

[Qemu-devel] [PATCH 07/16] virtio-scsi: Start/stop vhost

2012-04-18 Thread zwu . kernel
From: Stefan Hajnoczi This patch starts and stops vhost as the virtio device transitions through its status phases. Vhost can only be started once the guest reports its driver has successfully initialized, which means the virtqueues have been set up by the guest. Signed-off-by: Stefan Hajnoczi

[Qemu-devel] [PATCH 10/16] virtio-pci: check that event notification worked

2012-04-18 Thread zwu . kernel
From: Stefan Hajnoczi For sanity assert that event notification succeeds. Signed-off-by: Stefan Hajnoczi --- hw/virtio-pci.c |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/hw/virtio-pci.c b/hw/virtio-pci.c index dcd58d0..025a518 100644 --- a/hw/virtio-pci.c +++ b/

[Qemu-devel] [PATCH 15/16] vhost-scsi: move some definitions to its header file

2012-04-18 Thread zwu . kernel
From: Zhi Yong Wu Signed-off-by: Zhi Yong Wu --- hw/vhost-scsi.c |9 + hw/vhost-scsi.h | 11 +++ 2 files changed, 12 insertions(+), 8 deletions(-) diff --git a/hw/vhost-scsi.c b/hw/vhost-scsi.c index 54e6216..62f80bb 100644 --- a/hw/vhost-scsi.c +++ b/hw/vhost-scsi.c @@ -

[Qemu-devel] [PATCH 08/16] notifier: add validity check and notify function

2012-04-18 Thread zwu . kernel
From: Stefan Hajnoczi Event notifiers that have not had the event_notifier_init() function called on them are invalid. The event_notifier_valid() function checks whether or not an event notifier is valild. This can be used to check whether a notifier is in use or not. It sometimes useful to no

[Qemu-devel] [PATCH 11/16] vhost-scsi: add -vhost-scsi host device

2012-04-18 Thread zwu . kernel
From: Stefan Hajnoczi This patch adds a new type of host device that drives the vhost_scsi device. The syntax to add vhost-scsi is: qemu -vhost-scsi id=vhost-scsi0,wwpn=...,tpgt=123 The virtio-scsi emulated device will make use of vhost-scsi to process virtio-scsi requests inside the kernel

[Qemu-devel] [PATCH 13/16] virtio-scsi: WIP VHOST_SCSI_SET_ENDPOINT call

2012-04-18 Thread zwu . kernel
From: Stefan Hajnoczi Current we get -EINVAL, need to debug this. Signed-off-by: Stefan Hajnoczi Signed-off-by: Zhi Yong Wu --- hw/vhost-scsi.c | 19 ++- hw/virtio-scsi.h | 11 +-- 2 files changed, 19 insertions(+), 11 deletions(-) diff --git a/hw/vhost-scsi.c b/

[Qemu-devel] [PATCH 04/16] virtio-scsi: Fix tgpt typo to tpgt and use uint16_t

2012-04-18 Thread zwu . kernel
From: Stefan Hajnoczi Signed-off-by: Stefan Hajnoczi Signed-off-by: Zhi Yong Wu --- hw/virtio-scsi.h |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/virtio-scsi.h b/hw/virtio-scsi.h index 9cddacb..f66d12c 100644 --- a/hw/virtio-scsi.h +++ b/hw/virtio-scsi.h @@ -4

[Qemu-devel] [PATCH 03/16] virtio-scsi: Add vhost_vring_target ioctl struct

2012-04-18 Thread zwu . kernel
From: Stefan Hajnoczi The vhost-scsi device is associated with a tcm_vhost target using the VHOST_SCSI_SET_ENDPOINT and VHOST_SCSI_CLEAR_ENDPOINT ioctls that take a wwpn and tpgt. Signed-off-by: Stefan Hajnoczi --- hw/virtio-scsi.h |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-

Re: [Qemu-devel] [PATCH 14/20] qcow2: Factor out count_cow_clusters

2012-04-18 Thread Marcelo Tosatti
On Mon, Mar 12, 2012 at 04:19:45PM +0100, Kevin Wolf wrote: > Signed-off-by: Kevin Wolf > Reviewed-by: Stefan Hajnoczi > --- > block/qcow2-cluster.c | 55 > - > 1 files changed, 36 insertions(+), 19 deletions(-) Kevin, Autotest installed Fedor

[Qemu-devel] [PATCH 09/16] virtio-pci: support host notifiers in TCG mode

2012-04-18 Thread zwu . kernel
From: Stefan Hajnoczi Normally host notifiers are only used together with vhost-net in KVM mode. It is occassionally useful to use vhost with TCG mode, mainly for testing and development. This isn't hard to achieve, simply fall back to notifying the host notifier manually from qemu if KVM mode

[Qemu-devel] [PATCH 01/16] virtio-scsi: Add wwpn and tgpt properties

2012-04-18 Thread zwu . kernel
From: Stefan Hajnoczi A vhost-scsi device must be bound to a tcm_vhost target port. This will be done by issuing an ioctl to the device with the wwpn and tgpt. This patch adds these properties to the virtio-scsi-pci device: qemu -device virtio-scsi-pci,wwpn=$TARGET_WWN,tgpt=$TPGT Future pat

[Qemu-devel] [PATCH 05/16] virtio-scsi: Build virtio-scsi.o against vhost.o

2012-04-18 Thread zwu . kernel
From: Stefan Hajnoczi For the time being virtio-scsi.c will directly use vhost without a host device abstraction like virtio-net does. This patch builds virtio-scsi with vhost on KVM targets. Also make sure that virtio-pci.o can be built without virtio-scsi.o. Signed-off-by: Stefan Hajnoczi -

[Qemu-devel] [PATCH 02/16] vhost: Pass device path to vhost_dev_init()

2012-04-18 Thread zwu . kernel
From: Stefan Hajnoczi The path to /dev/vhost-net is currently hardcoded in vhost_dev_init(). This needs to be changed so that /dev/vhost-scsi can be used. Pass in the device path instead of hardcoding it. Signed-off-by: Stefan Hajnoczi Signed-off-by: Zhi Yong Wu --- hw/vhost.c |5 +++

Re: [Qemu-devel] [RFC PATCH v1 2/4] m25p80: initial verion

2012-04-18 Thread Peter Crosthwaite
Hi Andreas, So is there any standard policy on setting maintainer-ships for device models? When I pushed the cadence IP device models (hw/cadence*) for xilinx zynq there was insistence that they be in the MAINTAINERS, however like m25p80, they are not necessarily zynq specific (indeed cadence coul

[Qemu-devel] [Bug 985288] [NEW] scsi disk emulation doesn't enforce FUA (Force Unit Access) in write-back mode

2012-04-18 Thread Asad Saeed
Public bug reported: Microsoft NTFS utilizes the FUA bit in SCSI WRITE CDBs to insure integrity when a device advertises that it has write caching enabled. The FUA bit is meant to ensure a write is written to non-volatile storage before returning. This seems to not be enforced by QEMU's SCSI emul

Re: [Qemu-devel] qemu physical address

2012-04-18 Thread Xin Tong
but should not the address be within 1 - 4G-1 even with PAE. is not the PAE just using 64bits addresses as supposed to 32 bit ? what does the physical address bigger than 4G mean ? is not the physical address starting from 0 from the prospective of the processor ? Xin On Wed, Apr 18, 2012 at 4:

[Qemu-devel] Poking at m68k gitorious branch.

2012-04-18 Thread Rob Landley
My aboriginal linux project has bootable system images for a bunch of targets (see http://landley.net/aboriginal/bin) and I want to add m68k. For years it's been building a stub m68k system image, but the run-emulator.sh script that should theoretically launch it is just a stub. A user once got my

Re: [Qemu-devel] [SeaBIOS] [PATCH 0/3] switch to seavgabios

2012-04-18 Thread Kevin O'Connor
On Wed, Apr 18, 2012 at 03:07:35PM +0200, Gerd Hoffmann wrote: > seavgabios has no 32bit code at all at the moment. vesa pmi didn't seem > to be important enougth to change it. > > seabios is a 16/32bit hybrid with some code being compiled twice for > both modes; dunno how reusable the seabios in

Re: [Qemu-devel] [PATCH 0/3] switch to seavgabios

2012-04-18 Thread Kevin O'Connor
On Wed, Apr 18, 2012 at 10:45:06PM +0400, malc wrote: > On Wed, 18 Apr 2012, Gerd Hoffmann wrote: > > > We talked with malc briefly on irc yesterday, and this is what > > > he gave me: > > > > > > http://cvs.savannah.gnu.org/viewvc/vgabios/vbe.c?root=vgabios&r1=1.47&r2=1.48 > > > > > > this is no

Re: [Qemu-devel] [PATCH 10/14] qbus: move get_dev_path to DeviceState

2012-04-18 Thread Anthony Liguori
On 04/18/2012 04:20 PM, Andreas Färber wrote: Am 18.04.2012 22:56, schrieb Anthony Liguori: It should have never been a bus method. Signed-off-by: Anthony Liguori --- To DeviceClass. Ack. Regards, Anthony Liguori /-F

Re: [Qemu-devel] [PATCH 09/14] qdev: connect some links and move type to object

2012-04-18 Thread Anthony Liguori
On 04/18/2012 04:25 PM, Andreas Färber wrote: Am 18.04.2012 22:56, schrieb Anthony Liguori: This makes sysbus part of the root hierarchy and all busses children of their respective parent DeviceState. Signed-off-by: Anthony Liguori --- hw/qdev.c| 11 +-- qom/object.c | 12 +++

[Qemu-devel] [PATCH] qom: quick and dirty QOM filesystem based on FUSE

2012-04-18 Thread Anthony Liguori
Here's an example session: anthony@titi:~/git/qemu/QMP$ QMP_SERVER=/tmp/server.sock ./qom-fuse tmp anthony@titi:~/git/qemu/QMP$ ls tmp machine sysbus type anthony@titi:~/git/qemu/QMP$ ls tmp/machine i440fx peripheral peripheral-anon type unattached anthony@titi:~/git/qemu/QMP$ ls tmp/machine

[Qemu-devel] [PATCH] pc-sysfw: make sure to call qdev_init

2012-04-18 Thread Anthony Liguori
We're not actually calling qdev_init for the pc-sysfw device. Since we create the canonical path during realize, this was causing an assert to trigger when attempting to read a link pointing to pc-sysfw. Signed-off-by: Anthony Liguori --- hw/pc_sysfw.c |8 1 files changed, 8 insert

[Qemu-devel] [PATCH 1/3] qemu-ga: improve recovery options for fsfreeze

2012-04-18 Thread Michael Roth
guest-fsfreeze-thaw relies on state information obtained from guest-fsfreeze-freeze to determine what filesystems to unfreeze. This is unreliable due to the fact that that state does not account for FIFREEZE being issued by other processes, or previous instances of qemu-ga. This means in certain si

[Qemu-devel] [PATCH 2/3] qemu-ga: add a whitelist for fsfreeze-safe commands

2012-04-18 Thread Michael Roth
Currently we rely on fsfreeze/thaw commands disabling/enabling logging then having other commands check whether logging is disabled to avoid executing if they aren't safe for running while a filesystem is frozen. Instead, have an explicit whitelist of fsfreeze-safe commands, and consolidate loggin

[Qemu-devel] [PATCH 3/3] qemu-ga: persist tracking of fsfreeze state via filesystem

2012-04-18 Thread Michael Roth
Currently, qemu-ga may die/get killed/go away for whatever reason after guest-fsfreeze-freeze has been issued, and before guest-fsfreeze-thaw has been issued. This means the only way to unfreeze the guest is via VNC/network/console access, but obtaining that access after-the-fact can often be very

Re: [Qemu-devel] [PATCH 1/2] qemu-ga: suspend: fix possible SIGCHLD during close() and g_free()

2012-04-18 Thread Michael Roth
On Wed, Apr 18, 2012 at 04:30:48PM -0300, Luiz Capitulino wrote: > A child created by bios_supports_mode() could terminate during the call > to close() or g_free(). This could cause the SIGCHLD signal to be > deliveried in the midle of their execution. Possible problems range from > resource leak t

[Qemu-devel] [PATCH 08/14] config-host.mak: rename docdir to qemu_docdir

2012-04-18 Thread Eduardo Habkost
Signed-off-by: Eduardo Habkost --- Makefile |4 ++-- configure |2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/Makefile b/Makefile index a9bc796..4f43793 100644 --- a/Makefile +++ b/Makefile @@ -268,8 +268,8 @@ BLOBS= endif install-doc: $(DOCS) - $(INSTALL

Re: [Qemu-devel] [Qemu-ppc] [PATCH] target-ppc: Init dcache and icache size for e500 user mode

2012-04-18 Thread Scott Wood
On 04/18/2012 04:24 PM, Alexander Graf wrote: > > On 18.04.2012, at 18:27, Scott Wood wrote: > >> On 04/18/2012 10:40 AM, Alexander Graf wrote: >>> On 04/15/2012 06:14 PM, Andreas Färber wrote: diff --git a/target-ppc/translate_init.c b/target-ppc/translate_init.c index 86a915c..ba4b84d

[Qemu-devel] [PATCH 11/14] create_config: remove *dir block

2012-04-18 Thread Eduardo Habkost
Now only the qemu_*dir variables will become #defines. The other directory names aren't used by the C code. That means the following #defines won't be available in C code anymore: - CONFIG_QEMU_BINDIR - CONFIG_QEMU_LIBDIR - CONFIG_QEMU_INCLUDEDIR - CONFIG_QEMU_MANDIR - CONFIG_QEMU_SYSCONFDIR

[Qemu-devel] [PATCH 11/14] qbus: move get_fw_dev_path to DeviceClass

2012-04-18 Thread Anthony Liguori
It should have never been a bus method. Signed-off-by: Anthony Liguori --- hw/ide/qdev.c | 33 + hw/isa-bus.c | 31 +++ hw/pci.c | 31 +++ hw/qdev.c | 10 +- hw/qdev.h |2 +-

[Qemu-devel] [PATCH 14/14] qbus: initialize in standard way

2012-04-18 Thread Anthony Liguori
Signed-off-by: Anthony Liguori --- hw/qdev.c | 84 + 1 files changed, 51 insertions(+), 33 deletions(-) diff --git a/hw/qdev.c b/hw/qdev.c index 42476e9..c37a701 100644 --- a/hw/qdev.c +++ b/hw/qdev.c @@ -433,40 +433,35 @@ DeviceState

[Qemu-devel] [PATCH v5 00/14] configure: --with-confsuffix option

2012-04-18 Thread Eduardo Habkost
This is just a rebase of v4, because I noticed that v4 doesn't apply cleanly against current git master anymore (and I will soon submit a series that will depend on this one). Changes v4 -> v5: - Rebase against latest qemu.git Changes v3 -> v4: - Rebase against latest qemu.git - Changed patch

Re: [Qemu-devel] [Qemu-ppc] [PATCHv3] PPC: Fix interrupt MSR value for classic exception models.

2012-04-18 Thread Alexander Graf
On 18.04.2012, at 22:31, Jakub Jermar wrote: > On 04/18/2012 05:30 PM, Alexander Graf wrote: >> On 04/13/2012 06:53 PM, Mark Cave-Ayland wrote: >>> On 11/04/12 02:08, David Gibson wrote: >>> >>> Hi David, >>> > Commit 41557447d30eeb944e42069513df13585f5e6c7f introduced a new > method of

[Qemu-devel] [PATCH 09/14] qdev: connect some links and move type to object

2012-04-18 Thread Anthony Liguori
This makes sysbus part of the root hierarchy and all busses children of their respective parent DeviceState. Signed-off-by: Anthony Liguori --- hw/qdev.c| 11 +-- qom/object.c | 12 2 files changed, 17 insertions(+), 6 deletions(-) diff --git a/hw/qdev.c b/hw/qdev.c

Re: [Qemu-devel] [PATCH 09/14] qdev: connect some links and move type to object

2012-04-18 Thread Andreas Färber
Am 18.04.2012 22:56, schrieb Anthony Liguori: > This makes sysbus part of the root hierarchy and all busses children of their > respective parent DeviceState. > > Signed-off-by: Anthony Liguori > --- > hw/qdev.c| 11 +-- > qom/object.c | 12 > 2 files changed, 17 ins

Re: [Qemu-devel] [Qemu-ppc] [PATCH] target-ppc: Init dcache and icache size for e500 user mode

2012-04-18 Thread Alexander Graf
On 18.04.2012, at 22:16, Andreas Färber wrote: > Am 18.04.2012 17:40, schrieb Alexander Graf: >> On 04/15/2012 06:14 PM, Andreas Färber wrote: >>> Am 13.04.2012 14:00, schrieb Meador Inge: On 04/13/2012 06:40 AM, Andreas Färber wrote: > Am 12.04.2012 19:24, schrieb Scott Wood:

[Qemu-devel] [PATCH 10/14] config-host.mak: reorder variables a bit

2012-04-18 Thread Eduardo Habkost
Cosmetic change. Signed-off-by: Eduardo Habkost --- configure |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 3a38d28..d995615 100755 --- a/configure +++ b/configure @@ -2997,10 +2997,10 @@ echo "bindir=$bindir" >> $config_host_mak echo "l

[Qemu-devel] [PATCH 05/14] qdev: use wrapper for qdev_get_path

2012-04-18 Thread Anthony Liguori
This makes it easier to remove it from BusInfo. Signed-off-by: Anthony Liguori --- exec.c|4 ++-- hw/qdev.c | 16 hw/qdev.h |2 ++ savevm.c | 12 ++-- 4 files changed, 26 insertions(+), 8 deletions(-) diff --git a/exec.c b/exec.c index 77d6866..4632b24

Re: [Qemu-devel] [Qemu-ppc] [PATCH] target-ppc: Init dcache and icache size for e500 user mode

2012-04-18 Thread Alexander Graf
On 18.04.2012, at 18:27, Scott Wood wrote: > On 04/18/2012 10:40 AM, Alexander Graf wrote: >> On 04/15/2012 06:14 PM, Andreas Färber wrote: >>> diff --git a/target-ppc/translate_init.c b/target-ppc/translate_init.c >>> index 86a915c..ba4b84d 100644 >>> --- a/target-ppc/translate_init.c >>> +++ b/

[Qemu-devel] [PATCH 07/14] qdev: fix info qtree/qdm

2012-04-18 Thread Anthony Liguori
Don't rely on bus_info. I took a little liberty in the last commit as it would cause info qtree/info qdm to not show any useful information. But since this is not considered a supported interface, breaking it across a single commit seems okay. This commit makes info qtree/qdm work again. Signed

Re: [Qemu-devel] [PATCH 13/36] vmstate: port ppc cpu

2012-04-18 Thread Alexander Graf
On 11.04.2012, at 13:56, Andreas Färber wrote: > Am 19.03.2012 23:57, schrieb Juan Quintela: >> Added sdr1_vmstate because storing the value requires calling >> ppc_store_sdr1(). >> The position when the function is called also changes (I think it is save). >> >> Signed-off-by: Juan Quintela >

Re: [Qemu-devel] [PATCH 10/14] qbus: move get_dev_path to DeviceState

2012-04-18 Thread Andreas Färber
Am 18.04.2012 22:56, schrieb Anthony Liguori: > It should have never been a bus method. > > Signed-off-by: Anthony Liguori > --- To DeviceClass. /-F -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

[Qemu-devel] [PATCH 04/14] qdev: don't allow globals to be set by bus name

2012-04-18 Thread Anthony Liguori
This is technically a compatibility breaker. However: 1) libvirt does not rely on this (it always uses the driver name) 2) This behavior isn't actually documented anywhere (the docs just say driver). 3) I suspect there are less than three people on earth that even know this is possible (minu

[Qemu-devel] [PATCH 03/14] qdev: add qdev_add_properties

2012-04-18 Thread Anthony Liguori
This allows a base class to easily add properties. Signed-off-by: Anthony Liguori --- hw/qdev.c | 25 - hw/qdev.h |2 ++ 2 files changed, 14 insertions(+), 13 deletions(-) diff --git a/hw/qdev.c b/hw/qdev.c index 83a1db6..c8dda31 100644 --- a/hw/qdev.c +++ b/hw/qde

[Qemu-devel] [PATCH 10/14] qbus: move get_dev_path to DeviceState

2012-04-18 Thread Anthony Liguori
It should have never been a bus method. Signed-off-by: Anthony Liguori --- hw/pci.c | 75 - hw/qdev.c | 11 ++-- hw/qdev.h |2 +- hw/scsi-bus.c | 10 hw/usb/bus.c | 41 +++

[Qemu-devel] [PATCH 13/14] configure: change meaning of --datadir to Autoconf convention

2012-04-18 Thread Eduardo Habkost
Autoconf uses --datadir for the /usr/share directory, not the program-specific subdirectory inside /usr/share. This changes configure to match autoconf behavior. Note that this will break compatibility with existing build scripts. Signed-off-by: Eduardo Habkost --- configure |9 + 1

[Qemu-devel] [PATCH 13/14] qbus: make child devices links

2012-04-18 Thread Anthony Liguori
Make qbus children show up as link<> properties. There is no stable addressing for qbus children so we use an unstable naming convention. This is okay in QOM though because the composition name is expected to be what's stable. Signed-off-by: Anthony Liguori --- hw/acpi_piix4.c | 10

[Qemu-devel] [PATCH 08/14] qdev: convert busses to QEMU Object Model

2012-04-18 Thread Anthony Liguori
This is far less interesting than it sounds. We simply add an Object to each BusInfo and then register the types appropriately. Most of the interesting refactoring will follow in the next patches. Since we're changing fundamental type names (BusInfo -> BusClass), it all needs to convert at once.

[Qemu-devel] [PATCH 14/14] configure: add --with-confsuffix option

2012-04-18 Thread Eduardo Habkost
This will allow the user to make Qemu use a different subdirectory name inside $datadir and $sysconfdir, instead of "/qemu". Signed-off-by: Eduardo Habkost --- configure |9 ++--- 1 files changed, 6 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 4c0bb2f..6c12557

[Qemu-devel] [PATCH 3/5] move list of default config files to an array

2012-04-18 Thread Eduardo Habkost
More files will be added to the list, with additional attributes, later. Signed-off-by: Eduardo Habkost --- arch_init.c | 25 - 1 files changed, 16 insertions(+), 9 deletions(-) diff --git a/arch_init.c b/arch_init.c index 152cbbb..62332e9 100644 --- a/arch_init.c +++

Re: [Qemu-devel] [PATCH] mips: Fix BC1ANY[24]F instructions

2012-04-18 Thread Andreas Färber
Am 26.11.2011 14:37, schrieb Richard Sandiford: > There's some dodgy application of De Morgan's law in the emulation > of the MIPS BC1ANY[24]F instructions: they end up branching only > if all CCs are false, rather than if one CC is. > > Tested on mips64-linux-gnu, where it fixes the GCC MIPS3D te

[Qemu-devel] [PATCH 12/14] qbus: move print_dev to DeviceClass

2012-04-18 Thread Anthony Liguori
It should have never been a bus method. Signed-off-by: Anthony Liguori --- hw/isa-bus.c | 37 -- hw/pci.c | 79 +++ hw/qdev-monitor.c | 10 +++--- hw/qdev.h |3 +- hw/sysbus.c

[Qemu-devel] [PATCH 02/14] object: add object_property_foreach

2012-04-18 Thread Anthony Liguori
Provide a mechanism to walk through each property for an object. Signed-off-by: Anthony Liguori --- include/qemu/object.h | 26 ++ qom/object.c | 10 ++ 2 files changed, 36 insertions(+), 0 deletions(-) diff --git a/include/qemu/object.h b/include/qe

[Qemu-devel] [PATCH 01/14] qdev: fix adding of ptr properties

2012-04-18 Thread Anthony Liguori
ptr properties have neither a get/set or a print/parse which means that when they're added they aren't treated as static or legacy properties. Just assume properties like this are legacy properties and treat them as such. Signed-off-by: Anthony Liguori --- hw/qdev.c |5 - 1 files change

[Qemu-devel] [PATCH 00/14] qom: convert busses to QOM

2012-04-18 Thread Anthony Liguori
This is the last of the core QOM series. This series converts busses to QOM using a model where busses are proper objects that inherit from Object directly. Devices have a has-a relationship with any bus they implement. This series also creates link associated with the device/bus relationships.

Re: [Qemu-devel] [PATCH 2/3] qtest: enable qtest for most targets

2012-04-18 Thread Blue Swirl
On Wed, Apr 18, 2012 at 20:51, Andreas Färber wrote: > Am 18.04.2012 22:35, schrieb Anthony Liguori: >> On 04/18/2012 03:28 PM, Blue Swirl wrote: >>> On Tue, Apr 17, 2012 at 21:33, Anthony Liguori >>> wrote: Kernel loading is a hack.  I'll go out on a limb and say that most no

[Qemu-devel] [PATCH 4/5] implement -no-user-config command-line option

2012-04-18 Thread Eduardo Habkost
Signed-off-by: Eduardo Habkost --- arch_init.c | 10 +++--- arch_init.h |2 +- qemu-options.hx | 16 +--- vl.c|6 +- 4 files changed, 26 insertions(+), 8 deletions(-) diff --git a/arch_init.c b/arch_init.c index 62332e9..c5fc00f 100644 --- a/ar

Re: [Qemu-devel] [PATCH 2/3] qtest: enable qtest for most targets

2012-04-18 Thread Andreas Färber
Am 18.04.2012 22:35, schrieb Anthony Liguori: > On 04/18/2012 03:28 PM, Blue Swirl wrote: >> On Tue, Apr 17, 2012 at 21:33, Anthony Liguori >> wrote: >>> >>> >>> Kernel loading is a hack. I'll go out on a limb and say that most >>> non-x86 >>> boards are doing it completely wrong. Messing aroun

Re: [Qemu-devel] qemu physical address

2012-04-18 Thread Blue Swirl
On Wed, Apr 18, 2012 at 01:28, Xin Tong wrote: > I am reading how qemu refill TLB working. > > target-i386/helper.c > >    pte = pte & env->a20_mask; > >    /* Even if 4MB pages, we map only one 4KB page in the cache to >       avoid filling it too fast */ >    page_offset = (addr & TARGET_PAGE_MA

[Qemu-devel] [PATCH 12/14] create_config: simplify prefix=* block, remove CONFIG_QEMU_PREFIX

2012-04-18 Thread Eduardo Habkost
As now that block handles only the prefix variable, the code can be much simpler. This also removes the CONFIG_QEMU_PREFIX define as it is not used by any C code. Signed-off-by: Eduardo Habkost --- scripts/create_config |7 +-- 1 files changed, 1 insertions(+), 6 deletions(-) diff --git

[Qemu-devel] [PATCH 04/14] create_config: separate section for qemu_*dir variables (v2)

2012-04-18 Thread Eduardo Habkost
The generic *dir section will eventually go away and be replaced with qemu_* section. By now, both sections will be kept, while the variables get renamed on config-host.mak. With this patch, a XXXdir variable will become a CONFIG_QEMU_XXXDIR define, and a qemu_XXXdir variable will become CONFIG_QE

Re: [Qemu-devel] [PATCH 2/2] qemu-ga: suspend: handle EINTR

2012-04-18 Thread Luiz Capitulino
On Wed, 18 Apr 2012 21:08:28 +0100 Peter Maydell wrote: > On 18 April 2012 20:30, Luiz Capitulino wrote: > > The read() call in bios_supports_mode() can fail with EINTR if a child > > terminates during the call. Handle it. > > > > Signed-off-by: Luiz Capitulino > > --- > >  qga/commands-posix.c

[Qemu-devel] [PATCH 05/14] config-host.mak: rename datadir to qemu_datadir

2012-04-18 Thread Eduardo Habkost
Autoconf concept of "datadir" is supposed to be "$prefix/share", not "$prefix/share/PACKAGE", so using datadir for the Qemu-specific directory is confusing. The current C code that uses CONFIG_QEMU_DATADIR should be safe, as now create_config generates the same #define name (CONFIG_QEMU_DATADIR) f

Re: [Qemu-devel] [PATCH 2/3] qtest: enable qtest for most targets

2012-04-18 Thread Blue Swirl
On Wed, Apr 18, 2012 at 20:35, Anthony Liguori wrote: > On 04/18/2012 03:28 PM, Blue Swirl wrote: >> >> On Tue, Apr 17, 2012 at 21:33, Anthony Liguori >>  wrote: >>> >>> On 04/17/2012 03:59 PM, Peter Maydell wrote: On 17 April 2012 21:43, Blue Swirl    wrote: > > > On Tu

[Qemu-devel] [PATCH 5/5] move CPU definitions to /usr/share/qemu/cpus-x86_64.conf

2012-04-18 Thread Eduardo Habkost
Signed-off-by: Eduardo Habkost --- Makefile | 12 +++- arch_init.c |1 + sysconfigs/target/cpus-x86_64.conf | 128 ++ sysconfigs/target/target-x86_64.conf | 128 -- 4 fil

Re: [Qemu-devel] [PATCH 2/3] qtest: enable qtest for most targets

2012-04-18 Thread Blue Swirl
On Tue, Apr 17, 2012 at 20:59, Peter Maydell wrote: > On 17 April 2012 21:43, Blue Swirl wrote: >> On Tue, Apr 17, 2012 at 20:31, Peter Maydell >> wrote: >>> Well, it could. But we should make that decision based on whether it >>> makes sense and has a use case for actual users of the board, no

Re: [Qemu-devel] [PATCH 2/3] qtest: enable qtest for most targets

2012-04-18 Thread Anthony Liguori
On 04/18/2012 03:28 PM, Blue Swirl wrote: On Tue, Apr 17, 2012 at 21:33, Anthony Liguori wrote: On 04/17/2012 03:59 PM, Peter Maydell wrote: On 17 April 2012 21:43, Blue Swirlwrote: On Tue, Apr 17, 2012 at 20:31, Peter Maydell wrote: Well, it could. But we should make that decision

[Qemu-devel] [PATCH 02/14] configure: rename $docdir to $qemu_docdir

2012-04-18 Thread Eduardo Habkost
The autoconf "docdir" variable is used for the program-specific subdirectory, but anyway it's better to make it clear that the variable is used for the qemu-specific subdirectory path. Signed-off-by: Eduardo Habkost --- configure |8 1 files changed, 4 insertions(+), 4 deletions(-)

Re: [Qemu-devel] [Qemu-ppc] [PATCHv3] PPC: Fix interrupt MSR value for classic exception models.

2012-04-18 Thread Jakub Jermar
On 04/18/2012 05:30 PM, Alexander Graf wrote: > On 04/13/2012 06:53 PM, Mark Cave-Ayland wrote: >> On 11/04/12 02:08, David Gibson wrote: >> >> Hi David, >> Commit 41557447d30eeb944e42069513df13585f5e6c7f introduced a new method of calculating the MSR for the interrupt context. Howev

Re: [Qemu-devel] [Qemu-ppc] [PATCHv3] PPC: Fix interrupt MSR value for classic exception models.

2012-04-18 Thread Jakub Jermar
On 04/18/2012 05:30 PM, Alexander Graf wrote: > On 04/13/2012 06:53 PM, Mark Cave-Ayland wrote: >> On 11/04/12 02:08, David Gibson wrote: >> >> Hi David, >> Commit 41557447d30eeb944e42069513df13585f5e6c7f introduced a new method of calculating the MSR for the interrupt context. Howev

Re: [Qemu-devel] [RFC PATCH 2/3] iommu: Create basic group infrastructure and update AMD-Vi & Intel VT-d

2012-04-18 Thread Alex Williamson
On Wed, 2012-04-18 at 21:55 +1000, David Gibson wrote: > On Mon, Apr 02, 2012 at 03:14:46PM -0600, Alex Williamson wrote: > > IOMMU groups define the minimum granularity of the IOMMU. We therefore > > create groups using a dma_dev which is the effective requestor ID for > > the entire group. Addi

Re: [Qemu-devel] [PATCH 2/3] qtest: enable qtest for most targets

2012-04-18 Thread Blue Swirl
On Tue, Apr 17, 2012 at 21:33, Anthony Liguori wrote: > On 04/17/2012 03:59 PM, Peter Maydell wrote: >> >> On 17 April 2012 21:43, Blue Swirl  wrote: >>> >>> On Tue, Apr 17, 2012 at 20:31, Peter Maydell >>>  wrote: Well, it could. But we should make that decision based on whether it

[Qemu-devel] [PATCH 06/14] config-host.mak: rename confdir to qemu_confdir

2012-04-18 Thread Eduardo Habkost
$confdir is a confusing name, as it's not clear if it's "the system config dir" or "the Qemus-specific directory inside the config dir". $qemu_confdir makes it more clear. The current C code that uses CONFIG_QEMU_CONFDIR should be safe, as now create_config generates the same #define name (CONFIG_

[Qemu-devel] [PATCH 09/14] config-host.mak: remove CONFIG_QEMU_SHAREDIR

2012-04-18 Thread Eduardo Habkost
There is no code using that variable according to 'git grep', so kill it. Signed-off-by: Eduardo Habkost --- configure |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/configure b/configure index 33b7a25..3a38d28 100755 --- a/configure +++ b/configure @@ -3002,7 +3002,6 @

Re: [Qemu-devel] qemu softmmu inlined lookup sequence

2012-04-18 Thread Blue Swirl
On Wed, Apr 18, 2012 at 02:49, 陳韋任 wrote: > On Tue, Apr 17, 2012 at 08:17:09PM +, Blue Swirl wrote: >> On Tue, Apr 17, 2012 at 05:40, Xin Tong wrote: >> > that is possible. but if that is the case, why not split the tlb >> > walking and the tlb fill ? can anyone please confirm ? >> >> I sent

Re: [Qemu-devel] [Qemu-ppc] [PATCH] target-ppc: Init dcache and icache size for e500 user mode

2012-04-18 Thread Andreas Färber
Am 18.04.2012 17:40, schrieb Alexander Graf: > On 04/15/2012 06:14 PM, Andreas Färber wrote: >> Am 13.04.2012 14:00, schrieb Meador Inge: >>> On 04/13/2012 06:40 AM, Andreas Färber wrote: >>> Am 12.04.2012 19:24, schrieb Scott Wood: > On 04/12/2012 11:59 AM, Andreas Färber wrote: >> Am

[Qemu-devel] [PATCH 1/2] qemu-ga: suspend: fix possible SIGCHLD during close() and g_free()

2012-04-18 Thread Luiz Capitulino
A child created by bios_supports_mode() could terminate during the call to close() or g_free(). This could cause the SIGCHLD signal to be deliveried in the midle of their execution. Possible problems range from resource leak to segfault. Fix that by blocking SIGCHLD during those calls. Also, tries

Re: [Qemu-devel] [PATCH 01/14] configure: rename $datadir to $qemu_datadir

2012-04-18 Thread Eduardo Habkost
On Wed, Apr 18, 2012 at 01:57:51PM -0600, Eric Blake wrote: > On 04/18/2012 01:55 PM, Eduardo Habkost wrote: > > The variable is used for the qemu-specific directory and has a different > > meaning of the autoconf "datadir" variable (that's used for the > > $prefix/share directory, not for $prefix/

Re: [Qemu-devel] [PATCH 2/2] qemu-ga: suspend: handle EINTR

2012-04-18 Thread Peter Maydell
On 18 April 2012 20:30, Luiz Capitulino wrote: > The read() call in bios_supports_mode() can fail with EINTR if a child > terminates during the call. Handle it. > > Signed-off-by: Luiz Capitulino > --- >  qga/commands-posix.c |   12 >  1 file changed, 8 insertions(+), 4 deletions(-)

Re: [Qemu-devel] [RFC PATCH 1/3] iommu: Introduce iommu_group

2012-04-18 Thread Alex Williamson
On Wed, 2012-04-18 at 19:58 +1000, David Gibson wrote: > On Mon, Apr 02, 2012 at 03:14:40PM -0600, Alex Williamson wrote: > > IOMMUs often do not have visibility of individual devices in the > > system. Due to IOMMU design, bus topology, or device quirks, we > > can often only identify groups of d

[Qemu-devel] [PATCH 0/5] -no-user-config option, move CPU models to /usr/share

2012-04-18 Thread Eduardo Habkost
This is the first try of the new -no-user-config option. Patches 1 to 3 just move some code around, patch 4 just adds the new option without adding any new config file. Patch 5 finally creates a /usr/share/qemu /cpus-x86_64.conf file, with the CPU models we currently have on Qemu. Reference to pr

[Qemu-devel] [PATCH 1/5] move code to read default config files to a separate function

2012-04-18 Thread Eduardo Habkost
Function added to arch_init.c because it depends on arch-specific settings. Signed-off-by: Eduardo Habkost --- arch_init.c | 18 ++ arch_init.h |4 vl.c| 10 ++ 3 files changed, 24 insertions(+), 8 deletions(-) diff --git a/arch_init.c b/arch_init.c

[Qemu-devel] [PATCH 2/5] eliminate arch_config_name variable

2012-04-18 Thread Eduardo Habkost
Not needed anymore, as the code that uses the variable is already inside arch_init.c. Signed-off-by: Eduardo Habkost --- arch_init.c |3 +-- arch_init.h |2 -- 2 files changed, 1 insertions(+), 4 deletions(-) diff --git a/arch_init.c b/arch_init.c index 4008115..152cbbb 100644 --- a/arc

Re: [Qemu-devel] [PATCH 01/14] configure: rename $datadir to $qemu_datadir

2012-04-18 Thread Eric Blake
On 04/18/2012 01:55 PM, Eduardo Habkost wrote: > The variable is used for the qemu-specific directory and has a different > meaning of the autoconf "datadir" variable (that's used for the > $prefix/share directory, not for $prefix/share/PACKAGE). Automake uses the name $pkgdatadir for $prefix/shar

[Qemu-devel] [PATCH 07/14] Makefile: use $(qemu_confdir) instead of $(sysconfdir)/qemu

2012-04-18 Thread Eduardo Habkost
Instead of hardcoding the directory suffix, use the qemu-specific directory variable. Signed-off-by: Eduardo Habkost --- Makefile |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makefile index c8e4b53..a9bc796 100644 --- a/Makefile +++ b/Makefile @@ -281,8

[Qemu-devel] [PATCH 03/14] configure: rename $confdir to $qemu_confdir

2012-04-18 Thread Eduardo Habkost
There's no "confdir" variable on Autoconf, but it's good to make it clear that it's a variable for the Qemu-specific subdirectory inside $sysconfdir. Signed-off-by: Eduardo Habkost --- configure |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/configure b/configure ind

  1   2   3   >