Re: [PATCH] target/i386: Add ARCH_CAPABILITIES related bits into Icelake-Server CPU model

2020-03-16 Thread Paolo Bonzini
On 16/03/20 06:33, Xiaoyao Li wrote: > Current Icelake-Server CPU model lacks all the features enumerated by > MSR_IA32_ARCH_CAPABILITIES. > > Add them, so that guest of "Icelake-Server" can see all of them. > > Signed-off-by: Xiaoyao Li > --- > target/i386/cpu.c | 7 ++- > 1 file changed,

Re: [PATCH] checkpatch: enforce process for expected files

2020-03-16 Thread Stefan Hajnoczi
On Sun, Mar 15, 2020 at 07:35:46AM -0400, Michael S. Tsirkin wrote: > If the process documented in tests/qtest/bios-tables-test.c > is followed, then same patch never touches both expected > files and code. Teach checkpatch to enforce this rule. > > Signed-off-by: Michael S. Tsirkin > --- > >

[PATCH] travis.yml: Set G_MESSAGES_DEBUG do report GLib errors

2020-03-16 Thread Philippe Mathieu-Daudé
Since commit f5852efa293 we can display GLib errors with the QEMU error reporting API. Set it to the 'error' level, as this helps understanding failures from QEMU calls to GLib on Travis-CI. Signed-off-by: Philippe Mathieu-Daudé --- .travis.yml | 1 + 1 file changed, 1 insertion(+) diff --git

[Bug 1866870] Re: KVM Guest pauses after upgrade to Ubuntu 20.04

2020-03-16 Thread Christian Ehrhardt 
Thanks David! While bisecting on upstream git with just "-cpu Penryn" we have seen that it always works there. So it might be an interaction with some Ubuntu build/packaging/configure detail together with these old chips. While we still can't be sure if the VMX warnings are a red-herring

[PATCH v7 3/4] qcow2: add zstd cluster compression

2020-03-16 Thread Denis Plotnikov
zstd significantly reduces cluster compression time. It provides better compression performance maintaining the same level of the compression ratio in comparison with zlib, which, at the moment, is the only compression method available. The performance test results: Test compresses and

Re: [PATCH] cpus: avoid stucking in pause_all_vcpus due to race

2020-03-16 Thread Paolo Bonzini
On 16/03/20 09:37, Longpeng(Mike) wrote: > From: Longpeng > > We found an issue when repeat reboot in guest during migration, it cause the > migration thread never be waken up again. > > | >| > LOCK BQL | >

Re: [PATCH v6 1/4] qcow2: introduce compression type feature

2020-03-16 Thread Denis Plotnikov
Thanks for the comments. I'll make the fixes accordingly and re-sent the series shortly. Denis On 14.03.2020 00:40, Eric Blake wrote: On 3/12/20 4:22 AM, Denis Plotnikov wrote: The patch adds some preparation parts for incompatible compression type feature to qcow2 allowing the use different

[PATCH v3 2/4] linux-user, aarch64: sync syscall numbers with kernel v5.5

2020-03-16 Thread Laurent Vivier
Use helper script scripts/gensyscalls.sh to generate the file. This change TARGET_NR_fstatat64 by TARGET_NR_newfstatat that is correct because definitions from linux are: arch/arm64/include/uapi/asm/unistd.h #define __ARCH_WANT_NEW_STAT include/uapi/asm-generic/unistd.h #if

Re: [PATCH v2 3/8] qapi/misc: Restrict balloon-related commands to machine code

2020-03-16 Thread David Hildenbrand
On 16.03.20 01:03, Philippe Mathieu-Daudé wrote: > Signed-off-by: Philippe Mathieu-Daudé > --- > qapi/machine.json | 83 ++ > qapi/misc.json | 83 -- > include/sysemu/balloon.h | 2 +- > balloon.c

Re: [PATCH] target/rx/cpu: Use address_space_ldl() to read reset vector address

2020-03-16 Thread Peter Maydell
On Sun, 15 Mar 2020 at 13:49, Philippe Mathieu-Daudé wrote: > > From: Philippe Mathieu-Daudé > > The RX code flash is not a Masked ROM but a EEPROM (electrically > erasable programmable flash memory). > When implementing the flash hardware, the rom_ptr() returns NULL > and the reset vector is

[PATCH v4 3/6] virtio-net: implement RX RSS processing

2020-03-16 Thread Yuri Benditovich
If VIRTIO_NET_F_RSS negotiated and RSS is enabled, process incoming packets, calculate packet's hash and place the packet into respective RX virtqueue. Signed-off-by: Yuri Benditovich --- hw/net/virtio-net.c| 88 +- include/hw/virtio/virtio-net.h | 1

[PATCH v4 5/6] virtio-net: reference implementation of hash report

2020-03-16 Thread Yuri Benditovich
Suggest VIRTIO_NET_F_HASH_REPORT if specified in device parameters. If the VIRTIO_NET_F_HASH_REPORT is set, the device extends configuration space. If the feature is negotiated, the packet layout is extended to accomodate the hash information. In this case deliver packet's hash value and report

[PATCH v4 1/6] virtio-net: introduce RSS and hash report features

2020-03-16 Thread Yuri Benditovich
Signed-off-by: Yuri Benditovich --- hw/net/virtio-net.c | 65 + 1 file changed, 65 insertions(+) diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c index 3627bb1717..90b01221e9 100644 --- a/hw/net/virtio-net.c +++ b/hw/net/virtio-net.c @@ -71,6

[PATCH v4 2/6] virtio-net: implement RSS configuration command

