Re: [PATCH v5 3/3] ppc: Enable 2nd DAWR support on p10

2021-05-04 Thread David Gibson
On Wed, Apr 21, 2021 at 11:50:40AM +0530, Ravi Bangoria wrote: > Hi David, > > On 4/19/21 10:23 AM, David Gibson wrote: > > On Mon, Apr 12, 2021 at 05:14:33PM +0530, Ravi Bangoria wrote: > > > As per the PAPR, bit 0 of byte 64 in pa-features property indicates > > > availability of 2nd DAWR

Re: [PATCH] vfio/pci: Revert nvlink removal uAPI breakage

2021-05-04 Thread Christoph Hellwig
Looks good, Reviewed-by: Christoph Hellwig

[Bug 1905356] Re: No check for unaligned data access in ARM32 instructions

2021-05-04 Thread Thomas Huth
Richard's patches have been merged (see https://git.qemu.org/?p=qemu.git;a=commitdiff;h=4d753eb5fb03ee7bc71ec and the following ones), so I'm setting the state to "Fix committed" now. ** Changed in: qemu Status: Confirmed => Fix Committed -- You received this bug notification because you

[Bug 1868116] Re: QEMU monitor no longer works

2021-05-04 Thread Christian Ehrhardt 
@Thomas - there is a leftover task here and I've filed [1] for it in the new tracker. What is the right state to move this bug here into now? [1]: https://gitlab.com/qemu-project/qemu/-/issues/137 ** Bug watch added: gitlab.com/qemu-project/qemu/-/issues #137

[Bug 1759522] Re: windows qemu-img create vpc/vhdx error

2021-05-04 Thread Thomas Huth
This is an automated cleanup. This bug report has been moved to QEMU's new bug tracker on gitlab.com and thus gets marked as 'expired' now. Please continue with the discussion here: https://gitlab.com/qemu-project/qemu/-/issues/136 ** Changed in: qemu Status: New => Expired ** Bug

[PATCH 0/7] vhost-user-gpu: fix several security issues

2021-05-04 Thread Li Qiang
These security issue is low severity and is similar with the virtio-vga/virtio-gpu device. All of them can be triggered by the guest user. Li Qiang (7): vhost-user-gpu: fix memory disclosure in virgl_cmd_get_capset_info vhost-user-gpu: fix resource leak in 'vg_resource_create_2d'

[PATCH 4/7] vhost-user-gpu: fix memory link while calling 'vg_resource_unref'

2021-05-04 Thread Li Qiang
If the guest trigger following sequences, the attach_backing will be leaked: vg_resource_create_2d vg_resource_attach_backing vg_resource_unref This patch fix this by freeing 'res->iov' in vg_resource_destroy. Signed-off-by: Li Qiang ---

[PATCH 6/7] vhost-user-gpu: fix memory leak in 'virgl_resource_attach_backing'

2021-05-04 Thread Li Qiang
If 'virgl_renderer_resource_attach_iov' failed, the 'res_iovs' will be leaked. Signed-off-by: Li Qiang --- contrib/vhost-user-gpu/virgl.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/contrib/vhost-user-gpu/virgl.c b/contrib/vhost-user-gpu/virgl.c index

[PATCH 7/7] vhost-user-gpu: fix OOB write in 'virgl_cmd_get_capset'

2021-05-04 Thread Li Qiang
If 'virgl_cmd_get_capset' set 'max_size' to 0, the 'virgl_renderer_fill_caps' will write the data after the 'resp'. This patch avoid this by checking the returned 'max_size'. Signed-off-by: Li Qiang --- contrib/vhost-user-gpu/virgl.c | 4 1 file changed, 4 insertions(+) diff --git

[PATCH 3/7] vhost-user-gpu: fix memory leak in vg_resource_attach_backing

2021-05-04 Thread Li Qiang
Check whether the 'res' has already been attach_backing to avoid memory leak. Signed-off-by: Li Qiang --- contrib/vhost-user-gpu/vhost-user-gpu.c | 5 + 1 file changed, 5 insertions(+) diff --git a/contrib/vhost-user-gpu/vhost-user-gpu.c b/contrib/vhost-user-gpu/vhost-user-gpu.c index

[PATCH 5/7] vhost-user-gpu: fix memory leak in 'virgl_cmd_resource_unref'

2021-05-04 Thread Li Qiang
The 'res->iov' will be leaked if the guest trigger following sequences: virgl_cmd_create_resource_2d virgl_resource_attach_backing virgl_cmd_resource_unref This patch fixes this. Signed-off-by: Li Qiang --- contrib/vhost-user-gpu/virgl.c | 7 +++ 1 file changed, 7

[PATCH 2/7] vhost-user-gpu: fix resource leak in 'vg_resource_create_2d'

2021-05-04 Thread Li Qiang
Call 'vugbm_buffer_destroy' in error path to avoid resource leak. Signed-off-by: Li Qiang --- contrib/vhost-user-gpu/vhost-user-gpu.c | 1 + 1 file changed, 1 insertion(+) diff --git a/contrib/vhost-user-gpu/vhost-user-gpu.c b/contrib/vhost-user-gpu/vhost-user-gpu.c index

[PATCH 1/7] vhost-user-gpu: fix memory disclosure in virgl_cmd_get_capset_info

2021-05-04 Thread Li Qiang
Otherwise some of the 'resp' will be leaked to guest. Signed-off-by: Li Qiang --- contrib/vhost-user-gpu/virgl.c | 1 + 1 file changed, 1 insertion(+) diff --git a/contrib/vhost-user-gpu/virgl.c b/contrib/vhost-user-gpu/virgl.c index 9e6660c7ab..6a332d601f 100644 ---

