From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Mon, 12 Oct 2015 14:09:56 -0400 Subject: [U-Boot] [PATCH] part:efi: add GUID for linux file system data In-Reply-To: References: <20151012151406.GC23893@bill-the-cat> Message-ID: <20151012180956.GP23893@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Mon, Oct 12, 2015 at 07:48:48PM +0200, Patrick Delaunay wrote: > 2015-10-12 17:14 GMT+02:00 Tom Rini : > > > On Mon, Oct 12, 2015 at 05:03:15PM +0200, Patrick Delaunay wrote: > > > > > Signed-off-by: Patrick Delaunay > > > --- > > > > > > see > > https://en.wikipedia.org/wiki/GUID_Partition_Table#Partition_type_GUIDs > > > > > > nota 9 : > > > Previously, Linux used the same GUID for the data partitions as Windows > > > (Basic data partition: EBD0A0A2-B9E5-4433-87C0-68B6B72699C7). > > > Linux never had a separate unique partition type GUID defined for its > > > data partitions. > > > This created problems when dual-booting Linux and Windows in UEFI-GPT > > > setup. > > > The new GUID (Linux filesystem data: > > 0FC63DAF-8483-4772-8E79-3D69D8477DE4) > > > was defined jointly by GPT fdisk and GNU Parted developers. > > > It is identified as type code 0x8300 in GPT fdisk. > > > (See definitions in gdisk's parttypes.cc) > > > > Some of this should be in the main commit message, not the discarded > > part. > > > > I wasn't sure if these level of informations should be in commit message or > not. > > I can add in commit message : > > Previously, Linux used the same GUID for the data partitions as Windows > (Basic data partition: EBD0A0A2-B9E5-4433-87C0-68B6B72699C7). > This created problems when dual-booting Linux and Windows in UEFI-GPT > Setup, so a new GUID (Linux filesystem data: > 0FC63DAF-8483-4772-8E79-3D69D8477DE4) > was defined jointly by GPT fdisk and GNU Parted developers. Sounds good. > > > include/part_efi.h | 3 +++ > > > 1 file changed, 3 insertions(+) > > > > And without some changes to disk/part_efi.c I believe, this isn't used > > anywhere so not actually changing behavior yet :) > > yes today the behavior don't change yet. > > in fact only two GUID defines is really use in u-boot code : > > ./disk/part_efi.c:63:static efi_guid_t system_guid = PARTITION_SYSTEM_GUID; > ./disk/part_efi.c:449: &PARTITION_BASIC_DATA_GUID, 16); > > all the other existing value are not used today, so I add a other define > not used > because I don't see any simple way to use one the correct guid. > > I want to propose a patch to allow selection of partition type guid in > command gpt (with new option guid=) > but without dependancy with this patch > > and I plan to use this define and this new option to manage the partionning > on my project. OK. Please include this change then in the patch series you're going to post later on, thanks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: