Re: [Qemu-devel] [PATCH 0/7] Delete 16 *_cpu_class_by_name() functions

Eduardo Habkost writes: > On Wed, May 08, 2019 at 10:34:44AM +0200, Markus Armbruster wrote: >> Eduardo Habkost writes: >> >> > On Mon, May 06, 2019 at 01:53:28PM +0200, Markus Armbruster wrote: >> >> Eduardo Habkost writes: >> >> >> >> > This series adds a new CPUClass::class_name_format

Re: [Qemu-devel] [PATCH v2 1/2] vfio/mdev: add version attribute for mdev device

On Thu, May 09, 2019 at 11:38:34AM +0800, Alex Williamson wrote: > On Wed, 8 May 2019 23:10:55 -0400 > Yan Zhao wrote: > > > On Thu, May 09, 2019 at 05:22:42AM +0800, Alex Williamson wrote: > > > On Wed, 8 May 2019 07:27:40 -0400 > > > Yan Zhao wrote: > > > > > > > On Wed, May 08, 2019 at

Re: [Qemu-devel] [PATCH] configure: Require python3 >= 3.5

Eduardo Habkost writes: > The oldest python3 version in distros that will be supported by > QEMU 4.1 is 3.5.3 (the one in Debian Stretch). Error out if > running python3 < 3.5. > > We have a .travis.yml job configured to use Python 3.4. Change > it to use Python 3.5. > > Signed-off-by: Eduardo

Re: [Qemu-devel] [PATCH 2/3] spapr/xive: fix EQ page addresses above 64GB

On Wed, May 08, 2019 at 07:19:45PM +0200, Cédric Le Goater wrote: > The high order bits of the address of the OS event queue is stored in > bits [4-31] of word2 of the XIVE END internal structures and the low > order bits in word3. This structure is using Big Endian ordering and > computing the

Re: [Qemu-devel] [PATCH v2] target/ppc: Fix xxspltib

On Thu, May 09, 2019 at 06:17:33AM +1000, Anton Blanchard wrote: > xxspltib raises a VMX or a VSX exception depending on the register > set it is operating on. We had a check, but it was backwards. > > Fixes: f113283525a4 ("target-ppc: add xxspltib instruction") > Signed-off-by: Anton Blanchard

Re: [Qemu-devel] [PATCH 1/3] spapr/xive: EQ page should be naturally aligned

On Wed, May 08, 2019 at 07:19:44PM +0200, Cédric Le Goater wrote: > When the OS configures the EQ page in which to receive event > notifications from the XIVE interrupt controller, the page should be > naturally aligned. Add this check. > > Signed-off-by: Cédric Le Goater Applied, thanks. >

Re: [Qemu-devel] [PATCH 3/3] spapr/xive: print out the EQ page address in the monitor

On Wed, May 08, 2019 at 07:19:46PM +0200, Cédric Le Goater wrote: > This proved to be a useful information when debugging issues with OS > event queues allocated above 64GB. > > Signed-off-by: Cédric Le Goater Applied, thanks. > --- > hw/intc/spapr_xive.c | 5 +++-- > 1 file changed, 3

Re: [Qemu-devel] [PATCH 4/9] target/ppc: Fix lxvw4x, lxvh8x and lxvb16x

On Thu, May 09, 2019 at 10:33:24AM +1000, Anton Blanchard wrote: > Hi Mark, > > > Following on from this I've just gone through the load/store > > operations once again and spotted two things: > > > > > > 1) VSX_LOAD_SCALAR_DS has an extra get_cpu_vsrh() which can be removed > > > > diff --git

Re: [Qemu-devel] [PATCH] configure: Require python3 >= 3.5

On 08/05/2019 20.23, Eduardo Habkost wrote: > The oldest python3 version in distros that will be supported by > QEMU 4.1 is 3.5.3 (the one in Debian Stretch). Error out if > running python3 < 3.5. > > We have a .travis.yml job configured to use Python 3.4. Change > it to use Python 3.5. > >

Re: [Qemu-devel] [PULL v2 00/28] Kconfig for Arm machines

On 08/05/2019 18.45, Philippe Mathieu-Daudé wrote: > [clicked ctrl+enter too fast] > > On Wed, May 8, 2019 at 6:43 PM Philippe Mathieu-Daudé > wrote: >> On 5/8/19 5:33 PM, Thomas Huth wrote: >>> On 08/05/2019 17.09, Peter Maydell wrote: On Tue, 7 May 2019 at 14:45, Thomas Huth wrote:

Re: [Qemu-devel] [PATCH 26/26] tcg: Use tlb_fill probe from tlb_vaddr_to_host

On 4/29/19 10:41 AM, Peter Maydell wrote: > On Wed, 3 Apr 2019 at 05:05, Richard Henderson > wrote: >> >> Most of the existing users would continue around a loop which >> would fault the tlb entry in via a normal load/store. But for >> SVE we have a true non-faulting case which requires the new

Re: [Qemu-devel] [PULL 10/19] tests/boot_linux_console: increase timeout

Hi, > > Tests can also timeout due to slow downloads of test kernels. > > Any chance to run the downloads without timeout? > > I acknowledge this is an issue, and have thought about two possible > ways to solve it: > > 1) Downloading/caching/checking all the test assets in a job "pre-tests"

Re: [Qemu-devel] [PATCH v2 1/2] vfio/mdev: add version attribute for mdev device

On Wed, 8 May 2019 23:10:55 -0400 Yan Zhao wrote: > On Thu, May 09, 2019 at 05:22:42AM +0800, Alex Williamson wrote: > > On Wed, 8 May 2019 07:27:40 -0400 > > Yan Zhao wrote: > > > > > On Wed, May 08, 2019 at 05:18:26AM +0800, Alex Williamson wrote: > > > > On Sun, 5 May 2019 21:49:04

Re: [Qemu-devel] [PATCH v2 1/2] vfio/mdev: add version attribute for mdev device

On Thu, May 09, 2019 at 05:22:42AM +0800, Alex Williamson wrote: > On Wed, 8 May 2019 07:27:40 -0400 > Yan Zhao wrote: > > > On Wed, May 08, 2019 at 05:18:26AM +0800, Alex Williamson wrote: > > > On Sun, 5 May 2019 21:49:04 -0400 > > > Yan Zhao wrote: > > > > > > > version attribute is used

Re: [Qemu-devel] [PATCH 00/11] kvm/migration: support KVM_CLEAR_DIRTY_LOG

On Wed, May 08, 2019 at 01:55:07PM +0200, Paolo Bonzini wrote: > On 08/05/19 06:39, Peter Xu wrote: > >> The disadvantage of this is that you won't clear in the kernel those > >> dirty bits that come from other sources (e.g. vhost or > >> address_space_map). This can lead to double-copying of

Re: [Qemu-devel] [PATCH 17/26] target/s390x: Convert to CPUClass::tlb_fill

On 4/3/19 4:17 AM, David Hildenbrand wrote: >> +/* >> + * Note that handle_mmu_fault sets ilen to either 2 (for code) > This comment no longer matches. > >> + * or AUTO (for data). We can resolve AUTO now, as if it was >> + * set to UNWIND -- that will have been done via

[Qemu-devel] [PATCH v2] target/ppc: Fix xvabs[sd]p, xvnabs[sd]p, xvneg[sd]p, xvcpsgn[sd]p

We were using set_cpu_vsr*() when we should have used get_cpu_vsr*(). Fixes: 8b3b2d75c7c0 ("introduce get_cpu_vsr{l,h}() and set_cpu_vsr{l,h}() helpers for VSR register access") Signed-off-by: Anton Blanchard --- target/ppc/translate/vsx-impl.inc.c | 4 ++-- 1 file changed, 2 insertions(+), 2

Re: [Qemu-devel] [PATCH 23/26] target/xtensa: Convert to CPUClass::tlb_fill

On Tue, Apr 30, 2019 at 2:07 PM Max Filippov wrote: > On Tue, Apr 30, 2019 at 11:14 AM Max Filippov wrote: > > On Tue, Apr 30, 2019 at 10:44 AM Richard Henderson > > > And Peter's right that I should have kept EXC_USER. > > It appears to work as is: the EXC_USER is set up by the >

Re: [Qemu-devel] [PATCH 4/9] target/ppc: Fix lxvw4x, lxvh8x and lxvb16x

Hi Mark, > Following on from this I've just gone through the load/store > operations once again and spotted two things: > > > 1) VSX_LOAD_SCALAR_DS has an extra get_cpu_vsrh() which can be removed > > diff --git a/target/ppc/translate/vsx-impl.inc.c > b/target/ppc/translate/vsx-impl.inc.c

[Qemu-devel] [PATCH] target/ppc: Optimise VSX_LOAD_SCALAR_DS and VSX_VECTOR_LOAD_STORE

A few small optimisations: In VSX_LOAD_SCALAR_DS() we can don't need to read the VSR via get_cpu_vsrh(). Split VSX_VECTOR_LOAD_STORE() into two functions. Loads only need to write the VSRs (set_cpu_vsr*()) and stores only need to read the VSRs (get_cpu_vsr*()) Thanks to Mark Cave-Ayland for the

[Qemu-devel] [PATCH] Multiple ramfb enhancements

Pulled back the `qemu_create_displaysurface_guestmem` function to create the display surface so that the guest memory gets properly unmaped. Only allow one resolution change per guest boot, which prevents a crash when the guest writes garbage to the configuration space (e.g. when rebooting).

Re: [Qemu-devel] [PATCH] blockdev-backup: don't check aio_context too early

On 5/7/19 4:50 AM, Kevin Wolf wrote: > Am 06.05.2019 um 22:33 hat John Snow geschrieben: >> in blockdev_backup_prepare, we check to make sure that the target is >> associated with a compatible aio context. However, do_blockdev_backup is >> called later and has some logic to move the target to a

Re: [Qemu-devel] [PATCH v3 18/39] target/mips: Use env_cpu, env_archcpu

Hi Richard, Aleksandar. On 5/8/19 4:32 PM, Richard Henderson wrote: > On 5/8/19 1:15 AM, Aleksandar Markovic wrote: >> >> On May 8, 2019 2:19 AM, "Richard Henderson" > > wrote: >>> >>> >>> >> >> This commit message doesnˊt explain the reason for the change,

Re: [Qemu-devel] [PATCH v2 7/7] block: Ignore loosening perm restrictions failures

On 08.05.19 20:25, Max Reitz wrote: > We generally assume that loosening permission restrictions can never > fail. We have seen in the past that this assumption is wrong. This has > led to crashes because we generally pass _abort when loosening > permissions. > > However, a failure in such a

[Qemu-devel] [PATCH v3 2/5] iotests.py: Add qemu_nbd_early_pipe()

qemu_nbd_pipe() currently unconditionally reads qemu-nbd's output. That is not ideal because qemu-nbd may keep stderr open after the parent process has exited. Currently, the only user of qemu_nbd_pipe() is 147, which discards the whole output if the parent process returned success and only

Re: [Qemu-devel] [PATCH v2 1/2] vfio/mdev: add version attribute for mdev device

On Wed, 8 May 2019 07:27:40 -0400 Yan Zhao wrote: > On Wed, May 08, 2019 at 05:18:26AM +0800, Alex Williamson wrote: > > On Sun, 5 May 2019 21:49:04 -0400 > > Yan Zhao wrote: > > > > > version attribute is used to check two mdev devices' compatibility. > > > > > > The key point of this

[Qemu-devel] [PATCH v3 5/5] iotests: Let 233 run concurrently

common.nbd's nbd_server_set_tcp_port() tries to find a free port, and then uses it for the whole test run. However, this is racy because even if the port was free at the beginning, there is no guarantee it will continue to be available. Therefore, 233 currently cannot reliably be run

[Qemu-devel] [PATCH v3 4/5] iotests: Use qemu-nbd's --pid-file

Signed-off-by: Max Reitz --- tests/qemu-iotests/common.rc | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/tests/qemu-iotests/common.rc b/tests/qemu-iotests/common.rc index 93f87389b6..5502c3da2f 100644 --- a/tests/qemu-iotests/common.rc +++

[Qemu-devel] [PATCH v3 1/5] qemu-nbd: Add --pid-file option

--fork is a bit boring if there is no way to get the child's PID. This option helps. Signed-off-by: Max Reitz --- qemu-nbd.c| 11 +++ qemu-nbd.texi | 2 ++ 2 files changed, 13 insertions(+) diff --git a/qemu-nbd.c b/qemu-nbd.c index dca9e72cee..edb5195208 100644 --- a/qemu-nbd.c

[Qemu-devel] [PATCH v3 0/5] iotests: Let 233 run concurrently

Currently, 233 cannot reliably run concurrently to other NBD TCP tests. When it starts, it looks for a free port and then attempts to use that for the whole duration of the test run. This is a TOCTTOU race condition: It does not reserve that port, so another NBD TCP test that runs in parallel can

[Qemu-devel] [PATCH v3 3/5] qemu-nbd: Do not close stderr

We kept old_stderr specifically so we could keep emitting error message on stderr. However, qemu_daemon() closes stderr. Therefore, we need to dup() stderr to old_stderr before invoking qemu_daemon(). Signed-off-by: Max Reitz Reviewed-by: Eric Blake --- qemu-nbd.c | 3 ++- 1 file changed, 2

Re: [Qemu-devel] [PATCH v2 1/5] qemu-nbd: Add --pid-file option

On 08.05.19 15:22, Max Reitz wrote: > --fork is a bit boring if there is no way to get the child's PID. This > option helps. > > Signed-off-by: Max Reitz > --- > qemu-nbd.c| 12 > qemu-nbd.texi | 2 ++ > 2 files changed, 14 insertions(+) > > diff --git a/qemu-nbd.c

Re: [Qemu-devel] [PATCH] target/riscv: Only flush TLB if SATP.ASID changes

On Wed, 08 May 2019 10:38:35 PDT (-0700), jonat...@fintelia.io wrote: There is an analogous change for ARM here: https://patchwork.kernel.org/patch/10649857 Signed-off-by: Jonathan Behrens --- target/riscv/csr.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git

Re: [Qemu-devel] [Question] Memory hotplug clarification for Qemu ARM/virt

On 05/08/19 14:50, Robin Murphy wrote: > Hi Shameer, > > On 08/05/2019 11:15, Shameerali Kolothum Thodi wrote: >> Hi, >> >> This series here[0] attempts to add support for PCDIMM in QEMU for >> ARM/Virt platform and has stumbled upon an issue as it is not clear(at >> least >> from Qemu/EDK2 point

Re: [Qemu-devel] [PATCH 0/3] Export machine type deprecation info through QMP

On Wed, May 08, 2019 at 11:16:50AM +0200, Markus Armbruster wrote: > Eduardo Habkost writes: > > > On Tue, May 07, 2019 at 07:07:04AM +0200, Markus Armbruster wrote: > >> Eduardo Habkost writes: > >> > >> > This series adds machine type deprecation information to the > >> > output of the

Re: [Qemu-devel] [PATCH for-4.1 0/2] hw/alpha: Add the CY82C693UB southbridge in Kconfig

Paolo, Thomas, On 4/29/19 1:29 PM, Philippe Mathieu-Daudé wrote: > CC'ing Thomas who is a Kconfig expert. > > On 3/17/19 12:44 AM, Philippe Mathieu-Daudé wrote: >> Explicit the CY82C693UB southbridge used by the 264DP. >> >> Philippe Mathieu-Daudé (2): >> hw/isa/southbridge: Add the Cypress

Re: [Qemu-devel] [PATCH] target/riscv: Only flush TLB if SATP.ASID changes

On Wed, May 8, 2019 at 10:39 AM Jonathan Behrens wrote: > > There is an analogous change for ARM here: > https://patchwork.kernel.org/patch/10649857 > > Signed-off-by: Jonathan Behrens Reviewed-by: Alistair Francis Alistair > --- > target/riscv/csr.c | 4 +++- > 1 file changed, 3

[Qemu-devel] [PATCH v2] target/ppc: Fix xxspltib

xxspltib raises a VMX or a VSX exception depending on the register set it is operating on. We had a check, but it was backwards. Fixes: f113283525a4 ("target-ppc: add xxspltib instruction") Signed-off-by: Anton Blanchard --- target/ppc/translate/vsx-impl.inc.c | 8 1 file changed, 4

Re: [Qemu-devel] [Question] Memory hotplug clarification for Qemu ARM/virt

On 05/08/19 12:15, Shameerali Kolothum Thodi wrote: > Hi, > > This series here[0] attempts to add support for PCDIMM in QEMU for > ARM/Virt platform and has stumbled upon an issue as it is not clear(at least > from Qemu/EDK2 point of view) how in physical world the hotpluggable > memory is

Re: [Qemu-devel] [PATCH v3] i386: Add some MSR based features on Cascadelake-Server CPU model

On Wed, May 08, 2019 at 09:31:53AM +0800, Tao Xu wrote: > As noted in "c7a88b52f6 i386: Add new model of Cascadelake-Server" > Because MSR based feature has been supported by QEMU, we add > CPUID_7_0_EDX_ARCH_CAPABILITIES on Cascadelake-Server CPU model, > and add IA32_ARCH_CAPABILITIES MSR based

Re: [Qemu-devel] [PATCH v3 25/31] target/cris: Use tcg_gen_abs_tl

On 04.05.19 07:52, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target/cris/translate.c | 9 + > 1 file changed, 1 insertion(+), 8 deletions(-) > > diff --git a/target/cris/translate.c b/target/cris/translate.c > index b005a5c20e..31b40a57f9 100644 > ---

Re: [Qemu-devel] [PATCH 0/7] Delete 16 *_cpu_class_by_name() functions

On Wed, May 08, 2019 at 10:34:44AM +0200, Markus Armbruster wrote: > Eduardo Habkost writes: > > > On Mon, May 06, 2019 at 01:53:28PM +0200, Markus Armbruster wrote: > >> Eduardo Habkost writes: > >> > >> > This series adds a new CPUClass::class_name_format field, which > >> > allows us to

Re: [Qemu-devel] [PATCH 11/13] tests/vm: netbsd autoinstall, using serial console

On 08.05.2019 10:56, Gerd Hoffmann wrote: > Instead of fetching the prebuilt image from patchew download the install > iso and prepare the image locally. Install to disk, using the serial > console. Create qemu user, configure ssh login. Install packages > needed for qemu builds. > I

Re: [Qemu-devel] Update *BSD images with gnu-sed and bash

On 08.05.2019 10:07, Thomas Huth wrote: > On 08/05/2019 09.06, Kamil Rytarowski wrote: >> On 06.05.2019 12:12, Thomas Huth wrote: > [...] >>> Kamil, >>> >>> could you maybe help with the NetBSD image and the tests/vm/netbsd script? >>> >> >> Please be more specific what am I expected to do. > >

Re: [Qemu-devel] [PATCH v7 2/6] virtio-pmem: Add virtio pmem driver

On 5/8/19 4:12 AM, Pankaj Gupta wrote: > >> >> On 4/25/19 10:00 PM, Pankaj Gupta wrote: >> >>> +void host_ack(struct virtqueue *vq) >>> +{ >>> + unsigned int len; >>> + unsigned long flags; >>> + struct virtio_pmem_request *req, *req_buf; >>> + struct virtio_pmem *vpmem = vq->vdev->priv;

[Qemu-devel] [PATCH v2 5/7] block: Fix order in bdrv_replace_child()

We have to start by applying the permission restrictions to new_bs before we can loosen them on old_bs. See the comment for the explanation. Signed-off-by: Max Reitz Reviewed-by: Kevin Wolf --- block.c | 18 +- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git

[Qemu-devel] [PATCH v2 6/7] block: Add *tighten_restrictions to *check*_perm()

This patch makes three functions report whether the necessary permission change tightens restrictions or not. These functions are: - bdrv_check_perm() - bdrv_check_update_perm() - bdrv_child_check_perm() Callers can use this result to decide whether a failure is fatal or not (see the next

[Qemu-devel] [PATCH v2 4/7] block/commit: Drop bdrv_child_try_set_perm()

commit_top_bs never requests or unshares any permissions. There is no reason to make this so explicit here. Signed-off-by: Max Reitz Reviewed-by: Kevin Wolf --- block/commit.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/block/commit.c b/block/commit.c index 14e5bb394c..44b3083b84

[Qemu-devel] [PATCH v2 2/7] block: Add bdrv_child_refresh_perms()

If a block node uses bdrv_child_try_set_perm() to change the permission it takes on its child, the result may be very short-lived. If anything makes the block layer recalculate the permissions internally, it will invoke the node driver's .bdrv_child_perm() implementation. The permission/shared

[Qemu-devel] [PATCH v2 3/7] block/mirror: Fix child permissions

We cannot use bdrv_child_try_set_perm() to give up all restrictions on the child edge, and still have bdrv_mirror_top_child_perm() request BLK_PERM_WRITE. Fix this by making bdrv_mirror_top_child_perm() return 0/BLK_PERM_ALL when we want to give up all permissions, and replacing

[Qemu-devel] [PATCH v2 7/7] block: Ignore loosening perm restrictions failures

We generally assume that loosening permission restrictions can never fail. We have seen in the past that this assumption is wrong. This has led to crashes because we generally pass _abort when loosening permissions. However, a failure in such a case should actually be handled in quite the

[Qemu-devel] [PATCH v2 0/7] block: Ignore loosening perm restrictions failures

Hi, This series is mainly a fix for https://bugzilla.redhat.com/show_bug.cgi?id=1703793. The problem described there is that mirroring to a gluster volume, then switching off the volume makes qemu crash. There are two problems here: (1) file-posix reopens the FD all the time because it thinks

[Qemu-devel] [PATCH v2 1/7] file-posix: Update open_flags in raw_set_perm()

raw_check_perm() + raw_set_perm() can change the flags associated with the current FD. If so, we have to update BDRVRawState.open_flags accordingly. Otherwise, we may keep reopening the FD even though the current one already has the correct flags. Signed-off-by: Max Reitz Reviewed-by: Kevin

[Qemu-devel] [PATCH] configure: Require python3 >= 3.5

The oldest python3 version in distros that will be supported by QEMU 4.1 is 3.5.3 (the one in Debian Stretch). Error out if running python3 < 3.5. We have a .travis.yml job configured to use Python 3.4. Change it to use Python 3.5. Signed-off-by: Eduardo Habkost --- configure | 5 +++--

[Qemu-devel] [Bug 1828272] Re: 4.0 breaks keyboard autorepeat in guests with xserver

** Description changed: Description: In a linux/bsd guest within X, pressing and holding a key for a short time causes an endless repeat of that key in the guest. The release of the key gets ignored. Example 1: pressing and holding 'a' for a few seconds results in typing of

Re: [Qemu-devel] [PATCH] include/exec/poison: Mark TARGET_FMT_lu as poisoned, too

On 5/8/19 8:06 AM, Thomas Huth wrote: > We already poison TARGET_FMT_lx and TARGET_FMT_ld, but apparently > forgot to poison TARGET_FMT_lu, too. Do it now. > > Signed-off-by: Thomas Huth > --- > include/exec/poison.h | 1 + > 1 file changed, 1 insertion(+) Reviewed-by: Richard Henderson r~

[Qemu-devel] [Bug 1828272] Re: 4.0 breaks keyboard autorepeat in guests with xserver

** Description changed: Description: In a linux/bsd guest within X, pressing and holding a key for a short time causes an endless repeat of that key in the guest. The release of the key gets ignored. Example 1: pressing and holding 'a' for a few seconds results in typing of

[Qemu-devel] [Bug 1828272] [NEW] 4.0 breaks keyboard autorepeat in guests with xserver

Public bug reported: Description: In a linux/bsd guest within X, pressing and holding a key for a short time causes an endless repeat of that key in the guest. The release of the key gets ignored. Example 1: pressing and holding 'a' for a few seconds results in typing of '...'

Re: [Qemu-devel] [PATCH] Deprecate Python 2 support

On Wed, May 08, 2019 at 02:50:00PM +0200, Markus Armbruster wrote: > Daniel P. Berrangé writes: > > > On Tue, May 07, 2019 at 12:38:14PM +0200, Kevin Wolf wrote: > >> Am 03.05.2019 um 21:37 hat Eduardo Habkost geschrieben: > >> > Python 2 will reach end of life in January 1 2020. Declare it as

Re: [Qemu-devel] [PATCH] tests: Force Python I/O encoding for check-qapi-schema

On Wed, May 08, 2019 at 03:04:43PM +0200, Markus Armbruster wrote: > Eduardo Habkost writes: > > > On Tue, May 07, 2019 at 03:13:45PM +0100, Daniel P. Berrangé wrote: > >> On Mon, May 06, 2019 at 06:38:17PM -0300, Eduardo Habkost wrote: > >> > test-qapi.py doesn't force a specific encoding for

Re: [Qemu-devel] [PATCH v7 2/2] hw/arm: Add arm SBSA reference machine, devices part

On Wed, 8 May 2019 at 13:30, Hongbo Zhang wrote: > On Tue, 30 Apr 2019 at 22:17, Peter Maydell > wrote: > > > > On Thu, 18 Apr 2019 at 05:05, Hongbo Zhang > wrote: > > > > > > Following the previous patch, this patch adds peripheral devices to the > > > newly introduced SBSA-ref machine. > > >

Re: [Qemu-devel] [PATCH] target/riscv: Only flush TLB if SATP.ASID changes

On 5/8/19 10:38 AM, Jonathan Behrens wrote: > There is an analogous change for ARM here: > https://patchwork.kernel.org/patch/10649857 > > Signed-off-by: Jonathan Behrens > --- > target/riscv/csr.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) Reviewed-by: Richard Henderson r~

Re: [Qemu-devel] [PATCH v5 03/15] tests/tcg/aarch64: add system boot.S

Richard Henderson writes: > On 4/30/19 9:52 AM, Alex Bennée wrote: >> +.error: >> +.string "Terminated by exception.\n" > > Put it in .rodata just because we can? > >> +/* Page table setup (identity mapping). */ >> +adrpx0, ttb >> +add x0, x0, :lo12:ttb > > You are

