Re: [Qemu-devel] [PATCH 5/5] drive_open: Add invalidate option for block devices

2011-01-07 Thread Christoph Hellwig
On Tue, Jan 04, 2011 at 03:33:30PM +0100, Juan Quintela wrote: Linux allows to invalidate block devices. This is needed for the incoming migration part. It's not. The only thing that makes migration on block devices safe is using O_DIRECT, aka cache=none.

[Qemu-devel] Re: [PATCH] savevm: print migration failure to stderr rather than monitor

2011-01-07 Thread Jan Kiszka
Am 07.01.2011 08:18, Alex Williamson wrote: monitor_print only does anything for foreground commands, so we don't ever see this error message in the case of a 'migrate -d'. Your change needlessly steals the error from the monitor console where it belongs if migrate is used without -d. IIRC, mon

[Qemu-devel] Re: [PATCH v5 0/4] virtio: Use ioeventfd for virtqueue notify

2011-01-07 Thread Stefan Hajnoczi
On Thu, Jan 6, 2011 at 6:00 PM, Michael S. Tsirkin m...@redhat.com wrote: On Thu, Jan 06, 2011 at 04:41:50PM +, Stefan Hajnoczi wrote: Here are 4k sequential read results (cache=none) to check whether we see an ioeventfd performance regression with virtio-blk. The idea is to use a small

[Qemu-devel] Re: [PATCH 26/35] kvm: Eliminate KVMState arguments

2011-01-07 Thread Jan Kiszka
Am 06.01.2011 20:24, Anthony Liguori wrote: On 01/06/2011 11:56 AM, Marcelo Tosatti wrote: From: Jan Kiszkajan.kis...@siemens.com QEMU supports only one VM, so there is only one kvm_state per process, and we gain nothing passing a reference to it around. Eliminate any need to refer to it

Re: [Qemu-devel] [PATCH 1/3] tcg/arm: fix branch target change during code retranslation

2011-01-07 Thread Edgar E. Iglesias
On Thu, Jan 06, 2011 at 10:54:32PM +0100, Aurelien Jarno wrote: QEMU uses code retranslation to restore the CPU state when an exception happens. For it to work the retranslation must not modify the generated code. This is what is currently implemented in ARM TCG. However on CPU that don't

[Qemu-devel] [PATCH 3/4] target-arm: Add support for 'Standard FPSCR Value' as used by Neon

2011-01-07 Thread Peter Maydell
Add support to the ARM helper routines for a second fp_status value which should be used for operations which the ARM ARM indicates use ARM standard floating-point arithmetic rather than being controlled by the rounding/flush/NaN settings in the FPSCR. Signed-off-by: Peter Maydell

[Qemu-devel] [PATCH 2/4] target-arm: Fix implementation of VRSQRTS

2011-01-07 Thread Peter Maydell
The implementation of the ARM VRSQRTS instruction (which calculates (3 - op1 * op2) / 2) was missing the division operation. It also did not handle the special cases of (0,inf) and (inf,0). Signed-off-by: Peter Maydell peter.mayd...@linaro.org --- target-arm/helper.c | 10 +- 1 files

[Qemu-devel] [PATCH 1/4] softfloat: Add float32_is_zero_or_denormal() function

2011-01-07 Thread Peter Maydell
Add a utility function to softfloat to test whether a float32 is zero or denormal. Signed-off-by: Peter Maydell peter.mayd...@linaro.org --- fpu/softfloat.h |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/fpu/softfloat.h b/fpu/softfloat.h index 15052cc..1acf7af

[Qemu-devel] [PATCH 4/4] target-arm: Use the standard FPSCR value for VRSQRTS

2011-01-07 Thread Peter Maydell
VSQRTS always uses the standard FPSCR value as it is a Neon instruction. Signed-off-by: Peter Maydell peter.mayd...@linaro.org --- target-arm/helper.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/target-arm/helper.c b/target-arm/helper.c index d779055..8186500 100644

[Qemu-devel] [PATCH 0/4] target-arm: Fix VRSQRTS

2011-01-07 Thread Peter Maydell
This patch series fixes various bugs in VRSQRTS including the major one that it was calculating completely the wrong value (missing a division by 2). It also introduces the infrastructure to support the ARM Standard FPSCR Value as used for operations which use ARM standard floating-point

Re: [Qemu-devel] snapshots

2011-01-07 Thread Stefan Hajnoczi
On Fri, Jan 7, 2011 at 4:37 AM, chandra shekar chandrashekar...@gmail.com wrote: hi everyone i have got snapshot out of the image file now i would like to analyze the snapshot like opening it or identifying the fields in it can anyone suggest any softwares which i can use to open the snapshot

[Qemu-devel] [Bug 604872] Re: qemu-system-arm segfaults emulating versatile machine after running debootstrap --second-stage inside vm

2011-01-07 Thread Jani Monoses
should the fixes be applied to qemu-kvm if we plan on packaging qemu- meego for ARM support anyway? -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/604872 Title: qemu-system-arm segfaults emulating

[Qemu-devel] [Bug 604872] Re: qemu-system-arm segfaults emulating versatile machine after running debootstrap --second-stage inside vm

2011-01-07 Thread Loïc Minier
qemu-kvm is what's currently in the Ubuntu archive; I'm sure Peter will also arrange for the ubuntu-qemu-omap branch to get these fixes once they are in suitable shape. In any case, these fixes are also going upstream and will eventually bubble up to derived trees -- You received this bug

Re: [Qemu-devel] Re: [PATCH 06/13] Threadlet: Add dequeue_work threadlet API

2011-01-07 Thread Stefan Hajnoczi
On Thu, Jan 6, 2011 at 10:43 AM, Arun R Bharadwaj a...@linux.vnet.ibm.com wrote: * Stefan Hajnoczi stefa...@linux.vnet.ibm.com [2011-01-05 19:55:46]: On Tue, Jan 04, 2011 at 10:57:39AM +0530, Arun R Bharadwaj wrote: @@ -574,33 +574,39 @@ static void paio_remove(struct qemu_paiocb *acb)      

[Qemu-devel] Re: [Bug 604872] Re: qemu-system-arm segfaults emulating versatile machine after running debootstrap --second-stage inside vm

2011-01-07 Thread Ricardo Salveti
On Fri, Jan 7, 2011 at 7:53 AM, Jani Monoses j...@ubuntu.com wrote: should the fixes be applied to qemu-kvm if we plan on packaging qemu- meego for ARM support anyway? It depends on the size of the fix, if it's something simple we can for sure also update the qemu-kvm package. But as Loic

Re: [Qemu-devel] Re: [PATCH 2/3] piix: tag as not hotpluggable.

2011-01-07 Thread Michael S. Tsirkin
On Thu, Jan 06, 2011 at 04:45:20PM -0200, Marcelo Tosatti wrote: On Thu, Jan 06, 2011 at 04:34:38PM +0200, Michael S. Tsirkin wrote: On Thu, Jan 06, 2011 at 03:14:18PM +0100, Gerd Hoffmann wrote: Hi, Could you split the tested and untested parts to separate patches, noting the

[Qemu-devel] Re: [PATCH] add event queueing to USB HID

2011-01-07 Thread Ian Jackson
Gerd Hoffmann writes (Re: [PATCH] add event queueing to USB HID): On 12/23/10 15:57, Paolo Bonzini wrote: @@ -68,7 +77,7 @@ typedef struct USBHIDState { int protocol; uint8_t idle; int64_t next_idle_clock; -int changed; +int have_data, changed; What is the

[Qemu-devel] Re: [PATCH] add event queueing to USB HID

2011-01-07 Thread Ian Jackson
I wrote: Paulo: For reference, here is a diff of the relevant functionality. It's against qemu 0.10.0 (ish) but it may be a better starting point than what you used. I just had a go at seeing how that applies to current upstream master and it doesn't look like it would be hard to fix up. Ian.

Re: [Qemu-devel] [PATCH 2/3] tcg/arm: fix qemu_st64 for big endian targets

2011-01-07 Thread andrzej zaborowski
On 6 January 2011 22:54, Aurelien Jarno aurel...@aurel32.net wrote: Due to a typo, qemu_st64 doesn't properly byteswap the 32-bit low word of a 64 bit word before saving it. This patch fixes that. This is a good catch. Acked-by: Andrzej Zaborowski balr...@gmail.com Cheers

Re: [Qemu-devel] [PATCH 3/3] tcg/arm: improve constant loading

2011-01-07 Thread andrzej zaborowski
Hi, On 6 January 2011 22:54, Aurelien Jarno aurel...@aurel32.net wrote: Improve constant loading in two ways: - On all ARM versions, it's possible to load 0xff00 = -0x100 using  the mvn rd, #0. Fix the conditions. - On = ARMv6 versions, where movw and movt are not available, load the  

Re: [Qemu-devel] [PATCH 3/3] tcg/arm: improve constant loading

2011-01-07 Thread andrzej zaborowski
On 7 January 2011 13:52, andrzej zaborowski bal...@zabor.org wrote: On 6 January 2011 22:54, Aurelien Jarno aurel...@aurel32.net wrote: Improve constant loading in two ways: - On all ARM versions, it's possible to load 0xff00 = -0x100 using  the mvn rd, #0. Fix the conditions. - On =

Re: [Qemu-devel] [PATCH 3/3] tcg/arm: improve constant loading

2011-01-07 Thread Aurelien Jarno
On Fri, Jan 07, 2011 at 01:52:25PM +0100, andrzej zaborowski wrote: Hi, On 6 January 2011 22:54, Aurelien Jarno aurel...@aurel32.net wrote: Improve constant loading in two ways: - On all ARM versions, it's possible to load 0xff00 = -0x100 using  the mvn rd, #0. Fix the conditions.

Re: [Qemu-devel] [PATCH v2 2/4] piix: tag as not hotpluggable.

2011-01-07 Thread Aurelien Jarno
On Thu, Jan 06, 2011 at 03:14:38PM +0100, Gerd Hoffmann wrote: This patch tags all pci devices which belong to the piix3/4 chipsets as not hotpluggable (Host bridge, ISA bridge, IDE controller, ACPI bridge). Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/acpi_piix4.c |2 ++

[Qemu-devel] [PATCH] lsi53c895a: fix endianness issues

2011-01-07 Thread y
From: Aurelien Jarno aurel...@aurel32.net lsi_ram_read*() and lsi_ram_write*() are not consistent, one uses leXX_to_cpu() the other uses nothing. As the comment above the RAM declaration says: Script ram is stored as 32-bit words in host byteorder., remove the leXX_to_cpu() calls. This fixes the

[Qemu-devel] [PATCH 6/7] target-arm: Set privileged bit in TB flags correctly for M profile

2011-01-07 Thread Peter Maydell
M profile ARM cores don't have a CPSR mode field. Set the bit in the TB flags that indicates non-user mode correctly for these cores. Signed-off-by: Peter Maydell peter.mayd...@linaro.org --- target-arm/cpu.h | 17 - 1 files changed, 16 insertions(+), 1 deletions(-) diff --git

[Qemu-devel] [PATCH 2/7] target-arm: Translate with VFP-enabled from TB flags, not CPUState

2011-01-07 Thread Peter Maydell
When translating code, whether the VFP unit is enabled for this TB is stored in a bit in the TB flags. Use this rather than incorrectly reading the FPEXC from the CPUState passed to translation. Signed-off-by: Peter Maydell peter.mayd...@linaro.org --- target-arm/translate.c | 14

[Qemu-devel] [PATCH 0/7] target-arm: Translate based on TB flags, not CPUState

2011-01-07 Thread Peter Maydell
This patchset corrects a number of places in the ARM translation code which were generating code which was dependent on values in the CPUState structure which might change at runtime. This is a bad idea for two reasons. Firstly, we might try to reuse the generated code later when the assumptions

[Qemu-devel] [PATCH 3/7] target-arm: Translate with VFP len/stride from TB flags, not CPUState

2011-01-07 Thread Peter Maydell
When translating, the VFP vector length and stride for this TB are encoded in the TB flags; the CPUState copies may be different and must not be used. Signed-off-by: Peter Maydell peter.mayd...@linaro.org --- target-arm/translate.c | 10 +++--- 1 files changed, 7 insertions(+), 3

[Qemu-devel] [PATCH 7/7] target-arm: Translate with user-state from TB flags, not CPUState

2011-01-07 Thread Peter Maydell
When translating, get the user/priv state from the TB flags, not the CPUState. Signed-off-by: Peter Maydell peter.mayd...@linaro.org --- target-arm/translate.c |6 +- 1 files changed, 1 insertions(+), 5 deletions(-) diff --git a/target-arm/translate.c b/target-arm/translate.c index

[Qemu-devel] [PATCH 4/7] target-arm: Translate with Thumb state from TB flags, not CPUState

2011-01-07 Thread Peter Maydell
The Thumb/ARM state for the TB being translated should come from the TB flags, not the CPUState. Signed-off-by: Peter Maydell peter.mayd...@linaro.org --- target-arm/translate.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/target-arm/translate.c

[Qemu-devel] [PATCH 1/7] target-arm: Don't generate code specific to current CPU mode for SRS

2011-01-07 Thread Peter Maydell
When translating the SRS instruction, handle the store registers to stack of current mode case in the helper function rather than inline. This means the generated code does not make assumptions about the current CPU mode which might not be valid when the TB is executed later. Signed-off-by: Peter

[Qemu-devel] [PATCH 5/7] target-arm: Translate with condexec bits from TB flags, not CPUState

2011-01-07 Thread Peter Maydell
When translating, the condexec bits for the TB are in the TB flags; the CPUState condexec bits may be different. This patch fixes https://bugs.launchpad.net/bugs/604872 where we might segfault if we took an exception in the middle of a TB with an IT block, because when we came to retranslate in

Re: [Qemu-devel] [PATCH 2/2] linux-user: Fix incorrect NaN detection in ARM nwfpe emulation

2011-01-07 Thread Aurelien Jarno
On Thu, Jan 06, 2011 at 06:34:44PM +, Peter Maydell wrote: The code in the linux-user ARM nwfpe emulation was incorrectly checking only for quiet NaNs when it should have been checking for any kind of NaN. This is probably because the code in question was taken from the Linux kernel, whose

Re: [Qemu-devel] [PATCH 1/2] softfloat: Implement floatx80_is_any_nan() and float128_is_any_nan()

2011-01-07 Thread Aurelien Jarno
On Thu, Jan 06, 2011 at 06:34:43PM +, Peter Maydell wrote: Implement versions of float*_is_any_nan() for the floatx80 and float128 types. Signed-off-by: Peter Maydell peter.mayd...@linaro.org --- fpu/softfloat.h | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-)