Re: [PATCH] target/ppc: Do not check for LPCR[HAIL] on power10_v1.0 CPUs

2021-05-04 Thread David Gibson
On Tue, May 04, 2021 at 01:54:39PM +0200, Cédric Le Goater wrote: > On 5/4/21 12:49 PM, Nicholas Piggin wrote: > > Excerpts from Cédric Le Goater's message of May 4, 2021 7:59 pm: > >> The LPCR[HAIL] bit only applies to POWER10 DD2 CPUs. On POWER10 DD1, > >> the ail value should be extracted using

Re: [RFC PATCH v2 0/2] hw/ppc: code motion to compile without TCG

2021-05-04 Thread David Gibson
On Mon, May 03, 2021 at 07:21:11PM -0300, Fabiano Rosas wrote: > "Lucas Mateus Castro (alqotel)" writes: > > > After the feedback from v1 I reworked the patch with suggested ideas and > > this version has less duplicated code and is overall simpler. > > > > This patch series is still a WIP,

Re: [RFC PATCH v2 2/2] hw/ppc: Moved TCG code to spapr_hcall_tcg

2021-05-04 Thread David Gibson
On Tue, May 04, 2021 at 03:14:17PM -0300, Lucas Mateus Martins Araujo e Castro wrote: > > On 03/05/2021 01:34, David Gibson wrote: > > On Fri, Apr 30, 2021 at 03:40:47PM -0300, Lucas Mateus Castro (alqotel) > > wrote: > > > Moved h_enter, remove_hpte, h_remove, h_bulk_remove,h_protect and > > >

Re: [PATCH] target/ppc/spapr: Update H_GET_CPU_CHARACTERISTICS bits

2021-05-04 Thread David Gibson
On Tue, May 04, 2021 at 06:50:54PM +1000, Nicholas Piggin wrote: > Excerpts from David Gibson's message of May 4, 2021 10:41 am: > > On Mon, May 03, 2021 at 10:58:33PM +1000, Nicholas Piggin wrote: > >> There are several new bits added to the hcall which reflect new issues > >> found and new

Re: [PATCH v4 1/5] target/ppc: Fold gen_*_xer into their callers

2021-05-04 Thread David Gibson
On Tue, May 04, 2021 at 11:01:53AM -0300, Bruno Larsen (billionai) wrote: > folded gen_{read,write}_xer into their only callers, spr_{read,write}_xer > > Signed-off-by: Bruno Larsen (billionai) > Reviewed-by: Richard Henderson Good cleanup on its own. Applied to ppc-for-6.1. > --- >

Re: [PATCH v4 3/5] target/ppc: move SPR R/W callbacks to translate.c

2021-05-04 Thread David Gibson
On Tue, May 04, 2021 at 11:01:55AM -0300, Bruno Larsen (billionai) wrote: > Moved all read and write callbacks for SPRs away from > translate_init.c.inc and into translate.c; these functions are > TCG only, so this motion is required to enable building with > the flag disable-tcg > >

Re: [PATCH v4 2/5] target/ppc: renamed SPR registration functions

2021-05-04 Thread David Gibson
On Tue, May 04, 2021 at 11:01:54AM -0300, Bruno Larsen (billionai) wrote: > Renamed all gen_spr_* and gen_* functions specifically related to > registering SPRs to register_*_sprs and register_*, to avoid future > confusion with other TCG related code. > > Signed-off-by: Bruno Larsen (billionai)

Re: [PATCH 0/2] Fix for compat mode in P9 < 2.2

2021-05-04 Thread David Gibson
On Tue, May 04, 2021 at 09:11:28PM -0300, Fabiano Rosas wrote: > We dropped support in KVM for mixing MMU modes in machines that have > old POWER9 processors which don't support mixing. As usual I forgot > that compat mode exists: > > $ lscpu | grep pvr > Model: 2.1 (pvr 004e

[PATCH v8 4/4] target/arm: set ID_AA64ISAR0.TLB to 2 for max AARCH64 CPU type

2021-05-04 Thread Rebecca Cran
Indicate support for FEAT_TLBIOS and FEAT_TLBIRANGE by setting ID_AA64ISAR0.TLB to 2 for the max AARCH64 CPU type. Signed-off-by: Rebecca Cran Reviewed-by: Richard Henderson --- target/arm/cpu64.c | 1 + 1 file changed, 1 insertion(+) diff --git a/target/arm/cpu64.c b/target/arm/cpu64.c index

[PATCH v8 1/4] accel/tcg: Add TLB invalidation support for ranges of addresses

2021-05-04 Thread Rebecca Cran
Add functions to support the FEAT_TLBIRANGE ARMv8.4 feature that adds TLB invalidation instructions to invalidate ranges of addresses. Signed-off-by: Rebecca Cran --- accel/tcg/cputlb.c | 128 +++- include/exec/exec-all.h | 46 +++ 2 files changed, 171 insertions(+), 3

[PATCH v8 3/4] target/arm: Add support for FEAT_TLBIOS

2021-05-04 Thread Rebecca Cran
ARMv8.4 adds the mandatory FEAT_TLBIOS. It provides TLBI maintenance instructions that extend to the Outer Shareable domain. Signed-off-by: Rebecca Cran --- target/arm/cpu.h| 5 ++ target/arm/helper.c | 75 2 files changed, 80 insertions(+) diff --git