[Qemu-devel] [PATCH] target/riscv: Only flush TLB if SATP.ASID changes

There is an analogous change for ARM here: https://patchwork.kernel.org/patch/10649857 Signed-off-by: Jonathan Behrens --- target/riscv/csr.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/target/riscv/csr.c b/target/riscv/csr.c index 6083c782a1..1ec1222da1 100644 ---

Re: [Qemu-devel] [PATCH v10 13/13] qemu/bitops.h: Add extract8 and extract16

On 5/8/19 7:56 AM, Yoshinori Sato wrote: > +static inline uint16_t extract16(uint32_t value, int start, int length) s/uint32_t/uint16_t/ Aside from the possible value of the more restrictive asserts, I'm not sure what advantage you see in these routines. All arithmetic in C is promoted to type

Re: [Qemu-devel] [PATCH v10 12/13] hw/registerfields.h: Add 8bit and 16bit register macros.

On 5/8/19 7:56 AM, Yoshinori Sato wrote: > Some RX peripheral using 8bit and 16bit registers. > Added 8bit and 16bit APIs. > > Signed-off-by: Yoshinori Sato > --- > include/hw/registerfields.h | 32 +++- > 1 file changed, 31 insertions(+), 1 deletion(-) Reviewed-by:

Re: [Qemu-devel] [PATCH v10 10/13] Add rx-softmmu

