release: CA-O 2.2.17 con correctness de overclock (turbo, AMD, NVML, sonda GPU)

Author: Pyromesis

CHANGELOG.md

@@ -3,6 +3,16 @@
 Formato basado en [Keep a Changelog](https://keepachangelog.com/es-ES/1.1.0/).
 
 ## [Unreleased]
+## [2.2.17] - 2026-10-14
+### Corregido
+- **Perfil Rendimiento ya no apaga el turbo.** `KnobValue.FromIndex` dejaba `Scalar` a 0 y el preset agresivo (indice 2) escribia `PERFBOOSTMODE=0`: turbo apagado reportado como exito. El indice se refleja en `Scalar` y Conservador pasa a indice 1 (fabrica), nunca 0.
+- **Conservador AMD ya no pone la tarjeta al maximo.** ADLX habla en offset (-10..+15) y el catalogo en % de fabrica (100 = fabrica): escribir 100 directo lo recortaba al maximo. Nueva conversion `ToCatalog/ToDriverPercent` con stock del driver o inferido del rango.
+- **Offsets de reloj NVIDIA por fin disponibles.** Las versiones `16|1<<24` y `24|2<<24` no existen en `nvml.h` (una sola `v1` de 24 bytes): el driver devolvia `VERSION_MISMATCH` y los offsets nunca tenian bounds. Unificada a `24|1<<24`.
+- **Tuning sin veredicto ya no cuenta como aprobado.** Nueva sonda `NvmlGpuStabilityProbe` (reloj/utilizacion/temperatura/slowdown, 1 Hz): TDR/device-lost es fallo, ventana sin 70 % de carga media es `Pending` -> estado `Unverified` con restore al ultimo bueno. Si el driver recorta al mismo valor, el recorrido corta en `CeilingReached` con el valor real.
+- **Registry ya no mezcla vendors.** `AllCatalogKnobs` incluye AMD, una vez que hay GPU se ocultan los knobs de la contraria, `CompositeCeilingSource` concatena techos por vendor, y el rank de CPU deja de usar el orden del enum (PolicyOnly pisaba XTU).
+- **Textos honestos.** CPU deja de decir "sin MSR no hay OC": dice que ajusta turbo/frecuencia maxima del plan. Perfiles advierten que son valores fijos sin prueba; el auto avisa que necesita juego/benchmark con carga. Tour de Overclock actualizado.
+### Probado
+- Suite 1892 passed, 0 failed (Security 274, Benchmark 17, Infrastructure 215, Core 947, UI 379, Integration 60). 19 tests nuevos de correctness (turbo, Pending-no-pass, clamp, NVML version, AMD % stock, sonda GPU, rank CPU).
 ## [2.2.16] - 2026-10-13
 ### Corregido
 - **El aviso de ajustes pendientes ya dice cuales y como resolverlos.** Nombra cada ajuste pendiente, cuantos son y manda a la seccion 05 ESTABILIDAD con Revertir/Mantener/Descartar, en vez del generico "Resuelvelos primero".

README.md

@@ -1,12 +1,12 @@
 <Project>
   <!-- Single source of truth for versioning (FASE 28). All other files derive from here. -->
   <PropertyGroup>
-    <Version>2.2.16</Version>
-    <AssemblyVersion>2.2.16.0</AssemblyVersion>
-    <FileVersion>2.2.16.0</FileVersion>
-    <PackageVersion>2.2.16</PackageVersion>
-    <ProductVersion>2.2.16</ProductVersion>
-    <InformationalVersion>2.2.16+$(GHTag)</InformationalVersion>
+    <Version>2.2.17</Version>
+    <AssemblyVersion>2.2.17.0</AssemblyVersion>
+    <FileVersion>2.2.17.0</FileVersion>
+    <PackageVersion>2.2.17</PackageVersion>
+    <ProductVersion>2.2.17</ProductVersion>
+    <InformationalVersion>2.2.17+$(GHTag)</InformationalVersion>
   </PropertyGroup>
 </Project>
 

Version.props

@@ -114,14 +114,14 @@ Nunca se registran secretos, credenciales ni contenido de entrada del usuario (s
 - **i18n**: diccionario tipado es-ES/en-US (`Localizer`); migración a `.resw` planificada sin cambiar llamadas (`Localizer.Get(key)`).
 - **HAGS/VBS/MPO/etc.**: nunca tratados como ganancias universales; HAGS es WorkloadDependent+Conditional+RequiresReboot, VBS es Critical+SecurityTradeoff+ExpertOnly.
 
-## Pruebas (1873 passed, 0 failed — Release)
+## Pruebas (1892 passed, 0 failed — Release)
 
 | Suite | Cubre | Count |
 |---|---|---|
-| `CA-O.Core.Tests` | Contratos catálogo 93, `AnalysisStateStore` (save/load/corrupt), `GameCompatibility` (VBS bloqueado), transacciones, scoring, health, ledger one-shot, temporales a 10 min | 943 |
+| `CA-O.Core.Tests` | Contratos catálogo 93, `AnalysisStateStore` (save/load/corrupt), `GameCompatibility` (VBS bloqueado), transacciones, scoring, health, ledger one-shot, temporales a 10 min | 947 |
 | `CA-O.Security.Tests` | IPC validator (`Ping`/`Apply` cross-check), `IpcPingTests`, inyección | 274 |
 | `CA-O.Integration.Tests` | `E2EFlowsTests` 9 pruebas (8 casos numerados + benchmark: Abrir→Analizar→Persistir→Vanguard→Restore→Benchmark) + `TransactionJournalRecovery` + contrato docs + `PrivilegedServicePackagingTests` (payload self-contained) | 60 |
-| `CA-O.Infrastructure.Tests` | `HistoryRobustness` (malformed), `SnapshotRepository` (TX identity), `SystemContextCache` dual-TTL | 202 |
+| `CA-O.Infrastructure.Tests` | `HistoryRobustness` (malformed), `SnapshotRepository` (TX identity), `SystemContextCache` dual-TTL | 215 |
 | `CA-O.Benchmark.Tests` | `SystemBenchmarkRunner` (suelo 3%, mediana) | 17 |
-| `CA-O.UI.Tests` | `ViewModelTests` (Analyze/Dashboard con `SystemAnalysisService` + `Correlation`), Overclock (filas visuales + mood de tuning), tours por pagina (83 pasos, destinos verificados), auto-tuning con propuesta, puerta de restore point, aviso recovery con nombres | 377 |
-| **Total** | **Gates 1-5 `verify.ps1` + `build-release` con `gui-installer`** | **1873** |
+| `CA-O.UI.Tests` | `ViewModelTests` (Analyze/Dashboard con `SystemAnalysisService` + `Correlation`), Overclock (filas visuales + mood de tuning), tours por pagina (83 pasos, destinos verificados), auto-tuning con propuesta, puerta de restore point, aviso recovery con nombres | 379 |
+| **Total** | **Gates 1-5 `verify.ps1` + `build-release` con `gui-installer`** | **1892** |

docs/ARCHITECTURE.md

@@ -3,7 +3,7 @@
 ; (lee la versión de BuildConstants.cs y firma con CAO_SIGN_THUMBPRINT si existe)
 
 #ifndef AppVersion
-  #define AppVersion "2.2.16"
+  #define AppVersion "2.2.17"
 #endif
 #ifndef RepoRoot
   #define RepoRoot ".."

installer/CA-O.iss

@@ -89,9 +89,8 @@ public static class OverclockCatalog
             presets: []),
     ];
     /// <summary>
-    /// AMD knobs, served by amdadlx64.dll. Power limit only: ADLX reports it in
-    /// percent of TDP, which is exactly the catalog unit, so there is no anchor
-    /// to resolve and no conversion to get wrong. AMD clocks stay out on
+    /// AMD knobs, served by amdadlx64.dll. Power limit only, in % of stock like
+    /// the NVIDIA one; the backend converts from ADLX's offset-around-0 range. AMD clocks stay out on
     /// purpose: from Navi4 up SetGPUMaxFrequency takes an offset, before that
     /// an absolute frequency, and guessing which one a card expects is how a
     /// number the hardware never meant gets written.
@@ -123,7 +122,10 @@ public static class OverclockCatalog
             "knob_cpu_boost_mode", "knob_cpu_boost_mode_desc", requiresReboot: false,
             presets:
             [
-                (OverclockProfile.Conservative, KnobValue.FromIndex(0)),
+                // 0 = turbo off, 1 = turbo normal, 2 = turbo agresivo. Ningun perfil
+                // apaga el turbo: "Conservador" en una pestana de overclock es el
+                // comportamiento de fabrica, no un procesador mas lento.
+                (OverclockProfile.Conservative, KnobValue.FromIndex(1)),
                 (OverclockProfile.Balanced, KnobValue.FromIndex(2)),
                 (OverclockProfile.Performance, KnobValue.FromIndex(2)),
             ]),

src/CA-O.Core/Overclock/Catalog/OverclockCatalog.cs

@@ -30,7 +30,13 @@ private KnobValue(double scalar, int index)
 
     public static KnobValue FromScalar(double value) => new(value, 0);
 
-    public static KnobValue FromIndex(int index) => new(0d, index);
+    /// <summary>
+    /// A discrete value. The index is mirrored into <see cref="Scalar"/> on
+    /// purpose: backends write the setting as a number (powercfg takes the raw
+    /// index), and a discrete preset whose Scalar stayed 0 made the Performance
+    /// profile write PERFBOOSTMODE=0, turning turbo OFF while reporting success.
+    /// </summary>
+    public static KnobValue FromIndex(int index) => new(index, index);
 
     /// <summary>Scalar values are the common case in the catalog, so literals convert implicitly.</summary>
     public static implicit operator KnobValue(double value) => FromScalar(value);

src/CA-O.Core/Overclock/OverclockModel.cs

@@ -10,6 +10,9 @@ public enum ProbeMetric
     KernelBugcheck,
     EffectiveClockStall,
     GpuThermalLimit,
+
+    /// <summary>The GPU stopped answering mid-window (driver reset / TDR).</summary>
+    GpuDeviceLost,
     Composite,
 }
 
@@ -71,6 +74,17 @@ public sealed record ThermalLimitReached(string SensorName, double LimitCelsius,
         public bool IsFault => true;
     }
 
+    /// <summary>
+    /// La GPU dejó de responder durante la ventana: el driver se reinició (TDR) o el
+    /// dispositivo se perdió. Es la firma clásica de un offset de reloj inestable.
+    /// </summary>
+    public sealed record DeviceLost(string DeviceName, string ReasonKey) : ProbeEvidence
+    {
+        public override ProbeMetric Metric => ProbeMetric.GpuDeviceLost;
+
+        public bool IsFault => true;
+    }
+
     /// <summary>
     /// Un sensor que no se pudo leer. Se registra para que la UI pueda decir "sin datos"
     /// en lugar de dar a entender que todo va bien.
@@ -109,6 +123,7 @@ public sealed record ProbeSample(DateTimeOffset TakenAt, IReadOnlyList<ProbeEvid
         ProbeEvidence.KernelBugcheck b => b.IsFault,
         ProbeEvidence.ClockStall c => c.IsFault,
         ProbeEvidence.ThermalLimitReached t => t.IsFault,
+        ProbeEvidence.DeviceLost d => d.IsFault,
         _ => false,
     };
 }

src/CA-O.Core/Overclock/Stability/StabilityContracts.cs

@@ -62,6 +62,7 @@ public static IReadOnlyList<ProbeEvidence> Faults(ProbeRun run, StabilityThresho
         ProbeEvidence.KernelBugcheck => true,
         ProbeEvidence.ClockStall c => c.StallRatio > thresholds.MaxClockStallRatio,
         ProbeEvidence.ThermalLimitReached t => thresholds.ThermalLimitIsFault,
+        ProbeEvidence.DeviceLost => true,
         ProbeEvidence.SensorUnavailable => thresholds.SensorUnavailableIsFault,
         _ => false,
     };

src/CA-O.Core/Overclock/Stability/StabilityEvaluator.cs

@@ -53,6 +53,12 @@ public enum TuningStatus
 
     /// <summary>El backend no pudo ni aplicar el primer valor. No se abrió nada.</summary>
     Rejected,
+
+    /// <summary>
+    /// La sonda no pudo dar veredicto (sin carga, sin telemetría). Un punto sin
+    /// veredicto NO es un aprobado: se vuelve al último valor bueno y se cierra.
+    /// </summary>
+    Unverified,
 }
 
 public sealed record TuningRequest(
@@ -125,6 +131,7 @@ public async Task<TuningRunResult> RunAsync(
             cancellationToken.ThrowIfCancellationRequested();
 
             var proposed = KnobValue.FromScalar(value);
+
             var applied = await request.Target
                 .ApplyAsync(knob.Id, proposed, cancellationToken)
                 .ConfigureAwait(false);
@@ -135,6 +142,17 @@ public async Task<TuningRunResult> RunAsync(
                 return Build(request, TuningStatus.Rejected, lastGoodValue, null, points, applied.FailureReasonKey, null);
             }
 
+            // El driver recorta contra su propio techo. Si lo que de verdad recibió no
+            // pasa del último valor bueno, seguir subiendo solo repetiría el mismo punto:
+            // el techo del hardware está alcanzado y se dice con el valor real.
+            var actual = applied.Applied?.Scalar ?? value;
+            if (points.Count > 0 && actual <= lastGoodValue.Scalar + (request.Step * 1e-6))
+            {
+                return Build(request, TuningStatus.CeilingReached, lastGoodValue, lastGoodValue.Scalar, points, null, null);
+            }
+
+            proposed = KnobValue.FromScalar(actual);
+
             await WriteAsync(TransactionPhase.OverclockStepApplied, knob.Id.Value, cancellationToken).ConfigureAwait(false);
             await WriteAsync(TransactionPhase.OverclockCheckpointTaken, $"{knob.Id}={Format(proposed)}", cancellationToken).ConfigureAwait(false);
 
@@ -170,6 +188,16 @@ public async Task<TuningRunResult> RunAsync(
                 return Build(request, TuningStatus.StabilityCeilingFound, lastGoodValue, ceiling, points, null, null);
             }
 
+            if (verdict == StabilityVerdict.Pending)
+            {
+                // Sin veredicto no hay aprobado: una ventana sin carga o sin telemetría
+                // no demuestra nada. Se vuelve al último valor bueno y se cierra.
+                await RestoreAsync(request, lastGoodSnapshot, cancellationToken).ConfigureAwait(false);
+                var reason = run.AbortReasonKey ?? "overclock-tuning-no-verdict";
+                await WriteAsync(TransactionPhase.OverclockStepRejected, reason, cancellationToken).ConfigureAwait(false);
+                return Build(request, TuningStatus.Unverified, lastGoodValue, null, points, reason, null);
+            }
+
             await WriteAsync(TransactionPhase.OverclockTestPassed, knob.Id.Value, cancellationToken).ConfigureAwait(false);
 
             lastGoodValue = proposed;