1. Patchew
  2. Xen
  3. View series

[Xen-devel] [PATCH for-4.13] xen/passthrough: Render domains with %pd in the 'Q' debug handler

Andrew Cooper posted 1 patch 4 years, 5 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/xen tags/patchew/20191113134417.17299-1-andrew.cooper3@citrix.com
xen/drivers/passthrough/pci.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
[Xen-devel] [PATCH for-4.13] xen/passthrough: Render domains with %pd in the 'Q' debug handler
Posted by Andrew Cooper 4 years, 5 months ago 
IOMMUs are owned by DOM_XEN, and with XSA-302, DOM_IO is used for
quarantined domains.  Use %pd in the printk to render the system
domains more intelligently.

Before:
  (XEN) 0000:00:01.0 - dom 0   - node 0   - MSIs < >
  (XEN) 0000:00:00.0 - dom 0   - node 0   - MSIs < >
  (XEN) 0000:80:00.2 - dom 32754 - node 1   - MSIs < >
  (XEN) 0000:a0:00.2 - dom 32754 - node 1   - MSIs < >
  (XEN) 0000:c0:00.2 - dom 32754 - node 1   - MSIs < >
  (XEN) 0000:e0:00.2 - dom 32754 - node 1   - MSIs < >
  (XEN) 0000:00:00.2 - dom 32754 - node 0   - MSIs < >
  (XEN) 0000:20:00.2 - dom 32754 - node 0   - MSIs < >
  (XEN) 0000:40:00.2 - dom 32754 - node 0   - MSIs < >
  (XEN) 0000:60:00.2 - dom 32754 - node 0   - MSIs < >

After:
  (XEN) 0000:00:01.0 - d0 - node 0   - MSIs < >
  (XEN) 0000:00:00.0 - d0 - node 0   - MSIs < >
  (XEN) 0000:80:00.2 - d[XEN] - node 1   - MSIs < >
  (XEN) 0000:a0:00.2 - d[XEN] - node 1   - MSIs < >
  (XEN) 0000:c0:00.2 - d[XEN] - node 1   - MSIs < >
  (XEN) 0000:e0:00.2 - d[XEN] - node 1   - MSIs < >
  (XEN) 0000:00:00.2 - d[XEN] - node 0   - MSIs < >
  (XEN) 0000:20:00.2 - d[XEN] - node 0   - MSIs < >
  (XEN) 0000:40:00.2 - d[XEN] - node 0   - MSIs < >
  (XEN) 0000:60:00.2 - d[XEN] - node 0   - MSIs < >

Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
---
CC: Jan Beulich <JBeulich@suse.com>
CC: Juergen Gross <jgross@suse.com>

4.13 nice-to-have.  Not strictly required, but it is a low risk change which
improves diagnostics.
---
 xen/drivers/passthrough/pci.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/xen/drivers/passthrough/pci.c b/xen/drivers/passthrough/pci.c
