[PATCH v19 8/8] tests/bios-tables-test: add test cases for ACPI HMAT

2019-11-28 Thread Tao Xu
ACPI table HMAT has been introduced, QEMU now builds HMAT tables for Heterogeneous Memory with boot option '-numa node'. Add test cases on PC and Q35 machines with 2 numa nodes. Because HMAT is generated when system enable numa, the following tables need to be added for this test:

[PATCH v19 4/8] hmat acpi: Build Memory Proximity Domain Attributes Structure(s)

2019-11-28 Thread Tao Xu
From: Liu Jingqi HMAT is defined in ACPI 6.3: 5.2.27 Heterogeneous Memory Attribute Table (HMAT). The specification references below link: http://www.uefi.org/sites/default/files/resources/ACPI_6_3_final_Jan30.pdf It describes the memory attributes, such as memory side cache attributes and

[PATCH v19 1/8] numa: Extend CLI to provide initiator information for numa nodes

2019-11-28 Thread Tao Xu
In ACPI 6.3 chapter 5.2.27 Heterogeneous Memory Attribute Table (HMAT), The initiator represents processor which access to memory. And in 5.2.27.3 Memory Proximity Domain Attributes Structure, the attached initiator is defined as where the memory controller responsible for a memory proximity

[PATCH v19 3/8] numa: Extend CLI to provide memory side cache information

2019-11-28 Thread Tao Xu
From: Liu Jingqi Add -numa hmat-cache option to provide Memory Side Cache Information. These memory attributes help to build Memory Side Cache Information Structure(s) in ACPI Heterogeneous Memory Attribute Table (HMAT). Before using hmat-cache option, enable HMAT with -machine hmat=on.

Re: [PATCH v6] block/snapshot: rename Error ** parameter to more common errp

2019-11-28 Thread Max Reitz
On 27.11.19 20:25, Vladimir Sementsov-Ogievskiy wrote: > Signed-off-by: Vladimir Sementsov-Ogievskiy > Reviewed-by: Eric Blake > --- > > v6: merge corresponding header change here, so, v6 is merge of > [RFC v5 011/126] block/snapshot: rename Error ** parameter to more common > errp > and >

Re: [PATCH v4 4/6] s390x: Move clear reset

2019-11-28 Thread David Hildenbrand
On 28.11.19 09:35, Janosch Frank wrote: > On 11/28/19 8:09 AM, Thomas Huth wrote: >> On 27/11/2019 18.50, Janosch Frank wrote: >>> Let's also move the clear reset function into the reset handler. >>> >>> Signed-off-by: Janosch Frank >>> --- >>> target/s390x/cpu-qom.h | 1 + >>>

Re: [PATCH 2/4] target/arm: Abstract the generic timer frequency

2019-11-28 Thread Cédric Le Goater
On 28/11/2019 06:45, Andrew Jeffery wrote: > Prepare for SoCs such as the ASPEED AST2600 whose firmware configures > CNTFRQ to values significantly larger than the static 62.5MHz value > currently derived from GTIMER_SCALE. As the OS potentially derives its > timer periods from the CNTFRQ value

Re: [PATCH 4/4] ast2600: Configure CNTFRQ at 1125MHz

2019-11-28 Thread Cédric Le Goater
On 28/11/2019 06:45, Andrew Jeffery wrote: > This matches the configuration set by u-boot on the AST2600. > > Signed-off-by: Andrew Jeffery Reviewed-by: Cédric Le Goater > --- > hw/arm/aspeed_ast2600.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/hw/arm/aspeed_ast2600.c

Re: [PATCH v5] s390x: Move initial reset

2019-11-28 Thread Thomas Huth
On 28/11/2019 09.37, Janosch Frank wrote: Let's move the intial reset into the reset handler and cleanup afterwards. Signed-off-by: Janosch Frank Reviewed-by: David Hildenbrand --- Fixed the kvm vcpu reset. --- target/s390x/cpu-qom.h | 2 +- target/s390x/cpu.c | 46

[PATCH v19 2/8] numa: Extend CLI to provide memory latency and bandwidth information

2019-11-28 Thread Tao Xu
From: Liu Jingqi Add -numa hmat-lb option to provide System Locality Latency and Bandwidth Information. These memory attributes help to build System Locality Latency and Bandwidth Information Structure(s) in ACPI Heterogeneous Memory Attribute Table (HMAT). Before using hmat-lb option, enable

[PATCH v19 6/8] hmat acpi: Build Memory Side Cache Information Structure(s)

2019-11-28 Thread Tao Xu
From: Liu Jingqi This structure describes memory side cache information for memory proximity domains if the memory side cache is present and the physical device forms the memory side cache. The software could use this information to effectively place the data in memory to maximize the

Re: [PATCH 1/4] target/arm: Remove redundant scaling of nexttick

2019-11-28 Thread Cédric Le Goater
On 28/11/2019 06:45, Andrew Jeffery wrote: > The corner-case codepath was adjusting nexttick such that overflow > wouldn't occur when timer_mod() scaled the value back up. Remove a use > of GTIMER_SCALE and avoid unnecessary operations by calling > timer_mod_ns() directly. > > Signed-off-by:

[PATCH v19 0/8] Build ACPI Heterogeneous Memory Attribute Table (HMAT)

