Skip to content

Default partuuid parameter of _get_kernel_params_partition() to false - #3947

Closed
Mintsuki wants to merge 1 commit into
archlinux:masterfrom
Mintsuki:no_partuuid
Closed

Default partuuid parameter of _get_kernel_params_partition() to false#3947
Mintsuki wants to merge 1 commit into
archlinux:masterfrom
Mintsuki:no_partuuid

Conversation

@Mintsuki

@Mintsuki Mintsuki commented Nov 25, 2025

Copy link
Copy Markdown
Contributor

There isn't a reason I could think of for preferring PARTUUIDs to UUIDs, since as far as I know that should work fine with all supported filesystems (and GRUB already forces this to false). This is especially important when using MBR instead of GPT, as that doesn't have proper UUIDs, and it will instead use the kernel's own conversion that is based on the MBR ID and more prone to collisions.

@Mintsuki
Mintsuki requested a review from Torxed as a code owner November 25, 2025 14:25
@Mintsuki Mintsuki changed the title Default partuuid parameter of _get_kernel_params_partition() to true Default partuuid parameter of _get_kernel_params_partition() to false Nov 25, 2025
@svartkanin

Copy link
Copy Markdown
Collaborator

Couold you provide a description of why this is needed?

@Mintsuki

Copy link
Copy Markdown
Contributor Author

Couold you provide a description of why this is needed?

I have updated the original description.

…alse

There isn't a reason I could think of for preferring PARTUUIDs to UUIDs, since
as far as I know that should work fine with all supported filesystems (and GRUB
already forces this to false). This is especially important when using MBR
instead of GPT, as that doesn't have proper UUIDs, and it will instead use the
kernel's own conversion that is based on the MBR ID and more prone to
collisions.
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