index 18a7dc7224..8e501a79a8 100644
--- a/xen/drivers/passthrough/pci.c
+++ b/xen/drivers/passthrough/pci.c
@@ -1349,10 +1349,9 @@ static int _dump_pci_devices(struct pci_seg *pseg, void *arg)
 
     list_for_each_entry ( pdev, &pseg->alldevs_list, alldevs_list )
     {
-        printk("%04x:%02x:%02x.%u - dom %-3d - node %-3d - MSIs < ",
+        printk("%04x:%02x:%02x.%u - %pd - node %-3d - MSIs < ",
                pseg->nr, pdev->bus,
-               PCI_SLOT(pdev->devfn), PCI_FUNC(pdev->devfn),
-               pdev->domain ? pdev->domain->domain_id : -1,
+               PCI_SLOT(pdev->devfn), PCI_FUNC(pdev->devfn), pdev->domain,
                (pdev->node != NUMA_NO_NODE) ? pdev->node : -1);
         list_for_each_entry ( msi, &pdev->msi_list, list )
                printk("%d ", msi->irq);
-- 
2.11.0


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH for-4.13] xen/passthrough: Render domains with %pd in the 'Q' debug handler
Posted by Jan Beulich 4 years, 5 months ago 
On 13.11.2019 14:44, Andrew Cooper wrote:
> IOMMUs are owned by DOM_XEN, and with XSA-302, DOM_IO is used for
> quarantined domains.  Use %pd in the printk to render the system
> domains more intelligently.
> 
> Before:
>   (XEN) 0000:00:01.0 - dom 0   - node 0   - MSIs < >
>   (XEN) 0000:00:00.0 - dom 0   - node 0   - MSIs < >
>   (XEN) 0000:80:00.2 - dom 32754 - node 1   - MSIs < >
>   (XEN) 0000:a0:00.2 - dom 32754 - node 1   - MSIs < >
>   (XEN) 0000:c0:00.2 - dom 32754 - node 1   - MSIs < >
>   (XEN) 0000:e0:00.2 - dom 32754 - node 1   - MSIs < >
>   (XEN) 0000:00:00.2 - dom 32754 - node 0   - MSIs < >
>   (XEN) 0000:20:00.2 - dom 32754 - node 0   - MSIs < >
>   (XEN) 0000:40:00.2 - dom 32754 - node 0   - MSIs < >
>   (XEN) 0000:60:00.2 - dom 32754 - node 0   - MSIs < >
> 
> After:
>   (XEN) 0000:00:01.0 - d0 - node 0   - MSIs < >
>   (XEN) 0000:00:00.0 - d0 - node 0   - MSIs < >
>   (XEN) 0000:80:00.2 - d[XEN] - node 1   - MSIs < >
>   (XEN) 0000:a0:00.2 - d[XEN] - node 1   - MSIs < >
>   (XEN) 0000:c0:00.2 - d[XEN] - node 1   - MSIs < >
>   (XEN) 0000:e0:00.2 - d[XEN] - node 1   - MSIs < >
>   (XEN) 0000:00:00.2 - d[XEN] - node 0   - MSIs < >
>   (XEN) 0000:20:00.2 - d[XEN] - node 0   - MSIs < >
>   (XEN) 0000:40:00.2 - d[XEN] - node 0   - MSIs < >
>   (XEN) 0000:60:00.2 - d[XEN] - node 0   - MSIs < >
> 
> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>

Reviewed-by: Jan Beulich <jbeulich@suse.com>

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH for-4.13] xen/passthrough: Render domains with %pd in the 'Q' debug handler
Posted by Jürgen Groß 4 years, 5 months ago 
On 13.11.19 14:44, Andrew Cooper wrote:
> IOMMUs are owned by DOM_XEN, and with XSA-302, DOM_IO is used for
> quarantined domains.  Use %pd in the printk to render the system
> domains more intelligently.
> 
> Before:
>    (XEN) 0000:00:01.0 - dom 0   - node 0   - MSIs < >
>    (XEN) 0000:00:00.0 - dom 0   - node 0   - MSIs < >
>    (XEN) 0000:80:00.2 - dom 32754 - node 1   - MSIs < >
>    (XEN) 0000:a0:00.2 - dom 32754 - node 1   - MSIs < >
>    (XEN) 0000:c0:00.2 - dom 32754 - node 1   - MSIs < >
>    (XEN) 0000:e0:00.2 - dom 32754 - node 1   - MSIs < >
>    (XEN) 0000:00:00.2 - dom 32754 - node 0   - MSIs < >
>    (XEN) 0000:20:00.2 - dom 32754 - node 0   - MSIs < >
>    (XEN) 0000:40:00.2 - dom 32754 - node 0   - MSIs < >
>    (XEN) 0000:60:00.2 - dom 32754 - node 0   - MSIs < >
> 
> After:
>    (XEN) 0000:00:01.0 - d0 - node 0   - MSIs < >
>    (XEN) 0000:00:00.0 - d0 - node 0   - MSIs < >
>    (XEN) 0000:80:00.2 - d[XEN] - node 1   - MSIs < >
>    (XEN) 0000:a0:00.2 - d[XEN] - node 1   - MSIs < >
>    (XEN) 0000:c0:00.2 - d[XEN] - node 1   - MSIs < >
>    (XEN) 0000:e0:00.2 - d[XEN] - node 1   - MSIs < >
>    (XEN) 0000:00:00.2 - d[XEN] - node 0   - MSIs < >
>    (XEN) 0000:20:00.2 - d[XEN] - node 0   - MSIs < >
>    (XEN) 0000:40:00.2 - d[XEN] - node 0   - MSIs < >
>    (XEN) 0000:60:00.2 - d[XEN] - node 0   - MSIs < >
> 
> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>

Release-acked-by: Juergen Gross <jgross@suse.com>


Juergen

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

Patchew 2.1-devel-0870f84

© 2016 - 2024 Red Hat, Inc.