RE: [RFC 0/1] Use dmabufs for display updates instead of pixman

2021-03-09 Thread Kasireddy, Vivek
Hi Gerd, > Yes, it surely makes sense to go into that direction. > The patch as-is doesn't, it breaks the guest/host interface. > That's ok-ish for a quick proof-of-concept, but clearly not merge-able. > > > TODO: > > - Use Blob resources for getting meta-data such as modifier, format, etc. > >

Re: [RFC PATCH v3 02/10] net: Pad short frames to minimum size before send from SLiRP/TAP

2021-03-09 Thread Jason Wang
On 2021/3/8 6:22 下午, Peter Maydell wrote: On Mon, 8 Mar 2021 at 03:48, Jason Wang wrote: Do we need to care about other type of networking backends? E.g socket. Or at least we should keep the padding logic if we can't audit all of the backends. I think the key thing we need to do here is ma

RE: [RFC 1/1] virtio-gpu: Use dmabuf for display updates if possible instead of pixman

2021-03-09 Thread Kasireddy, Vivek
Hi Marc-Andre, +create = g_malloc0(sizeof(*create) + + res->iov_cnt * sizeof (struct udmabuf_create_item)); +if (!create) +return NULL; Pointless allocation check (g_malloc will abort if it failed to allocate) [Kasireddy, Vivek] Ok, will remove it in v2. +

[PATCH] virtio-gpu: Add spaces around operator

2021-03-09 Thread lijiejun
Fix code style. Operator needs spaces forend sides, and delete line space. Signed-off-by: lijiejun --- hw/display/virtio-gpu-3d.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/hw/display/virtio-gpu-3d.c b/hw/display/virtio-gpu-3d.c index 9eb4890..d989648 100644 --- a/hw/d

Re: [RFC PATCH v3 02/10] net: Pad short frames to minimum size before send from SLiRP/TAP

2021-03-09 Thread Bin Meng
Hi Jason, On Tue, Mar 9, 2021 at 4:23 PM Jason Wang wrote: > > > On 2021/3/8 6:22 下午, Peter Maydell wrote: > > On Mon, 8 Mar 2021 at 03:48, Jason Wang wrote: > >> Do we need to care about other type of networking backends? E.g socket. > >> > >> Or at least we should keep the padding logic if we

Re: [PATCH v2 4/7] hw/arm/smmu-common: Fix smmu_iotlb_inv_iova when asid is not set

2021-03-09 Thread Auger Eric
Hi Peter, On 3/8/21 5:37 PM, Peter Maydell wrote: > On Thu, 25 Feb 2021 at 09:15, Eric Auger wrote: >> >> If the asid is not set, do not attempt to locate the key directly >> as all inserted keys have a valid asid. >> >> Use g_hash_table_foreach_remove instead. >> >> Signed-off-by: Eric Auger >>

Re: [PATCH v4 0/3] vnc: support reload x509 certificates

2021-03-09 Thread Zihao Chang
Ping? Is this patch set being reviewed? Who can help to do this? Thanks Zihao On 2021/2/24 20:34, Zihao Chang wrote: > This series supports reload x509 certificates for vnc > 1. Support reload x509 certificates. > 2. Support reload vnc certificates. > 3. Add new qmp display-reload and implement

Re: [RFC PATCH v2] meson: Only generate trace files for selected targets

2021-03-09 Thread Claudio Fontana
On 3/8/21 6:48 PM, Philippe Mathieu-Daudé wrote: > On 3/8/21 5:46 PM, Stefan Hajnoczi wrote: >> On Wed, Feb 24, 2021 at 10:35:15PM +0100, Philippe Mathieu-Daudé wrote: >>> We don't need to generate trace files for targets we >>> are not building. Restrict the the ones selected. >>> >>> Signed-off-b

Re: [PATCH V2] virtio-net: calculating proper msix vectors on init

2021-03-09 Thread Philippe Mathieu-Daudé
On 3/9/21 5:23 AM, Jason Wang wrote: > Currently, the default msix vectors for virtio-net-pci is 3 which is > obvious not suitable for multiqueue guest, so we depends on the user > or management tools to pass a correct vectors parameter. In fact, we > can simplifying this by calculating the number

Re: [RFC PATCH v3 02/10] net: Pad short frames to minimum size before send from SLiRP/TAP

2021-03-09 Thread Jason Wang
On 2021/3/9 4:35 下午, Bin Meng wrote: Hi Jason, On Tue, Mar 9, 2021 at 4:23 PM Jason Wang wrote: On 2021/3/8 6:22 下午, Peter Maydell wrote: On Mon, 8 Mar 2021 at 03:48, Jason Wang wrote: Do we need to care about other type of networking backends? E.g socket. Or at least we should keep the

Re: [RFC PATCH v3 02/10] net: Pad short frames to minimum size before send from SLiRP/TAP

2021-03-09 Thread Bin Meng
Hi Jason, On Tue, Mar 9, 2021 at 4:57 PM Jason Wang wrote: > > > On 2021/3/9 4:35 下午, Bin Meng wrote: > > Hi Jason, > > > > On Tue, Mar 9, 2021 at 4:23 PM Jason Wang wrote: > >> > >> On 2021/3/8 6:22 下午, Peter Maydell wrote: > >>> On Mon, 8 Mar 2021 at 03:48, Jason Wang wrote: > Do we need

Re: [RFC PATCH v3 02/10] net: Pad short frames to minimum size before send from SLiRP/TAP

2021-03-09 Thread Bin Meng
Hi Jason, On Tue, Mar 9, 2021 at 5:00 PM Bin Meng wrote: > > Hi Jason, > > On Tue, Mar 9, 2021 at 4:57 PM Jason Wang wrote: > > > > > > On 2021/3/9 4:35 下午, Bin Meng wrote: > > > Hi Jason, > > > > > > On Tue, Mar 9, 2021 at 4:23 PM Jason Wang wrote: > > >> > > >> On 2021/3/8 6:22 下午, Peter Mayd

Re: [PATCH] virtio-gpu: Add spaces around operator

2021-03-09 Thread Philippe Mathieu-Daudé
(Cc'ing qemu-trivial@) On 3/9/21 8:42 AM, lijiejun wrote: > Fix code style. Operator needs spaces forend sides, and delete line space. "forend"? Otherwise: Reviewed-by: Philippe Mathieu-Daudé > > Signed-off-by: lijiejun > --- > hw/display/virtio-gpu-3d.c | 3 +-- > 1 file changed, 1 inserti

Re: [PATCH 0/6] hw/mips/gt64120: Minor fixes

2021-03-09 Thread Philippe Mathieu-Daudé
Hi Zoltan, On 3/5/21 5:21 PM, Philippe Mathieu-Daudé wrote: > Trivial fixes extracted from another series which became too big, > so I prefer to send them in a previous step. I just realized I meant to Cc you on this series but forgot :/ As this model is pretty close to your MV64361 one, and this

Re: [PATCH v5 4/8] vt82c686: Introduce abstract TYPE_VIA_ISA and base vt82c686b_isa on it

2021-03-09 Thread Philippe Mathieu-Daudé
On 3/5/21 2:02 AM, David Gibson wrote: > On Thu, Mar 04, 2021 at 11:42:10PM +0100, Philippe Mathieu-Daudé wrote: >> On 3/4/21 9:16 PM, BALATON Zoltan wrote: >>> On Thu, 4 Mar 2021, Philippe Mathieu-Daudé wrote: On 3/2/21 10:11 PM, BALATON Zoltan wrote: > To allow reusing ISA bridge emulati

Re: [PATCH v3 03/30] qapi/qom: Add ObjectOptions for authz-*

2021-03-09 Thread Daniel P . Berrangé
On Mon, Mar 08, 2021 at 05:54:13PM +0100, Kevin Wolf wrote: > This adds a QAPI schema for the properties of the authz-* objects. > > Signed-off-by: Kevin Wolf > Acked-by: Peter Krempa > Reviewed-by: Eric Blake > --- > qapi/authz.json | 61 +--- > qa

Re: [PATCH V2] virtio-net: calculating proper msix vectors on init

2021-03-09 Thread Stefan Hajnoczi
On Tue, Mar 09, 2021 at 12:23:14PM +0800, Jason Wang wrote: > Currently, the default msix vectors for virtio-net-pci is 3 which is > obvious not suitable for multiqueue guest, so we depends on the user > or management tools to pass a correct vectors parameter. In fact, we > can simplifying this by

Re: [RFC PATCH v2] meson: Only generate trace files for selected targets

2021-03-09 Thread Philippe Mathieu-Daudé
On 3/9/21 9:41 AM, Claudio Fontana wrote: > On 3/8/21 6:48 PM, Philippe Mathieu-Daudé wrote: >> On 3/8/21 5:46 PM, Stefan Hajnoczi wrote: >>> On Wed, Feb 24, 2021 at 10:35:15PM +0100, Philippe Mathieu-Daudé wrote: We don't need to generate trace files for targets we are not building. Rest

Re: [PATCH v3 09/30] qapi/qom: Add ObjectOptions for secret*, deprecate 'loaded'

2021-03-09 Thread Daniel P . Berrangé
On Mon, Mar 08, 2021 at 05:54:19PM +0100, Kevin Wolf wrote: > This adds a QAPI schema for the properties of the secret* objects. > > The 'loaded' property doesn't seem to make sense as an external > interface: It is automatically set to true in ucc->complete, and > explicitly setting it to true ea

Re: [PATCH v3 10/30] qapi/qom: Add ObjectOptions for tls-*, deprecate 'loaded'

2021-03-09 Thread Daniel P . Berrangé
On Mon, Mar 08, 2021 at 05:54:20PM +0100, Kevin Wolf wrote: > This adds a QAPI schema for the properties of the tls-* objects. > > The 'loaded' property doesn't seem to make sense as an external > interface: It is automatically set to true in ucc->complete, and > explicitly setting it to true earl

Re: [PATCH v3] bsd-user: Add new maintainers

2021-03-09 Thread Daniel P . Berrangé
On Mon, Mar 08, 2021 at 03:40:23PM -0700, Warner Losh wrote: > The FreeBSD project has a number of enhancements to bsd-user. These changes > have > evolved over the past 10 year, and aren't currently updated to the latest > version of qemu due to fluxuations in staffing causing us to fall behind i

Re: [PATCH v4 3/3] qmp: add new qmp display-reload

2021-03-09 Thread Markus Armbruster
Sorry for the delay, I missed this one. Zihao Chang writes: > This patch provides a new qmp to reload display configuration > without restart VM, but only reloading the vnc tls certificates > is implemented. > Example: > {"execute": "display-reload", "arguments":{"vnc-tls-certs": true}} > > Sign

Re: [PATCH v2] block/file-posix: Optimize for macOS

2021-03-09 Thread Kevin Wolf
Am 09.03.2021 um 05:52 hat Akihiko Odaki geschrieben: > 2021年3月9日(火) 0:37 Akihiko Odaki : > > > > 2021年3月9日(火) 0:17 Stefan Hajnoczi : > > > > > > The live migration compatibility issue is still present. Migrating to > > > another host might not work if the block limits are different. > > > > > > He

Re: [PATCH] coroutine: add libucontext as external library

2021-03-09 Thread Daniel P . Berrangé
On Mon, Mar 08, 2021 at 07:26:36PM -0800, Joelle van Dyne wrote: > iOS does not support ucontext natively for aarch64 and the sigaltstack is > also unsupported (even worse, it fails silently, see: > https://openradar.appspot.com/13002712 ) > > As a workaround we include a library implementation of

Re: [PATCH 2/3] memory: Provide 'base address' argument to mtree_print_mr()

2021-03-09 Thread Philippe Mathieu-Daudé
Hi Peter, On 3/9/21 12:40 AM, Peter Xu wrote:> On Sat, Mar 06, 2021 at 12:54:13AM +0100, Philippe Mathieu-Daudé wrote: >> @@ -3188,14 +3188,15 @@ void mtree_info(bool flatview, bool dispatch_tree, >> bool owner, bool disabled) >> >> QTAILQ_FOREACH(as, &address_spaces, address_spaces_link)

Re: [RFC 0/1] Use dmabufs for display updates instead of pixman

2021-03-09 Thread Gerd Hoffmann
Hi, > > That is pretty much mandatory. Without blob resources there is no concept > > of resources > > shared between host and guest in virtio-gpu, all data is explicitly copied > > with transfer > > commands. > [Kasireddy, Vivek] My understanding of virtio-gpu and the concept of > resources

[PATCH 0/4] vhost-user-blk-test: add vhost-user-blk server tests

2021-03-09 Thread Stefan Hajnoczi
These patches were dropped from Kevin's last block pull request due to a GitLab CI failure. The old CentOS 7 and opensuse Leap build images lack memfd_create(2) (introduced in Linux around 2014). The vhost-user-blk server had a dependency on memfd_create(2) due to VHOST_USER_PROTOCOL_F_INFLIGHT_SHM

[PATCH 4/4] vhost-user-blk-test: test discard/write zeroes invalid inputs

2021-03-09 Thread Stefan Hajnoczi
Exercise input validation code paths in block/export/vhost-user-blk-server.c. Signed-off-by: Stefan Hajnoczi --- tests/qtest/vhost-user-blk-test.c | 124 ++ 1 file changed, 124 insertions(+) diff --git a/tests/qtest/vhost-user-blk-test.c b/tests/qtest/vhost-user-blk

[PATCH 2/4] test: new qTest case to test the vhost-user-blk-server

2021-03-09 Thread Stefan Hajnoczi
From: Coiby Xu This test case has the same tests as tests/virtio-blk-test.c except for tests have block_resize. Since the vhost-user-blk export only serves one client one time, two exports are started by qemu-storage-daemon for the hotplug test. Suggested-by: Thomas Huth Signed-off-by: Coiby Xu

Re: [PATCH 1/2] hw: Model ASPEED's Hash and Crypto Engine

2021-03-09 Thread Philippe Mathieu-Daudé
On 3/3/21 8:03 AM, Joel Stanley wrote: > The HACE (Hash and Crpyto Engine) is a device that offloads MD5, SHA1, Typo "Crypto" > SHA2, RSA and other cryptographic algorithms. > > This initial model implements a subset of the device's functionality; > currently only direct access (non-scatter gath

[PATCH 1/4] block/export: disable VHOST_USER_PROTOCOL_F_INFLIGHT_SHMFD for now

2021-03-09 Thread Stefan Hajnoczi
The vhost-user in-flight shmfd feature has not been tested with qemu-storage-daemon's vhost-user-blk server. Disable this optional feature for now because it requires MFD_ALLOW_SEALING, which is not available in some CI environments. If we need this feature in the future it can be re-enabled after

Re: [PATCH v2 1/2] ramlist: Make dirty bitmap blocks of ramlist resizable

2021-03-09 Thread Stefan Hajnoczi
On Sat, Dec 26, 2020 at 03:11:53PM +0800, Keqian Zhu wrote: > > [...] > > >>> -for (j = old_num_blocks; j < new_num_blocks; j++) { > >>> -new_blocks->blocks[j] = bitmap_new(DIRTY_MEMORY_BLOCK_SIZE); > >>> +if (extend) { > >>> +for (j = cpy_num_blocks; j < n

[PATCH 3/4] tests/qtest: add multi-queue test case to vhost-user-blk-test

2021-03-09 Thread Stefan Hajnoczi
Signed-off-by: Stefan Hajnoczi --- tests/qtest/vhost-user-blk-test.c | 81 +-- 1 file changed, 76 insertions(+), 5 deletions(-) diff --git a/tests/qtest/vhost-user-blk-test.c b/tests/qtest/vhost-user-blk-test.c index f0fb09893e..61beee52d3 100644 --- a/tests/qtest/vh

Re: What is the intended behaviour of usb_host_get_port for root hubs?

2021-03-09 Thread Gerd Hoffmann
Hi, > Of course, reading the current code more carefully, usb_host_get_port > is never actually called with a hub device (let alone a root hub), Ah, the root hub itself. I've assumed you mean a device plugged into the root hub. That explains the confusion. Yes, passing through hubs is simply

Re: [PATCH v2 0/2] tests/acceptance: Test Aspeed ARM machines

2021-03-09 Thread Philippe Mathieu-Daudé
+Alex On 3/5/21 4:38 PM, Cédric Le Goater wrote: > On 3/5/21 4:12 PM, Peter Maydell wrote: >> On Thu, 4 Mar 2021 at 12:40, Joel Stanley wrote: >>> >>> v2: Fix ast2600 test, thanks Cédric for the review. >>> >>> This adds tests for the Aspeed ARM SoCs. The AST2400 and AST2500 tests >>> use OpenBMC

Re: [PATCH] ui/gtk: Remove NULL checks in gd_switch

2021-03-09 Thread Gerd Hoffmann
On Mon, Mar 08, 2021 at 11:07:13PM +0900, Akihiko Odaki wrote: > c821a58ee7 ("ui/console: Pass placeholder surface to display") > eliminated the possibility that NULL is passed as surface to > dpy_gfx_switch and removed some NULL checks from gd_switch, but the > removal was not thoroughly. Remainin

Re: [PATCH] coroutine: add libucontext as external library

2021-03-09 Thread Joelle van Dyne
On Tue, Mar 9, 2021 at 1:36 AM Daniel P. Berrangé wrote: > > On Mon, Mar 08, 2021 at 07:26:36PM -0800, Joelle van Dyne wrote: > > iOS does not support ucontext natively for aarch64 and the sigaltstack is > > also unsupported (even worse, it fails silently, see: > > https://openradar.appspot.com/13

Re: [PATCH] plugins: Expose physical addresses instead of device offsets

2021-03-09 Thread Peter Maydell
On Mon, 8 Mar 2021 at 20:14, Aaron Lindsay wrote: > > This allows plugins to query for full virtual-to-physical address > translation for a given `qemu_plugin_hwaddr` and stops exposing the > offset within the device itself. As this change breaks the API, > QEMU_PLUGIN_VERSION is incremented. > >

Re: [PATCH v2 4/4] slirp: feature detection for smbd

2021-03-09 Thread Peter Maydell
On Tue, 9 Mar 2021 at 00:30, Joelle van Dyne wrote: > > Replace Windows specific macro with a more generic feature detection > macro. Allows slirp smb feature to be disabled manually as well. > > Signed-off-by: Joelle van Dyne > --- Reviewed-by: Peter Maydell thanks -- PMM

Re: [RFC PATCH v3 02/10] net: Pad short frames to minimum size before send from SLiRP/TAP

2021-03-09 Thread Peter Maydell
On Tue, 9 Mar 2021 at 09:01, Bin Meng wrote: > > Hi Jason, > > On Tue, Mar 9, 2021 at 5:00 PM Bin Meng wrote: > > > > Hi Jason, > > > > On Tue, Mar 9, 2021 at 4:57 PM Jason Wang wrote: > > > > > > > > > On 2021/3/9 4:35 下午, Bin Meng wrote: > > > > Hi Jason, > > > > > > > > On Tue, Mar 9, 2021 at

Re: [PATCH v3] bsd-user: Add new maintainers

2021-03-09 Thread Peter Maydell
On Mon, 8 Mar 2021 at 22:43, Warner Losh wrote: > > The FreeBSD project has a number of enhancements to bsd-user. These changes > have > evolved over the past 10 year, and aren't currently updated to the latest > version of qemu due to fluxuations in staffing causing us to fall behind in > the >

Re: [RFC PATCH v3 02/10] net: Pad short frames to minimum size before send from SLiRP/TAP

2021-03-09 Thread Bin Meng
Hi Peter, On Tue, Mar 9, 2021 at 6:13 PM Peter Maydell wrote: > > On Tue, 9 Mar 2021 at 09:01, Bin Meng wrote: > > > > Hi Jason, > > > > On Tue, Mar 9, 2021 at 5:00 PM Bin Meng wrote: > > > > > > Hi Jason, > > > > > > On Tue, Mar 9, 2021 at 4:57 PM Jason Wang wrote: > > > > > > > > > > > > On

Re: [PATCH] coroutine: add libucontext as external library

2021-03-09 Thread Peter Maydell
On Tue, 9 Mar 2021 at 09:59, Joelle van Dyne wrote: > > On Tue, Mar 9, 2021 at 1:36 AM Daniel P. Berrangé wrote: > > The README here: > > > > https://github.com/kaniini/libucontext > > > > indicates that it is intentionally limiting what registers it saves > > and restores, explicitly excluding

[RFC PATCH 4/4] coroutine/rwlock: Wake writers in preference to readers

2021-03-09 Thread David Edmondson
A feature of the current rwlock is that if multiple coroutines hold a reader lock, all must be runnable. The unlock implementation relies on this, choosing to wake a single coroutine when the final read lock holder exits the critical section, assuming that it will wake a coroutine attempting to acq

[RFC PATCH 3/4] coroutine/mutex: Store the coroutine in the CoWaitRecord only once

2021-03-09 Thread David Edmondson
When taking the slow path for mutex acquisition, set the coroutine value in the CoWaitRecord in push_waiter(), rather than both there and in the caller. Signed-off-by: David Edmondson --- util/qemu-coroutine-lock.c | 1 - 1 file changed, 1 deletion(-) diff --git a/util/qemu-coroutine-lock.c b/u

[RFC PATCH 2/4] block/vdi: Don't assume that blocks are larger than VdiHeader

2021-03-09 Thread David Edmondson
Given that the block size is read from the header of the VDI file, a wide variety of sizes might be seen. Rather than re-using a block sized memory region when writing the VDI header, allocate an appropriately sized buffer. Signed-off-by: David Edmondson --- block/vdi.c | 10 ++ 1 file c

[RFC PATCH 1/4] block/vdi: When writing new bmap entry fails, don't leak the buffer

2021-03-09 Thread David Edmondson
If a new bitmap entry is allocated, requiring the entire block to be written, avoiding leaking the buffer allocated for the block should the write fail. Signed-off-by: David Edmondson --- block/vdi.c | 1 + 1 file changed, 1 insertion(+) diff --git a/block/vdi.c b/block/vdi.c index 5627e7d764..

[RFC PATCH 0/4] coroutine rwlock downgrade fix, minor VDI changes

2021-03-09 Thread David Edmondson
RFC because changing the coroutine code is scary and I'm new to it. Stressing the VDI code with qemu-img: qemu-img convert -p -W -m 16 -O vdi input.qcow2 output.vdi leads to a hang relatively quickly on a machine with sufficient CPUs. A similar test targetting either raw or qcow2 formats, or a

Re: [PATCH] coroutine: add libucontext as external library

2021-03-09 Thread Paolo Bonzini
On 09/03/21 10:59, Joelle van Dyne wrote: Does it make a difference if this is provided as an option and not as a replacement? Would it make sense to add some warning at configure time? Right now none of the concurrency backends are supported on iOS and it's possible support will go away on macOS

Re: [PATCH v2] block/file-posix: Optimize for macOS

2021-03-09 Thread Daniel P . Berrangé
On Tue, Mar 09, 2021 at 10:37:18AM +0100, Kevin Wolf wrote: > Am 09.03.2021 um 05:52 hat Akihiko Odaki geschrieben: > > 2021年3月9日(火) 0:37 Akihiko Odaki : > > > > > > 2021年3月9日(火) 0:17 Stefan Hajnoczi : > > > > > > > > The live migration compatibility issue is still present. Migrating to > > > > ano

Re: [RFC PATCH v2] meson: Only generate trace files for selected targets

2021-03-09 Thread Paolo Bonzini
On 09/03/21 09:41, Claudio Fontana wrote: In any case, I agree that just recursing all directories in target/ looking for trace.h / traceevents files could be better. Using the directory structure to find source files is a bad idea because you don't notice that you haven't "git add"ed a file u

[PATCH v3 0/7] Some vIOMMU fixes

2021-03-09 Thread Eric Auger
Hi, Here is a set of vIOMMU fixes: SMMUv3: - top SID computation overflow when handling SMMU_CMD_CFGI_ALL - internal IOTLB handling (changes related to range invalidation) - smmu_iotlb_inv_iova with asid = -1 - non power of 2 invalidation range handling. VIRTIO-IOMMU: - non power of 2 inva

[PATCH v3 5/7] hw/arm/smmuv3: Enforce invalidation on a power of two range

2021-03-09 Thread Eric Auger
As of today, the driver can invalidate a number of pages that is not a power of 2. However IOTLB unmap notifications and internal IOTLB invalidations work with masks leading to erroneous invalidations. In case the range is not a power of 2, split invalidations into power of 2 invalidations. When

[PATCH v3 2/7] dma: Introduce dma_aligned_pow2_mask()

2021-03-09 Thread Eric Auger
Currently get_naturally_aligned_size() is used by the intel iommu to compute the maximum invalidation range based on @size which is a power of 2 while being aligned with the @start address and less than the maximum range defined by @gaw. This helper is also useful for other iommu devices (virtio-i

[PATCH v3 1/7] intel_iommu: Fix mask may be uninitialized in vtd_context_device_invalidate

2021-03-09 Thread Eric Auger
With -Werror=maybe-uninitialized configuration we get ../hw/i386/intel_iommu.c: In function ‘vtd_context_device_invalidate’: ../hw/i386/intel_iommu.c:1888:10: error: ‘mask’ may be used uninitialized in this function [-Werror=maybe-uninitialized] 1888 | mask = ~mask; | ~^~~ A

[PATCH v3 3/7] virtio-iommu: Handle non power of 2 range invalidations

2021-03-09 Thread Eric Auger
Unmap notifiers work with an address mask assuming an invalidation range of a power of 2. Nothing mandates this in the VIRTIO-IOMMU spec. So in case the range is not a power of 2, split it into several invalidations. Signed-off-by: Eric Auger Reviewed-by: Peter Xu --- v2 -> v3: - added Peter'

[PATCH v3 6/7] hw/arm/smmuv3: Fix SMMU_CMD_CFGI_STE_RANGE handling

2021-03-09 Thread Eric Auger
If the whole SID range (32b) is invalidated (SMMU_CMD_CFGI_ALL), @end overflows and we fail to handle the command properly. Once this gets fixed, the current code really is awkward in the sense it loops over the whole range instead of removing the currently cached configs through a hash table look

[PATCH v3 4/7] hw/arm/smmu-common: Fix smmu_iotlb_inv_iova when asid is not set

2021-03-09 Thread Eric Auger
If the asid is not set, do not attempt to locate the key directly as all inserted keys have a valid asid. Use g_hash_table_foreach_remove instead. Signed-off-by: Eric Auger --- hw/arm/smmu-common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/arm/smmu-common.c b/hw/arm

Re: [PATCH] plugins: Expose physical addresses instead of device offsets

2021-03-09 Thread Alex Bennée
Aaron Lindsay writes: > Alex, > > I've now tested this change, and it is giving what appear to be valid > and correct physical addresses for both RAM and IO accesses in all the > cases I've thrown at it. My main concern with this patch at this point > is that I am concerned I may be breaking yo

Re: [PATCH] coroutine: add libucontext as external library

2021-03-09 Thread Daniel P . Berrangé
On Tue, Mar 09, 2021 at 11:24:08AM +0100, Paolo Bonzini wrote: > On 09/03/21 10:59, Joelle van Dyne wrote: > > Does it make a difference if this is provided as an option and not as > > a replacement? Would it make sense to add some warning at configure > > time? Right now none of the concurrency ba

[PATCH v3 7/7] hw/arm/smmuv3: Uniformize sid traces

2021-03-09 Thread Eric Auger
Convert all sid printouts to sid=0x%x. Signed-off-by: Eric Auger Reviewed-by: Philippe Mathieu-Daudé --- v2 -> v3: - added Philippe's R-b --- hw/arm/trace-events | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/hw/arm/trace-events b/hw/arm/trace-e

Re: [PATCH 0/9] hw/display/pl110, pxa2xx_lcd: Tidy up template headers

2021-03-09 Thread Gerd Hoffmann
On Fri, Mar 05, 2021 at 02:48:11PM +, Peter Maydell wrote: > Ping for review? Looks all sane from ui point of view. Acked-by: Gerd Hoffmann I suspect we still have some code duplication here due to each display device having its own set of line render functions. We should be able to share

Re: [PATCH] coroutine: add libucontext as external library

2021-03-09 Thread Daniel P . Berrangé
On Tue, Mar 09, 2021 at 10:20:07AM +, Peter Maydell wrote: > On Tue, 9 Mar 2021 at 09:59, Joelle van Dyne wrote: > > > > On Tue, Mar 9, 2021 at 1:36 AM Daniel P. Berrangé > > wrote: > > > The README here: > > > > > > https://github.com/kaniini/libucontext > > > > > > indicates that it is i

Re: [PATCH] coroutine: add libucontext as external library

2021-03-09 Thread Marc-André Lureau
Hi On Tue, Mar 9, 2021 at 2:36 PM Daniel P. Berrangé wrote: > On Tue, Mar 09, 2021 at 11:24:08AM +0100, Paolo Bonzini wrote: > > On 09/03/21 10:59, Joelle van Dyne wrote: > > > Does it make a difference if this is provided as an option and not as > > > a replacement? Would it make sense to add s

Re: [RFC RESEND PATCH 0/4] hw/arm/virt-acpi-build: Introduce iommu option for pci root bus

2021-03-09 Thread Wang Xingang
Hi, everyone! Do you have any suggestions? Please help review these patches, thanks very much. On 2021/2/27 16:33, Wang Xingang wrote: From: Xingang Wang These patches add support for configure iommu on/off for pci root bus, including primary bus and pxb root bus. At present, All root bus wil

[PULL 04/14] docker: OpenSBI build job depends on OpenSBI container

2021-03-09 Thread Thomas Huth
From: Philippe Mathieu-Daudé Add missing dependency build-opensbi -> docker-opensbi. Signed-off-by: Philippe Mathieu-Daudé Message-Id: <20210303130646.1494015-4-phi...@redhat.com> Reviewed-by: Willian Rampazzo Reviewed-by: Daniel P. Berrangé Signed-off-by: Thomas Huth --- .gitlab-ci.d/opens

[PULL 02/14] docker: Alpine build job depends on Alpine container

2021-03-09 Thread Thomas Huth
From: Philippe Mathieu-Daudé Add missing dependency build-system-alpine -> amd64-alpine-container. Signed-off-by: Philippe Mathieu-Daudé Message-Id: <20210303130646.1494015-2-phi...@redhat.com> Reviewed-by: Willian Rampazzo Reviewed-by: Daniel P. Berrangé Signed-off-by: Thomas Huth --- .git

[PULL 01/14] qtest: delete superfluous inclusions of qtest.h

2021-03-09 Thread Thomas Huth
From: Chen Qun There are 23 files that include the "sysemu/qtest.h", but they do not use any qtest functions. Signed-off-by: Chen Qun Acked-by: Markus Armbruster Message-Id: <20210226081414.205946-1-kuhn.chen...@huawei.com> Signed-off-by: Thomas Huth --- accel/tcg/cpu-exec.c| 1 -

[PULL 03/14] docker: EDK2 build job depends on EDK2 container

2021-03-09 Thread Thomas Huth
From: Philippe Mathieu-Daudé Add missing dependency build-edk2 -> docker-edk2. Signed-off-by: Philippe Mathieu-Daudé Message-Id: <20210303130646.1494015-3-phi...@redhat.com> Reviewed-by: Willian Rampazzo Reviewed-by: Daniel P. Berrangé Acked-by: Laszlo Ersek Signed-off-by: Thomas Huth ---

[PULL 00/14] Gitlab, testing and misc patches

2021-03-09 Thread Thomas Huth
Hi Peter, the following changes since commit 229a834518b950d56fd1bc94923276504d0ee9d4: Merge remote-tracking branch 'remotes/philmd-gitlab/tags/renesas-20210306' into staging (2021-03-08 15:45:48 +) are available in the Git repository at: https://gitlab.com/thuth/qemu.git tags/pull-re

[PULL 06/14] scripts/ci/gitlab-pipeline-status: split utlity function for HTTP GET

2021-03-09 Thread Thomas Huth
From: Cleber Rosa This simply splits out the code that does an HTTP GET so that it can be used for other API requests. Signed-off-by: Cleber Rosa Reviewed-by: Alex Bennée Reviewed-by: Wainer dos Santos Moschetta Message-Id: <20210222193240.921250-2-cr...@redhat.com> Signed-off-by: Thomas Huth

[PULL 05/14] meson: Re-enable the possibility to run "make check SPEED=slow"

2021-03-09 Thread Thomas Huth
"make check SPEED=slow" got lost in the conversion of the build system to meson - the tests were always running in "quick" mode. Fix it by passing the "-m" parameter to the test harness at the right spot in scripts/mtest2make.py. Signed-off-by: Thomas Huth Message-Id: <20210218172313.2217440-1-th

[PULL 10/14] docs/devel/qgraph: improve qgraph documentation

2021-03-09 Thread Thomas Huth
From: Emanuele Giuseppe Esposito Improve current qgraph documentation with a more concrete example and clearer motivation. This patch depends on the previous serie "libqos/qgraph: format qgraph comments for sphinx documentation" Signed-off-by: Emanuele Giuseppe Esposito Message-Id: <2021030109

[PULL 08/14] scripts/ci/gitlab-pipeline-status: give more info when pipeline not found

2021-03-09 Thread Thomas Huth
From: Cleber Rosa This includes both input parameters (project id and commit) in the message so to make it easier to debug returned API calls. Signed-off-by: Cleber Rosa Reviewed-by: Wainer dos Santos Moschetta Message-Id: <20210222193240.921250-4-cr...@redhat.com> Signed-off-by: Thomas Huth

[PULL 07/14] scripts/ci/gitlab-pipeline-status: give more information on failures

2021-03-09 Thread Thomas Huth
From: Cleber Rosa When an HTTP GET request fails, it's useful to go beyond the "not successful" message, and show the code returned by the server. Signed-off-by: Cleber Rosa Reviewed-by: Alex Bennée Reviewed-by: Wainer dos Santos Moschetta Message-Id: <20210222193240.921250-3-cr...@redhat.com

[PULL 12/14] Acceptance Tests: restore filtering of tests by target arch

2021-03-09 Thread Thomas Huth
From: Cleber Rosa Previously, tests were being filtered by the matching target architectures to be built. The benefit, compared to the current situation, is a more concise test job that won't show tests canceled because a matching QEMU binary was not found (those tests won't even be attempted).

Re: [RFC PATCH v2] meson: Only generate trace files for selected targets

2021-03-09 Thread Claudio Fontana
On 3/9/21 11:26 AM, Paolo Bonzini wrote: > On 09/03/21 09:41, Claudio Fontana wrote: >> In any case, I agree that just recursing all directories in target/ >> looking for trace.h / traceevents files could be better. > > Using the directory structure to find source files is a bad idea because > yo

[PULL 11/14] Acceptance Tests: restore downloading of VM images

2021-03-09 Thread Thomas Huth
From: Cleber Rosa The "get-vm-images" target defined in tests/Makefile.include is a prerequisite for "check-acceptance", so that those files get downloaded before the Avocado job even starts. It looks like on c401c058a1c a TARGETS variable was introduced with a different content than it was prev

[PULL 14/14] bsd-user: Add new maintainers

2021-03-09 Thread Thomas Huth
From: Warner Losh The FreeBSD project has a number of enhancements to bsd-user. These changes have evolved over the past 10 year, and aren't currently updated to the latest version of qemu due to fluxuations in staffing causing us to fall behind in the past. We're working on porting all the chang

[PULL 09/14] libqos/qgraph: format qgraph comments for sphinx documentation

2021-03-09 Thread Thomas Huth
From: Emanuele Giuseppe Esposito Change documentation style and fix minor typos in tests/qtest/libqos/qgraph.h to automatically generate sphinx documentation in docs/devel/qgraph.rst The mechanism explanation that once was in qgraph.h is now moved to qgraph.rst There is no functional change int

Re: [RFC PATCH 3/4] coroutine/mutex: Store the coroutine in the CoWaitRecord only once

2021-03-09 Thread Paolo Bonzini
On 09/03/21 11:21, David Edmondson wrote: When taking the slow path for mutex acquisition, set the coroutine value in the CoWaitRecord in push_waiter(), rather than both there and in the caller. Signed-off-by: David Edmondson --- util/qemu-coroutine-lock.c | 1 - 1 file changed, 1 deletion(-

Re: [PATCH v2] block/file-posix: Optimize for macOS

2021-03-09 Thread Kevin Wolf
Am 09.03.2021 um 11:26 hat Daniel P. Berrangé geschrieben: > On Tue, Mar 09, 2021 at 10:37:18AM +0100, Kevin Wolf wrote: > > Am 09.03.2021 um 05:52 hat Akihiko Odaki geschrieben: > > > 2021年3月9日(火) 0:37 Akihiko Odaki : > > > > > > > > 2021年3月9日(火) 0:17 Stefan Hajnoczi : > > > > > > > > > > The live

Re: [RFC PATCH v2] meson: Only generate trace files for selected targets

2021-03-09 Thread Philippe Mathieu-Daudé
On 3/9/21 11:26 AM, Paolo Bonzini wrote: > On 09/03/21 09:41, Claudio Fontana wrote: >> In any case, I agree that just recursing all directories in target/ >> looking for trace.h / traceevents files could be better. > > Using the directory structure to find source files is a bad idea because > you

Re: [PATCH] coroutine: add libucontext as external library

2021-03-09 Thread Paolo Bonzini
On 09/03/21 11:42, Marc-André Lureau wrote: If I remember correctly, there were objections because we wanted to have an implementation close to QEMU, so we could easily extend it, or add custom optimizations. I think it's quite mature now. The code that needs to stay close to QEMU is the loc

Re: [PATCH] block: remove format defaults from QemuOpts in bdrv_create_file()

2021-03-09 Thread Kevin Wolf
Am 08.03.2021 um 17:12 hat Stefano Garzarella geschrieben: > QemuOpts is usually created merging the QemuOptsList of format > and protocol. So, when the format calls bdr_create_file(), the 'opts' > parameter contains a QemuOptsList with a combination of format and > protocol default values. > > Th

Re: [PATCH v9 6/6] KVM: arm64: Document MTE capability and ioctl

2021-03-09 Thread Peter Maydell
On Mon, 1 Mar 2021 at 14:23, Steven Price wrote: > > A new capability (KVM_CAP_ARM_MTE) identifies that the kernel supports > granting a guest access to the tags, and provides a mechanism for the > VMM to enable it. > > A new ioctl (KVM_ARM_MTE_COPY_TAGS) provides a simple way for a VMM to > acces

Re: [PATCH v3] bsd-user: Add new maintainers

2021-03-09 Thread Philippe Mathieu-Daudé
On 3/8/21 11:40 PM, Warner Losh wrote: > The FreeBSD project has a number of enhancements to bsd-user. These changes > have > evolved over the past 10 year, and aren't currently updated to the latest > version of qemu due to fluxuations in staffing causing us to fall behind in > the > past. We're

Re: [PATCH v2] block/file-posix: Optimize for macOS

2021-03-09 Thread Akihiko Odaki
2021年3月9日(火) 19:26 Daniel P. Berrangé : > > On Tue, Mar 09, 2021 at 10:37:18AM +0100, Kevin Wolf wrote: > > Am 09.03.2021 um 05:52 hat Akihiko Odaki geschrieben: > > > 2021年3月9日(火) 0:37 Akihiko Odaki : > > > > > > > > 2021年3月9日(火) 0:17 Stefan Hajnoczi : > > > > > > > > > > The live migration compat

Re: [PULL 00/14] Gitlab, testing and misc patches

2021-03-09 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20210309104617.714908-1-th...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20210309104617.714908-1-th...@redhat.com Subject: [PULL 00/14] Gitlab, testing and misc patc

Re: [PATCH 1/4] hw/i386/pc: Get pflash MemoryRegion with sysbus_mmio_get_region()

2021-03-09 Thread Philippe Mathieu-Daudé
Michael, Paolo, if you Ack this patch I can queue the series via pflash-next. On 3/7/21 11:26 PM, Philippe Mathieu-Daudé wrote: > TYPE_PFLASH_CFI01 is a TYPE_SYS_BUS_DEVICE which registers its romd > MemoryRegion with sysbus_init_mmio(), so we can use the generic > sysbus_mmio_get_region() to get

Re: [RFC PATCH 4/4] coroutine/rwlock: Wake writers in preference to readers

2021-03-09 Thread Paolo Bonzini
On 09/03/21 11:21, David Edmondson wrote: A feature of the current rwlock is that if multiple coroutines hold a reader lock, all must be runnable. The unlock implementation relies on this, choosing to wake a single coroutine when the final read lock holder exits the critical section, assuming tha

Re: [RFC PATCH 4/4] coroutine/rwlock: Wake writers in preference to readers

2021-03-09 Thread Paolo Bonzini
On 09/03/21 11:21, David Edmondson wrote: -/* The critical section started in qemu_co_rwlock_wrlock. */ -qemu_co_queue_restart_all(&lock->queue); +/* The critical section started in qemu_co_rwlock_wrlock or + * qemu_co_rwlock_upgrade. + */ +qemu_co

Re: [RFC PATCH 3/4] coroutine/mutex: Store the coroutine in the CoWaitRecord only once

2021-03-09 Thread Philippe Mathieu-Daudé
On 3/9/21 11:21 AM, David Edmondson wrote: > When taking the slow path for mutex acquisition, set the coroutine > value in the CoWaitRecord in push_waiter(), rather than both there and > in the caller. > > Signed-off-by: David Edmondson > --- > util/qemu-coroutine-lock.c | 1 - > 1 file changed,

Re: [PATCH] ui/cocoa: Do not rely on the first argument

2021-03-09 Thread Peter Maydell
On Tue, 23 Feb 2021 at 11:06, Akihiko Odaki wrote: > > The first argument of the executable was used to get its path, but it is > not reliable because the executer can specify any arbitrary string. Use the > interfaces provided by QEMU and the platform to get those paths. > > This change also chan

Re: [RFC PATCH 1/4] block/vdi: When writing new bmap entry fails, don't leak the buffer

2021-03-09 Thread Philippe Mathieu-Daudé
On 3/9/21 11:21 AM, David Edmondson wrote: > If a new bitmap entry is allocated, requiring the entire block to be > written, avoiding leaking the buffer allocated for the block should > the write fail. > > Signed-off-by: David Edmondson Reviewed-by: Philippe Mathieu-Daudé > --- > block/vdi.c

[PATCH V2 trivial] Various spelling fixes

2021-03-09 Thread Michael Tokarev
An assorted set of spelling fixes in various places. Signed-off-by: Michael Tokarev Reviewed-by: Stefan Weil --- V1: https://lists.nongnu.org/archive/html/qemu-devel/2020-12/msg05959.html Changes: incorporated suggestions by Stefan Weil and added his R-b. Please note: this also patches one file

Re: [PATCH V2] virtio-net: calculating proper msix vectors on init

2021-03-09 Thread Stefano Garzarella
On Tue, Mar 09, 2021 at 12:23:14PM +0800, Jason Wang wrote: Currently, the default msix vectors for virtio-net-pci is 3 which is obvious not suitable for multiqueue guest, so we depends on the user or management tools to pass a correct vectors parameter. In fact, we can simplifying this by calcul

Re: [PULL 00/27] tcg patch queue

2021-03-09 Thread Peter Maydell
On Sat, 6 Mar 2021 at 21:36, Richard Henderson wrote: > > The following changes since commit 91e92cad67caca3bc4b8e920ddb5c8ca64aac9e1: > > Merge remote-tracking branch 'remotes/cohuck-gitlab/tags/s390x-20210305' > into staging (2021-03-05 19:04:47 +) > > are available in the Git repository

[PATCH v3] MAINTAINERS: Merge the Gitlab-CI section into the generic CI section

2021-03-09 Thread Thomas Huth
The status of the gitlab-CI files is currently somewhat confusing, and it is often not quite clear whether a patch should go via my tree or via the testing tree of Alex. That situation has grown historically... Initially, I was the only one using the gitlab-CI, just for my private repository there.

Re: [PATCH] ui/cocoa: Clear modifiers whenever possible

2021-03-09 Thread Gerd Hoffmann
On Fri, Mar 05, 2021 at 09:19:09PM +0900, Akihiko Odaki wrote: > ui/cocoa does not receive NSEventTypeFlagsChanged when it is not active, > and the modifier state can be desynchronized in such a situation. > > [NSEvent -modifierFlags] tells whether a modifier is *not* pressed, so > check it whenev

  1   2   3   4   5   6   7   >