Re: [PATCH v16 08/14] numa: Extend CLI to provide memory latency and bandwidth information

On 11/19/2019 7:03 PM, Igor Mammedov wrote: On Fri, 15 Nov 2019 15:53:46 +0800 Tao Xu wrote: From: Liu Jingqi Add -numa hmat-lb option to provide System Locality Latency and Bandwidth Information. These memory attributes help to build System Locality Latency and Bandwidth Information Structu

Re: [PATCH v3 14/33] serial-mm: add "regshift" property

Hi On Mon, Nov 18, 2019 at 6:54 PM Peter Maydell wrote: > > On Wed, 23 Oct 2019 at 18:34, Marc-André Lureau > wrote: > > > > And a property and rename "it_shift" field to "regshift", as it seems > > to be more popular (and I don't know what "it" stands for). > > > > Signed-off-by: Marc-André Lur

Re: [SeaBIOS] Re: 1.13 release?

On Mon, Nov 18, 2019 at 01:08:39PM -0500, Kevin O'Connor wrote: > On Wed, Nov 06, 2019 at 11:29:18AM -0500, Kevin O'Connor wrote: > > On Wed, Nov 06, 2019 at 12:12:55PM +0100, Gerd Hoffmann wrote: > > > On Wed, Oct 16, 2019 at 12:44:12PM +0200, Gerd Hoffmann wrote: > > > > Hi, > > > > > > > > Al

Re: [PATCH] spapr: Fix VSMT mode when it is not supported by the kernel

On Wed, 20 Nov 2019 15:36:53 +1100 David Gibson wrote: > On Tue, Nov 19, 2019 at 04:45:26PM +0100, Greg Kurz wrote: > > On Tue, 19 Nov 2019 15:06:51 +0100 > > Laurent Vivier wrote: > > > > > On 19/11/2019 02:00, David Gibson wrote: > > > > On Fri, Nov 08, 2019 at 05:47:59PM +0100, Greg Kurz wro

Re: [PATCH] Add minimal Hexagon target - First in a series of patches - linux-user changes + linux-user/hexagon + skeleton of target/hexagon - Files in target/hexagon/imported are from another project

On 11/20/19 3:26 AM, Aleksandar Markovic wrote: > > > On Tuesday, November 19, 2019, Richard Henderson > wrote: > > On 11/19/19 6:22 PM, Taylor Simpson wrote: > > - Laurent suggested I split the patch into two parts: linux-user and > target/hexag

Re: [PATCH 0/5] ppc/pnv: fix Homer/Occ mappings on multichip systems

Hello, On 19/11/2019 18:50, Balamuruhan S wrote: > Hi All, > > PowerNV fails to boot in multichip systems due to some misinterpretation > and mapping in Homer/Occ device models, this patchset fixes the > following, > > - Homer size is 4MB per chip and Occ common area size is 8MB > - Bar masks

Re: [PATCH v3 13/33] serial: start making SerialMM a sysbus device

Hi On Mon, Nov 18, 2019 at 6:43 PM Peter Maydell wrote: > > On Wed, 23 Oct 2019 at 18:33, Marc-André Lureau > wrote: > > > > Memory mapped serial device is in fact a sysbus device. The following > > patches will make use of sysbus facilities for resource and > > registration. > > > > Signed-off-

Re: [PATCH 5/5] hw/ppc/pnv_xscom: add PBA BARs for Power8 slw image

On 19/11/2019 18:50, Balamuruhan S wrote: > slw base and size mask are accessed during boot in homer_init_chip(), > so include BAR2 and BARMASK2 for Power8. > > Signed-off-by: Balamuruhan S > --- > hw/ppc/pnv_xscom.c | 10 -- > include/hw/ppc/pnv.h | 4 > 2 files changed, 12 inse

Re: [PATCH 4/5] hw/ppc/pnv_xscom: occ common area to be mapped only once

On 19/11/2019 18:50, Balamuruhan S wrote: > occ common area should be mapped once It's the same address on each chip. the question is how the HW knows from which chip the OCC access is being done ? How does it target the correct OCC if the address is the same ? > and disable it for every ot

Re: [PATCH 2/5] hw/ppc/pnv_xscom: PBA bar mask values are incorrect with homer/occ sizes

On 19/11/2019 18:50, Balamuruhan S wrote: > homer/occ sizes are calculated in skiboot with `(mask | 0xf) + 1`, > and from xscom access should return correct mask values instead of actual > sizes. > > Signed-off-by: Cédric Le Goater > Signed-off-by: Balamuruhan S > --- > hw/ppc/pnv_xscom.c |

Re: [PATCH 3/5] hw/ppc/pnv_xscom: Power8 occ common area is in PBA BAR 3

On 19/11/2019 18:50, Balamuruhan S wrote: > Fix incorrect PBA BAR and BARMASK value for Power8 occ common area > region where skiboot enum declaration have it in BAR 3 and BARMASK > is calculated BARMASK0 + BAR, > > enum P8_BAR { > P8_BAR_HOMER = 0, > P8_BAR_CENTAUR = 1, >

Re: [PATCH 1/5] hw/ppc/pnv: incorrect homer and occ common area size

On 19/11/2019 18:50, Balamuruhan S wrote: > Homer size is 4MB and OCC common area size is 8MB, but currently > it is assigned with bar mask value. Also pass on the occ sram > size 3 bits right shifted to initialize the size appropriately. > > Signed-off-by: Cédric Le Goater > Signed-off-by: Balam

[Bug 1853083] Re: qemu ppc64 4.0 boot AIX5.1 hung

AIX 5.1 is quite a bit older than POWER8, so I don't think that it will run with this processor anymore. You could try "power5" or "970fx" as CPU (maybe even the "40p" machine instead of "pseries"), but I guess it won't make a big difference - the QEMU pseries machine has been written for later ope

Re: [PATCH v16 09/14] numa: Extend CLI to provide memory side cache information

On 11/19/2019 7:47 PM, Igor Mammedov wrote: On Fri, 15 Nov 2019 15:53:47 +0800 Tao Xu wrote: From: Liu Jingqi Add -numa hmat-cache option to provide Memory Side Cache Information. These memory attributes help to build Memory Side Cache Information Structure(s) in ACPI Heterogeneous Memory At

Re: [RFC 5/5] spapr: Work around spurious warnings from vfio INTx initialization

On 20/11/2019 05:17, David Gibson wrote: > On Thu, Oct 17, 2019 at 03:42:06PM +0200, Cédric Le Goater wrote: >> On 17/10/2019 10:43, Cédric Le Goater wrote: >>> On 17/10/2019 07:42, David Gibson wrote: Traditional PCI INTx for vfio devices can only perform well if using an in-kernel irqch

Re: [PATCH v2] virtio-pci: disable vring processing when bus-mastering is disabled

Patchew URL: https://patchew.org/QEMU/20191120005003.27035-1-mdr...@linux.vnet.ibm.com/ Hi, This series failed the docker-quick@centos7 build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRIPT BE

[Bug 1853083] Re: qemu ppc64 4.0 boot AIX5.1 hung

I'm running this to start the AIX5.1 installation on qemu: #!/bin/bash qemu-system-ppc64 -cpu POWER8 -machine pseries -m 2048 -serial mon:stdio -hda aix-hdd.qcow2 -cdrom /Download/AIX5.1/VOLUME1.iso -prom-env boot-command='boot cdrom: -s verbose' -- You received this bug notification because y

[Bug 1853083] Re: qemu ppc64 4.0 boot AIX5.1 hung

and it got: [root@192 emu]# ./aix51 VNC server running on ::1:5900 qemu-system-ppc64: warning: TCG doesn't support requested feature, cap-cfpc=workaround qemu-system-ppc64: warning: TCG doesn't support requested feature, cap-sbbc=workaround qemu-system-ppc64: warning: TCG doesn't support requeste

Re: [PULL for-4.2-rc2 0/2] Tracing patches

Patchew URL: https://patchew.org/QEMU/20191119204551.240792-1-stefa...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Subject: [PULL for-4.2-rc2 0/2] Tracing patches Type: series Message-id: 20191119204551.240792-1-stefa...@redhat.

Re: [PATCH v4 17/19] spapr: Remove last pieces of SpaprIrq

On Mon, Oct 14, 2019 at 11:15:16AM +0200, Greg Kurz wrote: > On Sat, 12 Oct 2019 11:00:41 +1100 > David Gibson wrote: > > > On Fri, Oct 11, 2019 at 08:13:33AM +0200, Greg Kurz wrote: > > > On Fri, 11 Oct 2019 16:07:58 +1100 > > > David Gibson wrote: > > > > > > > On Thu, Oct 10, 2019 at 10:33:0

Re: [PATCH] exynos4210_gic: Suppress gcc9 format-truncation warnings

On Mon, Oct 14, 2019 at 01:51:39PM +0100, Peter Maydell wrote: > On Fri, 4 Oct 2019 at 04:10, David Gibson wrote: > > > > exynos4210_gic_realize() prints the number of cpus into some temporary > > buffers, but it only allows 3 bytes space for it. That's plenty - I'm > > pretty sure that existing

RE: [PATCH] Hexagon: Swap SIGRGMAX-1 and SIGRTMIN+1

Peter, Yeah, I was surprised not to see other targets encountering this problem. However, I don't understand how something under #ifdef TARGET_HEXAGON can break anything else. Could you point me to the patch set you mention? One generic solution I can think of is to reference a target-defined

RE: [PATCH] Add minimal Hexagon target - First in a series of patches - linux-user changes + linux-user/hexagon + skeleton of target/hexagon - Files in target/hexagon/imported are from another project

Thanks for the feedback Richard. Responses below ... Thanks, Taylor -Original Message- From: Richard Henderson Sent: Tuesday, November 19, 2019 1:34 PM To: Taylor Simpson ; laur...@vivier.eu; riku.voi...@iki.fi; qemu-devel@nongnu.org Subject: Re: [PATCH] Add minimal Hexagon target - F

RE: [PATCH] Add minimal Hexagon target - First in a series of patches - linux-user changes + linux-user/hexagon + skeleton of target/hexagon - Files in target/hexagon/imported are from another project

For the general DEBUG_HEX messages, I think the trace infrastructure isn't quite what I'm looking for. Here's a sample of what it prints Start packet: pc = 0x4002f0 Packet committed: pc = 0x4002f0 Regs written r18 = -69420 (0xfffef0d4) r19 = -69412 (0xfffef0dc) Stores memd

Re: [PATCH] spapr: Fix VSMT mode when it is not supported by the kernel

On Tue, Nov 19, 2019 at 04:45:26PM +0100, Greg Kurz wrote: > On Tue, 19 Nov 2019 15:06:51 +0100 > Laurent Vivier wrote: > > > On 19/11/2019 02:00, David Gibson wrote: > > > On Fri, Nov 08, 2019 at 05:47:59PM +0100, Greg Kurz wrote: > > >> On Fri, 8 Nov 2019 16:40:35 +0100 > > >> Laurent Vivier

Re: [PATCH] spapr: Fix VSMT mode when it is not supported by the kernel

On Tue, Nov 19, 2019 at 05:13:29PM +0100, Lukáš Doktor wrote: > Dne 19. 11. 19 v 16:45 Greg Kurz napsal(a): > > On Tue, 19 Nov 2019 15:06:51 +0100 > > Laurent Vivier wrote: > > > >> On 19/11/2019 02:00, David Gibson wrote: > >>> On Fri, Nov 08, 2019 at 05:47:59PM +0100, Greg Kurz wrote: > On

Re: [RFC v2 09/22] vfio/pci: add iommu_context notifier for pasid alloc/free

On Wed, Nov 06, 2019 at 12:14:50PM +, Liu, Yi L wrote: > > From: David Gibson [mailto:da...@gibson.dropbear.id.au] > > Sent: Tuesday, October 29, 2019 8:16 PM > > To: Liu, Yi L > > Subject: Re: [RFC v2 09/22] vfio/pci: add iommu_context notifier for pasid > > alloc/free > > > > On Thu, Oct 2

Re: [RFC 5/5] spapr: Work around spurious warnings from vfio INTx initialization

On Thu, Oct 17, 2019 at 03:42:06PM +0200, Cédric Le Goater wrote: > On 17/10/2019 10:43, Cédric Le Goater wrote: > > On 17/10/2019 07:42, David Gibson wrote: > >> Traditional PCI INTx for vfio devices can only perform well if using > >> an in-kernel irqchip. Therefore, vfio_intx_update() issues a

Re: [PATCH for-5.0 v5 06/23] ppc/xive: Introduce OS CAM line helpers

On Fri, Nov 15, 2019 at 05:24:19PM +0100, Cédric Le Goater wrote: > The OS CAM line has a special encoding exploited by the HW. Provide > helper routines to hide the details to the TIMA command handlers. This > also clarifies the endianness of different variables : 'qw1w2' is > big-endian and 'cam'

Re: [PATCH for-5.0 v5 07/23] ppc/xive: Check V bit in TM_PULL_POOL_CTX

On Fri, Nov 15, 2019 at 05:24:20PM +0100, Cédric Le Goater wrote: > A context should be 'valid' when pulled from the thread interrupt > context registers. > > Signed-off-by: Cédric Le Goater Applied, thanks. > --- > hw/intc/xive.c | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/

Re: [RFC 3/5] vfio/pci: Respond to KVM irqchip change notifier

On Thu, Oct 17, 2019 at 02:28:45PM -0600, Alex Williamson wrote: > On Thu, 17 Oct 2019 16:42:16 +1100 > David Gibson wrote: > > > VFIO PCI devices already respond to the pci intx routing notifier, in order > > to update kernel irqchip mappings when routing is updated. However this > > won't hand

Re: [PATCH v3 01/33] qdev: remove unused qdev_prop_int64

Hi On Mon, Nov 18, 2019 at 6:01 PM Peter Maydell wrote: > > On Wed, 23 Oct 2019 at 18:32, Marc-André Lureau > wrote: > > > > Signed-off-by: Marc-André Lureau > > --- > > hw/core/qdev-properties.c| 32 > > include/hw/qdev-properties.h | 3 --- > > 2 files c

Re: [PATCH 2/5] hw/ppc/pnv_xscom: PBA bar mask values are incorrect with homer/occ sizes

On Wed, Nov 20, 2019 at 08:31:03AM +0530, Balamuruhan S wrote: > On Wed, Nov 20, 2019 at 09:02:26AM +1100, David Gibson wrote: > > On Wed, Nov 20, 2019 at 09:00:32AM +1100, David Gibson wrote: > > > On Wed, Nov 20, 2019 at 08:56:18AM +1100, David Gibson wrote: > > > > On Tue, Nov 19, 2019 at 11:20:

Re: [PATCH 2/5] hw/ppc/pnv_xscom: PBA bar mask values are incorrect with homer/occ sizes

On Wed, Nov 20, 2019 at 09:02:26AM +1100, David Gibson wrote: > On Wed, Nov 20, 2019 at 09:00:32AM +1100, David Gibson wrote: > > On Wed, Nov 20, 2019 at 08:56:18AM +1100, David Gibson wrote: > > > On Tue, Nov 19, 2019 at 11:20:53PM +0530, Balamuruhan S wrote: > > > > homer/occ sizes are calculated

Re: [PATCH] linux-user/strace: Improve output of various syscalls

> > #ifdef TARGET_NR_ioctl > -{ TARGET_NR_ioctl, "ioctl" , NULL, NULL, NULL }, > +{ TARGET_NR_ioctl, "ioctl" , "%s(%d,%#x,%#x,%#x,%#x,%#x)", NULL, NULL }, > #endif ioctl() has only three arguments. Regards, Aleksandar

Re: [PATCH] Add minimal Hexagon target - First in a series of patches - linux-user changes + linux-user/hexagon + skeleton of target/hexagon - Files in target/hexagon/imported are from another project

On Tuesday, November 19, 2019, Richard Henderson < richard.hender...@linaro.org> wrote: > On 11/19/19 6:22 PM, Taylor Simpson wrote: > > - Laurent suggested I split the patch into two parts: linux-user and > target/hexagon. If I do that, which one should contain the changes to > common files (e.g

Re: [PULL 00/12] Misc patches for QEMU 4.2-rc

Patchew URL: https://patchew.org/QEMU/1574179728-35535-1-git-send-email-pbonz...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Subject: [PULL 00/12] Misc patches for QEMU 4.2-rc Type: series Message-id: 1574179728-35535-1-git-send

Re: [PATCH v9 Kernel 2/5] vfio iommu: Add ioctl defination to get dirty pages bitmap.

On Fri, Nov 15, 2019 at 11:21:33AM +0800, Alex Williamson wrote: > On Thu, 14 Nov 2019 21:40:35 -0500 > Yan Zhao wrote: > > > On Fri, Nov 15, 2019 at 05:06:25AM +0800, Alex Williamson wrote: > > > On Fri, 15 Nov 2019 00:26:07 +0530 > > > Kirti Wankhede wrote: > > > > > > > On 11/14/2019 1:37

RE: [PATCH v7 0/4] colo: Add support for continuous replication

> -Original Message- > From: Jason Wang > Sent: Tuesday, November 19, 2019 11:03 PM > To: Zhang, Chen ; Lukas Straub > ; qemu-devel > Cc: Kevin Wolf ; qemu-block bl...@nongnu.org>; Wen Congyang ; Max > Reitz ; Xie Changlong > Subject: Re: [PATCH v7 0/4] colo: Add support for continuou

RE: [PATCH v9 Kernel 2/5] vfio iommu: Add ioctl defination to get dirty pages bitmap.

> From: Alex Williamson [mailto:alex.william...@redhat.com] > Sent: Wednesday, November 20, 2019 7:17 AM > > On Fri, 15 Nov 2019 05:10:53 + > "Tian, Kevin" wrote: > > > > From: Alex Williamson > > > Sent: Friday, November 15, 2019 11:22 AM > > > > > > On Thu, 14 Nov 2019 21:40:35 -0500 > > >

[PATCH v2] virtio-pci: disable vring processing when bus-mastering is disabled

Currently the SLOF firmware for pseries guests will disable/re-enable a PCI device multiple times via IO/MEM/MASTER bits of PCI_COMMAND register after the initial probe/feature negotiation, as it tends to work with a single device at a time at various stages like probing and running block/network b

Re: [virtio-dev] Re: guest / host buffer sharing ...

On Tue, Nov 19, 2019 at 7:31 AM Liam Girdwood wrote: > > On Tue, 2019-11-12 at 14:55 -0800, Gurchetan Singh wrote: > > On Tue, Nov 12, 2019 at 5:56 AM Liam Girdwood > > wrote: > > > > > > On Mon, 2019-11-11 at 16:54 -0800, Gurchetan Singh wrote: > > > > On Tue, Nov 5, 2019 at 2:55 AM Gerd Hoffman

Re: [PATCH] audio: fix audio recording

On 2019-11-19 20:43, Richard Henderson wrote: On 11/19/19 9:01 AM, Philippe Mathieu-Daudé wrote: Cc'ing Zoltán. On 11/19/19 7:58 AM, Volker Rümelin wrote: With current code audio recording with all audio backends except PulseAudio and DirectSound is broken. The generic audio recording buffer m

Re: [PATCH] audio: fix audio recording

On 2019-11-19 07:58, Volker Rümelin wrote: With current code audio recording with all audio backends except PulseAudio and DirectSound is broken. The generic audio recording buffer management forgot to update the current read position after a read. Indeed, pos_emul is updated in audio_generic_p

Re: [PATCH 2/2] migration/multifd: not use multifd during postcopy

On Tue, Nov 19, 2019 at 11:55:52AM +0100, Juan Quintela wrote: >Wei Yang wrote: >> We don't support multifd during postcopy, but user still could enable >> both multifd and postcopy. This leads to migration failure. >> >> Skip multifd during postcopy. >> >> Signed-off-by: Wei Yang > >Reviewed-by:

Re: [PATCH v3 18/33] mips: inline serial_init

On Wednesday, October 23, 2019, Marc-André Lureau < marcandre.lur...@redhat.com> wrote: > The function is specific to mipssim, let's inline it. > > Signed-off-by: Marc-André Lureau > --- > hw/char/serial.c | 16 > hw/mips/mips_mipssim.c | 13 ++--- > include/hw

Re: [PATCH for-5.0 v5 04/23] ppc/pnv: Dump the XIVE NVT table

On Fri, Nov 15, 2019 at 05:24:17PM +0100, Cédric Le Goater wrote: > This is useful to dump the saved contexts of the vCPUs : configuration > of the base END index of the vCPU and the Interrupt Pending Buffer > register, which is updated when an interrupt can not be presented. > > When dumping the

Re: [PATCH] ppc/pnv: Create BMC devices at machine init

On Tue, Nov 19, 2019 at 07:49:33AM +0100, Cédric Le Goater wrote: > On 19/11/2019 01:52, David Gibson wrote: > > On Mon, Nov 18, 2019 at 10:22:22AM +0100, Cédric Le Goater wrote: > >> The BMC of the OpenPOWER systems monitors the machine state using > >> sensors, controls the power and controls the

Re: [PATCH for-5.0 v5 05/23] ppc/pnv: Quiesce some XIVE errors

On Fri, Nov 15, 2019 at 05:24:18PM +0100, Cédric Le Goater wrote: > When dumping the END and NVT tables, the error logging is too noisy. > > Signed-off-by: Cédric Le Goater Applied, thanks. > --- > hw/intc/pnv_xive.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git

Re: [PATCH] vfio: don't ignore return value of migrate_add_blocker

On Thu, 14 Nov 2019 14:34:49 +0100 Jens Freimann wrote: > When an error occurs in migrate_add_blocker() it sets a > negative return value and uses error pointer we pass in. > Instead of just looking at the error pointer check for a negative return > value and avoid a coverity error because the re

Re: [PATCH v9 Kernel 2/5] vfio iommu: Add ioctl defination to get dirty pages bitmap.

On Fri, 15 Nov 2019 05:10:53 + "Tian, Kevin" wrote: > > From: Alex Williamson > > Sent: Friday, November 15, 2019 11:22 AM > > > > On Thu, 14 Nov 2019 21:40:35 -0500 > > Yan Zhao wrote: > > > > > On Fri, Nov 15, 2019 at 05:06:25AM +0800, Alex Williamson wrote: > > > > On Fri, 15 Nov 20

[ANNOUNCE] QEMU 4.2.0-rc2 is now available

Hello, On behalf of the QEMU Team, I'd like to announce the availability of the third release candidate for the QEMU 4.2 release. This release is meant for testing purposes and should not be used in a production environment. http://download.qemu-project.org/qemu-4.2.0-rc2.tar.xz http://downl

Re: [PATCH 2/5] hw/ppc/pnv_xscom: PBA bar mask values are incorrect with homer/occ sizes

On Tue, Nov 19, 2019 at 11:20:53PM +0530, Balamuruhan S wrote: > homer/occ sizes are calculated in skiboot with `(mask | 0xf) + 1`, > and from xscom access should return correct mask values instead of actual > sizes. > > Signed-off-by: Cédric Le Goater > Signed-off-by: Balamuruhan S > --- >

Re: [PATCH for-5.0] xive/kvm: Trigger interrupts from userspace

On Tue, Nov 19, 2019 at 09:15:52AM +0100, Cédric Le Goater wrote: > On 19/11/2019 01:47, David Gibson wrote: > > On Mon, Nov 18, 2019 at 04:37:16PM +0100, Cédric Le Goater wrote: > >> On 18/11/2019 16:12, Greg Kurz wrote: > >>> When using the XIVE KVM device, the trigger page is directly accessible

Re: [PATCH 2/5] hw/ppc/pnv_xscom: PBA bar mask values are incorrect with homer/occ sizes

On Wed, Nov 20, 2019 at 09:00:32AM +1100, David Gibson wrote: > On Wed, Nov 20, 2019 at 08:56:18AM +1100, David Gibson wrote: > > On Tue, Nov 19, 2019 at 11:20:53PM +0530, Balamuruhan S wrote: > > > homer/occ sizes are calculated in skiboot with `(mask | 0xf) + 1`, > > > and from xscom access s

Re: [PATCH 2/5] hw/ppc/pnv_xscom: PBA bar mask values are incorrect with homer/occ sizes

On Wed, Nov 20, 2019 at 08:56:18AM +1100, David Gibson wrote: > On Tue, Nov 19, 2019 at 11:20:53PM +0530, Balamuruhan S wrote: > > homer/occ sizes are calculated in skiboot with `(mask | 0xf) + 1`, > > and from xscom access should return correct mask values instead of actual > > sizes. > > > >

Re: [PULL for-4.2-rc2 0/2] Tracing patches

On Tue, Nov 19, 2019 at 10:14 PM Aleksandar Markovic wrote: > > On Tue, Nov 19, 2019 at 9:46 PM Stefan Hajnoczi wrote: > > > > The following changes since commit f086f22d6c068ba151b0f6e81e75a64f130df712: > > > > Merge remote-tracking branch > > 'remotes/awilliam/tags/vfio-fixes-20191118.0' int

Re: [PATCH v35 05/13] target/avr: Add instruction translation - Arithmetic and Logic Instructions

Sounds good, thanks! On Tue, Nov 19, 2019 at 11:18 PM Aleksandar Markovic wrote: > > On Tue, Nov 19, 2019 at 9:09 PM Michael Rolnik wrote: > > > > Hi Aleksandar et al. > > > > how is it going? should I rebase or not? > > > > Michael > > > > I am just in the process of taking a detailed look to t

Re: [PATCH v35 05/13] target/avr: Add instruction translation - Arithmetic and Logic Instructions

On Tue, Nov 19, 2019 at 9:09 PM Michael Rolnik wrote: > > Hi Aleksandar et al. > > how is it going? should I rebase or not? > > Michael > I am just in the process of taking a detailed look to the whole series. I don't have any serious objection so far, I may just suggest a couple of things that

Re: [PULL for-4.2-rc2 0/2] Tracing patches

On Tue, Nov 19, 2019 at 9:46 PM Stefan Hajnoczi wrote: > > The following changes since commit f086f22d6c068ba151b0f6e81e75a64f130df712: > > Merge remote-tracking branch 'remotes/awilliam/tags/vfio-fixes-20191118.0' > into staging (2019-11-18 21:35:48 +) > > are available in the Git reposito

Re: [PATCH] Fix incorrect int->float conversions caught by clang -Wimplicit-int-float-conversion

Fangrui Song writes: > The warning will be enabled by default in clang 10. It is not available for > clang <= 9. > > qemu/migration/migration.c:2038:24: error: implicit conversion from 'long' to > 'double' changes value from 9223372036854775807 to 9223372036854775808 > [-Werror,-Wimplicit-

[PULL for-4.2-rc2 2/2] hw/mips/gt64xxx: Remove dynamic field width from trace events

From: Philippe Mathieu-Daudé Since not all trace backends support dynamic field width in format (dtrace via stap does not), replace by a static field width instead. We previously passed to the trace API 'width << 1' as the number of hex characters to display (the dynamic field width). We don't n

[PULL for-4.2-rc2 0/2] Tracing patches

The following changes since commit f086f22d6c068ba151b0f6e81e75a64f130df712: Merge remote-tracking branch 'remotes/awilliam/tags/vfio-fixes-20191118.0' into staging (2019-11-18 21:35:48 +) are available in the Git repository at: https://github.com/stefanha/qemu.git tags/tracing-pull-req

[PULL for-4.2-rc2 1/2] hw/block/pflash: Remove dynamic field width from trace events

From: Philippe Mathieu-Daudé Since not all trace backends support dynamic field width in format (dtrace via stap does not), replace by a static field width instead. We previously passed to the trace API 'width << 1' as the number of hex characters to display (the dynamic field width). We don't n

Re: [PATCH] linux-user/strace: Add missing signal strings

On 11/19/19 9:06 PM, Helge Deller wrote: >>> +#ifdef SIGSTKFLT >>> +    case TARGET_SIGSTKFLT: signal_name = "SIGSTKFLT"; break; >>> +#endif >> >> Wrong ifdef. > > Same here, see in linux-user/signal.c (line 50): > #ifdef SIGSTKFLT >     [SIGSTKFLT] = TARGET_SIGSTKFLT, > #endif But that one actua

Re: [PATCH v35 05/13] target/avr: Add instruction translation - Arithmetic and Logic Instructions

Hi Aleksandar et al. how is it going? should I rebase or not? Michael On Tue, Nov 5, 2019 at 4:38 PM Aleksandar Markovic wrote: > > On Tue, Nov 5, 2019 at 2:23 PM Richard Henderson > wrote: > > > > On 11/5/19 10:46 AM, Aleksandar Markovic wrote: > > > > > > > > > On Tuesday, November 5, 2019,

Re: [PATCH] linux-user/strace: Add missing signal strings

On 19.11.19 21:06, Helge Deller wrote: On 19.11.19 20:56, Richard Henderson wrote: On 11/19/19 7:51 PM, Helge Deller wrote: Add the textual representations of some missing target signals. Signed-off-by: Helge Deller diff --git a/linux-user/strace.c b/linux-user/strace.c index 3d4d684450..18b

Re: [PATCH] linux-user/strace: Add missing signal strings

On 19.11.19 20:56, Richard Henderson wrote: On 11/19/19 7:51 PM, Helge Deller wrote: Add the textual representations of some missing target signals. Signed-off-by: Helge Deller diff --git a/linux-user/strace.c b/linux-user/strace.c index 3d4d684450..18b57a9ef9 100644 --- a/linux-user/strace.c

Re: [PATCH v2 08/20] nvme: add support for the get log page command

On Tue, Nov 12, 2019 at 03:04:52PM +, Beata Michalska wrote: > Hi Klaus, > > > On Tue, 15 Oct 2019 at 11:45, Klaus Jensen wrote: > > +if (!nsid || (nsid != 0x && nsid > n->num_namespaces)) { > > +trace_nvme_err_invalid_ns(nsid, n->num_namespaces); > > +return NVME

Re: [PATCH] linux-user/strace: Add missing signal strings

On 11/19/19 7:51 PM, Helge Deller wrote: > Add the textual representations of some missing target signals. > > Signed-off-by: Helge Deller > > diff --git a/linux-user/strace.c b/linux-user/strace.c > index 3d4d684450..18b57a9ef9 100644 > --- a/linux-user/strace.c > +++ b/linux-user/strace.c > @@

Re: [PATCH v2 09/20] nvme: add support for the asynchronous event request command

On Tue, Nov 12, 2019 at 03:04:59PM +, Beata Michalska wrote: > Hi Klaus, > > On Tue, 15 Oct 2019 at 11:49, Klaus Jensen wrote: > > @@ -1188,6 +1326,9 @@ static int nvme_start_ctrl(NvmeCtrl *n) > > > > nvme_set_timestamp(n, 0ULL); > > > > +n->aer_timer = timer_new_ns(QEMU_CLOCK_VIRTUA

Re: [PATCH] audio: fix audio recording

On 11/19/19 9:01 AM, Philippe Mathieu-Daudé wrote: > Cc'ing Zoltán. > > On 11/19/19 7:58 AM, Volker Rümelin wrote: >> With current code audio recording with all audio backends >> except PulseAudio and DirectSound is broken. The generic audio >> recording buffer management forgot to update the curr

Re: [PATCH v1 0/2] s390x/cpumodel: Introduce "best" model variants

On Tue, Nov 19, 2019 at 12:00:14PM +0100, David Hildenbrand wrote: > On 19.11.19 11:36, Peter Maydell wrote: > > On Tue, 19 Nov 2019 at 09:59, David Hildenbrand wrote: > > > > > > On 19.11.19 10:22, Peter Maydell wrote: > > > > I don't hugely care about query-cpu-model-expansion. I > > > > just d

Re: [PATCH] Add minimal Hexagon target - First in a series of patches - linux-user changes + linux-user/hexagon + skeleton of target/hexagon - Files in target/hexagon/imported are from another project

On 11/19/19 6:22 PM, Taylor Simpson wrote: > - Laurent suggested I split the patch into two parts: linux-user and > target/hexagon. If I do that, which one should contain the changes to common > files (e.g., configure)? Also, note that we won't be able to build until > both patches are merged.

Re: [PATCH] Add minimal Hexagon target - First in a series of patches - linux-user changes + linux-user/hexagon + skeleton of target/hexagon - Files in target/hexagon/imported are from another project

On 11/19/19 12:58 AM, Taylor Simpson wrote: > +static abi_ulong get_sigframe(struct target_sigaction *ka, > + CPUHexagonState *regs, size_t framesize) > +{ > +abi_ulong sp = get_sp_from_cpustate(regs); > + > +/* This is the X/Open sanctioned signal stack switchi

Re: [PATCH] Hexagon: Swap SIGRGMAX-1 and SIGRTMIN+1

On Tue, 19 Nov 2019 at 19:07, Taylor Simpson wrote: > > Signed-off-by: Taylor Simpson > --- > linux-user/signal.c | 8 > 1 file changed, 8 insertions(+) > > diff --git a/linux-user/signal.c b/linux-user/signal.c > index 5ca6d62..ce3d27f 100644 > --- a/linux-user/signal.c > +++ b/linux-u

[PATCH] Hexagon: Swap SIGRGMAX-1 and SIGRTMIN+1

Signed-off-by: Taylor Simpson --- linux-user/signal.c | 8 1 file changed, 8 insertions(+) diff --git a/linux-user/signal.c b/linux-user/signal.c index 5ca6d62..ce3d27f 100644 --- a/linux-user/signal.c +++ b/linux-user/signal.c @@ -72,6 +72,14 @@ static uint8_t host_to_target_signal_tab

[PATCH] linux-user/strace: Improve output of various syscalls

Improve strace output of various syscalls which either have none or only int-type parameters. Signed-off-by: Helge Deller diff --git a/linux-user/strace.list b/linux-user/strace.list index 1de4319dcf..5163717087 100644 --- a/linux-user/strace.list +++ b/linux-user/strace.list @@ -26,7 +26,7 @@

[PATCH] linux-user/strace: Add missing signal strings

Add the textual representations of some missing target signals. Signed-off-by: Helge Deller diff --git a/linux-user/strace.c b/linux-user/strace.c index 3d4d684450..18b57a9ef9 100644 --- a/linux-user/strace.c +++ b/linux-user/strace.c @@ -146,6 +146,22 @@ print_signal(abi_ulong arg, int last)

Re: [PATCH 2/6] iotests: Skip test 079 if it is not possible to create large files

On 19/11/2019 18.50, Daniel P. Berrangé wrote: > On Tue, Nov 19, 2019 at 06:38:20PM +0100, Philippe Mathieu-Daudé wrote: >> On 11/19/19 6:34 PM, Thomas Huth wrote: >>> On 19/11/2019 18.29, Philippe Mathieu-Daudé wrote: On 11/19/19 6:08 PM, Thomas Huth wrote: > Test 079 fails in the arm64,

Re: [PULL 00/12] Misc patches for QEMU 4.2-rc

On Tue, 19 Nov 2019 at 16:10, Paolo Bonzini wrote: > > The following changes since commit 369e8f5bbd8a5301bde6fae22b93fe9288c552a5: > > buildfix: update texinfo menu (2019-11-18 10:33:29 +) > > are available in the git repository at: > > git://github.com/bonzini/qemu.git tags/for-upstream

Re: [RFC v5 065/126] XIVE: introduce ERRP_AUTO_PROPAGATE

On Fri, 11 Oct 2019 19:04:51 +0300 Vladimir Sementsov-Ogievskiy wrote: > If we want to add some info to errp (by error_prepend() or > error_append_hint()), we must use the ERRP_AUTO_PROPAGATE macro. > Otherwise, this info will not be added when errp == &fatal_err > (the program will exit prior to

Re: [PATCH] Add minimal Hexagon target - First in a series of patches - linux-user changes + linux-user/hexagon + skeleton of target/hexagon - Files in target/hexagon/imported are from another project

Le 19/11/2019 à 18:22, Taylor Simpson a écrit : > Thanks for all the feedback on the patch. I'll summarize my TODO list here. > Please let me know if there's anything I missed. > - Add a README file in the imported directory to make it clear that the code > comes from another project. Personal

Re: test-aio-multithread assertion

On Tue, 19 Nov 2019 at 17:55, Peter Maydell wrote: > Here's a gdb backtrace from as assertion failure under NetBSD > when running ./tests/test-aio-multithread -p /aio/multi/mutex/contended > (which is pretty replicable for me); does it look like the same > thing as the commit 2362a28ea1 issue? (Ap

Re: [RFC v5 035/126] PowerPC Machines: introduce ERRP_AUTO_PROPAGATE

On Fri, 11 Oct 2019 19:04:21 +0300 Vladimir Sementsov-Ogievskiy wrote: > If we want to add some info to errp (by error_prepend() or > error_append_hint()), we must use the ERRP_AUTO_PROPAGATE macro. > Otherwise, this info will not be added when errp == &fatal_err > (the program will exit prior to

Re: [RFC PATCH v1 1/8] vfio-ccw: Return IOINST_CC_NOT_OPERATIONAL for EIO

On Tue, 19 Nov 2019 13:02:20 +0100 Cornelia Huck wrote: > On Tue, 19 Nov 2019 12:23:40 +0100 > Halil Pasic wrote: > > > On Mon, 18 Nov 2019 19:13:34 +0100 > > Cornelia Huck wrote: > > > > > > EIO is returned by vfio-ccw mediated device when the backing > > > > host subchannel is not operation

Re: [PATCH] misc/pca9552: Add qom set and get

On 11/19/19 5:27 PM, Cédric Le Goater wrote: On 18/11/2019 18:48, Philippe Mathieu-Daudé wrote: Odd... I only received Cédric answer, not Joel patch. On 11/18/19 11:00 AM, Cédric Le Goater wrote: On 18/11/2019 07:17, Joel Stanley wrote: Following the pattern of the work recently done with the

[PATCH 3/5] hw/ppc/pnv_xscom: Power8 occ common area is in PBA BAR 3

Fix incorrect PBA BAR and BARMASK value for Power8 occ common area region where skiboot enum declaration have it in BAR 3 and BARMASK is calculated BARMASK0 + BAR, enum P8_BAR { P8_BAR_HOMER = 0, P8_BAR_CENTAUR = 1, P8_BAR_SLW = 2, P8_BAR_OCC_COMMON = 3, }; Signed-

Re: test-aio-multithread assertion

On Mon, 14 Oct 2019 at 11:50, Paolo Bonzini wrote: > > On 14/10/19 12:26, Peter Maydell wrote: > > On Thu, 5 Sep 2019 at 14:17, Peter Maydell wrote: > >> Here's a different intermittent assertion on Netbsd from > >> test-aio-multithread: > >> > >> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:

[PATCH 1/5] hw/ppc/pnv: incorrect homer and occ common area size

Homer size is 4MB and OCC common area size is 8MB, but currently it is assigned with bar mask value. Also pass on the occ sram size 3 bits right shifted to initialize the size appropriately. Signed-off-by: Cédric Le Goater Signed-off-by: Balamuruhan S --- hw/ppc/pnv_occ.c | 2 +- include/hw

[PATCH 4/5] hw/ppc/pnv_xscom: occ common area to be mapped only once

occ common area should be mapped once and disable it for every other chip. Signed-off-by: Cédric Le Goater Signed-off-by: Balamuruhan S --- hw/ppc/pnv_xscom.c | 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/hw/ppc/pnv_xscom.c b/hw/ppc/pnv_xscom.c index cb6d6

[PATCH 2/5] hw/ppc/pnv_xscom: PBA bar mask values are incorrect with homer/occ sizes

homer/occ sizes are calculated in skiboot with `(mask | 0xf) + 1`, and from xscom access should return correct mask values instead of actual sizes. Signed-off-by: Cédric Le Goater Signed-off-by: Balamuruhan S --- hw/ppc/pnv_xscom.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletio

[PATCH 5/5] hw/ppc/pnv_xscom: add PBA BARs for Power8 slw image

slw base and size mask are accessed during boot in homer_init_chip(), so include BAR2 and BARMASK2 for Power8. Signed-off-by: Balamuruhan S --- hw/ppc/pnv_xscom.c | 10 -- include/hw/ppc/pnv.h | 4 2 files changed, 12 insertions(+), 2 deletions(-) diff --git a/hw/ppc/pnv_xscom.c

Re: [PATCH 2/6] iotests: Skip test 079 if it is not possible to create large files

On Tue, Nov 19, 2019 at 06:38:20PM +0100, Philippe Mathieu-Daudé wrote: > On 11/19/19 6:34 PM, Thomas Huth wrote: > > On 19/11/2019 18.29, Philippe Mathieu-Daudé wrote: > > > On 11/19/19 6:08 PM, Thomas Huth wrote: > > > > Test 079 fails in the arm64, s390x and ppc64le LXD containers, which > > > >

[PATCH 0/5] ppc/pnv: fix Homer/Occ mappings on multichip systems

Hi All, PowerNV fails to boot in multichip systems due to some misinterpretation and mapping in Homer/Occ device models, this patchset fixes the following, - Homer size is 4MB per chip and Occ common area size is 8MB - Bar masks are used to calculate sizes of Homer/Occ in skiboot so return a

[Bug 1844817] Re: trace: dynamic width format syntax not validated

** Changed in: qemu Status: New => Fix Committed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1844817 Title: trace: dynamic width format syntax not validated Status in QEMU: Fix Committe

Re: [PATCH 2/6] iotests: Skip test 079 if it is not possible to create large files

On 11/19/19 6:34 PM, Thomas Huth wrote: On 19/11/2019 18.29, Philippe Mathieu-Daudé wrote: On 11/19/19 6:08 PM, Thomas Huth wrote: Test 079 fails in the arm64, s390x and ppc64le LXD containers, which apparently do not allow large files to be created. Test 079 tries to create a 4G sparse file, s

Re: [PATCH] Add minimal Hexagon target - First in a series of patches - linux-user changes + linux-user/hexagon + skeleton of target/hexagon - Files in target/hexagon/imported are from another project

On Tue, 19 Nov 2019 at 17:23, Taylor Simpson wrote: > - Laurent suggested I split the patch into two parts: linux-user and > target/hexagon. If I do that, which one should contain the changes to common > files (e.g., configure)? Also, note that we won't be able to build until > both patches a

Re: [PATCH 2/6] iotests: Skip test 079 if it is not possible to create large files

On 19/11/2019 18.29, Philippe Mathieu-Daudé wrote: > On 11/19/19 6:08 PM, Thomas Huth wrote: >> Test 079 fails in the arm64, s390x and ppc64le LXD containers, which >> apparently do not allow large files to be created. Test 079 tries to >> create a 4G sparse file, so check first whether we can real

Re: [PATCH 4/6] tests/test-util-filemonitor: Skip test on non-x86 Travis containers

On 11/19/19 6:08 PM, Thomas Huth wrote: test-util-filemonitor fails in restricted non-x86 Travis containers since they apparently blacklisted some required system calls there. Let's simply skip the test if we detect such an environment. Signed-off-by: Thomas Huth --- tests/test-util-filemonit

  1   2   3   >