Re: [Qemu-devel] [RFC] pxb: Restrict to x86

2017-01-08 Thread Marcel Apfelbaum
On 01/06/2017 09:41 PM, Michael S. Tsirkin wrote: On Fri, Jan 06, 2017 at 08:13:11AM +0200, Marcel Apfelbaum wrote: On 01/06/2017 07:04 AM, David Gibson wrote: The PCI Expander Bridge (PXB) device is essentially a hack to allow different PCIe devices to be assigned to different NUMA nodes on x8

[Qemu-devel] [PATCH v3 3/3] block: m25p80: Improve 1GiB Micron flash definition

2017-01-08 Thread Marcin Krzeminski
n25q00 and mt25q01 devices share the same JEDEC ID. The difference between those two devices is number of dies and one bit in extended JEDEC bytes. This commit adds proper entry for both devices by introduction the number of dies and and new 25q00 entries. Signed-off-by: Marcin Krzeminski --- hw

[Qemu-devel] [PATCH v3 1/3] block: m25p80: Add Quad Page Program 4byte

2017-01-08 Thread Marcin Krzeminski
Some flash chips have additional page program opcode that takes only 4 byte address. This commit adds support for such command in Qemu. Signed-off-by: Marcin Krzeminski Reviewed-by: Edgar E. Iglesias --- hw/block/m25p80.c | 4 1 file changed, 4 insertions(+) diff --git a/hw/block/m25p80.c

[Qemu-devel] [PATCH v3 0/3] block: m25p80: Improve mt25qu01g chip model

2017-01-08 Thread Marcin Krzeminski
This series introduce some improvememnt targeting mt25qu01g and adds support for n25q00 device. v3: * Removed debug printf v2: * Rework die erase command functionality. Since the number of dies is constant per chip, new field to flash description structure has been added (instead of dynamic code f

[Qemu-devel] [PATCH v3 2/3] block: m25p80: Introduce die erase command

2017-01-08 Thread Marcin Krzeminski
Modern big flash nor devices consist from more than one die. Some of them do not support chip erase and instead have die erase command that can erase one die only. This commit adds possibility to define number of dies in the chip and adds support for die erase command. Nor flash model is not strict

[Qemu-devel] [Bug 864490] Re: Windows 2008 x64 (SBS Server) freezes randomly when using more than 1 CPU core

2017-01-08 Thread Thomas Huth
Since nobody replied here within the last years: I think you should rather report problems with XEN / Proxmox to the XEN or Proxmox bugtracker instead. ** Changed in: qemu Status: New => Invalid -- You received this bug notification because you are a member of qemu- devel-ml, which is sub

[Qemu-devel] [Bug 839790] Re: -usbdevice tablet broken on win XP client

2017-01-08 Thread Thomas Huth
Can you reproduce this problem with the latest version of QEMU from http ://qemu-project.org/Download ? ** Changed in: qemu Status: New => Incomplete -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/b

Re: [Qemu-devel] [PATCH] net: optimize checksum computation

2017-01-08 Thread Dmitry Fleytman
> On 6 Jan 2017, at 10:08 AM, Ladi Prosek wrote: > > Very simple loop optimization with a significant performance impact. > > Microbenchmark results, modern x86-64: > > buffer size | speed up > +- > 1500| 1.7x > 64 | 1.5x > 8 | 1.15x > > Microben

[Qemu-devel] [Bug 884401] Re: PCI Passthrough for Digium TCE400P Codec Card Not working

2017-01-08 Thread Thomas Huth
According to the comment to Alex, this should have been fixed in newer versions, so setting status to "Fix Released" now. ** Changed in: qemu Status: New => Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://

[Qemu-devel] [Bug 887883] Re: Coverity scan revealed defects

2017-01-08 Thread Thomas Huth
AFAIK a lot of issues reported by coverity have been addressed in the recent versions of QEMU ... is there still anything left here that needs special attention? ** Changed in: qemu Status: New => Incomplete -- You received this bug notification because you are a member of qemu- devel-ml,

[Qemu-devel] [Bug 884401] Re: PCI Passthrough for Digium TCE400P Codec Card Not working

2017-01-08 Thread Thomas Huth
Oh, I meant "comment from Alex", not "comment to Alex", sorry! -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/884401 Title: PCI Passthrough for Digium TCE400P Codec Card Not working Status in QEMU:

Re: [Qemu-devel] [PATCH] m68k: QOMify the MCF Fast Ethernet Controller device

2017-01-08 Thread Thomas Huth
On 06.01.2017 10:58, Thomas Huth wrote: > When running qemu-system-m68k with the "-net" parameter (for example > simply "-net nic -net user"), there is currently a confusing warning > message saying: > > Warning: requested NIC (anonymous, model mcf_fec) was not created > (not supported by this m

[Qemu-devel] [Bug 888016] Re: RHEL 6.1 guest fails to boot with vhost

2017-01-08 Thread Thomas Huth
Can you still reproduce this problem with the latest version of QEMU from http://qemu-project.org/Download and recent host and guest Linux kernels? ** Changed in: qemu Status: New => Incomplete -- You received this bug notification because you are a member of qemu- devel-ml, which is subs

[Qemu-devel] [Bug 1654826] [NEW] Holding key down using input-linux freezes guest

2017-01-08 Thread mutedbytes
Public bug reported: Qemu release version 2.8.0 KVM, kernel 4.9.1 When using the -object input-linux capability in qemu for passthrough of input/evdev devices, I found that when a key is held for a few seconds or more (such as ctrl key), the guest system freezes until the key is released. In some

[Qemu-devel] [Bug 1654826] Re: Holding key down using input-linux freezes guest

2017-01-08 Thread mutedbytes
Thought I might add: The qemu command line option equivalents for mouse and keyboard: -object input-linux,id=kbd,evdev=/dev/input/by-path/platform-i8042-serio-0-event-kbd,repeat=on,grab_all=on \ -object input-linux,id=ms1,evdev=/dev/input/by-id/usb-ROCCAT_ROCCAT_Kone_Pure-event-mouse -- You

[Qemu-devel] [PATCH v2] m68k: QOMify the MCF Fast Ethernet Controller device

2017-01-08 Thread Thomas Huth
When running qemu-system-m68k with the "-net" parameter (for example simply "-net nic -net user"), there is currently a confusing warning message saying: Warning: requested NIC (anonymous, model mcf_fec) was not created (not supported by this machine?) This seems to happen because the MCF NIC h

[Qemu-devel] [PATCH] Add DOS support for RTL8139

2017-01-08 Thread Gerhard Wiesinger
Signed-off-by: Gerhard Wiesinger --- hw/net/rtl8139.c | 288 ++- 1 file changed, 264 insertions(+), 24 deletions(-) diff --git a/hw/net/rtl8139.c b/hw/net/rtl8139.c index f05e59c..5241fea 100644 --- a/hw/net/rtl8139.c +++ b/hw/net/rtl8139.c

[Qemu-devel] How to make dest host abort migration safely (was Re: [PATCH 4/4] kvm: Allow migration with invtsc)

2017-01-08 Thread Eduardo Habkost
On Fri, Jan 06, 2017 at 08:31:27AM -0200, Marcelo Tosatti wrote: > On Thu, Jan 05, 2017 at 10:19:50AM -0200, Eduardo Habkost wrote: > > On Thu, Jan 05, 2017 at 08:48:32AM -0200, Marcelo Tosatti wrote: > > > On Wed, Jan 04, 2017 at 11:36:31PM -0200, Eduardo Habkost wrote: > > > > On Wed, Jan 04, 201

[Qemu-devel] [PATCH v2 0/2] Allow migration with invtsc if TSC frequency is explicitly set

2017-01-08 Thread Eduardo Habkost
This series makes QEMU accept migration if tsc-frequency is explicitly set on configuration. As management software is required to keep device configuration the same on migration source or destination, explicit tsc-frequency will ensure that either: * The destination host has a matching TSC freque

[Qemu-devel] [PATCH v2 1/2] kvm: Simplify invtsc check

2017-01-08 Thread Eduardo Habkost
Instead of searching the table we have just built, we can check the env->features field directly. Reviewed-by: Marcelo Tosatti Signed-off-by: Eduardo Habkost --- target/i386/kvm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/target/i386/kvm.c b/target/i386/kvm.c index

[Qemu-devel] [PATCH v2 2/2] kvm: Allow invtsc migration if tsc-khz is set explicitly

2017-01-08 Thread Eduardo Habkost
We can safely allow a VM to be migrated with invtsc enabled if tsc-khz is set explicitly, because: * QEMU already refuses to start if it can't set the TSC frequency to the configured value. * Management software is already required to keep device configuration (including CPU configuration) the

Re: [Qemu-devel] [PATCH v2 0/2] Allow migration with invtsc if TSC frequency is explicitly set

2017-01-08 Thread no-reply
Hi, Your 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. Type: series Message-id: 20170108171330.11129-1-ehabk...@redhat.com Subject: [Qemu-devel] [PATCH v2 0/2] Allow migration with

Re: [Qemu-devel] [PATCH v2 0/2] Allow migration with invtsc if TSC frequency is explicitly set

2017-01-08 Thread Eduardo Habkost
On Sun, Jan 08, 2017 at 09:19:49AM -0800, no-re...@patchew.org wrote: > Hi, > > Your 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. > > Type: series > Message-id: 201701081713

[Qemu-devel] [PATCH v3 2/2] kvm: Allow invtsc migration if tsc-khz is set explicitly

2017-01-08 Thread Eduardo Habkost
We can safely allow a VM to be migrated with invtsc enabled if tsc-khz is set explicitly, because: * QEMU already refuses to start if it can't set the TSC frequency to the configured value. * Management software is already required to keep device configuration (including CPU configuration) the

[Qemu-devel] [PATCH v3 1/2] kvm: Simplify invtsc check

2017-01-08 Thread Eduardo Habkost
Instead of searching the table we have just built, we can check the env->features field directly. Reviewed-by: Marcelo Tosatti Signed-off-by: Eduardo Habkost --- target/i386/kvm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/target/i386/kvm.c b/target/i386/kvm.c index

[Qemu-devel] [PATCH v3 0/2] Allow migration with invtsc if TSC frequency is explicitly set

2017-01-08 Thread Eduardo Habkost
This series makes QEMU accept migration if tsc-frequency is explicitly set on configuration. As management software is required to keep device configuration the same on migration source or destination, explicit tsc-frequency will ensure that either: * The destination host has a matching TSC freque

[Qemu-devel] [PULL 04/11] tcg-mips: Add bswap32u and bswap64

2017-01-08 Thread Richard Henderson
From: Jin Guojie Without the mips32r2 instructions to perform swapping, bswap is quite large, dominating the size of each reverse-endian qemu_ld/qemu_st operation. Create two subroutines in the prologue block. The subroutines require extra reserved registers (TCG_TMP[2, 3]). Using these within

[Qemu-devel] [PULL 00/11] tcg host mips64 support

2017-01-08 Thread Richard Henderson
This is v6 of Jin Guojie's patch set, as tested by Aurelien Jarno, James Hogan, and Yun Qiang Su. Plus one more patch from me to fix a trival Werror in mips specific code with gcc 5.3. r~ Jin Guojie (10): tcg-mips: Move bswap code to a subroutine tcg-mips: Add mips64 opcodes tcg-mips: Su

[Qemu-devel] [PULL 02/11] tcg-mips: Add mips64 opcodes

2017-01-08 Thread Richard Henderson
From: Jin Guojie Since the mips manual tables are in octal, reorg all of the opcodes into that format for clarity. Note that the 64-bit opcodes are as yet unused. Tested-by: Aurelien Jarno Tested-by: James Hogan Tested-by: YunQiang Su Signed-off-by: Richard Henderson Signed-off-by: Jin Guoj

[Qemu-devel] [PULL 03/11] tcg-mips: Support 64-bit opcodes

2017-01-08 Thread Richard Henderson
From: Jin Guojie Bulk patch adding 64-bit opcodes into tcg_out_op. Note that mips64 is as yet neither complete nor enabled. Tested-by: Aurelien Jarno Tested-by: James Hogan Tested-by: YunQiang Su Signed-off-by: Richard Henderson Signed-off-by: Jin Guojie Message-Id: <1483592275-4496-4-git-

[Qemu-devel] [PULL 01/11] tcg-mips: Move bswap code to a subroutine

2017-01-08 Thread Richard Henderson
From: Jin Guojie Without the mips32r2 instructions to perform swapping, bswap is quite large, dominating the size of each reverse-endian qemu_ld/qemu_st operation. Create a subroutine in the prologue block. The subroutine requires extra reserved registers (TCG_TMP[2, 3]). Using these within qe

[Qemu-devel] [PULL 05/11] tcg-mips: Adjust move functions for mips64

2017-01-08 Thread Richard Henderson
From: Jin Guojie tcg_out_mov: using OPC_OR as most mips assemblers do; tcg_out_movi: extended to 64-bit immediate. Tested-by: Aurelien Jarno Tested-by: James Hogan Tested-by: YunQiang Su Signed-off-by: Richard Henderson Signed-off-by: Jin Guojie Message-Id: <1483592275-4496-6-git-send-email

[Qemu-devel] [PULL 06/11] tcg-mips: Adjust load/store functions for mips64

2017-01-08 Thread Richard Henderson
From: Jin Guojie tcg_out_ldst: using a generic ALIAS_PADD to avoid ifdefs tcg_out_ld: generates LD or LW tcg_out_st: generates SD or SW Tested-by: Aurelien Jarno Tested-by: James Hogan Tested-by: YunQiang Su Signed-off-by: Richard Henderson Signed-off-by: Jin Guojie Message-Id: <1483592275-

[Qemu-devel] [PULL 07/11] tcg-mips: Adjust prologue for mips64

2017-01-08 Thread Richard Henderson
From: Jin Guojie Take stack frame parameters out from the function body. Tested-by: Aurelien Jarno Tested-by: James Hogan Tested-by: YunQiang Su Signed-off-by: Richard Henderson Signed-off-by: Jin Guojie Message-Id: <1483592275-4496-8-git-send-email-jinguo...@loongson.cn> --- tcg/mips/tcg-

[Qemu-devel] [PULL 08/11] tcg-mips: Add tcg unwind info

2017-01-08 Thread Richard Henderson
From: Jin Guojie Tested-by: Aurelien Jarno Tested-by: James Hogan Tested-by: YunQiang Su Signed-off-by: Richard Henderson Signed-off-by: Jin Guojie Message-Id: <1483592275-4496-9-git-send-email-jinguo...@loongson.cn> --- tcg/mips/tcg-target.inc.c | 44 +++

[Qemu-devel] [PULL 11/11] translate-all: Avoid -Werror=switch-bool

2017-01-08 Thread Richard Henderson
gcc 5.3.0 diagnoses translate-all.c: In function ‘alloc_code_gen_buffer’: translate-all.c:756:17: error: switch condition has boolean value switch (buf2 != MAP_FAILED) { ^ Signed-off-by: Richard Henderson --- translate-all.c | 2 +- 1 file changed, 1 insertion(+), 1 de

[Qemu-devel] [PULL 10/11] tcg-mips: Adjust qemu_ld/st for mips64

2017-01-08 Thread Richard Henderson
From: Jin Guojie Tested-by: Aurelien Jarno Tested-by: James Hogan Tested-by: YunQiang Su Signed-off-by: Richard Henderson Signed-off-by: Jin Guojie Message-Id: <1483592275-4496-11-git-send-email-jinguo...@loongson.cn> --- tcg/mips/tcg-target.inc.c | 203 +

[Qemu-devel] [PULL 09/11] tcg-mips: Adjust calling conventions for mips64

2017-01-08 Thread Richard Henderson
From: Jin Guojie Tested-by: Aurelien Jarno Tested-by: James Hogan Tested-by: YunQiang Su Signed-off-by: Richard Henderson Signed-off-by: Jin Guojie Message-Id: <1483592275-4496-10-git-send-email-jinguo...@loongson.cn> --- tcg/mips/tcg-target.h | 19 +++ tcg/mips/tcg-targ

[Qemu-devel] [PATCH 0/5] Use non-blacklisted family/model/stepping for Haswell CPU model

2017-01-08 Thread Eduardo Habkost
A recent glibc commit[1] added a blacklist to ensure it won't use TSX on hosts that are known to have a broken TSX implementation. Our existing Haswell CPU model has a blacklisted family/model/stepping combination, so it has to be updated to make sure guests will really use TSX. This is done by pa

[Qemu-devel] [PATCH 5/5] i386: Change stepping of Haswell to non-blacklisted value

2017-01-08 Thread Eduardo Habkost
glibc blacklists TSX on Haswell CPUs with model==60 and stepping < 4. To make the Haswell CPU model more useful, make those guests actually use TSX by changing CPU stepping to 4. References: * glibc commit 2702856bf45c82cf8e69f2064f5aa15c0ceb6359 https://sourceware.org/git/?p=glibc.git;a=commit

[Qemu-devel] [PATCH 2/5] i386: host_vendor_fms() helper function

2017-01-08 Thread Eduardo Habkost
Helper function for code that needs to check the host CPU vendor/family/model/stepping values. Signed-off-by: Eduardo Habkost --- target/i386/cpu.h | 1 + target/i386/cpu.c | 28 2 files changed, 21 insertions(+), 8 deletions(-) diff --git a/target/i386/cpu.h b/tar

[Qemu-devel] [PATCH 1/5] i386: Add explicit array size to x86_cpu_vendor_words2str()

2017-01-08 Thread Eduardo Habkost
Add explicit array size to x86_cpu_vendor_words2str() to let the compiler check if we are really passing an array of the right size. GCC still doesn't print a warning when the array is too small[1], but clang already does. [1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=50584 Signed-off-by: Edu

[Qemu-devel] [PATCH 3/5] i386/kvm: Blacklist TSX on known broken hosts

2017-01-08 Thread Eduardo Habkost
Some Intel CPUs are known to have a broken TSX implementation. A microcode update from Intel disabled TSX on those CPUs, but GET_SUPPORTED_CPUID might be reporting it as supported if the hosts were not updated yet. Manually fixup the GET_SUPPORTED_CPUID data to ensure we will never enable TSX when

[Qemu-devel] [PATCH 4/5] pc: Add 2.9 machine-types

2017-01-08 Thread Eduardo Habkost
Cc: "Michael S. Tsirkin" Cc: Laszlo Ersek Cc: Igor Mammedov Signed-off-by: Eduardo Habkost --- Changes v1 -> v2: * Added extra backslash to PC_COMPAT_2_8 definition * Suggested-by: Laszlo Ersek --- include/hw/i386/pc.h | 2 ++ hw/i386/pc_piix.c| 15 --- hw/i386/pc_q35.c

Re: [Qemu-devel] [PATCH 0/5] Use non-blacklisted family/model/stepping for Haswell CPU model

2017-01-08 Thread no-reply
Hi, Your series seems to have some coding style problems. See output below for more information: Message-id: 20170108194041.10908-1-ehabk...@redhat.com Subject: [Qemu-devel] [PATCH 0/5] Use non-blacklisted family/model/stepping for Haswell CPU model Type: series === TEST SCRIPT BEGIN === #!/bin

Re: [Qemu-devel] [PATCH 0/5] Use non-blacklisted family/model/stepping for Haswell CPU model

2017-01-08 Thread Eduardo Habkost
Oops, there are 3 actual mistakes I missed among the false positives, below: On Sun, Jan 08, 2017 at 11:47:21AM -0800, no-re...@patchew.org wrote: [...] > === OUTPUT BEGIN === > Checking PATCH 1/5: i386: Add explicit array size to > x86_cpu_vendor_words2str()... > ERROR: line over 90 characters >

[Qemu-devel] Exporting kvm_max_guest_tsc_khz to userspace (was Re: [PATCH 4/4] kvm: Allow migration with invtsc)

2017-01-08 Thread Eduardo Habkost
On Fri, Jan 06, 2017 at 08:31:27AM -0200, Marcelo Tosatti wrote: [...] > > > > > > > Maybe your use case just needs a explicit "invtsc-migration=on" > > > > command-line flag without a mechanism to abort migration on > > > > mismatch? I can't tell. > > > > > > Again, there is no special case. > >

[Qemu-devel] [PATCH] ui: fix format specifier in vnc_client_io_error() to avoid break in build.

2017-01-08 Thread Rami Rosen
When building qemu after setting _VNC_DEBUG to 1 (see ui/vnc.h), we get the following error and the build breaks: ... ui/vnc.c: In function ‘vnc_client_io_error’: ui/vnc.c:1262:13: error: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘ssize_t’ [-Werror=format=] V

Re: [Qemu-devel] [PATCH 00/15] Postcopy: Hugepage support

2017-01-08 Thread Fam Zheng
On Fri, 01/06 18:59, Dr. David Alan Gilbert wrote: > * no-re...@patchew.org (no-re...@patchew.org) wrote: > > Hi, > > Checking PATCH 14/15: postcopy: Check for userfault+hugepage feature... > > ERROR: trailing whitespace > > #21: FILE: migration/postcopy-ram.c:85: > > +bool have_hp = false;

Re: [Qemu-devel] [RFC] pxb: Restrict to x86

2017-01-08 Thread David Gibson
On Sun, Jan 08, 2017 at 10:17:12AM +0200, Marcel Apfelbaum wrote: > On 01/06/2017 09:41 PM, Michael S. Tsirkin wrote: > > On Fri, Jan 06, 2017 at 08:13:11AM +0200, Marcel Apfelbaum wrote: > > > On 01/06/2017 07:04 AM, David Gibson wrote: > > > > The PCI Expander Bridge (PXB) device is essentially a

Re: [Qemu-devel] [PATCH v3 0/4] ppc: add a IBM 40p machine (RS/6000, PReP)

2017-01-08 Thread David Gibson
On Sat, Jan 07, 2017 at 04:23:39PM +0100, Hervé Poussineau wrote: > Hi, > > This patchset adds the emulation of the IBM RS/6000 7020 (40p). The real > machine is > able to run AIX (up to 4.3.3), Windows NT (up to 4.0 SP1), the beta of OS/2 > PowerPC, > Solaris, Linux, NetBSD/PReP ... > > I've t

Re: [Qemu-devel] Proposal PCI/PCIe device placement on PAPR guests

2017-01-08 Thread David Gibson
On Fri, Jan 06, 2017 at 06:34:29PM +0100, Andrea Bolognani wrote: > [Added Laine to CC, fixed qemu-devel address] > > On Thu, 2017-01-05 at 16:46 +1100, David Gibson wrote: > [...] > >   * To allow for hotplugged devices, libvirt should also add a number > > of additional, empty vPHBs (the PAP

Re: [Qemu-devel] [RFC] pxb: Restrict to x86

2017-01-08 Thread Michael S. Tsirkin
On Mon, Jan 09, 2017 at 10:48:24AM +1100, David Gibson wrote: > On Sun, Jan 08, 2017 at 10:17:12AM +0200, Marcel Apfelbaum wrote: > > On 01/06/2017 09:41 PM, Michael S. Tsirkin wrote: > > > On Fri, Jan 06, 2017 at 08:13:11AM +0200, Marcel Apfelbaum wrote: > > > > On 01/06/2017 07:04 AM, David Gibso

Re: [Qemu-devel] [PATCH qemu 2/2] spapr_pci: Advertise 16M IOMMU pages when available

2017-01-08 Thread Alexey Kardashevskiy
On 03/01/17 10:41, David Gibson wrote: > On Thu, Dec 22, 2016 at 04:22:12PM +1100, Alexey Kardashevskiy wrote: >> On sPAPR, IOMMU page size varies and if QEMU is running with RAM >> backed with hugepages, we can advertise this to the guest so does >> this patch. >> >> Signed-off-by: Alexey Kardashe

Re: [Qemu-devel] [PATCH for-2.8] qom: fix qemu_opts leak when hot unplug object

2017-01-08 Thread Gonglei (Arei)
Ping... Anybody can pick it up? Thanks! Regards, -Gonglei > -Original Message- > From: Gonglei (Arei) > Sent: Wednesday, November 23, 2016 7:50 PM > To: qemu-devel@nongnu.org > Cc: afaer...@suse.de; Gonglei (Arei) > Subject: [PATCH for-2.8] qom: fix qemu_opts leak when hot unplug object

Re: [Qemu-devel] [PATCH RESEND 0/5] hotplugging support for both cryptdoev and virtio crypto device

2017-01-08 Thread Gonglei (Arei)
Hi Michael, Ping... Regards, -Gonglei > -Original Message- > From: Gonglei (Arei) > Sent: Thursday, December 22, 2016 11:13 AM > To: qemu-devel@nongnu.org > Cc: m...@redhat.com; Gonglei (Arei) > Subject: [PATCH RESEND 0/5] hotplugging support for both cryptdoev and virtio > crypto dev

Re: [Qemu-devel] [PATCH v3] virtio-crypto: zeroize the key material before free

2017-01-08 Thread Gonglei (Arei)
Hi Michael, Ping... Regards, -Gonglei > -Original Message- > From: Gonglei (Arei) > Sent: Thursday, December 22, 2016 11:01 AM > To: qemu-devel@nongnu.org > Cc: m...@redhat.com; Gonglei (Arei) > Subject: [PATCH v3] virtio-crypto: zeroize the key material before free > > Common practic

Re: [Qemu-devel] [PATCH] virtio-crypto: fix possible integer and heap overflow

2017-01-08 Thread Gonglei (Arei)
Hi Michael, Ping... Regards, -Gonglei > -Original Message- > From: Gonglei (Arei) > Sent: Tuesday, January 03, 2017 2:50 PM > To: qemu-devel@nongnu.org > Cc: m...@redhat.com; liqiang...@360.cn; Gonglei (Arei); > qemu-sta...@nongnu.org > Subject: [PATCH] virtio-crypto: fix possible int

Re: [Qemu-devel] [PATCH] virtio-crypto: use the correct length for cipher operation

2017-01-08 Thread Gonglei (Arei)
Hi Michael, Ping... Regards, -Gonglei > -Original Message- > From: Gonglei (Arei) > Sent: Thursday, December 22, 2016 11:37 AM > To: qemu-devel@nongnu.org > Cc: m...@redhat.com; Gonglei (Arei) > Subject: [PATCH] virtio-crypto: use the correct length for cipher operation > > In some m

Re: [Qemu-devel] [PATCH v3 0/2] VT-d migration support

2017-01-08 Thread Peter Xu
On Fri, Jan 06, 2017 at 01:27:38PM +, Dr. David Alan Gilbert wrote: [...] > > (P.S. I found that split irqchip cannot work well with migration. Is > > this an known issue?) > > How did it fail? The keyboard hangs after migration. Command line for reference: $qemu -M q35,kernel-irqchip=sp

Re: [Qemu-devel] [PATCH qemu] target-ppc: kvm: make use of KVM_CREATE_SPAPR_TCE_64

2017-01-08 Thread Alexey Kardashevskiy
On 03/01/17 13:26, David Gibson wrote: > On Thu, Dec 22, 2016 at 12:13:12PM +1100, Alexey Kardashevskiy wrote: >> KVM_CAP_SPAPR_TCE capability allows creating TCE tables in KVM which >> allows having in-kernel acceleration for H_PUT_TCE_xxx hypercalls. >> However it only supports 32bit DMA windows

Re: [Qemu-devel] [PATCH v1 00/14] POWER9 TCG enablements - part10

2017-01-08 Thread David Gibson
On Fri, Jan 06, 2017 at 11:44:42AM +0530, Nikunj A Dadhania wrote: > This series contains 11 new instructions for POWER9 ISA3.0 > VSX Vector Insert/Extract Word > VSX Scalar Extract Exponent/Significand > VSX Scalar Convert > VSX Scalar Add QP > Various float related improv

Re: [Qemu-devel] [PATCH qemu] target-ppc: kvm: make use of KVM_CREATE_SPAPR_TCE_64

2017-01-08 Thread David Gibson
On Mon, Jan 09, 2017 at 01:38:26PM +1100, Alexey Kardashevskiy wrote: > On 03/01/17 13:26, David Gibson wrote: > > On Thu, Dec 22, 2016 at 12:13:12PM +1100, Alexey Kardashevskiy wrote: > >> KVM_CAP_SPAPR_TCE capability allows creating TCE tables in KVM which > >> allows having in-kernel acceleratio

Re: [Qemu-devel] [kvm-unit-tests PATCH v3 1/2] run_tests: put logs into per-test file

2017-01-08 Thread Peter Xu
On Fri, Jan 06, 2017 at 02:51:58PM +0100, Andrew Jones wrote: > On Fri, Jan 06, 2017 at 11:33:00AM +0800, Peter Xu wrote: > > We were using test.log before to keep all the test logs. This patch > > creates one log file per test case under logs/ directory with name > > "TESTNAME.log". > > > > A new

Re: [Qemu-devel] [PATCH] net: optimize checksum computation

2017-01-08 Thread Jason Wang
On 2017年01月08日 17:03, Dmitry Fleytman wrote: On 6 Jan 2017, at 10:08 AM, Ladi Prosek wrote: Very simple loop optimization with a significant performance impact. Microbenchmark results, modern x86-64: buffer size | speed up +- 1500| 1.7x 64 | 1.5x 8

Re: [Qemu-devel] [RFC] pxb: Restrict to x86

2017-01-08 Thread David Gibson
On Mon, Jan 09, 2017 at 03:58:03AM +0200, Michael S. Tsirkin wrote: > On Mon, Jan 09, 2017 at 10:48:24AM +1100, David Gibson wrote: > > On Sun, Jan 08, 2017 at 10:17:12AM +0200, Marcel Apfelbaum wrote: > > > On 01/06/2017 09:41 PM, Michael S. Tsirkin wrote: > > > > On Fri, Jan 06, 2017 at 08:13:11A

Re: [Qemu-devel] [kvm-unit-tests PATCH v3 2/2] run_tests: allow run tests in parallel

2017-01-08 Thread Peter Xu
On Fri, Jan 06, 2017 at 03:40:41PM +0100, Andrew Jones wrote: > On Fri, Jan 06, 2017 at 11:33:01AM +0800, Peter Xu wrote: > > run_task.sh is getting slow. This patch is trying to make it faster by > > running the tests concurrently. > > > > We provide a new parameter "-j" for the run_tests.sh, whi

Re: [Qemu-devel] [PATCH v2] m68k: QOMify the MCF Fast Ethernet Controller device

2017-01-08 Thread Jason Wang
On 2017年01月08日 19:51, Thomas Huth wrote: When running qemu-system-m68k with the "-net" parameter (for example simply "-net nic -net user"), there is currently a confusing warning message saying: Warning: requested NIC (anonymous, model mcf_fec) was not created (not supported by this machin

[Qemu-devel] [kvm-unit-tests PATCH v4 0/2] run_tests: support concurrent test execution

2017-01-08 Thread Peter Xu
v4: - add .gitignore for logs/ [Drew] - instead of create globals.bash, renaming function.bash into common.bash, put globals inside [Drew] - instead of removing logs/ directly when start run_tests, move it into logs.old so we at least have the last time result cached [Drew] - s/ut_/unittest_/ t

[Qemu-devel] [kvm-unit-tests PATCH v4 2/2] run_tests: allow run tests in parallel

2017-01-08 Thread Peter Xu
run_task.sh is getting slow. This patch is trying to make it faster by running the tests concurrently. We provide a new parameter "-j" for the run_tests.sh, which can be used to specify how many run queues we want for the tests. Default queue length is 1, which is the old behavior. Quick test on

Re: [Qemu-devel] [PATCH] hw/pci: use-after-free in pci_nic_init_nofail when nic device fails to initialize

2017-01-08 Thread Jason Wang
On 2017年01月08日 12:00, Michael S. Tsirkin wrote: On Fri, Jan 06, 2017 at 03:48:27PM -0800, Alex Kompel wrote: object_property_set_bool(OBJECT(dev), true, "realized", &err) in pci_nic_init_nofail may release the object if device fails to initialize which leads to use-after-free in error handling

[Qemu-devel] [kvm-unit-tests PATCH v4 1/2] run_tests: put logs into per-test file

2017-01-08 Thread Peter Xu
We were using test.log before to keep all the test logs. This patch creates one log file per test case under logs/ directory with name "TESTNAME.log". Meanwhile, we will keep the last time log into logs.old/. Renaming scripts/functions.bash into scripts/common.bash to store some more global variab

Re: [Qemu-devel] [PATCH] hw/pci: use-after-free in pci_nic_init_nofail when nic device fails to initialize

2017-01-08 Thread Jason Wang
On 2017年01月07日 07:48, Alex Kompel wrote: object_property_set_bool(OBJECT(dev), true, "realized", &err) in pci_nic_init_nofail may release the object if device fails to initialize which leads to use-after-free in error handling block. qdev_init_nofail does the same thing while holding the refere

[Qemu-devel] [Bug 881637] Re: QEMU fails to build on OpenBSD/hppa

2017-01-08 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/881637 Title: QEMU fails t

[Qemu-devel] [Bug 680758] Re: balloon only resizes by 2M

2017-01-08 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/680758 Title: balloon only

[Qemu-devel] [Bug 772275] Re: qemu-kvm-0.14.0 + kernel 2.6.35 : win2008r2 virtio nic hanging

2017-01-08 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/772275 Title: qemu-kvm-0.1

[Qemu-devel] [Bug 597351] Re: Slow UDP performance with virtio device

2017-01-08 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/597351 Title: Slow UDP per

Re: [Qemu-devel] [RESEND Patch v1 00/37] Implementation of vhost-pci for inter-vm commucation

2017-01-08 Thread Wei Wang
Hi Marc-André, Do you have any comments on the responses? Thanks. On 12/20/2016 12:32 PM, Wei Wang wrote: Hi Marc-André, thanks for the comments. On 12/20/2016 12:43 AM, Marc-André Lureau wrote: Hi Wei, On Mon, Dec 19, 2016 at 7:00 AM Wei Wang > wrote: This

Re: [Qemu-devel] [PATCH] hw/pci: use-after-free in pci_nic_init_nofail when nic device fails to initialize

2017-01-08 Thread Jason Wang
On 2017年01月09日 13:26, Alex Kompel wrote: Looks like this line got wrapped: "@@ -1805,13 +1805,7 @@ PCIDevice *pci_nic_init_nofail(NICInfo *nd, PCIBus *rootbus," Sorry about that. Could you unwrap it or use the attached text file? Thanks, -Alex Ok, applied. But please use git format-patch and

Re: [Qemu-devel] [PATCH v5 3/4] Plumb the HAXM-based hardware acceleration support

2017-01-08 Thread Yu Ning
On 1/6/2017 5:38, Paolo Bonzini wrote: On 05/01/2017 15:01, Paolo Bonzini wrote: In fact there is a race anyway: if (cpu->exit_request) { ret = 1; break; } cpu->exit_request

Re: [Qemu-devel] [PATCH v1 00/14] POWER9 TCG enablements - part10

2017-01-08 Thread Nikunj A Dadhania
David Gibson writes: > [ Unknown signature status ] > On Fri, Jan 06, 2017 at 11:44:42AM +0530, Nikunj A Dadhania wrote: >> This series contains 11 new instructions for POWER9 ISA3.0 >> VSX Vector Insert/Extract Word >> VSX Scalar Extract Exponent/Significand >> VSX Scalar Convert

[Qemu-devel] Question about add AF_ALG backend for virtio-crypto

2017-01-08 Thread Longpeng (Mike)
Hi guys, I'm one of Gonglei's virtio-crypto project members, and we plan to add a AF_ALG backend for virtio-crypto(there's only builtin-backend currently). I found that Catalin, Paolo and Stefan had discussed about this in 2015 (http://www.spinics.net/lists/kvm/msg115457.html), but it seems that

Re: [Qemu-devel] [PATCH] hw/pci: use-after-free in pci_nic_init_nofail when nic device fails to initialize

2017-01-08 Thread Alex Kompel
Looks like this line got wrapped: "@@ -1805,13 +1805,7 @@ PCIDevice *pci_nic_init_nofail(NICInfo *nd, PCIBus *rootbus," Sorry about that. Could you unwrap it or use the attached text file? Thanks, -Alex On Sun, Jan 8, 2017 at 8:06 PM, Jason Wang wrote: > > > On 2017年01月07日 07:48, Alex Kompel wro

Re: [Qemu-devel] [Qemu-ppc] [PATCH v3 0/4] ppc: add a IBM 40p machine (RS/6000, PReP)

2017-01-08 Thread Thomas Huth
On 07.01.2017 16:23, Hervé Poussineau wrote: > Hi, > > This patchset adds the emulation of the IBM RS/6000 7020 (40p). The real > machine is > able to run AIX (up to 4.3.3), Windows NT (up to 4.0 SP1), the beta of OS/2 > PowerPC, > Solaris, Linux, NetBSD/PReP ... > > I've tested current emulati

Re: [Qemu-devel] [PATCH 08/17] wctablet: drop debug code from wctablet_handler

2017-01-08 Thread Gerd Hoffmann
On Fr, 2017-01-06 at 07:17 -0600, Eric Blake wrote: > On 01/06/2017 02:55 AM, Gerd Hoffmann wrote: > > Signed-off-by: Gerd Hoffmann > > --- > > backends/wctablet.c | 6 -- > > 1 file changed, 6 deletions(-) > > Why not just squash this into 1/17? My intention is to keep the submission by An

[Qemu-devel] [Bug 1654826] Re: Holding key down using input-linux freezes guest

2017-01-08 Thread Gerd Hoffmann
quick workaround: drop "repeat=on". some guests seem to have problems with that, not debugged yet why. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1654826 Title: Holding key down using input-linu

Re: [Qemu-devel] [PATCH v2 3/4] migration: disallow migrate_add_blocker during migration

2017-01-08 Thread Greg Kurz
Funny... this was v3 and now it is v2 :) On Sun, 8 Jan 2017 13:04:47 +0530 Ashijeet Acharya wrote: > On Friday, January 6, 2017, Greg Kurz wrote: > > > Hi Ashijeet, > > > > Hello Greg, > > > > I didn't think hard enough while reviewing the changes for hw/9pfs/9p.c... > > I have > > some mor