2020-03-16 Thread Yuri Benditovich
Optionally report RSS feature. Handle RSS configuration command and keep RSS parameters in virtio-net device context. Signed-off-by: Yuri Benditovich --- hw/net/trace-events| 3 + hw/net/virtio-net.c| 189 +

Re: [PATCH] tools/virtiofsd: add support for --socket-group

2020-03-16 Thread Daniel P . Berrangé
On Sat, Mar 14, 2020 at 02:33:25PM +0100, Marc-André Lureau wrote: > Hi > > On Thu, Mar 12, 2020 at 11:49 AM Daniel P. Berrangé > wrote: > > > > On Thu, Mar 12, 2020 at 10:41:42AM +, Alex Bennée wrote: > > > If you like running QEMU as a normal user (very common for TCG runs) > > > but you

[Bug 1867601] [NEW] test-char not concurrent with unix socket

2020-03-16 Thread Philippe Mathieu-Daudé
Public bug reported: 'make check-unit' might fail when running multiple tests in parallel. Apparently occurred on OSX CI: https://travis-ci.org/github/philmd/qemu/jobs/662357430 Guess is same unix path used: static SocketAddress unixaddr = { .type = SOCKET_ADDRESS_TYPE_UNIX,

[PATCH v7 0/4] qcow2: Implement zstd cluster compression method

2020-03-16 Thread Denis Plotnikov
v7: * use qapi_enum_parse instead of the open-coding [Eric] * fix wording, typos and spelling [Eric] v6: * "block/qcow2-threads: fix qcow2_decompress" is removed from the series since it has been accepted by Max already * add compile time checking for Qcow2Header to be a

Re: [PATCH v9 02/10] scripts: Coccinelle script to use ERRP_AUTO_PROPAGATE()

2020-03-16 Thread Markus Armbruster
Vladimir Sementsov-Ogievskiy writes: > On 14.03.2020 00:54, Markus Armbruster wrote: >> Vladimir Sementsov-Ogievskiy writes: >> >>> 13.03.2020 18:42, Markus Armbruster wrote: Vladimir Sementsov-Ogievskiy writes: > 12.03.2020 19:36, Markus Armbruster wrote: >> I may have a

Re: [PATCH v7 3/5] virtio-iommu: Call iommu notifier for attach/detach

2020-03-16 Thread Auger Eric
Hi Bharat, On 3/16/20 9:58 AM, Bharat Bhushan wrote: > Hi Eric, > > On Mon, Mar 16, 2020 at 1:15 PM Bharat Bhushan > wrote: >> >> Hi Eric, >> >> On Mon, Mar 16, 2020 at 1:02 PM Auger Eric wrote: >>> >>> Hi Bharat, >>> >>> On 3/16/20 7:41 AM, Bharat Bhushan wrote: Hi Eric, On

Re: [PATCH 4/8] hw/ide: Move MAX_IDE_BUS define to one header

2020-03-16 Thread Philippe Mathieu-Daudé
On 3/16/20 7:53 AM, Markus Armbruster wrote: BALATON Zoltan writes: There are several definitions of MAX_IDE_BUS in different boards (some of them unused) with the same value. Move it to include/hw/ide/internal.h to have it in a central place. Signed-off-by: BALATON Zoltan This one feels

Re: [PATCH v7 3/5] virtio-iommu: Call iommu notifier for attach/detach

2020-03-16 Thread Bharat Bhushan
Hi Eric, On Mon, Mar 16, 2020 at 2:35 PM Auger Eric wrote: > > Hi Bharat, > > On 3/16/20 9:58 AM, Bharat Bhushan wrote: > > Hi Eric, > > > > On Mon, Mar 16, 2020 at 1:15 PM Bharat Bhushan > > wrote: > >> > >> Hi Eric, > >> > >> On Mon, Mar 16, 2020 at 1:02 PM Auger Eric wrote: > >>> > >>> Hi

[PATCH] cpus: avoid stucking in pause_all_vcpus due to race

2020-03-16 Thread Longpeng(Mike)
From: Longpeng We found an issue when repeat reboot in guest during migration, it cause the migration thread never be waken up again. | | LOCK BQL | ...| main_loop_should_exit

Re: [PATCH 6/8] hw/ide: Do ide_drive_get() within pci_ide_create_devs()

2020-03-16 Thread Philippe Mathieu-Daudé
On 3/16/20 7:23 AM, Markus Armbruster wrote: Paolo Bonzini writes: On 13/03/20 23:16, BALATON Zoltan wrote: +    pci_dev = pci_create_simple(pci_bus, -1, "cmd646-ide"); +    pci_ide_create_devs(pci_dev); Additionally, I think it may also make sense to move pci_ide_create_devs call into

Re: [PULL 132/136] mem-prealloc: optimize large guest startup

2020-03-16 Thread Laurent Vivier
Hi, a bug has been reported in launchpad for this patch: [Regression]Powerpc kvm guest unable to start with hugepage backed memory https://bugs.launchpad.net/qemu/+bug/1866962 Thanks, Laurent Le 25/02/2020 à 13:07, Paolo Bonzini a écrit : > From: bauerchen > > [desc]: > Large

[PATCH v3 3/4] linux-user, nios2: sync syscall numbers with kernel v5.5

2020-03-16 Thread Laurent Vivier
Use helper script scripts/gensyscalls.sh to generate the file. This adds TARGET_NR_llseek that was missing and remove syscalls 1024 to 1079. Add new syscalls from 288 (pkey_mprotect) to 434 (pidfd_open) Signed-off-by: Laurent Vivier Reviewed-by: Alistair Francis --- Notes: v2: add

