[Qemu-devel] Help debugging a regression in KVM Module

2015-08-14 Thread Peter Lieven
Hi, some time a go I stumbled across a regression in the KVM Module that has been introduced somewhere between 3.17 and 3.19. I have a rather old openSUSE guest with an XFS filesystem which realiably crashes after some live migrations. I originally believed that the issue might be related to

Re: [Qemu-devel] Win32 stdio not working if SDL is enabled

2015-08-14 Thread Daniel P. Berrange
On Thu, Aug 13, 2015 at 07:48:47PM +0200, Stefan Weil wrote: Am 13.08.2015 um 14:06 schrieb Daniel P. Berrange: When debugging some patches on Windows, I discovered that nothing printed to stderr ever appears on the console. Eventually I discovered that if I build with --disable-sdl, then

[Qemu-devel] [PATCH] block/iscsi: validate block size returned from target

2015-08-14 Thread Peter Lieven
It has been reported that at least tgtd returns a block size of 0 for LUN 0. To avoid running into divide by zero later on and protect against other problematic block sizes validate the block size right at connection time. Cc: qemu-sta...@nongnu.org Reported-by: Andrey Korolyov and...@xdel.ru

[Qemu-devel] [PATCH QEMU] vmstate: Remove redefinition of VMSTATE_UINT32_ARRAY

2015-08-14 Thread Soren Brinkmann
The macro is defined twice in identical ways. Signed-off-by: Soren Brinkmann soren.brinkm...@xilinx.com --- I have the feeling I'm missing a tiny one-letter difference or some ifdef, but I believe the mentioned macro is defined twice. Sören --- include/migration/vmstate.h | 3 --- 1

Re: [Qemu-devel] [Bug 1483070] [NEW] VIRTIO Sequential Write IOPS limits not working

2015-08-14 Thread Alberto Garcia
On Fri 14 Aug 2015 01:34:50 AM CEST, Peter Lieven p...@kamp.de wrote: IOPS limit does not work for VIRTIO devices if the disk workload is a sequential write. This is probably due to I/O request merging: Your benchmark application may generate 32 x 4KB write requests, but they are merged

Re: [Qemu-devel] [PATCH RFC] pseries: define coldplugged devices as configured

2015-08-14 Thread Laurent Vivier
On 14/08/2015 07:20, Bharata B Rao wrote: On Thu, Aug 13, 2015 at 02:53:02PM +0200, Laurent Vivier wrote: When a device is hotplugged, attach() sets configured to false, waiting an action from the OS to configure it and then to call ibm,configure-connector. On ibm,configure-connector, the

Re: [Qemu-devel] [PATCH RFC] pseries: define coldplugged devices as configured

2015-08-14 Thread Laurent Vivier
On 14/08/2015 07:20, Bharata B Rao wrote: On Thu, Aug 13, 2015 at 02:53:02PM +0200, Laurent Vivier wrote: When a device is hotplugged, attach() sets configured to false, waiting an action from the OS to configure it and then to call ibm,configure-connector. On ibm,configure-connector, the

Re: [Qemu-devel] [PATCH RFC] pseries: define coldplugged devices as configured

2015-08-14 Thread Bharata B Rao
On Fri, Aug 14, 2015 at 09:16:08AM +0200, Laurent Vivier wrote: On 14/08/2015 07:20, Bharata B Rao wrote: On Thu, Aug 13, 2015 at 02:53:02PM +0200, Laurent Vivier wrote: When a device is hotplugged, attach() sets configured to false, waiting an action from the OS to configure it and

[Qemu-devel] [Bug 1484925] [NEW] Segfault with custom vnc client

2015-08-14 Thread Uli Stärk
Public bug reported: Hey, I'm using Citrix XenServer 6.5. I worte a script that uses noVNC to connect to the rfb console via xapi. When I use GRML and try to boot it, the QEMU process segfaults and kills my VM. This happens when the screen resizes and the kernel is loading: recvfrom(3,

Re: [Qemu-devel] [PATCH] mirror: Fix coroutine reentrance

2015-08-14 Thread Jeff Cody
On Thu, Aug 13, 2015 at 10:41:50AM +0200, Kevin Wolf wrote: This fixes a regression introduced by commit dcfb3beb (mirror: Do zero write on target if sectors not allocated), which was reported to cause aborts with the message Co-routine re-entered recursively. The cause for this bug is the

Re: [Qemu-devel] [PATCH] spice: Allow to set password even if disable-ticketing was used

2015-08-14 Thread Daniel P. Berrange
On Fri, Aug 14, 2015 at 03:09:44PM +0200, Christophe Fergeau wrote: Hey, On Fri, Aug 14, 2015 at 01:54:59PM +0100, Daniel P. Berrange wrote: On Fri, Aug 14, 2015 at 02:47:15PM +0200, Christophe Fergeau wrote: Before commit b1ea7b79e1, it was possible to start with -spice

Re: [Qemu-devel] [Qemu-block] RFC cdrom in own thread?

2015-08-14 Thread Peter Lieven
Am 22.06.2015 um 23:54 schrieb John Snow: On 06/22/2015 09:09 AM, Peter Lieven wrote: Am 22.06.2015 um 11:25 schrieb Stefan Hajnoczi: On Fri, Jun 19, 2015 at 2:14 PM, Peter Lieven p...@kamp.de wrote: Am 18.06.2015 um 11:36 schrieb Stefan Hajnoczi: On Thu, Jun 18, 2015 at 10:29 AM, Peter

[Qemu-devel] [PULL 1/2] block/mirror: limit qiov to IOV_MAX elements

2015-08-14 Thread Jeff Cody
From: Stefan Hajnoczi stefa...@redhat.com If mirror has more free buffers than IOV_MAX, preadv(2)/pwritev(2) EINVAL failures may be encountered. It is possible to trigger this by setting granularity to a low value like 8192. This patch stops appending chunks once IOV_MAX is reached. The

[Qemu-devel] [PULL 0/2] Block job patches

2015-08-14 Thread Jeff Cody
The following changes since commit be1f13ac9d9fc21908975460652a72f5f0c018c5: Merge remote-tracking branch 'remotes/lalrae/tags/mips-20150813' into staging (2015-08-13 17:47:44 +0100) are available in the git repository at: g...@github.com:codyprime/qemu-kvm-jtc.git tags/block-pull-request

Re: [Qemu-devel] [PULL v2 00/20] SCSI, build, TCG, RCU, misc patches for 2015-08-12

2015-08-14 Thread Peter Maydell
On 14 August 2015 at 14:03, Paolo Bonzini pbonz...@redhat.com wrote: The following changes since commit cb48f67ad8c7b33c617d4f8144a27706e69fd688: bsd-user: Fix operand to cpu_x86_exec (2015-07-30 12:38:49 +0100) are available in the git repository at: git://github.com/bonzini/qemu.git

Re: [Qemu-devel] [PATCH] mirror: Fix coroutine reentrance

2015-08-14 Thread Jeff Cody
On Thu, Aug 13, 2015 at 10:41:50AM +0200, Kevin Wolf wrote: This fixes a regression introduced by commit dcfb3beb (mirror: Do zero write on target if sectors not allocated), which was reported to cause aborts with the message Co-routine re-entered recursively. The cause for this bug is the

[Qemu-devel] [PULL 2/2] mirror: Fix coroutine reentrance

2015-08-14 Thread Jeff Cody
From: Kevin Wolf kw...@redhat.com This fixes a regression introduced by commit dcfb3beb (mirror: Do zero write on target if sectors not allocated), which was reported to cause aborts with the message Co-routine re-entered recursively. The cause for this bug is the following code in

Re: [Qemu-devel] [Qemu-block] RFC cdrom in own thread?

2015-08-14 Thread Kevin Wolf
Am 14.08.2015 um 15:43 hat Peter Lieven geschrieben: Am 22.06.2015 um 23:54 schrieb John Snow: On 06/22/2015 09:09 AM, Peter Lieven wrote: Am 22.06.2015 um 11:25 schrieb Stefan Hajnoczi: On Fri, Jun 19, 2015 at 2:14 PM, Peter Lieven p...@kamp.de wrote: Am 18.06.2015 um 11:36 schrieb

Re: [Qemu-devel] [PATCH RFC 00/10] Enable repository wide style checking

2015-08-14 Thread Paul Eggert
Peter Maydell wrote: I just don't want a GPLv3-licensed file in the git repo and an integrated part of our build-and-test system... My kneejerk reaction is that the build procedures in question are large enough that they should stay GPLv3. If you don't want those files in your git repo you

[Qemu-devel] [PATCH] e500 ATMU register reads broken

2015-08-14 Thread Rudolf Marek
Hi all, I noticed that ATMU register reads on E500 are broken. Due to the wrong mask, some registers cannot be read and instead some other registers are read. Please see attached patch which fixes the problem. I also noticed that if there was an intention to have 1:1 PCI/CPU space mapping

Re: [Qemu-devel] about the patch kvmclock Ensure proper env-tsc value for kvmclock_current_nsec calculation

2015-08-14 Thread Marcin Gibuła
W dniu 2015-08-14 o 03:23, Li, Liang Z pisze: On Thu, Aug 13, 2015 at 01:25:29AM +, Li, Liang Z wrote: Hi Paolo Marcelo, Could please point out what issue the patch 317b0a6d8ba44e try to fix? I found in live migration the cpu_synchronize_all_states will be called twice, and it will

Re: [Qemu-devel] about the patch kvmclock Ensure proper env-tsc value for kvmclock_current_nsec calculation

2015-08-14 Thread Li, Liang Z
Subject: Re: [Qemu-devel] about the patch kvmclock Ensure proper env-tsc value for kvmclock_current_nsec calculation Thanks for your reply, I have read the thread in your email, what's the mean of 'switching from old to new disk', could give a detail description? The test case was

Re: [Qemu-devel] about the patch kvmclock Ensure proper env-tsc value for kvmclock_current_nsec calculation

2015-08-14 Thread Li, Liang Z
Could please point out what issue the patch 317b0a6d8ba44e try to fix? I found in live migration the cpu_synchronize_all_states will be called twice, and it will take more than 1 ms sometimes. I try to do some optimization but lack the knowledge about the background. What the

Re: [Qemu-devel] [PATCH RFC 02/10] maint: remove double semicolons in many files

2015-08-14 Thread Daniel P. Berrange
On Thu, Aug 13, 2015 at 06:57:55PM +0100, Peter Maydell wrote: On 31 July 2015 at 17:30, Daniel P. Berrange berra...@redhat.com wrote: A number of source files have statements accidentally terminated by a double semicolon - eg 'foo = bar;;'. This is harmless but a mistake none the less.

Re: [Qemu-devel] [RFC PATCH V7 07/19] protect TBContext with tb_lock.

2015-08-14 Thread Frederic Konrad
On 12/08/2015 20:20, Alex Bennée wrote: Frederic Konrad fred.kon...@greensocs.com writes: On 10/08/2015 17:27, fred.kon...@greensocs.com wrote: From: KONRAD Frederic fred.kon...@greensocs.com This protects TBContext with tb_lock to make tb_* thread safe. We can still have issue with

Re: [Qemu-devel] Plan for using softmmu with linux-user

2015-08-14 Thread Peter Maydell
On 14 August 2015 at 04:25, gchen gchen xili_gchen_5...@hotmail.com wrote: - If the performance of linux-user + softmmu + tci is not acceptable (at present, I am not quite sure), we have to implement SW64 tcg host target instead of tci. If you care even slightly about performance, then

Re: [Qemu-devel] [PATCH QEMU] vmstate: Remove redefinition of VMSTATE_UINT32_ARRAY

2015-08-14 Thread Peter Maydell
On 14 August 2015 at 07:16, Soren Brinkmann soren.brinkm...@xilinx.com wrote: The macro is defined twice in identical ways. Signed-off-by: Soren Brinkmann soren.brinkm...@xilinx.com --- I have the feeling I'm missing a tiny one-letter difference or some ifdef, but I believe the mentioned

Re: [Qemu-devel] about the patch kvmclock Ensure proper env-tsc value for kvmclock_current_nsec calculation

2015-08-14 Thread Marcin Gibuła
Thanks for your reply, I have read the thread in your email, what's the mean of 'switching from old to new disk', could give a detail description? The test case was like that (using libvirt): 1. Get VM running (linux, using kvmclock), 2. Use blockcopy to copy disk data from one location

Re: [Qemu-devel] [PATCH 00/10] translate-all.c thread-safety

2015-08-14 Thread Frederic Konrad
On 12/08/2015 18:40, Paolo Bonzini wrote: Hi, this is my attempt at 1) extracting upstreamable parts out of Fred's MTTCG, Can you take this one as well after the replace spinlock by QemuMutex: remove unused spinlock. Thanks, Fred and 2) documenting what's going on in user-mode MTTCG 3) fix

Re: [Qemu-devel] [PULL 0/4] target-mips queue

2015-08-14 Thread Peter Maydell
On 13 August 2015 at 17:45, Leon Alrae leon.al...@imgtec.com wrote: Hi, First target-mips pull request for 2.5 consisting of patches sent during 2.4 freeze. Thanks, Leon Cc: Peter Maydell peter.mayd...@linaro.org Cc: Aurelien Jarno aurel...@aurel32.net The following changes since

Re: [Qemu-devel] about the patch kvmclock Ensure proper env-tsc value for kvmclock_current_nsec calculation

2015-08-14 Thread Marcin Gibuła
So, the problem is cause by stop_vm(RUN_STATE_PAUSED), in this case the env-tsc is not updated, which lead to the issue. Is that right? I think so. If the cpu_clean_all_dirty() is needed just for the APIC status reason, I think we can do the cpu_synchronize_all_states() in do_vm_stop and

[Qemu-devel] [Bug 1484925] Re: Segfault with custom vnc client

2015-08-14 Thread Daniel Berrange
Can you attach GDB to your qemu-dm process and attempt to capture a full stack trace when it crashes (ie thread apply all backtrace) -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1484925 Title:

Re: [Qemu-devel] [PATCH] spice: Allow to set password even if disable-ticketing was used

2015-08-14 Thread Christophe Fergeau
On Fri, Aug 14, 2015 at 03:04:48PM +0100, Daniel P. Berrange wrote: Hmm, is oVirt using this via libvirt ? If so, I guess we have to fix it, as that would be a break in current usage. Yes this is done through libvirt. Before commit qemu-2.1.0-rc2~11^2, you could use virsh update-device with

Re: [Qemu-devel] [PATCH] spice: Allow to set password even if disable-ticketing was used

2015-08-14 Thread Daniel P. Berrange
On Fri, Aug 14, 2015 at 02:47:15PM +0200, Christophe Fergeau wrote: Before commit b1ea7b79e1, it was possible to start with -spice disable-ticketing, and then use the set_password spice command to enable ticketing with SPICE. Since commit b1ea7b79e1 this is no longer possible as

[Qemu-devel] [PULL 0/2] Block patches

2015-08-14 Thread Stefan Hajnoczi
The following changes since commit 2be4f242b50a84bf360df02480b173bfed161107: Merge remote-tracking branch 'remotes/ehabkost/tags/x86-pull-request' into staging (2015-08-04 16:51:24 +0100) are available in the git repository at: git://github.com/stefanha/qemu.git tags/block-pull-request

[Qemu-devel] [PULL 1/2] throttle: refuse bps_max/iops_max without bps/iops

2015-08-14 Thread Stefan Hajnoczi
The bps_max/iops_max values are meaningless without corresponding bps/iops values. Reported an error if bps_max/iops_max is given without bps/iops. Signed-off-by: Stefan Hajnoczi stefa...@redhat.com Reviewed-by: Alberto Garcia be...@igalia.com Message-id:

[Qemu-devel] [PULL 2/2] throttle: add throttle_max_is_missing_limit() test

2015-08-14 Thread Stefan Hajnoczi
Signed-off-by: Stefan Hajnoczi stefa...@redhat.com Reviewed-by: Alberto Garcia be...@igalia.com Message-id: 1438683733-2-3-git-send-email-stefa...@redhat.com --- tests/test-throttle.c | 21 + 1 file changed, 21 insertions(+) diff --git a/tests/test-throttle.c

Re: [Qemu-devel] [Qemu-block] RFC cdrom in own thread?

2015-08-14 Thread Peter Lieven
Am 14.08.2015 um 16:08 schrieb Kevin Wolf: Am 14.08.2015 um 15:43 hat Peter Lieven geschrieben: Am 22.06.2015 um 23:54 schrieb John Snow: On 06/22/2015 09:09 AM, Peter Lieven wrote: Am 22.06.2015 um 11:25 schrieb Stefan Hajnoczi: On Fri, Jun 19, 2015 at 2:14 PM, Peter Lieven p...@kamp.de

Re: [Qemu-devel] [PULL 0/2] Block job patches

2015-08-14 Thread Peter Maydell
On 14 August 2015 at 14:57, Jeff Cody jc...@redhat.com wrote: The following changes since commit be1f13ac9d9fc21908975460652a72f5f0c018c5: Merge remote-tracking branch 'remotes/lalrae/tags/mips-20150813' into staging (2015-08-13 17:47:44 +0100) are available in the git repository at:

Re: [Qemu-devel] [PULL v2 00/20] SCSI, build, TCG, RCU, misc patches for 2015-08-12

2015-08-14 Thread Paolo Bonzini
On 14/08/2015 16:21, Paolo Bonzini wrote: On 14/08/2015 15:53, Peter Maydell wrote: Hi; I'm afraid this failed to build on my w32 config: /home/petmay01/linaro/qemu-for-merges/block/dmg.c:1: warning: -fPIC ignored for target (all code is position independent) (I have

Re: [Qemu-devel] [PULL 0/2] Block job patches

2015-08-14 Thread Jeff Cody
On Fri, Aug 14, 2015 at 03:51:03PM +0100, Peter Maydell wrote: On 14 August 2015 at 14:57, Jeff Cody jc...@redhat.com wrote: The following changes since commit be1f13ac9d9fc21908975460652a72f5f0c018c5: Merge remote-tracking branch 'remotes/lalrae/tags/mips-20150813' into staging

[Qemu-devel] [PATCH v2 07/18] nvdimm: reserve address range for NVDIMM

2015-08-14 Thread Xiao Guangrong
NVDIMM reserves all the free range above 4G to do: - Persistent Memory (PMEM) mapping - implement NVDIMM ACPI device _DSM method Signed-off-by: Xiao Guangrong guangrong.x...@linux.intel.com --- hw/i386/pc.c | 12 ++-- hw/mem/nvdimm/pc-nvdimm.c | 13 +

Re: [Qemu-devel] [PULL v2 00/20] SCSI, build, TCG, RCU, misc patches for 2015-08-12

2015-08-14 Thread Paolo Bonzini
On 14/08/2015 15:53, Peter Maydell wrote: Hi; I'm afraid this failed to build on my w32 config: /home/petmay01/linaro/qemu-for-merges/block/dmg.c:1: warning: -fPIC ignored for target (all code is position independent) (I have warnings-are-errors enabled.) This is a very weird warning,

Re: [Qemu-devel] [Qemu-block] RFC cdrom in own thread?

2015-08-14 Thread Peter Lieven
Am 14.08.2015 um 16:08 schrieb Kevin Wolf: Am 14.08.2015 um 15:43 hat Peter Lieven geschrieben: Am 22.06.2015 um 23:54 schrieb John Snow: On 06/22/2015 09:09 AM, Peter Lieven wrote: Am 22.06.2015 um 11:25 schrieb Stefan Hajnoczi: On Fri, Jun 19, 2015 at 2:14 PM, Peter Lieven p...@kamp.de

Re: [Qemu-devel] [PULL v2 00/20] SCSI, build, TCG, RCU, misc patches for 2015-08-12

2015-08-14 Thread Peter Maydell
On 14 August 2015 at 15:21, Paolo Bonzini pbonz...@redhat.com wrote: On 14/08/2015 15:53, Peter Maydell wrote: Hi; I'm afraid this failed to build on my w32 config: /home/petmay01/linaro/qemu-for-merges/block/dmg.c:1: warning: -fPIC ignored for target (all code is position independent) (I

Re: [Qemu-devel] Plan for using softmmu with linux-user

2015-08-14 Thread Richard Henderson
On 08/14/2015 02:37 AM, gchen gchen wrote: - If I implement SW64 tcg backend, I guess, I cann't get help from qemu upstream: I don't think SW64 is valuable enough for upstream (either I am not sure that I can implment Alpha tcg backend in working time). It'll need some updating to apply

[Qemu-devel] [PATCH v2 09/18] nvdimm: build ACPI NFIT table

2015-08-14 Thread Xiao Guangrong
NFIT is defined in ACPI 6.0: 5.2.25 NVDIMM Firmware Interface Table (NFIT) Currently, we only support PMEM mode. Each device has 3 tables: - SPA table, define the PMEM region info - MEM DEV table, it has the @handle which is used to associate specified ACPI NVDIMM device we will introduce in

[Qemu-devel] [PATCH v2 01/18] acpi: allow aml_operation_region() working on 64 bit offset

2015-08-14 Thread Xiao Guangrong
Currently, the offset in OperationRegion is limited to 32 bit, extend it to 64 bit so that we can switch SSDT to 64 bit in later patch Signed-off-by: Xiao Guangrong guangrong.x...@linux.intel.com --- hw/acpi/aml-build.c | 2 +- include/hw/acpi/aml-build.h | 2 +- 2 files changed, 2

[Qemu-devel] [PATCH v2 03/18] acpi: add aml_derefof

2015-08-14 Thread Xiao Guangrong
Implement DeRefOf term which is used by NVDIMM _DSM method in later patch Signed-off-by: Xiao Guangrong guangrong.x...@linux.intel.com --- hw/acpi/aml-build.c | 8 include/hw/acpi/aml-build.h | 1 + 2 files changed, 9 insertions(+) diff --git a/hw/acpi/aml-build.c

[Qemu-devel] [PATCH v2 02/18] i386/acpi-build: allow SSDT to operate on 64 bit

2015-08-14 Thread Xiao Guangrong
Only 512M is left for MMIO below 4G and that are used by PCI, BIOS etc. Other components also reserve regions from their internal usage, e.g, [0xFED0, 0xFED0 + 0x400) is reserved for HPET Switch SSDT to 64 bit to use the huge free room above 4G. In the later patches, we will dynamical

[Qemu-devel] [PATCH v2 04/18] acpi: add aml_sizeof

2015-08-14 Thread Xiao Guangrong
Implement SizeOf term which is used by NVDIMM _DSM method in later patch Signed-off-by: Xiao Guangrong guangrong.x...@linux.intel.com --- hw/acpi/aml-build.c | 8 include/hw/acpi/aml-build.h | 1 + 2 files changed, 9 insertions(+) diff --git a/hw/acpi/aml-build.c

[Qemu-devel] [PATCH v2 13/18] nvdimm: build namespace config data

2015-08-14 Thread Xiao Guangrong
If @configdata is false, Qemu will build a static and readonly namespace in memory and use it serveing for DSM GET_CONFIG_SIZE/GET_CONFIG_DATA requests Signed-off-by: Xiao Guangrong guangrong.x...@linux.intel.com --- hw/mem/Makefile.objs | 3 +- hw/mem/nvdimm/acpi.c | 10 ++

[Qemu-devel] [PATCH v2 08/18] nvdimm: init backend memory mapping and config data area

2015-08-14 Thread Xiao Guangrong
The parameter @file is used as backed memory for NVDIMM which is divided into two parts if @dataconfig is true: - first parts is (0, size - 128K], which is used as PMEM (Persistent Memory) - 128K at the end of the file, which is used as Config Data Area, it's used to store Label namespace data

[Qemu-devel] [PATCH v2 05/18] acpi: add aml_create_field

2015-08-14 Thread Xiao Guangrong
Implement CreateField term which are used by NVDIMM _DSM method in later patch Signed-off-by: Xiao Guangrong guangrong.x...@linux.intel.com --- hw/acpi/aml-build.c | 14 ++ include/hw/acpi/aml-build.h | 1 + 2 files changed, 15 insertions(+) diff --git a/hw/acpi/aml-build.c

[Qemu-devel] [PATCH v2 12/18] nvdimm: save arg3 for NVDIMM device _DSM method

2015-08-14 Thread Xiao Guangrong
Check if the function (Arg2) has additional input info (arg3) and save the info if needed We only do the save on NVDIMM device since we are not going to support any function on root device Signed-off-by: Xiao Guangrong guangrong.x...@linux.intel.com --- hw/mem/nvdimm/acpi.c | 73

[Qemu-devel] [PATCH v2 06/18] pc: implement NVDIMM device abstract

2015-08-14 Thread Xiao Guangrong
Introduce pc-nvdimm device and it has two parameters: - @file, which is the backed memory file for NVDIMM device - @configdata, specify if we need to reserve 128k at the end of @file for nvdimm device's config data. Default is false If @configdata is false, Qemu will build a static and

[Qemu-devel] [PATCH 07/11] target-m68k: Use setcond for scc

2015-08-14 Thread Richard Henderson
Signed-off-by: Richard Henderson r...@twiddle.net --- target-m68k/translate.c | 20 +++- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/target-m68k/translate.c b/target-m68k/translate.c index ce48e2a..28c3e1e 100644 --- a/target-m68k/translate.c +++

[Qemu-devel] [PATCH v2 11/18] nvdimm: build ACPI nvdimm devices

2015-08-14 Thread Xiao Guangrong
NVDIMM devices is defined in ACPI 6.0 9.20 NVDIMM Devices This is a root device under \_SB and specified NVDIMM device are under the root device. Each NVDIMM device has _ADR which return its handle used to associate MEMDEV table in NFIT We reserve handle 0 for root device. In this patch, we save

[Qemu-devel] [PULL 3/3] tests: test rx recovery from cont

2015-08-14 Thread Stefan Hajnoczi
From: Jason Wang jasow...@redhat.com Rx should be recovered after cont. Signed-off-by: Jason Wang jasow...@redhat.com Message-id: 1437117954-16342-2-git-send-email-jasow...@redhat.com Signed-off-by: Stefan Hajnoczi stefa...@redhat.com --- tests/virtio-net-test.c | 52

[Qemu-devel] [PATCH 02/11] target-m68k: Some fixes to SR and flags management

2015-08-14 Thread Richard Henderson
Signed-off-by: Richard Henderson r...@twiddle.net --- target-m68k/cpu.h | 3 ++- target-m68k/helper.c| 17 +++-- target-m68k/op_helper.c | 5 +++-- target-m68k/translate.c | 2 +- 4 files changed, 21 insertions(+), 6 deletions(-) diff --git a/target-m68k/cpu.h

[Qemu-devel] [PATCH v2] spice: Allow to set password even if disable-ticketing was used

2015-08-14 Thread Christophe Fergeau
Before commit b1ea7b79e1, it was possible to start with -spice disable-ticketing, and then use the set_password spice command to enable ticketing with SPICE. Since commit b1ea7b79e1 this is no longer possible as qemu_spice_set_ticket() will return an error unless the 'auth' type is spice. When

[Qemu-devel] [PULL 2/3] tests: introduce basic pci test for virtio-net

2015-08-14 Thread Stefan Hajnoczi
From: Jason Wang jasow...@redhat.com Signed-off-by: Jason Wang jasow...@redhat.com Reviewed-by: Fam Zheng f...@redhat.com Message-id: 1437117954-16342-1-git-send-email-jasow...@redhat.com Signed-off-by: Stefan Hajnoczi stefa...@redhat.com --- tests/Makefile | 2 +-

[Qemu-devel] [PATCH] configure: Don't permit SDL or GTK on OSX

2015-08-14 Thread Peter Maydell
The cocoa GUI frontend assumes it is the only GUI (it redefines main() so it always gets control before the rest of QEMU), so it does not play well with other UIs like SDL or GTK. (Mostly people building QEMU on OSX don't have the necessary dependencies available for configure to build those other

Re: [Qemu-devel] [PATCH v2] qemu-doc.texi: Add information on compiling source code on Mac OS X

2015-08-14 Thread Peter Maydell
On 7 August 2015 at 21:57, Programmingkid programmingk...@gmail.com wrote: Add information on compiling source code on Mac OS X to the documentation. Signed-off-by: John Arbuckle programmingk...@gmail.com --- qemu-doc.texi | 44 +--- 1 files

[Qemu-devel] [PATCH 09/11] target-m68k: Optimize gen_flush_flags

2015-08-14 Thread Richard Henderson
--- target-m68k/translate.c | 56 + 1 file changed, 52 insertions(+), 4 deletions(-) diff --git a/target-m68k/translate.c b/target-m68k/translate.c index 6f60c6f..19097c2 100644 --- a/target-m68k/translate.c +++ b/target-m68k/translate.c @@ -461,18

Re: [Qemu-devel] [PATCH 00/11] Proposed format for m68k flags

2015-08-14 Thread Laurent Vivier
Le 14/08/2015 16:59, Richard Henderson a écrit : As promised a couple of days ago, with the addition of CC_OP_CMP, which wasn't in the text of my proposal the other day. From the looks of the generated code, I believe this is ideal. The following is based on Laurent's 8/30 Update cpu

[Qemu-devel] [PATCH v2 17/18] nvdimm: support NFIT_CMD_SET_CONFIG_DATA

2015-08-14 Thread Xiao Guangrong
Function 6 is used to set Namespace Label Data Signed-off-by: Xiao Guangrong guangrong.x...@linux.intel.com --- hw/mem/nvdimm/acpi.c | 40 1 file changed, 40 insertions(+) diff --git a/hw/mem/nvdimm/acpi.c b/hw/mem/nvdimm/acpi.c index 517d710..283228d

[Qemu-devel] [PATCH v2 00/18] implement vNVDIMM

2015-08-14 Thread Xiao Guangrong
Changlog: - Use litten endian for DSM method, thanks for Stefan's suggestion - introduce a new parameter, @configdata, if it's false, Qemu will build a static and readonly namespace in memory and use it serveing for DSM GET_CONFIG_SIZE/GET_CONFIG_DATA requests. In this case, no reserved

[Qemu-devel] [PATCH v2 16/18] nvdimm: support NFIT_CMD_GET_CONFIG_DATA

2015-08-14 Thread Xiao Guangrong
Function 5 is used to get Namespace Label Data Signed-off-by: Xiao Guangrong guangrong.x...@linux.intel.com --- hw/mem/nvdimm/acpi.c | 32 1 file changed, 32 insertions(+) diff --git a/hw/mem/nvdimm/acpi.c b/hw/mem/nvdimm/acpi.c index 0a5f2c2..517d710 100644 ---

[Qemu-devel] [PATCH v2 14/18] nvdimm: support NFIT_CMD_IMPLEMENTED function

2015-08-14 Thread Xiao Guangrong
__DSM is defined in ACPI 6.0: 9.14.1 _DSM (Device Specific Method) Function 0 is a query function. We do not support any function on root device and only 3 functions are support for NVDIMM device, NFIT_CMD_GET_CONFIG_SIZE, NFIT_CMD_GET_CONFIG_DATA and NFIT_CMD_SET_CONFIG_DATA, that means we

[Qemu-devel] [PATCH 10/11] target-m68k: Inline shifts

2015-08-14 Thread Richard Henderson
Signed-off-by: Richard Henderson r...@twiddle.net --- target-m68k/helper.c| 52 --- target-m68k/helper.h| 3 -- target-m68k/translate.c | 94 + 3 files changed, 72 insertions(+), 77 deletions(-) diff --git

[Qemu-devel] [PATCH v2 18/18] nvdimm: add maintain info

2015-08-14 Thread Xiao Guangrong
Add NVDIMM maintainer Signed-off-by: Xiao Guangrong guangrong.x...@linux.intel.com --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 978b717..86786e6 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -793,6 +793,12 @@ M: Jiri Pirko

[Qemu-devel] [PATCH v2 10/18] nvdimm: init the address region used by DSM method

2015-08-14 Thread Xiao Guangrong
This memory range is used to transfer data between ACPI in guest and Qemu, it occupies two pages: - one is RAM-based used to save the input info of _DSM method and Qemu reuse it store output info - another one is MMIO-based, ACPI write data to this page to transfer the control to Qemu

[Qemu-devel] [PATCH 00/11] Proposed format for m68k flags

2015-08-14 Thread Richard Henderson
As promised a couple of days ago, with the addition of CC_OP_CMP, which wasn't in the text of my proposal the other day. From the looks of the generated code, I believe this is ideal. The following is based on Laurent's 8/30 Update cpu flags management. FWIW, there's something in the last patch

[Qemu-devel] [PATCH 03/11] target-m68k: Remove incorrect clearing of cc_x

2015-08-14 Thread Richard Henderson
The CF docs certainly doesnt suggest this is true. Signed-off-by: Richard Henderson r...@twiddle.net --- target-m68k/helper.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/target-m68k/helper.c b/target-m68k/helper.c index c7b0fbe..6feb4bf 100644 --- a/target-m68k/helper.c +++

Re: [Qemu-devel] Win32 stdio not working if SDL is enabled

2015-08-14 Thread Liviu Ionescu
On 14 Aug 2015, at 15:15, Daniel P. Berrange berra...@redhat.com wrote: On Fri, Aug 14, 2015 at 12:14:15PM +0100, Daniel P. Berrange wrote: On Thu, Aug 13, 2015 at 07:48:47PM +0200, Stefan Weil wrote: ... that's a feature of SDL 1.2: stdout and stderr are by default redirected to files

[Qemu-devel] [PATCH 11/11] target-m68k: Inline addx, subx, negx

2015-08-14 Thread Richard Henderson
Signed-off-by: Richard Henderson r...@twiddle.net --- target-m68k/helper.c| 40 - target-m68k/helper.h| 2 -- target-m68k/translate.c | 67 + 3 files changed, 56 insertions(+), 53 deletions(-) diff --git

[Qemu-devel] [PULL 1/3] net/vmxnet3: Fix incorrect debug message

2015-08-14 Thread Stefan Hajnoczi
From: Dana Rubin dana.ru...@ravellosystems.com From: Dana Rubin dana.ru...@ravellosystems.com In commit 80da311d81, net/vmxnet3: Fix RX TCP/UDP checksum on partially summed packets a debug message was introduced in vmxnet3_rx_need_csum_calculate() for an unlikely input condition. The message

[Qemu-devel] [PATCH v2 15/18] nvdimm: support NFIT_CMD_GET_CONFIG_SIZE function

2015-08-14 Thread Xiao Guangrong
Function 4 is used to get Namespace lable size Signed-off-by: Xiao Guangrong guangrong.x...@linux.intel.com --- hw/mem/nvdimm/acpi.c | 70 1 file changed, 70 insertions(+) diff --git a/hw/mem/nvdimm/acpi.c b/hw/mem/nvdimm/acpi.c index

[Qemu-devel] [PATCH 05/11] target-m68k: Reorg flags handling

2015-08-14 Thread Richard Henderson
Separate all ccr bits. Continue to batch updates via cc_op. Signed-off-by: Richard Henderson r...@twiddle.net --- target-m68k/cpu.c | 2 +- target-m68k/cpu.h | 46 +++--- target-m68k/helper.c| 400 +--- target-m68k/helper.h| 6

[Qemu-devel] [PATCH 08/11] target-m68k: Optimize some comparisons

2015-08-14 Thread Richard Henderson
Signed-off-by: Richard Henderson r...@twiddle.net --- target-m68k/translate.c | 108 +--- 1 file changed, 102 insertions(+), 6 deletions(-) diff --git a/target-m68k/translate.c b/target-m68k/translate.c index 28c3e1e..6f60c6f 100644 ---

[Qemu-devel] [PATCH 06/11] target-m68k: Introduce DisasCompare

2015-08-14 Thread Richard Henderson
Signed-off-by: Richard Henderson r...@twiddle.net --- target-m68k/translate.c | 82 - 1 file changed, 60 insertions(+), 22 deletions(-) diff --git a/target-m68k/translate.c b/target-m68k/translate.c index b0cb39d..ce48e2a 100644 ---

[Qemu-devel] [PATCH 04/11] target-m68k: Replace helper_xflag_lt with setcond

2015-08-14 Thread Richard Henderson
Signed-off-by: Richard Henderson r...@twiddle.net --- target-m68k/helper.c| 5 - target-m68k/helper.h| 1 - target-m68k/translate.c | 14 +++--- 3 files changed, 7 insertions(+), 13 deletions(-) diff --git a/target-m68k/helper.c b/target-m68k/helper.c index 6feb4bf..a032947

Re: [Qemu-devel] [PATCH] spice: Allow to set password even if disable-ticketing was used

2015-08-14 Thread Christophe Fergeau
On Fri, Aug 14, 2015 at 03:35:21PM +0100, Daniel P. Berrange wrote: On Fri, Aug 14, 2015 at 02:47:15PM +0200, Christophe Fergeau wrote: Before commit b1ea7b79e1, it was possible to start with -spice disable-ticketing, and then use the set_password spice command to enable ticketing with

[Qemu-devel] [PATCH 01/11] target-m68k: Print flags properly

2015-08-14 Thread Richard Henderson
Signed-off-by: Richard Henderson r...@twiddle.net --- target-m68k/cpu.h | 1 + target-m68k/helper.c| 2 +- target-m68k/translate.c | 4 ++-- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/target-m68k/cpu.h b/target-m68k/cpu.h index 6d1a140..9708615 100644 ---

[Qemu-devel] [PULL 0/3] Net patches

2015-08-14 Thread Stefan Hajnoczi
The following changes since commit e95edefbd0559e1d0aa09549641b5d9af1f96fac: Merge remote-tracking branch 'remotes/sstabellini/tags/xen-migration-2.4-tag' into staging (2015-08-03 17:33:35 +0100) are available in the git repository at: git://github.com/stefanha/qemu.git

Re: [Qemu-devel] [PATCH] configure: Don't permit SDL or GTK on OSX

2015-08-14 Thread Daniel P. Berrange
On Fri, Aug 14, 2015 at 04:10:52PM +0100, Peter Maydell wrote: The cocoa GUI frontend assumes it is the only GUI (it redefines main() so it always gets control before the rest of QEMU), so it does not play well with other UIs like SDL or GTK. (Mostly people building QEMU on OSX don't have the

[Qemu-devel] [PATCH v2 0/4] scripts/qemu-gdb: Split into modules

2015-08-14 Thread Peter Maydell
This patch series splits scripts/qemu-gdb into separate files for each command, to make it easier to add new commands in future without them all going into a single huge file. Patches 1 and 2 do that split, and have been on the list before (and reviewed by Stefan). Patch 3 makes the script tell

[Qemu-devel] [PATCH v3] qemu-doc.texi: Add information on compiling source code on Mac OS X

2015-08-14 Thread Programmingkid
This patch will add information to the documentation web page on how to build Qemu on Mac OS X. Signed-off-by: John Arbuckle programmingk...@gmail.com --- Add info on additional libraries that need to be installed. qemu-doc.texi | 60 ++--

[Qemu-devel] [PATCH v2 2/4] scripts/qemu-gdb: Split CoroutineCommand into its own file

2015-08-14 Thread Peter Maydell
Split the implementation of CoroutineCommand into its own file. Signed-off-by: Peter Maydell peter.mayd...@linaro.org Reviewed-by: Stefan Hajnoczi stefa...@redhat.com --- scripts/qemu-gdb.py | 78 + scripts/qemugdb/coroutine.py | 91

Re: [Qemu-devel] Help debugging a regression in KVM Module

2015-08-14 Thread Peter Lieven
Am 14.08.2015 um 15:01 schrieb Paolo Bonzini: - Original Message - From: Peter Lieven p...@kamp.de To: qemu-devel@nongnu.org, k...@vger.kernel.org Cc: Paolo Bonzini pbonz...@redhat.com Sent: Friday, August 14, 2015 1:11:34 PM Subject: Help debugging a regression in KVM Module Hi,

Re: [Qemu-devel] [PATCH v1 08/10] xen/pt: Make xen_pt_unregister_device idempotent

2015-08-14 Thread Konrad Rzeszutek Wilk
@@ -818,10 +819,13 @@ static void xen_pt_unregister_device(PCIDevice *d) { XenPCIPassthroughState *s = XEN_PT_DEVICE(d); uint8_t machine_irq = s-machine_irq; -uint8_t intx = xen_pt_pci_intx(s); +uint8_t intx; int rc; -if (machine_irq) { + /*

Re: [Qemu-devel] Help debugging a regression in KVM Module

2015-08-14 Thread Alex Bennée
Peter Lieven p...@kamp.de writes: Hi, some time a go I stumbled across a regression in the KVM Module that has been introduced somewhere between 3.17 and 3.19. I have a rather old openSUSE guest with an XFS filesystem which realiably crashes after some live migrations. I originally

Re: [Qemu-devel] [PATCH v1 0/2] MAINTAINERS: Update the MAINTAINERS file

2015-08-14 Thread Alistair Francis
On Fri, Aug 14, 2015 at 10:17 AM, Peter Maydell peter.mayd...@linaro.org wrote: On 14 August 2015 at 18:15, Alistair Francis alistair.fran...@xilinx.com wrote: On Thu, Jul 16, 2015 at 1:35 PM, Alistair Francis alistair.fran...@xilinx.com wrote: Two simple patches that update the Xilinx

[Qemu-devel] [mttcg RFC v4 5/6] softmmu_template.h: move to multithreading

2015-08-14 Thread Alvise Rigo
Exploiting the tcg_excl_access_lock, port the helper_{le,be}_st_name to work in real multithreading. Suggested-by: Jani Kokkonen jani.kokko...@huawei.com Suggested-by: Claudio Fontana claudio.font...@huawei.com Signed-off-by: Alvise Rigo a.r...@virtualopensystems.com --- softmmu_template.h | 36

Re: [Qemu-devel] [PULL 0/2] Block job patches

2015-08-14 Thread Peter Maydell
On 14 August 2015 at 15:55, Jeff Cody jc...@redhat.com wrote: On Fri, Aug 14, 2015 at 03:51:03PM +0100, Peter Maydell wrote: Your pull req tag has not only these two commits in it, but also a merge commit (Merge branch 'block-next' into HEAD). Why is that? I was trying to keep a commit id

Re: [Qemu-devel] [PATCH 0/4] target-arm: Implement missing EL3 (and EL2) registers

2015-08-14 Thread Edgar E. Iglesias
On Fri, Aug 14, 2015 at 06:48:28PM +0100, Peter Maydell wrote: On 14 August 2015 at 18:42, Edgar E. Iglesias edgar.igles...@gmail.com wrote: On Fri, Aug 14, 2015 at 11:12:55AM +0100, Peter Maydell wrote: Ping? Hi! Sorry for the delay! I'll have a look at this over the weekend. Do

[Qemu-devel] [mttcg RFC v4 6/6] target-arm: Use a runtime helper for excl accesses

2015-08-14 Thread Alvise Rigo
Instead of using TCG's load and store instructions, use a runtime helper as a hook for the slow-path. This is a proof of concept to verify that this approach is actually working. At the moment only the 32bit STREX is relying on this new code-path and it's working as expected. Signed-off-by:

[Qemu-devel] [mttcg RFC v4 0/6] Atomic slow-path for mttcg

2015-08-14 Thread Alvise Rigo
This is just an update with the relevant patches needed to port the v4 of Slow-path for atomic instruction translation to mttcg v7. The full source code is available at the following GIT repository: https://git.virtualopensystems.com/dev/qemu-mt.git branch: slowpath-for-atomic-v4-mttcg The [PATCH

Re: [Qemu-devel] [PATCH 03/11] target-m68k: Remove incorrect clearing of cc_x

2015-08-14 Thread Andreas Schwab
Richard Henderson r...@twiddle.net writes: The CF docs certainly doesnt suggest this is true. Signed-off-by: Richard Henderson r...@twiddle.net --- target-m68k/helper.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/target-m68k/helper.c b/target-m68k/helper.c index

  1   2   >