On 5/8/19 7:56 AM, Yoshinori Sato wrote: > Signed-off-by: Yoshinori Sato > --- > configure | 8 > default-configs/rx-softmmu.mak | 7 +++ > include/sysemu/arch_init.h | 1 + > arch_init.c| 2 ++ > hw/Kconfig | 1 + > 5

Re: [Qemu-devel] [PATCH v10 05/13] target/rx: Miscellaneous files

On 5/8/19 7:56 AM, Yoshinori Sato wrote: > Signed-off-by: Yoshinori Sato > --- > target/rx/gdbstub.c | 112 > > target/rx/monitor.c | 38 > target/rx/Makefile.objs | 12 ++ > 3 files changed, 162 insertions(+) >

Re: [Qemu-devel] [PATCH v10 00/13] Add RX archtecture support

On 5/8/19 7:55 AM, Yoshinori Sato wrote: > Changes for v9. > - Fix "mov.l dsp5[rs],rd". You need the same change to the disassembler. r~

Re: [Qemu-devel] [PATCH v10 04/13] target/rx: RX disassembler

On 5/8/19 7:56 AM, Yoshinori Sato wrote: > Signed-off-by: Yoshinori Sato > --- > include/disas/dis-asm.h |5 + > target/rx/disas.c | 1480 > +++ > 2 files changed, 1485 insertions(+) > create mode 100644 target/rx/disas.c Reviewed-by:

