Re: [Qemu-devel] [PATCH 0/2] sparc64: cleanups

2010-05-26 Thread Blue Swirl
Thanks, applied both. On Tue, May 25, 2010 at 12:08 PM, Igor V. Kovalenko igor.v.kovale...@gmail.com wrote: - rename sun4u cpu to Ultrasparc IIi - cleanup pci bridge map (requires openbios change) v0-v1: split out rename of sun4u cpu to separate patch --- Igor V. Kovalenko (2):      

Re: [Qemu-devel] [PATCH] Default to cache=writeback

2010-05-26 Thread Stefan Hajnoczi
On Wed, May 26, 2010 at 7:13 PM, Alexander Graf ag...@suse.de wrote: In the previous discussion Anthony brought up the fact that cache=writeback is safe enough considering data integrity. If so, I don't see a reason not to use it as default, as it speeds up things a lot. cache=writeback is not

Re: [Qemu-devel] Re: [RFT][PATCH 07/15] qemu_irq: Add IRQ handlers with delivery feedback

2010-05-26 Thread Blue Swirl
On Tue, May 25, 2010 at 9:44 PM, Jan Kiszka jan.kis...@web.de wrote: Anthony Liguori wrote: On 05/25/2010 02:09 PM, Blue Swirl wrote: On Mon, May 24, 2010 at 8:13 PM, Jan Kiszkajan.kis...@web.de  wrote: From: Jan Kiszkajan.kis...@siemens.com This allows to communicate potential IRQ

[Qemu-devel] Re: [PATCH 6/8] enable event_notifier to use pipes

2010-05-26 Thread Paolo Bonzini
On 05/26/2010 08:19 PM, Richard Henderson wrote: On 05/26/2010 07:09 AM, Paolo Bonzini wrote: -#ifdef CONFIG_EVENTFD -#includesys/eventfd.h -#endif Is there a reason not to use eventfd if it is available? qemu_eventfd chooses between eventfd or pipes. The reads and writes are done so that

Re: [Qemu-devel] Re: hw/serial.c: Xmit fifo never used

2010-05-26 Thread Frank Mehnert
On Wednesday 26 May 2010, Stefano Stabellini wrote: I think the patch is correct. serial: fixed bug which prevented the use of the xmit fifo Signed-off-by: Frank Mehnert frank.mehn...@sun.com --- hw/serial.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/serial.c

[Qemu-devel] [PATCHv2-RFC 0/2] virtio: put last seen used index into ring itself

2010-05-26 Thread Michael S. Tsirkin
Here's a rewrite of the original patch with a new layout. I haven't tested it yet so no idea how this performs, but I think this addresses the cache bounce issue raised by Avi. Posting for early flames/comments. Generally, the Host end of the virtio ring doesn't need to see where Guest is up to

Re: [Qemu-devel] Re: [RFT][PATCH 07/15] qemu_irq: Add IRQ handlers with delivery feedback

2010-05-26 Thread Blue Swirl
On Tue, May 25, 2010 at 8:16 PM, Anthony Liguori anth...@codemonkey.ws wrote: On 05/25/2010 02:09 PM, Blue Swirl wrote: On Mon, May 24, 2010 at 8:13 PM, Jan Kiszkajan.kis...@web.de  wrote: From: Jan Kiszkajan.kis...@siemens.com This allows to communicate potential IRQ coalescing during

Re: [Qemu-devel] Re: [RFT][PATCH 07/15] qemu_irq: Add IRQ handlers with delivery feedback

2010-05-26 Thread Jan Kiszka
Blue Swirl wrote: On Tue, May 25, 2010 at 9:44 PM, Jan Kiszka jan.kis...@web.de wrote: Anthony Liguori wrote: On 05/25/2010 02:09 PM, Blue Swirl wrote: On Mon, May 24, 2010 at 8:13 PM, Jan Kiszkajan.kis...@web.de wrote: From: Jan Kiszkajan.kis...@siemens.com This allows to communicate

[Qemu-devel] [PATCHv2-RFC 1/2] virtio: support layout with avail ring before idx

2010-05-26 Thread Michael S. Tsirkin
This adds an (unused) option to put available ring before control (avail index, flags). This avoids cache line sharing between control and ring, and also makes it possible to extend avail control without incurring extra cache misses. Signed-off-by: Michael S. Tsirkin m...@redhat.com ---

Re: [Qemu-devel] Re: [RFT][PATCH 07/15] qemu_irq: Add IRQ handlers with delivery feedback

2010-05-26 Thread Blue Swirl
On Wed, May 26, 2010 at 8:08 AM, Gleb Natapov g...@redhat.com wrote: On Tue, May 25, 2010 at 11:44:50PM +0200, Jan Kiszka wrote: I think the real solution to coalescing is put the logic inside one device, in this case APIC because it has the information about irq delivery. APIC could

Re: [Qemu-devel] [PATCH -V3 2/7] virtio-9p: Rearrange fileop structures

2010-05-26 Thread Venkateswararao Jujjuri (JV)
Sripathi Kodi wrote: On Fri, 21 May 2010 14:26:05 -0700 Venkateswararao Jujjuri (JV) jv...@linux.vnet.ibm.com wrote: Hi JV, While I agree that this patch is nice to have, why is this part of the security model patchset? Is it required to implement the models? Initially I had a grand plan

Re: [Qemu-devel] Re: [RFT][PATCH 07/15] qemu_irq: Add IRQ handlers with delivery feedback

2010-05-26 Thread Blue Swirl
On Wed, May 26, 2010 at 8:09 PM, Jan Kiszka jan.kis...@web.de wrote: Blue Swirl wrote: On Tue, May 25, 2010 at 9:44 PM, Jan Kiszka jan.kis...@web.de wrote: Anthony Liguori wrote: On 05/25/2010 02:09 PM, Blue Swirl wrote: On Mon, May 24, 2010 at 8:13 PM, Jan Kiszkajan.kis...@web.de  wrote:

Re: [Qemu-devel] [PATCH 1/2] virtio-9p: make virtio-9p available to all POSIX systems

2010-05-26 Thread Blue Swirl
On Wed, May 26, 2010 at 6:02 PM, Venkateswararao Jujjuri (JV) jv...@linux.vnet.ibm.com wrote: Blue Swirl wrote: On Mon, May 24, 2010 at 8:46 PM, Venkateswararao Jujjuri (JV) jv...@linux.vnet.ibm.com wrote: Blue Swirl wrote: Field d_off in struct dirent is Linux specific. Signed-off-by: Blue

Re: [Qemu-devel] RFC: ehci - uhci handoff suggestions

2010-05-26 Thread Johannes Stezenbach
On Wed, May 26, 2010 at 08:00:33AM -0600, David S. Ahern wrote: On 05/26/2010 07:23 AM, Kevin Wolf wrote: Am 26.05.2010 15:06, schrieb David S. Ahern: My understanding is that the port routing happens internally to the host controller based on device speed - section 4.2 (pag 64) of:

[Qemu-devel] [PATCH] move net_handle_fd_param() into a common utility function

2010-05-26 Thread Alex Williamson
Move to monitor.c:monitor_handle_fd_param() as a common helper. Signed-off-by: Alex Williamson alex.william...@redhat.com --- I'd like to use this for the proposed device assignment configfd parameter since there's nothing net specific about it. monitor.c| 17 + monitor.h

Re: [Qemu-devel] Re: [RFT][PATCH 07/15] qemu_irq: Add IRQ handlers with delivery feedback

2010-05-26 Thread Jan Kiszka
Blue Swirl wrote: I think the real solution to coalescing is put the logic inside one device, in this case APIC because it has the information about irq delivery. APIC could monitor incoming RTC irqs for frequency information and whether they get delivered or not. If not, an internal timer is

[Qemu-devel] Re: [PATCH] Make cache=unsafe the default for -snapshot

2010-05-26 Thread Aurelien Jarno
On Wed, May 26, 2010 at 09:04:32PM +0200, Alexander Graf wrote: When using -snapshot we don't care about data integrity of the cow file at all, so let's disable flushing there and squeeze out the last drop of performance we could possibly get. Signed-off-by: Alexander Graf ag...@suse.de

[Qemu-devel] [PATCH -V4 4/7] virtio-9p: Implement Security model for mknod related files

2010-05-26 Thread Venkateswararao Jujjuri (JV)
In the mapped security model all the special files are created as regular files on the fileserver and appropriate mode bits are added to the extended attributes. These extended attributes are used to present this file as special file to the client. Signed-off-by: Venkateswararao Jujjuri

[Qemu-devel] [PATCH -V4 5/7] virtio-9p: Implemented security model for symlink and link.

