-
Notifications
You must be signed in to change notification settings - Fork 7
Align rootfs partition id with freedesktop GUIDs on all platforms #9
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
Comments
I suspect this is a remnant of these platforms shipping without an UEFI capable firmware. |
We imported the partition.conf file for RB1 from db-boot-tool from Linaro, and that GUID comes from here: |
Is there any known meaning to the GUID? Any boot firmware or other tool that might count on it? |
Not sure if this is legit, but this random repo lists it as "userdata" partition: And this one as Android user data: |
yes. The GUIDs are used to 'find' the partition. eg. the ROM CODE (PBL) will find the first stage bootloader (XBL) by looking for the GUID DEA0BA2C-CBDD-4805-B4F9-F428251C3E98. And it's the same thing for each stage of the boot. |
So which part might be break by changing the Android userdata partition? I guess this highlights that this is also OS specific, so really the partitions files here might be for a Linux boot, a Yocto A/B boot, or an Android A/B boot. We should still be consistent and use the right GUIDs when that makes sense, but I suspect we'll maintain partition files in OS-specific repositories. |
Using the userdata GUID was a 'bug'. For our rootfs partition we need to use the correct GUID (b921b045-1df0-41c3-af44-4c6f280d3fae) for Root Partition (64-bit ARM/AArch64). It won't break anything. |
As reported on qualcomm-linux/qcom-ptool#9, the GUID used for rootfs is incorrectly. It was wrongly inherited from the original partition XML file of an Android build, and the GUID which was used corresponds to what Android uses for userdata partition. This changes sets the correct GUID (b921b045-1df0-41c3-af44-4c6f280d3fae) for Root Partition (64-bit ARM/AArch64). Reported-by: Loic Minier <[email protected]> Signed-off-by: Nicolas Dechesne <[email protected]>
As reported on qualcomm-linux/qcom-ptool#9, the GUID used for rootfs is incorrectly. It was wrongly inherited from the original partition XML file of an Android build, and the GUID which was used corresponds to what Android uses for userdata partition. This changes sets the correct GUID (b921b045-1df0-41c3-af44-4c6f280d3fae) for Root Partition (64-bit ARM/AArch64).
Yeah, changing for the rootfs only won't really affect anything here. |
As reported on qualcomm-linux#9, the GUID used for rootfs is incorrectly. It was wrongly inherited from the original partition XML file of an Android build, and the GUID which was used corresponds to what Android uses for userdata partition. This changes sets the correct GUID (b921b045-1df0-41c3-af44-4c6f280d3fae) for Root Partition (64-bit ARM/AArch64). Reported-by: Loic Minier <[email protected]> Signed-off-by: Nicolas Dechesne <[email protected]> Signed-off-by: Sumit Garg <[email protected]>
Uh oh!
There was an error while loading. Please reload this page.
Hi,
In current qcom-ptool, these platforms define an ESP partition called "efi" (using efi.bin) using the proper GUID (
c12a7328-f81f-11d2-ba4b-00a0c93ec93b
) forEFI System Partition
per The Discoverable Partitions Specification (DPS):However while all of these platforms define a root partition called "rootfs" (using rootfs.img), only these two platforms use the correct GUID (
b921b045-1df0-41c3-af44-4c6f280d3fae
) forRoot Partition (64-bit ARM/AArch64)
:While these two use a GUID not defined in the DPS (
1B81E7E6-F50D-419B-A739-2AEEF8DA3335
):Notably, I don't think systemd recognizes this GUID.
The text was updated successfully, but these errors were encountered: