Project

General

Profile

Actions

Feature #7768

closed

Avoid vgatext dependency on agpmaster

Added by Gordon Ross over 6 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Start date:
2017-01-14
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
needs-triage
Gerrit CR:
External Bug:

Description

The vgatext driver /illumos-gate/usr/src/uts/intel/io/vgatext/
currently depends on the agpmaster driver (misc/agpmaster)
via a "-dy" flag in the Makefile here:
http://src.illumos.org/source/xref/illumos-gate/usr/src/uts/intel/vgatext/Makefile#49
Now that we're building the AGP drivers outside of illumos-gate,
it would be convenient if we did not have this dependency, or
at least if it were a run-time rather than link-time dependency.

The vgatext driver has a feature whereby it can pass through
ioctl calls to the misc/agpmaster module associated with the
attached VGA display (when found on a PCI bus). Such calls
are supported via "odd" minor devices (See AGPMASTER_MINOR)
http://src.illumos.org/source/search?q=AGPMASTER_MINOR&project=illumos-gate
There are no device nodes for these vgatext minor devices,
so this vgatext feature appears to be dead code.

This feature request will replace the (dead code) support for
agpmaster pass-through from vgatext with some syslog
messages (so we'll know if a consumer ever shows up).

The (old) misc/drm module in illumos gate also calls into
misc/agpmaster, optionally creating minor devices under
whatever top-level device linked misc/drm (i.e. i916) for
the agpmaster device. This relationship continues in the
newer DRM code over in the gfx-drm gate. For now, the
old drm code is still there in illumos-gate, but once that
goes away, no consumers of misc/agpmaster will remain
in illumos-gate.

Actions #1

Updated by Electric Monk over 6 years ago

  • Status changed from In Progress to Closed
  • % Done changed from 0 to 100

git commit d1aea6f139360e9e7f1504facb24f8521047b15c

commit  d1aea6f139360e9e7f1504facb24f8521047b15c
Author: Gordon Ross <gordon.w.ross@gmail.com>
Date:   2017-05-29T00:39:55.000Z

    7768 Avoid vgatext dependency on agpmaster
    Reviewed by: Toomas Soome <tsoome@me.com>
    Reviewed by: Yuri Pankov <yuri.pankov@nexenta.com>
    Approved by: Richard Lowe <richlowe@richlowe.net>

Actions

Also available in: Atom PDF