Re: [PATCH v9] s390x: protvirt: Fence huge pages

2020-03-16 Thread Janosch Frank
On 3/13/20 9:21 AM, Christian Borntraeger wrote: > > > On 12.03.20 17:25, Janosch Frank wrote: >> Let's bail out of the protected transition if we detect that huge >> pages might be in use. >> >> Signed-off-by: Janosch Frank >> --- >> >> I'd like to squash this into the unpack patch to give a

[PATCH v2] target/i386: Add ARCH_CAPABILITIES related bits into Icelake-Server CPU model

2020-03-16 Thread Xiaoyao Li
Current Icelake-Server CPU model lacks all the features enumerated by MSR_IA32_ARCH_CAPABILITIES. Add them, so that guest of "Icelake-Server" can see all of them. Signed-off-by: Xiaoyao Li --- v2: - Add it as a new version. --- target/i386/cpu.c | 13 + 1 file changed, 13

[PATCH v3 4/4] linux-user, openrisc: sync syscall numbers with kernel v5.5

2020-03-16 Thread Laurent Vivier
Use helper script scripts/gensyscalls.sh to generate the file. Add TARGET_NR_or1k_atomic Remove useless comments and blank lines. Define diretly the __NR_XXX64 syscalls rather than using the intermediate __NR3264 definition. Remove wrong cut'n'paste (like "#ifdef __ARCH_WANT_SYNC_FILE_RANGE2")

Re: [PATCH v4 2/3] mac_via: fix incorrect creation of mos6522 device in mac_via

2020-03-16 Thread Paolo Bonzini
On 16/03/20 07:03, Markus Armbruster wrote: > Paolo Bonzini writes: > >> On 15/03/20 15:56, Markus Armbruster wrote: The question is why they are not, i.e. where does the above reasoning break. >>> I don't know. But let's for the sake of the argument assume this >>> actually

Re: [PULL 132/136] mem-prealloc: optimize large guest startup

2020-03-16 Thread Paolo Bonzini
On 16/03/20 09:42, Laurent Vivier wrote: > Hi, > > a bug has been reported in launchpad for this patch: > > [Regression]Powerpc kvm guest unable to start with hugepage backed > memory > https://bugs.launchpad.net/qemu/+bug/1866962 Indeed, I'm sending the pull request with the fix

Re: [PATCH] docs/conf.py: Raise ConfigError for bad Sphinx Python version

2020-03-16 Thread Peter Maydell
On Fri, 13 Mar 2020 at 22:30, John Snow wrote: > When was ConfigError introduced, and what's our minimum Sphinx version? > (Hm, looks like it's not versioned, so I'll trust it's been around a while.) Yeah, it's been around a long time; our minimum Sphinx version is 1.3. thanks -- PMM

Re: [PATCH 0/2] Fix Cooperlake CPU model

2020-03-16 Thread Paolo Bonzini
On 16/03/20 02:39, Zhang, Cathy wrote: > On 1/7/2020 9:31 PM, Paolo Bonzini wrote: >> On 25/12/19 07:30, Xiaoyao Li wrote: >>> Current Cooperlake CPU model lacks VMX features which are introduced >>> by Paolo >>> several months ago, and it also lacks 2 security features in >>>

[PATCH v7 1/4] qcow2: introduce compression type feature

2020-03-16 Thread Denis Plotnikov
The patch adds some preparation parts for incompatible compression type feature to qcow2 allowing the use different compression methods for image clusters (de)compressing. It is implied that the compression type is set on the image creation and can be changed only later by image conversion, thus

Re: [PATCH 1/2] block: bdrv_set_backing_bs: fix use-after-free

2020-03-16 Thread Philippe Mathieu-Daudé
On 3/16/20 7:06 AM, Vladimir Sementsov-Ogievskiy wrote: There is a use-after-free possible: bdrv_unref_child() leaves bs->backing freed but not NULL. bdrv_attach_child may produce nested polling loop due to drain, than access of freed pointer is possible. I've produced the following crash on 30

Re: [PATCH v7 3/5] virtio-iommu: Call iommu notifier for attach/detach

2020-03-16 Thread Bharat Bhushan
Hi Eric, On Mon, Mar 16, 2020 at 1:15 PM Bharat Bhushan wrote: > > Hi Eric, > > On Mon, Mar 16, 2020 at 1:02 PM Auger Eric wrote: > > > > Hi Bharat, > > > > On 3/16/20 7:41 AM, Bharat Bhushan wrote: > > > Hi Eric, > > > > > > On Fri, Mar 13, 2020 at 8:11 PM Auger Eric wrote: > > >> > > >> Hi

[PATCH v3 1/4] scripts: add a script to generate syscall_nr.h

2020-03-16 Thread Laurent Vivier
This script is needed for targets based on asm-generic syscall numbers generation Signed-off-by: Laurent Vivier Reviewed-by: Alistair Francis Reviewed-by: Taylor Simpson Reviewed-by: Richard Henderson --- Notes: v3: remove useless upper command v2: add comments suggested by Taylor

[PATCH v3 0/4] linux-user: generate syscall_nr.h from linux unistd.h

2020-03-16 Thread Laurent Vivier
This series adds a script to generate syscall_nr.h for architectures that don't use syscall.tbl but asm-generic/unistd.h The script uses several cpp passes and filters result with a grep/sed/tr sequence. The result must be checked before being used, so it's why the script is not automatically

[PATCH v4 0/6] reference implementation of RSS and hash report

