[Qemu-devel] Any other cross ISA and system mode binary translator like QEMU?

2012-10-01 Thread Wei-Ren Chen
Hi all, Most binary translators I am aware of are user mode, i.e., they are only used to run guest application not OS. The system mode binary translators which do cross ISA translation I know so far are Transmeta CMS, IBM DAISY and QEMU. Any others out there? Thanks. Regards, chenwj [1] -- W

Re: [Qemu-devel] [PATCH V4 2/5] xen: Introduce xen_modified_memory.

2012-10-01 Thread Stefano Stabellini
On Thu, 27 Sep 2012, Anthony PERARD wrote: > This function is to be used during live migration. Every write access to the > guest memory should call this funcion so the Xen tools knows which pages are > dirty. > > Signed-off-by: Anthony PERARD Acked-by: Stefano Stabellini > hw/xen.h | 1 +

Re: [Qemu-devel] [PATCH V4 5/5] xen: Set the vram dirty when an error occur.

2012-10-01 Thread Stefano Stabellini
On Thu, 27 Sep 2012, Anthony PERARD wrote: > If the call to xc_hvm_track_dirty_vram() fails, then we set dirtybit on all > the > video ram. This case happens during migration. > > Signed-off-by: Anthony PERARD Acked-by: Stefano Stabellini > xen-all.c | 3 ++- > 1 file changed, 2 insertions(+

Re: [Qemu-devel] [PATCH V4 4/5] exec, memory: Call to xen_modified_memory.

2012-10-01 Thread Stefano Stabellini
On Thu, 27 Sep 2012, Anthony PERARD wrote: > This patch add some calls to xen_modified_memory to notify Xen about dirtybits > during migration. > > Signed-off-by: Anthony PERARD If I am not mistaken, this is the last patch that needs reviewing. Avi, are you OK with it? > exec.c | 1 + > me

Re: [Qemu-devel] a user here - pci-assign

2012-10-01 Thread lejeczek
hmm, still cannot get readon 5450 to work on win7-64, have changed -cpu to host but no fix no vfio in qemu-kvm-0.12.1.2-2.295.el6_3.2.x86_64 yes, I do blacklist modules at grub level and later in modprobe also is primary/secondary VGA setup somehow helped by qemu/components? in guest I can do

Re: [Qemu-devel] pwrite64 error because of argument position

2012-10-01 Thread Alexander Graf
On 30.09.2012, at 20:50, Alex Barcelo wrote: > This error may be a PPC specific problem, but I don't have another > environment where I can test it (i386 doesn't seem to use pwrite64), > so I ask for a bit of help/check. > > I am in a 32bit linux testing the qemu-ppc. > > My test program: > >

Re: [Qemu-devel] [Bug 1058225] [NEW] When setting hardware clock on linux guest, hwclock shows crazy date (in the year 2043)

2012-10-01 Thread Paolo Bonzini
Il 28/09/2012 19:21, Lucas Meneghel Rodrigues ha scritto: > Public bug reported: > > Very easy to reproduce: > > 1) Build the latest qemu.git (we've captured this on internal automated > testing, verified manually), the commit for reference is: > > 14:07:02 INFO | git commit ID is > 6f8fd2530e9a

[Qemu-devel] [PATCH 1/2] linux-user: ppc: mark as long long aligned

2012-10-01 Thread Alexander Graf
The PPC32 ABI dictates that long long (64bit) parameters are pass in odd/even register pairs. Because unlike ARM and MIPS we start at an odd register number, we can reuse the same aligning code that ARM and MIPS use. Signed-off-by: Alexander Graf --- linux-user/syscall.c |6 +- 1 files c

[Qemu-devel] [PATCH 2/2] linux-user: register align p{read, write}64

2012-10-01 Thread Alexander Graf
pread64 and pwrite64 pass 64bit parameters which for some architectures need to be aligned to special argument pairs, creating a gap argument. Handle this special case the same way we handle it in other places of the code. Reported-by: Alex Barcelo Signed-off-by: Alexander Graf --- linux-user/

[Qemu-devel] [PATCH] hw/arm_gic.c: Fix improper DPRINTF output.

2012-10-01 Thread Evgeny Voevodin
s->cpu_enabled is a massive, so s->cpu_enabled ? "En" : "Dis" returns "En" always. We should use s->cpu_enabled[cpu] here. Signed-off-by: Evgeny Voevodin --- hw/arm_gic.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/arm_gic.c b/hw/arm_gic.c index 55871fa..4024dae 100

[Qemu-devel] [PATCH] qtest: implement QTEST_STOP

2012-10-01 Thread Paolo Bonzini
It is quite difficult to debug qtest test cases without extra wrapper scripts for QEMU or similar. This patch adds a simple environment variable-based trigger that sends a STOP signal to the QEMU instance under test, before attempting to connect to its QMP session. This will block execution of th

[Qemu-devel] [PATCH 0/3] Fix autotest-exposed RTC bug 1058225

2012-10-01 Thread Paolo Bonzini
> Very easy to reproduce: > > 1) Build the latest qemu.git (we've captured this on internal automated > testing, verified manually), the commit for reference is: > > 14:07:02 INFO | git commit ID is > 6f8fd2530e9a530f237240daf1c981fa5df7f978 (tag v1.2.0-461-g6f8fd25) > > 2) Install a linux guest in

[Qemu-devel] [PATCH 1/3] rtc: fix overflow in mktimegm

2012-10-01 Thread Paolo Bonzini
When setting a date in 1980, Linux is actually disregarding the century byte and setting the year to 2080. This causes a year-2038 overflow in mktimegm. Fix this by doing the days-to-seconds computation in 64-bit math. Reported-by: Lucas Meneghel Rodrigues Signed-off-by: Paolo Bonzini --- cut

[Qemu-devel] [PATCH 2/3] rtc: map CMOS index 0x37 to 0x32 on read and writes

2012-10-01 Thread Paolo Bonzini
QEMU's attempt to implement the century byte cover two possible places for the byte. A common one on modern chipsets is 0x32, but QEMU also stores the value in 0x37 (apparently for IBM PS/2 compatibility---it's only been 25 years). To simplify the implementation of the century byte, store it only

