Project

General

Profile

Actions

Bug #14628

closed

ancient devicedb should be removed

Added by Rich Lowe 6 months ago. Updated 5 months ago.

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

100%

Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:
External Bug:

Description

Long ago, (prior to 'newboot' in S11 and S10u1) we booted via a real-mode bootloader that required its own special drivers, and took part in faking up an ieee1275-like device tree.
This changed with the switch to grub circa 2005, but the file was kept and maintained for reasons (rumoured to involve the Sun HCL process).

At the present time, the devicedb/master file is only used to facilitate the probing of ISA or onboard devices (on systems we support, this is always onboard devices via the LPC).
Any PCI etc, devices in the master file are unused and can be removed.

Of the devices which are left, many are very old and not actually possible to still be present, or exist largely to provide useful model strings in the device tree (the usefulness of these strings is questionable, since most of them are also impossible to connect to a supported system).

We should remove the devicedb file, and it's ad-hoc in-kernel parser, and just inline a table of those devices which still require this kind of support into our ACPI code (from which in almost every case, our ISA devices are enumerated).


Related issues

Related to illumos gate - Bug #13596: system/boot/real-mode package should not existNewJoshua M. Clulow

Actions
Actions #1

Updated by Electric Monk 6 months ago

  • Gerrit CR set to 2109
Actions #2

Updated by Joshua M. Clulow 6 months ago

  • Related to Bug #13596: system/boot/real-mode package should not exist added
Actions #3

Updated by Rich Lowe 5 months ago

testing: Build, boot in my environment and check devices are as before, boot in bhyve and test devices are as before.
I have also received reports that hyper-v and propolis are also fine

Actions #4

Updated by Electric Monk 5 months ago

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

git commit d4039345c8fe6e54a31d17d91e86e393fdcf401b

commit  d4039345c8fe6e54a31d17d91e86e393fdcf401b
Author: Richard Lowe <richlowe@richlowe.net>
Date:   2022-05-17T18:28:47.000Z

    14628 ancient devicedb should be removed
    Reviewed by: Robert Mustacchi <rm@fingolfin.org>
    Reviewed by: Toomas Soome <tsoome@me.com>
    Approved by: Dan McDonald <danmcd@mnx.io>

Actions

Also available in: Atom PDF