2020-03-16 Thread Yuri Benditovich
Support for VIRTIO_NET_F_RSS and VIRTIO_NET_F_HASH_REPORT features in QEMU for reference purpose. Implements Toeplitz hash calculation for incoming packets according to configuration provided by driver. Uses calculated hash for decision on receive virtqueue and/or reports the hash in the virtio

Re: [PATCH 0/2] Fix Cooperlake CPU model

2020-03-16 Thread Paolo Bonzini
On 16/03/20 11:19, Zhang, Cathy wrote: > Yes, I see they are already in master, but not in v4.2 yet, so will they > be in the next release v5.0? Yes, that's what master will become. Paolo

[PATCH v4 6/6] virtio-net: add migration support for RSS and hash report

2020-03-16 Thread Yuri Benditovich
Save and restore RSS/hash report configuration. Signed-off-by: Yuri Benditovich --- hw/net/virtio-net.c | 26 ++ 1 file changed, 26 insertions(+) diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c index a0614ad4e6..f343762a0f 100644 --- a/hw/net/virtio-net.c +++

Re: [PATCH v2 2/3] acpi: Add Windows ACPI Emulated Device Table (WAET)

2020-03-16 Thread Igor Mammedov
On Fri, 13 Mar 2020 16:50:08 +0200 Liran Alon wrote: > Microsoft introduced this ACPI table to avoid Windows guests performing > various workarounds for device erratas. As the virtual device emulated > by VMM may not have the errata. > > Currently, WAET allows hypervisor to inform guest about

[PATCH v2 0/2] thread: add lock guard macros

2020-03-16 Thread Stefan Hajnoczi
Lock guards automatically call qemu_(rec_)mutex_unlock() when returning from a function or leaving leaving a lexical scope. This simplifies code and eliminates leaks (especially in error code paths). This series adds lock guards for QemuMutex and QemuRecMutex. It does not convert the entire

Re: [PATCH v4 2/3] mac_via: fix incorrect creation of mos6522 device in mac_via

2020-03-16 Thread Markus Armbruster
Paolo Bonzini writes: > On 15/03/20 15:56, Markus Armbruster wrote: >>> >>> The question is why they are not, i.e. where does the above reasoning break. >> I don't know. But let's for the sake of the argument assume this >> actually worked. Asking for help in the monitor then *still* has side

Re: [PATCH 6/8] hw/ide: Do ide_drive_get() within pci_ide_create_devs()

2020-03-16 Thread Markus Armbruster
Paolo Bonzini writes: > On 13/03/20 23:16, BALATON Zoltan wrote: >>> >>> +    pci_dev = pci_create_simple(pci_bus, -1, "cmd646-ide"); >>> +    pci_ide_create_devs(pci_dev); >> >> Additionally, I think it may also make sense to move pci_ide_create_devs >> call into the realize methods of these

Re: [PATCH v7 2/5] virtio-iommu: Add iommu notifier for map/unmap

2020-03-16 Thread Bharat Bhushan
Hi Eric, On Fri, Mar 13, 2020 at 7:55 PM Auger Eric wrote: > > Hi Bharat, > On 3/13/20 8:48 AM, Bharat Bhushan wrote: > > This patch extends VIRTIO_IOMMU_T_MAP/UNMAP request to > > notify registered iommu-notifier. Which will call vfio > s/iommu-notifier/iommu-notifiers > > notifier to map/unmap

Re: [PATCH 0/8] Misc hw/ide legacy clean up

2020-03-16 Thread Markus Armbruster
BALATON Zoltan writes: > These are some clean ups to remove more legacy init functions and > lessen dependence on include/hw/ide.h with some simplifications in > board code. There should be no functional change. PATCH 1 could quote precedence more clearly in the commit message, but that's

Re: [PATCH v5 08/26] nvme: refactor device realization

2020-03-16 Thread Klaus Birkelund Jensen
On Feb 12 11:27, Maxim Levitsky wrote: > On Tue, 2020-02-04 at 10:51 +0100, Klaus Jensen wrote: > > This patch splits up nvme_realize into multiple individual functions, > > each initializing a different subset of the device. > > > > Signed-off-by: Klaus Jensen > > --- > > hw/block/nvme.c | 175

Re: [PATCH v5 17/26] nvme: allow multiple aios per command

2020-03-16 Thread Klaus Birkelund Jensen
On Feb 12 13:48, Maxim Levitsky wrote: > On Tue, 2020-02-04 at 10:51 +0100, Klaus Jensen wrote: > > This refactors how the device issues asynchronous block backend > > requests. The NvmeRequest now holds a queue of NvmeAIOs that are > > associated with the command. This allows multiple aios to be

[PATCH v7 2/4] qcow2: rework the cluster compression routine

2020-03-16 Thread Denis Plotnikov
The patch enables processing the image compression type defined for the image and chooses an appropriate method for image clusters (de)compression. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia --- block/qcow2-threads.c | 71

Re: [PATCH v7 3/5] virtio-iommu: Call iommu notifier for attach/detach

2020-03-16 Thread Bharat Bhushan
Hi Eric, On Fri, Mar 13, 2020 at 8:11 PM Auger Eric wrote: > > Hi Bharat > > On 3/13/20 8:48 AM, Bharat Bhushan wrote: > > iommu-notifier are called when a device is attached > IOMMU notifiers > > or detached to as address-space. > > This is needed for VFIO. > and vhost for detach > > > >

Re: [PATCH v2 1/8] target/i386: Restrict X86CPUFeatureWord to X86 targets