2019-11-28 Thread Tao Xu
This series of patches will build Heterogeneous Memory Attribute Table (HMAT) according to the command line. The ACPI HMAT describes the memory attributes, such as memory side cache attributes and bandwidth and latency details, related to the Memory Proximity Domain. The software is expected to

[PATCH v19 7/8] tests/numa: Add case for QMP build HMAT

2019-11-28 Thread Tao Xu
Check configuring HMAT usecase Reviewed-by: Igor Mammedov Suggested-by: Igor Mammedov Signed-off-by: Tao Xu --- Changes in v19: - Add some fail cases for hmat-cache when level=0 Changes in v18: - Rewrite the lines over 80 characters Chenges in v17: - Add some fail test cases

[PATCH v5] s390x: Move initial reset

2019-11-28 Thread Janosch Frank
Let's move the intial reset into the reset handler and cleanup afterwards. Signed-off-by: Janosch Frank Reviewed-by: David Hildenbrand --- Fixed the kvm vcpu reset. --- target/s390x/cpu-qom.h | 2 +- target/s390x/cpu.c | 46 +- target/s390x/cpu.h

Re: [PATCH v4 4/6] s390x: Move clear reset

2019-11-28 Thread Janosch Frank
On 11/28/19 8:09 AM, Thomas Huth wrote: > On 27/11/2019 18.50, Janosch Frank wrote: >> Let's also move the clear reset function into the reset handler. >> >> Signed-off-by: Janosch Frank >> --- >> target/s390x/cpu-qom.h | 1 + >> target/s390x/cpu.c | 58

[Bug 1846427] Re: 4.1.0: qcow2 corruption on savevm/quit/loadvm cycle

2019-11-28 Thread Michael Weiser
All my images are still fine after some heavy use with qemu-4.1.0 and fix patches applied. Just upgraded to 4.1.1 and will report back. But it's certainly looks like this bug is fixed for good. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed

Re: [PATCH] travis.yml: Run tcg tests with tci

2019-11-28 Thread Thomas Huth
On 27/11/2019 19.38, Alex Bennée wrote: Thomas Huth writes: So far we only have compile coverage for tci. But since commit 2f160e0f9797c7522bfd0d09218d0c9340a5137c ("tci: Add implementation for INDEX_op_ld16u_i64") has been included, we can also run the x86 TCG tests with tci, so let's

[PATCH v19 5/8] hmat acpi: Build System Locality Latency and Bandwidth Information Structure(s)

2019-11-28 Thread Tao Xu
From: Liu Jingqi This structure describes the memory access latency and bandwidth information from various memory access initiator proximity domains. The latency and bandwidth numbers represented in this structure correspond to rated latency and bandwidth for the platform. The software could use

[PATCH] block/nbd: fix memory leak in nbd_open()

2019-11-28 Thread pannengyuan
From: PanNengyuan In currently implementation there will be a memory leak when nbd_client_connect() returns error status. Here is an easy way to reproduce: 1. run qemu-iotests as follow and check the result with asan: ./check -raw 143 Following is the asan output backtrack: Direct leak of

Re: [RFC v5 000/126] error: auto propagated local_err

2019-11-28 Thread Markus Armbruster
Please accept my sincere apologies for taking so long to reply. A few thoughts before I dig deeper. Vladimir Sementsov-Ogievskiy writes: > Hi all! > > At the request of Markus: full version of errp propagation. Let's look > at it. Cover as much as possible, except inserting macro invocation >

Re: [PATCH] block/nbd: fix memory leak in nbd_open()

2019-11-28 Thread Stefano Garzarella
On Thu, Nov 28, 2019 at 04:40:10PM +0800, pannengy...@huawei.com wrote: Hi, I don't know nbd code very well, the patch LGTM, but just a comment below: > From: PanNengyuan > > In currently implementation there will be a memory leak when > nbd_client_connect() returns error status. Here is an

Re: [RFC v5 000/126] error: auto propagated local_err

2019-11-28 Thread Vladimir Sementsov-Ogievskiy
28.11.2019 11:54, Markus Armbruster wrote: > Please accept my sincere apologies for taking so long to reply. A few > thoughts before I dig deeper. > > Vladimir Sementsov-Ogievskiy writes: > >> Hi all! >> >> At the request of Markus: full version of errp propagation. Let's look >> at it. Cover

RE: [PATCH] Updating the GEM MAC IP to properly filter out the multicast addresses

2019-11-28 Thread Wasim, Bilal
Thanks for the pointers.. I will incorporate all these changes and post an updated thread asap.. -Original Message- From: Edgar E. Iglesias [mailto:edgar.igles...@gmail.com] Sent: Thursday, November 28, 2019 10:32 PM To: Wasim, Bilal Cc: qemu-devel@nongnu.org; alist...@alistair23.me;

Re: qom device lifecycle interaction with hotplug/hotunplug ?

2019-11-28 Thread Peter Maydell
On Thu, 28 Nov 2019 at 17:27, Igor Mammedov wrote: > > On Thu, 28 Nov 2019 16:00:06 + > Peter Maydell wrote: > > Once a device is hot-unplugged (and thus unrealized) is it valid > > for it to be re-hot-plugged, or is the assumption that it's then > > destroyed and a fresh device is created

Re: [PATCH v37 00/17] QEMU AVR 8 bit cores

2019-11-28 Thread Aleksandar Markovic
On Thursday, November 28, 2019, Philippe Mathieu-Daudé wrote: > On 11/28/19 2:46 PM, Michael Rolnik wrote: > >> I will rename them. >> > > Please wait comments from Richard before a version respin. > > Everything went well last 10 or so days, Michael and Sarah were responsive, the code and

Re: [PATCH 0/2] RFC: add -mem-shared option

2019-11-28 Thread Dr. David Alan Gilbert
* Marc-André Lureau (marcandre.lur...@redhat.com) wrote: > Hi, > > Setting up shared memory for vhost-user is a bit complicated from > command line, as it requires NUMA setup such as: m 4G -object > memory-backend-file,id=mem,size=4G,mem-path=/dev/shm,share=on -numa > node,memdev=mem. > >

libcap vs libcap-ng mess

2019-11-28 Thread Dr. David Alan Gilbert
Hi, We seem to have a bit of a mess with libcap and libcap-ng; and I'm not sure if we should try and untangle it. a) Our configure script has tests for both libcap and libcap-ng for libcap it says $cap, for libcap-ng it says $cap_ng (ok) If $cap is set - nothing happens? If $cap_ng is set

Re: [PATCH v2 2/2] travis.yml: Run tcg tests with tci

2019-11-28 Thread Stefan Weil
Am 28.11.19 um 16:35 schrieb Thomas Huth: > So far we only have compile coverage for tci. But since commit > 2f160e0f9797c7522bfd0d09218d0c9340a5137c ("tci: Add implementation > for INDEX_op_ld16u_i64") has been included now, we can also run the > "tcg" and "qtest" tests with tci, so let's enable

[PATCH v1 2/5] linux-user: convert target_mmap debug to tracepoint

2019-11-28 Thread Alex Bennée
It is a pain to re-compile when you need to debug and tracepoints are a fairly low impact way to instrument QEMU. Signed-off-by: Alex Bennée --- linux-user/mmap.c | 51 +++-- linux-user/trace-events | 1 + 2 files changed, 30 insertions(+), 22

Re: [PATCH 08/15] s390x: protvirt: KVM intercept changes

2019-11-28 Thread Cornelia Huck
On Thu, 28 Nov 2019 17:38:19 +0100 Janosch Frank wrote: > On 11/21/19 4:11 PM, Thomas Huth wrote: > > On 20/11/2019 12.43, Janosch Frank wrote: > >> Secure guests no longer intercept with code 4 for an instruction > >> interception. Instead they have codes 104 and 108 for secure > >>

Re: [PATCH v37 00/17] QEMU AVR 8 bit cores

2019-11-28 Thread Aleksandar Markovic
On Thursday, November 28, 2019, Alex Bennée wrote: > > Aleksandar Markovic writes: > > > On Thursday, November 28, 2019, Michael Rolnik > wrote: > > > >> I don't see why you say that the peripherals are inside the chip, there > is > >> CPU within target/avr directory and then there are some

RE: [PATCH] Updating the GEM MAC IP to properly filter out the multicast addresses

2019-11-28 Thread Wasim, Bilal
This was one of my first attempts, and so I was sure to miss something.. I've incorporated all the updates in this patch.. Let me know what you think about this.. net/cadence_gem: Updating the GEM MAC IP to properly filter out the multicast addresses. The current code makes a bad assumption

[PATCH v1 3/5] linux-user: add target_mmap_complete tracepoint

2019-11-28 Thread Alex Bennée
For full details we also want to see where the mmaps end up. Signed-off-by: Alex Bennée --- linux-user/mmap.c | 2 +- linux-user/trace-events | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/linux-user/mmap.c b/linux-user/mmap.c index c81fd85fbd2..a2c7037f1b6 100644

Re: [PATCH v2 0/2] Run tcg tests with tci on Travis

2019-11-28 Thread Philippe Mathieu-Daudé
On 11/28/19 4:35 PM, Thomas Huth wrote: It's now possible to run some TCG-based tests with our Tiny Code Generator Interpreter (TCI), too. These two patches enable the testing on Travis. Alex Bennée (1): configure: allow disable of cross compilation containers Thomas Huth (1):

Re: [PULL 0/5] i386 patches for QEMU 4.2-rc

2019-11-28 Thread Jens Freimann
On Wed, Nov 27, 2019 at 09:14:01AM +, Dr. David Alan Gilbert wrote: * Philippe Mathieu-Daudé (phi...@redhat.com) wrote: On 11/26/19 10:19 AM, no-re...@patchew.org wrote: > Patchew URL: https://patchew.org/QEMU/20191126085936.1689-1-pbonz...@redhat.com/ > > This series failed the

Re: [PATCH] hw: add compat machines for 5.0

2019-11-28 Thread Eduardo Habkost
On Thu, Nov 28, 2019 at 06:37:06PM +0100, Cornelia Huck wrote: > On Tue, 12 Nov 2019 11:48:11 +0100 > Cornelia Huck wrote: > > > Add 5.0 machine types for arm/i440fx/q35/s390x/spapr. > > > > For i440fx and q35, unversioned cpu models are still translated > > to -v1; I'll leave changing this (if

[for-5.0 0/4] Fixes for RMA size calculation

2019-11-28 Thread David Gibson
PAPR guests have a certain "Real Mode Area" - a subsection of memory which can be accessed when in guest real mode (that is, with the MMU "off" from the guest point of view). This is advertised to the guest in the device tree. We want to make the RMA as large as we can, to allow for flexibility

[for-5.0 2/4] spapr: Don't attempt to clamp RMA to VRMA constraint

2019-11-28 Thread David Gibson
The Real Mode Area (RMA) is the part of memory which a guest can access when in real (MMU off) mode. Of course, for a guest under KVM, the MMU isn't really turned off, it's just in a special translation mode - Virtual Real Mode Area (VRMA) - which looks like real mode in guest mode. The

[for-5.0 4/4] spapr: Correct clamping of RMA to Node 0 size

2019-11-28 Thread David Gibson
The Real Mode Area (RMA) needs to fit within Node 0 in NUMA configurations. We use a helper function spapr_node0_size() to calculate this. But that function doesn't actually get the size of Node 0, it gets the minimum size of all nodes, ever since b082d65a300 "spapr: Add a helper for node0_size

Re: [PATCH v19 2/8] numa: Extend CLI to provide memory latency and bandwidth information

2019-11-28 Thread Tao Xu
On 11/28/2019 7:50 PM, Markus Armbruster wrote: Tao Xu writes: From: Liu Jingqi Add -numa hmat-lb option to provide System Locality Latency and Bandwidth Information. These memory attributes help to build System Locality Latency and Bandwidth Information Structure(s) in ACPI Heterogeneous

Re: [PATCH v19 7/8] tests/numa: Add case for QMP build HMAT

2019-11-28 Thread Tao Xu
On 11/28/2019 7:53 PM, Thomas Huth wrote: On 28/11/2019 12.49, Markus Armbruster wrote: Tao Xu writes: Check configuring HMAT usecase Reviewed-by: Igor Mammedov Suggested-by: Igor Mammedov Signed-off-by: Tao Xu --- Changes in v19: - Add some fail cases for hmat-cache when level=0

[PATCH v2 2/4] ich9: fix getter type for sci_int property

2019-11-28 Thread Felipe Franciosi
When QOM APIs were added to ich9 in 6f1426ab, the getter for sci_int was written using uint32_t. However, the object property is uint8_t. This fixes the getter for correctness. Signed-off-by: Felipe Franciosi --- hw/isa/lpc_ich9.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-)

Re: [PATCH v2 1/2] configure: allow disable of cross compilation containers

2019-11-28 Thread Stefan Weil
Am 28.11.19 um 16:35 schrieb Thomas Huth: > From: Alex Bennée > > Our docker infrastructure isn't quite as multiarch as we would wish so > lets allow the user to disable it if they want. This will allow us to s/lets/let's/ ? Otherwise fine, thank you. Reviewed-by: Stefan Weil

[PATCH v1 5/5] linux-user: convert target_munmap debug to a tracepoint

2019-11-28 Thread Alex Bennée
Convert the final bit of DEBUG_MMAP to a tracepoint and remove the last remanents of the #ifdef hackery. Signed-off-by: Alex Bennée --- linux-user/mmap.c | 9 ++--- linux-user/trace-events | 1 + 2 files changed, 3 insertions(+), 7 deletions(-) diff --git a/linux-user/mmap.c

Re: [PATCH v37 17/17] target/avr: Update MAINTAINERS file

2019-11-28 Thread Philippe Mathieu-Daudé
On 11/27/19 6:52 PM, Michael Rolnik wrote: Include AVR maintaners in MAINTAINERS file Signed-off-by: Michael Rolnik --- MAINTAINERS | 11 +++ 1 file changed, 11 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 5e5e3e52d6..d7bfb62791 100644 --- a/MAINTAINERS +++

Re: [PATCH V2] block/nbd: fix memory leak in nbd_open()

2019-11-28 Thread pannengyuan
On 2019/11/28 21:36, Stefano Garzarella wrote: > On Thu, Nov 28, 2019 at 08:09:31PM +0800, pannengy...@huawei.com wrote: >> From: PanNengyuan >> >> In currently implementation there will be a memory leak when >> nbd_client_connect() returns error status. Here is an easy way to >> reproduce: >> >>

Re: [PATCH v19 0/8] Build ACPI Heterogeneous Memory Attribute Table (HMAT)

2019-11-28 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20191128082109.30081-1-tao3...@intel.com/ Hi, This series failed the docker-quick@centos7 build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRIPT BEGIN ===

Re: Network connection with COLO VM

2019-11-28 Thread Daniel Cho
Hi David, Zhang, Thanks for replying my question. We know why will occur this issue. As you said, the COLO VM's network needs colo-proxy to control packets, so the guest's interface should set the filter to solve the problem. But we found another question, when we set the fault-tolerance

Re: [PATCH] vfio-ccw: Fix error message

2019-11-28 Thread Cornelia Huck
On Thu, 28 Nov 2019 15:30:14 +0100 Boris Fiuczynski wrote: > Signed-off-by: Boris Fiuczynski > Reviewed-by: Eric Farman > --- > hw/vfio/ccw.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/vfio/ccw.c b/hw/vfio/ccw.c > index 6863f6c69f..3b5520ae75 100644 > ---

Re: [PATCH v4 6/6] s390x: kvm: Make kvm_sclp_service_call void

2019-11-28 Thread Cornelia Huck
On Wed, 27 Nov 2019 19:38:06 +0100 Janosch Frank wrote: > On 11/27/19 7:25 PM, Janosch Frank wrote: > > > > There's 0 (initiated), busy and operational and as far as I know we > > implement neither. > > That came out wrong... > s/operational/not operational/ > > We only implement "command

Re: [PATCH v2] virtio-pci: disable vring processing when bus-mastering is disabled

2019-11-28 Thread Halil Pasic
On Tue, 19 Nov 2019 18:50:03 -0600 Michael Roth wrote: [..] > I.e. the calling code is only scheduling a one-shot BH for > virtio_blk_data_plane_stop_bh, but somehow we end up trying to process > an additional virtqueue entry before we get there. This is likely due > to the following check in

Re: [PATCH v6] error: rename errp to errp_in where it is IN-argument

2019-11-28 Thread Markus Armbruster
Vladimir Sementsov-Ogievskiy writes: > 28.11.2019 17:23, Markus Armbruster wrote: >> Vladimir Sementsov-Ogievskiy writes: >> >>> Error **errp is almost always OUT-argument: it's assumed to be NULL, or >>> pointer to NULL-initialized pointer, or pointer to error_abort or >>> error_fatal, for

Re: [PATCH 2/4] target/arm: Abstract the generic timer frequency

2019-11-28 Thread Andrew Jeffery
On Thu, 28 Nov 2019, at 19:16, Cédric Le Goater wrote: > On 28/11/2019 06:45, Andrew Jeffery wrote: > > Prepare for SoCs such as the ASPEED AST2600 whose firmware configures > > CNTFRQ to values significantly larger than the static 62.5MHz value > > currently derived from GTIMER_SCALE. As the

Re: [PATCH v19 3/8] numa: Extend CLI to provide memory side cache information

2019-11-28 Thread Tao Xu
On 11/28/2019 9:57 PM, Igor Mammedov wrote: On Thu, 28 Nov 2019 12:50:36 +0100 Markus Armbruster wrote: Tao Xu writes: From: Liu Jingqi Add -numa hmat-cache option to provide Memory Side Cache Information. These memory attributes help to build Memory Side Cache Information Structure(s)

Re: [PATCH 1/7] target/ppc: Implement the VTB for HV access

2019-11-28 Thread David Gibson
On Thu, Nov 28, 2019 at 02:46:54PM +0100, Cédric Le Goater wrote: > From: Suraj Jitindar Singh > > The virtual timebase register (VTB) is a 64-bit register which > increments at the same rate as the timebase register, present on POWER8 > and later processors. > > The register is able to be

[for-5.0 3/4] spapr: Clean up RMA size calculation

2019-11-28 Thread David Gibson
Move the calculation of the Real Mode Area (RMA) size into a helper function. While we're there clean it up and correct it in a few ways: * Add comments making it clearer where the various constraints come from * Remove a pointless check that the RMA fits within Node 0 (we've just clamped

[PATCH v1 0/5] linux-user mmap debug cleanup

2019-11-28 Thread Alex Bennée
Hi, While debugging some wierd ELF loading bugs I realised our mmap debug code could do with a little clean-up so I removed the DEBUG_MMAP in favour of some tracepoints and extending the information that -d page gives you. Alex Bennée (5): linux-user: convert target_mprotect debug to

Re: [PATCH v1 0/1] s390x: protvirt: SCLP interpretation

2019-11-28 Thread no-reply
Patchew URL: https://patchew.org/QEMU/1574935984-16910-1-git-send-email-pmo...@linux.ibm.com/ Hi, This series seems to have some coding style problems. See output below for more information: Subject: [PATCH v1 0/1] s390x: protvirt: SCLP interpretation Type: series Message-id:

Re: [PATCH] io/channel-websock: treat 'binary' and no sub-protocol as the same

2019-11-28 Thread Yu-Chen Lin
Ping? Yu-Chen Lin 於 2019年11月23日 週六 11:43 寫道: > noVNC doesn't use 'binary' protocol by default after > commit c912230309806aacbae4295faf7ad6406da97617. > > It will cause qemu return 400 when handshaking. > > To overcome this problem and remain compatibility of > older noVNC client. > > We treat

Re: [PATCH v2] qga: fence guest-set-time if hwclock not available

2019-11-28 Thread Laszlo Ersek
Hi Cornelia, On 11/28/19 19:11, Cornelia Huck wrote: > The Posix implementation of guest-set-time invokes hwclock to > set/retrieve the time to/from the hardware clock. If hwclock > is not available, the user is currently informed that "hwclock > failed to set hardware clock to system time",

[PATCH v1 4/5] linux-user: log page table changes under -d page

2019-11-28 Thread Alex Bennée
The CPU_LOG_PAGE flag is woefully underused and could stand to do extra duty tracking page changes. If the user doesn't want to see the details as things change they still have the tracepoints available. Signed-off-by: Alex Bennée --- linux-user/mmap.c | 11 +++ 1 file changed, 7

Re: [PATCH 14/15] s390x: protvirt: Disable address checks for PV guest IO emulation

2019-11-28 Thread Thomas Huth
On 28/11/2019 17.10, Janosch Frank wrote: > On 11/28/19 4:28 PM, Thomas Huth wrote: >> On 20/11/2019 12.43, Janosch Frank wrote: >>> IO instruction data is routed through SIDAD for protected guests, so >>> adresses do not need to be checked, as this is kernel memory. >>> >>> Signed-off-by: Janosch

Re: [PATCH 2/2] Add -mem-shared option

2019-11-28 Thread Marc-André Lureau
Hi On Thu, Nov 28, 2019 at 9:25 PM Igor Mammedov wrote: > > On Thu, 28 Nov 2019 18:15:18 +0400 > Marc-André Lureau wrote: > > > Add an option to simplify shared memory / vhost-user setup. > > > > Currently, using vhost-user requires NUMA setup such as: > > -m 4G -object

Re: [PATCH 08/15] s390x: protvirt: KVM intercept changes

2019-11-28 Thread Janosch Frank
On 11/28/19 5:45 PM, Cornelia Huck wrote: > On Thu, 28 Nov 2019 17:38:19 +0100 > Janosch Frank wrote: > >> On 11/21/19 4:11 PM, Thomas Huth wrote: >>> On 20/11/2019 12.43, Janosch Frank wrote: Secure guests no longer intercept with code 4 for an instruction interception. Instead they

Re: [RFC 0/1] ATI R300 emulated grpahics card V2

2019-11-28 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20191128064350.20727-1-aaron.zakh...@gmail.com/ Hi, This series failed the docker-quick@centos7 build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRIPT

[PATCH v1 1/5] linux-user: convert target_mprotect debug to tracepoint

2019-11-28 Thread Alex Bennée
It is a pain to re-compile when you need to debug and tracepoints are a fairly low impact way to instrument QEMU. Signed-off-by: Alex Bennée --- linux-user/mmap.c | 17 + linux-user/trace-events | 3 +++ 2 files changed, 12 insertions(+), 8 deletions(-) diff --git

Re: [PATCH v2 2/2] travis.yml: Run tcg tests with tci

2019-11-28 Thread Stefan Weil
Am 28.11.19 um 22:06 schrieb Stefan Weil: > Am 28.11.19 um 16:35 schrieb Thomas Huth: > >> So far we only have compile coverage for tci. But since commit >> 2f160e0f9797c7522bfd0d09218d0c9340a5137c ("tci: Add implementation >> for INDEX_op_ld16u_i64") has been included now, we can also run the >>

Re: [PATCH v2 1/4] qom/object: enable setter for uint types

2019-11-28 Thread Marc-André Lureau
Hi On Thu, Nov 28, 2019 at 8:48 PM Felipe Franciosi wrote: > > Traditionally, the uint-specific property helpers only offer getters. > When adding object (or class) uint types, one must therefore use the > generic property helper if a setter is needed (and probably duplicate > some code writing

[for-5.0 1/4] spapr,ppc: Simplify signature of kvmppc_rma_size()

2019-11-28 Thread David Gibson
This function calculates the maximum size of the RMA as implied by the host's page size of structure of the VRMA (there are a number of other constraints on the RMA size which will supersede this one in many circumstances). The current interface takes the current RMA size estimate, and clamps it

Re: [PATCH 2/7] target/ppc: Work [S]PURR implementation and add HV support

2019-11-28 Thread David Gibson
On Thu, Nov 28, 2019 at 02:46:55PM +0100, Cédric Le Goater wrote: > From: Suraj Jitindar Singh > > The Processor Utilisation of Resources Register (PURR) and Scaled > Processor Utilisation of Resources Register (SPURR) provide an estimate > of the resources used by the thread, present on POWER7

Re: [PATCH v19 3/8] numa: Extend CLI to provide memory side cache information

2019-11-28 Thread Tao Xu
On 11/28/2019 7:50 PM, Markus Armbruster wrote: Tao Xu writes: From: Liu Jingqi Add -numa hmat-cache option to provide Memory Side Cache Information. These memory attributes help to build Memory Side Cache Information Structure(s) in ACPI Heterogeneous Memory Attribute Table (HMAT). Before

[for-5.0 3/4] spapr: Fold h_cas_compose_response() into h_client_architecture_support()

2019-11-28 Thread David Gibson
spapr_h_cas_compose_response() handles the last piece of the PAPR feature negotiation process invoked via the ibm,client-architecture-support OF call. Its only caller is h_client_architecture_support() which handles most of the rest of that process. I believe it was place in a separate file

Re: [PATCH v6] backends/cryptodev: drop local_err from cryptodev_backend_complete()

2019-11-28 Thread Markus Armbruster
Vladimir Sementsov-Ogievskiy writes: > No reason for local_err here, use errp directly instead. Related: "[PATCH v6] hw/vfio/ap: drop local_err from vfio_ap_realize". I'm surprised it's just two. Did you search for the anti-pattern systematically? > Signed-off-by: Vladimir Sementsov-Ogievskiy

Re: [PATCH v6] hw/core/qdev: cleanup Error ** variables

2019-11-28 Thread Markus Armbruster
Vladimir Sementsov-Ogievskiy writes: > Rename Error ** parameter in check_only_migratable to common errp. > > In device_set_realized: > > - Move "if (local_err != NULL)" closer to error setters. > > - Drop 'Error **local_errp': it doesn't save any LoCs, but it's very >unusual. > >

Re: [PATCH 0/2] RFC: add -mem-shared option

2019-11-28 Thread Gerd Hoffmann
On Thu, Nov 28, 2019 at 06:15:16PM +0400, Marc-André Lureau wrote: > Hi, > > Setting up shared memory for vhost-user is a bit complicated from > command line, as it requires NUMA setup such as: m 4G -object > memory-backend-file,id=mem,size=4G,mem-path=/dev/shm,share=on -numa > node,memdev=mem. >

Re: [PATCH v1 1/1] pc-bios/s390-ccw: fix sclp_get_loadparm_ascii

2019-11-28 Thread Christian Borntraeger
On 28.11.19 16:05, Peter Maydell wrote: > On Thu, 28 Nov 2019 at 12:48, Christian Borntraeger > wrote: >> >> >> >> On 28.11.19 13:45, Cornelia Huck wrote: >>> On Thu, 28 Nov 2019 13:35:29 +0100 >>> Christian Borntraeger wrote: >>> Ack. Conny, I think this would be really nice

Re: [PATCH 1/3] target/arm: Honor HCR_EL2.TID2 trapping requirements

2019-11-28 Thread Edgar E. Iglesias
On Thu, Nov 28, 2019 at 04:17:16PM +, Marc Zyngier wrote: > HCR_EL2.TID2 mandates that access from EL1 to CTR_EL0, CCSIDR_EL1, > CCSIDR2_EL1, CLIDR_EL1, CSSELR_EL1 are trapped to EL2, and QEMU > completely ignores it, making impossible for hypervisors to Nit: "making it impossible" >

Re: [PATCH 0/2] RFC: add -mem-shared option

2019-11-28 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20191128141518.628245-1-marcandre.lur...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Subject: [PATCH 0/2] RFC: add -mem-shared option Type: series Message-id:

Re: [PATCH v6] hw/vfio/ap: drop local_err from vfio_ap_realize

2019-11-28 Thread Markus Armbruster
Vladimir Sementsov-Ogievskiy writes: > No reason for local_err here, use errp directly instead. > > Signed-off-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Markus Armbruster

Re: [PATCH 0/2] RFC: add -mem-shared option

2019-11-28 Thread Marc-André Lureau
Hi On Fri, Nov 29, 2019 at 11:03 AM Gerd Hoffmann wrote: > > On Thu, Nov 28, 2019 at 06:15:16PM +0400, Marc-André Lureau wrote: > > Hi, > > > > Setting up shared memory for vhost-user is a bit complicated from > > command line, as it requires NUMA setup such as: m 4G -object > >

Re: [PATCH 0/2] RFC: add -mem-shared option

2019-11-28 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20191128141518.628245-1-marcandre.lur...@redhat.com/ Hi, This series failed the docker-quick@centos7 build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST

[for-5.0 1/4] spapr: Don't trigger a CAS reboot for XICS/XIVE mode changeover

2019-11-28 Thread David Gibson
PAPR allows the interrupt controller used on a POWER9 machine (XICS or XIVE) to be selected by the guest operating system, by using the ibm,client-architecture-support (CAS) feature negotiation call. Currently, if the guest selects an interrupt controller different from the one selected at

[RFC] smbios: does it make sense to present some host smbios information to guest?

2019-11-28 Thread Guoheyi
Hi folks, Right now some smbios fields are hard coded (like CPU nominal frequency), and some can be opted in, but there is no feasible way to present real backend hardware information to the front. In some scenario, the users of virtual machines may not be happy to see an unknown CPU model

[PATCH V3 2/2] block/nbd: fix memory leak in nbd_open()

2019-11-28 Thread pannengyuan
From: PanNengyuan In currently implementation there will be a memory leak when nbd_client_connect() returns error status. Here is an easy way to reproduce: 1. run qemu-iotests as follow and check the result with asan: ./check -raw 143 Following is the asan output backtrack: Direct leak of

[for-5.0 0/4] spapr: Improvements to CAS feature negotiation

2019-11-28 Thread David Gibson
This series contains several cleanups to the handling of the ibm,client-architecture-support firmware call used for boot time feature negotiation between the guest OS and the firmware & hypervisor. Mostly it's just internal polish, but one significant user visible change is that we no longer

[for-5.0 2/4] spapr: Improve handling of fdt buffer size

2019-11-28 Thread David Gibson
Previously, spapr_build_fdt() constructed the device tree in a fixed buffer of size FDT_MAX_SIZE. This is a bit inflexible, but more importantly it's awkward for the case where we use it during CAS. In that case the guest firmware supplies a buffer and we have to awkwardly check that what we

[for-5.0 4/4] spapr: Simplify ovec diff

2019-11-28 Thread David Gibson
spapr_ovec_diff(ov, old, new) has somewhat complex semantics. ov is set to those bits which are in new but not old, and it returns as a boolean whether or not there are any bits in old but not new. It turns out that both callers only care about the second, not the first. This is basically

Re: [PATCH v6] vnc: drop Error pointer indirection in vnc_client_io_error

2019-11-28 Thread Markus Armbruster
Vladimir Sementsov-Ogievskiy writes: > We don't need Error **, as all callers pass local Error object, which > isn't used after the call, or NULL. Use Error * instead. > > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > ui/vnc.h | 2 +- > ui/vnc.c | 20 +++- > 2 files

Re: [PATCH] hw: add compat machines for 5.0

2019-11-28 Thread Cornelia Huck
On Tue, 12 Nov 2019 11:48:11 +0100 Cornelia Huck wrote: > Add 5.0 machine types for arm/i440fx/q35/s390x/spapr. > > For i440fx and q35, unversioned cpu models are still translated > to -v1; I'll leave changing this (if desired) to the respective > maintainers. > > Signed-off-by: Cornelia Huck

Re: [PATCH v6] hmp: drop Error pointer indirection in hmp_handle_error

2019-11-28 Thread Markus Armbruster
Vladimir Sementsov-Ogievskiy writes: > We don't need Error **, as all callers pass local Error object, which > isn't used after the call. Use Error * instead. > > Signed-off-by: Vladimir Sementsov-Ogievskiy > Acked-by: Dr. David Alan Gilbert Reviewed-by: Markus Armbruster

Re: [PATCH] hw: add compat machines for 5.0

2019-11-28 Thread Cornelia Huck
On Thu, 28 Nov 2019 17:38:11 -0300 Eduardo Habkost wrote: > On Thu, Nov 28, 2019 at 06:37:06PM +0100, Cornelia Huck wrote: > > On Tue, 12 Nov 2019 11:48:11 +0100 > > Cornelia Huck wrote: > > > > > Add 5.0 machine types for arm/i440fx/q35/s390x/spapr. > > > > > > For i440fx and q35,

[PATCH V3 1/2] block/nbd: extract the common cleanup code

2019-11-28 Thread pannengyuan
From: PanNengyuan The BDRVNBDState cleanup code is common in two places, add nbd_free_bdrvstate_prop() function to do these cleanups (suggested by Stefano Garzarella). Signed-off-by: PanNengyuan --- block/nbd.c | 23 +-- 1 file changed, 13 insertions(+), 10 deletions(-)

[PATCH v20 3/8] numa: Extend CLI to provide memory side cache information

2019-11-28 Thread Tao Xu
From: Liu Jingqi Add -numa hmat-cache option to provide Memory Side Cache Information. These memory attributes help to build Memory Side Cache Information Structure(s) in ACPI Heterogeneous Memory Attribute Table (HMAT). Before using hmat-cache option, enable HMAT with -machine hmat=on.

Re: [PATCH v2] qga: fence guest-set-time if hwclock not available

2019-11-28 Thread Cornelia Huck
On Thu, 28 Nov 2019 19:38:00 +0100 Laszlo Ersek wrote: > Hi Cornelia, > > On 11/28/19 19:11, Cornelia Huck wrote: > > The Posix implementation of guest-set-time invokes hwclock to > > set/retrieve the time to/from the hardware clock. If hwclock > > is not available, the user is currently

Re: [PATCH v5] qga: add command guest-get-devices for reporting VirtIO devices

2019-11-28 Thread Daniel P . Berrangé
On Thu, Nov 28, 2019 at 01:45:13PM +0100, Tomáš Golembiovský wrote: > Add command for reporting devices on Windows guest. The intent is not so > much to report the devices but more importantly the driver (and its > version) that is assigned to the device. This gives caller the > information

Re: [PATCH v1 0/1] s390x: protvirt: SCLP interpretation

2019-11-28 Thread Pierre Morel
On 2019-11-28 13:28, Janosch Frank wrote: On 11/28/19 11:13 AM, Pierre Morel wrote: A new proposition: I think it would be wise to fork directly from handle_instruction instead to accept per default all instructions with with secure instruction interception code. Just in case future firmware

Re: [PATCH v37 00/17] QEMU AVR 8 bit cores

2019-11-28 Thread Aleksandar Markovic
On Thursday, November 28, 2019, Michael Rolnik wrote: > > > On Wed, Nov 27, 2019 at 11:06 PM Aleksandar Markovic < > aleksandar.m.m...@gmail.com> wrote: > >> On Wed, Nov 27, 2019 at 6:53 PM Michael Rolnik wrote: >> > >> > This series of patches adds 8bit AVR cores to QEMU. >> > All instruction,

[PATCH 1/7] target/ppc: Implement the VTB for HV access

2019-11-28 Thread Cédric Le Goater
From: Suraj Jitindar Singh The virtual timebase register (VTB) is a 64-bit register which increments at the same rate as the timebase register, present on POWER8 and later processors. The register is able to be read/written by the hypervisor and read by the supervisor. All other accesses are

[PATCH 4/7] target/ppc: Add SPR TBU40

2019-11-28 Thread Cédric Le Goater
From: Suraj Jitindar Singh The spr TBU40 is used to set the upper 40 bits of the timebase register, present on POWER5+ and later processors. This register can only be written by the hypervisor, and cannot be read. Signed-off-by: Suraj Jitindar Singh Reviewed-by: David Gibson Signed-off-by:

  1   2   3   >