[PATCH v8 0/4] aarch64: add support for FEAT_TLBIRANGE and FEAT_TLBIOS

2021-05-04 Thread Rebecca Cran
Improved readability and fixed a bug in tlb_flush_page_range_bits_by_mmuidx_async_0. Rebecca Cran (4): accel/tcg: Add TLB invalidation support for ranges of addresses target/arm: Add support for FEAT_TLBIRANGE target/arm: Add support for FEAT_TLBIOS target/arm: set ID_AA64ISAR0.TLB to 2

[PATCH v8 2/4] target/arm: Add support for FEAT_TLBIRANGE

2021-05-04 Thread Rebecca Cran
ARMv8.4 adds the mandatory FEAT_TLBIRANGE. It provides TLBI maintenance instructions that apply to a range of input addresses. Signed-off-by: Rebecca Cran --- target/arm/cpu.h| 5 + target/arm/helper.c | 296 2 files changed, 301 insertions(+) diff --git

Re: [PATCH 1/3] hw/i2c: add support for PMBus

2021-05-04 Thread Joel Stanley
On Tue, 4 May 2021 at 16:30, Titus Rwantare wrote: > > QEMU has support for SMBus devices, and PMBus is a more specific > implementation of SMBus. The additions made in this commit makes it easier to > add new PMBus devices to QEMU. > > https://pmbus.org/specification-archives/ I'm not a pmbus

[PATCH 0/2] Fix for compat mode in P9 < 2.2

2021-05-04 Thread Fabiano Rosas
We dropped support in KVM for mixing MMU modes in machines that have old POWER9 processors which don't support mixing. As usual I forgot that compat mode exists: $ lscpu | grep pvr Model: 2.1 (pvr 004e 1201) $ ~/qemu-system-ppc64 -machine

Re: [PATCH v4 0/3] nvdimm: Enable sync-dax property for nvdimm

2021-05-04 Thread Dan Williams
On Tue, May 4, 2021 at 2:03 AM Aneesh Kumar K.V wrote: > > On 5/4/21 11:13 AM, Pankaj Gupta wrote: > > > >> > >> What this patch series did was to express that property via a device > >> tree node and guest driver enables a hypercall based flush mechanism to > >> ensure persistence. > > > >

[PATCH 2/2] hw/ppc/spapr.c: Make sure the host supports the selected MMU mode

2021-05-04 Thread Fabiano Rosas
Starting with Linux kernel v5.12 we dropped support[1] in KVM for hosts that can't have their threads running in different MMU modes (POWER9 < DD2.2). In these hosts, KVM will no longer report the KVM_CAP_PPC_MMU_HASH_V3 capability[2] when the host is running Radix. For guests that support both

[PATCH 1/2] hw/ppc/spapr.c: Extract MMU mode error reporting into a function

2021-05-04 Thread Fabiano Rosas
A following patch will make use of it. Signed-off-by: Fabiano Rosas --- hw/ppc/spapr.c | 16 hw/ppc/spapr_hcall.c | 14 ++ include/hw/ppc/spapr.h | 1 + 3 files changed, 19 insertions(+), 12 deletions(-) diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c index

[PATCH v3 1/2] ui/cocoa: capture all keys and combos when mouse is grabbed

2021-05-04 Thread gustavo
From: Gustavo Noronha Silva Applications such as Gnome may use Alt-Tab and Super-Tab for different purposes, some use Ctrl-arrows so we want to allow qemu to handle everything when it captures the mouse/keyboard. However, Mac OS handles some combos like Command-Tab and Ctrl-arrows at an earlier

[PATCH v3 0/2] cocoa: keyboard quality of life

2021-05-04 Thread gustavo
From: Gustavo Noronha Silva v3 removes a rogue ; that made its way into v2 and makes the swap-option-command parameter off by default, so existing behaviour is maintained, as suggested by BALATON Zoltan. - This series adds two new options to the cocoa display: - full-grab causes it to

[PATCH v3 2/2] ui/cocoa: add option to swap Option and Command, enable by default

2021-05-04 Thread gustavo
From: Gustavo Noronha Silva On Mac OS X the Option key maps to Alt and Command to Super/Meta. This change swaps them around so that Alt is the key closer to the space bar and Meta/Super is between Control and Alt, like on non-Mac keyboards. It is a cocoa display option, disabled by default.

Re: [PATCH 1/3] hw/i2c: add support for PMBus

2021-05-04 Thread Titus Rwantare
Hi Phil, It looks big but the bulk of it is simple switch statements and definitions. Unless you mean you'd like it split despite that. Regards, Titus On Tue, 4 May 2021 at 16:49, Philippe Mathieu-Daudé wrote: > > Hi Titus, > > On 5/4/21 6:28 PM, Titus Rwantare wrote: > > QEMU has support for

Re: Gitlab Issue Tracker - Proposed Workflow

2021-05-04 Thread John Snow
On 5/4/21 4:24 PM, Peter Maydell wrote: On Tue, 4 May 2021 at 19:34, John Snow wrote: # Some concerns on the above scheme: - "In Progress" is not likely to be used faithfully and will fall out of sync often. It's not clear if there should be a difference between a bug having an assignee and a

Re: Gitlab Issue Tracker - Proposed Workflow

2021-05-04 Thread John Snow
On 5/4/21 2:52 PM, Daniel P. Berrangé wrote: On Tue, May 04, 2021 at 02:33:58PM -0400, John Snow wrote: I'm seeking feedback on our Gitlab issue handling workflow. (There's a TLDR at the bottom of the mail.) - In Progress: For issues that a developer is actively working on. The intent was to

