Re: vnc clipboard support

2021-01-29 Thread Gerd Hoffmann
Hi, > > (1) Have some guest agent (spice does it that way). > > Advantage: more flexible, allows more features. > > Disadvantage: requires agent in the guest. > > What about running the option to relay data to a VNC server in the > guest if there is one running? In other words, using

Re: [PATCH v4 0/4] meson: Try to clarify TCG / TCI options for new users

2021-01-29 Thread Paolo Bonzini
On 25/01/21 15:45, Philippe Mathieu-Daudé wrote: Since v3: - Rebased - Include fix for 23a77b2d18b ("build-system: clean up TCG/TCI configury") - Use get_option() (Claudio) - Use warning message suggested by Daniel - Drop 'Reword --enable-tcg-interpreter as --disable-native-tcg' (Paolo) Some new

Re: vnc clipboard support

2021-01-29 Thread Gerd Hoffmann
Hi, > Just a random thought: the guest should not be able to sniff, steal, or > overwrite host-side clipboard (selection) content, without the host user > explicitly requesting a clipboard operation. Sure. vncviewer has command line options for that, and you can also change it at runtime (F8 -

[PATCH RFC 0/1] QOM type names and QAPI

2021-01-29 Thread Markus Armbruster
QAPI has naming rules. docs/devel/qapi-code-gen.txt: === Naming rules and reserved names === All names must begin with a letter, and contain only ASCII letters, digits, hyphen, and underscore. There are two exceptions: enum values may start with a digit, and names that are downs

[PATCH RFC 1/1] hw: Replace anti-social QOM type names

2021-01-29 Thread Markus Armbruster
Several QOM type names contain ',': ARM,bitband-memory etraxfs,pic etraxfs,serial etraxfs,timer fsl,imx25 fsl,imx31 fsl,imx6 fsl,imx6ul fsl,imx7 grlib,ahbpnp grlib,apbpnp grlib,apbuart grlib,gptimer grlib,irqmp qemu,register SUNW,bpp

Re: [PATCH 0/3] block/nbd: fix crashers in reconnect while migrating

2021-01-29 Thread Vladimir Sementsov-Ogievskiy
29.01.2021 10:35, Roman Kagan wrote: On Fri, Jan 29, 2021 at 08:51:39AM +0300, Vladimir Sementsov-Ogievskiy wrote: 28.01.2021 23:14, Roman Kagan wrote: During the final phase of migration the NBD reconnection logic may encounter situations it doesn't expect during regular operation. This serie

Re: [PATCH v2 00/12] buildsys: Do not build various objects if not necessary

2021-01-29 Thread Paolo Bonzini
On 22/01/21 21:44, Philippe Mathieu-Daudé wrote: In this series we deselect a bunch of features when they not required, so less objects are built. While this reduce pressure on CI and slow systems, this is particularly helpful for developers regularly testing multiple build configurations. All

Re: [PULL 14/53] block: apply COR-filter to block-stream jobs

2021-01-29 Thread Max Reitz
On 29.01.21 06:26, Vladimir Sementsov-Ogievskiy wrote: 28.01.2021 21:38, Philippe Mathieu-Daudé wrote: Hi Andrey, On 1/26/21 3:19 PM, Max Reitz wrote: From: Andrey Shinkevich This patch completes the series with the COR-filter applied to block-stream operations. Adding the filter makes it p

Re: [PATCH] tests/Makefile.include: export PYTHON for check-block.sh

2021-01-29 Thread Thomas Huth
On 29/01/2021 06.13, Vladimir Sementsov-Ogievskiy wrote: check-block.sh called by make check-block rely on PYTHON variable being set. Fixes: f203080bbd9f9e5b31041b1f2afcd6040c5aaec5 Signed-off-by: Vladimir Sementsov-Ogievskiy --- Hi! As Peter reported, build fails on platforms where python3 is

Re: vnc clipboard support

2021-01-29 Thread Gerd Hoffmann
Hi, > Some things I keep in mind: > - the spice protocol had a number of iterations to fix some races. It would > be great not to repeat the same mistakes, and I don't know if VNC have the > same flaws or not. Sure. One of the reasons I've started this discussion ;) > - the GNOME desktop is w

Re: [PATCH v13 4/5] migration: implementation of background snapshot thread

2021-01-29 Thread Andrey Gruzdev
On 28.01.2021 21:29, Dr. David Alan Gilbert wrote: * Andrey Gruzdev (andrey.gruz...@virtuozzo.com) wrote: Introducing implementation of 'background' snapshot thread which in overall follows the logic of precopy migration while internally utilizes completely different mechanism to 'freeze' vmstat

[PULL 0/4] Trivial branch for 6.0 patches

2021-01-29 Thread Laurent Vivier
ial-branch-for-6.0-pull-request for you to fetch changes up to 77182df19f874adfc4ac98248606c154764baaa8: target/rx: Fix compiler errors for build with sanitizers (2021-01-29 09:31:= 53 +0100) Trivial patche

[PULL 1/4] MAINTAINERS: Update 9pfs tree URL

2021-01-29 Thread Laurent Vivier
From: Greg Kurz I've already moved my repositories to gitlab for extra CI coverage, and I won't use the ones at github anymore. Signed-off-by: Greg Kurz Reviewed-by: Christian Schoenebeck Message-Id: <161071814430.152031.14540382419012818908.st...@bahia.lan> Signed-off-by: Laurent Vivier ---

