WIP: test(vcores): counters for the #327 measurement (do not merge) #412

Closed

DuckNintendo wants to merge 4 commits from DuckNintendo/Longshot:perf/pr327-counters into main

pull from: DuckNintendo/Longshot:perf/pr327-counters

merge into: MagicBeanDev:main

MagicBeanDev:main

Conversation 0 Commits 4 Files changed 4 +326 -6

DuckNintendo commented 2026-11-14 10:58:43 +00:00

First-time contributor

• Copy link
• Copy Markdown

Testing only. Do not merge.

This is #327's code plus the temporary counters used for its Hyrule Field measurement.
The real change is #327; this PR is closed so it cannot be merged, and is kept only so
the counters and their diff stay available.

• Branch: perf/pr327-counters @ d0a06db7, based on #327's head f55b29d6.
• Difference from #327: host/hos_thread.c (the PERF counters) and host/control.c (the
  report call at quit). No scheduling change.
• #327: #327
• The Files changed page shows #327's files plus the counter commit.

**Testing only. Do not merge.** This is #327's code plus the temporary counters used for its Hyrule Field measurement. The real change is #327; this PR is closed so it cannot be merged, and is kept only so the counters and their diff stay available. - Branch: `perf/pr327-counters` @ `d0a06db7`, based on #327's head `f55b29d6`. - Difference from #327: `host/hos_thread.c` (the PERF counters) and `host/control.c` (the report call at quit). No scheduling change. - #327: https://git.magicbean.dev/MagicBeanDev/Longshot/pulls/327 - The Files changed page shows #327's files plus the counter commit.

DuckNintendo added 4 commits 2026-11-14 10:58:43 +00:00

fix(vcores): treat mask 0 as no affinity and prefer the cached core 071661d731

test(vcores): stub the core-cache helpers in the escape fixture 6b270ed941

fix(vcores): sync the guest TLS core cell at take f55b29d6ec

perf(vcores): temporary counters for the PR #327 measurement (not for merge) d0a06db786

DuckNintendo closed this pull request 2026-11-14 10:58:51 +00:00

DuckNintendo referenced this pull request 2026-11-14 10:58:52 +00:00

fix(vcores): honour mask 0 and keep the guest's core cache correct #327

DuckNintendo referenced this pull request 2026-11-14 10:58:59 +00:00

fix(vcores): honour mask 0 and keep the guest's core cache correct #327

Some checks failed

Hide checks

CI / Conventional Commit title (pull_request) Has been cancelled

Details

CI / Python source checks (ubuntu-24.04, part 1/2) (pull_request) Has been cancelled

Details

CI / Python source checks (ubuntu-24.04, part 2/2) (pull_request) Has been cancelled

Details

CI / SPIR-V fixtures (global narrow load/store, direct surface atomics, F2I immediate) (pull_request) Has been cancelled

Details

CI / Workflow lint (actionlint) (pull_request) Has been cancelled

Details

Third-party parts / As their notes say (pull_request) Has been cancelled

Details

Third-party parts / The same as upstream (pull_request) Has been cancelled

Details

PR description / Description follows the template (pull_request_target) Successful in 4s

Details

Pull request closed

Please reopen this pull request to perform a merge.

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

area/accessibility

Barriers for people with disabilities

  

area/audio

Sound and music

  

area/input

Controllers, keyboard, mouse

  

area/install

Installing, updating, building from your game

  

area/menu

In-game menu and settings

  

area/renderer

Vulkan renderer, shaders, effects

  

area/saves

Save files

  

area/upscaling

DLSS, FSR, frame generation

  

beanstalk: approved @ 1697603

Set by Beanstalk for show; the farm's database is what counts

  

beanstalk: approved @ 2c5a62e

Set by Beanstalk for show; the farm's database is what counts

  

beanstalk: approved @ 4fd82b7

Set by Beanstalk for show; the farm's database is what counts

  

beanstalk: approved @ 64d6d09

Set by Beanstalk for show; the farm's database is what counts

  

beanstalk: approved @ 6f91fae

Set by Beanstalk for show; the farm's database is what counts

  

beanstalk: approved @ 70a78f9

Set by Beanstalk for show; the farm's database is what counts

  

beanstalk: approved @ 8d361dc

Set by Beanstalk for show; the farm's database is what counts

  

beanstalk: approved @ 9d8356e

Set by Beanstalk for show; the farm's database is what counts

  

beanstalk: approved @ abf2227

Set by Beanstalk for show; the farm's database is what counts

  

beanstalk: approved @ d82f202

Set by Beanstalk for show; the farm's database is what counts

  

beanstalk: approved @ e5f1e40

Set by Beanstalk for show; the farm's database is what counts

  

beanstalk: approved @ ee8ab5f

Set by Beanstalk for show; the farm's database is what counts

  

ci: approved

A maintainer approved CircleCI for this fork PR (sticky; removed after 14 quiet days)

  

good first issue

A good start for a new contributor

  

gpu/amd

AMD GPUs

  

gpu/igpu

Integrated GPUs

  

gpu/intel

Intel GPUs

  

gpu/nvidia

NVIDIA GPUs

  

help wanted

Extra hands or knowledge wanted

  

kind

bug

Something doesn't work as it should

  

kind

chore

Build, CI, dependencies, tidying

  

kind

docs

Documentation

  

kind

feature

A new feature or an improvement

  

kind

performance

Frame rate, frame time, loading, memory

  

kind

question

A question rather than a change

  

needs measurements

Waiting for before/after measurements

  

platform/linux

Affects Linux (a change for every system carries both Windows and Linux)

  

platform/macos

macOS (Apple silicon or Intel Macs)

  

platform/steam-deck

Steam Deck

  

platform/windows

Affects Windows (a change for every system carries both Windows and Linux)

  

preview

Maintainers: deploy a preview of this PR's website (read its changes first). Removed after the run.

  

priority

critical

Crashes, data loss, or blocks most players

  

priority

high

Important: next in line

  

priority

low

Nice to have

  

regression

Worked in an earlier build, broken in a newer one: highest priority, above other bugs

  

status

blocked

Waits on something else first

  

status

confirmed

Reproduced or agreed on

  

status

duplicate

Already reported or proposed

  

status

invalid

Not a problem in this project

  

status

needs-info

Waiting for more information from the reporter

  

status

wontfix

Will not be worked on

  

testing wanted

Wanted: a test on other hardware (posts a test request in #test-requests)

  

ci: approved

No labels

area/accessibility

area/audio

area/input

area/install

area/menu

area/renderer

area/saves

area/upscaling

beanstalk: approved @ 1697603

beanstalk: approved @ 2c5a62e

beanstalk: approved @ 4fd82b7

beanstalk: approved @ 64d6d09

beanstalk: approved @ 6f91fae

beanstalk: approved @ 70a78f9

beanstalk: approved @ 8d361dc

beanstalk: approved @ 9d8356e

beanstalk: approved @ abf2227

beanstalk: approved @ d82f202

beanstalk: approved @ e5f1e40

beanstalk: approved @ ee8ab5f

ci: approved

good first issue

gpu/amd

gpu/igpu

gpu/intel

gpu/nvidia

help wanted

kind

bug

kind

chore

kind

docs

kind

feature

kind

performance

kind

question

needs measurements

platform/linux

platform/macos

platform/steam-deck

platform/windows

preview

priority

critical

priority

high

priority

low

regression

status

blocked

status

confirmed

status

duplicate

status

invalid

status

needs-info

status

wontfix

testing wanted

ci: approved

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

MagicBeanDev/Longshot!412

No description provided.