[Qemu-devel] [PATCH 1/3] spapr/xive: EQ page should be naturally aligned

When the OS configures the EQ page in which to receive event notifications from the XIVE interrupt controller, the page should be naturally aligned. Add this check. Signed-off-by: Cédric Le Goater --- hw/intc/spapr_xive.c | 6 ++ 1 file changed, 6 insertions(+) diff --git

Re: [Qemu-devel] [PATCH v10 03/13] target/rx: CPU definition

On 5/8/19 7:56 AM, Yoshinori Sato wrote: > Signed-off-by: Yoshinori Sato > --- > target/rx/cpu-qom.h | 52 > target/rx/cpu.h | 196 ++ > target/rx/cpu.c | 222 > > 3 files

[Qemu-devel] [PATCH 0/3] spapr/xive: fixes on EQ page addresses

Hello Here is a small series adding a check on the EQ page address alignment and fixing a severe issue when addresses are above 64GB. Thanks, C. Cédric Le Goater (3): spapr/xive: EQ page should be naturally aligned spapr/xive: fix EQ page addresses above 64GB spapr/xive: print out the EQ

Re: [Qemu-devel] [PATCH v10 02/13] target/rx: TCG helper

On 5/8/19 7:56 AM, Yoshinori Sato wrote: > Signed-off-by: Yoshinori Sato > --- > target/rx/helper.h| 31 > target/rx/helper.c| 148 > target/rx/op_helper.c | 481 > ++ > 3 files changed, 660 insertions(+) > create

[Qemu-devel] [PATCH 3/3] spapr/xive: print out the EQ page address in the monitor

This proved to be a useful information when debugging issues with OS event queues allocated above 64GB. Signed-off-by: Cédric Le Goater --- hw/intc/spapr_xive.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/hw/intc/spapr_xive.c b/hw/intc/spapr_xive.c index

Re: [Qemu-devel] [PATCH v10 01/13] target/rx: TCG translation

