Re: [PATCH v1 1/2] hw/i386: Make pit a property of common x86 base machine type

2022-01-11 Thread Xiaoyao Li
+ Paolo On 1/11/2022 3:35 PM, Xiaoyao Li wrote: Both pc and microvm have pit property individually. Let's just make it the property of common x86 base machine type. Signed-off-by: Xiaoyao Li --- hw/i386/microvm.c | 27 +-- hw/i386/pc.c | 24

[PATCH v2 5/5] tests/tcg/s390x: Test shift instructions

2022-01-11 Thread Ilya Leoshkevich
Add a test for each shift instruction in order to to prevent regressions. Signed-off-by: Ilya Leoshkevich --- tests/tcg/s390x/Makefile.target | 1 + tests/tcg/s390x/shift.c | 258 2 files changed, 259 insertions(+) create mode 100644

[PATCH v2 3/5] target/s390x: Fix cc_calc_sla_64() missing overflows

2022-01-11 Thread Ilya Leoshkevich
An overflow occurs for SLAG when at least one shifted bit is not equal to sign bit. Therefore, we need to check that `shift + 1` bits are neither all 0s nor all 1s. The current code checks only `shift` bits, missing some overflows. Fixes: cbe24bfa91d2 ("target-s390: Convert SHIFT, ROTATE SINGLE")

Re: [PATCH v10 0/7] Add vmnet.framework based network backend

2022-01-11 Thread Roman Bolshakov
On Wed, Jan 12, 2022 at 12:14:15AM +0300, Vladislav Yaroshchuk wrote: > macOS provides networking API for VMs called 'vmnet.framework': > https://developer.apple.com/documentation/vmnet > > We can provide its support as the new QEMU network backends which > represent three different

[RFC v4 20/21] vfio-user: migration support

2022-01-11 Thread John Johnson
Signed-off-by: John G Johnson Signed-off-by: Elena Ufimtseva Signed-off-by: Jagannathan Raman --- hw/vfio/user-protocol.h | 18 + hw/vfio/migration.c | 30 +-- hw/vfio/pci.c | 7 +++ hw/vfio/user.c | 54

Re: [PATCH v3 00/10] hw/dma: Use dma_addr_t type definition when relevant

2022-01-11 Thread Peter Xu
On Tue, Jan 11, 2022 at 07:42:59PM +0100, Philippe Mathieu-Daudé wrote: > Since v2: > - Split meson patch restricting fw_cfg (Richard) > - Reorder pci_dma_map() docstring (Peter, Richard) > - Move QEMUSGList in previous patch (David) > - Have dma_buf_read/dma_buf_write return dma_addr_t (Peter) >

Re: [PULL 00/13] Net patches

2022-01-11 Thread Jason Wang
在 2022/1/12 上午6:02, Vladislav Yaroshchuk 写道: вт, 11 янв. 2022 г., 5:10 AM Jason Wang : On Tue, Jan 11, 2022 at 12:49 AM Peter Maydell wrote: > > On Mon, 10 Jan 2022 at 03:40, Jason Wang wrote: > > > > The following changes since commit

Re: [PULL 00/13] Net patches

2022-01-11 Thread Vladislav Yaroshchuk
ср, 12 янв. 2022 г., 8:39 AM Jason Wang : > > 在 2022/1/12 上午6:02, Vladislav Yaroshchuk 写道: > > > > > > вт, 11 янв. 2022 г., 5:10 AM Jason Wang : > > > > On Tue, Jan 11, 2022 at 12:49 AM Peter Maydell > > wrote: > > > > > > On Mon, 10 Jan 2022 at 03:40, Jason Wang > > wrote:

Re: [PATCH] tests: Fix typo in check-help output

2022-01-11 Thread wangyanan (Y)
On 2022/1/12 1:55, Philippe Mathieu-Daudé wrote: Fix typo in 'make check-help' output. Signed-off-by: Philippe Mathieu-Daudé --- tests/Makefile.include | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/Makefile.include b/tests/Makefile.include index

[PATCH v2] usb: allow max 8192 bytes for desc

2022-01-11 Thread zhenwei pi
A device of USB video class usually uses larger desc structure, so use larger buffer to avoid failure. (dev-video.c is ready) This is an unlikely code path: 1, during guest startup, guest tries to probe device. 2, run 'lsusb' command in guest(or other similar commands). Reviewed-by: Daniel P.

RE: [RFC PATCH 6/7] x86: Use new XSAVE ioctls handling

2022-01-11 Thread Wang, Wei W
On Wednesday, January 12, 2022 10:51 AM, Zeng, Guang wrote: > To: Tian, Kevin ; Zhong, Yang ; > qemu-devel@nongnu.org > Cc: pbonz...@redhat.com; Christopherson,, Sean ; > jing2@linux.intel.com; Wang, Wei W > Subject: Re: [RFC PATCH 6/7] x86: Use new XSAVE ioctls handling > > On 1/11/2022

Re: [PATCH] clock-vmstate: Add missing END_OF_LIST

2022-01-11 Thread Luc Michel
On 10:19 Tue 11 Jan , Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > Add the missing VMSTATE_END_OF_LIST to vmstate_muldiv > > Fixes: 99abcbc7600 ("clock: Provide builtin multiplier/divider") > Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Luc Michel > ---

Re: [PATCH v2] vl: Add support to set properties when using JSON syntax for -device via -set option

2022-01-11 Thread MkfsSion
ping https://lore.kernel.org/qemu-devel/20211224072511.63894-1-mkfss...@mkfssion.com

Re: [PATCH v1 2/2] hw/i386: Make pic a property of common x86 base machine type

