Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Officially name the hypervisor extension "H" #1084

Merged
merged 1 commit into from
Apr 10, 2024

Conversation

a4lg
Copy link
Contributor

@a4lg a4lg commented Aug 8, 2023

Note that this is currently the initial change.

It does not replace other references to

  • the hypervisor extension to
  • the "H" extension.

Still, even without replacing those, it can be a first patch of the series.

TODO List (for complete change)

  • Check whether the name "Standard Extension for Hypervisor" is okay.
  • Determine where to replace (to the "H" extension)
  • Actual replacement (or not)

@kersten1
Copy link
Collaborator

Is this PR moving forward? I can help with the edits if they are approved.

@aswaterman
Copy link
Member

@jhauser-us should approve or reject.

@jhauser-us
Copy link
Collaborator

I think something like "Standard Extension for Hypervisor Support" would be a better name.

@aswaterman
Copy link
Member

The title would be "H" Standard Extension for Hypervisor Support, then. Can you update the PR, @a4lg?

@a4lg
Copy link
Contributor Author

a4lg commented Apr 10, 2024

@aswaterman Sorry for late. I'm back and will work on it.

@aswaterman
Copy link
Member

@a4lg Note that recently we decided to remove the word "Standard" from extension titles, so please make that change here, too. LGTM otherwise.

@a4lg
Copy link
Contributor Author

a4lg commented Apr 10, 2024

@aswaterman Thanks for letting me know (I was off for months and I'm glad to hear such changes to keep up).

This is the initial change in the series and does not replace other
references to "the hypervisor extension" to 'the "H" extension'.

Signed-off-by: Tsukasa OI <research_trasio@irq.a4lg.com>
@a4lg
Copy link
Contributor Author

a4lg commented Apr 10, 2024

Changed the commit title from "standard" to "base".

The use of the word "base" is likely non-canonical, but this is only in the commit title and still reflects the fact that this is the baseline for the hypervisor support (without some extensions expected to be implemented along with the base H extension).

@aswaterman
Copy link
Member

Fine with me. We do not want "Base" in the chapter title, though, similar to how "Base" does not appear in the M-mode or S-mode chapter titles.

@aswaterman aswaterman marked this pull request as ready for review April 10, 2024 01:16
@aswaterman aswaterman merged commit 6aaee4f into riscv:main Apr 10, 2024
2 checks passed
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.

4 participants