Skip to content

Nvidia unstable 11.9+cca/latest#4

Merged
NathanChenNVIDIA merged 2 commits intoNVIDIA:nvidia_unstable-11.9from
ianm-nv:nvidia_unstable-11.9+cca/latest
Nov 20, 2025
Merged

Nvidia unstable 11.9+cca/latest#4
NathanChenNVIDIA merged 2 commits intoNVIDIA:nvidia_unstable-11.9from
ianm-nv:nvidia_unstable-11.9+cca/latest

Conversation

@ianm-nv
Copy link
Collaborator

@ianm-nv ianm-nv commented Nov 19, 2025

Auto-convert pflash to ROM for ARM CCA guests

What this does

Reverts the previous patch that rejected pflash configurations for CCA guests, and replaces it with automatic pflash to ROM conversion.

Background

The old approach (commit 8a6274f) rejected any pflash loader when CCA was enabled, forcing users to manually specify ROM in their XML. This meant:

  • Separate XMLs needed for CCA vs non-CCA
  • virt-install commands would fail with CCA since it defaults to pflash for UEFI
  • Pain when switching between Realm and non-Realm modes

New approach

Instead of rejecting pflash, automatically convert it to ROM when <launchSecurity type='cca'> is present. NVRAM gets cleared automatically since ROM doesn't support it. Now the same XML works for both - just toggle the launchSecurity element.

What changed

  • Reverted the old "reject pflash" patch
  • Added auto-conversion in post-parse phase
  • Updated validation to verify conversion happened (sanity check)
  • Fixed ACPI validation to allow it with CCA guests
  • Updated docs

Testing

Tested on hardware. virt-install with pflash + CCA works now. Non-CCA VMs still use pflash normally.

…ests"

This reverts commit 8a6274f.

This patch is being replaced with an auto-conversion approach that converts
pflash to ROM for CCA guests instead of rejecting the configuration. This
allows the same domain XML to work for both CCA and non-CCA guests by simply
toggling the launchSecurity element.

Signed-off-by: Ian May <ianm@nvidia.com>
The CCA-enabled Arm virt machine doesn't support pflash devices when realm
mode is active. Automatically convert pflash loaders to ROM and ignore NVRAM
configuration when CCA launch security is enabled. This allows the same
domain XML to work for both CCA and non-CCA guests.

Signed-off-by: Ian May <ianm@nvidia.com>
@nvmochs nvmochs self-requested a review November 20, 2025 00:03
Copy link
Collaborator

@nvmochs nvmochs left a comment

Choose a reason for hiding this comment

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

This seems like a better approach. Changes LGTM!

Acked-by: Matthew R. Ochs <mochs@nvidia.com>

Copy link
Collaborator

@NathanChenNVIDIA NathanChenNVIDIA left a comment

Choose a reason for hiding this comment

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

Acked-by: Nathan Chen nathanc@nvidia.com

@NathanChenNVIDIA NathanChenNVIDIA merged commit c8dd7d6 into NVIDIA:nvidia_unstable-11.9 Nov 20, 2025
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