Re: [PATCH v2 0/4] hw/sparc: Kconfig fixes to build with/without the leon3 machine

2021-05-04 Thread Mark Cave-Ayland
On 28/04/2021 15:16, Philippe Mathieu-Daudé wrote: Missing review: 2-4 Since v1: - move cpu_check_irqs() to target/sparc/ (rth) This series fixes link failure when building either the leon3 machine or the sun4m ones. The problem is we have hardware specific code in the architectural

Re: [PATCH v3 0/6] hw/sparc/sun4m: Introduce Sun4mMachineClass to access sun4m_hwdefs

2021-05-04 Thread Mark Cave-Ayland
On 03/05/2021 18:12, Philippe Mathieu-Daudé wrote: Missing review: 6 Hi Mark, This series QOM'ify the sun4m machines. I need it for a further memory maxsize check. It is mostly code movement (and the diff-stat is good). Since v2: - use static const variable for hwdef (Richard) - added

Re: [RFC PATCH 07/27] virtio-snd: Add properties for class init

2021-05-04 Thread Shreyansh Chouhan
On Wed, 5 May 2021 at 02:00, Laurent Vivier wrote: > Hi Shreyansh, > > First of all, thank you for your work, I was expecting a virtio sound > device for some time... > > You're welcome :) > Le 04/05/2021 à 21:35, Shreyansh Chouhan a écrit : > > On Tue, 4 May 2021 at 19:02, Laurent Vivier

Re: [PATCH 1/3] hw/i2c: add support for PMBus

2021-05-04 Thread Philippe Mathieu-Daudé
Hi Titus, On 5/4/21 6:28 PM, Titus Rwantare wrote: > QEMU has support for SMBus devices, and PMBus is a more specific > implementation of SMBus. The additions made in this commit makes it easier to > add new PMBus devices to QEMU. > > https://pmbus.org/specification-archives/ > > Reviewed-by:

Re: [PATCH v4 0/5] target/ppc: Untangle CPU init from translation

2021-05-04 Thread Fabiano Rosas
"Bruno Larsen (billionai)" writes: > Based-on: ppc-for-6.1 tree > > This patch series aims to remove the logic of initializing CPU from > the file related to TCG translation. To achieve this, we have to make > it so registering SPRs isn't directly tied to TCG, and move code only > related to

Re: [RFC PATCH 07/27] virtio-snd: Add properties for class init

2021-05-04 Thread Laurent Vivier
Hi Shreyansh, First of all, thank you for your work, I was expecting a virtio sound device for some time... Le 04/05/2021 à 21:35, Shreyansh Chouhan a écrit : > On Tue, 4 May 2021 at 19:02, Laurent Vivier > wrote: > > There is nothing specific to PCI in that

Re: Gitlab Issue Tracker - Proposed Workflow

2021-05-04 Thread Peter Maydell
On Tue, 4 May 2021 at 19:34, John Snow wrote: > # Some concerns on the above scheme: > > - "In Progress" is not likely to be used faithfully and will fall out of > sync often. It's not clear if there should be a difference between a bug > having an assignee and a bug labeled "In Progress". I

Re: [PATCH 10/10] qcow2-refcount: check_refblocks(): add separate message for reserved

2021-05-04 Thread Eric Blake
On 5/4/21 10:20 AM, Vladimir Sementsov-Ogievskiy wrote: > Split checking for reserved bits out of aligned offset check. > > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > block/qcow2.h | 1 + > block/qcow2-refcount.c | 10 +- > 2 files changed, 10 insertions(+), 1

Re: [PATCH 09/10] qcow2-refcount: check_refcounts_l1(): check reserved bits

2021-05-04 Thread Eric Blake
On 5/4/21 10:20 AM, Vladimir Sementsov-Ogievskiy wrote: > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > block/qcow2.h | 1 + > block/qcow2-refcount.c | 6 ++ > 2 files changed, 7 insertions(+) > Reviewed-by: Eric Blake -- Eric Blake, Principal Software Engineer Red Hat,

Re: [PATCH 08/10] qcow2-refcount: improve style of check_refcounts_l1()

2021-05-04 Thread Eric Blake
On 5/4/21 10:20 AM, Vladimir Sementsov-Ogievskiy wrote: > - use g_autofree for l1_table > - better name for size in bytes variable > - reduce code blocks nesting > - whitespaces, braces, newlines > > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > block/qcow2-refcount.c | 97

[Bug 1892081] Re: Performance improvement when using "QEMU_FLATTEN" with softfloat type conversions

2021-05-04 Thread Thomas Huth
This is an automated cleanup. This bug report has been moved to QEMU's new bug tracker on gitlab.com and thus gets marked as 'expired' now. Please continue with the discussion here: https://gitlab.com/qemu-project/qemu/-/issues/134 ** Changed in: qemu Status: In Progress => Expired **

Re: [PATCH 07/10] qcow2-refcount: check_refcounts_l2(): check reserved bits

2021-05-04 Thread Eric Blake
On 5/4/21 10:20 AM, Vladimir Sementsov-Ogievskiy wrote: > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > block/qcow2.h | 1 + > block/qcow2-refcount.c | 12 +++- > 2 files changed, 12 insertions(+), 1 deletion(-) > Reviewed-by: Eric Blake -- Eric Blake, Principal

[Bug 1828608] Re: Chardev websocket might not support pasting more than a few chars

