Skip to content

Build an instance spec for the latest propolis#135

Merged
jmpesp merged 2 commits intomainfrom
latest_propolis
Apr 10, 2025
Merged

Build an instance spec for the latest propolis#135
jmpesp merged 2 commits intomainfrom
latest_propolis

Conversation

@jmpesp
Copy link
Contributor

@jmpesp jmpesp commented Apr 9, 2025

Update to use the latest propolis-server API: build an instance spec instead of the TOML file, and send that in the instance ensure.

Fixes #126

Update to use the latest propolis-server API: build an instance spec
instead of the TOML file, and send that in the instance ensure.

Fixes #126
@jmpesp jmpesp requested review from gjcolombo and rcgoodfellow April 9, 2025 18:35
@rcgoodfellow
Copy link
Collaborator

Poking around in a4x2, illumos seems a bit mad about a uart

ry@masaka:~/src/testbed/a4x2$ pfexec ./a4x2 serial g0
Entering VM console.
Use ^q to exit.
Press enter to continue.
pr  9 19:59:11 unknown asy: WARNING: asy1: UART @ 2f8 scratch register: expected 0x5a, got 0xff
Apr  9 19:59:11 unknown asy: WARNING: asy1: Cannot identify UART chip at 2f8
Apr  9 19:59:16 unknown asy: WARNING: asy1: UART @ 2f8 scratch register: expected 0x5a, got 0xff
Apr  9 19:59:16 unknown asy: WARNING: asy1: Cannot identify UART chip at 2f8

g0 console login: Apr  9 19:59:21 unknown asy: WARNING: asy1: UART @ 2f8 scratch register: expected 0x5a, got 0xff
Apr  9 19:59:21 unknown asy: WARNING: asy1: Cannot identify UART chip at 2f8
Apr  9 19:59:26 unknown asy: WARNING: asy1: UART @ 2f8 scratch register: expected 0x5a, got 0xff
Apr  9 19:59:26 unknown asy: WARNING: asy1: Cannot identify UART chip at 2f8

g0 console login:
g0 console login: Apr  9 19:59:31 unknown asy: WARNING: asy1: UART @ 2f8 scratch register: expected 0x5a, got 0xff
Apr  9 19:59:31 unknown asy: WARNING: asy1: Cannot identify UART chip at 2f8

g0 console login: Apr  9 19:59:36 unknown asy: WARNING: asy1: UART @ 2f8 scratch register: expected 0x5a, got 0xff
Apr  9 19:59:36 unknown asy: WARNING: asy1: Cannot identify UART chip at 2f8
Apr  9 19:59:41 unknown asy: WARNING: asy1: UART @ 2f8 scratch register: expected 0x5a, got 0xff
Apr  9 19:59:41 unknown asy: WARNING: asy1: Cannot identify UART chip at 2f8

@gjcolombo
Copy link

0x2F8 is the I/O port address for COM2. Propolis used to add all of COM1-COM3 implicitly to Falcon VMs, but it now requires them to be specified explicitly in the instance ensure inputs. I think the fix here might be to add COM2 and COM3 in Node::preflight (right-side line 789 of lib/src/lib.rs in this PR); as the rest of the function notes, COM4 will be enabled implicitly by the SoftNpuPciPort device that's already in the spec.

@jmpesp
Copy link
Contributor Author

jmpesp commented Apr 9, 2025

0x2F8 is the I/O port address for COM2. Propolis used to add all of COM1-COM3 implicitly to Falcon VMs, but it now requires them to be specified explicitly in the instance ensure inputs. I think the fix here might be to add COM2 and COM3 in Node::preflight (right-side line 789 of lib/src/lib.rs in this PR); as the rest of the function notes, COM4 will be enabled implicitly by the SoftNpuPciPort device that's already in the spec.

👍 thanks for digging into that, pushed the suggestion in a609c6d

@rcgoodfellow
Copy link
Collaborator

Things are looking better after a609c6d

Copy link
Collaborator

@rcgoodfellow rcgoodfellow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jmpesp! LGTM and a4x2 is working.

Copy link

@gjcolombo gjcolombo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for putting this together!

@jmpesp jmpesp merged commit 372a9a2 into main Apr 10, 2025
2 checks passed
@jmpesp jmpesp deleted the latest_propolis branch April 10, 2025 14:29
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.

Move to latest propolis

3 participants