[PULL 2/4] tcg/tci: Restrict tci_write_reg16() to 64-bit hosts

2021-01-29 Thread Laurent Vivier
From: Philippe Mathieu-Daudé Restrict tci_write_reg16() to 64-bit hosts to fix on 32-bit ones: [520/1115] Compiling C object libqemu-arm-linux-user.fa.p/tcg_tci.c.o FAILED: libqemu-arm-linux-user.fa.p/tcg_tci.c.o tcg/tci.c:132:1: error: 'tci_write_reg16' defined but not used [-Werror=unus

Re: vnc clipboard support

2021-01-29 Thread Marc-André Lureau
Hi On Fri, Jan 29, 2021 at 12:28 PM Gerd Hoffmann wrote: > > Well. That is a completely different level of desktop integration. > It surely makes sense to have that, and dbus-over-vsock looks like > a reasonable choice. > (using vsock & dbus also goes in the direction where UIs and remote serv

[PULL 3/4] net/slirp.c: Fix spelling error in error message

2021-01-29 Thread Laurent Vivier
From: Doug Evans DNS should be DHCP Signed-off-by: Doug Evans Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Samuel Thibault Message-Id: <20210122004251.843837-1-...@google.com> Signed-off-by: Laurent Vivier --- net/slirp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PULL 4/4] target/rx: Fix compiler errors for build with sanitizers

2021-01-29 Thread Laurent Vivier
From: Stefan Weil gcc (Debian 10.2.1-6) 10.2.1 20210110 aborts builds with enabled sanitizers: ../../../target/rx/op_helper.c: In function ‘helper_scmpu’: ../../../target/rx/op_helper.c:213:24: error: ‘tmp1’ may be used uninitialized in this function [-Werror=maybe-uninitialized] 213 | en

Re: [PATCH 4/7] ppc/pnv: Simplify pnv_bmc_create()

2021-01-29 Thread Cédric Le Goater
On 1/28/21 11:40 PM, David Gibson wrote: > On Thu, Jan 28, 2021 at 08:46:01AM +0100, Cédric Le Goater wrote: >> On 1/28/21 1:46 AM, Joel Stanley wrote: >>> On Tue, 26 Jan 2021 at 17:14, Cédric Le Goater wrote: and reuse pnv_bmc_set_pnor() to share the setting of the PNOR. Signe

Re: macOS (Big Sur, Apple Silicon) 'make check' fails in test-crypto-tlscredsx509

2021-01-29 Thread Roman Bolshakov
On Wed, Jan 27, 2021 at 06:59:17PM +, Daniel P. Berrangé wrote: > On Wed, Jan 27, 2021 at 07:56:16PM +0100, Stefan Weil wrote: > > Am 27.01.21 um 19:17 schrieb Daniel P. Berrangé: > > > > > On Wed, Jan 27, 2021 at 06:05:08PM +0100, Stefan Weil wrote: > > > > Am 27.01.21 um 17:53 schrieb Daniel

[PATCH] accel/kvm/kvm-all: Fix wrong return code handling in dirty log code

2021-01-29 Thread Thomas Huth
The kvm_vm_ioctl() wrapper already returns -errno if the ioctl itself returned -1, so the callers of kvm_vm_ioctl() should not check for -1 but for a value < 0 instead. This problem has been fixed once already in commit b533f658a98325d0e4 but that commit missed that the ENOENT error code is not fa

libtasn1 test suite fails on macOS Bug Sur with Apple Silicon

2021-01-29 Thread Roman Bolshakov
Hello, I'm seeing the test failures on macOS and there's a concern that the behaviour might affect GnuTLS and ultimately QEMU test suite. Are there any ideas of what might be causing it? Any help is appreciated. Thanks, Roman === GNU Libtasn1 4.16.

[PATCH] hw/sd: sd: Bypass the RCA check for CMD13 in SPI mode

2021-01-29 Thread Bin Meng
From: Bin Meng Unlike SD mode, when SD card is working in SPI mode, the argument of CMD13 is stuff bits. Hence we should bypass the RCA check. Signed-off-by: Bin Meng --- Based-on: http://patchwork.ozlabs.org/project/qemu-devel/list/?series=226787 hw/sd/sd.c | 3 ++- 1 file changed, 2 insert

Re: [PATCH] tests/Makefile.include: export PYTHON for check-block.sh

2021-01-29 Thread Vladimir Sementsov-Ogievskiy
29.01.2021 11:25, Thomas Huth wrote: On 29/01/2021 06.13, Vladimir Sementsov-Ogievskiy wrote: check-block.sh called by make check-block rely on PYTHON variable being set. Fixes: f203080bbd9f9e5b31041b1f2afcd6040c5aaec5 Signed-off-by: Vladimir Sementsov-Ogievskiy --- Hi! As Peter reported, bui

[RESEND] [PATCH v5] qga: Correct loop count in qmp_guest_get_vcpus()

2021-01-29 Thread lma--- via
The guest-get-vcpus returns incorrect vcpu info in case we hotunplug vcpus(not the last one). e.g.: A VM has 4 VCPUs: cpu0 + 3 hotunpluggable online vcpus(cpu1, cpu2 and cpu3). Hotunplug cpu2, Now only cpu0, cpu1 and cpu3 are present & online. ./qmp-shell /tmp/qmp-monitor.sock (QEMU) query-hotplu

Re: [PATCH v14 15/22] cpu: tcg_ops: move to tcg-cpu-ops.h, keep a pointer in CPUClass

2021-01-29 Thread Claudio Fontana
On 1/29/21 1:31 AM, Richard Henderson wrote: > On 1/27/21 11:28 PM, Claudio Fontana wrote: >> +/* >> + * NB: this should be covered by CONFIG_TCG, but it is unsafe to do it >> here, >> + * as this header is included by both ss_specific and ss_common code, >> + * leading to potentia

Re: [PATCH RFC 1/1] hw: Replace anti-social QOM type names

2021-01-29 Thread Paolo Bonzini
On 29/01/21 09:15, Markus Armbruster wrote: diff --git a/hw/block/fdc.c b/hw/block/fdc.c index 7fc547c62d..0dd158bf73 100644 --- a/hw/block/fdc.c +++ b/hw/block/fdc.c @@ -2547,7 +2547,7 @@ void sun4m_fdctrl_init(qemu_irq irq, hwaddr io_base, DeviceState *dev; FDCtrlSysBus *sys; -d

[PATCH] iotests/testrunner: fix recognition of python tests

2021-01-29 Thread Vladimir Sementsov-Ogievskiy
We should drop final '\n' before comparing with python3 shebang. Fixes: d74c754c924ca34e90b7c96ce2f5609d82c0e628 Signed-off-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/testrunner.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/qemu-iotests/testrunner.py b

[PATCH] vhost: Check for valid vdev in vhost_backend_handle_iotlb_msg

2021-01-29 Thread Eugenio Pérez
Not checking this can lead to invalid dev->vdev member access in vhost_device_iotlb_miss if backend issue an iotlb message in a bad timing, either maliciously or by a bug. Reproduced rebooting a guest with testpmd in txonly forward mode. #0 0x55994394 in vhost_device_iotlb_miss ( de

Re: [PATCH v7 07/13] confidential guest support: Introduce cgs "ready" flag

2021-01-29 Thread David Gibson
On Mon, Jan 18, 2021 at 07:47:30PM +, Dr. David Alan Gilbert wrote: > * David Gibson (da...@gibson.dropbear.id.au) wrote: > > The platform specific details of mechanisms for implementing > > confidential guest support may require setup at various points during > > initialization. Thus, it's no

Re: [PATCH v7 10/13] spapr: Add PEF based confidential guest support

2021-01-29 Thread David Gibson
On Thu, Jan 14, 2021 at 11:39:11AM +0100, Greg Kurz wrote: > On Thu, 14 Jan 2021 10:58:08 +1100 > David Gibson wrote: > > > Some upcoming POWER machines have a system called PEF (Protected > > Execution Facility) which uses a small ultravisor to allow guests to > > run in a way that they can't be

[Bug 1294227] Re: migration wrong handling of KVM_GET_DIRTY_LOG ioctl

2021-01-29 Thread Thomas Huth
Yet another try to fix this issue: https://lists.gnu.org/archive/html/qemu-devel/2021-01/msg07557.html -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1294227 Title: migration wrong handling of KVM_G

Re: [PATCH] vhost: Check for valid vdev in vhost_backend_handle_iotlb_msg

2021-01-29 Thread Eugenio Perez Martin
Version 2 of ("vhost-user: Check for iotlb callback in iotlb_miss") [1]. Starting a new series, since the title does not reflect the changes anymore. Thanks! [1] https://patchew.org/QEMU/20210127204449.745365-1-epere...@redhat.com/ On Fri, Jan 29, 2021 at 10:08 AM Eugenio Pérez wrote: > > Not

Re: [PATCH] vhost-user: Check for iotlb callback in iotlb_miss

2021-01-29 Thread Eugenio Perez Martin
On Fri, Jan 29, 2021 at 8:29 AM Jason Wang wrote: > > > On 2021/1/29 下午3:22, Eugenio Perez Martin wrote: > > On Fri, Jan 29, 2021 at 4:29 AM Jason Wang wrote: > >> > >> On 2021/1/28 下午5:37, Eugenio Perez Martin wrote: > >>> Hi Jason. > >>> > >>> On Thu, Jan 28, 2021 at 3:32 AM Jason Wang wrote:

[PATCH v5 3/5] hw/block/nvme: pull write pointer advancement to separate function

2021-01-29 Thread Klaus Jensen
From: Klaus Jensen In preparation for Simple Copy, pull write pointer advancement into a separate function that is independent on an NvmeRequest. Signed-off-by: Klaus Jensen --- hw/block/nvme.c | 16 +++- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/hw/block/nvme.

[PATCH v5 0/5] hw/block/nvme: add simple copy command

2021-01-29 Thread Klaus Jensen
From: Klaus Jensen Add support for TP 4065 ("Simple Copy Command"). Changes for v5 * rebased on nvme-next (support for zoned namespaces) Changes for v4 * merge for-loops (Keith) Changes for v3 * rebased on nvme-next * changed the default msrc value to a more reasonable 127 from 255

[PATCH v5 1/5] hw/block/nvme: remove unused parameter in check zone write

2021-01-29 Thread Klaus Jensen
From: Klaus Jensen Remove the unused NvmeCtrl parameter in nvme_check_zone_write. Signed-off-by: Klaus Jensen --- hw/block/nvme.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/hw/block/nvme.c b/hw/block/nvme.c index 8ac997735041..79d9563a17bd 100644 --- a/hw/block/

[PATCH v5 4/5] nvme: updated shared header for copy command

2021-01-29 Thread Klaus Jensen
From: Klaus Jensen Add new data structures and types for the Simple Copy command. Signed-off-by: Klaus Jensen Reviewed-by: Minwoo Im Acked-by: Stefan Hajnoczi --- include/block/nvme.h | 45 ++-- 1 file changed, 43 insertions(+), 2 deletions(-) diff --

[PATCH v5 2/5] hw/block/nvme: refactor zone resource management

2021-01-29 Thread Klaus Jensen
From: Klaus Jensen Zone transition handling and resource management is open coded (and semi-duplicated in the case of open, close and finish). In preparation for Simple Copy command support (which also needs to open zones for writing), consolidate into a set of 'nvme_zrm' functions and in the pr

Re: [PATCH RFC 0/1] QOM type names and QAPI

2021-01-29 Thread Markus Armbruster
Forgot to mention: Based-on: <20210125162402.1807394-1-arm...@redhat.com> [PATCH 0/3] Drop deprecated floppy config & bogus -drive if=T

Re: [PATCH RFC 1/1] hw: Replace anti-social QOM type names

2021-01-29 Thread Markus Armbruster
Paolo Bonzini writes: > On 29/01/21 09:15, Markus Armbruster wrote: >> diff --git a/hw/block/fdc.c b/hw/block/fdc.c >> index 7fc547c62d..0dd158bf73 100644 >> --- a/hw/block/fdc.c >> +++ b/hw/block/fdc.c >> @@ -2547,7 +2547,7 @@ void sun4m_fdctrl_init(qemu_irq irq, hwaddr io_base, >> DeviceSt

[PATCH v5 5/5] hw/block/nvme: add simple copy command

2021-01-29 Thread Klaus Jensen
From: Klaus Jensen Add support for TP 4065a ("Simple Copy Command"), v2020.05.04 ("Ratified"). The implementation uses a bounce buffer to first read in the source logical blocks, then issue a write of that bounce buffer. The default maximum number of source logical blocks is 128, translating to

Re: [PATCH] accel/kvm/kvm-all: Fix wrong return code handling in dirty log code

2021-01-29 Thread Philippe Mathieu-Daudé
On 1/29/21 9:43 AM, Thomas Huth wrote: > The kvm_vm_ioctl() wrapper already returns -errno if the ioctl itself > returned -1, so the callers of kvm_vm_ioctl() should not check for -1 > but for a value < 0 instead. > > This problem has been fixed once already in commit b533f658a98325d0e4 > but that

Re: [PATCH 3/7] ppc/pnv: Use skiboot addresses to load kernel and ramfs

2021-01-29 Thread Cédric Le Goater
On 1/28/21 11:36 PM, David Gibson wrote: > On Thu, Jan 28, 2021 at 08:02:41AM +0100, Cédric Le Goater wrote: >> On 1/28/21 1:45 AM, Joel Stanley wrote: >>> On Tue, 26 Jan 2021 at 17:11, Cédric Le Goater wrote: The current settings are useful to load large kernels (with debug) but it

[PATCH] pnv/ppc: Set default RAM size to 1 GB

2021-01-29 Thread Cédric Le Goater
Any value below will result in a skiboot crash : [0.034949905,3] MEM: Partial overlap detected between regions: [0.034959039,3] MEM: ibm,firmware-stacks [0x31c1-0x3a45] (new) [0.034968576,3] MEM: ibm,firmware-allocs-memory@0 [0x31c1-0x3840] [0.03498

Re: [PULL 14/53] block: apply COR-filter to block-stream jobs

2021-01-29 Thread Philippe Mathieu-Daudé
On 1/29/21 9:23 AM, Max Reitz wrote: > On 29.01.21 06:26, Vladimir Sementsov-Ogievskiy wrote: >> 28.01.2021 21:38, Philippe Mathieu-Daudé wrote: >>> Hi Andrey, >>> >>> On 1/26/21 3:19 PM, Max Reitz wrote: From: Andrey Shinkevich This patch completes the series with the COR-filter ap

Re: [PATCH] accel/kvm/kvm-all: Fix wrong return code handling in dirty log code

2021-01-29 Thread Paolo Bonzini
On 29/01/21 09:43, Thomas Huth wrote: The kvm_vm_ioctl() wrapper already returns -errno if the ioctl itself returned -1, so the callers of kvm_vm_ioctl() should not check for -1 but for a value < 0 instead. This problem has been fixed once already in commit b533f658a98325d0e4 but that commit mis

Re: [PATCH] accel/kvm/kvm-all: Fix wrong return code handling in dirty log code

2021-01-29 Thread Thomas Huth
On 29/01/2021 10.28, Philippe Mathieu-Daudé wrote: On 1/29/21 9:43 AM, Thomas Huth wrote: The kvm_vm_ioctl() wrapper already returns -errno if the ioctl itself returned -1, so the callers of kvm_vm_ioctl() should not check for -1 but for a value < 0 instead. This problem has been fixed once alr

Re: [PATCH] pnv/ppc: Set default RAM size to 1 GB

2021-01-29 Thread Philippe Mathieu-Daudé
On 1/29/21 10:29 AM, Cédric Le Goater wrote: > Any value below will result in a skiboot crash : > > [0.034949905,3] MEM: Partial overlap detected between regions: > [0.034959039,3] MEM: ibm,firmware-stacks [0x31c1-0x3a45] (new) > [0.034968576,3] MEM: ibm,firmware-al

Re: [PATCH 3/6] vhost-user: Return error code from slave_read()

2021-01-29 Thread Greg Kurz
On Mon, 25 Jan 2021 13:01:12 -0500 Vivek Goyal wrote: > Right now slave_read() is called through main event loop and does not > return error. In next few patches I want to call slave_read() from > vhost device shutdown path as well and want to know if an error > happened so that caller can give u

Re: [PATCH] tests/Makefile.include: export PYTHON for check-block.sh

2021-01-29 Thread Thomas Huth
On 29/01/2021 06.13, Vladimir Sementsov-Ogievskiy wrote: check-block.sh called by make check-block rely on PYTHON variable being set. Fixes: f203080bbd9f9e5b31041b1f2afcd6040c5aaec5 Signed-off-by: Vladimir Sementsov-Ogievskiy --- Hi! As Peter reported, build fails on platforms where python3 is

Re: [PATCH] iotests/testrunner: fix recognition of python tests

2021-01-29 Thread Thomas Huth
On 29/01/2021 10.06, Vladimir Sementsov-Ogievskiy wrote: We should drop final '\n' before comparing with python3 shebang. Fixes: d74c754c924ca34e90b7c96ce2f5609d82c0e628 Signed-off-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/testrunner.py | 2 +- 1 file changed, 1 insertion(+), 1

Re: [PATCH] iotests/297: pylint: ignore too many statements

2021-01-29 Thread Kevin Wolf
Am 28.01.2021 um 21:08 hat Vladimir Sementsov-Ogievskiy geschrieben: > 28.01.2021 23:04, Vladimir Sementsov-Ogievskiy wrote: > > Ignore two complains, which now lead to 297 failure on testenv.py and > > testrunner.py. > > > > Fixes: 2e5a2f57db481f18fcf70be2a36b1417370b8476 > > Fixes: d74c754c924ca

Re: macOS (Big Sur, Apple Silicon) 'make check' fails in test-crypto-tlscredsx509

2021-01-29 Thread Daniel P . Berrangé
On Fri, Jan 29, 2021 at 11:43:32AM +0300, Roman Bolshakov wrote: > On Wed, Jan 27, 2021 at 06:59:17PM +, Daniel P. Berrangé wrote: > > On Wed, Jan 27, 2021 at 07:56:16PM +0100, Stefan Weil wrote: > > > Am 27.01.21 um 19:17 schrieb Daniel P. Berrangé: > > > > > > > On Wed, Jan 27, 2021 at 06:05

Re: [PATCH] pnv/ppc: Set default RAM size to 1 GB

2021-01-29 Thread Greg Kurz
On Fri, 29 Jan 2021 10:29:36 +0100 Cédric Le Goater wrote: > Any value below will result in a skiboot crash : > > [0.034949905,3] MEM: Partial overlap detected between regions: > [0.034959039,3] MEM: ibm,firmware-stacks [0x31c1-0x3a45] (new) > [0.034968576,3] MEM:

Re: vnc clipboard support

2021-01-29 Thread Daniel P . Berrangé
On Thu, Jan 28, 2021 at 06:57:24PM +0100, Laszlo Ersek wrote: > On 01/28/21 18:12, Gerd Hoffmann wrote: > > Hi folks, > > > > I'm looking for a good way to implement cut+paste support for vnc. > > > > The vnc core protocol has support for text/plain cut+paste, and there > > is an extension addi

Re: [PATCH] tests: Replace deprecated ASN1 code

2021-01-29 Thread Daniel P . Berrangé
On Thu, Jan 28, 2021 at 06:15:23PM +0100, Stefan Weil wrote: > This fixes several compiler warnings on MacOS with Homebrew. Could you give an example of the warnings seen as I'm wondering why we won't have seen them already on other platforms ? > > Signed-off-by: Stefan Weil > --- > tests/cry

Re: [PULL 0/5] QAPI patches patches for 2021-01-28

2021-01-29 Thread Peter Maydell
On Thu, 28 Jan 2021 at 07:45, Markus Armbruster wrote: > > The following changes since commit bf159f0bdc7b8e7aa8342dedb3829ca744c1b612: > > Merge remote-tracking branch > 'remotes/edgar/tags/edgar/xilinx-next-2021-01-27.for-upstream' into staging > (2021-01-27 17:40:25 +) > > are available

Re: [PATCH] pnv/ppc: Set default RAM size to 1 GB

2021-01-29 Thread Greg Kurz
On Fri, 29 Jan 2021 10:39:12 +0100 Philippe Mathieu-Daudé wrote: > On 1/29/21 10:29 AM, Cédric Le Goater wrote: > > Any value below will result in a skiboot crash : > > > > [0.034949905,3] MEM: Partial overlap detected between regions: > > [0.034959039,3] MEM: ibm,firmware-stacks

[PATCH v14 0/5] UFFD write-tracking migration/snapshots

2021-01-29 Thread andrey . gruzdev--- via
This patch series is a kind of 'rethinking' of Denis Plotnikov's ideas he's implemented in his series '[PATCH v0 0/4] migration: add background snapshot'. Currently the only way to make (external) live VM snapshot is using existing dirty page logging migration mechanism. The main problem is that i

[PATCH v14 1/5] migration: introduce 'background-snapshot' migration capability

2021-01-29 Thread andrey . gruzdev--- via
Add new capability to 'qapi/migration.json' schema. Update migrate_caps_check() to validate enabled capability set against introduced one. Perform checks for required kernel features and compatibility with guest memory backends. Signed-off-by: Andrey Gruzdev Reviewed-by: Peter Xu Acked-by: Marku

[PATCH v14 3/5] migration: support UFFD write fault processing in ram_save_iterate()

2021-01-29 Thread andrey . gruzdev--- via
In this particular implementation the same single migration thread is responsible for both normal linear dirty page migration and procesing UFFD page fault events. Processing write faults includes reading UFFD file descriptor, finding respective RAM block and saving faulting page to the migration

[PATCH v14 4/5] migration: implementation of background snapshot thread

2021-01-29 Thread andrey . gruzdev--- via
Introducing implementation of 'background' snapshot thread which in overall follows the logic of precopy migration while internally utilizes completely different mechanism to 'freeze' vmstate at the start of snapshot creation. This mechanism is based on userfault_fd with wr-protection support and

[PATCH v14 2/5] migration: introduce UFFD-WP low-level interface helpers

2021-01-29 Thread andrey . gruzdev--- via
Glue code to the userfaultfd kernel implementation. Querying feature support, createing file descriptor, feature control, memory region registration, IOCTLs on registered registered regions. Signed-off-by: Andrey Gruzdev Reviewed-by: Peter Xu --- include/exec/memory.h | 1 + include/qemu

[PATCH v14 5/5] migration: introduce 'userfaultfd-wrlat.py' script

2021-01-29 Thread andrey . gruzdev--- via
Add BCC/eBPF script to analyze userfaultfd write fault latency distribution. Signed-off-by: Andrey Gruzdev Reviewed-by: Peter Xu --- scripts/userfaultfd-wrlat.py | 122 +++ 1 file changed, 122 insertions(+) create mode 100755 scripts/userfaultfd-wrlat.py diff -

Re: [PATCH] vfio/migrate: Move switch of dirty tracking into vfio_memory_listener

2021-01-29 Thread Keqian Zhu
On 2021/1/29 15:49, Paolo Bonzini wrote: > On 28/01/21 21:02, Dr. David Alan Gilbert wrote: >> * Paolo Bonzini (pbonz...@redhat.com) wrote: >>> On 11/01/21 08:34, Keqian Zhu wrote: +static void vfio_listener_log_start(MemoryListener *listener, +Memor

Re: [Bug 1913505] Re: Windows XP slow on Apple M1

2021-01-29 Thread BALATON Zoltan
The bug report says: "Qemu installed by using brew install qemu -s on M1" so I think it's whatver options homebrew used: https://formulae.brew.sh/formula/qemu -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/

Re: [Bug 1913669] [NEW] FPE in npcm7xx_adc_convert

2021-01-29 Thread Philippe Mathieu-Daudé
Cc'ing supporters: $ ./scripts/get_maintainer.pl -f hw/arm/npcm7xx.c Havard Skinnemoen (supporter:Nuvoton NPCM7xx) Tyrone Ting (supporter:Nuvoton NPCM7xx) On 1/29/21 3:36 AM, Alexander Bulekov wrote: > Public bug reported: > > Reproducer: > cat << EOF | ./qemu-system-aarch64 -M npcm750-evb \ >

Re: vnc clipboard support

2021-01-29 Thread Gerd Hoffmann
On Fri, Jan 29, 2021 at 12:34:32PM +0400, Marc-André Lureau wrote: > Hi > > On Fri, Jan 29, 2021 at 12:28 PM Gerd Hoffmann wrote: > > > > > Well. That is a completely different level of desktop integration. > > It surely makes sense to have that, and dbus-over-vsock looks like > > a reasonable

[Bug 1913668] Re: FPE in npcm7xx_pwm_calculate_freq

2021-01-29 Thread Philippe Mathieu-Daudé
** Tags added: arm ** Changed in: qemu Status: New => Confirmed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1913668 Title: FPE in npcm7xx_pwm_calculate_freq Status in QEMU: Confirmed

[Bug 1913667] Re: FPE in npcm7xx_clk_update_pll

2021-01-29 Thread Philippe Mathieu-Daudé
** Tags added: arm ** Changed in: qemu Status: New => Confirmed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1913667 Title: FPE in npcm7xx_clk_update_pll Status in QEMU: Confirmed Bug

Re: vnc clipboard support

2021-01-29 Thread Christophe de Dinechin
> On 29 Jan 2021, at 09:03, Gerd Hoffmann wrote: > > Hi, > >>> (1) Have some guest agent (spice does it that way). >>> Advantage: more flexible, allows more features. >>> Disadvantage: requires agent in the guest. >> >> What about running the option to relay data to a VNC server in

[PULL 02/46] target/arm: Conditionalize DBGDIDR

2021-01-29 Thread Peter Maydell
From: Richard Henderson Only define the register if it exists for the cpu. Signed-off-by: Richard Henderson Message-id: 20210120031656.737646-1-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/helper.c | 21 +++-- 1 file chan

[PULL 03/46] arm: rename xlnx-zcu102.canbusN properties

2021-01-29 Thread Peter Maydell
From: Paolo Bonzini The properties to attach a CANBUS object to the xlnx-zcu102 machine have a period in them. We want to use periods in properties for compound QAPI types, and besides the "xlnx-zcu102." prefix is both unnecessary and different from any other machine property name. Remove it.

[PULL 05/46] arm-virt: refactor gpios creation

2021-01-29 Thread Peter Maydell
From: Maxim Uvarov No functional change. Just refactor code to better support secure and normal world gpios. Signed-off-by: Maxim Uvarov Reviewed-by: Andrew Jones Signed-off-by: Peter Maydell --- hw/arm/virt.c | 57 --- 1 file changed, 36 inser

[PULL 08/46] target/arm: Replace magic value by MMU_DATA_LOAD definition

2021-01-29 Thread Peter Maydell
From: Philippe Mathieu-Daudé cpu_get_phys_page_debug() uses 'DATA LOAD' MMU access type. Signed-off-by: Philippe Mathieu-Daudé Message-id: 20210127232822.3530782-1-f4...@amsat.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/helper.c | 2 +- 1 file changed, 1 insert

[PULL 06/46] arm-virt: add secure pl061 for reset/power down

2021-01-29 Thread Peter Maydell
From: Maxim Uvarov Add secure pl061 for reset/power down machine from the secure world (Arm Trusted Firmware). Connect it with gpio-pwr driver. Signed-off-by: Maxim Uvarov Reviewed-by: Andrew Jones [PMM: Added mention of the new device to the documentation] Signed-off-by: Peter Maydell --- d

[PULL 01/46] target/arm: Implement ID_PFR2

2021-01-29 Thread Peter Maydell
From: Richard Henderson This was defined at some point before ARMv8.4, and will shortly be used by new processor descriptions. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20210120204400.1056582-1-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target

[PULL 12/46] darwin: remove redundant dependency declaration

2021-01-29 Thread Peter Maydell
From: Joelle van Dyne Meson will find CoreFoundation, IOKit, and Cocoa as needed. Reviewed-by: Peter Maydell Signed-off-by: Joelle van Dyne Message-id: 20210126012457.39046-...@getutm.app Signed-off-by: Peter Maydell --- configure | 1 - 1 file changed, 1 deletion(-) diff --git a/configure

[PULL 09/46] configure: Move preadv check to meson.build

2021-01-29 Thread Peter Maydell
Move the preadv availability check to meson.build. This is what we want to be doing for host-OS-feature-checks anyway, but it also fixes a problem with building for macOS with the most recent XCode SDK on a Catalina host. On that configuration, 'preadv()' is provided as a weak symbol, so that pro

[PULL 00/46] target-arm queue

2021-01-29 Thread Peter Maydell
.git tags/pull-target-arm-20210129 for you to fetch changes up to 11749122e1a86866591306d43603d2795a3dea1a: hw/arm/stellaris: Remove board-creation reset of STELLARIS_SYS (2021-01-29 10:47:29 +) target-arm queue: * Impleme

[PULL 11/46] osdep: build with non-working system() function

2021-01-29 Thread Peter Maydell
From: Joelle van Dyne Build without error on hosts without a working system(). If system() is called, return -1 with ENOSYS. Signed-off-by: Joelle van Dyne Message-id: 20210126012457.39046-...@getutm.app Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- meson.build | 1 +

[PULL 13/46] darwin: fix cross-compiling for Darwin

2021-01-29 Thread Peter Maydell
From: Joelle van Dyne Add objc to the Meson cross file as well as detection of Darwin. Reviewed-by: Peter Maydell Signed-off-by: Joelle van Dyne Reviewed-by: Philippe Mathieu-Daudé Message-id: 20210126012457.39046-...@getutm.app Signed-off-by: Peter Maydell --- configure | 4 1 file ch

[PULL 14/46] configure: cross compile should use x86_64 cpu_family

2021-01-29 Thread Peter Maydell
From: Joelle van Dyne Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Joelle van Dyne Message-id: 20210126012457.39046-...@getutm.app Signed-off-by: Peter Maydell --- configure | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/configure b/configure index e520a434065..

[PULL 16/46] darwin: remove 64-bit build detection on 32-bit OS

2021-01-29 Thread Peter Maydell
From: Joelle van Dyne A workaround added in early days of 64-bit OSX forced x86_64 if the host machine had 64-bit support. This creates issues when cross- compiling for ARM64. Additionally, the user can always use --cpu=* to manually set the host CPU and therefore this workaround should be remove

[PULL 07/46] hw/misc: Fix arith overflow in NPCM7XX PWM module

2021-01-29 Thread Peter Maydell
From: Hao Wu Fix potential overflow problem when calculating pwm_duty. 1. Ensure p->cmr and p->cnr to be from [0,65535], according to the hardware specification. 2. Changed duty to uint32_t. However, since MAX_DUTY * (p->cmr+1) can excceed UINT32_MAX, we convert them to uint64_t in computat

[PULL 18/46] hw/misc/pvpanic: split-out generic and bus dependent code

2021-01-29 Thread Peter Maydell
From: Mihai Carabas To ease the PCI device addition in next patches, split the code as follows: - generic code (read/write/setup) is being kept in pvpanic.c - ISA dependent code moved to pvpanic-isa.c Also, rename: - ISA_PVPANIC_DEVICE -> PVPANIC_ISA_DEVICE. - TYPE_PVPANIC -> TYPE_PVPANIC_ISA. -

[PULL 19/46] hw/misc/pvpanic: add PCI interface support

2021-01-29 Thread Peter Maydell
From: Mihai Carabas Add PCI interface support for PVPANIC device. Create a new file pvpanic-pci.c where the PCI specific routines reside and update the build system with the new files and config structure. Signed-off-by: Mihai Carabas Reviewed-by: Gerd Hoffmann Reviewed-by: Peter Maydell Sign

[PULL 17/46] hvf: Add hypervisor entitlement to output binaries

2021-01-29 Thread Peter Maydell
From: Alexander Graf In macOS 11, QEMU only gets access to Hypervisor.framework if it has the respective entitlement. Add an entitlement template and automatically self sign and apply the entitlement in the build. Signed-off-by: Alexander Graf Reviewed-by: Roman Bolshakov Tested-by: Roman Bols

[PULL 04/46] hw: gpio: implement gpio-pwr driver for qemu reset/poweroff

2021-01-29 Thread Peter Maydell
From: Maxim Uvarov Implement gpio-pwr driver to allow reboot and poweroff machine. This is simple driver with just 2 gpios lines. Current use case is to reboot and poweroff virt machine in secure mode. Secure pl066 gpio chip is needed for that. Signed-off-by: Maxim Uvarov Reviewed-by: Hao Wu R

[PULL 26/46] tests: Add a simple test of the CMSDK APB dual timer

2021-01-29 Thread Peter Maydell
Add a simple test of the CMSDK dual timer, since we're about to do some refactoring of how it is clocked. Signed-off-by: Peter Maydell Tested-by: Philippe Mathieu-Daudé Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Luc Michel Message-id: 20210128114145.20536-6-peter.mayd...@linaro.org Messa

[PULL 15/46] darwin: detect CoreAudio for build

2021-01-29 Thread Peter Maydell
From: Joelle van Dyne On iOS there is no CoreAudio, so we should not assume Darwin always has it. Signed-off-by: Joelle van Dyne Reviewed-by: Philippe Mathieu-Daudé Message-id: 20210126012457.39046-1...@getutm.app Signed-off-by: Peter Maydell --- configure | 35 ++

[PULL 24/46] tests: Add a simple test of the CMSDK APB timer

2021-01-29 Thread Peter Maydell
Add a simple test of the CMSDK APB timer, since we're about to do some refactoring of how it is clocked. Signed-off-by: Peter Maydell Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Luc Michel Tested-by: Philippe Mathieu-Daudé Message-id: 20210128114145.20536-4-peter.mayd...@linaro.org Messag

[PULL 10/46] configure: cross-compiling with empty cross_prefix

2021-01-29 Thread Peter Maydell
From: Joelle van Dyne The iOS toolchain does not use the host prefix naming convention. So we need to enable cross-compile options while allowing the PREFIX to be blank. Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Joelle van Dyne Message-id: 20210126012457.39046-...@getutm.app Signed-of

[PULL 31/46] hw/arm/armsse: Rename "MAINCLK" property to "MAINCLK_FRQ"

2021-01-29 Thread Peter Maydell
While we transition the ARMSSE code from integer properties specifying clock frequencies to Clock objects, we want to have the device provide both at once. We want the final name of the main input Clock to be "MAINCLK", following the hardware name. Unfortunately creating an input Clock with a name

[PULL 22/46] ptimer: Add new ptimer_set_period_from_clock() function

2021-01-29 Thread Peter Maydell
The ptimer API currently provides two methods for setting the period: ptimer_set_period(), which takes a period in nanoseconds, and ptimer_set_freq(), which takes a frequency in Hz. Neither of these lines up nicely with the Clock API, because although both the Clock and the ptimer track the freque

[PULL 21/46] tests/qtest: add a test case for pvpanic-pci

2021-01-29 Thread Peter Maydell
From: Mihai Carabas Add a test case for pvpanic-pci device. The scenario is the same as pvpanic ISA device, but is using the PCI bus. Signed-off-by: Mihai Carabas Acked-by: Thomas Huth Reviewed-by: Peter Maydell Signed-off-by: Mihai Carabas Signed-off-by: Peter Maydell --- tests/qtest/pvpa

[PULL 23/46] clock: Add new clock_has_source() function

2021-01-29 Thread Peter Maydell
Add a function for checking whether a clock has a source. This is useful for devices which have input clocks that must be wired up by the board as it allows them to fail in realize rather than ploughing on with a zero-period clock. Signed-off-by: Peter Maydell Reviewed-by: Luc Michel Reviewed-b

[PULL 25/46] tests: Add a simple test of the CMSDK APB watchdog

2021-01-29 Thread Peter Maydell
Add a simple test of the CMSDK watchdog, since we're about to do some refactoring of how it is clocked. Signed-off-by: Peter Maydell Reviewed-by: Luc Michel Tested-by: Philippe Mathieu-Daudé Reviewed-by: Philippe Mathieu-Daudé Message-id: 20210128114145.20536-5-peter.mayd...@linaro.org Message

[PULL 20/46] pvpanic : update pvpanic spec document

2021-01-29 Thread Peter Maydell
From: Mihai Carabas Add pvpanic PCI device support details in docs/specs/pvpanic.txt. Signed-off-by: Mihai Carabas Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- docs/specs/pvpanic.txt | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/docs/specs/p

[PULL 27/46] hw/timer/cmsdk-apb-timer: Rename CMSDKAPBTIMER struct to CMSDKAPBTimer

2021-01-29 Thread Peter Maydell
The state struct for the CMSDK APB timer device doesn't follow our usual naming convention of camelcase -- "CMSDK" and "APB" are both acronyms, but "TIMER" is not so should not be all-uppercase. Globally rename the struct to "CMSDKAPBTimer" (bringing it into line with CMSDKAPBWatchdog and CMSDKAPBD

  1   2   3   4   >