Skip to content

refactor: rename "bare metal" to "native host" across codebase#54

Merged
sgopinath1 merged 1 commit into
ROCm:mainfrom
shiv-tyagi:refactor/rename-bare-metal-to-native-host
May 28, 2026
Merged

refactor: rename "bare metal" to "native host" across codebase#54
sgopinath1 merged 1 commit into
ROCm:mainfrom
shiv-tyagi:refactor/rename-bare-metal-to-native-host

Conversation

@shiv-tyagi

@shiv-tyagi shiv-tyagi commented May 28, 2026

Copy link
Copy Markdown
Member

Summary

  • Replaces all occurrences of "bare metal" / "bare-metal" / BareMetal / bare_metal with the equivalent "native host" form throughout code, config, and comments.
  • The old term was misleading since the deployment mode works identically on VMs — "native host" accurately describes running directly on the host OS regardless of whether the underlying machine is physical or virtual.
  • Renames Backend::BareMetalBackend::NativeHost, BareMetalConfigNativeHostConfig, config key bare_metalnative_host, and all associated comments.

Companion to ROCm/spur#236.

Test plan

  • cargo check passes
  • CI validates build

The "bare metal" terminology was misleading since the deployment mode
works equally well on VMs. "Native host" accurately conveys running
directly on the host OS regardless of whether it's physical or virtual.

Renames enum variants, struct names, config keys, comments, and
example config.
@shiv-tyagi
shiv-tyagi force-pushed the refactor/rename-bare-metal-to-native-host branch from 8bfc8d5 to 0c8f5e1 Compare May 28, 2026 09:13
@shiv-tyagi
shiv-tyagi marked this pull request as ready for review May 28, 2026 09:13

@sgopinath1 sgopinath1 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@sgopinath1
sgopinath1 merged commit 6fdf133 into ROCm:main May 28, 2026
4 of 5 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.

2 participants