Skip to content

Makefile: Support usr-merged root filesystems#2781

Open
micromaomao wants to merge 1 commit into
microsoft:mainfrom
micromaomao:usr-merged-fs
Open

Makefile: Support usr-merged root filesystems#2781
micromaomao wants to merge 1 commit into
microsoft:mainfrom
micromaomao:usr-merged-fs

Conversation

@micromaomao

Copy link
Copy Markdown
Member

Without this UVM built with this Makefile doesn't boot when using Azure Linux rootfs.

This is a new version of #2485

Signed-off-by: Dominic Ayre dominicayre@microsoft.com Signed-off-by: Tingmao Wang tingmaowang@microsoft.com

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the rootfs packaging logic in the repository’s Makefile so the generated UVM delta rootfs can boot correctly when the input base root filesystem is usr-merged (e.g., Azure Linux), where /bin is effectively provided via /usr/bin.

Changes:

  • Detect whether the provided $(BASE) rootfs is usr-merged by examining how bin is represented.
  • When usr-merged, create rootfs/usr/bin and make rootfs/bin a symlink to usr/bin to match expected binary locations.
  • Preserve existing behavior for non-usr-merged base rootfs inputs by creating rootfs/bin as a real directory.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Makefile Outdated
@micromaomao micromaomao requested a review from helsaawy June 18, 2026 22:26
Comment thread Makefile Outdated
rm -rf rootfs
mkdir -p rootfs/bin/
rm -rf /tmp/base-rootfs && mkdir -p /tmp/base-rootfs
tar -xf $(BASE) -C /tmp/base-rootfs --no-anchored --no-recursion bin

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.

chatted offline with @helsaawy , $(BASE) isn't being handled in this file, but in Makefile.bootfiles. maybe we should have an explicit makefile flag for usr-merged rootfs instead.

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.

@micromaomao , can you address above and lets merge it.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I'm going to make the flag the default since we will not be building from Yocto again

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Actually to reduce opportunity of objection I've decided to make it non-default and will just update our build scripts instead

Without this UVM built with this Makefile doesn't boot when using Azure Linux
rootfs.

This is a new version of microsoft#2485

Signed-off-by: Dominic Ayre <dominicayre@microsoft.com>
Co-authored-by: Tingmao Wang <tingmaowang@microsoft.com>
Assisted-by: GitHub-Copilot
Signed-off-by: Tingmao Wang <tingmaowang@microsoft.com>
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.

6 participants