[Qemu-devel] [PATCH 3/3] rtc: implement century byte

2012-10-01 Thread Paolo Bonzini
Implement the century byte in the RTC emulation, and test that it works. This leads to some annoying compatibility code because we need to treat a value of 2000 for the base_year property as "use the century byte properly" (which would be a value of 0). The century byte will now be always-zero, ra

Re: [Qemu-devel] [PATCH] hw/arm_gic.c: Fix improper DPRINTF output.

2012-10-01 Thread Peter Maydell
On 1 October 2012 13:11, Evgeny Voevodin wrote: > s->cpu_enabled is a massive, so s->cpu_enabled ? "En" : "Dis" returns "is an array" > "En" always. We should use s->cpu_enabled[cpu] here. > > Signed-off-by: Evgeny Voevodin ...but nice catch. Reviewed-by: Peter Maydell -- PMM

Re: [Qemu-devel] [PATCH 18/22] target-i386: parse cpu_model string into set of stringified properties

2012-10-01 Thread Igor Mammedov
On Sun, 30 Sep 2012 12:16:27 + Blue Swirl wrote: > This would break build since strtok_r() isn't available on Mingw, it's > not Posix either. How about g_strsplit()? Thanks! Updated series at: https://github.com/imammedo/qemu/tree/x86-cpu-properties.WIP I'll post fixed [7/22] and [18/22] pa

[Qemu-devel] [PATCH 07/22 v4] target-i386: convert cpuid features into properties

2012-10-01 Thread Igor Mammedov
add property accessors for cpuid feature bits defined by *_feature_name arrays. Signed-off-by: Igor Mammedov --- v2: * replaced mask/ffs tricks by plain 'for (bit = 0; bit < 32; bit++)' as suggested by Eduardo Habkost v3: * check if property exists before adding it * rebased on top of

[Qemu-devel] [PATCH 18/22 v3] target-i386: parse cpu_model string into set of stringified properties

2012-10-01 Thread Igor Mammedov
cpu_model string does represent features in following format: ([+-]feat)|(feat=foo)|(feat) which makes it impossible directly use property infrastructure to set features on CPU. This patch introduces parser that splits CPU name from cpu_model and converts legacy features string into canonized set

Re: [Qemu-devel] [PATCH 1/2] linux-user: ppc: mark as long long aligned

2012-10-01 Thread Andreas Färber
Am 30.09.2012 03:32, schrieb Alexander Graf: > The PPC32 ABI dictates that long long (64bit) parameters are pass in odd/even > register pairs. Because unlike ARM and MIPS we start at an odd register > number, > we can reuse the same aligning code that ARM and MIPS use. > > Signed-off-by: Alexande

Re: [Qemu-devel] [PATCH 1/2] linux-user: ppc: mark as long long aligned

2012-10-01 Thread Alexander Graf
On 01.10.2012, at 15:04, Andreas Färber wrote: > Am 30.09.2012 03:32, schrieb Alexander Graf: >> The PPC32 ABI dictates that long long (64bit) parameters are pass in odd/even >> register pairs. Because unlike ARM and MIPS we start at an odd register >> number, >> we can reuse the same aligning c

Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility

2012-10-01 Thread Anthony Liguori
Jan Kiszka writes: > On 2012-10-01 11:31, Marcelo Tosatti wrote: > > It's not just about default configs. We need to validate if the > migration formats are truly compatible (qemu-kvm -> QEMU, the other way > around definitely not). For the command line switches, we could provide > a wrapper scri

Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility

2012-10-01 Thread Jan Kiszka
On 2012-10-01 15:19, Anthony Liguori wrote: > Jan Kiszka writes: > >> On 2012-10-01 11:31, Marcelo Tosatti wrote: >> >> It's not just about default configs. We need to validate if the >> migration formats are truly compatible (qemu-kvm -> QEMU, the other way >> around definitely not). For the com

Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility

2012-10-01 Thread Marcelo Tosatti
On Mon, Oct 01, 2012 at 08:19:29AM -0500, Anthony Liguori wrote: > Jan Kiszka writes: > > > On 2012-10-01 11:31, Marcelo Tosatti wrote: > > > > It's not just about default configs. We need to validate if the > > migration formats are truly compatible (qemu-kvm -> QEMU, the other way > > around de

Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility

2012-10-01 Thread Jan Kiszka
On 2012-10-01 15:31, Marcelo Tosatti wrote: > On Mon, Oct 01, 2012 at 08:19:29AM -0500, Anthony Liguori wrote: >> Jan Kiszka writes: >> >>> On 2012-10-01 11:31, Marcelo Tosatti wrote: >>> >>> It's not just about default configs. We need to validate if the >>> migration formats are truly compatible