[Qemu-devel] [Bug 688085] Re: Guest kernel hang during boot when KVM is active on i386 host

2011-01-07 Thread Serge Hallyn
Thanks very much for confirming. Forwarded the patch. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/688085 Title: Guest kernel hang during boot when KVM is active on i386 host Status in meego

Re: [Qemu-devel] [PATCH 0/2] ARM: fix commandline handling forsemihosted executables

2011-01-07 Thread riku voipio
Hi, Honestly, I have never tried ARM semihosting. Do you have some recommended test instructions? Riku On 12/23/2010 04:09 PM, Peter Maydell wrote: Riku, did you want to pick this one up as a linux-user patch? Otherwise it's on my list of patches which got no comments and will go into the

[Qemu-devel] [Bug 604872] Re: qemu-system-arm segfaults emulating versatile machine after running debootstrap --second-stage inside vm

2011-01-07 Thread Peter Maydell
I've now posted this patchset; it comes in 7 parts: http://patchwork.ozlabs.org/patch/77887/ http://patchwork.ozlabs.org/patch/77882/ http://patchwork.ozlabs.org/patch/77884/ http://patchwork.ozlabs.org/patch/77885/ http://patchwork.ozlabs.org/patch/77888/ http://patchwork.ozlabs.org/patch/77881/

[Qemu-devel] Re: [PATCH] savevm: print migration failure to stderr rather than monitor

2011-01-07 Thread Alex Williamson
On Fri, 2011-01-07 at 09:51 +0100, Jan Kiszka wrote: Am 07.01.2011 08:18, Alex Williamson wrote: monitor_print only does anything for foreground commands, so we don't ever see this error message in the case of a 'migrate -d'. Your change needlessly steals the error from the monitor console

[Qemu-devel] Re: [PATCH] savevm: print migration failure to stderr rather than monitor

2011-01-07 Thread Jan Kiszka
Am 07.01.2011 16:39, Alex Williamson wrote: On Fri, 2011-01-07 at 09:51 +0100, Jan Kiszka wrote: Am 07.01.2011 08:18, Alex Williamson wrote: monitor_print only does anything for foreground commands, so we don't ever see this error message in the case of a 'migrate -d'. Your change needlessly

Re: [Qemu-devel] [PATCH 1/2] softfloat: Implement floatx80_is_any_nan() and float128_is_any_nan()

2011-01-07 Thread Peter Maydell
On 7 January 2011 15:28, Aurelien Jarno aurel...@aurel32.net wrote: On Thu, Jan 06, 2011 at 06:34:43PM +, Peter Maydell wrote: Implement versions of float*_is_any_nan() for the floatx80 and float128 types. Signed-off-by: Peter Maydell peter.mayd...@linaro.org ---  fpu/softfloat.h |   11

Re: [Qemu-devel] [PATCH 3/3] tcg/arm: improve constant loading

2011-01-07 Thread andrzej zaborowski
On 7 January 2011 15:40, Aurelien Jarno aurel...@aurel32.net wrote: On Fri, Jan 07, 2011 at 01:52:25PM +0100, andrzej zaborowski wrote: Hi, On 6 January 2011 22:54, Aurelien Jarno aurel...@aurel32.net wrote: Improve constant loading in two ways: - On all ARM versions, it's possible to load

[Qemu-devel] Re: [PATCH] savevm: print migration failure to stderr rather than monitor

2011-01-07 Thread Alex Williamson
On Fri, 2011-01-07 at 16:46 +0100, Jan Kiszka wrote: Am 07.01.2011 16:39, Alex Williamson wrote: On Fri, 2011-01-07 at 09:51 +0100, Jan Kiszka wrote: Am 07.01.2011 08:18, Alex Williamson wrote: monitor_print only does anything for foreground commands, so we don't ever see this error

Re: [Qemu-devel] [PATCH 0/2] ARM: fix commandline handling forsemihosted executables

2011-01-07 Thread Peter Maydell
On 7 January 2011 15:46, Schildbach, Wolfgang ws...@dolby.com wrote: I usually compile a hello world (echoing its arguments) with rvds2.2 or above: If you don't have an ARM compiler available, I could send you the compiled binary to test. Peter, is there another way to generate binaries

[Qemu-devel] qemu-kvm vs. qemu: Terminate cpu loop on reset?

2011-01-07 Thread Jan Kiszka
Hi, does anyone immediately know if this hunk from vl.c @@ -1278,6 +1197,10 @@ void qemu_system_reset_request(void) } else { reset_requested = 1; } +if (cpu_single_env) { +cpu_single_env-stopped = 1; +cpu_exit(cpu_single_env); +}

[Qemu-devel] [PATCH V2] savevm: use error_report for vmstate_save error

2011-01-07 Thread Alex Williamson
If migration is done in the background with the -d option, mon is NULL and this error message is lost. Instead use error_report(). Signed-off-by: Alex Williamson alex.william...@redhat.com --- savevm.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/savevm.c

Re: [Qemu-devel] [PATCH 0/7] target-arm: Translate based on TB flags, not CPUState

2011-01-07 Thread Aurelien Jarno
Hi, On Fri, Jan 07, 2011 at 03:06:27PM +, Peter Maydell wrote: This patchset corrects a number of places in the ARM translation code which were generating code which was dependent on values in the CPUState structure which might change at runtime. This is a bad idea for two reasons.

Re: [Qemu-devel] [PATCH 1/7] target-arm: Don't generate code specific to current CPU mode for SRS

2011-01-07 Thread Aurelien Jarno
On Fri, Jan 07, 2011 at 03:06:28PM +, Peter Maydell wrote: When translating the SRS instruction, handle the store registers to stack of current mode case in the helper function rather than inline. This means the generated code does not make assumptions about the current CPU mode which

[Qemu-devel] Re: [PATCH V2] savevm: use error_report for vmstate_save error

2011-01-07 Thread Jan Kiszka
Am 07.01.2011 16:58, Alex Williamson wrote: If migration is done in the background with the -d option, mon is NULL and this error message is lost. Instead use error_report(). Signed-off-by: Alex Williamson alex.william...@redhat.com Reviewed-by: Jan Kiszka jan.kis...@siemens.com As already

[Qemu-devel] Re: [PATCH V2] savevm: use error_report for vmstate_save error

2011-01-07 Thread Alex Williamson
On Fri, 2011-01-07 at 17:03 +0100, Jan Kiszka wrote: Am 07.01.2011 16:58, Alex Williamson wrote: If migration is done in the background with the -d option, mon is NULL and this error message is lost. Instead use error_report(). Signed-off-by: Alex Williamson alex.william...@redhat.com

[Qemu-devel] [Bug 524447] Re: virsh save is very slow

2011-01-07 Thread Serge Hallyn
Hi, please test the qemu-kvm packages in ppa:serge-hallyn/virt for lucid (0.12.3+noroms-0ubuntu10slowsave2) and maverick (0.12.5+noroms- 0ubuntu7slowsave2), which have the proposed patch from upstream. If they succeed, then I will proceed with the SRU. -- You received this bug notification

Re: [Qemu-devel] [PATCH 0/7] target-arm: Translate based on TB flags, not CPUState

2011-01-07 Thread Peter Maydell
On 7 January 2011 16:01, Aurelien Jarno aurel...@aurel32.net wrote: In overall I think it's the correct approach to fix the issue, this is a really good cleanup. I have tested this patch series, and it clearly improve armv7 support. However I am surprised it doesn't fix the issue mentioned in

Re: [Qemu-devel] [PATCH 0/7] target-arm: Translate based on TB flags, not CPUState

2011-01-07 Thread Peter Maydell
On 7 January 2011 16:01, Aurelien Jarno aurel...@aurel32.net wrote: My other concern is about the definition of the individual bits in the flags. I have seen that you have tried to summarize the usage in the patch 6, but the masks and shifts are still duplicated in different files, which may

Re: [Qemu-devel] [PATCH] lsi53c895a: fix endianness issues

2011-01-07 Thread Stefan Hajnoczi
On Fri, Jan 7, 2011 at 2:43 PM, y...@volta.aurel32.net wrote: From: Aurelien Jarno aurel...@aurel32.net lsi_ram_read*() and lsi_ram_write*() are not consistent, one uses leXX_to_cpu() the other uses nothing. As the comment above the RAM declaration says: Script ram is stored as 32-bit words

Re: [Qemu-devel] [PATCH 0/7] target-arm: Translate based on TB flags, not CPUState

2011-01-07 Thread David Turner
For what is worth, this was fixed in the Android emulator with the following patch: http://android.git.kernel.org/?p=platform/external/qemu.git;a=commit;h=01e9608cb62901d13b330f851a260a2082e81a06

[Qemu-devel] Re: [PATCH V2] savevm: use error_report for vmstate_save error

2011-01-07 Thread Jan Kiszka
Am 07.01.2011 17:10, Alex Williamson wrote: On Fri, 2011-01-07 at 17:03 +0100, Jan Kiszka wrote: Am 07.01.2011 16:58, Alex Williamson wrote: If migration is done in the background with the -d option, mon is NULL and this error message is lost. Instead use error_report(). Signed-off-by: Alex

Re: [Qemu-devel] [PATCH 0/2] ARM: fix commandline handling forsemihosted executables

2011-01-07 Thread riku voipio
On 01/07/2011 05:56 PM, Peter Maydell wrote: Or you can just test by hand: Got it. For anyone wondering into this thread, if you use ubuntu/linaro toolchains to compile the example below, you need a -marm flag for the compiler or you'll get a weird error on swi... Looks good, I'll add

Re: [Qemu-devel] [PATCH] lsi53c895a: fix endianness issues

2011-01-07 Thread Aurelien Jarno
On Fri, Jan 07, 2011 at 04:21:29PM +, Stefan Hajnoczi wrote: On Fri, Jan 7, 2011 at 2:43 PM, y...@volta.aurel32.net wrote: From: Aurelien Jarno aurel...@aurel32.net lsi_ram_read*() and lsi_ram_write*() are not consistent, one uses leXX_to_cpu() the other uses nothing. As the comment

[Qemu-devel] Re: qemu-kvm vs. qemu: Terminate cpu loop on reset?

2011-01-07 Thread Gleb Natapov
On Fri, Jan 07, 2011 at 04:57:31PM +0100, Jan Kiszka wrote: Hi, does anyone immediately know if this hunk from vl.c @@ -1278,6 +1197,10 @@ void qemu_system_reset_request(void) } else { reset_requested = 1; } +if (cpu_single_env) { +cpu_single_env-stopped

Re: [Qemu-devel] [PATCH 1/7] target-arm: Don't generate code specific to current CPU mode for SRS

2011-01-07 Thread Peter Maydell
On 7 January 2011 16:01, Aurelien Jarno aurel...@aurel32.net wrote: On Fri, Jan 07, 2011 at 03:06:28PM +, Peter Maydell wrote: When translating the SRS instruction, handle the store registers to stack of current mode case in the helper function rather than inline. This means the generated

[Qemu-devel] Re: qemu-kvm vs. qemu: Terminate cpu loop on reset?

2011-01-07 Thread Gleb Natapov
On Fri, Jan 07, 2011 at 05:59:34PM +0100, Jan Kiszka wrote: Am 07.01.2011 17:53, Gleb Natapov wrote: On Fri, Jan 07, 2011 at 04:57:31PM +0100, Jan Kiszka wrote: Hi, does anyone immediately know if this hunk from vl.c @@ -1278,6 +1197,10 @@ void qemu_system_reset_request(void) }

[Qemu-devel] Linux as VirtualBox quest OS with QEMU running Solaris

2011-01-07 Thread Mateusz Loskot
Hi, First, I'm sorry if my question does not belong here. The qemu-devel says it's devel, but I can't find any qemu-users mailing list. I have no experience with QEMU. I've been using x86-only virtualization software like VirtualBox, VMWare and others. I need to run Solaris (SPARC) OS and I'd

[Qemu-devel] Re: qemu-kvm vs. qemu: Terminate cpu loop on reset?

2011-01-07 Thread Jan Kiszka
Am 07.01.2011 18:16, Gleb Natapov wrote: On Fri, Jan 07, 2011 at 05:59:34PM +0100, Jan Kiszka wrote: Am 07.01.2011 17:53, Gleb Natapov wrote: On Fri, Jan 07, 2011 at 04:57:31PM +0100, Jan Kiszka wrote: Hi, does anyone immediately know if this hunk from vl.c @@ -1278,6 +1197,10 @@ void

Re: [Qemu-devel] Linux as VirtualBox quest OS with QEMU running Solaris

2011-01-07 Thread Michal Suchanek
On 7 January 2011 18:28, Mateusz Loskot mate...@loskot.net wrote: Hi, First, I'm sorry if my question does not belong here. The qemu-devel says it's devel, but I can't find any qemu-users mailing list. I have seen it once but could not find it either when searching for a qemu mailing list to

Re: [Qemu-devel] [PATCH 0/7] target-arm: Translate based on TB flags, not CPUState

2011-01-07 Thread Peter Maydell
On 7 January 2011 16:01, Aurelien Jarno aurel...@aurel32.net wrote: My other concern is about the definition of the individual bits in the flags. I have seen that you have tried to summarize the usage in the patch 6, but the masks and shifts are still duplicated in different files, which may

[Qemu-devel] Re: qemu-kvm vs. qemu: Terminate cpu loop on reset?

2011-01-07 Thread Gleb Natapov
On Fri, Jan 07, 2011 at 06:30:57PM +0100, Jan Kiszka wrote: Am 07.01.2011 18:16, Gleb Natapov wrote: On Fri, Jan 07, 2011 at 05:59:34PM +0100, Jan Kiszka wrote: Am 07.01.2011 17:53, Gleb Natapov wrote: On Fri, Jan 07, 2011 at 04:57:31PM +0100, Jan Kiszka wrote: Hi, does anyone

[Qemu-devel] [PATCH 1/4] qemu-doc: Clean whitespace

2011-01-07 Thread Stefan Weil
Remove blanks at line endings. Signed-off-by: Stefan Weil w...@mail.berlios.de --- qemu-doc.texi |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/qemu-doc.texi b/qemu-doc.texi index 7ce8999..21d8a82 100644 --- a/qemu-doc.texi +++ b/qemu-doc.texi @@ -832,7 +832,7 @@

[Qemu-devel] [PATCH 3/4] qemu-doc: Add missing blanks

2011-01-07 Thread Stefan Weil
Signed-off-by: Stefan Weil w...@mail.berlios.de --- qemu-doc.texi |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/qemu-doc.texi b/qemu-doc.texi index 744ba73..a1e6130 100644 --- a/qemu-doc.texi +++ b/qemu-doc.texi @@ -203,7 +203,7 @@ Intel 82801AA AC97 Audio compatible

[Qemu-devel] [PATCH 4/4] qemu-doc: Spelling fixes

2011-01-07 Thread Stefan Weil
neccessary - necessary Keberos - Kerberos emuilated - emulated transciever - transceiver emulaton - emulation inital - initial MingGW - MinGW Signed-off-by: Stefan Weil w...@mail.berlios.de --- qemu-doc.texi | 20 ++-- 1 files changed, 10 insertions(+), 10 deletions(-) diff

[Qemu-devel] [PATCH 2/4] qemu-doc: Add missing menu entry

2011-01-07 Thread Stefan Weil
Each @section should have a menu entry and a @node entry. Signed-off-by: Stefan Weil w...@mail.berlios.de --- qemu-doc.texi |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/qemu-doc.texi b/qemu-doc.texi index 21d8a82..744ba73 100644 --- a/qemu-doc.texi +++

Re: [Qemu-devel] Linux as VirtualBox quest OS with QEMU running Solaris

2011-01-07 Thread Stefan Weil
Am 07.01.2011 18:28, schrieb Mateusz Loskot: Hi, First, I'm sorry if my question does not belong here. The qemu-devel says it's devel, but I can't find any qemu-users mailing list. I have no experience with QEMU. I've been using x86-only virtualization software like VirtualBox, VMWare and

[Qemu-devel] Re: qemu-kvm vs. qemu: Terminate cpu loop on reset?

2011-01-07 Thread Jan Kiszka
Am 07.01.2011 18:53, Gleb Natapov wrote: On Fri, Jan 07, 2011 at 06:30:57PM +0100, Jan Kiszka wrote: Am 07.01.2011 18:16, Gleb Natapov wrote: On Fri, Jan 07, 2011 at 05:59:34PM +0100, Jan Kiszka wrote: Am 07.01.2011 17:53, Gleb Natapov wrote: On Fri, Jan 07, 2011 at 04:57:31PM +0100, Jan

[Qemu-devel] Re: qemu-kvm vs. qemu: Terminate cpu loop on reset?

2011-01-07 Thread Jan Kiszka
Am 07.01.2011 19:24, Jan Kiszka wrote: Am 07.01.2011 18:53, Gleb Natapov wrote: On Fri, Jan 07, 2011 at 06:30:57PM +0100, Jan Kiszka wrote: Am 07.01.2011 18:16, Gleb Natapov wrote: On Fri, Jan 07, 2011 at 05:59:34PM +0100, Jan Kiszka wrote: Am 07.01.2011 17:53, Gleb Natapov wrote: On Fri,

[Qemu-devel] [PATCH v3] savevm: Fix no_migrate

2011-01-07 Thread Alex Williamson
The no_migrate save state flag is currently only checked in the last phase of migration. This means that we potentially waste a lot of time and bandwidth with the live state handlers before we ever check the no_migrate flags. The error message printed when we catch a non-migratable device

[Qemu-devel] Re: [PATCH V2] savevm: use error_report for vmstate_save error

2011-01-07 Thread Alex Williamson
On Fri, 2011-01-07 at 17:27 +0100, Jan Kiszka wrote: Am 07.01.2011 17:10, Alex Williamson wrote: On Fri, 2011-01-07 at 17:03 +0100, Jan Kiszka wrote: Am 07.01.2011 16:58, Alex Williamson wrote: If migration is done in the background with the -d option, mon is NULL and this error message

[Qemu-devel] Re: [PATCH v3] savevm: Fix no_migrate

2011-01-07 Thread Jan Kiszka
Am 07.01.2011 19:39, Alex Williamson wrote: The no_migrate save state flag is currently only checked in the last phase of migration. This means that we potentially waste a lot of time and bandwidth with the live state handlers before we ever check the no_migrate flags. The error message

[Qemu-devel] Re: qemu-kvm vs. qemu: Terminate cpu loop on reset?

2011-01-07 Thread Gleb Natapov
On Fri, Jan 07, 2011 at 07:24:00PM +0100, Jan Kiszka wrote: Am 07.01.2011 18:53, Gleb Natapov wrote: On Fri, Jan 07, 2011 at 06:30:57PM +0100, Jan Kiszka wrote: Am 07.01.2011 18:16, Gleb Natapov wrote: On Fri, Jan 07, 2011 at 05:59:34PM +0100, Jan Kiszka wrote: Am 07.01.2011 17:53, Gleb

[Qemu-devel] Re: qemu-kvm vs. qemu: Terminate cpu loop on reset?

2011-01-07 Thread Jan Kiszka
Am 07.01.2011 20:10, Gleb Natapov wrote: We are on a good track now. I predict that we will be left with only one or two major additional features in qemu-kvm in a few months from now, no more duplications with subtle differences, and production-grade kvm upstream stability. You are

[Qemu-devel] [PATCH] Fix trivial endianness bugs

2011-01-07 Thread Stefan Weil
Replace endianess - endianness. Signed-off-by: Stefan Weil w...@mail.berlios.de --- audio/sdlaudio.c| 20 ++-- block/vdi.c |4 ++-- hw/xilinx_ethlite.c |2 +- target-mips/cpu.h |2 +- usb-bsd.c |2 +- 5 files changed, 15 insertions(+), 15

[Qemu-devel] [PATCH] qemu-tech: Spelling fixes

2011-01-07 Thread Stefan Weil
Signed-off-by: Stefan Weil w...@mail.berlios.de --- qemu-tech.texi |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/qemu-tech.texi b/qemu-tech.texi index 2e2a081..138e3ce 100644 --- a/qemu-tech.texi +++ b/qemu-tech.texi @@ -516,7 +516,7 @@ timers, especially together

[Qemu-devel] [PATCH] tcg/README: Spelling fixes

2011-01-07 Thread Stefan Weil
Signed-off-by: Stefan Weil w...@mail.berlios.de --- tcg/README |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/tcg/README b/tcg/README index 68d27ff..a2b69dd 100644 --- a/tcg/README +++ b/tcg/README @@ -75,11 +75,11 @@ destroyed, but local temporaries and globals

Re: [Qemu-devel] Linux as VirtualBox quest OS with QEMU running Solaris

2011-01-07 Thread Blue Swirl
On Fri, Jan 7, 2011 at 5:28 PM, Mateusz Loskot mate...@loskot.net wrote: Hi, First, I'm sorry if my question does not belong here. The qemu-devel says it's devel, but I can't find any qemu-users mailing list. A forum exists: http://qemu-forum.ipi.fi/ I have no experience with QEMU. I've

[Qemu-devel] [PATCH 4/7] softfloat: Implement floatx80_is_any_nan() and float128_is_any_nan()

2011-01-07 Thread Riku Voipio
From: Peter Maydell peter.mayd...@linaro.org Implement versions of float*_is_any_nan() for the floatx80 and float128 types. Acked-by: Aurelien Jarno aurel...@aurel32.net Signed-off-by: Peter Maydell peter.mayd...@linaro.org Signed-off-by: Riku Voipio riku.voi...@iki.fi --- fpu/softfloat.h |

[Qemu-devel] [PATCH 2/7] linux-user: Support ioctls whose parameter size is not constant

2011-01-07 Thread Riku Voipio
From: Peter Maydell peter.mayd...@linaro.org Some ioctls (for example FS_IOC_FIEMAP) use structures whose size is not constant. The generic argument conversion code in do_ioctl() cannot handle this, so add support for implementing a special-case handler for a particular ioctl which does the

[Qemu-devel] [PATCH 0/7] linux-user fixes for pull

2011-01-07 Thread Riku Voipio
From: Riku Voipio riku.voi...@nokia.com The following changes since commit 2a704b137f1acfbd972aa6e9f031c5015c7e28cb: cris: Avoid useless tmp in t_gen_cc_jmp() (2011-01-07 12:50:38 +0100) are available in the git repository at: git://gitorious.org/qemu-maemo/qemu.git linux-user-for-upstream

[Qemu-devel] [PATCH 5/7] linux-user: Fix incorrect NaN detection in ARM nwfpe emulation

2011-01-07 Thread Riku Voipio
From: Peter Maydell peter.mayd...@linaro.org The code in the linux-user ARM nwfpe emulation was incorrectly checking only for quiet NaNs when it should have been checking for any kind of NaN. This is probably because the code in question was taken from the Linux kernel, whose copy of the

[Qemu-devel] [PATCH 3/7] linux-user: Implement FS_IOC_FIEMAP ioctl

2011-01-07 Thread Riku Voipio
From: Peter Maydell peter.mayd...@linaro.org Implement the FS_IOC_FIEMAP ioctl using the new support for custom handling of ioctls; this is needed because the struct that is passed includes a variable-length array. Signed-off-by: Peter Maydell peter.mayd...@linaro.org Signed-off-by: Riku Voipio

[Qemu-devel] [PATCH 6/7] Fix commandline handling for ARM semihosted executables

2011-01-07 Thread Riku Voipio
From: Wolfgang Schildbach ws...@dolby.com Use the copy of the command line that loader_build_argptr() sets up in guest memory as the command line to return from the ARM SYS_GET_CMDLINE semihosting call. Previously we were using a pointer to memory which had already been freed before the guest