2020-03-16 Thread Philippe Mathieu-Daudé
On 3/16/20 1:29 AM, Aleksandar Markovic wrote: On Monday, March 16, 2020, Philippe Mathieu-Daudé > wrote: Move out x86-specific structures from generic machine code. Philippe, I a kind of have hard time understanding what is achieved with this patch. Is this

Re: [PATCH v7 3/5] virtio-iommu: Call iommu notifier for attach/detach

2020-03-16 Thread Bharat Bhushan
Hi Eric, On Mon, Mar 16, 2020 at 1:02 PM Auger Eric wrote: > > Hi Bharat, > > On 3/16/20 7:41 AM, Bharat Bhushan wrote: > > Hi Eric, > > > > On Fri, Mar 13, 2020 at 8:11 PM Auger Eric wrote: > >> > >> Hi Bharat > >> > >> On 3/13/20 8:48 AM, Bharat Bhushan wrote: > >>> iommu-notifier are called

Re: [PATCH v5 57/60] target/riscv: vector slide instructions

2020-03-16 Thread LIU Zhiwei
On 2020/3/15 13:16, Richard Henderson wrote: On 3/12/20 7:58 AM, LIU Zhiwei wrote: +#define GEN_VEXT_VSLIDEUP_VX(NAME, ETYPE, H, CLEAR_FN)\ +void HELPER(NAME)(void *vd, void *v0, target_ulong s1, void *vs2, \ +CPURISCVState *env, uint32_t desc)

Re: [PATCH v5 15/26] nvme: bump supported specification to 1.3

2020-03-16 Thread Klaus Birkelund Jensen
On Feb 12 12:35, Maxim Levitsky wrote: > On Tue, 2020-02-04 at 10:51 +0100, Klaus Jensen wrote: > > Add new fields to the Identify Controller and Identify Namespace data > > structures accoding to NVM Express 1.3d. > > > > NVM Express 1.3d requires the following additional features: > > -

Re: [PATCH v5 14/26] nvme: make sure ncqr and nsqr is valid

2020-03-16 Thread Klaus Birkelund Jensen
On Feb 12 12:30, Maxim Levitsky wrote: > On Tue, 2020-02-04 at 10:51 +0100, Klaus Jensen wrote: > > 0x is not an allowed value for NCQR and NSQR in Set Features on > > Number of Queues. > > > > Signed-off-by: Klaus Jensen > > --- > > hw/block/nvme.c | 4 > > 1 file changed, 4

Re: [PATCH v7 2/5] virtio-iommu: Add iommu notifier for map/unmap

2020-03-16 Thread Auger Eric
Hi Bharat, On 3/16/20 7:36 AM, Bharat Bhushan wrote: > Hi Eric, > > On Fri, Mar 13, 2020 at 7:55 PM Auger Eric wrote: >> >> Hi Bharat, >> On 3/13/20 8:48 AM, Bharat Bhushan wrote: >>> This patch extends VIRTIO_IOMMU_T_MAP/UNMAP request to >>> notify registered iommu-notifier. Which will call

Re: [PATCH v7 3/5] virtio-iommu: Call iommu notifier for attach/detach

2020-03-16 Thread Auger Eric
Hi Bharat, On 3/16/20 7:41 AM, Bharat Bhushan wrote: > Hi Eric, > > On Fri, Mar 13, 2020 at 8:11 PM Auger Eric wrote: >> >> Hi Bharat >> >> On 3/13/20 8:48 AM, Bharat Bhushan wrote: >>> iommu-notifier are called when a device is attached >> IOMMU notifiers >>> or detached to as address-space.

Re: [PATCH v2 00/12] user-mode: Prune build dependencies (part 1)

