Re: [Qemu-devel] [RFC 0/9] Introduce light weight PC platform pc-lite

2016-06-19 Thread Claudio Fontana
Hi, On 17.06.2016 10:14, Chao Peng wrote: > This patchset is against commit 585fcd4 (Merge remote-tracking branch > 'remotes/bonzini/tags/for-upstream' into staging) on master branch. I > also put it on github: > > https://github.com/chao-p/qemu pc-lite-v1 > > Although we have run the patchset

Re: [Qemu-devel] [PATCH 2/2] configure: Make AVX2 test robust to non-ELF systems

2016-06-19 Thread Peter Maydell
On 19 June 2016 at 06:53, Stefan Weil wrote: > Am 19.06.2016 um 00:05 schrieb Peter Maydell: >> From: Peter Maydell >> >> The AVX2 optimization test assumes that the object format >> is ELF and the system has the readelf utility. If this isn't >> true then configure might fail or emit a warning (

Re: [Qemu-devel] [PATCH 2/7] QEMU does not currently support host pages that are larger than guest pages, likely due to glibc using fixed mmap requests.

2016-06-19 Thread Peter Maydell
On 19 June 2016 at 01:11, Timothy Pearson wrote: > Attempting to use host pages larger than the guest leads to > alignment errors during ELF load in the best case, and an > initialization failure inside NPTL in the worst case, causing > all fork() requests inside the guest to fail. > > Warn when t

Re: [Qemu-devel] [PATCH 3/7] Pass select() arguments directly to do_select() on x86 platforms

2016-06-19 Thread Peter Maydell
On 19 June 2016 at 01:15, Timothy Pearson wrote: > This matches the calling conventions in the Linux kernel and > resolves select() hangs on i386/x86_64 guests. > > Signed-off-by: Timothy Pearson > --- > linux-user/syscall.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --gi

Re: [Qemu-devel] [PATCH 5/7] TIOCGPTN and related terminal control ioctls were not converted to the guest ioctl format on x86_64 targets. Convert these ioctls to enable terminal functionality on x86_6

2016-06-19 Thread Peter Maydell
On 19 June 2016 at 01:15, Timothy Pearson wrote: > Signed-off-by: Timothy Pearson > --- > linux-user/x86_64/termbits.h | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/linux-user/x86_64/termbits.h b/linux-user/x86_64/termbits.h > index 1c3445c..5fc4639 100644

Re: [Qemu-devel] [PATCH 2/2] configure: Make AVX2 test robust to non-ELF systems

2016-06-19 Thread Stefan Weil
Am 19.06.2016 um 11:41 schrieb Peter Maydell: > On 19 June 2016 at 06:53, Stefan Weil wrote: >> Am 19.06.2016 um 00:05 schrieb Peter Maydell: >>> From: Peter Maydell >>> >>> The AVX2 optimization test assumes that the object format >>> is ELF and the system has the readelf utility. If this isn't

Re: [Qemu-devel] [Qemu-ppc] [PATCH] ppc: Improve emulation of THRM registers

2016-06-19 Thread BALATON Zoltan
On Sun, 19 Jun 2016, Benjamin Herrenschmidt wrote: The 75x and 74xx processors have some thermal monitoring SPRs that some OSes such as MacOS do use. Our current "dumb" implementation isn't good enough and will cause some versions of MacOS to hang during boot. This lifts an improved emulation fr

[Qemu-devel] [Bug 1594069] Re: SIMD instructions translated to scalar host instructions

2016-06-19 Thread pranith
** Changed in: qemu Status: New => Confirmed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1594069 Title: SIMD instructions translated to scalar host instructions Status in QEMU: Confirme

Re: [Qemu-devel] [Qemu-ppc] [PATCH 01/10] ppc: Fix rfi/rfid/hrfi/... emulation

2016-06-19 Thread Cédric Le Goater
On 06/19/2016 01:35 AM, Benjamin Herrenschmidt wrote: > On Fri, 2016-06-17 at 16:32 +0200, Cédric Le Goater wrote: >> The instruction set PPC_POWER_BR contains nearly all the deleted >> instructions from isa2. rfi is not part of it and should. Also, only >> the cpus "PowerPC 601*" make a use of i

Re: [Qemu-devel] [Qemu-ppc] [PATCH 01/10] ppc: Fix rfi/rfid/hrfi/... emulation

2016-06-19 Thread Alexander Graf
> Am 19.06.2016 um 14:49 schrieb Cédric Le Goater : > >> On 06/19/2016 01:35 AM, Benjamin Herrenschmidt wrote: >>> On Fri, 2016-06-17 at 16:32 +0200, Cédric Le Goater wrote: >>> The instruction set PPC_POWER_BR contains nearly all the deleted >>> instructions from isa2. rfi is not part of it an

Re: [Qemu-devel] [Bug 1594069] [NEW] SIMD instructions translated to scalar host instructions

2016-06-19 Thread Peter Maydell
On 19 June 2016 at 06:33, Timothy Pearson wrote: > Public bug reported: > > SIMD instructions inside the guest (NEON, MMX, SSE, SSE2, AVX) are > translated to scalar instructions on the host instead of SIMD > instructions. It appears that there have been a few efforts to rectify > this [1], and e

Re: [Qemu-devel] [Qemu-ppc] [PATCH 01/10] ppc: Fix rfi/rfid/hrfi/... emulation

2016-06-19 Thread Benjamin Herrenschmidt
On Sun, 2016-06-19 at 14:49 +0200, Cédric Le Goater wrote: > > So my test is correct in the context of what we emulate today. > > OK. so this is an openbios issue when run under a ppc64. shouldn't we  > be using an openbios-ppc64 in that case ?   > > > We could be more strict with the rfi instru

[Qemu-devel] [PATCH] ppc: Improve emulation of THRM registers

2016-06-19 Thread G 3
On 6/19/16 12:12 AM, qemu-devel-requ...@nongnu.org wrote: Message: 5 Date: Sun, 19 Jun 2016 10:54:13 +1000 From: Benjamin Herrenschmidt To:qemu-...@nongnu.org Cc:qemu-devel@nongnu.org,da...@gibson.dropbear.id.au, Mark Cave-Ayland , Alexander Graf Subject: [Qemu-devel] [PATCH

Re: [Qemu-devel] [PATCH v2] ppc / sparc: Add a tester for checking whether OpenBIOS runs successfully

2016-06-19 Thread Artyom Tarasenko
On Fri, Jun 17, 2016 at 3:56 PM, Mark Cave-Ayland wrote: Since the mac99 and g3beige PowerPC machines recently broke without being noticed, it would be good to have a tester for "make check" that detects such issues immediately. A simple way to test the firmware

Re: [Qemu-devel] [PATCH 4/5] x86: Allow physical address bits to be set

2016-06-19 Thread Marcel Apfelbaum
On 06/17/2016 07:07 PM, Laszlo Ersek wrote: On 06/17/16 11:52, Igor Mammedov wrote: On Fri, 17 Jun 2016 11:17:54 +0200 Gerd Hoffmann wrote: On Fr, 2016-06-17 at 10:43 +0200, Paolo Bonzini wrote: On 17/06/2016 10:15, Dr. David Alan Gilbert wrote: Larger is a problem if the guest tries to ma

Re: [Qemu-devel] [PATCH 06/10] machine: Add machine_register_compat_props() function

2016-06-19 Thread Marcel Apfelbaum
On 06/15/2016 11:32 PM, Eduardo Habkost wrote: Move the compat_props handling to core machine code. Signed-off-by: Eduardo Habkost --- hw/core/machine.c | 16 include/hw/boards.h | 1 + vl.c| 9 ++--- 3 files changed, 19 insertions(+), 7 deletions(-

Re: [Qemu-devel] [PATCH 07/10] vl: Set errp to &error_abort on machine compat_props

2016-06-19 Thread Marcel Apfelbaum
On 06/15/2016 11:32 PM, Eduardo Habkost wrote: Use the new GlobalProperty.errp field to handle compat_props errors. Example output before this change: (with an intentionally broken entry added to PC_COMPAT_1_3 just for testing) $ qemu-system-x86_64 -machine pc-1.3 qemu-system-x86_64: hw/c

Re: [Qemu-devel] [PATCH 10/10] machine: Skip global registration for non-existing classes

2016-06-19 Thread Marcel Apfelbaum
On 06/15/2016 11:32 PM, Eduardo Habkost wrote: MachineClass::compat_props may point to class names that are not compiled into the QEMU binary. Skip registering those as global properties. This will allow the qdev global property code to implement stricter checks on the global property values in t

Re: [Qemu-devel] [PATCH 01/10] qdev: Don't stop applying globals on first error

2016-06-19 Thread Marcel Apfelbaum
On 06/15/2016 11:32 PM, Eduardo Habkost wrote: qdev_prop_set_globals_for_type() stops applying global properties on the first error. It is a leftover from when QEMU exited on any error when applying global property. Now we print a warning about the first error, bug ignore all other global propert

Re: [Qemu-devel] [PATCH 10/10] machine: Skip global registration for non-existing classes

2016-06-19 Thread Marcel Apfelbaum
On 06/19/2016 07:39 PM, Marcel Apfelbaum wrote: On 06/15/2016 11:32 PM, Eduardo Habkost wrote: MachineClass::compat_props may point to class names that are not compiled into the QEMU binary. Skip registering those as global properties. This will allow the qdev global property code to implement s

Re: [Qemu-devel] [Qemu-ppc] [PATCH 01/10] ppc: Fix rfi/rfid/hrfi/... emulation

2016-06-19 Thread Cédric Le Goater
On 06/19/2016 03:00 PM, Alexander Graf wrote: > > >> Am 19.06.2016 um 14:49 schrieb Cédric Le Goater : >> >>> On 06/19/2016 01:35 AM, Benjamin Herrenschmidt wrote: On Fri, 2016-06-17 at 16:32 +0200, Cédric Le Goater wrote: The instruction set PPC_POWER_BR contains nearly all the deleted

Re: [Qemu-devel] [Qemu-ppc] [PATCH 01/10] ppc: Fix rfi/rfid/hrfi/... emulation

2016-06-19 Thread Cédric Le Goater
On 06/19/2016 04:08 PM, Benjamin Herrenschmidt wrote: > On Sun, 2016-06-19 at 14:49 +0200, Cédric Le Goater wrote: >>> So my test is correct in the context of what we emulate today. >> >> OK. so this is an openbios issue when run under a ppc64. shouldn't we >> be using an openbios-ppc64 in that ca

Re: [Qemu-devel] [PATCH v2] ppc / sparc: Add a tester for checking whether OpenBIOS runs successfully

2016-06-19 Thread Richard Henderson
On 06/19/2016 08:26 AM, Artyom Tarasenko wrote: On Fri, Jun 17, 2016 at 3:56 PM, Mark Cave-Ayland wrote: Since the mac99 and g3beige PowerPC machines recently broke without being noticed, it would be good to have a tester for "make check" that detects such issues immediately. A simple way to te

Re: [Qemu-devel] [PATCH v8 00/12] 8bit AVR cores

2016-06-19 Thread Richard Henderson
On 06/18/2016 12:55 PM, Michael Rolnik wrote: This series of patches adds 8bit AVR cores to QEMU. All instruction, except BREAK/DES/SPM/SPMX, are implemented. Not fully tested yet. However I was able to execute simple code with functions. e.g fibonacci calculation. This series of patches includ

[Qemu-devel] [Bug 1585008] Re: Windows 7 guests hang on bootup when qxl video is used

2016-06-19 Thread Serge Hallyn
Thanks. ** Also affects: qemu Importance: Undecided Status: New -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1585008 Title: Windows 7 guests hang on bootup when qxl video is used Statu

Re: [Qemu-devel] [PATCH v8 00/12] 8bit AVR cores

2016-06-19 Thread Michael Rolnik
it did compile for me. how do you compile? what machine? On Sun, Jun 19, 2016 at 8:45 PM, Richard Henderson wrote: > On 06/18/2016 12:55 PM, Michael Rolnik wrote: > >> This series of patches adds 8bit AVR cores to QEMU. >> All instruction, except BREAK/DES/SPM/SPMX, are implemented. Not fully >>

Re: [Qemu-devel] [PATCH v8 00/12] 8bit AVR cores

2016-06-19 Thread Michael Rolnik
I am building on MAC. I just do ./configure --target-list=avr-softmmu build no errors On Sun, Jun 19, 2016 at 8:50 PM, Michael Rolnik wrote: > it did compile for me. > how do you compile? what machine? > > On Sun, Jun 19, 2016 at 8:45 PM, Richard Henderson > wrote: > >> On 06/18/2016 12:55 PM,

Re: [Qemu-devel] [PATCH 2/7] QEMU does not currently support host pages that are larger than guest pages, likely due to glibc using fixed mmap requests.

2016-06-19 Thread Richard Henderson
On 06/19/2016 02:46 AM, Peter Maydell wrote: On 19 June 2016 at 01:11, Timothy Pearson wrote: Attempting to use host pages larger than the guest leads to alignment errors during ELF load in the best case, and an initialization failure inside NPTL in the worst case, causing all fork() requests i

Re: [Qemu-devel] [PATCH v8 00/12] 8bit AVR cores

2016-06-19 Thread Richard Henderson
On 06/19/2016 10:50 AM, Michael Rolnik wrote: it did compile for me. how do you compile? what machine? Fedora 23, gcc 5.3.1. r~

[Qemu-devel] [PATCH] ppc64: disable gen_pause() for linux-user mode

2016-06-19 Thread Laurent Vivier
While trying to install a fedora container with "lxc-create -t fedora -- -I qemu-ppc64" the installation abort with the following error: qemu: fatal: Unknown exception 0x65537. Aborting NIP 004000927924 LR 0040009e325c CTR 004000927480 XER CPU#0 MSR 90010280600

[Qemu-devel] kvm_arch_put_registers and xsave

2016-06-19 Thread Alex Bligh
I'm still plugging away at my Hypervisor.Framework port (now rewritten to be another accelerator like kvm). It appears to be setting up memory regions, and I'm now working on the equivalent of kvm_cpu_exec. I see an exit code 33 on the first call, which is an invalid VMCS. This is unsurprising as

Re: [Qemu-devel] kvm_arch_put_registers and xsave

2016-06-19 Thread Peter Maydell
On 19 June 2016 at 20:36, Alex Bligh wrote: > Looking at kvm_arch_put_registers it appears to be pretty complicated. > It appears to 'put' each individual register type, as well as putting > an xsave region. I'm a bit confused why the ordinary registers 'put' > are not then overwritten by the xsav

Re: [Qemu-devel] [PATCH] ppc: Improve emulation of THRM registers

2016-06-19 Thread Benjamin Herrenschmidt
On Sun, 2016-06-19 at 11:09 -0400, G 3 wrote: >   On 6/19/16 12:12 AM, qemu-devel-requ...@nongnu.org wrote: > > Message: 5 > > Date: Sun, 19 Jun 2016 10:54:13 +1000 > > From: Benjamin Herrenschmidt > > To:qemu-...@nongnu.org > > Cc:qemu-devel@nongnu.org,da...@gibson.dropbear.id.au, Mark > > Cav

[Qemu-devel] KVM VM hangs forever in R state within lock

2016-06-19 Thread Andrey Korolyov
Hi, I`ve observed this issue previously on an old 3.10 branch but wrote it off due to inability to reproduce in any meaningful way. Currently I am seeing it on 3.10 branch where all KVM-related and RCU-related issues are patched more or less for well-known issues. Way to obtain a problematic stat

Re: [Qemu-devel] [Qemu-ppc] [PATCH 01/10] ppc: Fix rfi/rfid/hrfi/... emulation

2016-06-19 Thread Benjamin Herrenschmidt
On Sun, 2016-06-19 at 19:23 +0200, Cédric Le Goater wrote: > > You can run a 32-bit OS or firmware on ppc64, but it needs to know that > > it's running on a 64-bit chip and do a few things differently. > > yes sure but qemu would still allow rfi under 64bit CPUs, that is what  > I was concerned ab

[Qemu-devel] [Bug 1585008] Re: Windows 7 guests hang on bootup when qxl video is used

2016-06-19 Thread Anthony de Beus
Downgraded to qemu-2.4.0-1 (on Arch), problem doesn't exist there. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1585008 Title: Windows 7 guests hang on bootup when qxl video is used Status in QEM

[Qemu-devel] [Bug 1585008] Re: Windows 7 guests hang on bootup when qxl video is used

2016-06-19 Thread Anthony de Beus
and doesn't exist in qemu-2.5.1-1; the next version I have in Arch is qemu-2.6.0-1, which is the current one where the problem exists. So, something changed between 2.5 and 2.6 Sorry about the multiple posts. -- You received this bug notification because you are a member of qemu- devel-ml, whic

Re: [Qemu-devel] kvm_arch_put_registers and xsave

2016-06-19 Thread Alex Bligh
On 19 Jun 2016, at 21:54, Peter Maydell wrote: > The purpose of the get/put functions, broadly, is "copy > state from the hypervisor into QEMU's cpu state struct > and vice-versa". The specific details are down to KVM's ABI > (and to historical details like some KVM ioctls being newer > or optio

[Qemu-devel] [PATCH] ppc: Improve emulation of THRM registers

2016-06-19 Thread Programmingkid
On Jun 19, 2016, at 12:12 AM, qemu-devel-requ...@nongnu.org wrote: > Message: 5 > Date: Sun, 19 Jun 2016 10:54:13 +1000 > From: Benjamin Herrenschmidt > To: qemu-...@nongnu.org > Cc: qemu-devel@nongnu.org, da...@gibson.dropbear.id.au, Mark > Cave-Ayland , Alexander Graf > > Subj

Re: [Qemu-devel] [Qemu-ppc] [PATCH 01/10] ppc: Fix rfi/rfid/hrfi/... emulation

2016-06-19 Thread Benjamin Herrenschmidt
On Sun, 2016-06-19 at 19:21 +0200, Cédric Le Goater wrote: > But, I could not boot  ./darwinppc-602.cdr with on a 970. That might > be another issue. Right, the issue is that the kernel in Darwin 6.0.2 doesn't support the 970 :-) I think the first MacOS X to support 970 was some special build of

[Qemu-devel] [RFC 4/8] linux-user: Rework exclusive operation mechanism

2016-06-19 Thread Sergey Fedorov
From: Sergey Fedorov A single variable 'pending_cpus' was used for both counting currently running CPUs and for signalling the pending exclusive operation request. To prepare for supporting operations which requires a quiescent state, like translation buffer flush, it is useful to keep a counter

[Qemu-devel] [RFC 5/8] linux-user: Add qemu_cpu_is_self() and qemu_cpu_kick()

2016-06-19 Thread Sergey Fedorov
From: Sergey Fedorov Signed-off-by: Sergey Fedorov Signed-off-by: Sergey Fedorov --- linux-user/main.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/linux-user/main.c b/linux-user/main.c index 485336f78b8f..0093a8008c8e 100644 --- a/linux-user/main.c +++ b/linux-user/main.c @

[Qemu-devel] [RFC 1/8] cpus: pass CPUState to run_on_cpu helpers

2016-06-19 Thread Sergey Fedorov
From: Alex Bennée CPUState is a fairly common pointer to pass to these helpers. This means if you need other arguments for the async_run_on_cpu case you end up having to do a g_malloc to stuff additional data into the routine. For the current users this isn't a massive deal but for MTTCG this get

[Qemu-devel] [RFC 0/8] cpu-exec: Safe work in quiescent state

2016-06-19 Thread Sergey Fedorov
From: Sergey Fedorov Hi, This RFC series is a follow-up for a discussion on the subject [1]. Basically, this series is intended to show a possible way to perform operations on quiescent state, so that we can discuss if it a sane way to do this. The goal is to implement such a mechanism which ca

[Qemu-devel] [RFC 2/8] cpus: Move common code out of {async_, }run_on_cpu()

2016-06-19 Thread Sergey Fedorov
From: Sergey Fedorov Move the code common between run_on_cpu() and async_run_on_cpu() into a new function queue_work_on_cpu(). Signed-off-by: Sergey Fedorov Signed-off-by: Sergey Fedorov --- cpus.c | 42 ++ 1 file changed, 18 insertions(+), 24 deletions

[Qemu-devel] [RFC 3/8] cpus: Add 'qemu_work_cond' usage wrappers

2016-06-19 Thread Sergey Fedorov
From: Sergey Fedorov Introducing these wrappers is a step towards CPU work support in user-mode emulation. Signed-off-by: Sergey Fedorov Signed-off-by: Sergey Fedorov --- cpus.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/cpus.c b/cpus.c index 04687c85b

[Qemu-devel] [RFC 7/8] cpu-exec-common: Introduce async_safe_run_on_cpu()

2016-06-19 Thread Sergey Fedorov
From: Sergey Fedorov This patch is based on the ideas found in work of KONRAD Frederic [1], Alex Bennée [2], and Alvise Rigo [3]. This mechanism allows to perform an operation safely in a quiescent state. Quiescent state means: (1) no vCPU is running and (2) BQL in system-mode or 'exclusive_lock

Re: [Qemu-devel] [Qemu-ppc] [PATCH 01/10] ppc: Fix rfi/rfid/hrfi/... emulation

2016-06-19 Thread Benjamin Herrenschmidt
On Mon, 2016-06-20 at 08:15 +1000, Benjamin Herrenschmidt wrote: > On Sun, 2016-06-19 at 19:21 +0200, Cédric Le Goater wrote: > > But, I could not boot  ./darwinppc-602.cdr with on a 970. That > > might > > be another issue. > > Right, the issue is that the kernel in Darwin 6.0.2 doesn't support >

[Qemu-devel] [RFC 8/8] tcg: Make tb_flush() thread safe

2016-06-19 Thread Sergey Fedorov
From: Sergey Fedorov Use async_safe_run_on_cpu() to make tb_flush() thread safe. Signed-off-by: Sergey Fedorov Signed-off-by: Sergey Fedorov --- translate-all.c | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/translate-all.c b/translate-all.c index 3f402dfe04f

[Qemu-devel] [RFC 6/8] linux-user: Support CPU work queue

2016-06-19 Thread Sergey Fedorov
From: Sergey Fedorov Make CPU work core functions common between system and user-mode emulation. User-mode does not have BQL, so flush_queued_work() is protected by 'exclusive_lock'. Signed-off-by: Sergey Fedorov Signed-off-by: Sergey Fedorov --- cpu-exec-common.c | 83 +

Re: [Qemu-devel] [PATCH 1/2] tcg: Fix name for high-half register

2016-06-19 Thread David Gibson
On Fri, Jun 17, 2016 at 10:03:26PM -0700, Richard Henderson wrote: > Signed-off-by: Richard Henderson Reviewed-by: David Gibson > --- > tcg/tcg.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tcg/tcg.c b/tcg/tcg.c > index 254427b..154ffe8 100644 > --- a/tcg/tcg.c > +

Re: [Qemu-devel] [PATCH] ppc64: disable gen_pause() for linux-user mode

2016-06-19 Thread David Gibson
On Sun, Jun 19, 2016 at 09:19:24PM +0200, Laurent Vivier wrote: > While trying to install a fedora container with > "lxc-create -t fedora -- -I qemu-ppc64" the installation abort with > the following error: > > qemu: fatal: Unknown exception 0x65537. Aborting > > NIP 004000927924 LR 004

Re: [Qemu-devel] [PATCH] tests: Use '+=' to add additional tests, not '='

2016-06-19 Thread David Gibson
On Fri, Jun 17, 2016 at 03:16:17PM +0200, Thomas Huth wrote: > The recent commit that added the prom-env-test accidentially > overwrote the check-qtest-ppc-y, check-qtest-ppc64-y and > check-qtest-sparc-y variables instead of extending them. > > Fixes: fcbf4a3c0c576eec1321f9cff4fa0dd8e0b1a82f > Si

Re: [Qemu-devel] [PATCH] powerpc/mm: Update the WIMG check during H_ENTER

2016-06-19 Thread David Gibson
On Fri, Jun 17, 2016 at 04:07:20PM +0530, Aneesh Kumar K.V wrote: > Support for 0 value for memeory coherence is optional and with ppc64 > we can always enable memory coherence. Linux kernel did that during > the development of 4.7 kernel. But that resulted in failure in Qemu > in H_ENTER hcall due

Re: [Qemu-devel] [SeaBIOS] [PATCH v2] fw/msr_feature_control: add support to set MSR_IA32_FEATURE_CONTROL

2016-06-19 Thread Haozhong Zhang
On 06/17/16 11:26, Kevin O'Connor wrote: > On Fri, Jun 17, 2016 at 03:20:10PM +0800, Haozhong Zhang wrote: > > OS usually expects BIOS to set certain bits in MSR_IA32_FEATURE_CONTROL > > for some features (e.g. VMX and LMCE). QEMU provides a fw_cfg file > > "etc/msr_feature_control" to advise bits

Re: [Qemu-devel] [Qemu-ppc] [PATCH] ppc: Improve emulation of THRM registers

2016-06-19 Thread David Gibson
On Sun, Jun 19, 2016 at 02:14:12PM +0200, BALATON Zoltan wrote: > On Sun, 19 Jun 2016, Benjamin Herrenschmidt wrote: > > The 75x and 74xx processors have some thermal monitoring SPRs that > > some OSes such as MacOS do use. Our current "dumb" implementation > > isn't good enough and will cause some

Re: [Qemu-devel] [RFC 1/8] cpus: pass CPUState to run_on_cpu helpers

2016-06-19 Thread David Gibson
On Mon, Jun 20, 2016 at 01:28:26AM +0300, Sergey Fedorov wrote: > From: Alex Bennée > > CPUState is a fairly common pointer to pass to these helpers. This means > if you need other arguments for the async_run_on_cpu case you end up > having to do a g_malloc to stuff additional data into the routi

Re: [Qemu-devel] [QEMU 1/7] balloon: speed up inflating & deflating process

2016-06-19 Thread Li, Liang Z
> > > > virtqueue_push(vq, elem, offset); @@ -374,6 +489,7 @@ static > > uint64_t virtio_balloon_get_features(VirtIODevice *vdev, uint64_t f, > > VirtIOBalloon *dev = VIRTIO_BALLOON(vdev); > > f |= dev->host_features; > > virtio_add_feature(&f, VIRTIO_BALLOON_F_STATS_VQ); >

[Qemu-devel] [PATCH] hw/intc/arm_gicv3: Add missing break

2016-06-19 Thread Shannon Zhao
From: Shannon Zhao These are spotted by coverity 1356936 and 1356937. Signed-off-by: Shannon Zhao --- hw/intc/arm_gicv3_cpuif.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/intc/arm_gicv3_cpuif.c b/hw/intc/arm_gicv3_cpuif.c index 5b2972e..4633172 100644 --- a/hw/intc/arm_gicv3_cpui

Re: [Qemu-devel] [PATCH v4 1/3] target-i386: KVM: add basic Intel LMCE support

2016-06-19 Thread Haozhong Zhang
On 06/17/16 13:20, Eduardo Habkost wrote: > On Fri, Jun 17, 2016 at 09:26:57AM +0800, Haozhong Zhang wrote: > [...] > > > > static void mce_init(X86CPU *cpu) > > > > { > > > > CPUX86State *cenv = &cpu->env; > > > > unsigned int bank; > > > > +Error *local_err = NULL; > > > > > > >

Re: [Qemu-devel] [QEMU 2/7] virtio-balloon: add drop cache support

2016-06-19 Thread Li, Liang Z
> On Mon, Jun 13, 2016 at 06:16:44PM +0800, Liang Li wrote: > > virtio-balloon can make use of the amount of free memory to determine > > the amount of memory to be filled in the balloon, but the amount of > > free memory will be effected by the page cache, which can be reclaimed. > > Drop the cach

Re: [Qemu-devel] [PATCH v2] target-ppc: Fix rlwimi, rlwinm, rlwnm

2016-06-19 Thread David Gibson
On Sun, Jun 19, 2016 at 03:34:35AM +1000, Anton Blanchard wrote: > Hi rth, > > > In 63ae0915f8ec, I arranged to use a 32-bit rotate, without > > considering the effect of a mask value that wraps around to > > the high bits of the word. > > Thanks, that passes my tests. > > Tested-by: Anton Blanc

Re: [Qemu-devel] [PATCH v4 2/3] target-i386: add migration support for Intel LMCE

2016-06-19 Thread Haozhong Zhang
On 06/17/16 14:20, Eduardo Habkost wrote: > On Fri, Jun 17, 2016 at 10:01:05AM +0800, Haozhong Zhang wrote: > > On 06/16/16 14:58, Eduardo Habkost wrote: > > > On Thu, Jun 16, 2016 at 07:40:20PM +0200, Paolo Bonzini wrote: > > > > > > > > > > > > On 16/06/2016 19:36, Eduardo Habkost wrote: > > >

Re: [Qemu-devel] [PATCH] target-ppc: Correct ppc3500_spin initial TLB size

2016-06-19 Thread da...@gibson.dropbear.id.au
On Fri, Jun 17, 2016 at 10:55:47PM +, Scott Wood wrote: > On 06/17/2016 05:13 PM, Aaron Larson wrote: > > When e500 PPC is booted multi-core, the non-boot cores are started via > > the spin table. ppce500_spin.c:spin_kick() calls > > mmubooke_create_initial_mapping() to allocate a 64MB TLB ent

Re: [Qemu-devel] [Qemu-ppc] [PATCH 01/10] ppc: Fix rfi/rfid/hrfi/... emulation

2016-06-19 Thread David Gibson
On Mon, Jun 20, 2016 at 07:12:38AM +1000, Benjamin Herrenschmidt wrote: > On Sun, 2016-06-19 at 19:23 +0200, Cédric Le Goater wrote: > > > You can run a 32-bit OS or firmware on ppc64, but it needs to know that > > > it's running on a 64-bit chip and do a few things differently. > > > > yes sure b

[Qemu-devel] [Bug 1591628] Re: 2.6.0 hangs linux vm using vfio for pci passthrough of graphics card

2016-06-19 Thread Alex Williamson
Please test the patch in the link below and send your email address (privately if preferred) so I can provide proper attributes for Reported-by. Thanks. https://paste.fedoraproject.org/381971/46638926/ -- You received this bug notification because you are a member of qemu- devel-ml, which is su

Re: [Qemu-devel] [PATCH 0/5] hw/net: Don't use cpu_to_*w() and *_to_cpup()

2016-06-19 Thread Jason Wang
On 2016年06月17日 01:17, Peter Maydell wrote: This patchset converts a handful of network devices to use ld*_p() and st*_p() instead of cpu_to_*w() and *_to_cpup(). This is the last lot of conversion patches; I have the "delete the implementations from bswap.h" patch, and will send that out once

Re: [Qemu-devel] [QEMU 4/7] balloon: get free page info from guest

2016-06-19 Thread Li, Liang Z
> On Mon, Jun 13, 2016 at 06:16:46PM +0800, Liang Li wrote: > > Add a new feature to get the free page information from guest, the > > free page information is saved in a bitmap. Please note that 'free > > page' only means these pages are free before the request, some of the > > pages will become n

Re: [Qemu-devel] [QEMU 7/7] migration: skip free pages during live migration

2016-06-19 Thread Li, Liang Z
> On Mon, Jun 13, 2016 at 06:16:49PM +0800, Liang Li wrote: > > After sending out the request for free pages, live migration process > > will start without waiting for the free page bitmap is ready. If the > > free page bitmap is not ready when doing the 1st > > migration_bitmap_sync() after ram_sa

Re: [Qemu-devel] [PATCH] vmxnet3: Fix reading/writing guest memory specially when behind an IOMMU

2016-06-19 Thread Jason Wang
On 2016年06月18日 18:24, KarimAllah Ahmed wrote: When a PCI device lives behind an IOMMU, it should use 'pci_dma_*' family of functions when any transfer from/to guest memory is required while 'cpu_physical_memory_*' family of functions completely bypass any MMU/IOMMU in the system. vmxnet3 was e

Re: [Qemu-devel] [RFC PATCH V4 0/4] Introduce COLO-compare

2016-06-19 Thread Jason Wang
On 2016年06月17日 10:25, Zhang Chen wrote: Hi~ jason. I tried a lot of ways to make it run in compare thread, but it not work. Because that: void g_main_context_push_thread_default (GMainContext *context); Acquires context and sets it as the thread-default context for the current thread. So,

Re: [Qemu-devel] [PATCH v4 3/3] i386: publish advised value of MSR_IA32_FEATURE_CONTROL via fw_cfg

2016-06-19 Thread Haozhong Zhang
On 06/17/16 13:21, Raj, Ashok wrote: > On Fri, Jun 17, 2016 at 07:31:08PM +0200, Laszlo Ersek wrote: > > >> > > >> On 16/06/2016 08:06, Haozhong Zhang wrote: > > >>> It's a prerequisite that certain bits of MSR_IA32_FEATURE_CONTROL should > > >>> be set before some features (e.g. VMX and LMCE) can

Re: [Qemu-devel] [QEMU 6/7] kvm: Add two new arch specific functions

2016-06-19 Thread Li, Liang Z
> > --- a/target-arm/kvm.c > > +++ b/target-arm/kvm.c > > @@ -627,3 +627,17 @@ int kvm_arch_msi_data_to_gsi(uint32_t data) { > > return (data - 32) & 0x; > > } > > + > > +unsigned long get_guest_max_pfn(void) { > > +/* To be done */ > > + > > +return 0; > > +} > > + > > +unsigned

Re: [Qemu-devel] [PATCH] pci_register_bar: cleanup

2016-06-19 Thread Cao jin
Hi Michael, Would you please take a look at this one? It has been forgotten for a long time. On 05/18/2016 08:31 PM, Michael S. Tsirkin wrote: On Wed, May 18, 2016 at 02:17:20PM +0200, Paolo Bonzini wrote: On 18/05/2016 14:16, Cao jin wrote: I guess maybe this one is more suitable for trivi

Re: [Qemu-devel] [RFC PATCH V4 0/4] Introduce COLO-compare

2016-06-19 Thread Zhang Chen
On 06/20/2016 11:03 AM, Jason Wang wrote: On 2016年06月17日 10:25, Zhang Chen wrote: Hi~ jason. I tried a lot of ways to make it run in compare thread, but it not work. Because that: void g_main_context_push_thread_default (GMainContext *context); Acquires context and sets it as the thread-de

Re: [Qemu-devel] [V11 2/4] hw/i386: ACPI IVRS table

2016-06-19 Thread Peter Xu
On Sat, Jun 18, 2016 at 02:34:26PM +0200, Jan Kiszka wrote: > On 2016-06-18 14:32, Peter Xu wrote: > > On Sat, Jun 18, 2016 at 11:18:29AM +0300, David Kiarie wrote: [...] > >> I missed this is the last version of the patch I should fix it in next > >> version. > >> > >> On taking a closer look at

Re: [Qemu-devel] [PATCH 1/2] hw/misc: Add a model for the ASPEED System Control Unit

2016-06-19 Thread Andrew Jeffery
On Fri, 2016-06-17 at 15:22 +0100, Peter Maydell wrote: On 16 June 2016 at 08:48, Andrew Jeffery wrote: The SCU is a collection of chip-level control registers that manage the various functions supported by the AST2400. Typically the bits control interactions with clocks, external hardware or re

Re: [Qemu-devel] [PATCH v3 0/2] Let PKGVERSION include the "git describe" output

2016-06-19 Thread Changlong Xie
For this patch, i think you forget to add qemu-version.h and/or qemu-version.h.tmp in .gitignore changlox ~/w/qemu/qemu% gs On branch master Your branch is up-to-date with 'origin/master'. Untracked files: (use "git add ..." to include in what will be committed) qemu-version.h

[Qemu-devel] [Bug 1594239] [NEW] After adding more scsi disks for Aarch64 virtual machine, start the VM and got Qemu Error

2016-06-19 Thread Kevin Zhao
Public bug reported: Description === Using virt-manager to create a VM in Aarch64, Ubuntu 16.04. Add scsi disk to the VM. After add four or more scsi disks, start the VM and will got Qemu error. Steps to reproduce == 1.Use virt-manager to create a VM. 2.After the VM is st

Re: [Qemu-devel] [RFC PATCH V4 0/4] Introduce COLO-compare

2016-06-19 Thread Jason Wang
On 2016年06月20日 11:27, Zhang Chen wrote: On 06/20/2016 11:03 AM, Jason Wang wrote: On 2016年06月17日 10:25, Zhang Chen wrote: Hi~ jason. I tried a lot of ways to make it run in compare thread, but it not work. Because that: void g_main_context_push_thread_default (GMainContext *context);

Re: [Qemu-devel] [PATCH 2/2] Explicitly print out default vnc option in use

2016-06-19 Thread Markus Armbruster
Robert Hu writes: > On Wed, 2016-06-08 at 16:22 +0200, Markus Armbruster wrote: >> Robert Hu writes: >> >> > On Mon, 2016-06-06 at 09:28 +0200, Markus Armbruster wrote: >> >> Robert Hu writes: >> >> >> >> > On Tue, 2016-05-31 at 13:17 +0200, Markus Armbruster wrote: >> >> >> Robert Hu writes

Re: [Qemu-devel] [Xen-devel] [PATCH v2] xen: fix qdisk BLKIF_OP_DISCARD for 32/64 word size mix

2016-06-19 Thread Juergen Gross
On 17/06/16 18:10, Stefano Stabellini wrote: > On Fri, 17 Jun 2016, Paul Durrant wrote: >>> -Original Message- >>> From: Juergen Gross [mailto:jgr...@suse.com] >>> Sent: 17 June 2016 11:40 >>> To: Paul Durrant; Jan Beulich >>> Cc: Anthony Perard; xen-devel; sstabell...@kernel.org; qemu- >>>

[Qemu-devel] [PATCH v9 05/13] mptsas: change msi property type

2016-06-19 Thread Cao jin
>From uint32 to enum OnOffAuto, and give it a shorter name. cc: Paolo Bonzini cc: Michael S. Tsirkin cc: Markus Armbruster cc: Marcel Apfelbaum Reviewed-by: Markus Armbruster Signed-off-by: Cao jin --- hw/scsi/mptsas.c | 5 +++-- hw/scsi/mptsas.h | 3 ++- 2 files changed, 5 insertions(+),

[Qemu-devel] [PATCH v9 07/13] pci bridge dev: change msi property type

2016-06-19 Thread Cao jin
>From bit to enum OnOffAuto. cc: Michael S. Tsirkin cc: Markus Armbruster cc: Marcel Apfelbaum Reviewed-by: Markus Armbruster Signed-off-by: Cao jin --- hw/pci-bridge/pci_bridge_dev.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/hw/pci-bridge/pci_bridge_

[Qemu-devel] [PATCH v9 10/13] mptsas: remove unnecessary internal msi state flag

2016-06-19 Thread Cao jin
internal flag msi_in_use in unnecessary, msi_uninit() could be called directly, and msi_enabled() is enough to check device msi state. cc: Markus Armbruster cc: Marcel Apfelbaum cc: Paolo Bonzini cc: Michael S. Tsirkin Reviewed-by: Markus Armbruster Signed-off-by: Cao jin --- hw/scsi/mptsa

[Qemu-devel] [PATCH v9 04/13] intel-hda: change msi property type

2016-06-19 Thread Cao jin
>From uint32 to enum OnOffAuto. cc: Gerd Hoffmann cc: Michael S. Tsirkin cc: Markus Armbruster cc: Marcel Apfelbaum Reviewed-by: Markus Armbruster Signed-off-by: Cao jin --- hw/audio/intel-hda.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/hw/audio/intel-hda

[Qemu-devel] [PATCH v9 01/13] change pvscsi_init_msi() type to void

2016-06-19 Thread Cao jin
Nobody use its return value, so change the type to void. cc: Michael S. Tsirkin cc: Paolo Bonzini cc: Markus Armbruster cc: Marcel Apfelbaum Reviewed-by: Markus Armbruster Acked-by: Dmitry Fleytman Reviewed-by: Marcel Apfelbaum Signed-off-by: Cao jin --- hw/scsi/vmw_pvscsi.c | 4 +--- 1

Re: [Qemu-devel] [Qemu-ppc] [PATCH 01/10] ppc: Fix rfi/rfid/hrfi/... emulation

2016-06-19 Thread Cédric Le Goater
On 06/19/2016 11:12 PM, Benjamin Herrenschmidt wrote: > On Sun, 2016-06-19 at 19:23 +0200, Cédric Le Goater wrote: >>> You can run a 32-bit OS or firmware on ppc64, but it needs to know that >>> it's running on a 64-bit chip and do a few things differently. >> >> yes sure but qemu would still allow

[Qemu-devel] [PATCH v9 00/13] Add param Error ** for msi_init()--part2

2016-06-19 Thread Cao jin
rebased against upstream, and passed make check. changelog: 1. vmw_pvscsi: for compatibility, leave the field msi_used alone. 2. since patch "msi_init: change return value to 0 on success" has been adopted first, the patch "megasas: Fix check for msi_init() failure" isn't necessary anymore,

Re: [Qemu-devel] [RFC 0/9] Introduce light weight PC platform pc-lite

2016-06-19 Thread Chao Peng
On Fri, Jun 17, 2016 at 03:24:59PM +0200, Paolo Bonzini wrote: > > > On 17/06/2016 10:14, Chao Peng wrote: > > Basically: > > - it removes old ISA devices and support only PCI devices; > > I think you need to keep at least the RTC, otherwise where does Linux > get the time of day from? PV clock

[Qemu-devel] [PATCH v9 06/13] megasas: change msi/msix property type

2016-06-19 Thread Cao jin
>From bit to enum OnOffAuto. cc: Hannes Reinecke cc: Paolo Bonzini cc: Michael S. Tsirkin cc: Markus Armbruster cc: Marcel Apfelbaum Reviewed-by: Markus Armbruster Signed-off-by: Cao jin --- hw/scsi/megasas.c | 28 +++- 1 file changed, 11 insertions(+), 17 deletion

[Qemu-devel] [PATCH v9 02/13] mptsas: change .realize function name

2016-06-19 Thread Cao jin
All the other devices` .realize function name are xxx_realize, except this one. cc: Michael S. Tsirkin cc: Marcel Apfelbaum cc: Paolo Bonzini cc: Markus Armbruster Reviewed-by: Markus Armbruster Reviewed-by: Marcel Apfelbaum Signed-off-by: Cao jin --- hw/scsi/mptsas.c | 4 ++-- 1 file cha

Re: [Qemu-devel] [PATCH v9 06/13] megasas: change msi/msix property type

2016-06-19 Thread Hannes Reinecke
On 06/20/2016 08:13 AM, Cao jin wrote: > From bit to enum OnOffAuto. > > cc: Hannes Reinecke > cc: Paolo Bonzini > cc: Michael S. Tsirkin > cc: Markus Armbruster > cc: Marcel Apfelbaum > > Reviewed-by: Markus Armbruster > Signed-off-by: Cao jin > --- > hw/scsi/megasas.c | 28 +++--

Re: [Qemu-devel] [Qemu-ppc] [PATCH 01/10] ppc: Fix rfi/rfid/hrfi/... emulation

2016-06-19 Thread Cédric Le Goater
On 06/20/2016 04:19 AM, David Gibson wrote: > On Mon, Jun 20, 2016 at 07:12:38AM +1000, Benjamin Herrenschmidt wrote: >> On Sun, 2016-06-19 at 19:23 +0200, Cédric Le Goater wrote: You can run a 32-bit OS or firmware on ppc64, but it needs to know that it's running on a 64-bit chip and do

[Qemu-devel] [PATCH v9 09/13] megasas: remove unnecessary megasas_use_msi()

2016-06-19 Thread Cao jin
megasas overwrites user configuration when msi_init fail to flag internal msi state, which is unsuitable. megasa_use_msi() is unnecessary, we can call msi_uninit() directly when unrealize, even no need to call msi_enabled() first. cc: Hannes Reinecke cc: Paolo Bonzini cc: Markus Armbruster cc:

[Qemu-devel] [PATCH v9 03/13] usb xhci: change msi/msix property type

2016-06-19 Thread Cao jin
>From bit to enum OnOffAuto cc: Gerd Hoffmann cc: Michael S. Tsirkin cc: Markus Armbruster cc: Marcel Apfelbaum Reviewed-by: Markus Armbruster Signed-off-by: Cao jin --- hw/usb/hcd-xhci.c | 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/hw/usb/hcd-xhci.c

Re: [Qemu-devel] [RFC 0/9] Introduce light weight PC platform pc-lite

2016-06-19 Thread Chao Peng
On Sun, Jun 19, 2016 at 06:51:04AM +0300, Michael S. Tsirkin wrote: > On Fri, Jun 17, 2016 at 04:14:08AM -0400, Chao Peng wrote: > > - it is FAST; > > Any numbers to demonstrate just how fast it is and fast at what? On a 2.30GHz Haswell server, guest kernel booting time is 59.9ms by following tes

[Qemu-devel] [PATCH v9 13/13] vmw_pvscsi: remove unnecessary internal msi state flag

2016-06-19 Thread Cao jin
Internal flag msi_used is uncesessary, msi_uninit() could be called directly, msi_enabled() is enough to check device msi state. But for migration compatibility, keep the field in structure. cc: Paolo Bonzini cc: Dmitry Fleytman cc: Markus Armbruster cc: Marcel Apfelbaum cc: Michael S. Tsirki

[Qemu-devel] [PATCH v9 12/13] e1000e: remove unnecessary internal msi state flag

2016-06-19 Thread Cao jin
Internal big flag E1000E_USE_MSI is unnecessary, also is the helper function: e1000e_init_msi(), e1000e_cleanup_msi(), so, remove them all. cc: Dmitry Fleytman cc: Jason Wang cc: Markus Armbruster cc: Marcel Apfelbaum cc: Michael S. Tsirkin Signed-off-by: Cao jin --- hw/net/e1000e.c | 33 +

Re: [Qemu-devel] [PATCH 2/3] replay: allow replay stopping and restarting

2016-06-19 Thread Pavel Dovgalyuk
> From: Paolo Bonzini [mailto:pbonz...@redhat.com] > > From: "Pavel Dovgalyuk" > > This patch fixes bug with stopping and restarting replay > > through monitor. > > > > Signed-off-by: Pavel Dovgalyuk > > --- > > block/blkreplay.c| 18 +- > > cpus.c |

  1   2   >