Re: [Qemu-devel] [PULL for Kevin 00/19] Block job improvements part 1

2012-10-01 Thread Paolo Bonzini
Il 28/09/2012 20:23, Kevin Wolf ha scritto: > Am 28.09.2012 17:22, schrieb Paolo Bonzini: >> Kevin, >> >> the following changes since commit ac05f3492421caeb05809ffa02c6198ede179e43: >> >> add a boot parameter to set reboot timeout (2012-09-25 20:05:04 -0500) >> >> are available in the git reposi

Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility

2012-10-01 Thread Paolo Bonzini
Il 01/10/2012 15:19, Anthony Liguori ha scritto: > I think it's time for the distros to drop qemu-kvm and just ship > qemu.git. Is there anything else that needs to happen to make that > switch? Perhaps change the default to -machine accel=kvm:tcg? Paolo

Re: [Qemu-devel] [PATCH 2/2] cpu_physical_memory_write_rom() needs to do TB invalidates

2012-10-01 Thread Pavel Hrdina
On 09/12/2012 07:57 AM, David Gibson wrote: On Mon, Sep 10, 2012 at 03:27:45PM +0200, Andreas Färber wrote: Am 10.09.2012 04:30, schrieb David Gibson: cpu_physical_memory_write_rom(), despite the name, can also be used to write images into RAM - and will often be used that way if the machine us

Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility

2012-10-01 Thread Michael Tokarev
01.10.2012 17:36, Jan Kiszka wrote: > On 2012-10-01 15:31, Marcelo Tosatti wrote: >>> Default nic is rtl8139 vs. e1000. >> >> Config file (as suggested earlier on this thread). > > If you need to append -config bla, you can also specify the desired NIC > explicitly - I see no value in the former.

Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility

2012-10-01 Thread Jan Kiszka
On 2012-10-01 15:38, Paolo Bonzini wrote: > Il 01/10/2012 15:19, Anthony Liguori ha scritto: >> I think it's time for the distros to drop qemu-kvm and just ship >> qemu.git. Is there anything else that needs to happen to make that >> switch? > > Perhaps change the default to -machine accel=kvm:tc

Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility

2012-10-01 Thread Jan Kiszka
On 2012-10-01 15:44, Michael Tokarev wrote: > 01.10.2012 17:36, Jan Kiszka wrote: >> On 2012-10-01 15:31, Marcelo Tosatti wrote: > Default nic is rtl8139 vs. e1000. >>> >>> Config file (as suggested earlier on this thread). >> >> If you need to append -config bla, you can also specify the des

Re: [Qemu-devel] a user here - pci-assign

2012-10-01 Thread Alex Williamson
On Mon, 2012-10-01 at 12:10 +0100, lejeczek wrote: > hmm, still cannot get readon 5450 to work on win7-64, have > changed -cpu to host but no fix > no vfio in qemu-kvm-0.12.1.2-2.295.el6_3.2.x86_64 > yes, I do blacklist modules at grub level and later in > modprobe also You probably want to try

Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility

2012-10-01 Thread Anthony Liguori
Paolo Bonzini writes: > Il 01/10/2012 15:19, Anthony Liguori ha scritto: >> I think it's time for the distros to drop qemu-kvm and just ship >> qemu.git. Is there anything else that needs to happen to make that >> switch? > > Perhaps change the default to -machine accel=kvm:tcg? > > Paolo I wou

Re: [Qemu-devel] a user here - pci-assign

2012-10-01 Thread lejeczek
all these attempts and tests I've been doing in 2.6.39-200.32.1.el6uek.x86_64(oracle) now when I try rhel 2.6.32-279.5.1.el6.x86_64(everything else, hw&soft stay the same) I get: ,for win7-64 guest: pci-stub :26:00.0: irq 104 for MSI/MSI-X pci-stub :26:00.0: restoring config space at

Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility

2012-10-01 Thread Alexander Graf
On 01.10.2012, at 15:19, Anthony Liguori wrote: > Jan Kiszka writes: > >> On 2012-10-01 11:31, Marcelo Tosatti wrote: >> >> It's not just about default configs. We need to validate if the >> migration formats are truly compatible (qemu-kvm -> QEMU, the other way >> around definitely not). For

Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility (fwd)

2012-10-01 Thread Serge Hallyn
We (Ubuntu) plan to switch to qemu in the next release which opens in November. I suppose there's likely to be a hiccough or two, but I can't think of any offhand. -serge Quoting Scott Moser (smo...@ubuntu.com): > you should have been added here. > > > -- Forwarded message -- >

Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility

2012-10-01 Thread Paolo Bonzini
Il 01/10/2012 16:07, Alexander Graf ha scritto: >>> -drive ...,boot= <- this is ignored > > It's ignored, but useful for certain things. I don't know how many of > our users use boot= today, but it's certainly still in the code, and > supported. I honestly wouldn't mind to carry a SUSE specific pa

Re: [Qemu-devel] [PATCH] hw: Add support for new LSI Logic devices.

2012-10-01 Thread Paolo Bonzini
Il 29/09/2012 16:35, Don Slutz ha scritto: >> > How important is the big endian support? > > lsi53c895a.c says: > > /* ??? Need to check if the {read,write}[wl] routines work properly on >big-endian targets. */ > > I could do the same, I.E. code it up and submit it un-tested on big > endian

Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility (fwd)

