[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH RESEND v2 0/3] To add HMP interface to dump PCI MSI-X table/P
|
From: |
Michael S. Tsirkin |
|
Subject: |
Re: [PATCH RESEND v2 0/3] To add HMP interface to dump PCI MSI-X table/PBA |
|
Date: |
Tue, 13 Jul 2021 16:59:52 -0400 |
On Mon, Jul 12, 2021 at 04:58:09PM -0700, Dongli Zhang wrote:
> I have just rebased the v2 patchset on top of most recent qemu to resend.
>
> This patch is to introduce the new HMP command to dump the MSI-X table/PBA.
>
> Here is the RFC v1:
>
> https://lists.gnu.org/archive/html/qemu-devel/2021-04/msg04673.html
>
> The idea was inspired by below discussion:
>
> https://lists.gnu.org/archive/html/qemu-devel/2021-03/msg09020.html
Fails build for microblaze:
/scm/qemu/build/../softmmu/qdev-monitor.c:1030: undefined reference to
`msix_dump_info'
collect2: error: ld returned 1 exit status
[2525/8445] Compiling C object
libqemu-microblaze-softmmu.fa.p/meson-generated_.._qapi_qapi-visit-misc-target.c.o
>
> (qemu) info msix -d /machine/peripheral/vscsi0
> Msg L.Addr Msg U.Addr Msg Data Vect Ctrl
> 0xfee00000 0x00000000 0x00004041 0x00000000
> 0xfee00000 0x00000000 0x00004051 0x00000000
> 0xfee00000 0x00000000 0x00004061 0x00000000
> 0xfee00000 0x00000000 0x00004071 0x00000000
> 0xfee01000 0x00000000 0x000040b1 0x00000000
> 0xfee02000 0x00000000 0x000040c1 0x00000000
> 0xfee03000 0x00000000 0x000040d1 0x00000000
>
> MSI-X PBA
> 0 0 0 0 0 0 0
>
> virtio pci vector info:
> config: 0
> queue 0: 1
> queue 1: 2
> queue 2: 3
> queue 3: 4
> queue 4: 5
> queue 5: 6
>
>
> Changed since RFC v1:
> - Add heading to MSI-X table (suggested by David Alan Gilbert)
> - Add device specific interface, e.g., to dump virtio-pci queue-to-vector
> mapping (Suggested By Jason)
>
>
> hmp-commands-info.hx | 14 +++++++++
> hw/pci/msix.c | 63 +++++++++++++++++++++++++++++++++++++++++
> hw/virtio/virtio-pci.c | 22 ++++++++++++++
> hw/virtio/virtio.c | 10 +++++++
> include/hw/pci/msix.h | 2 ++
> include/hw/pci/pci.h | 3 ++
> include/hw/virtio/virtio.h | 2 ++
> include/monitor/hmp.h | 1 +
> softmmu/qdev-monitor.c | 36 +++++++++++++++++++++++
> 9 files changed, 153 insertions(+)
>
> Thank you very much!
>
> Dongli Zhang
>