Hi Janusz,

On Wed Jan 21, 2026 at 12:42 PM CET, Janusz Krzysztofik wrote:
> In a short listing, information about PCI parents of DRM devices, as well
> as about DRM children of PCI GPU devices is now printed.  In a long
> listing in turn, the latter is also printed.  Apply a similar approach to
> PCIe bridge upstream ports: print information about their PCI GPU children
> and also their codenames in both formats, and inform about upstream ports
> of PCIe bridges found on PCI discrete GPU cards in the short format.
>
> Signed-off-by: Janusz Krzysztofik <[email protected]>
> ---
>  lib/igt_device_scan.c | 22 +++++++++++++++++-----
>  1 file changed, 17 insertions(+), 5 deletions(-)
>
> diff --git a/lib/igt_device_scan.c b/lib/igt_device_scan.c
> index e86da001a9..69ea8d2eaf 100644
> --- a/lib/igt_device_scan.c
> +++ b/lib/igt_device_scan.c
> @@ -250,6 +250,8 @@ struct igt_device {
>       char *codename; /* For grouping by codename */
>       enum dev_type dev_type; /* For grouping by integrated/discrete */
>  
> +     char *pci_gpu; /* Filled for upstream bridge ports */
> +
>       struct igt_list_head link;
>  };
>  
> @@ -1058,6 +1060,9 @@ static void update_or_add_parent(struct udev *udev,
>       igt_assert(bridge_idev);
>  
>       bridge_idev->dev_type = DEVTYPE_BRIDGE;
> +     bridge_idev->pci_gpu = parent_idev->pci_slot_name;
> +     bridge_idev->codename = parent_idev->codename;
> +     parent_idev->parent = bridge_idev;
I haven’t checked the release process myself. Have you verified that freeing 
this
memory won’t lead to any problems?
 (Maybe use strdup??)

-- 
Best regards,
Sebastian

Reply via email to