2012-10-01 Thread Peter Maydell
On 1 October 2012 15:15, Serge Hallyn wrote: > We (Ubuntu) plan to switch to qemu in the next release which opens in > November. I suppose there's likely to be a hiccough or two, but I can't > think of any offhand. Are you planning to do that for all CPU target architectures, or to maintain the

Re: [Qemu-devel] [PATCH 2/2] cpu_physical_memory_write_rom() needs to do TB invalidates

2012-10-01 Thread Anthony Liguori
Pavel Hrdina writes: > On 09/12/2012 07:57 AM, David Gibson wrote: >> On Mon, Sep 10, 2012 at 03:27:45PM +0200, Andreas Färber wrote: >>> Am 10.09.2012 04:30, schrieb David Gibson: cpu_physical_memory_write_rom(), despite the name, can also be used to write images into RAM - and will of

[Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it

2012-10-01 Thread Jan Kiszka
If we built a target for a host that supports KVM in principle, set the default accelerator to KVM as well. This also means the start of QEMU will fail to start if KVM support turns out to be unavailable at runtime. Signed-off-by: Jan Kiszka --- kvm-all.c |1 + kvm-stub.c |1 + kvm.h

[Qemu-devel] [PATCH v2 0/9] Embedded NBD server

2012-10-01 Thread Paolo Bonzini
This series rebases the previous qemu-sockets patches for error propagation and uses the new QAPI-friendly socket functions in the embedded NBD server. The changes are due to Orit's patches being now in, some early parts being in Luiz's queue, and glusterfs patches not having touched qemu-sockets.

[Qemu-devel] [PATCH v2 1/9] build: add QAPI files to the tools

2012-10-01 Thread Paolo Bonzini
We need them because qemu-sockets will soon be using SocketAddress. Signed-off-by: Paolo Bonzini --- Makefile.objs | 3 ++- 1 file modificato, 2 inserzioni(+). 1 rimozione(-) diff --git a/Makefile.objs b/Makefile.objs index 4412757..03da150 100644 --- a/Makefile.objs +++ b/Makefile.objs @@ -47,

[Qemu-devel] [PATCH v2 3/9] qemu-sockets: add error propagation to inet_parse

2012-10-01 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- qemu-sockets.c | 41 + 1 file modificato, 21 inserzioni(+), 20 rimozioni(-) diff --git a/qemu-sockets.c b/qemu-sockets.c index 1f14e8b..bf1f794 100644 --- a/qemu-sockets.c +++ b/qemu-sockets.c @@ -511,7 +511,7 @@ err: }

[Qemu-devel] [PATCH v2 4/9] qemu-sockets: add error propagation to Unix socket functions

2012-10-01 Thread Paolo Bonzini
This patch starts harmonizing unix_* and inet_* functions. Signed-off-by: Paolo Bonzini --- nbd.c | 4 ++-- qemu-char.c | 4 ++-- qemu-sockets.c | 40 qemu_socket.h | 8 qga/channel-posix.c | 2 +- ui/vnc.c

[Qemu-devel] [PATCH v2 7/9] block: add close notifiers

2012-10-01 Thread Paolo Bonzini
The first user of close notifiers will be the embedded NBD server. It is possible to use them to do some of the ad hoc processing (e.g. for block jobs and I/O limits) that is currently done by bdrv_close. Acked-by: Kevin Wolf Signed-off-by: Paolo Bonzini --- Makefile.objs | 3 ++- block.c

[Qemu-devel] [PATCH v2 5/9] qemu-sockets: return IPSocketAddress from inet_parse

2012-10-01 Thread Paolo Bonzini
Prepare inet_parse so that it can be easily reused in socket_parse. socket_parse will be public and will help converting addresses accepted by HMP to SocketAddress structures accepted by the implementation of QMP commands. Signed-off-by: Paolo Bonzini --- qemu-sockets.c | 119 +++

[Qemu-devel] [PATCH v2 6/9] qemu-sockets: add socket_listen, socket_connect, socket_parse

2012-10-01 Thread Paolo Bonzini
These are QAPI-friendly versions of the qemu-sockets functions. They support IP sockets, Unix sockets, and named file descriptors, using a QAPI union to dispatch to the correct function. Signed-off-by: Paolo Bonzini --- qemu-sockets.c | 95 +++

[Qemu-devel] [PATCH v2 2/9] qapi: add socket address types

2012-10-01 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- qapi-schema.json | 53 + 1 file modificato, 53 inserzioni(+) diff --git a/qapi-schema.json b/qapi-schema.json index 191d921..b443a99 100644 --- a/qapi-schema.json +++ b/qapi-schema.json @@ -2367,6 +2367,59 @@

[Qemu-devel] [PATCH v2 9/9] hmp: add NBD server commands

2012-10-01 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- hmp-commands.hx | 29 + hmp.c | 55 +++ hmp.h | 2 ++ 3 file modificati, 86 inserzioni(+) diff --git a/hmp-commands.hx b/hmp-commands.hx index ed67e99..b0af484 10

[Qemu-devel] [PATCH v2 8/9] qmp: add NBD server commands

2012-10-01 Thread Paolo Bonzini
Adding an NBD server inside QEMU is trivial, since all the logic is in nbd.c and can be shared easily between qemu-nbd and QEMU itself. The main difference is that qemu-nbd serves a single unnamed export, while QEMU serves named exports. Signed-off-by: Paolo Bonzini --- Makefile.objs| 2 +-

