Project

General

Profile

Feature #9585

extend pcibus enumerator to expose driver instance as property on pcifn/pciexfn nodes

Added by Rob Johnston about 2 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Start date:
2018-06-07
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:

Description

The topo nodes that represent PCI/PCIe functions (pcifn/pciexfn nodes) expose the associated driver via the "driver" property in the "io" property group. It would be nice if it also exposed the driver instance in the same place.

Note that this change has already been integrated into illumos-joyent via the following changeset:

commit 4cbfb4e7e093f73a0396f7354b72ffeb3c5ab608
Author: Rob Johnston <rob.johnston@joyent.com>
Date:   Sat Apr 7 07:30:35 2018 +0000

    OS-6870 need topo maps for the SMCI,SSG-2029P-ACR24L
    OS-6876 authority and system pgroups missing on some nodes
    OS-6877 extend pcibus enumerator to expose driver instance as property on pcifn/pciexfn nodes
    Reviewed by: Robert Mustacchi <rm@joyent.com>
    Approved by: Joshua M. Clulow <josh@sysmgr.org>

History

#1

Updated by Rob Johnston about 2 years ago

The testing notes for the original illumos-joyent integration are available at the URL below:

https://smartos.org/bugview/OS-6877

As a sanity test of the upstream port of this change, I onu'd my openindiana workstation and booted a BE with this change and ran fmtopo and verified that the pcibus topology was correct and now included the additional "instance" property on the pcifn and pciexfn nodes.

#2

Updated by Electric Monk about 2 years ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100

git commit 73797b86226967e602e2bfd913f917fd5d1d8272

commit  73797b86226967e602e2bfd913f917fd5d1d8272
Author: Rob Johnston <rob.johnston@joyent.com>
Date:   2018-06-08T00:02:09.000Z

    9585 extend pcibus enumerator to expose driver instance as property on pcifn/pciexfn nodes
    Reviewed by: Igor Kozhukhov <igor@dilos.org>
    Reviewed by: Yuri Pankov <yuripv@yuripv.net>
    Reviewed by: Ken Mays <maybird1776@yahoo.com>
    Approved by: Dan McDonald <danmcd@joyent.com>

Also available in: Atom PDF