Re: [RFC PATCH v4 17/29] Hexagon (target/hexagon) generator phase 1 - C preprocessor for semantics

2020-09-29 Thread Philippe Mathieu-Daudé
On 9/28/20 7:28 PM, Taylor Simpson wrote: > Run the C preprocessor across the instruction definition files and macro > definitoin file to expand macros and prepare the semantics_generated.pyinc Typo "definition". > file. The resulting file contains one entry with the semantics for each >

Re: [PULL 1/1] tests/acpi: drop unnecessary files

2020-09-29 Thread Thomas Huth
On 29/09/2020 13.13, Michael S. Tsirkin wrote: > The last pull added a ton of useless files by mistake. > Drop them all. > > Fixes: 0ed93f4c05896 ("update golden master DSDT binary table blobs for q35") > Signed-off-by: Michael S. Tsirkin > --- > tests/test-qapi-event.c |

Re: [PATCH v10 13/13] tests/acpi: add DSDT.hpbrroot DSDT table blob to test global i440fx hotplug

2020-09-29 Thread Michael S. Tsirkin
On Tue, Sep 29, 2020 at 04:35:50PM +0530, Ani Sinha wrote: > On Tue, Sep 29, 2020 at 4:25 PM Michael S. Tsirkin wrote: > > > > On Tue, Sep 29, 2020 at 04:11:45PM +0530, Ani Sinha wrote: > > > On Tue, Sep 29, 2020 at 4:07 PM Michael S. Tsirkin > > > wrote: > > > > > > > > On Tue, Sep 29, 2020 at

Re: [RFC PATCH v4 15/29] Hexagon (target/hexagon) utility functions

2020-09-29 Thread Philippe Mathieu-Daudé
On 9/28/20 7:28 PM, Taylor Simpson wrote: > Utility functions called by various instructions > > Signed-off-by: Taylor Simpson > --- > target/hexagon/arch.h | 42 +++ > target/hexagon/conv_emu.h | 50 +++ > target/hexagon/fma_emu.h | 27 ++ > target/hexagon/arch.c | 354

Re: [PULL v4 00/48] virtio,pc,acpi: fixes, tests

2020-09-29 Thread Peter Maydell
On Tue, 29 Sep 2020 at 08:21, Michael S. Tsirkin wrote: > > Changes from v3: > - dropped reconnect tests > - added more fixes, an smbios feature > > The following changes since commit 74504514b154ebebdff577d88e4bf5c13074e9ed: > > Merge remote-tracking branch

Re: [PULL v4 00/48] virtio,pc,acpi: fixes, tests

2020-09-29 Thread Michael S. Tsirkin
On Tue, Sep 29, 2020 at 12:07:01PM +0100, Peter Maydell wrote: > On Tue, 29 Sep 2020 at 12:04, Michael S. Tsirkin wrote: > > > > On Tue, Sep 29, 2020 at 03:20:52AM -0400, Michael S. Tsirkin wrote: > > > Changes from v3: > > > - dropped reconnect tests > > > - added more fixes, an smbios feature >

[PATCH v6 14/14] tests/acceptance: add reverse debugging test

2020-09-29 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This is a test for GDB reverse debugging commands: reverse step and reverse continue. Every test in this suite consists of two phases: record and replay. Recording saves the execution of some instructions and makes an initial VM snapshot to allow reverse execution. Replay

Re: [PATCH v10 13/13] tests/acpi: add DSDT.hpbrroot DSDT table blob to test global i440fx hotplug

2020-09-29 Thread Ani Sinha
On Tue, Sep 29, 2020 at 4:28 PM Michael S. Tsirkin wrote: > > On Tue, Sep 29, 2020 at 04:20:15PM +0530, Ani Sinha wrote: > > as > > > > On Tue, Sep 29, 2020 at 4:11 PM Ani Sinha wrote: > > > > > > On Tue, Sep 29, 2020 at 4:07 PM Michael S. Tsirkin > > > wrote: > > > > > > > > On Tue, Sep 29,

[PATCH v6 12/14] replay: describe reverse debugging in docs/replay.txt

2020-09-29 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This patch updates the documentation and describes usage of the reverse debugging in QEMU+GDB. Signed-off-by: Pavel Dovgalyuk Reviewed-by: Alex Bennée -- v4 changes: - added an example of the command line for reverse debugging of the diskless machine ---

[PULL 0/1] acpi: fixup

2020-09-29 Thread Michael S. Tsirkin
The following changes since commit 213057383c9f73a17cfe635b204d88e11f918df1: Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into staging (2020-09-29 11:10:29 +0100) are available in the Git repository at: git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_upstream

[PATCH v6 11/14] gdbstub: add reverse continue support in replay mode

2020-09-29 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This patch adds support of the reverse continue operation for gdbstub. Reverse continue finds the last breakpoint that would happen in normal execution from the beginning to the current moment. Implementation of the reverse continue replays the execution twice: to find the

Re: [PULL v4 00/48] virtio,pc,acpi: fixes, tests

2020-09-29 Thread Peter Maydell
On Tue, 29 Sep 2020 at 12:04, Michael S. Tsirkin wrote: > > On Tue, Sep 29, 2020 at 03:20:52AM -0400, Michael S. Tsirkin wrote: > > Changes from v3: > > - dropped reconnect tests > > - added more fixes, an smbios feature > > > > The following changes since commit

[PATCH v6 10/14] gdbstub: add reverse step support in replay mode

2020-09-29 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk GDB remote protocol supports two reverse debugging commands: reverse step and reverse continue. This patch adds support of the first one to the gdbstub. Reverse step is intended to step one instruction in the backwards direction. This is not possible in regular execution.

