In message <3081716.hHqAuc6tWs@ravel>, Olivier Certner writes:
> --nextPart2431391.THHZn3L5Ee
> Content-Transfer-Encoding: 7Bit
> Content-Type: text/plain; charset="utf-8"; protected-headers="v1"
> From: Olivier Certner <[email protected]>
> To: Dag-Erling =?UTF-8?B?U23DuHJncmF2?= <[email protected]>
> Date: Tue, 03 Feb 2026 16:14:10 +0100
> Message-ID: <3081716.hHqAuc6tWs@ravel>
> In-Reply-To: <[email protected]>
> MIME-Version: 1.0
>
> > This doesn't work.  You are reusing the same OID already used by the
> > Intel subdriver.  The result is that whichever one happens to be first
> > on the linker command line works and the other one doesn't.  You have to
> > either use a different OID for each subdriver or have both subdrivers
> > share a single boolean.
>
> Yes, we are aware.  There's already https://reviews.freebsd.org/D54528 about 
> that.
>
> We did not feel this was urgent, especially since I intend to retire `machdep
> .hwpstate_pkg_ctrl` from the AMD driver entirely (and obsolete it in the Inte
> l driver) in favor of an easier to use interface, but this has lasted for a w
> hile now and could prevent the same knob from working in existing Intel cases
> .
>
> I'll move D54528 in the coming days.
>
> -- 
> Olivier Certner

One of the subsequent commits broke the commit that allows disabling of 
CPPC through a tunable.

Setting sysctl machdep.hwpstate_amd_cppc_enable failed: 12


-- 
Cheers,
Cy Schubert <[email protected]>
FreeBSD UNIX:  <[email protected]>   Web:  https://FreeBSD.org
NTP:           <[email protected]>    Web:  https://nwtime.org

                        e**(i*pi)+1=0



Reply via email to