2022-01-11 Thread Xiaoyao Li
+ Paolo On 1/11/2022 3:35 PM, Xiaoyao Li wrote: Legacy PIC (8259) cannot be supported for TDX guests since TDX module doesn't allow directly interrupt injection. Using posted interrupts for the PIC is not a viable option as the guest BIOS/kernel will not do EOI for PIC IRQs, i.e. will leave

Re: [PATCH] qdev-core.h: Fix wrongly named reference to TYPE_SPLIT_IRQ

2022-01-11 Thread wangyanan (Y)
On 2022/1/12 1:26, Peter Maydell wrote: Fix a comment in qdev-core.h where we incorrectly referred to TYPE_IRQ_SPLIT when we meant TYPE_SPLIT_IRQ. Signed-off-by: Peter Maydell --- include/hw/qdev-core.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH v6 09/23] target/riscv: Implement AIA local interrupt priorities

2022-01-11 Thread Frank Chang
On Wed, Jan 12, 2022 at 1:18 AM Anup Patel wrote: > > > On Mon, Jan 10, 2022 at 6:38 PM Frank Chang > wrote: > > > > Anup Patel 於 2021年12月30日 週四 下午8:38寫道: > >> > >> From: Anup Patel > >> > >> The AIA spec defines programmable 8-bit priority for each local > interrupt > >> at M-level, S-level

[PATCH v2 1/5] target/s390x: Fix SLDA sign bit index

2022-01-11 Thread Ilya Leoshkevich
David Hildenbrand noticed that sign bit index for SLDA is wrong: since SLDA operates on 64-bit values, it should be 63, not 31. Fixes: a79ba3398a0a ("target-s390: Convert SHIFT DOUBLE") Signed-off-by: Ilya Leoshkevich --- target/s390x/tcg/insn-data.def | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH v2 0/5] target/s390x: Fix shift instructions

2022-01-11 Thread Ilya Leoshkevich
v1: https://lists.nongnu.org/archive/html/qemu-devel/2022-01/msg02035.html v1 -> v2: Fix cc_calc_sla_32(). Fix cc_calc_sla_64(). Fix SLDA sign bit index. Inline help_l2_shift(). Fix wout_r1_D32(). Add all shift instructions to the test.

[RFC v4 16/21] vfio-user: dma map/unmap operations

2022-01-11 Thread John Johnson
Add ability to do async operations during memory transactions Signed-off-by: Jagannathan Raman Signed-off-by: Elena Ufimtseva Signed-off-by: John G Johnson --- hw/vfio/user-protocol.h | 32 +++ include/hw/vfio/vfio-common.h | 9 +- hw/vfio/common.c | 63 +---

Re: [RFC PATCH 6/7] x86: Use new XSAVE ioctls handling

2022-01-11 Thread Zeng Guang
On 1/11/2022 10:30 AM, Tian, Kevin wrote: From: Zeng, Guang Sent: Monday, January 10, 2022 5:47 PM On 1/10/2022 4:40 PM, Tian, Kevin wrote: From: Zhong, Yang Sent: Friday, January 7, 2022 5:32 PM From: Jing Liu Extended feature has large state while current kvm_xsave only allows 4KB. Use

Re: [PULL 00/13] Net patches

2022-01-11 Thread Roman Bolshakov
On Wed, Jan 12, 2022 at 01:39:28PM +0800, Jason Wang wrote: > > 在 2022/1/12 上午6:02, Vladislav Yaroshchuk 写道: > > > > > > вт, 11 янв. 2022 г., 5:10 AM Jason Wang : > > > > On Tue, Jan 11, 2022 at 12:49 AM Peter Maydell > > wrote: > > > > > > On Mon, 10 Jan 2022 at 03:40, Jason

Re: [PATCH] target/s390x: Fix 32-bit shifts

2022-01-11 Thread Ilya Leoshkevich
On Tue, 2022-01-11 at 15:22 +0100, David Hildenbrand wrote: > On 10.01.22 19:59, Ilya Leoshkevich wrote: > > Both 32- and 64-bit shifts use lowest 6 address bits. The current > > code > > special-cases 32-bit shifts to use only 5 bits, which is not > > correct. > > > > I assume for 32-bit

[PATCH v2 4/5] target/s390x: Fix shifting 32-bit values for more than 31 bits

2022-01-11 Thread Ilya Leoshkevich
According to PoP, both 32- and 64-bit shifts use lowest 6 address bits. The current code special-cases 32-bit shifts to use only 5 bits, which is not correct. For example, shifting by 32 bits currently preserves the initial value, however, it's supposed zero it out instead. Fix by merging sh32

Re: [RFC PATCH v3 5/7] audio/coreaudio: Remove a deprecation warning on macOS 12

2022-01-11 Thread Roman Bolshakov
On Mon, Jan 10, 2022 at 02:09:59PM +0100, Philippe Mathieu-Daudé wrote: > When building on macOS 12 we get: > > audio/coreaudio.c:50:5: error: 'kAudioObjectPropertyElementMaster' is > deprecated: first deprecated in macOS 12.0 [-Werror,-Wdeprecated-declarations] >

Re: [PULL 00/13] Net patches

2022-01-11 Thread Jason Wang
On Wed, Jan 12, 2022 at 3:10 PM Roman Bolshakov wrote: > > On Wed, Jan 12, 2022 at 01:39:28PM +0800, Jason Wang wrote: > > > > 在 2022/1/12 上午6:02, Vladislav Yaroshchuk 写道: > > > > > > > > > вт, 11 янв. 2022 г., 5:10 AM Jason Wang : > > > > > > On Tue, Jan 11, 2022 at 12:49 AM Peter Maydell >

[PATCH v2 2/5] target/s390x: Fix SRDA CC calculation