2010-05-26 Thread Venkateswararao Jujjuri (JV)
Signed-off-by: Venkateswararao Jujjuri jv...@linux.vnet.ibm.com --- hw/file-op-9p.h |4 +- hw/virtio-9p-local.c | 98 - hw/virtio-9p.c | 24 + 3 files changed, 99 insertions(+), 27 deletions(-) diff --git

[Qemu-devel] [PATCH -V4 6/7] virtio-9p: Implemented Security model for lstat and fstat

2010-05-26 Thread Venkateswararao Jujjuri (JV)
Signed-off-by: Venkateswararao Jujjuri jv...@linux.vnet.ibm.com --- hw/virtio-9p-local.c | 62 ++--- 1 files changed, 58 insertions(+), 4 deletions(-) diff --git a/hw/virtio-9p-local.c b/hw/virtio-9p-local.c index 395a33f..11f3650 100644 ---

Re: [Qemu-devel] Re: [RFT][PATCH 07/15] qemu_irq: Add IRQ handlers with delivery feedback

2010-05-26 Thread Paul Brook
At the other extreme, would it be possible to make the educated guests aware of the virtualization also in clock aspect: virtio-clock? The guest doesn't even need to be aware of virtualization. It just needs to be able to accommodate the lack of guaranteed realtime behavior. The fundamental

[Qemu-devel] [PATCH -V4 1/7] virtio-9p: Introduces an option to specify the security model.

2010-05-26 Thread Venkateswararao Jujjuri (JV)
The new option is: -fsdev fstype,id=myid,path=/share_path/,security_model=[mapped|passthrough] -virtfs fstype,path=/share_path/,security_model=[mapped|passthrough],mnt_tag=tag In the case of mapped security model, files are created with QEMU user credentials and the client-user's credentials are

[Qemu-devel] [PATCH -V4 2/7] virtio-9p: Rearrange fileop structures

2010-05-26 Thread Venkateswararao Jujjuri (JV)
Signed-off-by: Venkateswararao Jujjuri jv...@linux.vnet.ibm.com --- hw/virtio-9p.c | 185 ++-- hw/virtio-9p.h | 92 2 files changed, 138 insertions(+), 139 deletions(-) diff --git a/hw/virtio-9p.c

[Qemu-devel] [Bug 586175] [NEW] Windows XP/2003 doesn't boot

2010-05-26 Thread tekditt
Public bug reported: Hello everyone, my qemu doesn't boot any Windows XP/2003 installations if I try to boot the image. If I boot the install cd first, it's boot manager counts down and triggers the boot on it's own. That's kinda stupid. I'm using libvirt, but even by a simple qemu-kvm

[Qemu-devel] Re: [PATCH-V4 0/7] virtio-9p:Introducing security model for VirtFS

2010-05-26 Thread Andy Lutomirski
Venkateswararao Jujjuri (JV) wrote: This patch series introduces the security model for VirtFS. Brief description of this patch series: It introduces two type of security models for VirtFS. They are: mapped and passthrough. The following is common to both security models. * Client's VFS

[Qemu-devel] [PATCH] pci: clean up of pci_set_default_subsystem_id().

2010-05-26 Thread Isaku Yamahata
Use pci accessor function. don't return value because it always return 0 and the caller doesn't check the return value. Signed-off-by: Isaku Yamahata yamah...@valinux.co.jp --- hw/pci.c | 12 +--- 1 files changed, 5 insertions(+), 7 deletions(-) diff --git a/hw/pci.c b/hw/pci.c index

[Qemu-devel] [PATCH] pci-hotplug: make them aware of pci domain.

2010-05-26 Thread Isaku Yamahata
add helper function which converts root bus to pci domain. make functions which ignored domain aware of pci domain. Signed-off-by: Isaku Yamahata yamah...@valinux.co.jp --- hw/pci-hotplug.c |7 --- hw/pci.c | 24 +++- hw/pci.h |2 ++ 3 files

[Qemu-devel] [PATCH] pci: fix pci_default_read_config().

2010-05-26 Thread Isaku Yamahata
address and config_size are both unsigned. So check which is bigger before minus operation. Otherwise the result of minus can be unexpected big value. Signed-off-by: Isaku Yamahata yamah...@valinux.co.jp --- hw/pci.c |9 +++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git

<    1   2