On 5/8/19 7:55 AM, Yoshinori Sato wrote: > +static bool trans_XCHG_mr(DisasContext *ctx, arg_XCHG_mr *a) > +{ > +TCGv mem, addr; > +mem = tcg_temp_new(); > +switch (a->mi) { > +case 0: /* dsp[rs].b */ > +case 1: /* dsp[rs].w */ > +case 2: /* dsp[rs].l */ > +addr =

[Qemu-devel] [PATCH 2/3] spapr/xive: fix EQ page addresses above 64GB

The high order bits of the address of the OS event queue is stored in bits [4-31] of word2 of the XIVE END internal structures and the low order bits in word3. This structure is using Big Endian ordering and computing the value requires some simple arithmetic which happens to be wrong. The mask

Re: [Qemu-devel] [PULL v2 00/28] Kconfig for Arm machines

[clicked ctrl+enter too fast] On Wed, May 8, 2019 at 6:43 PM Philippe Mathieu-Daudé wrote: > On 5/8/19 5:33 PM, Thomas Huth wrote: > > On 08/05/2019 17.09, Peter Maydell wrote: > >> On Tue, 7 May 2019 at 14:45, Thomas Huth wrote: > >>> > >>> Hi Peter, > >>> > >>> the following changes since

Re: [Qemu-devel] [PULL v2 00/28] Kconfig for Arm machines

On 5/8/19 5:33 PM, Thomas Huth wrote: > On 08/05/2019 17.09, Peter Maydell wrote: >> On Tue, 7 May 2019 at 14:45, Thomas Huth wrote: >>> >>> Hi Peter, >>> >>> the following changes since commit a6ae23831b05a11880b40f7d58e332c45a6b04f7: >>> >>> Merge remote-tracking branch >>>

Re: [Qemu-devel] [PATCH v10 07/13] hw/timer: RX62N internal timer modules

On 5/8/19 4:56 PM, Yoshinori Sato wrote: > renesas_tmr: 8bit timer modules. > renesas_cmt: 16bit compare match timer modules. > This part use many renesas's CPU. > Hardware manual. > https://www.renesas.com/us/en/doc/products/mpumcu/doc/rx_family/r01uh0033ej0140_rx62n.pdf > > Signed-off-by:

Re: [Qemu-devel] [PATCH 0/3] acpi: More trace points

tagged, thanks! On Wed, May 08, 2019 at 01:19:47PM +0200, Markus Armbruster wrote: > Ping? > > Markus Armbruster writes: > > > I wrote these patches to help me debug an unplug failure. I expect > > them to be helpful for others, too. > > > > Markus Armbruster (3): > > acpi/piix4: Convert

Re: [Qemu-devel] [PATCH v10 01/13] target/rx: TCG translation

On 5/8/19 4:55 PM, Yoshinori Sato wrote: > This part only supported RXv1 instructions. > Instruction manual. > https://www.renesas.com/us/en/doc/products/mpumcu/doc/rx_family/r01us0032ej0120_rxsm.pdf > > Signed-off-by: Yoshinori Sato > --- > target/rx/translate.c | 2432 >

Re: [Qemu-devel] [PATCH v10 06/13] hw/intc: RX62N interrupt controller (ICUa)

On 5/8/19 4:56 PM, Yoshinori Sato wrote: > This implementation supported only ICUa. > Hardware manual. > https://www.renesas.com/us/en/doc/products/mpumcu/doc/rx_family/r01uh0033ej0140_rx62n.pdf > > Signed-off-by: Yoshinori Sato > --- > include/hw/intc/rx_icu.h | 49 +++ > hw/intc/rx_icu.c

Re: [Qemu-devel] [PATCH 4/6] fsdev: Error out when unsupported option is passed

On 5/7/19 3:45 AM, Greg Kurz wrote: > Each fsdriver only supports a subset of the options that can be passed > to -fsdev. Unsupported options are simply ignored. This could cause the > user to erroneously think QEMU has a bug. > > Enforce strict checking of supported options for all fsdrivers.

Re: [Qemu-devel] [PATCH v10 08/13] hw/char: RX62N serial communication interface (SCI)

On 5/8/19 4:56 PM, Yoshinori Sato wrote: > This module supported only non FIFO type. > Hardware manual. > https://www.renesas.com/us/en/doc/products/mpumcu/doc/rx_family/r01uh0033ej0140_rx62n.pdf > > Signed-off-by: Yoshinori Sato > --- > include/hw/char/renesas_sci.h | 45 ++ >

Re: [Qemu-devel] [PULL 0/5] NBD patches for 2019-05-07

On Tue, 7 May 2019 at 16:03, Eric Blake wrote: > > The following changes since commit 19eb2d4e736dc895f31fbd6b520e514f10cc08e0: > > Merge remote-tracking branch 'remotes/thibault/tags/samuel-thibault' into > staging (2019-05-07 10:43:32 +0100) > > are available in the Git repository at: > >

Re: [Qemu-devel] [PATCH v10 01/13] target/rx: TCG translation

On 5/8/19 4:55 PM, Yoshinori Sato wrote: > This part only supported RXv1 instructions. > Instruction manual. > https://www.renesas.com/us/en/doc/products/mpumcu/doc/rx_family/r01us0032ej0120_rxsm.pdf > > Signed-off-by: Yoshinori Sato > --- > target/rx/translate.c | 2432 >

Re: [Qemu-devel] [PATCH v10 05/13] target/rx: Miscellaneous files

On 5/8/19 4:56 PM, Yoshinori Sato wrote: > Signed-off-by: Yoshinori Sato > --- > target/rx/gdbstub.c | 112 > > target/rx/monitor.c | 38 > target/rx/Makefile.objs | 12 ++ > 3 files changed, 162 insertions(+) >

Re: [Qemu-devel] [PATCH v10 10/13] Add rx-softmmu

On 5/8/19 4:56 PM, Yoshinori Sato wrote: > Signed-off-by: Yoshinori Sato > --- > configure | 8 > default-configs/rx-softmmu.mak | 7 +++ > include/sysemu/arch_init.h | 1 + > arch_init.c| 2 ++ > hw/Kconfig | 1 + > 5

Re: [Qemu-devel] [PATCH v10 09/13] hw/rx: RX Target hardware definition

Hi Yoshinori, "v10"... I'm a bit late at reviewing this, sorry. On 5/8/19 4:56 PM, Yoshinori Sato wrote: > rx62n - RX62N cpu. cpu -> MCU (or SoC) > rxqemu - QEMU virtual target. You declare it with MACHINE_TYPE_NAME("rx-qemu"), please use one or another to be consistent. Why not name it

Re: [Qemu-devel] [PATCH 6/6] virtfs: Fix documentation of -fsdev and -virtfs

On 07/05/2019 10.45, Greg Kurz wrote: > This fixes several things: > - add "id" description to -virtfs documentation > - split the description into several lines in both usage and documentation > for accurateness and clarity > - add documentation and usage of the synth fsdriver > - add

[Qemu-devel] [PATCH v2] block: Use bdrv_unref_child() for all children in bdrv_close()

bdrv_unref_child() does the following things: - Updates the child->bs->inherits_from pointer. - Calls bdrv_detach_child() to remove the BdrvChild from bs->children. - Calls bdrv_unref() to unref the child BlockDriverState. When bdrv_unref_child() was introduced in commit 33a604075c it was

Re: [Qemu-devel] [PULL v2 00/28] Kconfig for Arm machines

On 08/05/2019 17.09, Peter Maydell wrote: > On Tue, 7 May 2019 at 14:45, Thomas Huth wrote: >> >> Hi Peter, >> >> the following changes since commit a6ae23831b05a11880b40f7d58e332c45a6b04f7: >> >> Merge remote-tracking branch >> 'remotes/ehabkost/tags/python-next-pull-request' into staging

Re: [Qemu-devel] [PATCH v10 04/13] target/rx: RX disassembler

On 5/8/19 4:56 PM, Yoshinori Sato wrote: > Signed-off-by: Yoshinori Sato Tested-by: Philippe Mathieu-Daudé > --- > include/disas/dis-asm.h |5 + > target/rx/disas.c | 1480 > +++ > 2 files changed, 1485 insertions(+) > create mode 100644

Re: [Qemu-devel] [PATCH v7 2/6] virtio-pmem: Add virtio pmem driver

> > > > > +int virtio_pmem_flush(struct nd_region *nd_region) > > > +{ > > > +int err; > > > +unsigned long flags; > > > +struct scatterlist *sgs[2], sg, ret; > > > +struct virtio_device *vdev = nd_region->provider_data; > > > +struct virtio_pmem *vpmem =

Re: [Qemu-devel] [PATCH 0/2] fix qmp-cont vs migration-finish race-crash

On 24/01/2019 13.25, Vladimir Sementsov-Ogievskiy wrote: > Hi. > > It's a simple fix for problems reported in "Aborts in iotest 169" > by Max: > https://lists.gnu.org/archive/html/qemu-devel/2019-01/msg05907.html > > In thread Kevin described that a problem itself is bigger and needs > more

[Qemu-devel] [PATCH v10 01/13] target/rx: TCG translation

This part only supported RXv1 instructions. Instruction manual. https://www.renesas.com/us/en/doc/products/mpumcu/doc/rx_family/r01us0032ej0120_rxsm.pdf Signed-off-by: Yoshinori Sato --- target/rx/translate.c | 2432 target/rx/insns.decode |

Re: [Qemu-devel] [PULL v2 00/28] Kconfig for Arm machines

On Tue, 7 May 2019 at 14:45, Thomas Huth wrote: > > Hi Peter, > > the following changes since commit a6ae23831b05a11880b40f7d58e332c45a6b04f7: > > Merge remote-tracking branch > 'remotes/ehabkost/tags/python-next-pull-request' into staging (2019-05-03 > 15:26:09 +0100) > > are available in

[Qemu-devel] [PATCH] include/exec/poison: Mark TARGET_FMT_lu as poisoned, too

We already poison TARGET_FMT_lx and TARGET_FMT_ld, but apparently forgot to poison TARGET_FMT_lu, too. Do it now. Signed-off-by: Thomas Huth --- include/exec/poison.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/exec/poison.h b/include/exec/poison.h index 1a7a57baae..b862320fa6

[Qemu-devel] [PATCH v10 04/13] target/rx: RX disassembler

Signed-off-by: Yoshinori Sato --- include/disas/dis-asm.h |5 + target/rx/disas.c | 1480 +++ 2 files changed, 1485 insertions(+) create mode 100644 target/rx/disas.c diff --git a/include/disas/dis-asm.h b/include/disas/dis-asm.h index

  1   2   3   >