2022-01-11 Thread Ilya Leoshkevich
SRDA uses r1_D32 for binding the first operand and s64 for setting CC. cout_s64() relies on o->out being the shift result, however, wout_r1_D32() clobbers it. Fix by using a temporary. Fixes: a79ba3398a0a ("target-s390: Convert SHIFT DOUBLE") Signed-off-by: Ilya Leoshkevich ---

Re: [PULL 00/13] Net patches

2022-01-11 Thread Jason Wang
On Wed, Jan 12, 2022 at 2:19 PM Vladislav Yaroshchuk wrote: > > > > ср, 12 янв. 2022 г., 8:39 AM Jason Wang : >> >> >> 在 2022/1/12 上午6:02, Vladislav Yaroshchuk 写道: >> > >> > >> > вт, 11 янв. 2022 г., 5:10 AM Jason Wang : >> > >> > On Tue, Jan 11, 2022 at 12:49 AM Peter Maydell >> >

Re: [PATCH v9 05/10] ACPI ERST: support for ACPI ERST feature

2022-01-11 Thread Ani Sinha
On Tue, Dec 14, 2021 at 2:33 AM Eric DeVolder wrote: > > Ani, an inline response below. > Thanks! > eric > > On 12/10/21 08:09, Ani Sinha wrote: > > On Thu, Dec 9, 2021 at 11:24 PM Eric DeVolder > > wrote: > >> > >> Ani, inline responses below. eric > >> > >> On 12/9/21 00:29, Ani Sinha wrote:

Re: [PATCH] target/ppc: Fix e6500 boot

2022-01-11 Thread ma...@locati.it
>From: "Cédric Le Goater" c...@kaod.org >To: "ma...@locati.it" ma...@locati.it, bala...@eik.bme.hu >Cc: faro...@linux.ibm.com, qemu-devel@nongnu.org, qemu-...@nongnu.org, >danielhb...@gmail.com >Date: Mon, 10 Jan 2022 09:04:39 +0100 >Subject: Re: [PATCH] target/ppc: Fix e6500 boot > >Hello Mario,

Re: [PATCH v3 2/2] This patch includes i3c instance in ast2600 soc.

2022-01-11 Thread Cédric Le Goater
On 1/11/22 09:45, Troy Lee wrote: v3: - Remove unrelated changes to SPI2 address - Remove controller irq line v2: Rebase to mainline QEMU Signed-off-by: Troy Lee Reviewed-by: Cédric Le Goater You should consider updating the documentation :

Re: [PATCH v2] linux-user: Fix clang warning for nios2-linux-user code

2022-01-11 Thread Laurent Vivier
Le 11/01/2022 à 09:29, Peter Maydell a écrit : The clang in Ubuntu 18.04 (10.0.0-4ubuntu1) produces a warning on the code added in commit f5ef0e518d03 where we use a shifted expression in a boolean context: ../../linux-user/elfload.c:2423:16: error: converting the result of '<<' to a boolean

Re: [RFC PATCH v2 20/44] i386/tdx: Parse tdx metadata and store the result into TdxGuestState