2020-03-16 Thread Philippe Mathieu-Daudé
On 3/16/20 1:16 AM, Aleksandar Markovic wrote: On Monday, March 16, 2020, Philippe Mathieu-Daudé > wrote: This is the first part of a series reducing user-mode dependencies. By stripping out unused code, the build and testing time is reduced (as is space

Re: [PATCH v5 09/26] nvme: add temperature threshold feature

2020-03-16 Thread Klaus Birkelund Jensen
On Feb 12 11:31, Maxim Levitsky wrote: > On Tue, 2020-02-04 at 10:51 +0100, Klaus Jensen wrote: > > It might seem wierd to implement this feature for an emulated device, > > but it is mandatory to support and the feature is useful for testing > > asynchronous event request support, which will be

Re: [PATCH v5 10/26] nvme: add support for the get log page command

2020-03-16 Thread Klaus Birkelund Jensen
On Feb 12 11:35, Maxim Levitsky wrote: > On Tue, 2020-02-04 at 10:51 +0100, Klaus Jensen wrote: > > Add support for the Get Log Page command and basic implementations of > > the mandatory Error Information, SMART / Health Information and Firmware > > Slot Information log pages. > > > > In

Re: [PATCH v5 12/26] nvme: add missing mandatory features

2020-03-16 Thread Klaus Birkelund Jensen
On Feb 12 12:27, Maxim Levitsky wrote: > On Tue, 2020-02-04 at 10:51 +0100, Klaus Jensen wrote: > > Add support for returning a resonable response to Get/Set Features of > > mandatory features. > > > > Signed-off-by: Klaus Jensen > > --- > > hw/block/nvme.c | 57

[PATCH v7 4/4] iotests: 287: add qcow2 compression type test

2020-03-16 Thread Denis Plotnikov
The test checks fulfilling qcow2 requiriements for the compression type feature and zstd compression type operability. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/287 | 128 + tests/qemu-iotests/287.out

[PATCH 1/2] block: bdrv_set_backing_bs: fix use-after-free

2020-03-16 Thread Vladimir Sementsov-Ogievskiy
There is a use-after-free possible: bdrv_unref_child() leaves bs->backing freed but not NULL. bdrv_attach_child may produce nested polling loop due to drain, than access of freed pointer is possible. I've produced the following crash on 30 iotest with modified code. It does not reproduce on

Re: [PATCH v5 16/26] nvme: refactor prp mapping

2020-03-16 Thread Klaus Birkelund Jensen
On Feb 12 13:44, Maxim Levitsky wrote: > On Tue, 2020-02-04 at 10:51 +0100, Klaus Jensen wrote: > > Refactor nvme_map_prp and allow PRPs to be located in the CMB. The logic > > ensures that if some of the PRP is in the CMB, all of it must be located > > there, as per the specification. > > To be

[PATCH 2/2] block/qcow2: zero data_file child after free

2020-03-16 Thread Vladimir Sementsov-Ogievskiy
data_file being NULL doesn't seem to be a correct state, but it's better than dead pointer and simpler to debug. Signed-off-by: Vladimir Sementsov-Ogievskiy --- block/qcow2.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/block/qcow2.c b/block/qcow2.c index d44b45633d..6cdefe059f 100644

[PATCH 0/2] zero pointer after bdrv_unref_child

2020-03-16 Thread Vladimir Sementsov-Ogievskiy
Hi all! I faced use-after-free of bs->backing pointer after bdrv_unref_child in bdrv_set_backing_hd. Fix it, and do similar thing for s->data_file in qcow2.c. I'm not sure that this is the full fix. Is it safe to keep bs->backing during bdrv_unref_child itself? Is it safe to keep bs->backing

Re: [PATCH 1/8] hw/ide: Get rid of piix3_init functions

2020-03-16 Thread Markus Armbruster
BALATON Zoltan writes: > This removes pci_piix3_ide_init() and pci_piix3_xen_ide_init() > functions similar to clean up done to other ide devices. Got a commit hash for "done to other ide devices"? > Signed-off-by: BALATON Zoltan

Re: [PATCH 4/8] hw/ide: Move MAX_IDE_BUS define to one header

2020-03-16 Thread Markus Armbruster
BALATON Zoltan writes: > There are several definitions of MAX_IDE_BUS in different boards (some > of them unused) with the same value. Move it to include/hw/ide/internal.h > to have it in a central place. > > Signed-off-by: BALATON Zoltan This one feels a bit questionable. The number of

Re: [PATCH v9 02/10] scripts: Coccinelle script to use ERRP_AUTO_PROPAGATE()

2020-03-16 Thread Vladimir Sementsov-Ogievskiy
On 14.03.2020 00:54, Markus Armbruster wrote: Vladimir Sementsov-Ogievskiy writes: 13.03.2020 18:42, Markus Armbruster wrote: Vladimir Sementsov-Ogievskiy writes: 12.03.2020 19:36, Markus Armbruster wrote: I may have a second look tomorrow with fresher eyes, but let's get this out now

Re: [PATCH v5 21/26] nvme: add support for scatter gather lists

2020-03-16 Thread Klaus Birkelund Jensen
On Feb 12 14:07, Maxim Levitsky wrote: > On Tue, 2020-02-04 at 10:52 +0100, Klaus Jensen wrote: > > For now, support the Data Block, Segment and Last Segment descriptor > > types. > > > > See NVM Express 1.3d, Section 4.4 ("Scatter Gather List (SGL)"). > > > > Signed-off-by: Klaus Jensen > >

Re: [PATCH v5 20/26] nvme: handle dma errors

2020-03-16 Thread Klaus Birkelund Jensen
On Feb 12 13:52, Maxim Levitsky wrote: > On Tue, 2020-02-04 at 10:52 +0100, Klaus Jensen wrote: > > Handling DMA errors gracefully is required for the device to pass the > > block/011 test ("disable PCI device while doing I/O") in the blktests > > suite. > > > > With this patch the device passes

Re: [PATCH v5 22/26] nvme: support multiple namespaces

2020-03-16 Thread Klaus Birkelund Jensen
On Feb 12 14:34, Maxim Levitsky wrote: > On Tue, 2020-02-04 at 10:52 +0100, Klaus Jensen wrote: > > This adds support for multiple namespaces by introducing a new 'nvme-ns' > > device model. The nvme device creates a bus named from the device name > > ('id'). The nvme-ns devices then connect to

Re: [PATCH 0/2] Fix Cooperlake CPU model

2020-03-16 Thread Zhang, Cathy
On 3/16/2020 4:41 PM, Paolo Bonzini wrote: On 16/03/20 02:39, Zhang, Cathy wrote: On 1/7/2020 9:31 PM, Paolo Bonzini wrote: On 25/12/19 07:30, Xiaoyao Li wrote: Current Cooperlake CPU model lacks VMX features which are introduced by Paolo several months ago, and it also lacks 2 security

Re: [PATCH v2] python/qemu/qmp.py: QMP debug with VM label

2020-03-16 Thread Oksana Voshchana
Hi Eduardo I'm already fixing it. Thank you, On Sun, Mar 15, 2020 at 5:39 PM Eduardo Habkost wrote: > On Thu, Mar 12, 2020 at 04:05:47PM +0200, Oksana Vohchana wrote: > > QEMUMachine writes some messages to the default logger. > > But it sometimes hard to read the output if we have requests to

Re: [PATCH] softmmu/vl.c: Handle '-cpu help' and '-device help' before 'no default machine'

2020-03-16 Thread Kashyap Chamarthy
[Cc: Markus; he'd be pleasantly surprised with this, if he already hadn't noticed this, as he was also mildly annoyed about this the other day.] On Fri, Mar 13, 2020 at 05:24:47PM +, Peter Maydell wrote: > Currently if you try to ask for the list of CPUs for a target > architecture which does

[PULL 0/6] Audio 20200316 patches

2020-03-16 Thread Gerd Hoffmann
The following changes since commit 61c265f0660ee476985808c8aa7915617c44fd53: Merge remote-tracking branch 'remotes/dgilbert/tags/pull-migration-20200313a' into staging (2020-03-13 10:33:04 +) are available in the Git repository at: git://git.kraxel.org/qemu tags/audio-20200316-pull

Re: [PATCH v5 07/50] multi-process: define mpqemu-link object

2020-03-16 Thread Stefan Hajnoczi
On Tue, Mar 10, 2020 at 11:26:23AM -0700, Elena Ufimtseva wrote: > On Tue, Mar 10, 2020 at 04:09:41PM +, Stefan Hajnoczi wrote: > > On Mon, Feb 24, 2020 at 03:54:58PM -0500, Jagannathan Raman wrote: > > > +msg->num_fds = 0; > > > +for (chdr = CMSG_FIRSTHDR(); chdr != NULL; > > > +

Re: [PATCH] modules: load modules from versioned /var/run dir

2020-03-16 Thread Stefan Hajnoczi
On Tue, Mar 10, 2020 at 12:47:49PM +0100, Christian Ehrhardt wrote: > On Tue, Mar 10, 2020 at 10:39 AM Stefan Hajnoczi wrote: > > On Fri, Mar 06, 2020 at 02:26:48PM +0100, Christian Ehrhardt wrote: > And finally this has to be considered an "offer" by qemu to the packagers > to fix a real field

[PATCH 0/2] avoid integer overflow

2020-03-16 Thread Yifei Jiang
the constant default type is "int", when the constant is shifted to the left, it may exceed 32 bits, resulting in integer overflowing. So constant type need change to "long" Yifei Jiang (2): tcg: avoid integer overflow accel/tcg: avoid integer overflow accel/tcg/cputlb.c | 6 +++---

[PULL 2/6] audio: change naming scheme of FLOAT_CONV macros

2020-03-16 Thread Gerd Hoffmann
From: Volker Rümelin This patch changes the naming scheme of the FLOAT_CONV_TO and FLOAT_CONV_FROM macros to the scheme used in mixeng_template.h. Signed-off-by: Volker Rümelin Message-id: 20200308193321.20668-2-vr_q...@t-online.de Signed-off-by: Gerd Hoffmann --- audio/mixeng.c | 22

Re: [PATCH v2 0/6] mostly changes related to audio float samples

2020-03-16 Thread Gerd Hoffmann
On Sun, Mar 08, 2020 at 08:29:05PM +0100, Volker Rümelin wrote: > v2: > - "qapi/audio: add documentation for AudioFormat" >   Markus suggested to correct a spelling mistake. > > - "audio: add audiodev format=f32 option documentation" >   New patch. Pull request sent. thanks, Gerd

[PATCH 01/11] MAINTAINERS: Fix KVM path expansion glob

2020-03-16 Thread Philippe Mathieu-Daudé
The KVM files has been moved from target-ARCH to the target/ARCH/ folder in commit fcf5ef2a. Fix the pathname expansion. Fixes: fcf5ef2a ("Move target-* CPU file into a target/ folder") Signed-off-by: Philippe Mathieu-Daudé --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [PATCH 0/5] QEMU Gating CI

2020-03-16 Thread Cleber Rosa
- Original Message - > From: "Peter Maydell" > To: "Cleber Rosa" > Cc: "Alex Bennée" , "QEMU Developers" > , "Fam Zheng" , > "Eduardo Habkost" , "Beraldo Leal" , > "Philippe Mathieu-Daudé" > , "Thomas Huth" , "Wainer dos Santos > Moschetta" , "Erik > Skultety" , "Willian Rampazzo"

[PATCH v4] python/qemu/qmp.py: QMP debug with VM label

2020-03-16 Thread Oksana Vohchana
QEMUMachine writes some messages to the default logger. But it sometimes hard to read the output if we have requests to more than one VM. This patch adds a label to the logger in the debug mode. Signed-off-by: Oksana Vohchana --- v2: - Instead of shown the label in the message it provides the

[PATCH v4 4/6] tap: allow extended virtio header with hash info

2020-03-16 Thread Yuri Benditovich
Signed-off-by: Yuri Benditovich --- net/tap.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/net/tap.c b/net/tap.c index 6207f61f84..47de7fdeb6 100644 --- a/net/tap.c +++ b/net/tap.c @@ -63,6 +63,14 @@ typedef struct TAPState { Notifier exit; } TAPState;

Re: [PATCH v3 0/4] linux-user: generate syscall_nr.h from linux unistd.h

2020-03-16 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200316085620.309769-1-laur...@vivier.eu/ Hi, This series failed the asan 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 === #!/bin/bash

[PATCH 03/11] MAINTAINERS: Add an entry for the HAX accelerator

2020-03-16 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé --- Cc: Sergio Andres Gomez Del Real Cc: Vincent Palatin Cc: Yu Ning Cc: Tao Wu Cc: haxm-t...@intel.com Cc: Colin Xu Cc: Hang Yuan Cc: David Chou Cc: Wenchao Wang --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS

Re: [PATCH RESEND v2] block/nvme: introduce PMR support from NVMe 1.4 spec

2020-03-16 Thread Stefan Hajnoczi
On Wed, Mar 11, 2020 at 11:08:27PM -0700, Klaus Birkelund Jensen wrote: > On Mar 11 15:54, Andrzej Jakowski wrote: > > On 3/11/20 2:20 AM, Stefan Hajnoczi wrote: > > > Please try: > > > > > > $ git grep pmem > > > > > > backends/hostmem-file.c is the backend that can be used and the > > >

Re: [PATCH 0/5] QEMU Gating CI

2020-03-16 Thread Peter Maydell
On Thu, 12 Mar 2020 at 22:16, Cleber Rosa wrote: > The quick answer is: > > $ git push g...@gitlab.com:qemu-project/qemu.git my-branch:staging So I did this bit... > Once that push happens, you could use: > > $ contrib/ci/scripts/gitlab-pipeline-status --verbose --wait ...but this script

[Bug 1866870] Re: KVM Guest pauses after upgrade to Ubuntu 20.04

2020-03-16 Thread Boris Derzhavets
Verification new packages to be installed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1866870 Title: KVM Guest pauses after upgrade to Ubuntu 20.04 Status in QEMU: New Status in qemu package

Re: [PATCH 04/11] MAINTAINERS: Add an entry for the HVF accelerator

2020-03-16 Thread Philippe Mathieu-Daudé
Hi Roman, On 3/16/20 1:12 PM, Roman Bolshakov wrote: Hi Philippe, I can take the ownership if nobody wants it. At the moment I'm working on APIC for HVF to get kvm-unit-tests fixed. Next items on the list (in no particular order): * MMX emulation * SSE emulation * qxl display * gdb stub *

[PULL 6/6] audio: add audiodev format=f32 option documentation

2020-03-16 Thread Gerd Hoffmann
From: Volker Rümelin The documentaion for -audiodev format=f32 option was missing. Signed-off-by: Volker Rümelin Message-id: 20200308193321.20668-6-vr_q...@t-online.de Signed-off-by: Gerd Hoffmann --- qemu-options.hx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PULL 5/6] audio: fix saturation nonlinearity in clip_* functions

2020-03-16 Thread Gerd Hoffmann
From: Volker Rümelin The current positive limit for the saturation nonlinearity is only correct if the type of the result has 8 bits or less. Signed-off-by: Volker Rümelin Message-id: 20200308193321.20668-5-vr_q...@t-online.de Signed-off-by: Gerd Hoffmann --- audio/mixeng_template.h | 5

[PATCH v2 2/2] lockable: add QemuRecMutex support

2020-03-16 Thread Stefan Hajnoczi
The polymorphic locking macros don't support QemuRecMutex yet. Add it so that lock guards can be used with QemuRecMutex. Convert TCG plugins functions that benefit from these macros. Manual qemu_rec_mutex_lock/unlock() callers are left unmodified in cases where clarity would not improve by

[PATCH 06/11] accel/Kconfig: Extract accel selectors into their own config

2020-03-16 Thread Philippe Mathieu-Daudé
Move the accel selectors from the global Kconfig.host to their own Kconfig file. Signed-off-by: Philippe Mathieu-Daudé --- Makefile | 1 + Kconfig.host | 7 --- accel/Kconfig | 6 ++ 3 files changed, 7 insertions(+), 7 deletions(-) create mode 100644 accel/Kconfig diff --git

[PATCH 05/11] Makefile: Write MINIKCONF variables as one entry per line

2020-03-16 Thread Philippe Mathieu-Daudé
Having one entry per line helps reviews/refactors. As we are going to modify the MINIKCONF variables, split them now to ease further review. Signed-off-by: Philippe Mathieu-Daudé --- Makefile | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makefile index

[PATCH 00/11] accel: Allow targets to use Kconfig, disable semihosting by default

2020-03-16 Thread Philippe Mathieu-Daudé
This series include generic patches I took of the KVM/ARM specific series which will follow. - List orphan accelerators in MAINTAINERS - Add accel/Kconfig - Allow targets to use their how Kconfig - Enforce semihosting on ARM/LM32/MIPS, disable it elsewhere Previous RFC for semihosting posted

Re: [PATCH 0/5] QEMU Gating CI

2020-03-16 Thread Peter Maydell
On Mon, 16 Mar 2020 at 12:04, Cleber Rosa wrote: > A few possible reasons come to my mind: > > 1) It usually takes a few seconds after the push for the pipeline to > > 2) If you've pushed to a repo different than gitlab.com/qemu-project/qemu, >you'd have to tweak the project ID

[PATCH 07/11] accel/Kconfig: Add the TCG selector

2020-03-16 Thread Philippe Mathieu-Daudé
Expose the CONFIG_TCG selector to let minikconf.py uses it. When building with --disable-tcg build, this helps to deselect devices that are TCG-dependent. Signed-off-by: Philippe Mathieu-Daudé --- Makefile | 1 + accel/Kconfig | 3 +++ 2 files changed, 4 insertions(+) diff --git

[PATCH v2] hw/scsi/vmw_pvscsi: Remove assertion for kick after reset

2020-03-16 Thread Liran Alon
From: Elazar Leibovich When running Ubuntu 3.13.0-65-generic guest, QEMU sometimes crashes during guest ACPI reset. It crashes on assert(s->rings_info_valid) in pvscsi_process_io(). Analyzing the crash revealed that it happens when userspace issues a sync during a reboot syscall. Below are

  1   2   3   4   5   6   7   >