Re: [PATCH v10 13/13] tests/acpi: add DSDT.hpbrroot DSDT table blob to test global i440fx hotplug

2020-09-29 Thread Ani Sinha
On Tue, Sep 29, 2020 at 4:25 PM Michael S. Tsirkin wrote: > > On Tue, Sep 29, 2020 at 04:11:45PM +0530, Ani Sinha wrote: > > On Tue, Sep 29, 2020 at 4:07 PM Michael S. Tsirkin wrote: > > > > > > On Tue, Sep 29, 2020 at 04:02:07PM +0530, Ani Sinha wrote: > > > > On Tue, Sep 29, 2020 at 4:00 PM

[PATCH v6 06/14] replay: introduce info hmp/qmp command

2020-09-29 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This patch introduces 'info replay' monitor command and corresponding qmp request. These commands request the current record/replay mode, replay log file name, and the instruction count (number of recorded/replayed instructions). The instruction count can be used with the

Re: [PULL v4 00/48] virtio,pc,acpi: fixes, tests

2020-09-29 Thread Michael S. Tsirkin
On Tue, Sep 29, 2020 at 03:20:52AM -0400, Michael S. Tsirkin wrote: > Changes from v3: > - dropped reconnect tests > - added more fixes, an smbios feature > > The following changes since commit 74504514b154ebebdff577d88e4bf5c13074e9ed: > > Merge remote-tracking branch

[PATCH v6 09/14] replay: flush rr queue before loading the vmstate

2020-09-29 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk Non-empty record/replay queue prevents saving and loading the VM state, because it includes pending bottom halves and block coroutines. But when the new VM state is loaded, we don't have to preserve the consistency of the current state anymore. Therefore this patch just

[PATCH v6 13/14] replay: create temporary snapshot at debugger connection

2020-09-29 Thread Pavel Dovgalyuk
When record/replay does not uses overlays for storing the snapshots, user is not capable of issuing reverse debugging commands. This patch adds creation of the VM snapshot on the temporary overlay image, when the debugger connects to QEMU. Therefore the execution can be rewind to the moment of the

[PATCH v6 03/14] qcow2: introduce icount field for snapshots

2020-09-29 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This patch introduces the icount field for saving within the snapshot. It is required for navigation between the snapshots in record/replay mode. Signed-off-by: Pavel Dovgalyuk Acked-by: Kevin Wolf --- block/qcow2-snapshot.c |7 +++ block/qcow2.h |3

[PATCH v6 04/14] migration: introduce icount field for snapshots

2020-09-29 Thread Pavel Dovgalyuk
Saving icount as a parameters of the snapshot allows navigation between them in the execution replay scenario. This information can be used for finding a specific snapshot for proceeding the recorded execution to the specific moment of the time. E.g., 'reverse step' action (introduced in one of

Re: [PATCH v26 05/17] vfio: Add VM state change handler to know state of VM

2020-09-29 Thread Dr. David Alan Gilbert
* Cornelia Huck (coh...@redhat.com) wrote: > On Wed, 23 Sep 2020 04:54:07 +0530 > Kirti Wankhede wrote: > > > VM state change handler gets called on change in VM's state. This is used > > to set > > VFIO device state to _RUNNING. > > > > Signed-off-by: Kirti Wankhede > > Reviewed-by: Neo Jia

[PATCH v6 08/14] replay: implement replay-seek command

2020-09-29 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This patch adds hmp/qmp commands replay_seek/replay-seek that proceed the execution to the specified instruction count. The command automatically loads nearest snapshot and replays the execution to find the desired instruction count. Signed-off-by: Pavel Dovgalyuk

[PATCH v6 02/14] replay: provide an accessor for rr filename

2020-09-29 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This patch adds an accessor function for the name of the record/replay log file. Adding an accessor instead of making variable global, prevents accidental modification of this variable by other modules. Signed-off-by: Pavel Dovgalyuk Reviewed-by: Alex Bennée Reviewed-by:

[PATCH v6 05/14] qapi: introduce replay.json for record/replay-related stuff

2020-09-29 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This patch adds replay.json file. It will be used for adding record/replay-related data structures and commands. Signed-off-by: Pavel Dovgalyuk Reviewed-by: Markus Armbruster Reviewed-by: Alex Bennée --- MAINTAINERS |1 + include/sysemu/replay.h |1

Re: [PATCH] PoC: Rust binding for QAPI (qemu-ga only, for now)

2020-09-29 Thread Paolo Bonzini
On 29/09/20 12:34, Marc-André Lureau wrote: > That would not be backwards compatible as you would have to set all > optional fields.  Every time the command grows a new optional argument, > all clients would have to specify it; if a command becomes optional, > you'd have to add

[PATCH v6 07/14] replay: introduce breakpoint at the specified step

2020-09-29 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This patch introduces replay_break, replay_delete_break qmp and hmp commands. These commands allow stopping at the specified instruction. It may be useful for debugging when there are some known events that should be investigated. replay_break command has one argument -

[PATCH v6 00/14] Reverse debugging

2020-09-29 Thread Pavel Dovgalyuk
GDB remote protocol supports reverse debugging of the targets. It includes 'reverse step' and 'reverse continue' operations. The first one finds the previous step of the execution, and the second one is intended to stop at the last breakpoint that would happen when the program is executed

[PATCH v6 01/14] replay: don't record interrupt poll

2020-09-29 Thread Pavel Dovgalyuk
Interrupt poll is not a real interrupt event. It is needed only for thread safety. This interrupt is used for i386 and converted to hardware interrupt by cpu_handle_interrupt function. Therefore it is not needed to be recorded, because hardware interrupt will be recorded after converting.

[Bug 1897680] Re: memory address over 0x2000_7ffc is not accessible in mps2-an505

2020-09-29 Thread Peter Maydell
** Changed in: qemu Status: New => Invalid -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1897680 Title: memory address over 0x2000_7ffc is not accessible in mps2-an505 Status in QEMU:

[PATCH v11 1/3] misc: Add versal-usb2-ctrl-regs module

2020-09-29 Thread Sai Pavan Boddu
This module emulates control registers of versal usb2 controller, this is added just to make guest happy. In general this module would control the phy-reset signal from usb controller, data coherency of the transactions, signals the host system errors received from controller. Signed-off-by: Sai

Re: [PATCH v10 13/13] tests/acpi: add DSDT.hpbrroot DSDT table blob to test global i440fx hotplug

2020-09-29 Thread Michael S. Tsirkin
On Tue, Sep 29, 2020 at 04:20:15PM +0530, Ani Sinha wrote: > as > > On Tue, Sep 29, 2020 at 4:11 PM Ani Sinha wrote: > > > > On Tue, Sep 29, 2020 at 4:07 PM Michael S. Tsirkin wrote: > > > > > > On Tue, Sep 29, 2020 at 04:02:07PM +0530, Ani Sinha wrote: > > > > On Tue, Sep 29, 2020 at 4:00 PM

Re: [PATCH v10 13/13] tests/acpi: add DSDT.hpbrroot DSDT table blob to test global i440fx hotplug

2020-09-29 Thread Michael S. Tsirkin
On Tue, Sep 29, 2020 at 04:11:45PM +0530, Ani Sinha wrote: > On Tue, Sep 29, 2020 at 4:07 PM Michael S. Tsirkin wrote: > > > > On Tue, Sep 29, 2020 at 04:02:07PM +0530, Ani Sinha wrote: > > > On Tue, Sep 29, 2020 at 4:00 PM Ani Sinha wrote: > > > > > > > > In your pull request the following

[PATCH v11 3/3] Versal: Connect DWC3 controller with virt-versal

2020-09-29 Thread Sai Pavan Boddu
From: Vikram Garhwal Connect dwc3 controller and usb2-reg module to xlnx-versal SOC, its placed in iou of lpd domain and configure it as dual port host controller. Add the respective guest dts nodes for "xlnx-versal-virt" machine. Signed-off-by: Vikram Garhwal Signed-off-by: Sai Pavan Boddu

[PATCH v11 0/3] Add Versal usb model

2020-09-29 Thread Sai Pavan Boddu
This patch series attempts to make 'hcd-xhci' an independent model so it can be used by both pci and system-bus interface. Changes for V2: Make XHCIState non-qom Use container_of functions for retriving pci device instance Initialize the AddressSpace pointer in PATCH 1/3 itself

[PATCH v11 2/3] usb: Add DWC3 model

2020-09-29 Thread Sai Pavan Boddu
From: Vikram Garhwal This patch adds skeleton model of dwc3 usb controller attached to xhci-sysbus device. It defines global register space of DWC3 controller, global registers control the AXI/AHB interfaces properties, external FIFO support and event count support. All of which are

Re: [PATCH v10 13/13] tests/acpi: add DSDT.hpbrroot DSDT table blob to test global i440fx hotplug

2020-09-29 Thread Ani Sinha
as On Tue, Sep 29, 2020 at 4:11 PM Ani Sinha wrote: > > On Tue, Sep 29, 2020 at 4:07 PM Michael S. Tsirkin wrote: > > > > On Tue, Sep 29, 2020 at 04:02:07PM +0530, Ani Sinha wrote: > > > On Tue, Sep 29, 2020 at 4:00 PM Ani Sinha wrote: > > > > > > > > In your pull request the following patch

Re: [PATCH v4 00/14] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set

2020-09-29 Thread Klaus Jensen
On Sep 28 22:54, Damien Le Moal wrote: > On 2020/09/29 6:25, Keith Busch wrote: > > On Mon, Sep 28, 2020 at 08:36:48AM +0200, Klaus Jensen wrote: > >> On Sep 28 02:33, Dmitry Fomichev wrote: > >>> You are making it sound like the entire WDC series relies on this > >>> approach. > >>> Actually,

Re: [PATCH v10 13/13] tests/acpi: add DSDT.hpbrroot DSDT table blob to test global i440fx hotplug

2020-09-29 Thread Ani Sinha
On Tue, Sep 29, 2020 at 4:07 PM Michael S. Tsirkin wrote: > > On Tue, Sep 29, 2020 at 04:02:07PM +0530, Ani Sinha wrote: > > On Tue, Sep 29, 2020 at 4:00 PM Ani Sinha wrote: > > > > > > In your pull request the following patch is completely screwed up: > > > > > > commit

Re: [PATCH v26 17/17] qapi: Add VFIO devices migration stats in Migration stats

2020-09-29 Thread Dr. David Alan Gilbert
* Kirti Wankhede (kwankh...@nvidia.com) wrote: > Added amount of bytes transferred to the target VM by all VFIO devices > > Signed-off-by: Kirti Wankhede > --- > > Note: Comments from v25 for this patch are not addressed yet. > https://www.mail-archive.com/qemu-devel@nongnu.org/msg715620.html >

Re: [PATCH v4] introduce vfio-user protocol specification

2020-09-29 Thread Stefan Hajnoczi
On Tue, Sep 15, 2020 at 07:29:17AM -0700, Thanos Makatos wrote: > This patch introduces the vfio-user protocol specification (formerly > known as VFIO-over-socket), which is designed to allow devices to be > emulated outside QEMU, in a separate process. vfio-user reuses the > existing VFIO

Re: [PATCH v4] introduce vfio-user protocol specification

2020-09-29 Thread Stefan Hajnoczi
On Mon, Sep 28, 2020 at 09:58:37AM +, Thanos Makatos wrote: > > It should be accompanied by a test in tests/. PCI-level testing APIS for > > BARs, configuration space, interrupts, etc are available in > > tests/qtest/libqos/pci.h. The test case needs to include a vfio-user > > device backend

Re: [PATCH v10 13/13] tests/acpi: add DSDT.hpbrroot DSDT table blob to test global i440fx hotplug

2020-09-29 Thread Michael S. Tsirkin
On Tue, Sep 29, 2020 at 04:02:07PM +0530, Ani Sinha wrote: > On Tue, Sep 29, 2020 at 4:00 PM Ani Sinha wrote: > > > > In your pull request the following patch is completely screwed up: > > > > commit cda2006eded0ed91974e1d9e7f9f288e65812a3e > > Author: Ani Sinha > > Date: Tue Sep 29 03:22:52

Re: [PATCH v10 13/13] tests/acpi: add DSDT.hpbrroot DSDT table blob to test global i440fx hotplug

2020-09-29 Thread Ani Sinha
On Tue, Sep 29, 2020 at 4:02 PM Ani Sinha wrote: > > On Tue, Sep 29, 2020 at 4:00 PM Ani Sinha wrote: > > > > In your pull request the following patch is completely screwed up: > > > > commit cda2006eded0ed91974e1d9e7f9f288e65812a3e > > Author: Ani Sinha > > Date: Tue Sep 29 03:22:52 2020

Re: [PATCH] PoC: Rust binding for QAPI (qemu-ga only, for now)

2020-09-29 Thread Marc-André Lureau
Hi On Tue, Sep 29, 2020 at 2:15 PM Paolo Bonzini wrote: > On 29/09/20 09:45, Marc-André Lureau wrote: > > Hi > > > > On Tue, Sep 22, 2020 at 9:08 PM Paolo Bonzini > > wrote: > > > So a QEMU D-Bus interface could have a name like org.qemu.Qemu51, > > >

Re: [PATCH qemu v9] spapr: Implement Open Firmware client interface

2020-09-29 Thread Alexey Kardashevskiy
On 16/07/2020 23:22, David Gibson wrote: On Thu, Jul 16, 2020 at 07:04:56PM +1000, Alexey Kardashevskiy wrote: Ping? I kinda realize it is not going to replace SLOF any time soon but still... Yeah, I know. I just haven't had time to consider it. Priority starvation. Still? :) On

Re: [PATCH v10 13/13] tests/acpi: add DSDT.hpbrroot DSDT table blob to test global i440fx hotplug

2020-09-29 Thread Ani Sinha
On Tue, Sep 29, 2020 at 4:00 PM Ani Sinha wrote: > > In your pull request the following patch is completely screwed up: > > commit cda2006eded0ed91974e1d9e7f9f288e65812a3e > Author: Ani Sinha > Date: Tue Sep 29 03:22:52 2020 -0400 > > tests/acpi: update golden master DSDT binary table

Re: [PATCH v10 13/13] tests/acpi: add DSDT.hpbrroot DSDT table blob to test global i440fx hotplug

2020-09-29 Thread Ani Sinha
In your pull request the following patch is completely screwed up: commit cda2006eded0ed91974e1d9e7f9f288e65812a3e Author: Ani Sinha Date: Tue Sep 29 03:22:52 2020 -0400 tests/acpi: update golden master DSDT binary table blobs for q35 This is not my patch. It has all sorts of changes

Re: [Bug 1897680] Re: memory address over 0x2000_7ffc is not accessible in mps2-an505

2020-09-29 Thread Changho Choi
Thank you for all your kind answers and suggestions. I also have confirmed that the memory address over 0x2000_8000 is not accessible in the real board. Regards, Changho Choi 2020년 9월 29일 (화) 오후 6:25, Peter Maydell <1897...@bugs.launchpad.net>님이 작성: > (If you were just interested in having a

[Bug 1857269] Re: Keyboard not fully working on Windows version

2020-09-29 Thread Thomas Huth
** Changed in: qemu Status: New => Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1857269 Title: Keyboard not fully working on Windows version Status in QEMU: Fix Released

Re: [PATCH] hw/rtc/m48t59: Simplify m48t59_init()

2020-09-29 Thread Philippe Mathieu-Daudé
On 9/24/20 5:00 PM, Laurent Vivier wrote: > Le 24/09/2020 à 16:53, Philippe Mathieu-Daudé a écrit : >> On 9/16/20 4:50 AM, David Gibson wrote: >>> On Mon, Sep 14, 2020 at 12:24:25PM +0200, Philippe Mathieu-Daudé wrote: As the 'io_base' argument of m48t59_init() is unused (set to 0),

Re: [PATCH v26 07/17] vfio: Register SaveVMHandlers for VFIO device

2020-09-29 Thread Dr. David Alan Gilbert
* Philippe Mathieu-Daudé (phi...@redhat.com) wrote: > On 9/23/20 1:24 AM, Kirti Wankhede wrote: > > Define flags to be used as delimeter in migration file stream. > > Typo "delimiter". > > > Added .save_setup and .save_cleanup functions. Mapped & unmapped migration > > region from these

Re: [PATCH] PoC: Rust binding for QAPI (qemu-ga only, for now)

2020-09-29 Thread Paolo Bonzini
On 29/09/20 09:45, Marc-André Lureau wrote: > Hi > > On Tue, Sep 22, 2020 at 9:08 PM Paolo Bonzini > wrote: > > So a QEMU D-Bus interface could have a name like org.qemu.Qemu51, > > org.qemu.Qemu52.. for example, if we can't provide better API >

Re: [PATCH] vfio-ccw: plug memory leak while getting region info

2020-09-29 Thread Cornelia Huck
On Mon, 28 Sep 2020 12:17:01 +0200 Cornelia Huck wrote: > vfio_get_dev_region_info() unconditionally allocates memory > for a passed-in vfio_region_info structure (and does not re-use > an already allocated structure). Therefore, we have to free > the structure we pass to that function in

Re: [PULL 0/1] migration queue

2020-09-29 Thread Peter Maydell
On Mon, 28 Sep 2020 at 16:23, Dr. David Alan Gilbert (git) wrote: > > From: "Dr. David Alan Gilbert" > > The following changes since commit 8d16e72f2d4df2c9e631393adf1669a1da7efe8a: > > Merge remote-tracking branch > 'remotes/dgilbert/tags/pull-migration-20200925a' into staging (2020-09-25 >

[PULL 7/9] virtio-gpu: set physical dimensions for EDID

2020-09-29 Thread Gerd Hoffmann
From: Marc-André Lureau Signed-off-by: Marc-André Lureau Message-id: 20200927145751.365446-7-marcandre.lur...@redhat.com Signed-off-by: Gerd Hoffmann --- include/hw/virtio/virtio-gpu.h | 1 + hw/display/virtio-gpu-base.c | 2 ++ hw/display/virtio-gpu.c| 2 ++ 3 files changed, 5

Re: Outline for VHOST_USER_PROTOCOL_F_VDPA

2020-09-29 Thread Michael S. Tsirkin
On Tue, Sep 29, 2020 at 09:57:51AM +0100, Stefan Hajnoczi wrote: > On Tue, Sep 29, 2020 at 02:09:55AM -0400, Michael S. Tsirkin wrote: > > On Mon, Sep 28, 2020 at 10:25:37AM +0100, Stefan Hajnoczi wrote: > > > Why extend vhost-user with vDPA? > > > > > > Reusing

[PULL 8/9] virtio-vga: implement big-endian-framebuffer property

2020-09-29 Thread Gerd Hoffmann
Allows to switch the (vga mode) framebuffer into bigendian mode by setting the property, simliar to stdvga. Signed-off-by: Gerd Hoffmann Message-id: 20200928085335.21961-2-kra...@redhat.com --- hw/display/virtio-vga.c | 19 +++ 1 file changed, 19 insertions(+) diff --git

[PULL 3/9] edid: use physical dimensions if available

2020-09-29 Thread Gerd Hoffmann
From: Marc-André Lureau Replace dpi with width_mm/height_mm in qemu_edid_info. Use it when set (non-zero) to compute the DPI and generate the EDID. Signed-off-by: Marc-André Lureau Message-id: 20200927145751.365446-3-marcandre.lur...@redhat.com Signed-off-by: Gerd Hoffmann ---

[PULL 4/9] ui: add getter for UIInfo

2020-09-29 Thread Gerd Hoffmann
From: Marc-André Lureau The following patch is going to introduce extra fields / details to UIInfo. Add a getter and keep the current values, instead of memset(0) Signed-off-by: Marc-André Lureau Message-id: 20200927145751.365446-4-marcandre.lur...@redhat.com Signed-off-by: Gerd Hoffmann ---

[PULL 1/9] virtio-gpu-3d: fix abnormal display after a warm reboot

2020-09-29 Thread Gerd Hoffmann
From: Guoqing Zhang When resetting virtio-gpu, virgl_renderer_reset() should be called to ensure that the virglrenderer status is correct. Signed-off-by: Guoqing Zhang Reviewed-by: Qi Liu Message-id: 20200918111632.37354-1-zhangguoqing.ker...@bytedance.com Signed-off-by: Gerd Hoffmann ---

[PULL 9/9] ppc/pseries: enable big-endian-framebuffer quirk for bochs-display and virtio-vga

2020-09-29 Thread Gerd Hoffmann
Already done for stdvga and secondary-vga, bochs-display and virtio-vga support the big-endian-framebuffer property too. Fixes blue console background at boot (offb firmware console). Buglink: https://bugzilla.redhat.com/show_bug.cgi?id=1881912 Signed-off-by: Gerd Hoffmann Message-id:

[PULL 6/9] spice: get monitors physical dimension

2020-09-29 Thread Gerd Hoffmann
From: Marc-André Lureau With upcoming Spice server version 0.14.4, the monitor configuration can contain additional fields for the monitor physical dimensions. Signed-off-by: Marc-André Lureau Message-id: 20200927145751.365446-6-marcandre.lur...@redhat.com Signed-off-by: Gerd Hoffmann ---

[PULL 0/9] Vga 20200929 patches

2020-09-29 Thread Gerd Hoffmann
The following changes since commit 74504514b154ebebdff577d88e4bf5c13074e9ed: Merge remote-tracking branch 'remotes/alistair/tags/pull-register-20200927'= into staging (2020-09-28 16:49:10 +0100) are available in the Git repository at: git://git.kraxel.org/qemu tags/vga-20200929-pull

[PULL 5/9] spice: remove the single monitor config logic

2020-09-29 Thread Gerd Hoffmann
From: Marc-André Lureau Introduced in commit 9c956e646178fee8c14ce7dfae5a9d7cb901876c ("spice: prepare for upcoming spice-server change"), the new logic never materialized in the spice server source tree. Let's remove it for now, until it actually changes in Spice. Signed-off-by: Marc-André

[PULL 2/9] edid: fix physical display size computation

2020-09-29 Thread Gerd Hoffmann
From: Marc-André Lureau Divide the resolution by the DPI, and multiply to mm. Note the computation done for edid[21/22] is correct (in cm). Signed-off-by: Marc-André Lureau Message-id: 20200927145751.365446-2-marcandre.lur...@redhat.com Signed-off-by: Gerd Hoffmann ---

Re: [PATCH 2/2] hw/arm/sbsa-ref : allocate IRQs for SMMUv3

2020-09-29 Thread Philippe Mathieu-Daudé
On 9/29/20 11:42 AM, Graeme Gregory wrote: > Original commit did not allocate IRQs for the SMMUv3 in the irqmap > effectively using irq 0->3 (shared with other devices). Assuming > original intent was to allocate unique IRQs then add an allocation > to the irqmap. > > Fixes: e9fdf453240 ("hw/arm:

Re: [PATCH 16/16] tests/acceptance: Test the MIPSsim machine

2020-09-29 Thread Philippe Mathieu-Daudé
On 9/29/20 11:38 AM, Alex Bennée wrote: > > Philippe Mathieu-Daudé writes: > >> +Alex/Daniel >> >> On 9/28/20 10:33 PM, Willian Rampazzo wrote: >>> On Mon, Sep 28, 2020 at 2:31 PM Philippe Mathieu-Daudé >>> wrote: Add a test for the mipssim machine, based on the recommended

Re: [PATCH v5 0/7] vhost-user-blk: fix the migration issue and enhance qtests

2020-09-29 Thread Dima Stepanov
On Tue, Sep 29, 2020 at 03:13:09AM -0400, Michael S. Tsirkin wrote: > On Sun, Sep 27, 2020 at 09:48:28AM +0300, Dima Stepanov wrote: > > On Thu, Sep 24, 2020 at 07:26:14AM -0400, Michael S. Tsirkin wrote: > > > On Fri, Sep 11, 2020 at 11:39:42AM +0300, Dima Stepanov wrote: > > > > v4 -> v5: > > >

Re: [RFC PATCH 0/7] block-backend: Introduce I/O hang

2020-09-29 Thread cenjiahui
On 2020/9/28 18:57, Kevin Wolf wrote: > Am 27.09.2020 um 15:04 hat Ying Fang geschrieben: >> A VM in the cloud environment may use a virutal disk as the backend storage, >> and there are usually filesystems on the virtual block device. When backend >> storage is temporarily down, any I/O issued

Re: [PATCH v6 09/21] docs/interop: Convert qemu-qmp-ref to rST

2020-09-29 Thread Peter Maydell
On Tue, 29 Sep 2020 at 09:42, Markus Armbruster wrote: > > Appears to break documented make target html: > > $ make -C bld-x86 html > make: Entering directory '/work/armbru/qemu/bld-x86' > make: *** No rule to make target 'html'. Stop. > make: Leaving directory '/work/armbru/qemu/bld-x86'

[PATCH 1/2] hw/arm/sbsa-ref : Fix SMMUv3 Initialisation

2020-09-29 Thread Graeme Gregory
SMMUv3 has an error in previous patch where a i was transposed to a 1 meaning interrupts would not have been correctly assigned to the SMMUv3 instance. Fixes: 48ba18e6d3f3 ("hw/arm/sbsa-ref: Simplify by moving the gic in the machine state") Signed-off-by: Graeme Gregory --- hw/arm/sbsa-ref.c |

[PATCH 2/2] hw/arm/sbsa-ref : allocate IRQs for SMMUv3

2020-09-29 Thread Graeme Gregory
Original commit did not allocate IRQs for the SMMUv3 in the irqmap effectively using irq 0->3 (shared with other devices). Assuming original intent was to allocate unique IRQs then add an allocation to the irqmap. Fixes: e9fdf453240 ("hw/arm: Add arm SBSA reference machine, devices part")

[PATCH 0/2] hw/arm/sbsa-ref : small fixes to smmuv3 initialisation

2020-09-29 Thread Graeme Gregory
Fix two issues with the smmuv3 initialisation, first where a previous patch had transposed an i to a 1. The second an assumption that the IRQs allocated were meant to be unique and not 0 based. Graeme Gregory (2): hw/arm/sbsa-ref : Fix SMMUv3 Initialisation hw/arm/sbsa-ref : allocate IRQs for

Re: [PATCH v10 13/13] tests/acpi: add DSDT.hpbrroot DSDT table blob to test global i440fx hotplug

2020-09-29 Thread Michael S. Tsirkin
On Tue, Sep 29, 2020 at 02:21:06PM +0530, Ani Sinha wrote: > On Tue, Sep 29, 2020 at 1:53 PM Ani Sinha wrote: > > > > On Tue, Sep 29, 2020 at 12:56 PM Michael S. Tsirkin wrote: > > > > > > On Tue, Sep 29, 2020 at 12:53:17PM +0530, Ani Sinha wrote: > > > > On Tue, Sep 29, 2020 at 12:42 PM Michael

Re: [Bug 1897680] [NEW] memory address over 0x2000_7ffc is not accessible in mps2-an505

2020-09-29 Thread Philippe Mathieu-Daudé
On 9/29/20 8:57 AM, Changho Choi wrote: > Public bug reported: > > I currently run qemu with the following options > `qemu-system-aarch64 -machine mps2-an505 -cpu cortex-m33 -m 16` > > For some reason, memory address over 0x2000_7ffc is not accessible. > It can be tested in gdb as follow. > >

Re: [PATCH v6 09/21] docs/interop: Convert qemu-qmp-ref to rST

2020-09-29 Thread Peter Maydell
On Tue, 29 Sep 2020 at 09:28, Markus Armbruster wrote: > > Peter Maydell writes: > > diff --git a/qapi/meson.build b/qapi/meson.build > > index 2b2872a41d8..a287ca5d9d7 100644 > > --- a/qapi/meson.build > > +++ b/qapi/meson.build > > @@ -97,7 +97,7 @@ foreach module : qapi_all_modules > >

Re: [PATCH 16/16] tests/acceptance: Test the MIPSsim machine

2020-09-29 Thread Alex Bennée
Philippe Mathieu-Daudé writes: > +Alex/Daniel > > On 9/28/20 10:33 PM, Willian Rampazzo wrote: >> On Mon, Sep 28, 2020 at 2:31 PM Philippe Mathieu-Daudé >> wrote: >>> >>> Add a test for the mipssim machine, based on the recommended >>> test setup from Thomas Huth: >>>

Re: [PULL 0/5] Usb 20200928 patches

2020-09-29 Thread Peter Maydell
On Mon, 28 Sep 2020 at 14:23, Gerd Hoffmann wrote: > > The following changes since commit 8d16e72f2d4df2c9e631393adf1669a1da7efe8a: > > Merge remote-tracking branch > 'remotes/dgilbert/tags/pull-migration-20200925a' into staging (2020-09-25 > 14:46:18 +0100) > > are available in the Git

Re: [PATCH v6 08/21] docs/interop: Convert qemu-ga-ref to rST

2020-09-29 Thread Peter Maydell
On Tue, 29 Sep 2020 at 09:20, Markus Armbruster wrote: > > Peter Maydell writes: > > > Convert qemu-ga-ref to rST format. This includes dropping > > the plain-text, pdf and info format outputs for this document; > > as with all our other Sphinx-based documentation, we provide > > HTML and

[Bug 1897680] Re: memory address over 0x2000_7ffc is not accessible in mps2-an505

2020-09-29 Thread Peter Maydell
(If you were just interested in having a large area of contiguous RAM, the "PSRAM" is the largest lump on this board: it's 16MB starting at 0x8000_.) -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU.

[Bug 1897680] Re: memory address over 0x2000_7ffc is not accessible in mps2-an505

2020-09-29 Thread Peter Maydell
PS: you don't need to pass "-cpu cortex-m33" as it is the default for the mps2-an505 board, and you don't need to pass a -m argument either, as 16MB is the fixed value for this board. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU.

[Bug 1897680] Re: memory address over 0x2000_7ffc is not accessible in mps2-an505

2020-09-29 Thread Peter Maydell
This is expected behaviour. The memory at 0x2000_ in this board is the "FPGA block RAM", and there is only 32K of it, so it finishes at 0x2000_7fff, and attempts to access beyond that will fail because there is no device or memory at the address immediately after it in the memory map. -- You

Re: [PATCH 0/2] Update leftover comments that mention Texinfo

2020-09-29 Thread Philippe Mathieu-Daudé
On 9/29/20 9:58 AM, Markus Armbruster wrote: > Markus Armbruster (2): > qemu-img-cmds.hx: Update comment that mentions Texinfo > target/i386/cpu: Update comment that mentions Texinfo > > target/i386/cpu.c | 3 ++- > qemu-img-cmds.hx | 2 +- > 2 files changed, 3 insertions(+), 2 deletions(-)

Re: [PATCH 16/16] tests/acceptance: Test the MIPSsim machine

2020-09-29 Thread Philippe Mathieu-Daudé
+Alex/Daniel On 9/28/20 10:33 PM, Willian Rampazzo wrote: > On Mon, Sep 28, 2020 at 2:31 PM Philippe Mathieu-Daudé > wrote: >> >> Add a test for the mipssim machine, based on the recommended >> test setup from Thomas Huth: >> https://www.mail-archive.com/qemu-devel@nongnu.org/msg606846.html >>

Re: [PATCH v6 07/21] docs/sphinx: Add new qapi-doc Sphinx extension

2020-09-29 Thread Peter Maydell
On Tue, 29 Sep 2020 at 07:54, Markus Armbruster wrote: > > Peter Maydell writes: > > > Some of our documentation is auto-generated from documentation > > comments in the JSON schema. > > > > For Sphinx, rather than creating a file to include, the most natural > > way to handle this is to have a

Re: Outline for VHOST_USER_PROTOCOL_F_VDPA

2020-09-29 Thread Stefan Hajnoczi
On Tue, Sep 29, 2020 at 02:09:55AM -0400, Michael S. Tsirkin wrote: > On Mon, Sep 28, 2020 at 10:25:37AM +0100, Stefan Hajnoczi wrote: > > Why extend vhost-user with vDPA? > > > > Reusing VIRTIO emulation code for vhost-user backends > >

Re: [PATCH 00/16] hw/mips: Set CPU frequency

2020-09-29 Thread Philippe Mathieu-Daudé
On 9/29/20 4:46 AM, no-re...@patchew.org wrote: > Patchew URL: https://patchew.org/QEMU/20200928171539.788309-1-f4...@amsat.org/ > time make docker-test-quick@centos7 SHOW_ENV=1 J=14 NETWORK=1 > --- > qemu-system-aarch64: falling back to tcg > socket_accept failed: Resource temporarily unavailable

[PATCH v2 2/2] util/vfio-helpers: Rework the IOVA allocator to avoid IOVA reserved regions

2020-09-29 Thread Eric Auger
Introduce the qemu_vfio_find_fixed/temp_iova helpers which respectively allocate IOVAs from the bottom/top parts of the usable IOVA range, without picking within host IOVA reserved windows. The allocation remains basic: if the size is too big for the remaining of the current usable IOVA range, we

Re: [PATCH v2 7/8] qemu/bswap: Use compiler __builtin_bswap() on NetBSD

2020-09-29 Thread Peter Maydell
On Mon, 28 Sep 2020 at 23:02, Kamil Rytarowski wrote: > > Personally, I prefer using the system headers. but if you want to use > the GCC builtins, please go for it. I'd agree if the system header approach was cross-platform or if this was a BSD-only program or if we were aiming for complete

[PATCH v2 1/2] util/vfio-helpers: Collect IOVA reserved regions

2020-09-29 Thread Eric Auger
The IOVA allocator currently ignores host reserved regions. As a result some chosen IOVAs may collide with some of them, resulting in VFIO MAP_DMA errors later on. This happens on ARM where the MSI reserved window quickly is encountered: [0x800, 0x810]. since 5.4 kernel, VFIO returns the

Re: [PATCH v6 05/21] scripts/qapi/parser.py: improve doc comment indent handling

2020-09-29 Thread Peter Maydell
On Mon, 28 Sep 2020 at 20:16, Markus Armbruster wrote: > > Peter Maydell writes: > > +Descriptions of arguments can span multiple lines. The description > > +text can start on the line following the '@argname:', in which case > > +it must not be indented at all. It can also start on the same

[PATCH v2 0/2] NVMe passthrough: Take into account host IOVA reserved regions

2020-09-29 Thread Eric Auger
The current IOVA allocator allocates within the [0x1, 1ULL << 39] window, without paying attention to the host IOVA reserved regions. This prevents NVMe passthtrough from working on ARM as the fixed IOVAs rapidly grow up to the MSI reserved region [0x800, 0x810] causing some VFIO MAP

Re: [PATCH v6 14/21] meson.build: Make manuals depend on source to Sphinx extensions

2020-09-29 Thread Markus Armbruster
Peter Maydell writes: > The automatic dependency handling for Sphinx manuals only makes the output > depend on the input documentation files. This means that if you edit > the Python source of an extension then the documentation won't be > rebuilt. > > Create a list of all the source files for

Re: [PATCH v10 13/13] tests/acpi: add DSDT.hpbrroot DSDT table blob to test global i440fx hotplug

2020-09-29 Thread Ani Sinha
On Tue, Sep 29, 2020 at 1:53 PM Ani Sinha wrote: > > On Tue, Sep 29, 2020 at 12:56 PM Michael S. Tsirkin wrote: > > > > On Tue, Sep 29, 2020 at 12:53:17PM +0530, Ani Sinha wrote: > > > On Tue, Sep 29, 2020 at 12:42 PM Michael S. Tsirkin > > > wrote: > > > > > > > > On Fri, Sep 18, 2020 at

Re: [PULL v4 00/48] virtio,pc,acpi: fixes, tests

2020-09-29 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200929071948.281157-1-...@redhat.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 BEGIN ===

Re: [PATCH v6 13/21] meson.build: Move SPHINX_ARGS to top level meson.build file

2020-09-29 Thread Markus Armbruster
Peter Maydell writes: > We're going to want to use SPHINX_ARGS in both docs/meson.build > and tests/qapi-schema/meson.build. Move the definition up to the > top level file so it is available to both subdirectories. > > Signed-off-by: Peter Maydell > --- > docs/meson.build | 8 >

Re: [PATCH v6 09/21] docs/interop: Convert qemu-qmp-ref to rST

2020-09-29 Thread Markus Armbruster
Appears to break documented make target html: $ make -C bld-x86 html make: Entering directory '/work/armbru/qemu/bld-x86' make: *** No rule to make target 'html'. Stop. make: Leaving directory '/work/armbru/qemu/bld-x86'

Re: [PATCH v6 12/21] tests/qapi-schema: Convert doc-good.json to rST-style strong/emphasis

2020-09-29 Thread Markus Armbruster
Peter Maydell writes: > doc-good.json currently uses the old *strong* and _emphasis_ markup. > As part of the conversion to rST this needs to switch to **strong** > and *emphasis*, because rST uses underscores as part of its markup > of hyperlinks and will otherwise warn about the syntax error.

Re: [PATCH v6 11/21] qga/qapi-schema.json: Add some headings

2020-09-29 Thread Markus Armbruster
Peter Maydell writes: > Add some section headings to the QGA json; this is purely so that we > have some H1 headings, as otherwise each command ends up being > visible in the interop/ manual's table of contents. In an ideal > world there might be a proper 'Introduction' section the way there is

Re: [PATCH v6 09/21] docs/interop: Convert qemu-qmp-ref to rST

2020-09-29 Thread Markus Armbruster
Peter Maydell writes: > Convert qemu-qmp-ref to rST format. This includes dropping > the plain-text, pdf and info format outputs for this document; > as with all our other Sphinx-based documentation, we provide > HTML and manpage only. > > The qemu-qmp-ref.rst is somewhat more stripped down than

<    1   2   3   4   5   6   >