Re: [Qemu-devel] [PATCH v3 41/45] windbg: implemented kd_api_search_memory

2017-11-28 Thread Ladi Prosek
On Tue, Nov 21, 2017 at 3:11 PM, Mihail Abakumov wrote: > Signed-off-by: Mihail Abakumov > Signed-off-by: Pavel Dovgalyuk > Signed-off-by: Dmitriy Koltunov > --- > include/exec/windbgstub-utils.h |

Re: [Qemu-devel] [PATCH] BCM2837 and machine raspi3

2017-11-28 Thread bzt bzt
On Tue, Nov 28, 2017 at 6:54 PM, Andrew Baumann < andrew.baum...@microsoft.com> wrote: > > From: bzt bzt [mailto:bztem...@gmail.com] > > Sent: Tuesday, 28 November 2017 03:27 > > > Yes, I agree. I've provided a parameterised version on Oct 24, which > does not > > have a separate bcm2837

Re: [Qemu-devel] [PATCH] i386: turn off l3-cache property by default

2017-11-28 Thread Longpeng (Mike)
On 2017/11/29 14:01, Roman Kagan wrote: > On Wed, Nov 29, 2017 at 01:20:38PM +0800, Longpeng (Mike) wrote: >> On 2017/11/29 5:13, Eduardo Habkost wrote: >> >>> [CCing the people who were copied in the original patch that >>> enabled l3cache] >>> >>> On Tue, Nov 28, 2017 at 11:20:27PM +0300,

Re: [Qemu-devel] [PATCH v3 40/45] windbg: implemented kd_api_read_msr and kd_api_write_msr

2017-11-28 Thread Ladi Prosek
On Tue, Nov 21, 2017 at 3:11 PM, Mihail Abakumov wrote: > Signed-off-by: Mihail Abakumov > Signed-off-by: Pavel Dovgalyuk > Signed-off-by: Dmitriy Koltunov > --- > include/exec/windbgstub-utils.h |

Re: [Qemu-devel] [PATCH] BCM2837 and machine raspi3

2017-11-28 Thread bzt bzt
On Tue, Nov 28, 2017 at 12:56 PM, Peter Maydell wrote: > On 28 November 2017 at 11:26, bzt bzt wrote: > > (Although I have a question. I'm not sure what's the preferred > > way to get MachineClass* object in bcm2836. Use a MachineState* cast on >

Re: [Qemu-devel] [PATCH v3 35/45] windbg: debug exception subscribing

2017-11-28 Thread Ladi Prosek
On Tue, Nov 21, 2017 at 3:10 PM, Mihail Abakumov wrote: > Added handler registration of gdb debug exception. Its exception also can be > used for windbg. > > Signed-off-by: Mihail Abakumov > Signed-off-by: Pavel Dovgalyuk

[Qemu-devel] [Bug 1735082] [NEW] NVME pass through in th eguest VM

2017-11-28 Thread Nitin
Public bug reported: Hi Qemu Team i am new in qemu and trying for nvme pass through .. for that i used below git repo for nvme https://github.com/famz/qemu/tree/nvme and trying to launch the VM by below qemu command .. /usr/local/bin/qemu-system-x86_64 -name sl7.0  -m 1024 -object memory-

Re: [Qemu-devel] [PATCH] thread: move detach_thread from creating thread to created thread

2017-11-28 Thread no-reply
Hi, This series failed automatic build test. Please find the testing commands and their output below. If you have docker installed, you can probably reproduce it locally. Subject: [Qemu-devel] [PATCH] thread: move detach_thread from creating thread to created thread Type: series Message-id:

Re: [Qemu-devel] Block layer complexity: what to do to keep it under control?

2017-11-28 Thread Jeff Cody
On Wed, Nov 29, 2017 at 11:55:02AM +0800, Fam Zheng wrote: > Hi all, > > As we move forwards with new features in the block layer, the chances of > tricky > bugs happening have been increasing alongside - block jobs, coroutines, > throttling, AioContext, op blockers and image locking combined

Re: [Qemu-devel] device_del fail

2017-11-28 Thread Fam Zheng
On Wed, 11/29 13:43, Lying wrote: > Hello everybody, I encounter a error on my vm guest > I remove my device on my guest is failed, following is detail message. > I didn't add my audio device before running my guest, So i add audio with > "device_add" after running > Then i can use it, so i

Re: [Qemu-devel] [PATCH] i386: turn off l3-cache property by default

2017-11-28 Thread Roman Kagan
On Wed, Nov 29, 2017 at 06:17:40AM +0200, Michael S. Tsirkin wrote: > On Tue, Nov 28, 2017 at 11:20:27PM +0300, Denis V. Lunev wrote: > > > There's one thing I don't understand in your test case: if you > > > just found out that Linux will behave worse if it assumes that > > > the VCPUs are

[Qemu-devel] device_del fail

2017-11-28 Thread Lying
Hello everybody, I encounter a error on my vm guest I remove my device on my guest is failed, following is detail message. I didn't add my audio device before running my guest, So i add audio with "device_add" after running Then i can use it, so i remove it with "device_del", but when i do it

Re: [Qemu-devel] [PATCH] i386: turn off l3-cache property by default

2017-11-28 Thread Gonglei (Arei)
> -Original Message- > From: rka...@virtuozzo.com [mailto:rka...@virtuozzo.com] > Sent: Wednesday, November 29, 2017 1:56 PM > To: Gonglei (Arei) > Cc: Eduardo Habkost; Denis V. Lunev; longpeng; Michael S. Tsirkin; Denis > Plotnikov; pbonz...@redhat.com; r...@twiddle.net;

Re: [Qemu-devel] [PATCH] i386: turn off l3-cache property by default

2017-11-28 Thread Roman Kagan
On Wed, Nov 29, 2017 at 01:20:38PM +0800, Longpeng (Mike) wrote: > On 2017/11/29 5:13, Eduardo Habkost wrote: > > > [CCing the people who were copied in the original patch that > > enabled l3cache] > > > > On Tue, Nov 28, 2017 at 11:20:27PM +0300, Denis V. Lunev wrote: > >> On 11/28/2017 10:58

Re: [Qemu-devel] [PATCH] i386: turn off l3-cache property by default

2017-11-28 Thread rka...@virtuozzo.com
On Wed, Nov 29, 2017 at 01:57:14AM +, Gonglei (Arei) wrote: > > On Tue, Nov 28, 2017 at 11:20:27PM +0300, Denis V. Lunev wrote: > > > On 11/28/2017 10:58 PM, Eduardo Habkost wrote: > > > > On Fri, Nov 24, 2017 at 04:26:50PM +0300, Denis Plotnikov wrote: > > > >> Commit 14c985cffa "target-i386:

Re: [Qemu-devel] [PATCH] i386: turn off l3-cache property by default

2017-11-28 Thread Roman Kagan
On Tue, Nov 28, 2017 at 07:13:26PM -0200, Eduardo Habkost wrote: > [CCing the people who were copied in the original patch that > enabled l3cache] Thanks, and sorry to have forgotten to do so in the patch! > On Tue, Nov 28, 2017 at 11:20:27PM +0300, Denis V. Lunev wrote: > > On 11/28/2017 10:58

Re: [Qemu-devel] [PATCH] i386: turn off l3-cache property by default

2017-11-28 Thread Longpeng (Mike)
On 2017/11/29 5:13, Eduardo Habkost wrote: > [CCing the people who were copied in the original patch that > enabled l3cache] > > On Tue, Nov 28, 2017 at 11:20:27PM +0300, Denis V. Lunev wrote: >> On 11/28/2017 10:58 PM, Eduardo Habkost wrote: >>> Hi, >>> >>> On Fri, Nov 24, 2017 at 04:26:50PM

Re: [Qemu-devel] [PATCH 12/25] spapr: introduce a XIVE interrupt presenter model

2017-11-28 Thread David Gibson
On Thu, Nov 23, 2017 at 02:29:42PM +0100, Cédric Le Goater wrote: > The XIVE interrupt presenter exposes a set of rings, also called > Thread Interrupt Management Areas (TIMA), to handle priority > management and interrupt acknowledgment among other things. There is > one ring per level of

Re: [Qemu-devel] [PATCH 08/25] spapr: introduce a skeleton for the XIVE interrupt controller

2017-11-28 Thread David Gibson
On Tue, Nov 28, 2017 at 10:44:03AM +, Cédric Le Goater wrote: > On 11/28/2017 05:40 AM, David Gibson wrote: > > On Thu, Nov 23, 2017 at 02:29:38PM +0100, Cédric Le Goater wrote: > >> The XIVE interrupt controller uses a set of tables to redirect exception > >> from event sources to CPU

Re: [Qemu-devel] [PATCH 10/25] spapr: add MMIO handlers for the XIVE interrupt sources

2017-11-28 Thread David Gibson
On Tue, Nov 28, 2017 at 06:33:06PM +, Cédric Le Goater wrote: > On 11/28/2017 06:38 AM, David Gibson wrote: > > On Thu, Nov 23, 2017 at 02:29:40PM +0100, Cédric Le Goater wrote: > >> Each interrupt source is associated with a two bit state machine > >> called an Event State Buffer (ESB). The

Re: [Qemu-devel] [PATCH] i386: turn off l3-cache property by default

2017-11-28 Thread Roman Kagan
On Tue, Nov 28, 2017 at 08:50:54PM +0100, Paolo Bonzini wrote: > On 28/11/2017 19:54, Michael S. Tsirkin wrote: > >> Now there's a downside: with L3 cache the Linux scheduler is more eager > >> to wake up tasks on sibling CPUs, resulting in unnecessary cross-vCPU > >> interactions and therefore

Re: [Qemu-devel] [Qemu-ppc] [PATCH v2] pseries: fix TCG migration

2017-11-28 Thread Bharata B Rao
On Wed, Nov 29, 2017 at 10:58:29AM +1100, Suraj Jitindar Singh wrote: > On Tue, 2017-11-28 at 18:43 +0100, Laurent Vivier wrote: > > Migration of pseries is broken with TCG because > > QEMU tries to restore KVM MMU state unconditionally. > > > > The result is a SIGSEGV in kvm_vm_ioctl(): > > > >

Re: [Qemu-devel] [PATCH] i386: turn off l3-cache property by default

2017-11-28 Thread Michael S. Tsirkin
On Tue, Nov 28, 2017 at 11:20:27PM +0300, Denis V. Lunev wrote: > > There's one thing I don't understand in your test case: if you > > just found out that Linux will behave worse if it assumes that > > the VCPUs are sharing a L3 cache, why are you configuring a > > 8-core VCPU topology explicitly?

Re: [Qemu-devel] [PATCH 3/4] cryptodev-vhost-user: add crypto session handler

2017-11-28 Thread Michael S. Tsirkin
On Wed, Nov 29, 2017 at 02:33:59AM +, Gonglei (Arei) wrote: > > > -Original Message- > > From: Paolo Bonzini [mailto:pbonz...@redhat.com] > > Sent: Tuesday, November 28, 2017 7:20 PM > > To: Gonglei (Arei); qemu-devel@nongnu.org > > Cc: m...@redhat.com; Huangweidong (C);

[Qemu-devel] [PATCH qemu-web v2] Suggest qemu-web prefix in the introductory post

2017-11-28 Thread Fam Zheng
Chances are qemu-devel@nongnu.org patches only adding new files can be picked up by patchew and treated like a qemu.git patch. Patchew already ignores [qemu-web] patches, so suggest it in this article as well. (Are there better places to document this?) Signed-off-by: Fam Zheng

Re: [Qemu-devel] [PATCH qemu-web] Suggest qemu-web prefix in the introductory post

2017-11-28 Thread Fam Zheng
On Tue, 11/28 20:54, Eric Blake wrote: > On 11/28/2017 07:58 PM, Fam Zheng wrote: > > Chances are qemu-devel@nongnu.org patches only adding new files can be > > picked up by patchew and treated like a qemu.git patch. Patchew already > > ignores [qemu-web] patches, so suggest it in this article as

[Qemu-devel] Block layer complexity: what to do to keep it under control?

2017-11-28 Thread Fam Zheng
Hi all, As we move forwards with new features in the block layer, the chances of tricky bugs happening have been increasing alongside - block jobs, coroutines, throttling, AioContext, op blockers and image locking combined together make a large and complex picture that is hard to fully understand

[Qemu-devel] [Bug 1735049] [NEW] Need MTTCG support for x86 guests

2017-11-28 Thread Timothy Pearson
Public bug reported: MTTCG support is notably absent for x86_64 guests. The last Wiki update on MTTCG was back in 2015, and I am having some difficulty determining the current status of the underlying requirements to enable this feature on x86 hosts. For instance, has support for strong-on-weak

Re: [Qemu-devel] [qemu-s390x] [PATCH v1 for-2.12 03/15] s390x/tcg: rip out dead tpi code

2017-11-28 Thread Thomas Huth
On 28.11.2017 21:33, David Hildenbrand wrote: > It is broken and not even wired up. We'll add a new handler soon, but > that will live somewhere else. > > Signed-off-by: David Hildenbrand > --- > hw/s390x/css.c | 6 -- > include/hw/s390x/css.h | 1 - >

Re: [Qemu-devel] [PATCH qemu-web] Suggest qemu-web prefix in the introductory post

2017-11-28 Thread Eric Blake
On 11/28/2017 07:58 PM, Fam Zheng wrote: Chances are qemu-devel@nongnu.org patches only adding new files can be picked up by patchew and treated like a qemu.git patch. Patchew already ignores [qemu-web] patches, so suggest it in this article as well. (Are there better places to document this?)

Re: [Qemu-devel] [PATCH 3/4] cryptodev-vhost-user: add crypto session handler

2017-11-28 Thread Gonglei (Arei)
> -Original Message- > From: Paolo Bonzini [mailto:pbonz...@redhat.com] > Sent: Tuesday, November 28, 2017 7:20 PM > To: Gonglei (Arei); qemu-devel@nongnu.org > Cc: m...@redhat.com; Huangweidong (C); stefa...@redhat.com; Zhoujian > (jay); pa...@linux.vnet.ibm.com; longpeng;

Re: [Qemu-devel] [PATCH] vhost-user spec: Clarify policy on setting log_base

2017-11-28 Thread Jason Wang
On 2017年11月29日 02:06, Michael S. Tsirkin wrote: On Tue, Nov 28, 2017 at 03:46:44PM +0200, Victor Kaplansky wrote: From: Victor Kaplansky If we allow qemu to change logging area after it was already established, it may require from the backend to acquire a lock

Re: [Qemu-devel] [PATCH] Add a blog post about HAXM acceleration on Windows

2017-11-28 Thread Fam Zheng
On Wed, 11/29 09:30, Yu Ning wrote: > > > On 11/29/2017 9:25, Fam Zheng wrote: > > On Tue, 11/28 13:57, Eric Blake wrote: > > > On 11/22/2017 04:43 AM, no-re...@patchew.org wrote: > > > > Hi, > > > > > > > > This series failed automatic build test. Please find the testing > > > > commands and

[Qemu-devel] [PATCH qemu-web] Suggest qemu-web prefix in the introductory post

2017-11-28 Thread Fam Zheng
Chances are qemu-devel@nongnu.org patches only adding new files can be picked up by patchew and treated like a qemu.git patch. Patchew already ignores [qemu-web] patches, so suggest it in this article as well. (Are there better places to document this?) Signed-off-by: Fam Zheng

Re: [Qemu-devel] [PATCH] i386: turn off l3-cache property by default

2017-11-28 Thread Gonglei (Arei)
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Wednesday, November 29, 2017 5:13 AM > To: Denis V. Lunev; longpeng; Michael S. Tsirkin > Cc: Denis Plotnikov; pbonz...@redhat.com; r...@twiddle.net; > qemu-devel@nongnu.org; rka...@virtuozzo.com; Gonglei

Re: [Qemu-devel] [PATCH v3] vhost: Cancel migration when vhost-user process restarted during migration

2017-11-28 Thread Ying Fang
On 2017/11/28 18:18, Dr. David Alan Gilbert wrote: > * fangying (fangyi...@huawei.com) wrote: >> QEMU will abort when vhost-user process is restarted during migration >> and vhost_log_global_start/stop is called. The reason is clear that >> vhost_dev_set_log returns -1 because network connection

Re: [Qemu-devel] [PATCH] etsec: fix IRQ (un)masking

2017-11-28 Thread David Gibson
On Mon, Nov 27, 2017 at 08:42:13PM -0600, Michael Davidsaver wrote: > Interrupt conditions occurring while masked are not being > signaled when later unmasked. > The fix is to raise/lower IRQs when IMASK is changed. > > To avoid problems like this in future, consolidate > IRQ pin update logic in

Re: [Qemu-devel] [PATCH 03/17] i2c: add mpc8540 i2c controller

2017-11-28 Thread David Gibson
On Mon, Nov 27, 2017 at 01:05:11PM -0600, Michael Davidsaver wrote: > On 11/27/2017 01:12 AM, David Gibson wrote: > > On Sun, Nov 26, 2017 at 03:59:01PM -0600, Michael Davidsaver wrote: > >> Signed-off-by: Michael Davidsaver > >> --- > >> hw/i2c/Makefile.objs | 1 + > >>

Re: [Qemu-devel] [PATCH] Add a blog post about HAXM acceleration on Windows

2017-11-28 Thread Yu Ning
On 11/29/2017 9:25, Fam Zheng wrote: On Tue, 11/28 13:57, Eric Blake wrote: On 11/22/2017 04:43 AM, no-re...@patchew.org wrote: Hi, This series failed automatic build test. Please find the testing commands and their output below. If you have docker installed, you can probably reproduce it

Re: [Qemu-devel] [PATCH v2 0/5] backup improvements part 1

2017-11-28 Thread Fam Zheng
On Tue, 11/28 14:44, Jeff Cody wrote: > > Yes, I can bring in through my tree, unless Fam prefers to have it - I'll > > pull it in via my block-next branch. > > > > Rather than keep this on my email queue: > > Applied to my block-next branch: > > git://github.com/codyprime/qemu-kvm-jtc

Re: [Qemu-devel] [PATCH v2] pseries: fix TCG migration

2017-11-28 Thread David Gibson
On Tue, Nov 28, 2017 at 06:43:10PM +0100, Laurent Vivier wrote: > Migration of pseries is broken with TCG because > QEMU tries to restore KVM MMU state unconditionally. > > The result is a SIGSEGV in kvm_vm_ioctl(): > > #0 kvm_vm_ioctl (s=0x0, type=-2146390353) > at

Re: [Qemu-devel] [PATCH v2 0/5] backup improvements part 1

2017-11-28 Thread Fam Zheng
On Mon, 11/13 17:41, John Snow wrote: > Jeff, I think this ought to go through your tree, but I'll sign off on it. > > Fam: Do you agree? Sounds good. Fam

Re: [Qemu-devel] [PATCH] Add a blog post about HAXM acceleration on Windows

2017-11-28 Thread Fam Zheng
On Tue, 11/28 13:57, Eric Blake wrote: > On 11/22/2017 04:43 AM, no-re...@patchew.org wrote: > > Hi, > > > > This series failed automatic build test. Please find the testing commands > > and > > their output below. If you have docker installed, you can probably > > reproduce it > > locally. > >

Re: [Qemu-devel] [PATCH v3] virtio_balloon: include disk/file caches memory statistics

2017-11-28 Thread Michael S. Tsirkin
On Sun, Nov 12, 2017 at 01:05:38PM +0100, Tomáš Golembiovský wrote: > Add a new field VIRTIO_BALLOON_S_CACHES to virtio_balloon memory > statistics protocol. The value represents all disk/file caches. > > In this case it corresponds to the sum of values > Buffers+Cached+SwapCached from

Re: [Qemu-devel] [PATCH v8 0/5] fw_cfg: add DMA operations & etc/vmcoreinfo support

2017-11-28 Thread Michael S. Tsirkin
On Thu, Nov 23, 2017 at 03:02:01PM +0100, Marc-André Lureau wrote: > Hi, > > This series adds DMA operations support to the qemu fw_cfg kernel > module and populates "etc/vmcoreinfo" with vmcoreinfo location > details. > > Note: the support for this entry handling has been merged for upcoming >

Re: [Qemu-devel] [PULL 1/9] hw/acpi-build: Make assignment statement of next_base easy to read

2017-11-28 Thread Dou Liyang
Hi, Igor, At 11/28/2017 11:07 PM, Igor Mammedov wrote: On Thu, 5 Oct 2017 17:36:30 -0300 Eduardo Habkost wrote: From: Dou Liyang It may be hard to read the assignment statement of "next_base", so S/next_base += (1ULL << 32) -

Re: [Qemu-devel] [PULL 2/9] ACPI/unit-test: Add a new testcase for RAM allocation in numa node

2017-11-28 Thread Dou Liyang
Hi Igor, At 11/28/2017 11:08 PM, Igor Mammedov wrote: On Thu, 5 Oct 2017 17:36:31 -0300 Eduardo Habkost wrote: From: Dou Liyang As QEMU supports the memory-less node, it is possible that there is no RAM in the first numa node(also be called

Re: [Qemu-devel] [PATCH v3 1/3] hostmem-file: add "align" option

2017-11-28 Thread Haozhong Zhang
[ it looks my email client had some glitch and failed to add mst in cc list ] copy mst On 11/27/17 12:35 +0800, Haozhong Zhang wrote: > When mmap(2) the backend files, QEMU uses the host page size > (getpagesize(2)) by default as the alignment of mapping address. > However, some backends may

Re: [Qemu-devel] [PATCH v3 1/3] hostmem-file: add "align" option

2017-11-28 Thread Haozhong Zhang
On 11/27/17 23:07 -0200, Eduardo Habkost wrote: > On Mon, Nov 27, 2017 at 12:35:15PM +0800, Haozhong Zhang wrote: > > When mmap(2) the backend files, QEMU uses the host page size > > (getpagesize(2)) by default as the alignment of mapping address. > > However, some backends may require alignments

Re: [Qemu-devel] [PATCH v3 1/3] hostmem-file: add "align" option

2017-11-28 Thread Haozhong Zhang
copy mst On 11/27/17 12:35 +0800, Haozhong Zhang wrote: > When mmap(2) the backend files, QEMU uses the host page size > (getpagesize(2)) by default as the alignment of mapping address. > However, some backends may require alignments different than the page > size. For example, mmap a device DAX

Re: [Qemu-devel] [Qemu-ppc] [PATCH v2 3/4] tests: ibm, configure-connector RTAS call implementation

2017-11-28 Thread Michael Roth
Quoting Daniel Henrique Barboza (2017-11-09 06:35:30) > > > On 11/06/2017 03:46 PM, Laurent Vivier wrote: > > On 31/10/2017 21:43, Daniel Henrique Barboza wrote: > >> 'ibm,configure-connector' hypercall is used by the guest OS > >> to start the configuration of a given device, where the > >>

Re: [Qemu-devel] [Qemu-ppc] [PATCH v2] pseries: fix TCG migration

2017-11-28 Thread Suraj Jitindar Singh
On Tue, 2017-11-28 at 18:43 +0100, Laurent Vivier wrote: > Migration of pseries is broken with TCG because > QEMU tries to restore KVM MMU state unconditionally. > > The result is a SIGSEGV in kvm_vm_ioctl(): > > #0 kvm_vm_ioctl (s=0x0, type=-2146390353) > at

Re: [Qemu-devel] [Qemu-ppc] [PATCH v2] pseries: fix TCG migration

2017-11-28 Thread Suraj Jitindar Singh
On Tue, 2017-11-28 at 17:26 -0200, Daniel Henrique Barboza wrote: > > On 11/28/2017 03:43 PM, Laurent Vivier wrote: > > Migration of pseries is broken with TCG because > > QEMU tries to restore KVM MMU state unconditionally. > > > > The result is a SIGSEGV in kvm_vm_ioctl(): > > > >#0

Re: [Qemu-devel] [PATCH 2/2] ide: fix crash in IDE cdrom read

2017-11-28 Thread John Snow
On 11/28/2017 07:10 AM, Denis V. Lunev wrote: > There is the following crash reported from the field in QEMU 2.9: > bdrv_inc_in_flight (bs=bs@entry=0x0) > blk_aio_prwv > blk_aio_preadv > ide_buffered_readv > cd_read_sector Is ide_atapi_cmd_reply_end missing from the call

[Qemu-devel] [PATCH] firmware: Use PTR_ERR_OR_ZERO()

2017-11-28 Thread Vasyl Gomonovych
Fix ptr_ret.cocci warnings: drivers/firmware/efi/efi.c:610:8-14: WARNING: PTR_ERR_OR_ZERO can be used Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR Generated by: scripts/coccinelle/api/ptr_ret.cocci Signed-off-by: Vasyl Gomonovych ---

Re: [Qemu-devel] [PATCH v3 0/5] Add a valid_cpu_types property

2017-11-28 Thread Alistair Francis
On Tue, Nov 7, 2017 at 3:30 PM, Alistair Francis wrote: > On Mon, Oct 23, 2017 at 12:26 AM, Alistair Francis > wrote: >> On Wed, Oct 18, 2017 at 12:31 AM, Alistair Francis >> wrote: >>> There are numorous QEMU

Re: [Qemu-devel] [PATCH-2.12 v1 3/3] xilinx_spips: Use memset instead of a for loop to zero registers

2017-11-28 Thread KONRAD Frederic
On 11/28/2017 11:31 PM, Alistair Francis wrote: Use memset() instead of a for loop to zero all of the registers. Signed-off-by: Alistair Francis --- hw/ssi/xilinx_spips.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git

Re: [Qemu-devel] [PATCH] qga-win: VSS: Handle VSS_E_PROVIDER_ALREADY_REGISTERED error

2017-11-28 Thread Michael Roth
Quoting Sameeh Jubran (2017-11-22 05:30:14) > From: Sameeh Jubran > > We can get VSS_E_PROVIDER_ALREADY_REGISTERED error if for some > reason the uninstall of qemu-ga didn't complete successfully. > In this case we can try to unregister the component service > and attempt to

Re: [Qemu-devel] [PATCH-2.12 v1 1/3] xilinx_spips: Update the QSPI Mod ID reset value

2017-11-28 Thread KONRAD Frederic
On 11/28/2017 11:31 PM, Alistair Francis wrote: Update the reset value to match the latest ZynqMP register spec. Signed-off-by: Alistair Francis --- hw/ssi/xilinx_spips.c | 1 + 1 file changed, 1 insertion(+) diff --git a/hw/ssi/xilinx_spips.c

[Qemu-devel] [PATCH-2.12 v1 2/3] xilinx_spis: Set all of the reset values

2017-11-28 Thread Alistair Francis
Following the ZynqMP register spec let's ensure that all reset values are set. Signed-off-by: Alistair Francis --- hw/ssi/xilinx_spips.c | 38 +- include/hw/ssi/xilinx_spips.h | 2 +- 2 files changed, 34 insertions(+), 6

[Qemu-devel] [PATCH-2.12 v1 1/3] xilinx_spips: Update the QSPI Mod ID reset value

2017-11-28 Thread Alistair Francis
Update the reset value to match the latest ZynqMP register spec. Signed-off-by: Alistair Francis --- hw/ssi/xilinx_spips.c | 1 + 1 file changed, 1 insertion(+) diff --git a/hw/ssi/xilinx_spips.c b/hw/ssi/xilinx_spips.c index ad1b2ba79f..899db814ee 100644 ---

[Qemu-devel] [PATCH-2.12 v1 0/3] Update the reset values of the Xilinx ZynqMP QSPI

2017-11-28 Thread Alistair Francis
Update the reset values of the Xilinx ZynqMP QSPI device to match the resister spec here: https://www.xilinx.com/html_docs/registers/ug1087/ug1087-zynq-ultrascale-registers.html Based-on: 20171126231634.9531-14-frasse.igles...@gmail.com Alistair Francis (3): xilinx_spips: Update the QSPI Mod

[Qemu-devel] [PATCH-2.12 v1 3/3] xilinx_spips: Use memset instead of a for loop to zero registers

2017-11-28 Thread Alistair Francis
Use memset() instead of a for loop to zero all of the registers. Signed-off-by: Alistair Francis --- hw/ssi/xilinx_spips.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/hw/ssi/xilinx_spips.c b/hw/ssi/xilinx_spips.c index

Re: [Qemu-devel] [PATCH] i386: turn off l3-cache property by default

2017-11-28 Thread Eduardo Habkost
[CCing the people who were copied in the original patch that enabled l3cache] On Tue, Nov 28, 2017 at 11:20:27PM +0300, Denis V. Lunev wrote: > On 11/28/2017 10:58 PM, Eduardo Habkost wrote: > > Hi, > > > > On Fri, Nov 24, 2017 at 04:26:50PM +0300, Denis Plotnikov wrote: > >> Commit 14c985cffa

Re: [Qemu-devel] [PATCH] vhost-user spec: Clarify policy on setting log_base

2017-11-28 Thread Victor Kaplansky
> From: "Michael S. Tsirkin" > To: "Victor Kaplansky" > Cc: qemu-devel@nongnu.org, "Maxime Coquelin" , > "Jason Wang" > Sent: Tuesday, November 28, 2017 8:45:43 PM > Subject: Re: [PATCH] vhost-user spec:

[Qemu-devel] [PATCH v1 for-2.12 15/15] s390x/tcg: drop potential_page_fault()

2017-11-28 Thread David Hildenbrand
Only one user left, get rid of it so we don't get any new users. Signed-off-by: David Hildenbrand --- target/s390x/translate.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/target/s390x/translate.c b/target/s390x/translate.c index

[Qemu-devel] [PATCH v1 for-2.12 14/15] s390x/tcg: drop program_interrupt()

2017-11-28 Thread David Hildenbrand
All users are gone, we can finally drop it and make sure that all new new program interrupt injections are reminded of the retaddr - as they have to use program_interrupt_ra() now. Signed-off-by: David Hildenbrand --- target/s390x/cpu.h | 1 - target/s390x/interrupt.c |

[Qemu-devel] [PATCH v1 for-2.12 12/15] s390x/tcg: use program_interrupt_ra() in SACF

2017-11-28 Thread David Hildenbrand
Convert this user, too. Signed-off-by: David Hildenbrand --- target/s390x/cc_helper.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/s390x/cc_helper.c b/target/s390x/cc_helper.c index f008897e84..5ead3abbc1 100644 --- a/target/s390x/cc_helper.c +++

[Qemu-devel] [PATCH v1 for-2.12 09/15] s390x/tcg: use program_interrupt_ra() in SCLP Service Call

2017-11-28 Thread David Hildenbrand
Now we can drop potential_page_fault(). Signed-off-by: David Hildenbrand --- target/s390x/misc_helper.c | 3 +-- target/s390x/translate.c | 1 - 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/target/s390x/misc_helper.c b/target/s390x/misc_helper.c index

[Qemu-devel] [PATCH v1 for-2.12 08/15] s390x/tcg: io instructions don't need potential_page_fault()

2017-11-28 Thread David Hildenbrand
As we handle the retaddr in all cases properly now, we can drop it. Signed-off-by: David Hildenbrand --- target/s390x/translate.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/target/s390x/translate.c b/target/s390x/translate.c index 85d0a6c3af..d0859c4bc7

[Qemu-devel] [PATCH v1 for-2.12 13/15] s390x/tcg: use program_interrupt_ra() in STSI

2017-11-28 Thread David Hildenbrand
STSI needs some more love, but let's do one step at a time. We can now drop potential_page_fault(). Signed-off-by: David Hildenbrand --- target/s390x/misc_helper.c | 2 +- target/s390x/translate.c | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git

[Qemu-devel] [PATCH v1 for-2.12 07/15] s390x: handle exceptions during s390_cpu_virt_mem_rw() correctly (TCG)

2017-11-28 Thread David Hildenbrand
s390_cpu_virt_mem_rw() must always return, so callers can react on an exception (e.g. see ioinst_handle_stcrw()). For TCG, there was one case where a cpu loop exit was triggered. Fix that up. However, for TCG we always have to exit the cpu loop (and restore the cpu state before that) if we

[Qemu-devel] [PATCH v1 for-2.12 10/15] s390x/tcg: use program_interrupt_ra() in DIAG

2017-11-28 Thread David Hildenbrand
Now we can drop the two save statements in the translate function. Signed-off-by: David Hildenbrand --- target/s390x/misc_helper.c | 2 +- target/s390x/translate.c | 3 --- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/target/s390x/misc_helper.c

[Qemu-devel] [PATCH v1 for-2.12 11/15] s390x/tcg: use program_interrupt_ra() in per_check_exception()

2017-11-28 Thread David Hildenbrand
We can now drop updating the cc. Signed-off-by: David Hildenbrand --- target/s390x/misc_helper.c | 2 +- target/s390x/translate.c | 3 --- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/target/s390x/misc_helper.c b/target/s390x/misc_helper.c index

[Qemu-devel] [PATCH v1 for-2.12 03/15] s390x/tcg: rip out dead tpi code

2017-11-28 Thread David Hildenbrand
It is broken and not even wired up. We'll add a new handler soon, but that will live somewhere else. Signed-off-by: David Hildenbrand --- hw/s390x/css.c | 6 -- include/hw/s390x/css.h | 1 - target/s390x/internal.h | 1 - target/s390x/ioinst.c | 26

[Qemu-devel] [PATCH v1 for-2.12 05/15] s390x/pci: pass the retaddr to all PCI instructions

2017-11-28 Thread David Hildenbrand
Once we wire up TCG, we will need the retaddr to correctly inject program interrupts. As we want to get rid of the function program_interrupt(), convert PCI code too. For KVM, we can simply pass a 0. Convert program_interrupt() to program_interrupt_ra() directly, making use of the passed

[Qemu-devel] [PATCH v1 for-2.12 06/15] s390x/diag: pass the retaddr into handle_diag_308()

2017-11-28 Thread David Hildenbrand
Needed to later drop potential_page_fault() from the diag TCG translate function. Convert program_interrupt() to program_interrupt_ra() directly, making use of the passed address. Signed-off-by: David Hildenbrand --- target/s390x/diag.c| 14 +++---

[Qemu-devel] [PATCH v1 for-2.12 04/15] s390x/ioinst: pass the retaddr to all IO instructions

2017-11-28 Thread David Hildenbrand
TCG needs the retaddr when injecting an interrupt. Let's just pass it along and use 0 for KVM. The value will be completely ignored for KVM. Convert program_interrupt() to program_interrupt_ra() directly, making use of the passed address. Signed-off-by: David Hildenbrand ---

[Qemu-devel] [PATCH v1 for-2.12 02/15] s390x/tcg: get rid of runtime_exception()

2017-11-28 Thread David Hildenbrand
Let's use program_interrupt_ra() instead. Signed-off-by: David Hildenbrand --- target/s390x/fpu_helper.c | 2 +- target/s390x/int_helper.c | 14 +++--- target/s390x/internal.h| 2 -- target/s390x/misc_helper.c | 16 4 files changed, 8

[Qemu-devel] [PATCH v1 for-2.12 00/15] s390x/tcg: cleanup and fix program interrupts

2017-11-28 Thread David Hildenbrand
I have quite some patches on my queue for 2.12. (booting Fedora 26/27 guests, floating interrupts, machine checks, missing instructions ...) So let's start slowly :) This series gets rid of program_interrupt() and potential_page_fault(). We now always properly restore the cpu state when

[Qemu-devel] [PATCH v1 for-2.12 01/15] s390x/tcg: introduce and use program_interrupt_ra()

2017-11-28 Thread David Hildenbrand
Allows to easily convert more callers of program_interrupt() and to easily introduce new exceptions without forgetting about the cpu state reset. Use program_interrupt_ra() in places where we already had the same pattern. Signed-off-by: David Hildenbrand ---

Re: [Qemu-devel] [PATCH v3 1/3] Switch AMD CFI flash to use new MMIO API

2017-11-28 Thread Eric Blake
On 11/28/2017 02:05 PM, Eric Blake wrote: Better yet, please fix the PFLASH_DEBUG code to avoid bitrot in the first place, by rewriting the bad pattern: into the good pattern: #ifdef PFLASH_DEBUG # define PFLASH_PRINT 1 #else # define PFLASH_PRINT 0 #endif #define DPRINTF(fmt, ...) \ do {

Re: [Qemu-devel] [PATCH] i386: turn off l3-cache property by default

2017-11-28 Thread Denis V. Lunev
On 11/28/2017 10:58 PM, Eduardo Habkost wrote: > Hi, > > On Fri, Nov 24, 2017 at 04:26:50PM +0300, Denis Plotnikov wrote: >> Commit 14c985cffa "target-i386: present virtual L3 cache info for vcpus" >> introduced and set by default exposing l3 to the guest. >> >> The motivation behind it was that

Re: [Qemu-devel] [PATCH] i386: turn off l3-cache property by default

2017-11-28 Thread Eduardo Habkost
Hi, On Fri, Nov 24, 2017 at 04:26:50PM +0300, Denis Plotnikov wrote: > Commit 14c985cffa "target-i386: present virtual L3 cache info for vcpus" > introduced and set by default exposing l3 to the guest. > > The motivation behind it was that in the Linux scheduler, when waking up > a task on a

Re: [Qemu-devel] [PATCH] i386: turn off l3-cache property by default

2017-11-28 Thread Eduardo Habkost
On Tue, Nov 28, 2017 at 08:50:54PM +0100, Paolo Bonzini wrote: > On 28/11/2017 19:54, Michael S. Tsirkin wrote: > >> Now there's a downside: with L3 cache the Linux scheduler is more eager > >> to wake up tasks on sibling CPUs, resulting in unnecessary cross-vCPU > >> interactions and therefore

Re: [Qemu-devel] [PATCH v3 1/3] Switch AMD CFI flash to use new MMIO API

2017-11-28 Thread Eric Blake
On 11/28/2017 01:47 PM, Michael Nawrocki wrote: I suspect you'll find that the change of type for 'ret' here and the 'value' argument to pflash_write() will break compilation with PFLASH_DEBUG defined, because the type won't match the DPRINTF format strings any more. You could either fix up the

Re: [Qemu-devel] [PATCH] Add a blog post about HAXM acceleration on Windows

2017-11-28 Thread Eric Blake
On 11/22/2017 04:43 AM, no-re...@patchew.org wrote: Hi, This series failed automatic build test. Please find the testing commands and their output below. If you have docker installed, you can probably reproduce it locally. Subject: [Qemu-devel] [PATCH] Add a blog post about HAXM acceleration

Re: [Qemu-devel] [PATCH v4 1/1] migration: Use proper types in json

2017-11-28 Thread Eric Blake
On 11/22/2017 06:45 AM, Juan Quintela wrote: We use int for everything (int64_t), and then we check that value is between 0 and 255. Change it to the valid types. This change only happens for HMP. QMP always use bytes and similars. s/similars/similar/ Signed-off-by: Juan Quintela

Re: [Qemu-devel] [PATCH] i386: turn off l3-cache property by default

2017-11-28 Thread Paolo Bonzini
On 28/11/2017 19:54, Michael S. Tsirkin wrote: >> Now there's a downside: with L3 cache the Linux scheduler is more eager >> to wake up tasks on sibling CPUs, resulting in unnecessary cross-vCPU >> interactions and therefore exessive halts and IPIs. E.g. "perf bench >> sched pipe -i 10" gives

Re: [Qemu-devel] [PATCH v3 3/3] Add partial flash interleaving to AMD CFI devices

2017-11-28 Thread Michael Nawrocki
On 11/23/2017 08:46 AM, Peter Maydell wrote: On 13 November 2017 at 17:31, Mike Nawrocki wrote: This mirrors the interleaving support already in place in pflash_cfi01.c, using the max_device_width and device_width properties. Signed-off-by: Mike Nawrocki

Re: [Qemu-devel] [PATCH v3 2/3] Enable 8-byte wide access to AMD CFI devices

2017-11-28 Thread Michael Nawrocki
On 11/23/2017 06:49 AM, Peter Maydell wrote: On 13 November 2017 at 16:14, Mike Nawrocki wrote: Signed-off-by: Mike Nawrocki --- hw/block/pflash_cfi02.c | 143 ++-- 1 file

Re: [Qemu-devel] [PATCH v3 1/3] Switch AMD CFI flash to use new MMIO API

2017-11-28 Thread Michael Nawrocki
On 11/23/2017 06:27 AM, Peter Maydell wrote: On 23 November 2017 at 11:26, Peter Maydell wrote: On 13 November 2017 at 16:14, Mike Nawrocki wrote: Signed-off-by: Mike Nawrocki ---

Re: [Qemu-devel] [PATCH v2 0/5] backup improvements part 1

2017-11-28 Thread Jeff Cody
On Tue, Nov 14, 2017 at 12:00:06AM -0500, Jeff Cody wrote: > On Mon, Nov 13, 2017 at 05:41:13PM -0500, John Snow wrote: > > Jeff, I think this ought to go through your tree, but I'll sign off on it. > > > > Fam: Do you agree? > > > > (Probably -next material, but I'm sending this email because

Re: [Qemu-devel] [Qemu-ppc] [PATCH v2] pseries: fix TCG migration

2017-11-28 Thread Daniel Henrique Barboza
On 11/28/2017 03:43 PM, Laurent Vivier wrote: Migration of pseries is broken with TCG because QEMU tries to restore KVM MMU state unconditionally. The result is a SIGSEGV in kvm_vm_ioctl(): #0 kvm_vm_ioctl (s=0x0, type=-2146390353) at qemu/accel/kvm/kvm-all.c:2032 #1

Re: [Qemu-devel] [PATCH 3/4] cryptodev-vhost-user: add crypto session handler

2017-11-28 Thread Michael S. Tsirkin
On Tue, Nov 28, 2017 at 11:45:31AM +0100, Paolo Bonzini wrote: > On 28/11/2017 11:43, Gonglei (Arei) wrote: > >> As far as I understand, VIRTIO_CRYPTO_CIPHER_CREATE_SESSION is called as > >> a result of sending a message on the control virtqueue. > > > > VIRTIO_CRYPTO_CIPHER_CREATE_SESSION is a

Re: [Qemu-devel] [PATCH] ioapic/tracing: Remove last DPRINTFs

2017-11-28 Thread Michael S. Tsirkin
On Thu, Nov 02, 2017 at 06:03:10PM +, Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > Remove the last few DPRINTFs from hw/intc/ioapic.c and turn > them into tracing. In one case it's a new trace, in the others > it's just adding a parameter to

Re: [Qemu-devel] [PATCH] i386: turn off l3-cache property by default

2017-11-28 Thread Michael S. Tsirkin
On Fri, Nov 24, 2017 at 04:26:50PM +0300, Denis Plotnikov wrote: > Commit 14c985cffa "target-i386: present virtual L3 cache info for vcpus" > introduced and set by default exposing l3 to the guest. > > The motivation behind it was that in the Linux scheduler, when waking up > a task on a sibling

Re: [Qemu-devel] [PATCH] vhost-user spec: Clarify policy on setting log_base

2017-11-28 Thread Michael S. Tsirkin
On Tue, Nov 28, 2017 at 01:34:19PM -0500, Victor Kaplansky wrote: > > From: "Michael S. Tsirkin" > > To: "Victor Kaplansky" > > Cc: qemu-devel@nongnu.org, "Maxime Coquelin" , > > "Jason Wang" > > Sent:

Re: [Qemu-devel] [PATCH-for-2.12 v3 0/3] nvdimm: fixes for (non-)dax backends

2017-11-28 Thread Michael S. Tsirkin
On Mon, Nov 27, 2017 at 12:35:14PM +0800, Haozhong Zhang wrote: > Previous versions can be found at > v2: https://lists.gnu.org/archive/html/qemu-devel/2017-06/msg01203.html > v1: https://lists.gnu.org/archive/html/qemu-devel/2017-05/msg05919.html > > Changes in v3: > * Add an option 'align'

Re: [Qemu-devel] [PATCH for 2.11 0/2] QEMU crashes with CD device without media

2017-11-28 Thread Denis V. Lunev
On 11/28/2017 08:28 PM, Peter Maydell wrote: > On 28 November 2017 at 17:01, John Snow wrote: >> It's not your fault, it's mine for letting this go for a release. It's >> just unfortunate timing. I'm looking at (at least) the IDE portion of >> this and the underlying cause in

  1   2   3   >