From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1f4xG1-0006Dn-Op for mharc-grub-devel@gnu.org; Sat, 07 Apr 2018 19:28:29 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40267) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1f4xFz-0006D0-Pf for grub-devel@gnu.org; Sat, 07 Apr 2018 19:28:29 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1f4xFy-0003MR-Em for grub-devel@gnu.org; Sat, 07 Apr 2018 19:28:27 -0400 Received: from mail-pl0-x244.google.com ([2607:f8b0:400e:c01::244]:35794) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1f4xFy-0003MF-4o for grub-devel@gnu.org; Sat, 07 Apr 2018 19:28:26 -0400 Received: by mail-pl0-x244.google.com with SMTP id 61-v6so2853476plb.2 for ; Sat, 07 Apr 2018 16:28:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:subject:date:message-id; bh=LtJkxfOtVEsRyFnDWEeE0IrFAtG2juBc7+zn9ge8Fg8=; b=N6TNtuBrw+rJ6gxPLiYPYlQGAFYz3GrA7qBtfIsoQM7JF68o1mcU+XoqIJaIpiKQrm 5B+VDGbZDArzTgOueXmyn3sAWi768ymd6DjeZlYKOw8ld4xbkUmxSBCmLtmM68k6BCHE bcJslZk3XDj2jrcUzzD/PDuBTtCNjcatDE0SGEMeR1XU37IrpJj2ewX6q+1b3RUeY50d BwuBvEMpMMXkDPyh2J1hYM4+/OnEaJixEz+5nY/yoyOKjFqksb+kiigIwfXZZeBGPdld st2mXd8Wb7zh5UnAIibdB7VVBtPJt/HzJWyHseVSDOwc/iE0pYWmuvkbIRLh+BnrWIH9 u17g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id; bh=LtJkxfOtVEsRyFnDWEeE0IrFAtG2juBc7+zn9ge8Fg8=; b=YkZVsJUNdyAryrT4X9T0CEPnoQQB2lpr9bbii/7BbyNtBpG4cuTwlUqSwk58m+ggOJ rjCg0rN7whAqgRwQuJt5PrmHOL7CLOVCwTq+S4988If5t998iJEAFwoPEzK4XdBWGl0P ek6bCLMyZXLD57cI/iyUt0vdWFMvRVay/bZgX67fM8+MM2oScGONW0qfh7xc1a8tr9dA MFpYFLZ/n3SI9wz2vmgIJ5Qw32+AGeSvDtt3GxIBpsfZ55V9lrUd7S1g+d5xjTQaR2GT umQdvqqcTKNanRaEVrMX88fWQBXzcRY/YAnoWPV8ZJaJT6ThXUf+kDV6BRV2oqI9zU7q I9fA== X-Gm-Message-State: ALQs6tA5Dh65Zr0CVKf1t50EKWKWfeA4ZqI4oBu8BwCrwySChVl+96Ov ZFFAQs69jwoZOwPy3ByLvmWxXA== X-Google-Smtp-Source: AIpwx49neXHWj8nWzTnN788TFQdqRUPWt+6R4MNwTKR6kngFg9ql2VSqkp3utnj81vz5wW3Kz9wPuQ== X-Received: by 2002:a17:902:4324:: with SMTP id i33-v6mr19881467pld.379.1523143704239; Sat, 07 Apr 2018 16:28:24 -0700 (PDT) Received: from localhost.localdomain ([2601:602:9c01:e86a:2ec2:3a19:cd81:76ab]) by smtp.googlemail.com with ESMTPSA id 85sm26092944pfz.155.2018.04.07.16.28.22 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sat, 07 Apr 2018 16:28:23 -0700 (PDT) From: Nicholas Vinson To: grub-devel@gnu.org Subject: [GRUB PARTUUID PATCH V9 0/5] Add PARTUUID detection support Date: Sat, 7 Apr 2018 16:28:09 -0700 Message-Id: X-Mailer: git-send-email 2.17.0 X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2607:f8b0:400e:c01::244 X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: The development of GNU GRUB List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 07 Apr 2018 23:28:29 -0000 Changes from Patch v8: - Renamed GRUB_ENABLE_LINUX_PARTUUID to GRUB_DISABLE_LINUX_PARTUUID - Updated the 10_linux logic so GRUB_ENABLE_LINUX_PARTUUID and GRUB_ENABLE_LINUX_UUID would behave more independently. - Documented interactions of GRUB_DISABLE_LINUX_UUID, GRUB_DISABLE_LINUX_PARTUUID, and initramfs detection in commit message. - Added optional patch that sets GRUB_DISABLE_LINUX_PARTUUID to true by default. - Fixed merge conflicts between this patchset and upstream's master branch. Changes from Patch v7: - Changed checks in probe_partuuid() to use the variable 'p' instead of disk->partition - Moved 'disk->partition = p;' assignment to inside the 'p & p->parent == NULL' statement - Moved 'disk->partition = p->parent;' assignment to before partmap name checks. - Fixed formatting issues. - Copied Daniel Kiper's reviewed-by line to unaltered patches. Changes from Patch v6: - Corrected spelling and grammatical errors in description text for GRUB_ENABLE_LINUX_PARTUUID - Moved disk->partition save & restore logic to beginning and end of probe_partuuid() - Fixed formatting errors in probe_partuuid Changes from Patch v5: - Added sign-off by lines - Fixed formatting errors found by Daniel Kiper Changes from Patch v4: - Updated grub.texi to reflect new behavior for GRUB_ENABLE_LINUX_PARTUUID - Updated 10_linux.in logic to favor the PARTUUID when GRUB_ENABLE_LINUX_PARTUUID is enabled and GRUB_DISABLE_LINUX_UUID is disabled. Changes from Patch v3: - Removed flex-2.6.3 compatibility patch - Removed Steve Kenton's patch Changes from Patch v2: - Added flex-2.6.3 compatibility patch - Fixed a GPT partition read error - Added Steve Kenton's patch - Changed struct grub_part_gpt_type name to struct grub_part_gpt_part_guid - Changed grub_part_gpt_type_t typedef name to grub_part_gpt_guid_t - Added sprint_gpt_guid to Steve Kenton's patch - Updated v1 and Steve Kenton's patch to use similar methods when reading partition GUIDs. Changes from Patch v1: - Added GRUB_ENABLE_LINUX_PARTUUID variable description to grub.texi - Removed added gpt_part_guid copy logic from grub_gpt_partition_map_iterate() - Removed added NT disk signature copy logic from grub_partition_msdos_iterate() - Removed modifications to partition number increment logic - Removed added guid union definition. - Added GRUB_ENABLE_LINUX_PARTUUID to grub-mkconfig.in export list - Moved PRINT_GPT_PARTTYPE printing logic to print_gpt_guid() function in grub-probe.c - Updated PRINT_GPT_PARTTYPE case to call print_gpt_guid() function in grub-probe.c. - Created probe_partuuid() function in grub-probe.c - Updated print == PRINT_PARTUUID check logic in probe() to call probe_partuuid(). - Updated UUID logic in 10_linux.in to enable root=PARTUUID feature only if GRUB_DISABLE_LINUX_UUID is not set to true, and GRUB_DEVICE_PARTUUID is not empty, GRUB_ENABLE_LINUX_PARTUUID is set to true. Hello, This is a request to add PARTUUID detection support grub-probe for MBR and GPT partition schemes. The Linux kernel supports mounting the root filesystem by Linux device name or by the Partition [GU]UID. GRUB's mkconfig, however, currently only supports specifying the rootfs in the kernel command-line by Linux device name unless an initramfs is also present. When an initramfs is present GRUB's mkconfig will set the kernel's root parameter value to either the Linux device name or to the filesystem [GU]UID. Therefore, the only way to protect a Linux system from failing to boot when its Linux storage device names change is to either manually edit grub.cfg or /etc/default/grub and append root=PARTUUID=xxx to the command-line or create an initramfs that understands how to mount devices by filesystem [G]UID and let grub-mkconfig pass the filesystem [GU]UID to the initramfs. The goal of this patch set is to enable root=PARTUUID=xxx support in grub-mkconfig, so that users don't have to manually edit /etc/default/grub or grub.cfg, or create an initramfs for the sole purpose of having a robust bootloader configuration for Linux. Thanks, Nicholas Vinson Nicholas Vinson (5): Centralize guid prints Update grub_gpt_partentry Add PARTUUID detection support to grub-probe Update grub script template files Default to disabling partition UUID support docs/grub.texi | 11 +++++++ grub-core/disk/ldm.c | 2 +- grub-core/partmap/gpt.c | 4 +-- include/grub/gpt_partition.h | 8 ++--- util/grub-install.c | 2 +- util/grub-mkconfig.in | 3 ++ util/grub-probe.c | 76 +++++++++++++++++++++++++++++++++++--------- util/grub.d/10_linux.in | 22 +++++++++++-- 8 files changed, 102 insertions(+), 26 deletions(-) -- 2.16.3