Re: Proposed new /etc/rc.d/drvctl script

[Paul Goyette <paul%whooppee.com@localhost>]

On Wed, 9 Apr 2025, Christoph Badura wrote:

> Isn't this barking up the wrong tree?
>
> First of all, WTF amdsmn4)?  Apparently there is no man page.

AMD system management bus.  Talk to riastradh about the missing man
page!  :)  amdsmn provides an attachment for amdzentemp(4) sensor.

> Our autoconfiguration system handles this case just fine with match
> priorities.
>
> That should work for modules too.  martin@ noted this in the PR.
>
> I recently came into a possession of an Infinite Noise TRNG that attaches
> as a ucom(4) by default.  The to-be-written driver should attach it as a
> rng device and use match priority to make that happen.  So this is a more
> generic problem that needs solving in the module and autoconfiguration
> framework.

Match priority doesn't work here..

Initially we have only one driver present, pchb.  The second, higher
priority driver arrives later (when the module gets loaded) but by that
time it is too late - pchb is already "claimiing" the attachment.  We
need to "evict" pchb to make room for amdsmn.  But with our current
autoconfig system the amdsmn_attach doesn't even get called.  We skip
over the "occupied" attachments.


> How do you plan to get rid of the script once we've fixed the underlying
> issues.

IF we rewrite autoconf stuff to enable auto-eviction, we can simply
obsolete this fiile.  Setting drvctl=NO in rc.conf is the default to
disable the script.

> Your proposed script pampers of the symptoms in a crude way.
>
> You can easily use this script locally until the underlying issue(s) have
> been fixed.
>
> On Mon, Apr 07, 2025 at 11:58:13PM -0700, Paul Goyette wrote:
> > I'd like to commit this within the next few days unless there are
> > strong reasons against.
>
> To be frank: NO!  If you still want to commit this, I suggset you get
> approval by core@ first.
>
> --chris
>
> !DSPAM:67f63aed162954462316739!

+---------------------+--------------------------+----------------------+
| Paul Goyette (.sig) | PGP Key fingerprint:     | E-mail addresses:    |
| (Retired)           | 1B11 1849 721C 56C8 F63A | paul%whooppee.com@localhost    |
| Software Developer  | 6E2E 05FD 15CE 9F2D 5102 | pgoyette%netbsd.org@localhost  |
| & Network Engineer  |                          | pgoyette99%gmail.com@localhost |
+---------------------+--------------------------+----------------------+