Skip to content

CPU topology v2 phase 1: introduce CpuSelection and regression tests#7

Merged
PrimeBuild-pc merged 3 commits into
mainfrom
codex-cpu-topology-v2-phase-1
May 19, 2026
Merged

CPU topology v2 phase 1: introduce CpuSelection and regression tests#7
PrimeBuild-pc merged 3 commits into
mainfrom
codex-cpu-topology-v2-phase-1

Conversation

@PrimeBuild-pc
Copy link
Copy Markdown
Owner

Description

  • Introduce CpuSelection, ProcessorRef, CpuSelectionMetadata, CpuTopologySignature and minimal CpuTopologySnapshot.
  • Adds regression tests for legacy long affinity mask bugs above 64 logical processors.
  • Documents CoreMask.ToProcessorAffinity aliasing bug for CPU index 64.
  • Adds conversion helpers from legacy affinity mask to CpuSelection and back when safely representable.
  • Does not change runtime affinity application, UI, presets, profiles, or ProcessService behavior.

Checklist

  • dotnet test "ThreadPilot_1.sln" --configuration Release --no-restore passes.
  • No runtime behavior changes.
  • Existing profile compatibility preserved.
  • Legacy conversion returns null when selection cannot be represented as a safe single-group <64-bit mask.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 19, 2026

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 93.15068% with 10 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
Models/CpuSelection.cs 93.15% 7 Missing and 3 partials ⚠️

📢 Thoughts on this report? Let us know!

@PrimeBuild-pc PrimeBuild-pc merged commit 3c17c98 into main May 19, 2026
2 checks passed
@PrimeBuild-pc PrimeBuild-pc deleted the codex-cpu-topology-v2-phase-1 branch May 19, 2026 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants