Skip to content

Add platform support to README - #1967

Open
yoshuawuyts wants to merge 1 commit into
mainfrom
platform-support
Open

yoshuawuyts wants to merge 1 commit into
mainfrom
platform-support

Conversation

@yoshuawuyts

Copy link
Copy Markdown
Contributor

This came from a conversation yesterday with @simongdavies, but we're currently not communicating our platform support clearly anywhere. That includes base hypervisor support, but also support for features like debugging and crash dumps.

The second part of the diff here is because I ran "format tables" on the entire file. I figured that would make it a bit more legible, so I kept it in.

I hope I got all the info right in the table; please have a look. Thanks!

Signed-off-by: Yosh <github@yosh.is>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The documentation-only changes have one minor, nonblocking accuracy correction.

1 open issue
What changed in this PR

The README documents Hyperlight’s platform and feature support.

Changes:

  • Adds support tables for Linux, Windows, and macOS.
  • Lists hypervisor, debugger, and crash dump availability.
  • Formats the repository structure table.
File Description
README.md Documents platform support and formats existing tables.

Review effort: Balanced

Comment thread README.md
| ------------ | ---------- | --------------- | -------- | ---------- |
| aarch64 | HVF | ✅ Yes | ❌ No | ❌ No |

¹ Not yet covered by CI

@simongdavies simongdavies left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

At line 14 the README says "Supports KVM, MSHV, and Windows Hypervisor Platform".

We should probably add AVF/HVF there or remove that line.

Also as copilot noted we do have CI for Windows/WHP/ARM so I think it should be noted as supported not experimental

Comment thread README.md

### Linux

| Architecture | Hypervisor | General support | Debugger | Crashdumps |

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nit: should we make the headers "Guest Debugging"/ Guest Crashdumps?

This branch has not been deployed

No deployments
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.

3 participants