Skip to content

Conversation

iximeow
Copy link
Member

@iximeow iximeow commented Sep 23, 2025

yours truly built a Helios which does not load viona. of course, i only discovered this after i ran propolis-server and got told it couldn't open /dev/vmmctl even though truss very clearly reported it opened just fine.

along the way i was wondering why we have cache_api_version and i think i've intuited part of why, so i added some words there too. i was initially hoping to include the path where OpenOptions::...open() fails and rely on that going all the way up, but errno-ifying it and back means Error::other() is useless there.

for reference, propolis-server on my broken environment now says:

# ./target/debug/propolis-server run /rpool/devel/OVMF_CODE.fd '[::1]:12400'
Sep 23 22:08:24.931 INFO metric registration is disabled, no metric data will be produced by this server
Sep 23 22:08:24.931 ERRO Failed to open /dev/viona
Error: API version checks

Caused by:
    0: checking version of viona
    1: IO Error
    2: No such file or directory (os error 2)

@iximeow iximeow merged commit a9b6fc3 into oxidecomputer:master Sep 24, 2025
11 checks passed
iximeow added a commit to oxidecomputer/omicron that referenced this pull request Oct 18, 2025
Propolis changes:

* oxidecomputer/propolis#950
* oxidecomputer/propolis#952
* oxidecomputer/propolis#951
* oxidecomputer/propolis#954
* oxidecomputer/propolis#957
* oxidecomputer/propolis#960
* oxidecomputer/propolis#961
* oxidecomputer/propolis#955

Crucible changes:

* oxidecomputer/crucible#1773
* oxidecomputer/crucible#1774
* oxidecomputer/crucible#1780
* oxidecomputer/crucible#1778

Crucible shouldn't have functional changes here, Propolis' big ones are
@sunshowers' work moving Propolis to versioned APIs, plus propolis#960
turning the crank on MAXCPU.

propolis#961 changes the initial Milan CPU profile one last time before
the release in service of propolis#959. Propolis will clear [this
bit](https://github.com/oxidecomputer/omicron/blob/d74f5e3f1ae0a378dcdb9795a0ada2426702b046/nexus/src/app/instance_platform/cpu_platform.rs#L423).
Later we want to actually set up leaf 8000_001E, so after this merges
I'll have a followup to remove that leaf from the inital Milan
definition to keep the profile constant when `propolis-server` is
smarter about the leaf.
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