Re: [Qemu-devel] [PATCH 2/2] cpu_physical_memory_write_rom() needs to do TB invalidates

2012-10-01 Thread Pavel Hrdina
On 10/01/2012 04:28 PM, Anthony Liguori wrote: Pavel Hrdina writes: On 09/12/2012 07:57 AM, David Gibson wrote: On Mon, Sep 10, 2012 at 03:27:45PM +0200, Andreas Färber wrote: Am 10.09.2012 04:30, schrieb David Gibson: cpu_physical_memory_write_rom(), despite the name, can also be used to w

Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility

2012-10-01 Thread Bruce Rogers
>>> On 10/1/2012 at 07:19 AM, Anthony Liguori wrote: > Jan Kiszka writes: > >> On 2012-10-01 11:31, Marcelo Tosatti wrote: >> >> It's not just about default configs. We need to validate if the >> migration formats are truly compatible (qemu-kvm -> QEMU, the other way >> around definitely not).

Re: [Qemu-devel] failed to fail over between two bonded interface under user mode network stack

2012-10-01 Thread Mulyadi Santosa
anybody could help? On Sun, Sep 30, 2012 at 3:12 PM, Mulyadi Santosa wrote: > Hi all... > > Recently, I did experiments to see what bonding can do. Therefore I > run Qemu using following command: > qemu-system-i386 -m 512 -hda ./centos.qcow2 \ > -net nic,model=e1000,vlan=0 -net user,vlan=0 \ >

Re: [Qemu-devel] [PATCH v2] x86: Implement SMEP and SMAP

2012-10-01 Thread Anthony Liguori
"H. Peter Anvin" writes: > From: "H. Peter Anvin" > > This patch implements Supervisor Mode Execution Prevention (SMEP) and > Supervisor Mode Access Prevention (SMAP) for x86. The purpose of the > patch, obviously, is to help kernel developers debug the support for > those features. > > A fair

Re: [Qemu-devel] [PATCH v6 0/4] VFIO-based PCI device assignment

2012-10-01 Thread Anthony Liguori
Alex Williamson writes: > v6: > Update patch 4/4 so Makefile just uses CONFIG_LINUX and > avoids all the noise in configure. > > Also available in git here: > > git://github.com/awilliam/qemu-vfio.git > branch: vfio-for-qemu > tag: vfio-pci-for-qemu-v6 Applied. Thanks. Regards, Anthony Lig

Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it

2012-10-01 Thread Anthony Liguori
Jan Kiszka writes: > If we built a target for a host that supports KVM in principle, set the > default accelerator to KVM as well. This also means the start of QEMU > will fail to start if KVM support turns out to be unavailable at > runtime. > > Signed-off-by: Jan Kiszka > --- > kvm-all.c |

Re: [Qemu-devel] [PATCH 2/2] cpu_physical_memory_write_rom() needs to do TB invalidates

2012-10-01 Thread Anthony Liguori
Pavel Hrdina writes: > On 10/01/2012 04:28 PM, Anthony Liguori wrote: >> Pavel Hrdina writes: >> >>> On 09/12/2012 07:57 AM, David Gibson wrote: On Mon, Sep 10, 2012 at 03:27:45PM +0200, Andreas Färber wrote: > Am 10.09.2012 04:30, schrieb David Gibson: >> cpu_physical_memory_write_

Re: [Qemu-devel] [PATCH 2/2] cpu_physical_memory_write_rom() needs to do TB invalidates

2012-10-01 Thread Pavel Hrdina
On 10/01/2012 06:21 PM, Anthony Liguori wrote: Pavel Hrdina writes: On 10/01/2012 04:28 PM, Anthony Liguori wrote: Pavel Hrdina writes: On 09/12/2012 07:57 AM, David Gibson wrote: On Mon, Sep 10, 2012 at 03:27:45PM +0200, Andreas Färber wrote: Am 10.09.2012 04:30, schrieb David Gibson:

Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it

2012-10-01 Thread Andreas Färber
Hello Jan, Am 01.10.2012 16:34, schrieb Jan Kiszka: > If we built a target for a host that supports KVM in principle, set the > default accelerator to KVM as well. This also means the start of QEMU > will fail to start if KVM support turns out to be unavailable at > runtime. >From a distro point

Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it

2012-10-01 Thread Daniel P. Berrange
On Mon, Oct 01, 2012 at 06:43:00PM +0200, Andreas Färber wrote: > Hello Jan, > > Am 01.10.2012 16:34, schrieb Jan Kiszka: > > If we built a target for a host that supports KVM in principle, set the > > default accelerator to KVM as well. This also means the start of QEMU > > will fail to start if

Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility (fwd)

2012-10-01 Thread Serge Hallyn
Quoting Peter Maydell (peter.mayd...@linaro.org): > On 1 October 2012 15:15, Serge Hallyn wrote: > > We (Ubuntu) plan to switch to qemu in the next release which opens in > > November. I suppose there's likely to be a hiccough or two, but I can't > > think of any offhand. > > Are you planning to

Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it

2012-10-01 Thread Aurelien Jarno
On Mon, Oct 01, 2012 at 11:20:41AM -0500, Anthony Liguori wrote: > Jan Kiszka writes: > > > If we built a target for a host that supports KVM in principle, set the > > default accelerator to KVM as well. This also means the start of QEMU > > will fail to start if KVM support turns out to be unava