2021-05-04 Thread Thomas Huth
This is an automated cleanup. This bug report has been moved to QEMU's new bug tracker on gitlab.com and thus gets marked as 'expired' now. Please continue with the discussion here: https://gitlab.com/qemu-project/qemu/-/issues/133 ** Changed in: qemu Status: New => Expired ** Bug

Re: [PATCH 06/10] qcow2-refcount: check_refcounts_l2(): check l2_bitmap

2021-05-04 Thread Eric Blake
On 5/4/21 10:20 AM, Vladimir Sementsov-Ogievskiy wrote: > Check subcluster bitmap of the l2 entry for different types of > clusters: > > - for compressed it must be zero > - for allocated check consistency of two parts of the bitmap > - for unallocated all subclusters should be unallocated >

Re: [PATCH 05/10] qcow2-refcount: fix_l2_entry_by_zero(): also zero L2 entry bitmap

2021-05-04 Thread Eric Blake
On 5/4/21 10:20 AM, Vladimir Sementsov-Ogievskiy wrote: > We'll reuse the function to fix wrong L2 entry bitmap. Support it now. > > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > block/qcow2-refcount.c | 18 +++--- > 1 file changed, 15 insertions(+), 3 deletions(-) > > diff

[Bug 1861404] Re: AVX instruction VMOVDQU implementation error for YMM registers

2021-05-04 Thread Thomas Huth
This is an automated cleanup. This bug report has been moved to QEMU's new bug tracker on gitlab.com and thus gets marked as 'expired' now. Please continue with the discussion here: https://gitlab.com/qemu-project/qemu/-/issues/132 ** Changed in: qemu Status: New => Expired ** Bug

Re: [RFC PATCH 07/27] virtio-snd: Add properties for class init

2021-05-04 Thread Shreyansh Chouhan
On Tue, 4 May 2021 at 19:02, Laurent Vivier wrote: > There is nothing specific to PCI in that code, why do you prevent the use > of virtio-snd as a MMIO > device? > > I am sorry I do not understand your question completely. If by preventing the use of virtio-snd, you mean why did I add the PCI

Re: [PATCH 04/10] qcow2-refcount: introduce fix_l2_entry_by_zero()

2021-05-04 Thread Eric Blake
On 5/4/21 10:20 AM, Vladimir Sementsov-Ogievskiy wrote: > Split fix_l2_entry_by_zero() out of check_refcounts_l2() to be > reused in further patch. > > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > block/qcow2-refcount.c | 87 +- > 1 file changed,

[Bug 1658141] Re: QEMU's default msrs handling causes Windows 10 64 bit to crash

2021-05-04 Thread Thomas Huth
This is an automated cleanup. This bug report has been moved to QEMU's new bug tracker on gitlab.com and thus gets marked as 'expired' now. Please continue with the discussion here: https://gitlab.com/qemu-project/qemu/-/issues/131 ** Changed in: qemu Status: New => Expired ** Bug

Re: [PATCH 03/10] qcow2: introduce qcow2_parse_compressed_l2_entry() helper

2021-05-04 Thread Eric Blake
On 5/4/21 10:20 AM, Vladimir Sementsov-Ogievskiy wrote: > Add helper to parse compressed l2_entry and use it everywhere instead > of opencoding. open-coding > > Note, that in most places we move to precise coffset/csize instead of > sector-aligned. Still it should work good enough for updating

[Bug 1868116] Re: QEMU monitor no longer works

2021-05-04 Thread Thomas Huth
Looking at the comments, it seems to me that this was an issue in VTE that got fixed. Is there still anything left to do for upstream QEMU here? ** Changed in: qemu Status: New => Incomplete -- You received this bug notification because you are a member of qemu- devel-ml, which is

Re: [PATCH 02/10] qcow2: compressed read: simplify cluster descriptor passing

2021-05-04 Thread Eric Blake
On 5/4/21 10:20 AM, Vladimir Sementsov-Ogievskiy wrote: > Let's pass the whole L2 entry and not bother with > L2E_COMPRESSED_OFFSET_SIZE_MASK. > > It also helps further refactoring that adds generic > qcow2_parse_compressed_l2_entry() helper. > > Signed-off-by: Vladimir Sementsov-Ogievskiy >

Re: [PATCH 01/10] qcow2-refcount: improve style of check_refcounts_l2()

2021-05-04 Thread Eric Blake
On 5/4/21 10:20 AM, Vladimir Sementsov-Ogievskiy wrote: > - don't use same name for size in bytes and in entries > - use g_autofree for l2_table > - add whitespace > - fix block comment style > > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > block/qcow2-refcount.c | 47

Re: [PATCH] migration: do not restart VM after successful snapshot-load

2021-05-04 Thread Dr. David Alan Gilbert
* Paolo Bonzini (pbonz...@redhat.com) wrote: > The HMP loadvm code is calling load_snapshot rather than > qmp_snapshot_load, in order to bypass the job infrastructure. The code > around it is almost the same, with one difference: hmp_loadvm is > restarting the VM if load_snapshot fails,

Re: Gitlab Issue Tracker - Proposed Workflow

2021-05-04 Thread Daniel P . Berrangé
On Tue, May 04, 2021 at 02:33:58PM -0400, John Snow wrote: > I'm seeking feedback on our Gitlab issue handling workflow. > (There's a TLDR at the bottom of the mail.) > > > # Background: > > Last KVM Forum, I started experimenting with the Gitlab issue tracker: >

Gitlab Issue Tracker - Proposed Workflow

2021-05-04 Thread John Snow
I'm seeking feedback on our Gitlab issue handling workflow. (There's a TLDR at the bottom of the mail.) # Background: Last KVM Forum, I started experimenting with the Gitlab issue tracker: https://gitlab.com/qemu-project/qemu/-/issues At the time, I hastily added some labels to roughly

Re: [RFC PATCH v2 2/2] hw/ppc: Moved TCG code to spapr_hcall_tcg

2021-05-04 Thread Lucas Mateus Martins Araujo e Castro
On 03/05/2021 01:34, David Gibson wrote: On Fri, Apr 30, 2021 at 03:40:47PM -0300, Lucas Mateus Castro (alqotel) wrote: Moved h_enter, remove_hpte, h_remove, h_bulk_remove,h_protect and h_read to spapr_hcall_tcg.c, added h_tcg_only to be used in a !TCG environment in spapr_hcall.c and changed

Re: [PATCH v11 5/6] KVM: arm64: ioctl to fetch/store tags in a guest

2021-05-04 Thread Catalin Marinas
On Thu, Apr 29, 2021 at 05:06:07PM +0100, Steven Price wrote: > On 27/04/2021 18:58, Catalin Marinas wrote: > > On Fri, Apr 16, 2021 at 04:43:08PM +0100, Steven Price wrote: > > > diff --git a/arch/arm64/include/uapi/asm/kvm.h > > > b/arch/arm64/include/uapi/asm/kvm.h > > > index

Re: [PATCH v11 2/6] arm64: kvm: Introduce MTE VM feature

2021-05-04 Thread Catalin Marinas
On Thu, Apr 29, 2021 at 05:06:41PM +0100, Steven Price wrote: > On 28/04/2021 18:07, Catalin Marinas wrote: > > I probably asked already but is the only way to map a standard RAM page > > (not device) in stage 2 via the fault handler? One case I had in mind > > was something like get_user_pages()

Re: [PATCH] migration: do not restart VM after successful snapshot-load

2021-05-04 Thread Daniel P . Berrangé
On Tue, May 04, 2021 at 12:58:26PM -0400, Paolo Bonzini wrote: > The HMP loadvm code is calling load_snapshot rather than > qmp_snapshot_load, in order to bypass the job infrastructure. The code > around it is almost the same, with one difference: hmp_loadvm is > restarting the VM if

[Bug 1857226] Re: 'set_link net0 off' not working with e1000e driver

2021-05-04 Thread Jannik
Still reproducible on Fedora 34 with QEMU 5.2.0. ** Changed in: qemu Status: Incomplete => New -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1857226 Title: 'set_link net0 off' not working

Re: [PATCH] migration: do not restart VM after successful snapshot-load

2021-05-04 Thread Eric Blake
On 5/4/21 11:58 AM, Paolo Bonzini wrote: > The HMP loadvm code is calling load_snapshot rather than > qmp_snapshot_load, in order to bypass the job infrastructure. The code > around it is almost the same, with one difference: hmp_loadvm is > restarting the VM if load_snapshot fails,

[PATCH] migration: do not restart VM after successful snapshot-load

2021-05-04 Thread Paolo Bonzini
The HMP loadvm code is calling load_snapshot rather than qmp_snapshot_load, in order to bypass the job infrastructure. The code around it is almost the same, with one difference: hmp_loadvm is restarting the VM if load_snapshot fails, qmp_snapshot_load is doing so if load_snapshot succeeds. Fix

Re: [PATCH] qcow2: set bdi->is_dirty

2021-05-04 Thread Kirill Tkhai
On 04.05.2021 19:06, Vladimir Sementsov-Ogievskiy wrote: > Set bdi->is_dirty, so that qemu-img info could show dirty flag. > > After this commit the following check will show '"dirty-flag": true': > > ./build/qemu-img create -f qcow2 -o lazy_refcounts=on x 1M > ./build/qemu-io x > qemu-io> write

[PATCH 1/2] Consistent function names for sifive uart read and write function

2021-05-04 Thread Lukas Jünger
Signed-off-by: Lukas Jünger --- hw/char/sifive_uart.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/hw/char/sifive_uart.c b/hw/char/sifive_uart.c index 3a00ba7f00..cb70374ead 100644 --- a/hw/char/sifive_uart.c +++ b/hw/char/sifive_uart.c @@ -65,7 +65,7 @@

Re: [PATCH v4 4/5] target/ppc: turned SPR R/W callbacks not static

2021-05-04 Thread Richard Henderson
On 5/4/21 7:01 AM, Bruno Larsen (billionai) wrote: To be able to compile translate_init.c.inc as a standalone file, we have to make the callbacks accessible outside of translate.c; This patch does exactly that Signed-off-by: Bruno Larsen (billionai) --- target/ppc/spr_tcg.h | 134

Re: [RESEND PATCH 05/32] vl: Add "sgx-epc" option to expose SGX EPC sections to guest

2021-05-04 Thread Paolo Bonzini
On 04/05/21 18:20, Sean Christopherson wrote: If it's just CPUID, one possibility could be to mark the EPC sections specially in KVM_SET_USER_MEMORY_REGION and synthesize the leaves within KVM; or even look inside the VMA structs and detect EPC regions that way. I experimented with those

[PATCH 0/2] QOMify Sifive UART model

2021-05-04 Thread Lukas Jünger
Hello, This patch QOMifies the Sifive UART model. It is split into two commits. The first commit makes the sifive uart read and write function names more consistent. The second commit QOMifies the model. Looking forward to hear from you, Lukas Lukas Jünger (2): Consistent function names for

[PATCH 2/3] hw/misc: add ADM1272 device