[Qemu-devel] Re: qemu-kvm vs. qemu: Terminate cpu loop on reset?

2011-01-07 Thread Gleb Natapov
On Fri, Jan 07, 2011 at 08:33:20PM +0100, Jan Kiszka wrote: Am 07.01.2011 20:10, Gleb Natapov wrote: We are on a good track now. I predict that we will be left with only one or two major additional features in qemu-kvm in a few months from now, no more duplications with subtle differences,

[Qemu-devel] [PATCH v4] savevm: Fix no_migrate

2011-01-07 Thread Alex Williamson
The no_migrate save state flag is currently only checked in the last phase of migration. This means that we potentially waste a lot of time and bandwidth with the live state handlers before we ever check the no_migrate flags. The error message printed when we catch a non-migratable device

[Qemu-devel] [PATCH 6/7] target-i386: Use deposit operation.

2011-01-07 Thread Richard Henderson
Use this for assignment to the low byte or low word of a register. Signed-off-by: Richard Henderson r...@twiddle.net --- target-i386/translate.c | 34 ++ 1 files changed, 6 insertions(+), 28 deletions(-) diff --git a/target-i386/translate.c

[Qemu-devel] [PATCH 7/7] target-ppc: Use deposit operation.

2011-01-07 Thread Richard Henderson
Use this in implementing rl[wd]imi, at least for the cases that don't require true rotation. Signed-off-by: Richard Henderson r...@twiddle.net --- target-ppc/translate.c | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/target-ppc/translate.c

[Qemu-devel] [PATCH 0/7] Define deposit tcg operation

2011-01-07 Thread Richard Henderson
Emulating i386 -- particularly in 16-bit mode -- requires quite a few bitfield insert operations, to handle byte and word stores into the dword registers. On several hosts, this can be done natively, without resorting to a sequence of and and or instructions. Even i386 as a host can do better

[Qemu-devel] [PATCH 4/7] tcg-ia64: Implement deposit operation.

2011-01-07 Thread Richard Henderson
Signed-off-by: Richard Henderson r...@twiddle.net --- tcg/ia64/tcg-target.c | 92 + tcg/ia64/tcg-target.h |2 + 2 files changed, 94 insertions(+), 0 deletions(-) diff --git a/tcg/ia64/tcg-target.c b/tcg/ia64/tcg-target.c index

[Qemu-devel] [PATCH 2/7] tcg-ppc: Implement deposit operation.

2011-01-07 Thread Richard Henderson
Signed-off-by: Richard Henderson r...@twiddle.net --- tcg/ppc/tcg-target.c | 17 - tcg/ppc/tcg-target.h |1 + 2 files changed, 17 insertions(+), 1 deletions(-) diff --git a/tcg/ppc/tcg-target.c b/tcg/ppc/tcg-target.c index 7970268..2947d1b 100644 --- a/tcg/ppc/tcg-target.c

[Qemu-devel] [PATCH 1/7] tcg: Define deposit as an optional operation.

2011-01-07 Thread Richard Henderson
Signed-off-by: Richard Henderson r...@twiddle.net --- tcg/README| 14 ++ tcg/tcg-op.h | 40 tcg/tcg-opc.h |6 ++ tcg/tcg.c | 13 + 4 files changed, 73 insertions(+), 0 deletions(-) diff --git a/tcg/README

[Qemu-devel] [PATCH 5/7] tcg-i386: Implement deposit operation.

2011-01-07 Thread Richard Henderson
Special case deposits that are implementable with byte and word stores. Otherwise implement with double-word shift plus rotates. Expose tcg_reg_alloc to the backend for allocation of scratch registers. There's an edge condition that cannot actually happen at the moment due to a bug elsewhere in

Re: [Qemu-devel] [PATCH 0/7] Define deposit tcg operation

2011-01-07 Thread Peter Maydell
On 7 January 2011 22:42, Richard Henderson r...@twiddle.net wrote: Emulating i386 -- particularly in 16-bit mode -- requires quite a few bitfield insert operations, to handle byte and word stores into the dword registers.  On several hosts, this can be done natively, without resorting to a

[Qemu-devel] Re: [PATCH 26/35] kvm: Eliminate KVMState arguments

2011-01-07 Thread Anthony Liguori
On 01/07/2011 03:03 AM, Jan Kiszka wrote: Am 06.01.2011 20:24, Anthony Liguori wrote: On 01/06/2011 11:56 AM, Marcelo Tosatti wrote: From: Jan Kiszkajan.kis...@siemens.com QEMU supports only one VM, so there is only one kvm_state per process, and we gain nothing passing a reference

[Qemu-devel] Re: [PATCH 2/7] tcg-ppc: Implement deposit operation.

2011-01-07 Thread malc
On Fri, 7 Jan 2011, Richard Henderson wrote: Signed-off-by: Richard Henderson r...@twiddle.net --- tcg/ppc/tcg-target.c | 17 - tcg/ppc/tcg-target.h |1 + 2 files changed, 17 insertions(+), 1 deletions(-) PPC bits look fine to me. [..snip..] --

Re: [Qemu-devel] [PATCH 3/7] tcg-hppa: Implement deposit operation.

2011-01-07 Thread Stuart Brady
On Fri, Jan 07, 2011 at 02:42:59PM -0800, Richard Henderson wrote: Signed-off-by: Richard Henderson r...@twiddle.net --- tcg/hppa/tcg-target.c | 58 +++- tcg/hppa/tcg-target.h |1 + 2 files changed, 53 insertions(+), 6 deletions(-) diff

  1   2   >