[Qemu-devel] RE: KVM call agenda for Sept 21

2010-09-20 Thread Nakajima, Jun
Avi Kivity wrote on Mon, 20 Sep 2010 at 09:50:55: > On 09/20/2010 06:44 PM, Chris Wright wrote: >> Please send in any agenda items you are interested in covering. >> > nested vmx: the resurrection. Nice to see it progressing again, but > there's still a lot of ground to cover. Perhaps we can

[Qemu-devel] Re: Caching modes

2010-09-20 Thread Anthony Liguori
On 09/20/2010 06:17 PM, Christoph Hellwig wrote: On Mon, Sep 20, 2010 at 03:11:31PM -0500, Anthony Liguori wrote: All read and write requests SHOULD avoid any type of caching in the host. Any write request MUST complete after the next level of storage reports that the write request has comp

[Qemu-devel] Re: Caching modes

2010-09-20 Thread Christoph Hellwig
On Mon, Sep 20, 2010 at 03:11:31PM -0500, Anthony Liguori wrote: > >>All read and write requests SHOULD avoid any type of caching in the > >>host. Any write request MUST complete after the next level of storage > >>reports that the write request has completed. A flush from the guest > >>MUST comp

Re: [Qemu-devel] Re: Problem with virtio

2010-09-20 Thread Frans de Boer
On 09/20/2010 11:19 PM, Vadim Rozenfeld wrote: > On Mon, 2010-09-20 at 22:22 +0200, Frans de Boer wrote: > >> On 09/20/2010 07:46 PM, Michael S. Tsirkin wrote: >> >>> On Sun, Sep 19, 2010 at 11:13:16PM +0200, Frans de Boer wrote: >>> >>> On 09/19/2010 05:53 PM, Michael S. Tsi

[Qemu-devel] [PATCH 1/2] mips_fulong2e: Fix format strings

2010-09-20 Thread Stefan Weil
Fix two compiler warnings (when format attribute is applied) and one error (missing %) in format strings. Cc: Aurelien Jarno Cc: Blue Swirl Signed-off-by: Stefan Weil --- hw/mips_fulong2e.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/hw/mips_fulong2e.c b/hw/mi

[Qemu-devel] Re: [PATCH] net: delay peer host device delete

2010-09-20 Thread Michael S. Tsirkin
On Mon, Sep 20, 2010 at 03:38:36PM -0500, Anthony Liguori wrote: > On 09/20/2010 03:27 PM, Michael S. Tsirkin wrote: > >On Mon, Sep 20, 2010 at 03:20:59PM -0500, Anthony Liguori wrote: > >>On 09/20/2010 02:44 PM, Michael S. Tsirkin wrote: > I think the only workable approach that doesn't involv

Re: [Qemu-devel] Re: Problem with virtio

2010-09-20 Thread Vadim Rozenfeld
On Mon, 2010-09-20 at 22:22 +0200, Frans de Boer wrote: > On 09/20/2010 07:46 PM, Michael S. Tsirkin wrote: > > On Sun, Sep 19, 2010 at 11:13:16PM +0200, Frans de Boer wrote: > > > >> On 09/19/2010 05:53 PM, Michael S. Tsirkin wrote: > >> > >> On Sat, Sep 18, 2010 at 11:13:12PM +0200, Frans

[Qemu-devel] Re: [PATCH] net: delay peer host device delete

2010-09-20 Thread Michael S. Tsirkin
On Mon, Sep 20, 2010 at 02:22:18PM -0500, Anthony Liguori wrote: > On 09/20/2010 01:59 PM, Michael S. Tsirkin wrote: > >>You can also initiate the unplug from the OS without the ACPI event > >>ever happening. I suspect that in our current implementation, that > >>means that we'll automatically del

[Qemu-devel] Re: [PATCH] net: delay peer host device delete

2010-09-20 Thread Anthony Liguori
On 09/20/2010 03:37 PM, Michael S. Tsirkin wrote: On Mon, Sep 20, 2010 at 03:38:36PM -0500, Anthony Liguori wrote: On 09/20/2010 03:27 PM, Michael S. Tsirkin wrote: On Mon, Sep 20, 2010 at 03:20:59PM -0500, Anthony Liguori wrote: On 09/20/2010 02:44 PM, Michael S. Tsirkin wro

[Qemu-devel] Re: [PATCH] net: delay peer host device delete

2010-09-20 Thread Michael S. Tsirkin
On Mon, Sep 20, 2010 at 02:28:42PM -0500, Anthony Liguori wrote: > On 09/20/2010 02:15 PM, Michael S. Tsirkin wrote: > >On Mon, Sep 20, 2010 at 01:39:00PM -0500, Anthony Liguori wrote: > >>On 09/20/2010 01:24 PM, Michael S. Tsirkin wrote: > >>>On Mon, Sep 20, 2010 at 01:14:12PM -0500, Anthony Liguo

Re: [Qemu-devel] [PATCH] e1000: Pad short frames to minimum size (60 bytes)

2010-09-20 Thread Michael S. Tsirkin
On Mon, Sep 20, 2010 at 10:40:35PM +0200, Edgar E. Iglesias wrote: > On Mon, Sep 20, 2010 at 03:31:32PM -0500, Anthony Liguori wrote: > > On 09/20/2010 05:42 AM, Michael S. Tsirkin wrote: > > > On Sun, Sep 19, 2010 at 07:36:51AM +0100, Stefan Hajnoczi wrote: > > > > > >> On Sat, Sep 18, 2010 at

[Qemu-devel] Re: [PATCH 13/15] Use a few more gcc warning flags

2010-09-20 Thread Blue Swirl
On Fri, Sep 10, 2010 at 9:00 PM, Blue Swirl wrote: > If the compiler supports the following warning flags, use them: > > -Wformat-security -Wformat-y2k -Winit-self -Wignored-qualifiers > -Wmissing-include-dirs -Wclobbered > > Currently, these flags don't produce any warnings. I noticed that in fa