2021-05-04 Thread Titus Rwantare
The ADM1272 is a PMBus compliant Hot Swap Controller and Digital Power Monitor by Analog Devices. This commit adds support for interfacing with it, and support for setting and monitoring sensor limits. Datasheet: https://www.analog.com/media/en/technical-documentation/data-sheets/ADM1272.pdf

[PATCH 2/2] QOMify sifive_uart model

2021-05-04 Thread Lukas Jünger
Signed-off-by: Lukas Jünger --- include/hw/char/sifive_uart.h | 6 +-- hw/char/sifive_uart.c | 72 ++- 2 files changed, 65 insertions(+), 13 deletions(-) diff --git a/include/hw/char/sifive_uart.h b/include/hw/char/sifive_uart.h index

[PATCH 3/3] hw/misc: add MAX34451 device

2021-05-04 Thread Titus Rwantare
The MAX34451 is a Maxim power-supply system manager that can monitor up to 16 voltage rails or currents. It also contains a temperature sensor and supports up to four external temperature sensors. This commit adds support for interfacing with it, and setting limits on the supported sensors.

Re: remove the nvlink2 pci_vfio subdriver v2

2021-05-04 Thread Jason Gunthorpe
On Tue, May 04, 2021 at 04:23:40PM +0200, Daniel Vetter wrote: > Just my 2cents from drm (where we deprecate old gunk uapi quite often): > Imo it's best to keep the uapi headers as-is, but exchange the > documentation with a big "this is removed, never use again" warning: We in RDMA have been

Re: remove the nvlink2 pci_vfio subdriver v2

2021-05-04 Thread Daniel Vetter
On Tue, May 04, 2021 at 12:53:27PM -0300, Jason Gunthorpe wrote: > On Tue, May 04, 2021 at 04:23:40PM +0200, Daniel Vetter wrote: > > > Just my 2cents from drm (where we deprecate old gunk uapi quite often): > > Imo it's best to keep the uapi headers as-is, but exchange the > > documentation with

[PATCH 1/3] hw/i2c: add support for PMBus

2021-05-04 Thread Titus Rwantare
QEMU has support for SMBus devices, and PMBus is a more specific implementation of SMBus. The additions made in this commit makes it easier to add new PMBus devices to QEMU. https://pmbus.org/specification-archives/ Reviewed-by: Hao Wu Signed-off-by: Titus Rwantare --- hw/arm/Kconfig

[PATCH 0/3] Add support for PMBus in QEMU

2021-05-04 Thread Titus Rwantare
Hello, This patch series adds an interface to start supporting PMBus devices in QEMU. I’ve included two PMBus devices: MAX34451 and ADM1272. PMBus is a variant of SMBus meant for digital management of power supplies. PMBus adds to the SMBus standard by defining a number of constants and commands

Re: [PATCH] qcow2: set bdi->is_dirty

2021-05-04 Thread Eric Blake
On 5/4/21 11:06 AM, Vladimir Sementsov-Ogievskiy wrote: > Set bdi->is_dirty, so that qemu-img info could show dirty flag. > > After this commit the following check will show '"dirty-flag": true': > > ./build/qemu-img create -f qcow2 -o lazy_refcounts=on x 1M > ./build/qemu-io x > qemu-io> write

Re: [RESEND PATCH 05/32] vl: Add "sgx-epc" option to expose SGX EPC sections to guest

2021-05-04 Thread Sean Christopherson
On Tue, May 04, 2021, Paolo Bonzini wrote: > On 04/05/21 02:09, Sean Christopherson wrote: > > Is there a way to process "-device sgx-epc..." before vCPUs are realized? > > The > > ordering problem was the only reason I added a dedicated option. > > If it's just CPUID, one possibility could be

Re: [PATCH v4 3/5] target/ppc: move SPR R/W callbacks to translate.c

2021-05-04 Thread Richard Henderson
On 5/4/21 7:01 AM, Bruno Larsen (billionai) wrote: Moved all read and write callbacks for SPRs away from translate_init.c.inc and into translate.c; these functions are TCG only, so this motion is required to enable building with the flag disable-tcg Signed-off-by: Bruno Larsen (billionai)

Re: [PATCH v6 11/12] qtest/bios-tables-test: Make test build-independent from accelerator

2021-05-04 Thread Philippe Mathieu-Daudé
On 5/3/21 11:19 PM, Eric Blake wrote: > On 5/3/21 4:10 PM, Philippe Mathieu-Daudé wrote: >> Now than we can probe if the TCG accelerator is available > > that Oops. I'll wait for Igor review before respinning. > >> at runtime with a QMP command, do it once at the beginning >> and only register

[PATCH] qcow2: set bdi->is_dirty

2021-05-04 Thread Vladimir Sementsov-Ogievskiy
Set bdi->is_dirty, so that qemu-img info could show dirty flag. After this commit the following check will show '"dirty-flag": true': ./build/qemu-img create -f qcow2 -o lazy_refcounts=on x 1M ./build/qemu-io x qemu-io> write 0 1M After "write" command success, kill the qemu-io process: kill

Re: [PATCH] vfio/pci: Revert nvlink removal uAPI breakage

2021-05-04 Thread Greg Kurz
On Tue, 04 May 2021 09:52:02 -0600 Alex Williamson wrote: > Revert the uAPI changes from the below commit with notice that these > regions and capabilities are no longer provided. > > Fixes: b392a1989170 ("vfio/pci: remove vfio_pci_nvlink2") > Reported-by: Greg Kurz > Signed-off-by: Alex

Re: [PATCH] vfio/pci: Revert nvlink removal uAPI breakage

