Feature #7768

Avoid vgatext dependency on agpmaster

Added by Gordon Ross over 1 year ago. Updated 11 months ago.

Status:ClosedStart date:2017-01-14
Priority:NormalDue date:
Assignee:Gordon Ross% Done:


Target version:-
Difficulty:Medium Tags:needs-triage


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:
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)
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.


#1 Updated by Electric Monk 11 months ago

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

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>

Also available in: Atom