Project

General

Profile

Feature #11257

Add DDI support for Upgradable Firmware Modules

Added by Rob Johnston 4 months ago. Updated 3 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
kernel
Start date:
Due date:
% Done:

100%

Estimated time:
Difficulty:
Hard
Tags:

Description

This ticket covers porting the DDI UFM (Upgradeable Firmware Modules) subsystem from illumos-joyent to upstream.

For background, please refer to the section "UFM Visibility" in RFD 89:

https://github.com/joyent/rfd/blob/master/rfd/0089/README.md

This ticket covers the following changes to:
  • extensions the Device Driver Interface (DDI) so that drivers have a standard mechanism for reporting firmware information.
  • modifications to the mptsas and i40e drivers to leverage the new driver interfaces
  • addition of a pseudo driver (ufmtest) to facilitate automated testing
  • addition of a set of automated tests
  • addition of manpages to describe the DDI UFM interfaces and the ufm(7d) driver and its ioctl interface

A separate ticket will be filed to cover userland changes to extend libtopo such that UFM information can be exposed in the topology.

History

#1

Updated by Rob Johnston 3 months ago

Testing

Please see the detailed testing notes from the original SmartOS integration, detailed here:

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

Additionally, , I onu'd the changes onto and OpenIndiana VM and booted into that BE. Then I installed system/test/os-test package and ran the ddi-ufm tests. I also manually inspected the rendered man pages.

#2

Updated by Electric Monk 3 months ago

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

git commit 508a0e8cf1600b06c1f7361ad76e736710d3fdf8

commit  508a0e8cf1600b06c1f7361ad76e736710d3fdf8
Author: Rob Johnston <rob.johnston@joyent.com>
Date:   2019-07-26T16:34:02.000Z

    11257 Add DDI support for Upgradable Firmware Modules
    11258 Add libtopo support for Upgradable Firmware Modules
    Reviewed by: Robert Mustacchi <robert.mustacchi@joyent.com>
    Reviewed by: Patrick Mooney <patrick.mooney@joyent.com>
    Reviewed by: Toomas Soome <tsoome@me.com>
    Reviewed by: Yuri Pankov <yuri.pankov@nexenta.com>
    Approved by: Richard Lowe <richlowe@richlowe.net>

Also available in: Atom PDF