Re: [Qemu-devel] [PATCH] fpu/softfloat.c: Return correctly signed values from uint64_to_float32

2012-10-01 Thread Aurelien Jarno
On Fri, Sep 28, 2012 at 04:17:03PM +0100, Peter Maydell wrote: > The uint64_to_float32() conversion function was incorrectly always > returning numbers with the sign bit set (ie negative numbers). Correct > this so we return positive numbers instead. > > Signed-off-by: Peter Maydell > --- > As fa

Re: [Qemu-devel] pwrite64 error because of argument position

2012-10-01 Thread Alex Barcelo
ok, thank you very much. Now I understand the problem... and I see the correct way of mending it. Your patch works for me. >On Mon, Oct 1, 2012 at 1:52 PM, Alexander Graf wrote: > > On 30.09.2012, at 20:50, Alex Barcelo wrote: > >> This error may be a PPC specific problem, but I don't have anoth

Re: [Qemu-devel] pwrite64 error because of argument position

2012-10-01 Thread Alexander Graf
On 01.10.2012, at 18:57, Alex Barcelo wrote: > ok, thank you very much. Now I understand the problem... and I see the > correct way of mending it. > > Your patch works for me. Awesome. Could you please put your Tested-by tag below the patches I sent to the ML earlier to day then? :) Alex

Re: [Qemu-devel] [PATCH] fpu/softfloat.c: Return correctly signed values from uint64_to_float32

2012-10-01 Thread Aurelien Jarno
On Fri, Sep 28, 2012 at 04:42:16PM +0100, Peter Maydell wrote: > On 28 September 2012 16:17, Peter Maydell wrote: > > The uint64_to_float32() conversion function was incorrectly always > > returning numbers with the sign bit set (ie negative numbers). Correct > > this so we return positive numbers

Re: [Qemu-devel] [PATCH v2 4/9] qemu-sockets: add error propagation to Unix socket functions

2012-10-01 Thread Luiz Capitulino
On Mon, 1 Oct 2012 16:52:19 +0200 Paolo Bonzini wrote: > This patch starts harmonizing unix_* and inet_* functions. > > Signed-off-by: Paolo Bonzini > --- > nbd.c | 4 ++-- > qemu-char.c | 4 ++-- > qemu-sockets.c | 40 > q

Re: [Qemu-devel] [PATCH v6 3/4] vfio: vfio-pci device assignment driver

2012-10-01 Thread Alex Barcelo
This: struct vfio_iommu_type1_dma_map map = { .argsz = sizeof(map), .flags = VFIO_DMA_MAP_FLAG_READ, .vaddr = (__u64)vaddr, .iova = iova, .size = size, }; (around line 771) breaks in my environment. I am in a crosschain environment on a i386 buildi

Re: [Qemu-devel] [PATCH 1/2] linux-user: ppc: mark as long long aligned

2012-10-01 Thread malc
On Sun, 30 Sep 2012, Alexander Graf wrote: > The PPC32 ABI dictates that long long (64bit) parameters are pass in odd/even > register pairs. Because unlike ARM and MIPS we start at an odd register > number, > we can reuse the same aligning code that ARM and MIPS use. > > Signed-off-by: Alexander

Re: [Qemu-devel] [PATCH 2/2] linux-user: register align p{read, write}64

2012-10-01 Thread Alex Barcelo
On Sun, Sep 30, 2012 at 3:32 AM, Alexander Graf wrote: > pread64 and pwrite64 pass 64bit parameters which for some architectures need > to be aligned to special argument pairs, creating a gap argument. > > Handle this special case the same way we handle it in other places of the > code. > > Repor

Re: [Qemu-devel] [PATCH 2/7] tcg: Optimize add2 + sub2

2012-10-01 Thread Aurelien Jarno
On Thu, Sep 27, 2012 at 04:28:47PM -0700, Richard Henderson wrote: > On 09/27/2012 04:20 PM, Aurelien Jarno wrote: > > I understand that we can't easily insert an instruction, so the > > limitation comes from here, but is it really something happening often? > > It will certainly appear sometimes.

Re: [Qemu-devel] [PATCH v2 0/9] Embedded NBD server

2012-10-01 Thread Luiz Capitulino
On Mon, 1 Oct 2012 16:52:15 +0200 Paolo Bonzini wrote: > This series rebases the previous qemu-sockets patches for error > propagation and uses the new QAPI-friendly socket functions in the > embedded NBD server. The changes are due to Orit's patches being > now in, some early parts being in Lu

Re: [Qemu-devel] [PATCH 2/7] tcg: Optimize add2 + sub2

2012-10-01 Thread Richard Henderson
On 2012-09-30 00:04, Blue Swirl wrote: >> We can't do complete constant folding because we lack "mov2", >> > or the ability to insert opcodes in the stream. But we can >> > at least canonicalize add2 operand ordering and simplify >> > add2 to add when the lowpart adds a constant 0. > Couldn't we i

Re: [Qemu-devel] [PATCH 2/7] tcg: Optimize add2 + sub2

2012-10-01 Thread Richard Henderson
On 2012-10-01 10:46, Aurelien Jarno wrote: > For the first add2 that seemed to have work correctly, this patch > optimized 0.2% of them. I am not sure it worth it as is. You're probably right. > I think optimizing add2, and in general all *2 ops is a good idea, but > we should be able to do more

[Qemu-devel] [PATCH] vfio_pci: fix build on 32-bit systems

2012-10-01 Thread Anthony Liguori
We cannot cast directly from pointer to uint64. Cc: Alex Williamson Cc: Alex Barcelo Reported-by: Alex Barcelo Signed-off-by: Anthony Liguori --- hw/vfio_pci.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/vfio_pci.c b/hw/vfio_pci.c index a24558a..a1eeced 100644

Re: [Qemu-devel] [PATCH 4/7] tcg: Optimize double-word comparisons against zero

2012-10-01 Thread Aurelien Jarno
On Thu, Sep 27, 2012 at 10:19:54AM -0700, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > tcg/optimize.c | 51 +++ > 1 file changed, 51 insertions(+) > > diff --git a/tcg/optimize.c b/tcg/optimize.c > index d39926e..c972e4f 1006

Re: [Qemu-devel] [PATCH 5/7] tcg: Split out subroutines from do_constant_folding_cond

2012-10-01 Thread Aurelien Jarno
On Thu, Sep 27, 2012 at 10:19:55AM -0700, Richard Henderson wrote: > We can re-use these for implementing double-word folding. > > Signed-off-by: Richard Henderson > --- > tcg/optimize.c | 146 > - > 1 file changed, 81 insertions(+), 65 de

Re: [Qemu-devel] [PATCH 4/7] tcg: Optimize double-word comparisons against zero

2012-10-01 Thread Richard Henderson
On 2012-10-01 11:43, Aurelien Jarno wrote: > While it's a nice optimization to have, one that seems to happen a lot > more often is the two high parts being equal. It happens when the guest > is working on (u)int32_t. It depends on what target you're looking at. For alpha guest, all branches are

Re: [Qemu-devel] [PATCH 6/7] tcg: Tidy brcond optimization

2012-10-01 Thread Aurelien Jarno
On Thu, Sep 27, 2012 at 10:19:56AM -0700, Richard Henderson wrote: > Do the memset once. Don't reset_temp before doing so. > > Signed-off-by: Richard Henderson > --- > tcg/optimize.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/tcg/optimize.c b/tcg/optimize.c

Re: [Qemu-devel] [PATCH] vfio_pci: fix build on 32-bit systems