[Qemu-devel] Re: [PATCH] net: delay peer host device delete

2010-09-20 Thread Anthony Liguori
On 09/20/2010 02:37 PM, Michael S. Tsirkin wrote: On Mon, Sep 20, 2010 at 02:22:18PM -0500, Anthony Liguori wrote: On 09/20/2010 01:59 PM, Michael S. Tsirkin wrote: You can also initiate the unplug from the OS without the ACPI event ever happening. I suspect that in our current imple

Re: [Qemu-devel] [PATCH 1/2] Add support for gcc format attribute gnu_printf

2010-09-20 Thread Stefan Weil
Am 20.09.2010 21:18, schrieb Blue Swirl: On Mon, Sep 13, 2010 at 8:02 PM, Stefan Weil wrote: Since version 4.4.x, gcc supports additional format attributes. __attribute__ ((format (gnu_printf, 1, 2))) should be used instead of __attribute__ ((format (printf, 1, 2)) because QEMU alwa

Re: [Qemu-devel] [PATCH] e1000: Pad short frames to minimum size (60 bytes)

2010-09-20 Thread Edgar E. Iglesias
On Mon, Sep 20, 2010 at 03:31:32PM -0500, Anthony Liguori wrote: > On 09/20/2010 05:42 AM, Michael S. Tsirkin wrote: > > On Sun, Sep 19, 2010 at 07:36:51AM +0100, Stefan Hajnoczi wrote: > > > >> On Sat, Sep 18, 2010 at 10:27 PM, Edgar E. Iglesias > >> wrote: > >> > >>> This doesn't look

[Qemu-devel] [PATCH] Move macros GCC_ATTR and GCC_FMT_ATTR to common header file

2010-09-20 Thread Stefan Weil
By moving the definition of GCC_ATTR and GCC_FMT_ATTR from audio_int.h to qemu-common.h these macros are now generally available for further patches which add the gcc format attribute. Newer gcc versions support format gnu_printf which is better suited for use in QEMU than format printf (QEMU alwa

[Bug 638955] Re: [Qemu-devel] [PATCH] e1000: Pad short frames to minimum size (60 bytes)

2010-09-20 Thread Edgar E. Iglesias
On Mon, Sep 20, 2010 at 10:44:34PM +0200, Michael S. Tsirkin wrote: > On Mon, Sep 20, 2010 at 10:40:35PM +0200, Edgar E. Iglesias wrote: > > On Mon, Sep 20, 2010 at 03:31:32PM -0500, Anthony Liguori wrote: > > > On 09/20/2010 05:42 AM, Michael S. Tsirkin wrote: > > > > On Sun, Sep 19, 2010 at 07:36

[Bug 638955] Re: [Qemu-devel] [PATCH] e1000: Pad short frames to minimum size (60 bytes)

2010-09-20 Thread Anthony Liguori
On 09/20/2010 03:44 PM, Michael S. Tsirkin wrote: >>> From f77c3143f3fbefdfa2f0cc873c2665b5aa78e8c9 Mon Sep 17 00:00:00 2001 >>> From: Anthony Liguori >>> Date: Mon, 20 Sep 2010 15:29:31 -0500 >>> Subject: [PATCH] tap: make sure packets are at least 40 bytes long >>> >>> This is required by ethern

Re: [Qemu-devel] [PATCH] e1000: Pad short frames to minimum size (60 bytes)

2010-09-20 Thread Michael S. Tsirkin
On Mon, Sep 20, 2010 at 03:31:32PM -0500, Anthony Liguori wrote: > On 09/20/2010 05:42 AM, Michael S. Tsirkin wrote: > >On Sun, Sep 19, 2010 at 07:36:51AM +0100, Stefan Hajnoczi wrote: > >>On Sat, Sep 18, 2010 at 10:27 PM, Edgar E. Iglesias > >> wrote: > >>>This doesn't look right. AFAIK, MAC's do

[Qemu-devel] [PATCH 2/2] mips_malta: Fix format strings

2010-09-20 Thread Stefan Weil
Fix two compiler warnings (when format attribute is applied). Cc: Aurelien Jarno Cc: Blue Swirl Signed-off-by: Stefan Weil --- hw/mips_malta.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/mips_malta.c b/hw/mips_malta.c index ec95cd8..1cb7880 100644 --- a/hw/mi

[Qemu-devel] Re: [PATCH] net: delay peer host device delete

2010-09-20 Thread Anthony Liguori
On 09/20/2010 03:27 PM, Michael S. Tsirkin wrote: On Mon, Sep 20, 2010 at 03:20:59PM -0500, Anthony Liguori wrote: On 09/20/2010 02:44 PM, Michael S. Tsirkin wrote: I think the only workable approach that doesn't involve new commands is to change the semantics of the existing

[Qemu-devel] Re: Problem with virtio

2010-09-20 Thread Frans de Boer
On 09/20/2010 07:46 PM, Michael S. Tsirkin wrote: > On Sun, Sep 19, 2010 at 11:13:16PM +0200, Frans de Boer wrote: > >> On 09/19/2010 05:53 PM, Michael S. Tsirkin wrote: >> >> On Sat, Sep 18, 2010 at 11:13:12PM +0200, Frans de Boer wrote: >> >> >> On 09/17/2010 10:10 AM, Frans de Boe

Re: [Qemu-devel] [PATCH] e1000: Pad short frames to minimum size (60 bytes)

2010-09-20 Thread Edgar E. Iglesias
On Mon, Sep 20, 2010 at 03:31:32PM -0500, Anthony Liguori wrote: > On 09/20/2010 05:42 AM, Michael S. Tsirkin wrote: > > On Sun, Sep 19, 2010 at 07:36:51AM +0100, Stefan Hajnoczi wrote: > > > >> On Sat, Sep 18, 2010 at 10:27 PM, Edgar E. Iglesias > >> wrote: > >> > >>> This doesn't look

[Qemu-devel] Re: [PATCH] net: delay peer host device delete

2010-09-20 Thread Anthony Liguori
On 09/20/2010 02:44 PM, Michael S. Tsirkin wrote: I think the only workable approach that doesn't involve new commands is to change the semantics of the existing ones. Make netdev_del work regardless of whether the device is still present. You would need to reference count the actual netdev s

[Qemu-devel] Re: [PATCH] net: delay peer host device delete

2010-09-20 Thread Michael S. Tsirkin
On Mon, Sep 20, 2010 at 03:20:59PM -0500, Anthony Liguori wrote: > On 09/20/2010 02:44 PM, Michael S. Tsirkin wrote: > > > >>I think the only workable approach that doesn't involve new commands > >>is to change the semantics of the existing ones. > >> > >>Make netdev_del work regardless of whether

Re: [Qemu-devel] Win2k host problem with {get,free}{addr,name}info()

2010-09-20 Thread Anthony Liguori
On 09/20/2010 03:03 PM, Blue Swirl wrote: On Mon, Sep 20, 2010 at 6:41 PM, Blue Swirl wrote: On Mon, Sep 20, 2010 at 6:26 PM, Anthony Liguori wrote: On 09/19/2010 11:16 AM, Blue Swirl wrote: On Wed, Sep 15, 2010 at 7:25 PM, Anthony Liguori wrote: On 09/15/2010

[Qemu-devel] Re: [PATCH v7] Introduce qemu_madvise()

2010-09-20 Thread Blue Swirl
On Sun, Sep 19, 2010 at 10:11 AM, Andreas Färber wrote: > From: Andreas Färber > > vl.c has a Sun-specific hack to supply a prototype for madvise(), > but the call site has apparently moved to arch_init.c. > > Haiku doesn't implement madvise() in favor of posix_madvise(). > OpenBSD and Solaris 10

[Qemu-devel] Re: [PATCH] net: delay peer host device delete

2010-09-20 Thread Michael S. Tsirkin
On Mon, Sep 20, 2010 at 03:15:45PM -0500, Anthony Liguori wrote: > On 09/20/2010 02:37 PM, Michael S. Tsirkin wrote: > >On Mon, Sep 20, 2010 at 02:22:18PM -0500, Anthony Liguori wrote: > >>On 09/20/2010 01:59 PM, Michael S. Tsirkin wrote: > You can also initiate the unplug from the OS without t

Re: [Qemu-devel] [PATCH] e1000: Pad short frames to minimum size (60 bytes)

2010-09-20 Thread Anthony Liguori
On 09/20/2010 05:42 AM, Michael S. Tsirkin wrote: On Sun, Sep 19, 2010 at 07:36:51AM +0100, Stefan Hajnoczi wrote: On Sat, Sep 18, 2010 at 10:27 PM, Edgar E. Iglesias wrote: This doesn't look right. AFAIK, MAC's dont pad on receive. I agree. NICs that do padding will do it

[Qemu-devel] Re: [PULL] vhost,e1000 fixes

2010-09-20 Thread Michael S. Tsirkin
On Mon, Sep 20, 2010 at 07:35:36PM +, Blue Swirl wrote: > On Mon, Sep 20, 2010 at 7:21 PM, Michael S. Tsirkin wrote: > > On Mon, Sep 20, 2010 at 06:23:55PM +, Blue Swirl wrote: > >> On Mon, Sep 20, 2010 at 6:08 PM, Michael S. Tsirkin > >> wrote: > >> > This fixes a bug in vhost error han

[Qemu-devel] Re: Caching modes

2010-09-20 Thread Anthony Liguori
On 09/20/2010 02:34 PM, Christoph Hellwig wrote: On Mon, Sep 20, 2010 at 11:53:02AM -0500, Anthony Liguori wrote: cache=none All read and write requests SHOULD avoid any type of caching in the host. Any write request MUST complete after the next level of storage reports that the write requ

Re: [Qemu-devel] Win2k host problem with {get,free}{addr,name}info()

2010-09-20 Thread Blue Swirl
On Mon, Sep 20, 2010 at 6:41 PM, Blue Swirl wrote: > On Mon, Sep 20, 2010 at 6:26 PM, Anthony Liguori > wrote: >> On 09/19/2010 11:16 AM, Blue Swirl wrote: >>> >>> On Wed, Sep 15, 2010 at 7:25 PM, Anthony Liguori >>>  wrote: >>> On 09/15/2010 02:11 PM, Blue Swirl wrote: >

[Qemu-devel] Re: [PULL] vhost,e1000 fixes

2010-09-20 Thread Blue Swirl
On Mon, Sep 20, 2010 at 7:21 PM, Michael S. Tsirkin wrote: > On Mon, Sep 20, 2010 at 06:23:55PM +, Blue Swirl wrote: >> On Mon, Sep 20, 2010 at 6:08 PM, Michael S. Tsirkin wrote: >> > This fixes a bug in vhost error handling >> > (also triggers build warning with vhost enabled) >> > and fixes

[Qemu-devel] Re: Caching modes

2010-09-20 Thread Christoph Hellwig
On Mon, Sep 20, 2010 at 11:53:02AM -0500, Anthony Liguori wrote: > cache=none > > All read and write requests SHOULD avoid any type of caching in the > host. Any write request MUST complete after the next level of storage > reports that the write request has completed. A flush from the guest

[Qemu-devel] Re: [PATCH] net: delay peer host device delete

2010-09-20 Thread Anthony Liguori
On 09/20/2010 02:15 PM, Michael S. Tsirkin wrote: On Mon, Sep 20, 2010 at 01:39:00PM -0500, Anthony Liguori wrote: On 09/20/2010 01:24 PM, Michael S. Tsirkin wrote: On Mon, Sep 20, 2010 at 01:14:12PM -0500, Anthony Liguori wrote: On 09/20/2010 12:14 PM, Michael S. Tsirkin wro

[Qemu-devel] Re: KVM call agenda for Sept 21

2010-09-20 Thread Anthony Liguori
On 09/20/2010 02:15 PM, Michael S. Tsirkin wrote: On Mon, Sep 20, 2010 at 06:50:55PM +0200, Avi Kivity wrote: On 09/20/2010 06:44 PM, Chris Wright wrote: Please send in any agenda items you are interested in covering. nested vmx: the resurrection. Nice to see it progressi

[Qemu-devel] Re: [PULL] vhost,e1000 fixes

2010-09-20 Thread Michael S. Tsirkin
On Mon, Sep 20, 2010 at 06:23:55PM +, Blue Swirl wrote: > On Mon, Sep 20, 2010 at 6:08 PM, Michael S. Tsirkin wrote: > > This fixes a bug in vhost error handling > > (also triggers build warning with vhost enabled) > > and fixes e1000 handling of short frames. > > > > Discussion on best ways t

[Qemu-devel] Re: [HaikuPorts-devs] [PATCH 0/8] Add Haiku host support

2010-09-20 Thread Ingo Weinhold
On 2010-09-20 at 00:50:42 [+0200], Andreas Färber wrote: > The following patch series together with qemu_madvise() allows to build and > run QEMU on Haiku, using GCC 4.x. > > Patches 1-5 add support for Haiku hosts to build ppc-softmmu. > They are based on qemu_madvise() v7, since Haiku does no

[Qemu-devel] Re: [PATCH] net: delay peer host device delete

2010-09-20 Thread Michael S. Tsirkin
On Mon, Sep 20, 2010 at 01:39:00PM -0500, Anthony Liguori wrote: > On 09/20/2010 01:24 PM, Michael S. Tsirkin wrote: > >On Mon, Sep 20, 2010 at 01:14:12PM -0500, Anthony Liguori wrote: > >>On 09/20/2010 12:14 PM, Michael S. Tsirkin wrote: > >>>On Mon, Sep 20, 2010 at 11:56:56AM -0500, Anthony Liguo

Re: [Qemu-devel] [PATCH 1/2] Add support for gcc format attribute gnu_printf

2010-09-20 Thread Blue Swirl
On Mon, Sep 13, 2010 at 8:02 PM, Stefan Weil wrote: > Since version 4.4.x, gcc supports additional format attributes. >    __attribute__ ((format (gnu_printf, 1, 2))) > should be used instead of >    __attribute__ ((format (printf, 1, 2)) > because QEMU always uses standard format strings (even wi

[Qemu-devel] Re: KVM call agenda for Sept 21

2010-09-20 Thread Michael S. Tsirkin
On Mon, Sep 20, 2010 at 06:50:55PM +0200, Avi Kivity wrote: > On 09/20/2010 06:44 PM, Chris Wright wrote: > >Please send in any agenda items you are interested in covering. > > > > nested vmx: the resurrection. Nice to see it progressing again, but > there's still a lot of ground to cover. Perh

[Qemu-devel] Re: [PATCH] net: delay peer host device delete

2010-09-20 Thread Anthony Liguori
On 09/20/2010 01:59 PM, Michael S. Tsirkin wrote: You can also initiate the unplug from the OS without the ACPI event ever happening. I suspect that in our current implementation, that means that we'll automatically delete the device which may have strange effects on management tools. So it pro

Re: [Qemu-devel] [PATCH] mips_malta: Detect format errors for function prom_set

2010-09-20 Thread Blue Swirl
On Sat, Sep 11, 2010 at 6:46 AM, Stefan Weil wrote: > Add the necessary gcc attribute and fix the detected errors. This is identical to fulong patch, so my comments on that one apply also here. > > Cc: Aurelien Jarno > Signed-off-by: Stefan Weil > --- >  hw/mips_malta.c |    6 -- >  1 file

[Qemu-devel] Re: [PATCH] net: delay peer host device delete

2010-09-20 Thread Michael S. Tsirkin
On Mon, Sep 20, 2010 at 01:19:48PM -0500, Anthony Liguori wrote: > On 09/20/2010 01:14 PM, Anthony Liguori wrote: > >Here's what makes sense to me: > > > >1) async device remove + poll device status/removal notification + > >remove backend > > > >The management tool needs to determine when the devi

Re: [Qemu-devel] [PATCH] mips_fulong2e: Detect format errors for function prom_set

2010-09-20 Thread Blue Swirl
On Sat, Sep 11, 2010 at 6:46 AM, Stefan Weil wrote: > Add the necessary gcc attribute and fix the detected errors. The fixes are correct, so those parts could be applied. However the gnu_printf part will break with older compilers unless 1/2 is applied first. That part should also use GCC_FMT_AT

Re: [Qemu-devel] [PATCH 2/2] Use new gcc format attribute gnu_printf

2010-09-20 Thread Blue Swirl
On Mon, Sep 13, 2010 at 8:02 PM, Stefan Weil wrote: > Since version 4.4.x, gcc supports additional format attributes. >    __attribute__ ((format (gnu_printf, 1, 2))) > should be used instead of >    __attribute__ ((format (printf, 1, 2)) > because QEMU always uses standard format strings (even wi

Re: [Qemu-devel] Caching modes

2010-09-20 Thread Anthony Liguori
On 09/20/2010 01:37 PM, Blue Swirl wrote: It would be nice to have additional mode, like cache=always, where even flushes MAY be ignored. This would max out the performance. That's cache=unsafe and we have it. I ignored it for the purposes of this discussion. Guest disk cache. For al

Re: [Qemu-devel] Win2k host problem with {get,free}{addr,name}info()

2010-09-20 Thread Blue Swirl
On Mon, Sep 20, 2010 at 6:26 PM, Anthony Liguori wrote: > On 09/19/2010 11:16 AM, Blue Swirl wrote: >> >> On Wed, Sep 15, 2010 at 7:25 PM, Anthony Liguori >>  wrote: >> >>> >>> On 09/15/2010 02:11 PM, Blue Swirl wrote: >>> Hi, I tried to test QEMU on Win2k, but there are run ti

[Qemu-devel] Re: [PATCH] net: delay peer host device delete

2010-09-20 Thread Anthony Liguori
On 09/20/2010 01:24 PM, Michael S. Tsirkin wrote: On Mon, Sep 20, 2010 at 01:14:12PM -0500, Anthony Liguori wrote: On 09/20/2010 12:14 PM, Michael S. Tsirkin wrote: On Mon, Sep 20, 2010 at 11:56:56AM -0500, Anthony Liguori wrote: On 09/20/2010 11:47 AM, Michael S. Tsirkin wro

Re: [Qemu-devel] Caching modes

2010-09-20 Thread Blue Swirl
On Mon, Sep 20, 2010 at 4:53 PM, Anthony Liguori wrote: > Moving to a separate thread since this has come up a few times and I think > we need to discuss the assumptions a bit more. > > This is how I understand the caching modes should behave and what guarantees > a guest gets. > > cache=none > >

[Qemu-devel] Installing on qemu nw312

2010-09-20 Thread Bhasker C V
Hi, Just for fun, I was trying to install Novell netware server 3.12 on qemu. I get an error while mounting SYS volume and haven't been successful although I am sure that I have installed the same software before on vmware and it worked well. I can send a screenshot of the error message in

Re: [Qemu-devel] Win2k host problem with {get,free}{addr,name}info()

2010-09-20 Thread Anthony Liguori
On 09/19/2010 11:16 AM, Blue Swirl wrote: On Wed, Sep 15, 2010 at 7:25 PM, Anthony Liguori wrote: On 09/15/2010 02:11 PM, Blue Swirl wrote: Hi, I tried to test QEMU on Win2k, but there are run time errors because of missing {get,free}{addr,name}info() functions. After adding dummy d

[Qemu-devel] Re: [PATCH] net: delay peer host device delete

2010-09-20 Thread Michael S. Tsirkin
On Mon, Sep 20, 2010 at 01:14:12PM -0500, Anthony Liguori wrote: > On 09/20/2010 12:14 PM, Michael S. Tsirkin wrote: > >On Mon, Sep 20, 2010 at 11:56:56AM -0500, Anthony Liguori wrote: > >>On 09/20/2010 11:47 AM, Michael S. Tsirkin wrote: > >>>On Mon, Sep 20, 2010 at 11:41:45AM -0500, Anthony Liguo

[Qemu-devel] Re: [PULL] vhost,e1000 fixes

2010-09-20 Thread Blue Swirl
On Mon, Sep 20, 2010 at 6:08 PM, Michael S. Tsirkin wrote: > This fixes a bug in vhost error handling > (also triggers build warning with vhost enabled) > and fixes e1000 handling of short frames. > > Discussion on best ways to fix the e1000 issue > is still ongoing but the  bug is severe enough >

[Qemu-devel] Re: [PATCH v3 00/13] pcie port switch emulators

2010-09-20 Thread Michael S. Tsirkin
On Wed, Sep 15, 2010 at 02:38:13PM +0900, Isaku Yamahata wrote: > Here is v3 of the patch series. > I didn't address the pcie_init() issue yet with v3 because > there are already many changes. So I'd like to get feed back > before going too far. The issue would be addressed with the next spin > if

[Qemu-devel] Re: [PULL] vhost,e1000 fixes

2010-09-20 Thread Anthony Liguori
On 09/20/2010 01:08 PM, Michael S. Tsirkin wrote: This fixes a bug in vhost error handling (also triggers build warning with vhost enabled) and fixes e1000 handling of short frames. Discussion on best ways to fix the e1000 issue is still ongoing but the bug is severe enough for some guests and

[Qemu-devel] Re: [PATCH v3 04/13] pcie: add pcie constants to pcie_regs.h

2010-09-20 Thread Michael S. Tsirkin
On Wed, Sep 15, 2010 at 02:38:17PM +0900, Isaku Yamahata wrote: > add pcie constants to pcie_regs.h. > Those constants should go to Linux pci_regs.h and then the file should > go away eventually. > > Signed-off-by: Isaku Yamahata > --- > Changes v2 -> v3: > - moved out pcie constants from pcie.c

Re: [Qemu-devel] [PATCH] net: delay peer host device delete

2010-09-20 Thread Anthony Liguori
On 09/20/2010 01:14 PM, Anthony Liguori wrote: Here's what makes sense to me: 1) async device remove + poll device status/removal notification + remove backend The management tool needs to determine when the device is gone and remove the backend. 2) sync device remove + remove backend Com

Re: [Qemu-devel] [PATCH v3 00/15] GCC warning flags

2010-09-20 Thread Blue Swirl
On Mon, Sep 20, 2010 at 11:28 AM, Stefan Weil wrote: > Am 10.09.2010 22:58, schrieb Blue Swirl: >> >> I updated the descriptions and cleaned up 6/16. >> >> Blue Swirl (15): >> Check for errors during BIOS or kernel load >> linux-user: fix socklen_t comparisons >> linux-user: fix types in a compari

[Qemu-devel] [PULL] vhost,e1000 fixes

2010-09-20 Thread Michael S. Tsirkin
This fixes a bug in vhost error handling (also triggers build warning with vhost enabled) and fixes e1000 handling of short frames. Discussion on best ways to fix the e1000 issue is still ongoing but the bug is severe enough for some guests and the fix is safe enough that I feel we should have it

Re: [Qemu-devel] [PATCH] net: delay peer host device delete

2010-09-20 Thread Anthony Liguori
On 09/20/2010 12:14 PM, Michael S. Tsirkin wrote: On Mon, Sep 20, 2010 at 11:56:56AM -0500, Anthony Liguori wrote: On 09/20/2010 11:47 AM, Michael S. Tsirkin wrote: On Mon, Sep 20, 2010 at 11:41:45AM -0500, Anthony Liguori wrote: On 09/20/2010 11:30 AM, Michael S. Tsirkin wro

[Qemu-devel] Re: [PATCH] Make NIC model fallback to default when specified model is not supported

2010-09-20 Thread Michael S. Tsirkin
On Mon, Sep 20, 2010 at 11:47:59AM +0200, Michal Novotny wrote: > Hi, > > this is the patch to introduce a NIC model fallback to default when model > specified is not supported. It's been tested on i386-softmmu target on > i386 host using the Windows XP x86 virtual machine and by trying to setup >

Re: [Qemu-devel] [PATCH] Fix vhost_net compilation errors for i386-softmmu target

2010-09-20 Thread Blue Swirl
On Mon, Sep 20, 2010 at 9:36 AM, Michal Novotny wrote: > Hi, > there were compilation errors when I was trying to compile i386-softmmu > target on i386 > host (running on Fedora-13 with development version of qemu downloaded from > git). > > There were errors of comparison of unsigned expression w

[Qemu-devel] Re: [PATCH] Fix vhost_net compilation errors for i386-softmmu target

2010-09-20 Thread Michael S. Tsirkin
On Mon, Sep 20, 2010 at 11:36:58AM +0200, Michal Novotny wrote: > Hi, > there were compilation errors when I was trying to compile > i386-softmmu target on i386 > host (running on Fedora-13 with development version of qemu > downloaded from git). > > There were errors of comparison of unsigned exp

[Qemu-devel] Re: [PATCH] e1000: Pad short frames to minimum size (60 bytes)

2010-09-20 Thread Michael S. Tsirkin
On Sat, Sep 18, 2010 at 09:43:45PM +0100, Stefan Hajnoczi wrote: > The OpenIndiana (Solaris) e1000g driver drops frames that are too long > or too short. It expects to receive frames of at least the Ethernet > minimum size. ARP requests in particular are small and will be dropped > if they are no

Re: [Qemu-devel] Win2k host problem with {get,free}{addr,name}info()

2010-09-20 Thread Blue Swirl
On Mon, Sep 20, 2010 at 4:16 AM, Joe Ross wrote: > On Wednesday, September 15, 2010, Anthony Liguori > wrote: >> Win2k is EOL so I don't think it's useful for us to support it as a host.   >> So any type of patch is just going to add additional complexity for very >> little real gain. >> > > Wh

[Qemu-devel] Re: Re: Problem with virtio

2010-09-20 Thread Michael S. Tsirkin
On Sun, Sep 19, 2010 at 11:13:16PM +0200, Frans de Boer wrote: > On 09/19/2010 05:53 PM, Michael S. Tsirkin wrote: > > On Sat, Sep 18, 2010 at 11:13:12PM +0200, Frans de Boer wrote: > > > On 09/17/2010 10:10 AM, Frans de Boer wrote: > > > On 09/17/2010 01:40 AM, Luiz Ca

[Qemu-devel] [PATCH 0/2] powerpc: Improve BookE emulation (v2)

2010-09-20 Thread Edgar E. Iglesias
Improve BookE emulation in preparation for virtex 5 support. Once this is OK, the Xilinx specific parts will follow. Cheers, Edgar v2: * Fix MMU emulation details and other comments from A. Graf. Edgar E. Iglesias (2): powerpc: Improve emulation of the BookE MMU powerpc: Make the decr interr

[Qemu-devel] [PATCH 1/2] powerpc: Improve emulation of the BookE MMU

2010-09-20 Thread Edgar E. Iglesias
Improve the emulation of the BookE MMU to be able to boot linux on virtex5 boards. Signed-off-by: Edgar E. Iglesias --- target-ppc/cpu.h|3 +++ target-ppc/helper.c | 38 ++ 2 files changed, 33 insertions(+), 8 deletions(-) diff --git a/target-ppc/cp

[Qemu-devel] [PATCH 2/2] powerpc: Make the decr interrupt type overridable

2010-09-20 Thread Edgar E. Iglesias
Make it possible for boards to override the kind of interrupt to be signaled when the decr timer hits. The 405's signal PIT interrupts while the 440's signal DECR. Signed-off-by: Edgar E. Iglesias --- hw/ppc.c | 16 +--- hw/ppc.h |4 +++- hw/ppc4xx_devs.c |2

Re: [Qemu-devel] [PATCH] net: delay peer host device delete

2010-09-20 Thread Michael S. Tsirkin
On Mon, Sep 20, 2010 at 11:56:56AM -0500, Anthony Liguori wrote: > On 09/20/2010 11:47 AM, Michael S. Tsirkin wrote: > >On Mon, Sep 20, 2010 at 11:41:45AM -0500, Anthony Liguori wrote: > >>On 09/20/2010 11:30 AM, Michael S. Tsirkin wrote: > >>>With -netdev, virtio devices present offload > >>>featu

Re: [Qemu-devel] [PATCH] net: delay peer host device delete

2010-09-20 Thread Anthony Liguori
On 09/20/2010 11:47 AM, Michael S. Tsirkin wrote: On Mon, Sep 20, 2010 at 11:41:45AM -0500, Anthony Liguori wrote: On 09/20/2010 11:30 AM, Michael S. Tsirkin wrote: With -netdev, virtio devices present offload features to guest, depending on the backend used. Thus, removing host ntede

Re: [Qemu-devel] [PATCH] net: delay peer host device delete

2010-09-20 Thread Michael S. Tsirkin
On Mon, Sep 20, 2010 at 11:41:45AM -0500, Anthony Liguori wrote: > On 09/20/2010 11:30 AM, Michael S. Tsirkin wrote: > >With -netdev, virtio devices present offload > >features to guest, depending on the backend used. > >Thus, removing host ntedev peer while guest is > >active leads to guest-visibl

[Qemu-devel] Caching modes

2010-09-20 Thread Anthony Liguori
Moving to a separate thread since this has come up a few times and I think we need to discuss the assumptions a bit more. This is how I understand the caching modes should behave and what guarantees a guest gets. cache=none All read and write requests SHOULD avoid any type of caching in the

[Qemu-devel] Re: KVM call agenda for Sept 21

2010-09-20 Thread Avi Kivity
On 09/20/2010 06:44 PM, Chris Wright wrote: Please send in any agenda items you are interested in covering. nested vmx: the resurrection. Nice to see it progressing again, but there's still a lot of ground to cover. Perhaps we can involve Intel to speed things up? -- error compiling com

[Qemu-devel] KVM call agenda for Sept 21

2010-09-20 Thread Chris Wright
Please send in any agenda items you are interested in covering. thanks, -chris

Re: [Qemu-devel] [PATCH] net: delay peer host device delete

2010-09-20 Thread Anthony Liguori
On 09/20/2010 11:30 AM, Michael S. Tsirkin wrote: With -netdev, virtio devices present offload features to guest, depending on the backend used. Thus, removing host ntedev peer while guest is active leads to guest-visible inconsistency and/or crashes. See e.g. https://bugzilla.redhat.com/show_bug

[Qemu-devel] [PATCH] net: delay peer host device delete

2010-09-20 Thread Michael S. Tsirkin
With -netdev, virtio devices present offload features to guest, depending on the backend used. Thus, removing host ntedev peer while guest is active leads to guest-visible inconsistency and/or crashes. See e.g. https://bugzilla.redhat.com/show_bug.cgi?id=623735 As a solution, while guest (NIC) pee

Re: [Qemu-devel] [RFC] block-queue: Delay and batch metadata writes

2010-09-20 Thread Anthony Liguori
On 09/20/2010 10:55 AM, Kevin Wolf wrote: Am 20.09.2010 17:40, schrieb Anthony Liguori: On 09/20/2010 10:08 AM, Kevin Wolf wrote: If you're comfortable with a writeback cache for metadata, then you should also be comfortable with a writeback cache for data in which case, cache=writeba

[Qemu-devel] Patch for Cortex-A9 NEON instructions

2010-09-20 Thread Pham Van Thiet
Hello everybody, Some days ago, I read a topic about "QEMU state of ARM NEON support" from address: http://comments.gmane.org/gmane.comp.emulators.qemu/65999 I dowloaded test file anh run it on QEMU for cortex-a8 because the reference test file is written for cortex-a8 and I also listed some ins

[Qemu-devel] QEMU's not support NEON instructions (NEON run incorrectly)

2010-09-20 Thread Pham Van Thiet
Hello everybody, I arm building VP8 (WebM) for QEMU, when I configure VP8 for ARMv6 then it run ok on QEMU but when I build it with ARMv7 then it's not run on QEMU, I asked on WebM's forum of Google and they said that QEMU does not support VP8 NEON code right now I ran debug to test their replys

Re: [Qemu-devel] [RFC] block-queue: Delay and batch metadata writes

2010-09-20 Thread Avi Kivity
On 09/20/2010 05:51 PM, Anthony Liguori wrote: On 09/20/2010 10:08 AM, Kevin Wolf wrote: If you're comfortable with a writeback cache for metadata, then you should also be comfortable with a writeback cache for data in which case, cache=writeback is the answer. Well, there is a difference: W

Re: [Qemu-devel] [RFC] block-queue: Delay and batch metadata writes

2010-09-20 Thread Kevin Wolf
Am 20.09.2010 17:40, schrieb Anthony Liguori: > On 09/20/2010 10:08 AM, Kevin Wolf wrote: >>> If you're comfortable with a writeback cache for metadata, then you >>> should also be comfortable with a writeback cache for data in which >>> case, cache=writeback is the answer. >>> >> Well, there

Re: [Qemu-devel] [RFC] block-queue: Delay and batch metadata writes

2010-09-20 Thread Anthony Liguori
On 09/20/2010 10:08 AM, Kevin Wolf wrote: If you're comfortable with a writeback cache for metadata, then you should also be comfortable with a writeback cache for data in which case, cache=writeback is the answer. Well, there is a difference: We don't pollute the host page cache with gu

Re: [Qemu-devel] [RFC] block-queue: Delay and batch metadata writes

2010-09-20 Thread Anthony Liguori
On 09/20/2010 10:33 AM, Kevin Wolf wrote: Am 20.09.2010 16:56, schrieb Anthony Liguori: +void blkqueue_flush(BlockQueue *bq) +{ +qemu_mutex_lock(&bq->flush_lock); + +/* Process any left over requests */ +while (QTAILQ_FIRST(&bq->queue)) { +blkqueue_process_request(bq); +

Re: [Qemu-devel] [RFC] block-queue: Delay and batch metadata writes

2010-09-20 Thread Kevin Wolf
Am 20.09.2010 17:33, schrieb Avi Kivity: > On 09/20/2010 05:08 PM, Kevin Wolf wrote: >>> >>> Let's expand it a bit more: >>> >>> 1. Update refcount table >>> 2. bdrv_flush >>> 3. Update L2 entry >>> 4. Write data to disk >>> 5. Report write complete >>> >>> I'm struggling to understand how

Re: [Qemu-devel] Re: [PATCH v3] blkverify: Add block driver for verifying I/O

2010-09-20 Thread Kevin Wolf
Am 20.09.2010 17:22, schrieb Stefan Hajnoczi: > Thanks for your comments Kevin! > > I'd like to merge the overlapping I/O patch I sent today since > blkverify is not in mainline yet. Are you okay with that or should I > keep them separate? If that works better for you, I don't mind merging them.

Re: [Qemu-devel] [RFC] block-queue: Delay and batch metadata writes

2010-09-20 Thread Anthony Liguori
On 09/20/2010 10:08 AM, Kevin Wolf wrote: If you're comfortable with a writeback cache for metadata, then you should also be comfortable with a writeback cache for data in which case, cache=writeback is the answer. Well, there is a difference: We don't pollute the host page cache with gues

Re: [Qemu-devel] [RFC] block-queue: Delay and batch metadata writes

2010-09-20 Thread Avi Kivity
On 09/20/2010 05:33 PM, Avi Kivity wrote: On 09/20/2010 05:08 PM, Kevin Wolf wrote: > > Let's expand it a bit more: > > 1. Update refcount table > 2. bdrv_flush > 3. Update L2 entry > 4. Write data to disk > 5. Report write complete > > I'm struggling to understand how a thread helps ou

Re: [Qemu-devel] [RFC] block-queue: Delay and batch metadata writes

2010-09-20 Thread Avi Kivity
On 09/20/2010 05:08 PM, Kevin Wolf wrote: > > Let's expand it a bit more: > > 1. Update refcount table > 2. bdrv_flush > 3. Update L2 entry > 4. Write data to disk > 5. Report write complete > > I'm struggling to understand how a thread helps out. This sequence becomes: 1. Update refcou

Re: [Qemu-devel] [RFC] block-queue: Delay and batch metadata writes

2010-09-20 Thread Kevin Wolf
Am 20.09.2010 16:56, schrieb Anthony Liguori: >>> +void blkqueue_flush(BlockQueue *bq) >>> +{ >>> +qemu_mutex_lock(&bq->flush_lock); >>> + >>> +/* Process any left over requests */ >>> +while (QTAILQ_FIRST(&bq->queue)) { >>> +blkqueue_process_request(bq); >>> +} >>> + >>> +

[Qemu-devel] [Bug 611142] Re: seabios should have native scsi support

2010-09-20 Thread Scott Moser
** Tags removed: ec2-images -- seabios should have native scsi support https://bugs.launchpad.net/bugs/611142 You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. Status in QEMU: New Status in “qemu-kvm” package in Ubuntu: New Status in “sea

Re: [Qemu-devel] Re: [PATCH v3] blkverify: Add block driver for verifying I/O

2010-09-20 Thread Stefan Hajnoczi
Thanks for your comments Kevin! I'd like to merge the overlapping I/O patch I sent today since blkverify is not in mainline yet. Are you okay with that or should I keep them separate? On Mon, Sep 20, 2010 at 3:48 PM, Kevin Wolf wrote: > Am 04.09.2010 17:34, schrieb Stefan Hajnoczi: >> +static v

Re: [Qemu-devel] [RFC] block-queue: Delay and batch metadata writes

2010-09-20 Thread Kevin Wolf
Am 20.09.2010 16:31, schrieb Anthony Liguori: > On 09/20/2010 08:56 AM, Kevin Wolf wrote: >> I won't get this ready until I leave for vacation on Wednesday, so I thought >> I >> could just as well post it as an RFC in this state. >> >> With this patch applied, qcow2 doesn't directly access the ima

Re: [Qemu-devel] [RFC] block-queue: Delay and batch metadata writes

2010-09-20 Thread Anthony Liguori
On 09/20/2010 09:31 AM, Anthony Liguori wrote: If we delay the operation and get three of these sequences queued before actually executing, we end up with the following result, saving two syncs: 1. Update refcount table (req 1) 2. Update refcount table (req 2) 3. Update refcount table (req 3)

[Qemu-devel] Re: [PATCH v3] blkverify: Add block driver for verifying I/O

2010-09-20 Thread Kevin Wolf
Am 04.09.2010 17:34, schrieb Stefan Hajnoczi: > The blkverify block driver makes investigating image format data > corruption much easier. A raw image initialized with the same contents > as the test image (e.g. qcow2 file) must be provided. The raw image > mirrors read/write operations and is us

Re: [Qemu-devel] Re: [PATCH] Make NIC model fallback to default when specified model is not supported

2010-09-20 Thread Anthony Liguori
On 09/20/2010 06:15 AM, Michal Novotny wrote: Well, this way I guess we should have 2, maybe 3 different approaches - different for qemu itself and for upstream version Xen using the older version of qemu-dm and RHEL-5 version. Therefore I think we should drop the patch for qemu (the one sent

Re: [Qemu-devel] [RFC] block-queue: Delay and batch metadata writes

2010-09-20 Thread Anthony Liguori
On 09/20/2010 08:56 AM, Kevin Wolf wrote: I won't get this ready until I leave for vacation on Wednesday, so I thought I could just as well post it as an RFC in this state. With this patch applied, qcow2 doesn't directly access the image file any more for metadata, but rather goes through the ne

Re: [Qemu-devel] [PATCH] nbd: correctly manage default port

2010-09-20 Thread Kevin Wolf
Am 17.09.2010 20:37, schrieb Laurent Vivier: > block/nbd.c: use default port number when none is specified > qemu-nbd.c: use IANA-assigned port number: 10809 > > Signed-off-by: Laurent Vivier Thanks, applied to the block branch. Kevin

[Qemu-devel] Re: [PATCH 2/4] powerpc: Make the decr interrupt type overridable

2010-09-20 Thread Alexander Graf
Am 20.09.2010 um 14:11 schrieb "Edgar E. Iglesias" : > On Mon, Sep 20, 2010 at 12:42:41PM +0200, Alexander Graf wrote: >> Edgar E. Iglesias wrote: >>> Make it possible for boards to override the kind of interrupt >>> to be signaled when the decr timer hits. The 405's signal PIT >>> interrupts whi

  1   2   >