[Qemu-devel] test mail

2017-10-17 Thread li.shanlue
it's a test mail, another line.

[Qemu-devel] [PATCH] docker: cleanup temp directory after test

2017-10-17 Thread Peter Xu
There are temp directories named "docker-src.*" after doing docker tests. I don't see much point in keeping that (it only contains the qemu.tar which is exactly current tree, and the copied "run" file). Let's remove it after test finished. CC: "Alex Bennée" CC: Fam Zheng CC: "Philippe Mathieu-D

[Qemu-devel] [Resend][PATCH] qga-win: don't hang if vss hold writes timeout

2017-10-17 Thread Chen Hanxiao
From: Chen Hanxiao When VM is in a heavy IO, if the command "guest-fsfreeze-freeze" is executed, VSS may timeout when trying to hold writes. Inside guest, Event ID 12298(VSS_ERROR_HOLD_WRITES_TIMEOUT) is logged in the Event Viewer. At that time, if we call AbortBackup, qga may hang forever. Th

Re: [Qemu-devel] [RFC 2/2] KVM: add virtio-pmem driver

2017-10-17 Thread Christoph Hellwig
I think this driver is at entirely the wrong level. If you want to expose pmem to a guest with flushing assist do it as pmem, and not a block driver.

[Qemu-devel] 答复: test mail

2017-10-17 Thread li.shanlue
reply for test mail, i just recieve my test mail. --原始邮件-- 发件人:李善略640438 收件人: ; 日 期 :2017年10月17日 15:08 主 题 :[Qemu-devel] test mail it's a test mail, another line.

Re: [Qemu-devel] [RFC 0/6] enable numa configuration before machine_init() from HMP/QMP

2017-10-17 Thread Igor Mammedov
On Mon, 16 Oct 2017 17:36:36 +0100 "Daniel P. Berrange" wrote: > On Mon, Oct 16, 2017 at 06:22:50PM +0200, Igor Mammedov wrote: > > Series allows to configure NUMA mapping at runtime using QMP/HMP > > interface. For that to happen it introduces a new '-paused' CLI option > > which allows to pause

Re: [Qemu-devel] [RFC 2/2] KVM: add virtio-pmem driver

2017-10-17 Thread Pankaj Gupta
> > I think this driver is at entirely the wrong level. > > If you want to expose pmem to a guest with flushing assist do it > as pmem, and not a block driver. Are you saying do it as existing i.e ACPI pmem like interface? The reason we have created this new driver is exiting pmem driver does n

Re: [Qemu-devel] [PATCH v2] monitor: fix dangling CPU pointer

2017-10-17 Thread Igor Mammedov
On Mon, 16 Oct 2017 20:47:56 +0200 Greg Kurz wrote: ... > Note that the resolution should really return a CPU object, otherwise > we have a bug. This is achieved by relying on object_resolve_path() > and CPU() instead of calling object_resolve_path_type(path, TYPE_CPU). Did you forget to remove t

Re: [Qemu-devel] [PATCH] s390x/event-facility: variable-length event masks

2017-10-17 Thread Cornelia Huck
On Mon, 16 Oct 2017 13:11:19 -0400 "Jason J. Herne" wrote: > On 10/16/2017 09:44 AM, Cornelia Huck wrote: > > On Wed, 11 Oct 2017 09:39:53 -0400 > > "Jason J. Herne" wrote: > >> ... > > > > Out of curiousity: Do you have a guest that can verify this for mask > > lengths != 4? Given that the

Re: [Qemu-devel] [PATCH] s390x/event-facility: variable-length event masks

2017-10-17 Thread Cornelia Huck
On Wed, 11 Oct 2017 09:39:53 -0400 "Jason J. Herne" wrote: > From: Cornelia Huck > > The architecture supports masks of variable length for sclp write > event mask. We currently only support 4 byte event masks, as that > is what Linux uses. > > Let's extend this to the maximum mask length supp

Re: [Qemu-devel] [RFC 2/2] KVM: add virtio-pmem driver

2017-10-17 Thread Christoph Hellwig
On Tue, Oct 17, 2017 at 03:40:56AM -0400, Pankaj Gupta wrote: > Are you saying do it as existing i.e ACPI pmem like interface? > The reason we have created this new driver is exiting pmem driver > does not define proper semantics for guest flushing requests. At this point I'm caring about the Linu

Re: [Qemu-devel] [PATCH v2] monitor: fix dangling CPU pointer

2017-10-17 Thread Greg Kurz
On Tue, 17 Oct 2017 09:52:02 +0200 Igor Mammedov wrote: > On Mon, 16 Oct 2017 20:47:56 +0200 > Greg Kurz wrote: > > ... > > Note that the resolution should really return a CPU object, otherwise > > we have a bug. This is achieved by relying on object_resolve_path() > > and CPU() instead of call

Re: [Qemu-devel] [PATCH] docker: cleanup temp directory after test

2017-10-17 Thread Fam Zheng
On Tue, 10/17 15:12, Peter Xu wrote: > There are temp directories named "docker-src.*" after doing docker > tests. I don't see much point in keeping that (it only contains the > qemu.tar which is exactly current tree, and the copied "run" file). > Let's remove it after test finished. > > CC: "Ale

[Qemu-devel] [PATCH v3] monitor: fix dangling CPU pointer

2017-10-17 Thread Greg Kurz
If a CPU selected with the "cpu" command is hot-unplugged then "info cpus" causes QEMU to exit: (qemu) device_del cpu1 (qemu) info cpus qemu:qemu_cpu_kick_thread: No such process This happens because "cpu" stores the pointer to the selected CPU into the monitor structure. When the CPU is hot-unpl

Re: [Qemu-devel] [RFC 4/6] CLI: add -paused option

2017-10-17 Thread Igor Mammedov
On Mon, 16 Oct 2017 17:35:15 +0100 "Daniel P. Berrange" wrote: > On Mon, Oct 16, 2017 at 06:22:54PM +0200, Igor Mammedov wrote: > > This really needs to have a commit message that provides justification > for why this option is needed when we already have -S that is used > to allow configuration

[Qemu-devel] [PULL 4/4] cirrus: fix oob access in mode4and5 write functions

2017-10-17 Thread Gerd Hoffmann
Move dst calculation into the loop, so we apply the mask on each interation and will not overflow vga memory. Cc: Prasad J Pandit Reported-by: Niu Guoxiang Signed-off-by: Gerd Hoffmann Message-id: 20171011084314.21752-1-kra...@redhat.com --- hw/display/cirrus_vga.c | 6 ++ 1 file changed,

[Qemu-devel] [PULL 1/4] vga: drop line_offset variable

2017-10-17 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann --- hw/display/vga.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/hw/display/vga.c b/hw/display/vga.c index ed24ef7076..bf774e3402 100644 --- a/hw/display/vga.c +++ b/hw/display/vga.c @@ -1464,7 +1464,7 @@ static void vga_draw_graphic(VGA

[Qemu-devel] [PULL 2/4] vga: handle cirrus vbe mode wraparounds.

2017-10-17 Thread Gerd Hoffmann
Commit "3d90c62548 vga: stop passing pointers to vga_draw_line* functions" is incomplete. It doesn't handle the case that the vga rendering code tries to create a shared surface, i.e. a pixman image backed by vga video memory. That can not work in case the guest display wraps from end of video me

[Qemu-devel] [PULL 0/4] Vga 20171017 patches

2017-10-17 Thread Gerd Hoffmann
The following changes since commit c5bbcaa4b7c0f8a322bebe9ec563560178a68b55: Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into staging (2017-10-16 17:29:16 +0100) are available in the git repository at: git://git.kraxel.org/qemu tags/vga-20171017-pull-request

[Qemu-devel] [PULL 3/4] vga: add ram_addr_t cast

2017-10-17 Thread Gerd Hoffmann
Reported by Coverity. Fixes: CID 1381409 Signed-off-by: Gerd Hoffmann Message-id: 20171010141323.14049-4-kra...@redhat.com --- hw/display/vga.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/display/vga.c b/hw/display/vga.c index 1afceb6f16..1d19f6bc48 100644 --- a/hw/dis

Re: [Qemu-devel] [RFC 2/2] KVM: add virtio-pmem driver

2017-10-17 Thread Pankaj Gupta
> > Are you saying do it as existing i.e ACPI pmem like interface? > > The reason we have created this new driver is exiting pmem driver > > does not define proper semantics for guest flushing requests. > > At this point I'm caring about the Linux-internal interface, and > for that it should be i

Re: [Qemu-devel] [PATCH v1 0/2] s390x/tcg: LAP support using immediate TLB invalidation

2017-10-17 Thread Cornelia Huck
On Mon, 16 Oct 2017 22:23:56 +0200 David Hildenbrand wrote: > Details about Low-Address Protection can be found in description of > patch 1 and 2. It is basically a subpage protection of the first two > pages of every address space (for which it is enabled). > > We can achieve this by simply dir

Re: [Qemu-devel] [RFC 4/6] CLI: add -paused option

2017-10-17 Thread Paolo Bonzini
On 16/10/2017 19:17, Eduardo Habkost wrote: >> Should (or could) "-M none" be changed in a backwards-compatible way to >> allow such preconfiguration? For example >> >> qemu -M none -monitor stdio >> (qemu) machine-set-options pc,accel=kvm >> (qemu) c > Sounds like an interesting idea. It w

Re: [Qemu-devel] [PULL 0/5] m68k, qtest and config patches

2017-10-17 Thread Peter Maydell
On 16 October 2017 at 14:12, Thomas Huth wrote: > The following changes since commit f90ea7ba7c5ae7010ee0ce062207ae42530f57d6: > > Merge remote-tracking branch > 'remotes/pmaydell/tags/pull-target-arm-20171012' into staging (2017-10-12 > 17:06:50 +0100) > > are available in the git repository

Re: [Qemu-devel] [RFC 4/6] CLI: add -paused option

2017-10-17 Thread Igor Mammedov
On Mon, 16 Oct 2017 14:59:16 -0200 Eduardo Habkost wrote: > On Mon, Oct 16, 2017 at 06:22:54PM +0200, Igor Mammedov wrote: > > Signed-off-by: Igor Mammedov > > --- > > include/sysemu/sysemu.h | 1 + > > qemu-options.hx | 15 ++ > > qmp.c | 5 + > > vl

Re: [Qemu-devel] [PATCH v2 00/39] remove i386/pc dependency from non-PC world (part 1)

2017-10-17 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20171017001209.32276-1-f4...@amsat.org Subject: [Qemu-devel] [PATCH v2 00/39] remove i386/pc dependency from non-PC world (part 1) === TEST SCRIPT BEGIN === #!/bin/bash BAS

[Qemu-devel] [Bug 1723927] Re: Linux or windows guest boot failed by uefi on CPU of Intel Xeon X5675

2017-10-17 Thread Laszlo Ersek (Red Hat)
"-bios /usr/share/qemu-kvm/OVMF_CODE.fd" is *never* a correct option to boot OVMF. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1723927 Title: Linux or windows guest boot failed by uefi on CPU of

Re: [Qemu-devel] [PATCH v1 0/2] s390x/tcg: LAP support using immediate TLB invalidation

2017-10-17 Thread David Hildenbrand
On 17.10.2017 10:47, Cornelia Huck wrote: > On Mon, 16 Oct 2017 22:23:56 +0200 > David Hildenbrand wrote: > >> Details about Low-Address Protection can be found in description of >> patch 1 and 2. It is basically a subpage protection of the first two >> pages of every address space (for which it

Re: [Qemu-devel] [RFC 4/6] CLI: add -paused option

2017-10-17 Thread Igor Mammedov
On Tue, 17 Oct 2017 10:47:40 +0200 Paolo Bonzini wrote: > On 16/10/2017 19:17, Eduardo Habkost wrote: > >> Should (or could) "-M none" be changed in a backwards-compatible way to > >> allow such preconfiguration? For example > >> > >> qemu -M none -monitor stdio > >> (qemu) machine-set-optio

[Qemu-devel] [PATCH] x86/cpufeatures: Enable new AVX512 cpu features

2017-10-17 Thread Yang Zhong
Intel IceLake cpu has added new cpu features,AVX512VBMI2/GFNI/ VAES/AVX512VNNI/AVX512BITALG/VPCLMULQDQ. Those new cpu features need expose to guest VM. The bit definition: CPUID.(EAX=7,ECX=0):ECX[bit 06] AVX512VBMI2 CPUID.(EAX=7,ECX=0):ECX[bit 08] GFNI CPUID.(EAX=7,ECX=0):ECX[bit 09] VAES CPUID.(E

Re: [Qemu-devel] [PATCH v1 0/2] s390x/tcg: LAP support using immediate TLB invalidation

2017-10-17 Thread Cornelia Huck
On Tue, 17 Oct 2017 11:22:19 +0200 David Hildenbrand wrote: > On 17.10.2017 10:47, Cornelia Huck wrote: > > On Mon, 16 Oct 2017 22:23:56 +0200 > > David Hildenbrand wrote: > > > >> Details about Low-Address Protection can be found in description of > >> patch 1 and 2. It is basically a subpag

Re: [Qemu-devel] [PULL 00/11] Ui 20171016 patches

2017-10-17 Thread Peter Maydell
On 16 October 2017 at 14:16, Gerd Hoffmann wrote: > The following changes since commit f90ea7ba7c5ae7010ee0ce062207ae42530f57d6: > > Merge remote-tracking branch > 'remotes/pmaydell/tags/pull-target-arm-20171012' into staging (2017-10-12 > 17:06:50 +0100) > > are available in the git repositor

Re: [Qemu-devel] [PATCH v2 37/39] i386/pc: extract port92 from pc.c

2017-10-17 Thread Philippe Mathieu-Daudé
> Why are you doing this? I really fail to see the rationale here. Please > add proper patch descriptions! Oops sorry :/ Mostly because while moving things around I read this comment laying around since commit 4b78a802ffa: /* port 92 stuff: could be split off */ No further rational (except

Re: [Qemu-devel] [PULL 0/1] 9pfs patches for 2.11 20171016

2017-10-17 Thread Peter Maydell
On 16 October 2017 at 16:13, Greg Kurz wrote: > The following changes since commit 40a1e8ac2e10155b5df13a2508ac080b00cd7e23: > > Merge remote-tracking branch 'remotes/elmarco/tags/vu-pull-request' into > staging (2017-10-16 10:22:39 +0100) > > are available in the git repository at: > > https

[Qemu-devel] Clean Block Driver Shutdown

2017-10-17 Thread Peter Lieven
Hi, I noticed that Qemu quits at several points with an exit() if the supplied parameters in the commandline are incorrect. This at some stages happens after there have already been connections to storage backends established. These connections are not cleanly shut down in this case. For posix

[Qemu-devel] [PATCH] linux-user/main: support dfilter

2017-10-17 Thread Alex Bennée
This adds the -dfilter support to linux-user. There is a minor checkpatch complaint about formatting which I've ignored for aesthetic reasons. Signed-off-by: Alex Bennée Reviewed-by: Laurent Vivier Reviewed-by: Richard Henderson --- v2 - add r-b tags - expand comment to mention checkpatch

Re: [Qemu-devel] [PATCH v2 30/39] hw/display/vga: remove the old i386/pc dependency

2017-10-17 Thread Philippe Mathieu-Daudé
Hi Thomas, >> diff --git a/hw/display/vga_int.h b/hw/display/vga_int.h >> index ad34a1f048..fe23b81442 100644 >> --- a/hw/display/vga_int.h >> +++ b/hw/display/vga_int.h >> @@ -25,8 +25,9 @@ >> #ifndef HW_VGA_INT_H >> #define HW_VGA_INT_H >> >> -#include "hw/hw.h" >> +#include "exec/ioport.h"

Re: [Qemu-devel] [RFC 4/6] CLI: add -paused option

2017-10-17 Thread Laszlo Ersek
On 10/17/17 10:17, Igor Mammedov wrote: > On Mon, 16 Oct 2017 17:35:15 +0100 > "Daniel P. Berrange" wrote: > >> On Mon, Oct 16, 2017 at 06:22:54PM +0200, Igor Mammedov wrote: >> >> This really needs to have a commit message that provides justification >> for why this option is needed when we alre

Re: [Qemu-devel] [PATCH v2 36/39] i386/pc: move pvpanic.c from hw/misc/ to hw/i386/

2017-10-17 Thread Paolo Bonzini
On 17/10/2017 02:12, Philippe Mathieu-Daudé wrote: > Signed-off-by: Philippe Mathieu-Daudé pvpanic does not _have_ to be i386-specific, and losing the configurability with default-configs/ is a bit of a pity (there are people who change the configurations), so I think it's better to leave out thi

Re: [Qemu-devel] [PATCH v2 35/39] i386/pc: move vmport.c from hw/misc/ to hw/i386/

2017-10-17 Thread Paolo Bonzini
On 17/10/2017 02:12, Philippe Mathieu-Daudé wrote: > Signed-off-by: Philippe Mathieu-Daudé Keeping the configurability is desirable here too, but moving to hw/i386 makes a lot of sense in this case. Paolo > --- > default-configs/i386-softmmu.mak | 1 - > default-configs/x86_64-softmmu.mak |

Re: [Qemu-devel] [PATCH v9 07/12] migration: Create thread infrastructure for multifd recv side

2017-10-17 Thread Dr. David Alan Gilbert
* Juan Quintela (quint...@redhat.com) wrote: > We make the locking and the transfer of information specific, even if we > are still receiving things through the main thread. > > Signed-off-by: Juan Quintela > > -- > > We split when we create the main channel and where we start the main > migrat

Re: [Qemu-devel] [PATCH v2 28/39] hw/acpi: ACPI_PM_* defines are not restricted to i386 arch

2017-10-17 Thread Paolo Bonzini
On 17/10/2017 02:11, Philippe Mathieu-Daudé wrote: > this allows to remove the old i386/pc dependency on acpi/core. enable_tco is specific to i386/pc. You can move it to include/hw/acpi/ich9.h. Paolo > Signed-off-by: Philippe Mathieu-Daudé > --- > include/hw/acpi/acpi.h | 12 > i

Re: [Qemu-devel] [PATCH v2 30/39] hw/display/vga: remove the old i386/pc dependency

2017-10-17 Thread Thomas Huth
On 17.10.2017 12:49, Philippe Mathieu-Daudé wrote: > Hi Thomas, > >>> diff --git a/hw/display/vga_int.h b/hw/display/vga_int.h >>> index ad34a1f048..fe23b81442 100644 >>> --- a/hw/display/vga_int.h >>> +++ b/hw/display/vga_int.h >>> @@ -25,8 +25,9 @@ >>> #ifndef HW_VGA_INT_H >>> #define HW_VGA_I

Re: [Qemu-devel] [PATCH v2 2/8] s390x/css: IO instr handler ending control

2017-10-17 Thread Halil Pasic
On 10/12/2017 01:44 PM, Halil Pasic wrote: > > > On 10/12/2017 08:58 AM, Thomas Huth wrote: >> On 10.10.2017 13:41, Halil Pasic wrote: [..] >> So yes, please don't do a "typedef unsigned int IOInstEnding" either. I >> think the best match for QEMU would be a >> >> typedef enum IOInstEnding { >>

Re: [Qemu-devel] [PATCH v9 08/12] migration: Test new fd infrastructure

2017-10-17 Thread Dr. David Alan Gilbert
* Juan Quintela (quint...@redhat.com) wrote: > We just send the address through the alternate channels and test that it > is ok. > > Signed-off-by: Juan Quintela I remember questions on this patch from last time as well; this is just test isn't it, and all this gets changed in later patches. So

Re: [Qemu-devel] [RFC 4/6] CLI: add -paused option

2017-10-17 Thread Peter Krempa
On Tue, Oct 17, 2017 at 12:56:28 +0200, Laszlo Ersek wrote: > On 10/17/17 10:17, Igor Mammedov wrote: > > On Mon, 16 Oct 2017 17:35:15 +0100 > > "Daniel P. Berrange" wrote: > > > >> On Mon, Oct 16, 2017 at 06:22:54PM +0200, Igor Mammedov wrote: > >> > >> This really needs to have a commit message

Re: [Qemu-devel] [PATCH] linux-user/main: support dfilter

2017-10-17 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20171017103514.5859-1-alex.ben...@linaro.org Subject: [Qemu-devel] [PATCH] linux-user/main: support dfilter === TEST SCRIPT BEGIN === #!/bin/bash BASE=base n=1 total=$(git l

[Qemu-devel] [PULL 4/6] egl-helpers: add dmabuf import support

2017-10-17 Thread Gerd Hoffmann
Add helper function to import a dma-buf as opengl texture. Also add a helper to release the texture again. Signed-off-by: Gerd Hoffmann Message-id: 20171010135453.6704-5-kra...@redhat.com --- include/ui/egl-helpers.h | 3 +++ ui/egl-helpers.c | 46 +++

[Qemu-devel] [PULL 0/6] Opengl 20171017 patches

2017-10-17 Thread Gerd Hoffmann
The following changes since commit c5bbcaa4b7c0f8a322bebe9ec563560178a68b55: Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into staging (2017-10-16 17:29:16 +0100) are available in the git repository at: git://git.kraxel.org/qemu tags/opengl-20171017-pull-request

[Qemu-devel] [PULL 1/6] console: add support for dmabufs

2017-10-17 Thread Gerd Hoffmann
This patch adds support for dma-bufs to the qemu console interfaces. It adds a new "struct QemuDmaBuf" to represent a dmabuf with accociated metatdata (size, format). It adds three functions (and DisplayChangeListenerOps operations) to set a dma-buf as display scanout, as cursor and to release a d

[Qemu-devel] [PULL 6/6] egl-headless: add dmabuf support

2017-10-17 Thread Gerd Hoffmann
Add support for the new dmabuf interface. Signed-off-by: Gerd Hoffmann Message-id: 20171010135453.6704-7-kra...@redhat.com --- ui/egl-headless.c | 59 +-- 1 file changed, 57 insertions(+), 2 deletions(-) diff --git a/ui/egl-headless.c b/ui/egl

[Qemu-devel] [PULL 3/6] opengl: add flipping vertex shader

2017-10-17 Thread Gerd Hoffmann
Add vertex shader which flips the texture upside down while blitting it. Add argument to qemu_gl_run_texture_blit() to enable flipping. Signed-off-by: Gerd Hoffmann Message-id: 20171010135453.6704-4-kra...@redhat.com --- Makefile | 4 +++- include/ui/shader.h

[Qemu-devel] [PULL 2/6] opengl: move shader init from console-gl.c to shader.c

2017-10-17 Thread Gerd Hoffmann
With the upcoming dmabuf support in qemu there will be more users of the shaders than just console-gl.c. So rename ConsoleGLState to QemuGLShader, rename some functions too, move code from console-gl.c to shaders.c. No functional change. Signed-off-by: Gerd Hoffmann Message-id: 20171010135453.6

[Qemu-devel] [PULL 5/6] egl-helpers: add egl_texture_blit and egl_texture_blend

2017-10-17 Thread Gerd Hoffmann
egl_texture_blit() blits a texture, simliar to egl_fb_blit() but by rendering the texture to the screen instead of using a framebuffer blit. egl_texture_blend() renders a texture with alpha blending, will be used to render the cursor to the screen. Signed-off-by: Gerd Hoffmann Message-id: 201710

[Qemu-devel] [Bug 1723984] Re: ID_MMFR0 has an invalid value on aarch64 cpu (A57, A53)

2017-10-17 Thread Peter Maydell
QEMU's behaviour in this case is matching the hardware. We claim to model an r1p0 (based on the MIDR value we report), and for the r1p0 the A53 and A57 reported the ID_MMFR0 as 0x10101105 -- this is documented in the TRMs for that rev of the CPUs. r1p3 reports the 0x10201105 you describe, but this

Re: [Qemu-devel] [qemu-s390x] [PATCH v2 2/8] s390x/css: IO instr handler ending control

2017-10-17 Thread Thomas Huth
On 17.10.2017 13:10, Halil Pasic wrote: > > > On 10/12/2017 01:44 PM, Halil Pasic wrote: >> >> >> On 10/12/2017 08:58 AM, Thomas Huth wrote: >>> On 10.10.2017 13:41, Halil Pasic wrote: > [..] >>> So yes, please don't do a "typedef unsigned int IOInstEnding" either. I >>> think the best match for

Re: [Qemu-devel] [RFC QEMU PATCH v3 00/10] Implement vNVDIMM for Xen HVM guest

2017-10-17 Thread Paolo Bonzini
On 14/10/2017 00:46, Stefano Stabellini wrote: > On Fri, 13 Oct 2017, Jan Beulich wrote: > On 13.10.17 at 13:13, wrote: >>> To Jan, Andrew, Stefano and Anthony, >>> >>> what do you think about allowing QEMU to build the entire guest ACPI >>> and letting SeaBIOS to load it? ACPI builder code in

Re: [Qemu-devel] [Qemu-block] Clean Block Driver Shutdown

2017-10-17 Thread Kevin Wolf
Am 17.10.2017 um 12:33 hat Peter Lieven geschrieben: > I noticed that Qemu quits at several points with an exit() if the > supplied parameters in the commandline are incorrect. This at some > stages happens after there have already been connections to storage > backends established. Maybe we need

Re: [Qemu-devel] [Qemu-block] Clean Block Driver Shutdown

2017-10-17 Thread Peter Lieven
Am 17.10.2017 um 13:46 schrieb Kevin Wolf: Am 17.10.2017 um 12:33 hat Peter Lieven geschrieben: I noticed that Qemu quits at several points with an exit() if the supplied parameters in the commandline are incorrect. This at some stages happens after there have already been connections to storage

Re: [Qemu-devel] [PULL v1 00/11] Merge QIO 2017-10-16

2017-10-17 Thread Peter Maydell
On 16 October 2017 at 21:16, Daniel P. Berrange wrote: > The following changes since commit 79b2a13aa81724228166c794f48eb75bfb696b88: > > Merge remote-tracking branch 'remotes/ericb/tags/pull-nbd-2017-10-14' into > staging (2017-10-16 15:54:42 +0100) > > are available in the git repository at:

Re: [Qemu-devel] [qemu-s390x] [PATCH v2 2/8] s390x/css: IO instr handler ending control

2017-10-17 Thread Cornelia Huck
On Tue, 17 Oct 2017 13:28:57 +0200 Thomas Huth wrote: > On 17.10.2017 13:10, Halil Pasic wrote: > > > > > > On 10/12/2017 01:44 PM, Halil Pasic wrote: > >> > >> > >> On 10/12/2017 08:58 AM, Thomas Huth wrote: > >>> On 10.10.2017 13:41, Halil Pasic wrote: > > [..] > >>> So yes, please do

Re: [Qemu-devel] [RFC QEMU PATCH v3 00/10] Implement vNVDIMM for Xen HVM guest

2017-10-17 Thread Haozhong Zhang
On 10/17/17 13:45 +0200, Paolo Bonzini wrote: > On 14/10/2017 00:46, Stefano Stabellini wrote: > > On Fri, 13 Oct 2017, Jan Beulich wrote: > > On 13.10.17 at 13:13, wrote: > >>> To Jan, Andrew, Stefano and Anthony, > >>> > >>> what do you think about allowing QEMU to build the entire guest ACP

[Qemu-devel] [PATCH] watch_mem_write: implement 8-byte accesses

2017-10-17 Thread Paolo Bonzini
Aligned 8-byte memory writes by a 64-bit target on a 64-bit host should always turn into atomic 8-byte writes on the host, however a write write watchpoint would end up tearing the 8-byte write into two 4-byte writes in access_with_adjusted_size(). Reported-by: Andrew Baumann Signed-off-by: Paolo

Re: [Qemu-devel] [PATCH v1 0/2] s390x/tcg: LAP support using immediate TLB invalidation

2017-10-17 Thread Cornelia Huck
On Mon, 16 Oct 2017 22:23:56 +0200 David Hildenbrand wrote: > Details about Low-Address Protection can be found in description of > patch 1 and 2. It is basically a subpage protection of the first two > pages of every address space (for which it is enabled). > > We can achieve this by simply dir

Re: [Qemu-devel] [PATCH v2 36/39] i386/pc: move pvpanic.c from hw/misc/ to hw/i386/

2017-10-17 Thread Philippe Mathieu-Daudé
On 10/17/2017 08:04 AM, Paolo Bonzini wrote:> pvpanic does not _have_ to be i386-specific, and losing the > configurability with default-configs/ is a bit of a pity (there are > people who change the configurations), so I think it's better to leave > out this change. Good to know, OK! (patch dropp

Re: [Qemu-devel] [PATCH 02/13] target/arm: Check PMCNTEN for whether PMCCNTR is enabled

2017-10-17 Thread Peter Maydell
On 19 April 2017 at 18:41, Aaron Lindsay wrote: > Signed-off-by: Aaron Lindsay > --- > target/arm/helper.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/target/arm/helper.c b/target/arm/helper.c > index 8cb7a94..391 100644 > --- a/target/arm/helper.c > +++ b/target

Re: [Qemu-devel] [PATCH 02/13] target/arm: Check PMCNTEN for whether PMCCNTR is enabled

2017-10-17 Thread Peter Maydell
On 30 September 2017 at 03:08, Aaron Lindsay wrote: > Signed-off-by: Aaron Lindsay > --- > target/arm/helper.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/target/arm/helper.c b/target/arm/helper.c > index 8be78ea..40c9273 100644 > --- a/target/arm/helper.c > +++ b/ta

[Qemu-devel] [Bug 1723984] Re: ID_MMFR0 has an invalid value on aarch64 cpu (A57, A53)

2017-10-17 Thread Bug
Oh I see. I didn't check older TRM since the ARM ARM was quite strict on the value, sorry. I'll read the MIDR to have a more robust code then. Thank you. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs

Re: [Qemu-devel] [RFC 1/3] hw/intc/arm_gicv3_its: Don't abort on table save/restore

2017-10-17 Thread Auger Eric
Hi Peter, On 12/10/2017 13:54, Peter Maydell wrote: > On 27 September 2017 at 15:56, Eric Auger wrote: >> The ITS is not properly reset at the moment. It is possible the >> GITS_BASER.valid is set and the in-kernel ITS caches are not >> empty (list of devices, collections, LPIs) while data structu

[Qemu-devel] [PATCH v4 RFC 9/8] nbd: Minimal structured read for client

2017-10-17 Thread Vladimir Sementsov-Ogievskiy
Minimal implementation: for structured error only error_report error message. Signed-off-by: Vladimir Sementsov-Ogievskiy --- it's an updated [PATCH v3 13/13] nbd: Minimal structured read for client https://lists.gnu.org/archive/html/qemu-devel/2017-10/msg02760.html from [PATCH v3 00/13] nbd

Re: [Qemu-devel] [qemu-s390x] [PATCH v2 2/8] s390x/css: IO instr handler ending control

2017-10-17 Thread Halil Pasic
On 10/17/2017 02:13 PM, Cornelia Huck wrote: > On Tue, 17 Oct 2017 13:28:57 +0200 > Thomas Huth wrote: > >> On 17.10.2017 13:10, Halil Pasic wrote: >>> >>> >>> On 10/12/2017 01:44 PM, Halil Pasic wrote: On 10/12/2017 08:58 AM, Thomas Huth wrote: > On 10.10.2017 13:41, Hal

[Qemu-devel] [PATCH v2 00/43] Windbg supporting

2017-10-17 Thread Mihail Abakumov
An update of: v1: https://lists.nongnu.org/archive/html/qemu-devel/2017-09/msg07092.html We made the debugger module WinDbg (like GDB) for QEMU. This is the replacement of the remote stub in Windows kernel. Used for remote Windows kernel debugging without debugging mode. WinDbg is a m

[Qemu-devel] [PATCH v2 01/43] windbg: added empty windbgstub files

2017-10-17 Thread Mihail Abakumov
Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- Makefile.target |5 + include/exec/windbgstub-utils.h | 18 ++ include/exec/windbgstub.h | 17 + stubs/Makefile.objs |

[Qemu-devel] [PATCH v2 03/43] windbg: modified windbgkd.h

2017-10-17 Thread Mihail Abakumov
Added useful name arrays of some defines. Not used yet. Needs for the future. Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- include/exec/windbgkd.h | 75 +++ 1 file changed, 75 insertions(+) diff

[Qemu-devel] [PATCH v2 09/43] windbg: handler of fs/gs register

2017-10-17 Thread Mihail Abakumov
Added handler of fs/gs register. It tries to find and check KPCR and version address. Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- include/exec/windbgstub-utils.h |8 ++ target/i386/windbgstub.c| 49 +++

[Qemu-devel] [PATCH v2 06/43] windbg: added WindbgState

2017-10-17 Thread Mihail Abakumov
Added definition of the WindbgState struct and its initialization. Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- windbgstub.c | 28 1 file changed, 28 insertions(+) diff --git a/windbgstub.c b/windbgstub.c inde

[Qemu-devel] [PATCH v2 02/43] windbg: added windbg's KD header file

2017-10-17 Thread Mihail Abakumov
Header file from windbg's source code describing the main structures. Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- include/exec/windbgkd.h | 873 +++ include/exec/windbgstub-utils.h |1 2 fil

[Qemu-devel] [PATCH v2 19/43] windbg: implemented windbg_process_control_packet

2017-10-17 Thread Mihail Abakumov
Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- windbgstub.c | 26 +- 1 file changed, 25 insertions(+), 1 deletion(-) diff --git a/windbgstub.c b/windbgstub.c index dc9eba3e3c..b2d4881ee8 100755 --- a/windbgstub.c +++

[Qemu-devel] [PATCH v2 13/43] windbg: handler of parsing context

2017-10-17 Thread Mihail Abakumov
Implemented handler of parsing context. Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- windbgstub.c | 36 ++-- 1 file changed, 34 insertions(+), 2 deletions(-) diff --git a/windbgstub.c b/windbgstub.c index 8

[Qemu-devel] [PATCH v2 04/43] windbg: added '-windbg' option

2017-10-17 Thread Mihail Abakumov
This option starts windbg server. Signed-off-by: Mihail Abakumov Acked-by: Alistair Francis Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- qemu-options.hx |8 vl.c|8 2 files changed, 16 insertions(+) diff --git a/qemu-options.hx b/qe

[Qemu-devel] [PATCH v2 07/43] windbg: added chardev

2017-10-17 Thread Mihail Abakumov
Added chardev for listening to windbg. Target device is a parameter in the '-windbg' option. Signed-off-by: Mihail Abakumov Acked-by: Alistair Francis Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- windbgstub.c | 30 ++ 1 file changed, 30 ins

[Qemu-devel] [PATCH v2 22/43] windbg: implemented kd_api_read_virtual_memory and kd_api_write_virtual_memory

2017-10-17 Thread Mihail Abakumov
Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- include/exec/windbgstub-utils.h |2 ++ windbgstub-utils.c | 47 +++ windbgstub.c|8 +++ 3 files changed, 57 insert

[Qemu-devel] [PATCH v2 14/43] windbg: init DBGKD_ANY_WAIT_STATE_CHANGE

2017-10-17 Thread Mihail Abakumov
Added function for init DBGKD_ANY_WAIT_STATE_CHANGE. It is a header of 'state change' packets. Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- include/exec/windbgstub-utils.h |8 ++ target/i386/windbgstub.c| 49 +

[Qemu-devel] [PATCH v2 08/43] windbg: hook to wrmsr operation

2017-10-17 Thread Mihail Abakumov
Inserted hook to wrmsr operation. Windows Kernel put address on KPCR struct to fs/gs (x32/x64) register. Needs catch this moment and allow windbg server handle packets. Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- include/exec/windbgstub-u

[Qemu-devel] [PATCH v2 05/43] windbg: added helper features

2017-10-17 Thread Mihail Abakumov
Added some helper features for windbgstub. Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- include/exec/windbgstub-utils.h | 34 ++ include/exec/windbgstub.h |6 ++ 2 files changed, 40 insertions(

[Qemu-devel] [PATCH v2 23/43] windbg: kernel's structures

2017-10-17 Thread Mihail Abakumov
Defined Windows kernel's structures (CPU_CONTEXT and CPU_KSPECIAL_REGISTERS) for i386 and x64_86. Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- target/i386/windbgstub.c | 247 ++ 1 file changed, 2

[Qemu-devel] [PATCH v2 15/43] windbg: sized data buffer

2017-10-17 Thread Mihail Abakumov
Defined useful sized data buffer. It contains pointer to data and size of this data. Also, defined some macros for init and free. Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- include/exec/windbgstub-utils.h | 16 1 file c

[Qemu-devel] [PATCH v2 10/43] windbg: structures for parsing data stream

2017-10-17 Thread Mihail Abakumov
Added structures for parsing data stream from windbg to packet. Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- include/exec/windbgstub-utils.h | 11 +++ windbgstub.c| 30 ++ 2 files c

[Qemu-devel] [PATCH v2 11/43] windbg: parsing data stream

2017-10-17 Thread Mihail Abakumov
Added function of parsing data stream from windbg to packet. Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- windbgstub.c | 111 +- 1 file changed, 110 insertions(+), 1 deletion(-) diff

[Qemu-devel] [PATCH v2 29/43] windbg: implemented windbg_write_ks_regs

2017-10-17 Thread Mihail Abakumov
Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- target/i386/windbgstub.c | 111 ++ 1 file changed, 111 insertions(+) diff --git a/target/i386/windbgstub.c b/target/i386/windbgstub.c index b22584a45a.

[Qemu-devel] [PATCH v2 25/43] windbg: implemented kd_api_read_control_space and kd_api_write_control_space

2017-10-17 Thread Mihail Abakumov
Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- include/exec/windbgstub-utils.h |2 + target/i386/windbgstub.c| 89 +++ windbgstub.c|8 3 files changed, 99 insertions

[Qemu-devel] [PATCH v2 17/43] windbg: generate LoadSymbolsStateChange

2017-10-17 Thread Mihail Abakumov
Added function for generate LoadSymbolsStateChange packet. Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- include/exec/windbgstub-utils.h |1 + target/i386/windbgstub.c| 15 +++ 2 files changed, 16 insertions(+) diff

[Qemu-devel] [PATCH v2 12/43] windbg: send data and control packets

2017-10-17 Thread Mihail Abakumov
Added functions for send data and control packets to windbg. Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- windbgstub.c | 56 1 file changed, 56 insertions(+) diff --git a/windbgstub

[Qemu-devel] [PATCH v2 30/43] windbg: implemented windbg_set_sr

2017-10-17 Thread Mihail Abakumov
Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- target/i386/windbgstub.c | 17 - 1 file changed, 16 insertions(+), 1 deletion(-) diff --git a/target/i386/windbgstub.c b/target/i386/windbgstub.c index fbb42d05bd..d67ad8cf0c 100

[Qemu-devel] [PATCH v2 31/43] windbg: implemented windbg_set_dr

2017-10-17 Thread Mihail Abakumov
Defined useful macros for breakpoints. Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- target/i386/windbgstub.c | 50 +- 1 file changed, 49 insertions(+), 1 deletion(-) diff --git a/target/i386/win

[Qemu-devel] [PATCH v2 16/43] windbg: generate ExceptionStateChange

2017-10-17 Thread Mihail Abakumov
Added function for generate ExceptionStateChange packet. Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- include/exec/windbgstub-utils.h |2 ++ target/i386/windbgstub.c| 20 +++- 2 files changed, 21 insertions(+),

[Qemu-devel] [PATCH v2 18/43] windbg: windbg_vm_stop

2017-10-17 Thread Mihail Abakumov
Added function for stop vm. Also, ExceptionStateChange data is generated and sent here. Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- windbgstub.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/windbgstu

[Qemu-devel] [PATCH v2 37/43] windbg: implemented kd_api_read_io_space and kd_api_write_io_space

2017-10-17 Thread Mihail Abakumov
Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- include/exec/windbgstub-utils.h |2 + windbgstub-utils.c | 62 +++ windbgstub.c|8 + 3 files changed, 72 insertion

[Qemu-devel] [PATCH v2 36/43] windbg: implemented kd_api_continue

2017-10-17 Thread Mihail Abakumov
Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- include/exec/windbgstub-utils.h |1 + windbgstub-utils.c | 15 +++ windbgstub.c|5 + 3 files changed, 21 insertions(+) diff --git a/inclu

[Qemu-devel] [PATCH v2 21/43] windbg: implemented windbg_process_manipulate_packet

2017-10-17 Thread Mihail Abakumov
Signed-off-by: Mihail Abakumov Signed-off-by: Pavel Dovgalyuk Signed-off-by: Dmitriy Koltunov --- include/exec/windbgstub-utils.h |2 ++ windbgstub-utils.c | 10 ++ windbgstub.c| 26 +- 3 files changed, 37 insertions(+), 1

  1   2   3   4   >