2021-05-04 Thread Cornelia Huck
On Tue, 04 May 2021 09:52:02 -0600 Alex Williamson wrote: > Revert the uAPI changes from the below commit with notice that these > regions and capabilities are no longer provided. > > Fixes: b392a1989170 ("vfio/pci: remove vfio_pci_nvlink2") > Reported-by: Greg Kurz > Signed-off-by: Alex

Re: [PULL 0/5] bsd-user: minor cleanup patches

2021-05-04 Thread Peter Maydell
On Fri, 30 Apr 2021 at 18:46, Warner Losh wrote: > > The following changes since commit ffa090bc56e73e287a63261e70ac02c0970be61a: > > target/s390x: fix s390_probe_access to check PAGE_WRITE_ORG for > writeability (2021-04-23 14:10:56 +0100) > > are available in the Git repository at: > >

Re: [PATCH 4/7] tests/acceptance: Sun4uMachine: Remove dependency to LinuxKernelTest

2021-05-04 Thread Philippe Mathieu-Daudé
Hi Wainer, On 5/4/21 12:43 AM, Wainer dos Santos Moschetta wrote: > The Sun4uMachine class inherit from LinuxKernelTest to effectively only use > the KERNEL_COMMON_COMMAND_LINE attribute. This change remove that unneeded > dependency, making Sun4uMachine self-content. It is odd because the test

Re: [PATCH v4 2/5] target/ppc: renamed SPR registration functions

2021-05-04 Thread Richard Henderson
On 5/4/21 7:01 AM, Bruno Larsen (billionai) wrote: Renamed all gen_spr_* and gen_* functions specifically related to registering SPRs to register_*_sprs and register_*, to avoid future confusion with other TCG related code. Signed-off-by: Bruno Larsen (billionai) ---

Re: [RFC PATCH v2 0/2] hw/ppc: code motion to compile without TCG

2021-05-04 Thread Lucas Mateus Martins Araujo e Castro
Thanks, it will be quite helpful. Also, I agree with Bruno including this information somewhere would be quite good in my opinion. From: Fabiano Rosas Sent: Monday, May 3, 2021 7:21 PM To: Lucas Mateus Martins Araujo e Castro ; qemu-devel@nongnu.org ;

[Bug 1828867] Re: QEmu translation is incorrect when using REX in combination with LAHF/SAHF

2021-05-04 Thread Thomas Huth
This is an automated cleanup. This bug report has been moved to QEMU's new bug tracker on gitlab.com and thus gets marked as 'expired' now. Please continue with the discussion here: https://gitlab.com/qemu-project/qemu/-/issues/130 ** Changed in: qemu Status: New => Expired ** Bug

[PATCH] vfio/pci: Revert nvlink removal uAPI breakage

2021-05-04 Thread Alex Williamson
Revert the uAPI changes from the below commit with notice that these regions and capabilities are no longer provided. Fixes: b392a1989170 ("vfio/pci: remove vfio_pci_nvlink2") Reported-by: Greg Kurz Signed-off-by: Alex Williamson --- Greg (Kurz), please double check this resolves the issue.

Re: [PATCH v11 1/6] arm64: mte: Sync tags for pages where PTE is untagged

2021-05-04 Thread Catalin Marinas
On Thu, Apr 29, 2021 at 05:06:05PM +0100, Steven Price wrote: > On 27/04/2021 18:43, Catalin Marinas wrote: > > On Fri, Apr 16, 2021 at 04:43:04PM +0100, Steven Price wrote: > > > diff --git a/arch/arm64/include/asm/pgtable.h > > > b/arch/arm64/include/asm/pgtable.h > > > index

Re: [PATCH] virtio-blk: drop deprecated scsi=on|off property

2021-05-04 Thread Eduardo Habkost
On Tue, May 04, 2021 at 03:32:55PM +0100, Stefan Hajnoczi wrote: > On Thu, Apr 29, 2021 at 02:03:52PM -0400, Eduardo Habkost wrote: > > On Thu, Apr 29, 2021 at 04:52:21PM +0100, Stefan Hajnoczi wrote: > > > Live migrating old guests from an old QEMU with the SCSI feature bit > > > enabled will

Re: [PATCH v3 03/26] DAX: vhost-user: Rework slave return values

2021-05-04 Thread Stefan Hajnoczi
On Wed, Apr 28, 2021 at 12:00:37PM +0100, Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > All the current slave handlers on the qemu side generate an 'int' > return value that's squashed down to a bool (!!ret) and stuffed into > a uint64_t (field of a union) to be

Re: [PATCH v3 04/26] DAX: libvhost-user: Route slave message payload

2021-05-04 Thread Stefan Hajnoczi
On Wed, Apr 28, 2021 at 12:00:38PM +0100, Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > Route the uint64 payload from message replies on the slave back up > through vu_process_message_reply and to the callers. > > Signed-off-by: Dr. David Alan Gilbert > Reviewed-by:

Re: remove the nvlink2 pci_vfio subdriver v2

2021-05-04 Thread Alex Williamson
On Tue, 4 May 2021 16:11:31 +0200 Greg Kurz wrote: > On Tue, 4 May 2021 15:30:15 +0200 > Greg Kroah-Hartman wrote: > > > On Tue, May 04, 2021 at 03:20:34PM +0200, Greg Kurz wrote: > > > On Tue, 4 May 2021 14:59:07 +0200 > > > Greg Kroah-Hartman wrote: > > > > > > > On Tue, May 04, 2021

  1   2   3   4   >