svn commit: r320088 - in stable/11/usr.sbin/bsdinstall: partedit scripts

[Ed Maste]

Author: emaste
Date: Mon Jun 19 03:07:00 2017
New Revision: 320088
URL: https://svnweb.freebsd.org/changeset/base/320088

Log:
  bsdinstall: use consistent EFI configuration across platforms
  
  MFC r320007:
  - increase arm64 EFI partition to 200M, as x86
  - use EFI_BOOTPART_SIZE and EFI_BOOTPART_PATH macros on x86
  - increase ZFS EFI partition to 200M
  
  MFC r320008: bsdinstall: correct comment after r320007
  
  PR:		201898
  Approved by:	re (kib)
  Relnotes:	Yes
  Sponsored by:	The FreeBSD Foundation

Modified:
  stable/11/usr.sbin/bsdinstall/partedit/partedit_arm64.c
  stable/11/usr.sbin/bsdinstall/partedit/partedit_x86.c
  stable/11/usr.sbin/bsdinstall/scripts/zfsboot
Directory Properties:
  stable/11/   (props changed)

Modified: stable/11/usr.sbin/bsdinstall/partedit/partedit_arm64.c
==============================================================================
--- stable/11/usr.sbin/bsdinstall/partedit/partedit_arm64.c	Sun Jun 18 21:53:17 2017	(r320087)
+++ stable/11/usr.sbin/bsdinstall/partedit/partedit_arm64.c	Mon Jun 19 03:07:00 2017	(r320088)
@@ -34,8 +34,8 @@
 
 #include "partedit.h"
 
-/* EFI partition size in KB */
-#define	EFI_BOOTPART_SIZE	(50 * 1024)
+/* EFI partition size in bytes */
+#define	EFI_BOOTPART_SIZE	(200 * 1024 * 1024)
 #define	EFI_BOOTPART_PATH	"/boot/boot1.efifat"
 
 const char *
@@ -73,7 +73,7 @@ bootpart_size(const char *scheme)
 	if (strcmp(scheme, "GPT") != 0)
 		return (0);
 
-	return ((EFI_BOOTPART_SIZE) * 1024);
+	return (EFI_BOOTPART_SIZE);
 }
 
 const char *

Modified: stable/11/usr.sbin/bsdinstall/partedit/partedit_x86.c
==============================================================================
--- stable/11/usr.sbin/bsdinstall/partedit/partedit_x86.c	Sun Jun 18 21:53:17 2017	(r320087)
+++ stable/11/usr.sbin/bsdinstall/partedit/partedit_x86.c	Mon Jun 19 03:07:00 2017	(r320088)
@@ -32,6 +32,10 @@
 
 #include "partedit.h"
 
+/* EFI partition size in bytes */
+#define	EFI_BOOTPART_SIZE	(200 * 1024 * 1024)
+#define	EFI_BOOTPART_PATH	"/boot/boot1.efifat"
+
 static const char *
 x86_bootmethod(void)
 {
@@ -99,7 +103,7 @@ bootpart_size(const char *scheme)
 	if (strcmp(x86_bootmethod(), "BIOS") == 0)
 		return (512*1024);
 	else 
-		return (200*1024*1024);
+		return (EFI_BOOTPART_SIZE);
 
 	return (0);
 }
@@ -137,7 +141,7 @@ partcode_path(const char *part_type, const char *fs_ty
 
 	if (strcmp(part_type, "GPT") == 0) {
 		if (strcmp(x86_bootmethod(), "UEFI") == 0)
-			return ("/boot/boot1.efifat");
+			return (EFI_BOOTPART_PATH);
 		else if (strcmp(fs_type, "zfs") == 0)
 			return ("/boot/gptzfsboot");
 		else

Modified: stable/11/usr.sbin/bsdinstall/scripts/zfsboot
==============================================================================
--- stable/11/usr.sbin/bsdinstall/scripts/zfsboot	Sun Jun 18 21:53:17 2017	(r320087)
+++ stable/11/usr.sbin/bsdinstall/scripts/zfsboot	Mon Jun 19 03:07:00 2017	(r320088)
@@ -848,7 +848,7 @@ zfs_create_diskpart()
 
 			f_eval_catch $funcname gpart \
 			             "$GPART_ADD_ALIGN_LABEL_WITH_SIZE" \
-			             "$align_small" efiboot$index efi 800k $disk ||
+			             "$align_small" efiboot$index efi 200M $disk ||
 			             return $FAILURE
 			f_eval_catch $funcname gpart "$GPART_BOOTCODE_PARTONLY" \
 			             /boot/boot1.efifat 1 $disk ||