2022-01-11 Thread Laszlo Ersek
On 01/10/22 13:09, Xiaoyao Li wrote: > On 1/10/2022 7:01 PM, Gerd Hoffmann wrote: If you go without pflash, then you likely will not have a standards-conformant UEFI variable store. (Unless you reimplement the variable arch protocols in edk2 on top of something else than the

Re: [PATCH] linux-user: Fix clang warning for nios2-linux-user code

2022-01-11 Thread Peter Maydell
On Tue, 11 Jan 2022 at 07:01, Laurent Vivier wrote: > > Le 10/01/2022 à 20:17, Peter Maydell a écrit : > > The clang in Ubuntu 18.04 (10.0.0-4ubuntu1) produces a warning > > on the code added in commit f5ef0e518d03 where we use a > > shifted expression in a boolean context: > > > >

[PATCH v2] linux-user: Fix clang warning for nios2-linux-user code

2022-01-11 Thread Peter Maydell
The clang in Ubuntu 18.04 (10.0.0-4ubuntu1) produces a warning on the code added in commit f5ef0e518d03 where we use a shifted expression in a boolean context: ../../linux-user/elfload.c:2423:16: error: converting the result of '<<' to a boolean always evaluates to true

Re: [PATCH 0/8] target/ppc: powerpc_excp improvements [40x] (3/n)

2022-01-11 Thread Cédric Le Goater
On 1/10/22 19:15, Fabiano Rosas wrote: This is the first series of the exception model-specific changes. I intend to keep this same structure for the rest of the exception models. == preparation == First few patches are cleanups specific to this model. This comes first because I'm using some

Re: [PATCH] softmmu/device_tree: Remove redundant pointer assignment

2022-01-11 Thread Andrew Jones
On Tue, Jan 11, 2022 at 11:27:58AM +0800, Yanan Wang wrote: > The pointer assignment "const char *p = path;" in function > qemu_fdt_add_path is unnecessary. Let's remove it and just > use the "path" passed in. No functional change. > > Suggested-by: Richard Henderson > Signed-off-by: Yanan Wang

Re: [PATCH v2 19/37] console: save current scanout details

2022-01-11 Thread Marc-André Lureau
Hi Akihiko On Tue, Jan 11, 2022 at 7:30 AM Akihiko Odaki wrote: > > Hi, > > I found this brings an inconsistency and a flaw to scanout semantics and > think the inconsistency should be fixed or this should be reverted > before the next release comes up. > > The inconsistency is in the handling

[PATCH 1/1] util: adjust coroutine pool size to virtio block queue

2022-01-11 Thread Hiroki Narukawa
Coroutine pool size was 64 from long ago, and the basis was organized in the commit message in c740ad92. At that time, virtio-blk queue-size and num-queue were not configuable, and equivalent values were 128 and 1. Coroutine pool size 64 was fine then. Later queue-size and num-queue got

[PATCH 0/1 v2] Patch to adjust coroutine pool size adaptively

2022-01-11 Thread Hiroki Narukawa
Resending patch with fixing atomic access method to use qatomic_read(). We encountered random disk IO performance drop since qemu-5.0.0, and this patch fixes it. Commit message in c740ad92 implied to adjust coroutine pool size adaptively, so I tried to implement this. Could you review this

Re: [PATCH v3 0/2] Aspeed I3C device model

2022-01-11 Thread Cédric Le Goater
On 1/11/22 09:45, Troy Lee wrote: This series of patch introduce a dummy implemenation of aspeed i3c model, and it provide just enough information for guest machine. However, the driver probing is still failed, but it will not cause kernel panic. v3: - Remove unused AspeedI3CClass - Refine

Re: [PATCH 2/2] block/rbd: workaround for ceph issue #53784

2022-01-11 Thread Peter Lieven
Am 10.01.22 um 15:18 schrieb Stefano Garzarella: > On Mon, Jan 10, 2022 at 12:41:54PM +0100, Peter Lieven wrote: >> librbd had a bug until early 2022 that affected all versions of ceph that >> supported fast-diff. This bug results in reporting of incorrect offsets >> if the offset parameter to

Re: [PATCH v2 1/2] hw/misc: Implementating dummy AST2600 I3C model

2022-01-11 Thread Cédric Le Goater
Hello Troy, + +memory_region_init_io(>iomem, OBJECT(s), _i3c_ops, s, +TYPE_ASPEED_I3C, ASPEED_I3C_NR_REGS << 2); + +sysbus_init_mmio(sbd, >iomem); I would add a container region containing all the regions : memory_region_init(>iomem_container, OBJECT(s),

Re: [PATCH] ui/cocoa: capture screencast with AVAssetWriter

2022-01-11 Thread Peter Maydell
On Tue, 11 Jan 2022 at 07:09, Zhang Chen wrote: > > To record screencast, AVAssetWriter APIs were called for each > cocoa_update call. > > Commands for start/stop recording were added to View menu. This seems a bit of an odd feature -- why doesn't the OS just permit screen recording of any

[PATCH v3 1/2] Introduce a dummy AST2600 I3C model.

2022-01-11 Thread Troy Lee
Aspeed 2600 SDK enables I3C support by default. The I3C driver will try to reset the device controller and setup through device address table register. This dummy model response these register with default value listed on ast2600v10 datasheet chapter 54.2. If the device address table register

[PATCH v3 2/2] This patch includes i3c instance in ast2600 soc.

2022-01-11 Thread Troy Lee
v3: - Remove unrelated changes to SPI2 address - Remove controller irq line v2: Rebase to mainline QEMU Signed-off-by: Troy Lee --- hw/arm/aspeed_ast2600.c | 16 include/hw/arm/aspeed_soc.h | 3 +++ 2 files changed, 19 insertions(+) diff --git a/hw/arm/aspeed_ast2600.c

[PATCH v3 0/2] Aspeed I3C device model

2022-01-11 Thread Troy Lee
This series of patch introduce a dummy implemenation of aspeed i3c model, and it provide just enough information for guest machine. However, the driver probing is still failed, but it will not cause kernel panic. v3: - Remove unused AspeedI3CClass - Refine memory region - Refine register reset -

Re: [PATCH 0/3] virtio-iommu: Support VIRTIO_IOMMU_F_BYPASS_CONFIG

2022-01-11 Thread Eric Auger
Hi Jean, Michael, On 9/30/21 8:50 PM, Jean-Philippe Brucker wrote: > Replace the VIRTIO_IOMMU_F_BYPASS feature with > VIRTIO_IOMMU_F_BYPASS_CONFIG, which enables a config space bit to switch > global bypass on and off. > > Add a boot-bypass option, which defaults to 'on' to be in line with >

Re: [PATCH v3 1/2] Introduce a dummy AST2600 I3C model.

2022-01-11 Thread Cédric Le Goater
On 1/11/22 09:45, Troy Lee wrote: Aspeed 2600 SDK enables I3C support by default. The I3C driver will try to reset the device controller and setup through device address table register. This dummy model response these register with default value listed on ast2600v10 datasheet chapter 54.2. If

Re: [PATCH v4 4/5] ppc/pnv: Introduce user creatable pnv-phb4 devices

2022-01-11 Thread Daniel Henrique Barboza
On 1/11/22 06:56, Cédric Le Goater wrote: On 1/11/22 01:55, Daniel Henrique Barboza wrote: This patch introduces pnv-phb4 user creatable devices that are created in a similar manner as pnv-phb3 devices, allowing the user to interact with the PHBs directly instead of creating PCI Express

Re: [PATCH 0/3] virtio-iommu: Support VIRTIO_IOMMU_F_BYPASS_CONFIG

2022-01-11 Thread Eric Auger
Hi Jean, On 1/11/22 11:13 AM, Jean-Philippe Brucker wrote: > Hi Eric, > > On Tue, Jan 11, 2022 at 10:02:12AM +0100, Eric Auger wrote: >> Hi Jean, Michael, >> >> On 9/30/21 8:50 PM, Jean-Philippe Brucker wrote: >>> Replace the VIRTIO_IOMMU_F_BYPASS feature with >>> VIRTIO_IOMMU_F_BYPASS_CONFIG,

Re: [PATCH v4 2/7] tcg/arm: Remove use_armv5t_instructions

2022-01-11 Thread Peter Maydell
On Sat, 8 Jan 2022 at 06:33, Richard Henderson wrote: > > This is now always true, since we require armv6. > > Signed-off-by: Richard Henderson Reviewed-by: Peter Maydell thanks -- PMM

Re: [PULL 0/2] SD/MMC patches for 2022-01-08

2022-01-11 Thread Peter Maydell
On Mon, 10 Jan 2022 at 19:20, Philippe Mathieu-Daudé wrote: > > On 1/10/22 17:02, Peter Maydell wrote: > > On Sat, 8 Jan 2022 at 21:59, Philippe Mathieu-Daudé wrote: > >> > >> Hi Richard, > >> > >> This is the SD/MMC PR that ought to be sent previously. > >> > >> The following changes since

Re: [PATCH v4 7/7] tcg/arm: Support raising sigbus for user-only

2022-01-11 Thread Peter Maydell
On Sat, 8 Jan 2022 at 06:33, Richard Henderson wrote: > > Signed-off-by: Richard Henderson > --- > +static void tcg_out_test_alignment(TCGContext *s, bool is_ld, TCGReg addrlo, > + TCGReg addrhi, unsigned a_bits) > @@ -1709,6 +1778,10 @@ static void

Re: [PATCH] usb: allow max 8192 bytes for desc

2022-01-11 Thread Daniel P . Berrangé
On Tue, Jan 11, 2022 at 12:21:42PM +, Peter Maydell wrote: > On Tue, 11 Jan 2022 at 10:54, zhenwei pi wrote: > > > > A device of USB video class usually uses larger desc structure, so > > use larger buffer to avoid failure. (dev-video.c is ready) > > > > Allocating memory dynamically by

Re: Re: [PATCH] usb: allow max 8192 bytes for desc

2022-01-11 Thread zhenwei pi
On 1/11/22 8:25 PM, Daniel P. Berrangé wrote: On Tue, Jan 11, 2022 at 12:21:42PM +, Peter Maydell wrote: On Tue, 11 Jan 2022 at 10:54, zhenwei pi wrote: A device of USB video class usually uses larger desc structure, so use larger buffer to avoid failure. (dev-video.c is ready)

[PATCH v3 3/6] migration: ram_release_pages() always receive 1 page as argument

2022-01-11 Thread Juan Quintela
Remove the pages argument. And s/pages/page/ Signed-off-by: Juan Quintela Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Peter Xu -- - Use 1LL instead of casts (philmd) - Change the whole 1ULL for TARGET_PAGE_SIZE Signed-off-by: Juan Quintela --- migration/ram.c | 8 1 file

Re: [PATCH v2 1/5] libvhost-user: Add vu_rem_mem_reg input validation

2022-01-11 Thread Stefan Hajnoczi
On Mon, Jan 10, 2022 at 07:43:06PM +, Raphael Norwitz wrote: > On Mon, Jan 10, 2022 at 04:36:34AM -0500, Michael S. Tsirkin wrote: > > On Thu, Jan 06, 2022 at 06:47:26AM +, Raphael Norwitz wrote: > > > Signed-off-by: Raphael Norwitz > > > > > > Raphael any chance you can add a bit more

RE: [PATCH 1/1] util: adjust coroutine pool size to virtio block queue

2022-01-11 Thread Hiroki Narukawa
> On Thu, Jan 06, 2022 at 05:20:57PM +0900, Hiroki Narukawa wrote: > > Phil, thanks for notifying me. > > > Coroutine pool size was 64 from long ago, and the basis was organized in > > the commit message in c740ad92. > > > > At that time, virtio-blk queue-size and num-queue were not

Re: "Startup" meeting (was Re: Meeting today?)

2022-01-11 Thread Mark Burton
That is my understanding… See you there! Cheers Mark. > On 11 Jan 2022, at 11:20, Philippe Mathieu-Daudé wrote: > > Hi, > > Just checking in, this call is scheduled for today, 3pm CEST, right? > > Here is the KVM call calendar: >

Re: [PATCH v2 1/9] hw/nvram: Restrict stub to sysemu and tools

2022-01-11 Thread Philippe Mathieu-Daudé
On 1/7/22 01:17, Richard Henderson wrote: > On 1/4/22 12:54 AM, Philippe Mathieu-Daudé wrote: >> +if have_system or have_tools > ... >>   if have_system >> +  stub_ss.add(files('fw_cfg.c')) > > Disconnect in tests? (I assume s/tests/stubs/). fw_cfg-interface.c exposes the fw_cfg QOM interface,

Re: [PATCH v4 3/7] tcg/arm: Remove use_armv6_instructions

2022-01-11 Thread Peter Maydell
On Sat, 8 Jan 2022 at 06:33, Richard Henderson wrote: > > This is now always true, since we require armv6. > > Signed-off-by: Richard Henderson Reviewed-by: Peter Maydell thanks -- PMM

[PATCH v3 5/6] migration: simplify do_compress_ram_page

2022-01-11 Thread Juan Quintela
The goto is not needed at all. Signed-off-by: Juan Quintela --- migration/ram.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/migration/ram.c b/migration/ram.c index fa49d22e69..422c6bce28 100644 --- a/migration/ram.c +++ b/migration/ram.c @@ -1341,12 +1341,11

Re: [PATCH v4 4/5] ppc/pnv: Introduce user creatable pnv-phb4 devices

2022-01-11 Thread Cédric Le Goater
On 1/11/22 01:55, Daniel Henrique Barboza wrote: This patch introduces pnv-phb4 user creatable devices that are created in a similar manner as pnv-phb3 devices, allowing the user to interact with the PHBs directly instead of creating PCI Express Controllers that will create a certain amount of

Re: [PATCH v4 5/5] ppc/pnv: turn pnv_phb4_update_regions() into static

2022-01-11 Thread Cédric Le Goater
On 1/11/22 01:55, Daniel Henrique Barboza wrote: Its only callers are inside pnv_phb4.c. Signed-off-by: Daniel Henrique Barboza Reviewed-by: Cédric Le Goater Thanks, C. --- hw/pci-host/pnv_phb4.c | 52 +- include/hw/pci-host/pnv_phb4.h | 1 -

Re: [PATCH] clock-vmstate: Add missing END_OF_LIST

2022-01-11 Thread Peter Maydell
On Tue, 11 Jan 2022 at 10:19, Dr. David Alan Gilbert (git) wrote: > > From: "Dr. David Alan Gilbert" > > Add the missing VMSTATE_END_OF_LIST to vmstate_muldiv > > Fixes: 99abcbc7600 ("clock: Provide builtin multiplier/divider") > Signed-off-by: Dr. David Alan Gilbert > --- >

[PATCH] usb: allow max 8192 bytes for desc

2022-01-11 Thread zhenwei pi
A device of USB video class usually uses larger desc structure, so use larger buffer to avoid failure. (dev-video.c is ready) Allocating memory dynamically by g_malloc of the orignal version of this change, Philippe suggested just using the stack. Test the two versions of qemu binary, the size of

Re: [PATCH v2] block-backend: prevent dangling BDS pointers across aio_poll()

2022-01-11 Thread Stefan Hajnoczi
On Wed, Dec 15, 2021 at 04:31:26PM +0100, Kevin Wolf wrote: > Am 15.12.2021 um 12:28 hat Stefan Hajnoczi geschrieben: > > On Tue, Dec 14, 2021 at 03:59:49PM +0100, Kevin Wolf wrote: > > > Am 14.12.2021 um 15:35 hat Stefan Hajnoczi geschrieben: > > > > The BlockBackend root child can change when

Re: [PATCH v4 1/7] tcg/arm: Drop support for armv4 and armv5 hosts

2022-01-11 Thread Peter Maydell
On Sat, 8 Jan 2022 at 06:33, Richard Henderson wrote: > > Support for unaligned accesses is difficult for pre-v6 hosts. > While debian still builds for armv4, we cannot use a compile > time test, so test the architecture at runtime and error out. > > Signed-off-by: Richard Henderson

Re: [PATCH v4 4/7] tcg/arm: Check alignment for ldrd and strd

2022-01-11 Thread Peter Maydell
On Sat, 8 Jan 2022 at 06:33, Richard Henderson wrote: > > We will shortly allow the use of unaligned memory accesses, > and these require proper alignment. Use get_alignment_bits > to verify and remove USING_SOFTMMU. > > Signed-off-by: Richard Henderson > --- > tcg/arm/tcg-target.c.inc | 22

Re: [PULL v2 00/10] virtio: revert config interrupt changes

2022-01-11 Thread Peter Maydell
On Mon, 10 Jan 2022 at 21:04, Michael S. Tsirkin wrote: > > Changes from v1: > added tags to commit logs. > > The following changes since commit ca745d2277496464b54fd832c15c45d0227325bb: > > tests: acpi: Add updated TPM related tables (2022-01-07 19:30:13 -0500) > > are available in the Git

Re: [PATCH v4 6/7] tcg/arm: Reserve a register for guest_base

2022-01-11 Thread Peter Maydell
On Sat, 8 Jan 2022 at 06:33, Richard Henderson wrote: > > Reserve a register for the guest_base using aarch64 for reference. > By doing so, we do not have to recompute it for every memory load. > > Signed-off-by: Richard Henderson > --- > tcg/arm/tcg-target.c.inc | 39

[PATCH v1] util/oslib-posix: Fix missing unlock in the error path of os_mem_prealloc()

2022-01-11 Thread David Hildenbrand
We're missing an unlock in case installing the signal handler failed. Fortunately, we barely see this error in real life. Fixes: a960d6642d39 ("util/oslib-posix: Support concurrent os_mem_prealloc() invocation") Fixes: CID 1468941 Cc: Paolo Bonzini Cc: Michael S. Tsirkin Cc: Pankaj Gupta Cc:

Re: [PATCH 4/5] usb: allow max 8192 bytes for desc

2022-01-11 Thread Daniel P . Berrangé
On Mon, Dec 27, 2021 at 10:27:33PM +0800, zhenwei pi wrote: > A device of USB video class usually uses larger desc structure, so > use larger buffer to avoid failure. > > Signed-off-by: zhenwei pi > --- > hw/usb/desc.c | 15 --- > hw/usb/desc.h | 1 + > 2 files changed, 9

[PATCH v1] libvhost-user: Map shared RAM with MAP_NORESERVE to support virtio-mem with hugetlb

2022-01-11 Thread David Hildenbrand
For fd-based shared memory, MAP_NORESERVE is only effective for hugetlb, otherwise it's ignored. Older Linux versions that didn't support reservation of huge pages ignored MAP_NORESERVE completely. The first client to mmap a hugetlb fd without MAP_NORESERVE will trigger reservation of huge pages

Re: [PATCH v2 19/37] console: save current scanout details

2022-01-11 Thread Akihiko Odaki
On 2022/01/11 17:23, Marc-André Lureau wrote: Hi Akihiko On Tue, Jan 11, 2022 at 7:30 AM Akihiko Odaki wrote: Hi, I found this brings an inconsistency and a flaw to scanout semantics and think the inconsistency should be fixed or this should be reverted before the next release comes up.

[PATCH v3 2/6] migration: We only need last_stage in two places

2022-01-11 Thread Juan Quintela
We only need last_stage in two places and we are passing it all around. Just add a field to RAMState that passes it. Signed-off-by: Juan Quintela Reviewed-by: Peter Xu --- Repeat subject (philmd suggestion) --- migration/ram.c | 41 ++--- 1 file changed,

[PATCH v3 0/6] migration: misc cleanups

2022-01-11 Thread Juan Quintela
Hi Changes since v2: - rebase - address comments - use TARGET_PAGE_SIZE and remove casts - remove TARGET_PAGE_MASK from compression code. Nothing else use them. And if offsets are not aligned, we are in big trouble. Please, review. Changes since v1: - Add reviewed tags for reviewed patches

Re: [PATCH v4 1/5] ppc/pnv: set phb4 properties in stk_realize()

2022-01-11 Thread Cédric Le Goater
On 1/11/22 01:55, Daniel Henrique Barboza wrote: Moving all phb4 properties setup to stk_realize() keeps this logic in a single place instead of having it scattered between stk_realize() and pec_realize(). 'phb->index' can be retrieved using stack->stack_no and pnv_phb4_pec_get_phb_id(),

Re: [PATCH v4 2/5] ppc/pnv: move PHB4 XSCOM init to phb4_realize()

2022-01-11 Thread Cédric Le Goater
On 1/11/22 01:55, Daniel Henrique Barboza wrote: The 'stack->phb_regs_mr' PHB4 passthrough XSCOM initialization relies on 'stack->phb' being not NULL. Moving 'stack->phb_regs_mr' region_init() and add_subregion() to phb4_realize() time is a natural thing to do since it's strictly PHB related.

Re: [PATCH v4 3/5] ppc/pnv: turn 'phb' into a pointer in struct PnvPhb4PecStack

2022-01-11 Thread Cédric Le Goater
On 1/11/22 01:55, Daniel Henrique Barboza wrote: At this moment, stack->phb is the plain PnvPHB4 device itself instead of a pointer to the device. This will present a problem when adding user creatable devices because we can't deal with this struct and the realize() callback from the user

Re: [PATCH 0/3] virtio-iommu: Support VIRTIO_IOMMU_F_BYPASS_CONFIG

2022-01-11 Thread Jean-Philippe Brucker
Hi Eric, On Tue, Jan 11, 2022 at 10:02:12AM +0100, Eric Auger wrote: > Hi Jean, Michael, > > On 9/30/21 8:50 PM, Jean-Philippe Brucker wrote: > > Replace the VIRTIO_IOMMU_F_BYPASS feature with > > VIRTIO_IOMMU_F_BYPASS_CONFIG, which enables a config space bit to switch > > global bypass on and

[PATCH] clock-vmstate: Add missing END_OF_LIST

2022-01-11 Thread Dr. David Alan Gilbert (git)
From: "Dr. David Alan Gilbert" Add the missing VMSTATE_END_OF_LIST to vmstate_muldiv Fixes: 99abcbc7600 ("clock: Provide builtin multiplier/divider") Signed-off-by: Dr. David Alan Gilbert --- hw/core/clock-vmstate.c | 1 + 1 file changed, 1 insertion(+) diff --git a/hw/core/clock-vmstate.c

Re: [PATCH] usb: allow max 8192 bytes for desc

2022-01-11 Thread Philippe Mathieu-Daudé
On 1/11/22 11:49, zhenwei pi wrote: > A device of USB video class usually uses larger desc structure, so > use larger buffer to avoid failure. (dev-video.c is ready) > > Allocating memory dynamically by g_malloc of the orignal version of > this change, Philippe suggested just using the stack.

[PATCH v3 1/6] migration: All this fields are unsigned

2022-01-11 Thread Juan Quintela
So printing it as %d is wrong. Notice that for the channel id, that is an uint8_t, but I changed it anyways for consistency. Signed-off-by: Juan Quintela Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Peter Xu --- migration/multifd-zlib.c | 20 ++-- migration/multifd-zstd.c

Re: [PULL v4 00/18] Build system and KVM changes for 2021-12-23

2022-01-11 Thread Peter Maydell
On Mon, 10 Jan 2022 at 15:01, Paolo Bonzini wrote: > > The following changes since commit afe33262585565b64df706c62b4b0f6e0ad30c71: > > Merge tag 'pull-riscv-to-apply-20220108' of github.com:alistair23/qemu into > staging (2022-01-07 22:09:24 -0800) > > are available in the Git repository at:

Re: "Startup" meeting (was Re: Meeting today?)

2022-01-11 Thread Philippe Mathieu-Daudé
Hi, Just checking in, this call is scheduled for today, 3pm CEST, right? Here is the KVM call calendar: https://www.google.com/calendar/embed?src=dG9iMXRqcXAzN3Y4ZXZwNzRoMHE4a3BqcXNAZ3JvdXAuY2FsZW5kYXIuZ29vZ2xlLmNvbQ On 1/6/22 12:23, Daniel P. Berrangé wrote: > No one objected, so I think we

Re: "Startup" meeting (was Re: Meeting today?)

2022-01-11 Thread Daniel P . Berrangé
On Tue, Jan 11, 2022 at 11:20:54AM +0100, Philippe Mathieu-Daudé wrote: > Hi, > > Just checking in, this call is scheduled for today, 3pm CEST, right? > > Here is the KVM call calendar: >

Re: [PATCH v1 1/2] hw/i386: Make pit a property of common x86 base machine type

2022-01-11 Thread Sergio Lopez
On Tue, Jan 11, 2022 at 03:35:27PM +0800, Xiaoyao Li wrote: > Both pc and microvm have pit property individually. Let's just make it > the property of common x86 base machine type. > > Signed-off-by: Xiaoyao Li > --- > hw/i386/microvm.c | 27 +-- > hw/i386/pc.c

Re: [PATCH v1 2/2] hw/i386: Make pic a property of common x86 base machine type

2022-01-11 Thread Sergio Lopez
On Tue, Jan 11, 2022 at 03:35:28PM +0800, Xiaoyao Li wrote: > Legacy PIC (8259) cannot be supported for TDX guests since TDX module > doesn't allow directly interrupt injection. Using posted interrupts > for the PIC is not a viable option as the guest BIOS/kernel will not > do EOI for PIC IRQs,

Re: [PATCH 2/2] block/rbd: workaround for ceph issue #53784

2022-01-11 Thread Stefano Garzarella
Hi Peter, On Tue, Jan 11, 2022 at 10:10:16AM +0100, Peter Lieven wrote: Hi Stefano, thanks for the feedback. Please note that you also need the other patch or you will sooner or later run into another assertion as soon as rbd snapshots are involved. Yep, I tested with the entire series

Re: [PATCH v4 5/7] tcg/arm: Support unaligned access for softmmu

2022-01-11 Thread Peter Maydell
On Sat, 8 Jan 2022 at 06:33, Richard Henderson wrote: > > From armv6, the architecture supports unaligned accesses. > All we need to do is perform the correct alignment check > in tcg_out_tlb_read. > > Signed-off-by: Richard Henderson > --- > tcg/arm/tcg-target.c.inc | 39

[PATCH] Fix null pointer dereference in util/fdmon-epoll.c

2022-01-11 Thread Daniella Lee
Orginal qemu commit hash: de3f5223fa4cf8bfc5e3fe1fd495ddf468edcdf7 In util/fdmon-epoll.c, function fdmon_epoll_update, variable "old_node" maybe NULL with the condition, while it is directly used in the statement and may lead to null pointer dereferencen problem. Variable "r" in the condition is

[PATCH v2 00/13] arm gicv3 ITS: Various bug fixes and refactorings

2022-01-11 Thread Peter Maydell
I've been working on the ITS to add support for the GICv4 functionality. In the course of that I found a handful of bugs in it and also some places where the code benefited from refactoring to make it a better base to put in the GICv4 parts. This patchset is just the bugfixes and cleanups, because

[PATCH v2 02/13] hw/intc/arm_gicv3_its: Convert int ID check to num_intids convention

2022-01-11 Thread Peter Maydell
The bounds check on the number of interrupt IDs is correct, but doesn't match our convention; change the variable name, initialize it to the 2^n value rather than (2^n)-1, and use >= instead of > in the comparison. Signed-off-by: Peter Maydell Reviewed-by: Alex Bennée ---

[PATCH v2 08/13] hw/intc/arm_gicv3_its: Fix return codes in process_mapti()

2022-01-11 Thread Peter Maydell
Fix process_mapti() to consistently return CMD_STALL for memory errors and CMD_CONTINUE for parameter errors, as we claim in the comments that we do. Signed-off-by: Peter Maydell Reviewed-by: Alex Bennée Reviewed-by: Richard Henderson --- hw/intc/arm_gicv3_its.c | 28

Re: [PULL 06/26] target/ppc: Improve logging in Radix MMU

2022-01-11 Thread Cédric Le Goater
On 1/11/22 17:51, Peter Maydell wrote: On Tue, 4 Jan 2022 at 07:31, Cédric Le Goater wrote: Signed-off-by: Cédric Le Goater Reviewed-by: Richard Henderson Message-Id: <20211222071002.1568894-1-...@kaod.org> Signed-off-by: Cédric Le Goater --- target/ppc/mmu-radix64.c | 55

[PATCH] hw/arm: kudo add lm75s behind bus 1 switch at 75

2022-01-11 Thread Patrick Venture
Reviewed-by: Hao Wu Signed-off-by: Patrick Venture --- hw/arm/npcm7xx_boards.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/hw/arm/npcm7xx_boards.c b/hw/arm/npcm7xx_boards.c index 7d0f3148be..d701e5cc55 100644 --- a/hw/arm/npcm7xx_boards.c +++

[PATCH v2 09/13] hw/intc/arm_gicv3_its: Fix return codes in process_mapc()

2022-01-11 Thread Peter Maydell
Fix process_mapc() to consistently return CMD_STALL for memory errors and CMD_CONTINUE for parameter errors, as we claim in the comments that we do. Signed-off-by: Peter Maydell Reviewed-by: Alex Bennée Reviewed-by: Richard Henderson --- hw/intc/arm_gicv3_its.c | 8 +++- 1 file changed, 3

Re: [PATCH v4 07/14] vfio-user: run vfio-user context

2022-01-11 Thread Stefan Hajnoczi
On Mon, Jan 10, 2022 at 05:56:25PM +, John Levon wrote: > On Thu, Jan 06, 2022 at 01:35:32PM +, Stefan Hajnoczi wrote: > > > > > >> +static void vfu_object_attach_ctx(void *opaque) > > > > >> +{ > > > > >> +VfuObject *o = opaque; > > > > >> +GPollFD pfds[1]; > > > > >> +int

Re: [PATCH] clock-vmstate: Add missing END_OF_LIST

2022-01-11 Thread Peter Maydell
On Tue, 11 Jan 2022 at 10:19, Dr. David Alan Gilbert (git) wrote: > > From: "Dr. David Alan Gilbert" > > Add the missing VMSTATE_END_OF_LIST to vmstate_muldiv > > Fixes: 99abcbc7600 ("clock: Provide builtin multiplier/divider") > Signed-off-by: Dr. David Alan Gilbert > --- >

  1   2   3   4   >