2012-10-01 Thread Alex Williamson
On Mon, 2012-10-01 at 13:41 -0500, Anthony Liguori wrote: > We cannot cast directly from pointer to uint64. > > Cc: Alex Williamson > Cc: Alex Barcelo > Reported-by: Alex Barcelo > Signed-off-by: Anthony Liguori > --- > hw/vfio_pci.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(

Re: [Qemu-devel] [PATCH 7/7] tcg: Do constant folding on double-word comparisons

2012-10-01 Thread Aurelien Jarno
On Thu, Sep 27, 2012 at 10:19:57AM -0700, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > tcg/optimize.c | 134 > - > 1 file changed, 94 insertions(+), 40 deletions(-) > > diff --git a/tcg/optimize.c b/tcg/optimize.c >

[Qemu-devel] [PATCH v2] target-xtensa: de-optimize EXTUI

2012-10-01 Thread Aurelien Jarno
Now that "and" with 0xff, 0x and 0x and "shr" with 0 shift are optimized in tcg/tcg-op.h there is no need to do it in target-xtensa/translate.c. Cc: Max Filippov Signed-off-by: Aurelien Jarno --- target-xtensa/translate.c | 22 ++ 1 file changed, 2 insertions(+

Re: [Qemu-devel] [PATCH v10] kvm: notify host when the guest is panicked

2012-10-01 Thread Luiz Capitulino
On Wed, 29 Aug 2012 13:18:54 +0800 Wen Congyang wrote: > We can know the guest is panicked when the guest runs on xen. > But we do not have such feature on kvm. What's the status of this series? It got lost in my queue and I ended up not reviewing it, but it seems to be stuck. > > Another pur

[Qemu-devel] [PATCH] tcg: remove obsolete jmp op

2012-10-01 Thread Aurelien Jarno
The TCG jmp operation doesn't really make sense in the QEMU context, it is unused, it is not implemented by some targets, and it is wrongly implemented by some others. This patch simply removes it. Cc: Andrzej Zaborowski Cc: Richard Henderson Cc: Vassili Karpov (malc) Cc: Alexander Graf Cc: B

Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it

2012-10-01 Thread Anthony Liguori
"Daniel P. Berrange" writes: > On Mon, Oct 01, 2012 at 06:43:00PM +0200, Andreas Färber wrote: >> Hello Jan, >> >> Am 01.10.2012 16:34, schrieb Jan Kiszka: >> > If we built a target for a host that supports KVM in principle, set the >> > default accelerator to KVM as well. This also means the st

Re: [Qemu-devel] [PATCH v2 4/9] qemu-sockets: add error propagation to Unix socket functions

2012-10-01 Thread Paolo Bonzini
Il 01/10/2012 19:17, Luiz Capitulino ha scritto: >> > if (bind(sock, (struct sockaddr*) &un, sizeof(un)) < 0) { >> > -fprintf(stderr, "bind(unix:%s): %s\n", un.sun_path, >> > strerror(errno)); >> > +error_set(errp, QERR_SOCKET_BIND_FAILED); > This drops error information, maki

Re: [Qemu-devel] [PATCH] device tree: simplify dumpdtb code

2012-10-01 Thread Aurelien Jarno
On Sun, Sep 23, 2012 at 11:29:32PM +0200, Alexander Graf wrote: > As per Peter's suggestion, we can use glib to write out a buffer in whole to > a file, simplifying the code dramatically. > > Signed-off-by: Alexander Graf > > --- > > This patch applies on top of the generic dumpdtb remodeling p

Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it

2012-10-01 Thread Paolo Bonzini
> But libguest can set it's accelerator option to whatever it wants. > > If your running QEMU under a VM, it's pretty reasonable to have to > use a special option IMHO. It's also reasonable to have consecutive releases change defaults in a more "friendly" way (i.e. from tcg to kvm:tcg), especial

Re: [Qemu-devel] [PATCH] vfio_pci: fix build on 32-bit systems

2012-10-01 Thread Anthony Liguori
Anthony Liguori writes: > We cannot cast directly from pointer to uint64. > > Cc: Alex Williamson > Cc: Alex Barcelo > Reported-by: Alex Barcelo > Signed-off-by: Anthony Liguori Applied. Regards, Anthony Liguori > --- > hw/vfio_pci.c |2 +- > 1 files changed, 1 insertions(+), 1 delet

Re: [Qemu-devel] [PATCH v10] kvm: notify host when the guest is panicked

2012-10-01 Thread Marcelo Tosatti
On Mon, Oct 01, 2012 at 03:57:40PM -0300, Luiz Capitulino wrote: > On Wed, 29 Aug 2012 13:18:54 +0800 > Wen Congyang wrote: > > > We can know the guest is panicked when the guest runs on xen. > > But we do not have such feature on kvm. > > What's the status of this series? > > It got lost in my

[Qemu-devel] KVM call agenda for 2012-10-01

2012-10-01 Thread Juan Quintela
hi Please send in any agenda topics you are interested in. Later, Juan.

Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it

2012-10-01 Thread Anthony Liguori
Paolo Bonzini writes: >> But libguest can set it's accelerator option to whatever it wants. >> >> If your running QEMU under a VM, it's pretty reasonable to have to >> use a special option IMHO. > > It's also reasonable to have consecutive releases change defaults in > a more "friendly" way (i.e

Re: [Qemu-devel] [PATCH] fpu/softfloat.c: Return correctly signed values from uint64_to_float32

2012-10-01 Thread Aurelien Jarno
On Fri, Sep 28, 2012 at 04:17:03PM +0100, Peter Maydell wrote: > The uint64_to_float32() conversion function was incorrectly always > returning numbers with the sign bit set (ie negative numbers). Correct > this so we return positive numbers instead. > > Signed-off-by: Peter Maydell > --- > As fa

Re: [Qemu-devel] [PATCH] fpu/softfloat.c: Remove pointless shift of always-zero value

2012-10-01 Thread Aurelien Jarno
On Mon, Sep 24, 2012 at 05:28:35PM +0100, Peter Maydell wrote: > In float16_to_float32, when returning an infinity, just pass zero > as the mantissa argument to packFloat32(), rather than shifting > a value which we know must be zero. > > Signed-off-by: Peter Maydell > --- > Spotted by the clang

Re: [Qemu-devel] [PATCH v2] target-xtensa: de-optimize EXTUI

2012-10-01 Thread Max Filippov
On Mon, Oct 1, 2012 at 10:54 PM, Aurelien Jarno wrote: > Now that "and" with 0xff, 0x and 0x and "shr" with 0 shift > are optimized in tcg/tcg-op.h there is no need to do it in > target-xtensa/translate.c. > > Cc: Max Filippov > Signed-off-by: Aurelien Jarno > --- > target-xtensa/tr

Re: [Qemu-devel] KVM call agenda for 2012-10-01

2012-10-01 Thread Anthony Liguori
Juan Quintela writes: > hi > > Please send in any agenda topics you are interested in. 1) TODO to finish off qemu-kvm.git/master... Regards, Anthony Liguori > > Later, Juan.

Re: [Qemu-devel] [PATCH] tcg: remove obsolete jmp op

2012-10-01 Thread Richard Henderson
On 2012-10-01 12:00, Aurelien Jarno wrote: > The TCG jmp operation doesn't really make sense in the QEMU context, it > is unused, it is not implemented by some targets, and it is wrongly > implemented by some others. > > This patch simply removes it. > > Cc: Andrzej Zaborowski > Cc: Richard Hend

Re: [Qemu-devel] [PATCH] qtest: implement QTEST_STOP

2012-10-01 Thread Anthony Liguori
Paolo Bonzini writes: > It is quite difficult to debug qtest test cases without extra wrapper > scripts for QEMU or similar. This patch adds a simple environment > variable-based trigger that sends a STOP signal to the QEMU instance > under test, before attempting to connect to its QMP session.

Re: [Qemu-devel] [PATCH] Add option to mlock guest and qemu memory

2012-10-01 Thread Satoru Moriya
Hi Jan, Thank you for reviewing. On 09/28/2012 04:05 AM, Jan Kiszka wrote: > On 2012-09-28 01:21, Satoru Moriya wrote: >> We have some plans to migrate old enterprise systems which require >> low latency (msec order) to kvm virtualized environment. Usually, >> we uses mlock to preallocate and pin

  1   2   >