Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 25 Apr 2014 01:38:57 +0000 (UTC)
From:      Glen Barber <gjb@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r264907 - in head/release: amd64 i386
Message-ID:  <201404250138.s3P1cvHn050580@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: gjb
Date: Fri Apr 25 01:38:57 2014
New Revision: 264907
URL: http://svnweb.freebsd.org/changeset/base/264907

Log:
  Refactor make-memstick.sh to avoid creating the 'dangerously
  dedicated' partition scheme, reported to cause the memstick.img
  to fail to boot.
  
  Similar to how make-memstick.sh worked on stable/8, use makefs(8)
  to create the actual filesystem.  Then calculate the size of the
  resulting image file, create the GPT partition scheme, then dd(1)
  the filesystem created with makefs(8) to the freebsd-ufs GPT
  partition.
  
  This was tested on a known-working machine[1] for regression, and
  a known-not-working machine[2] to ensure the boot issue has been
  resolved.
  
  Testers:	myself [1], db [2]
  MFC After:	1 week
  Sponsored by:	The FreeBSD Foundation

Modified:
  head/release/amd64/make-memstick.sh
  head/release/i386/make-memstick.sh

Modified: head/release/amd64/make-memstick.sh
==============================================================================
--- head/release/amd64/make-memstick.sh	Fri Apr 25 01:20:10 2014	(r264906)
+++ head/release/amd64/make-memstick.sh	Fri Apr 25 01:38:57 2014	(r264907)
@@ -3,7 +3,7 @@
 # This script generates a "memstick image" (image that can be copied to a
 # USB memory stick) from a directory tree.  Note that the script does not
 # clean up after itself very well for error conditions on purpose so the
-# problem can be diagnosed (full filesystem most likely but ...).
+# problem can be diagnosed.
 #
 # Usage: make-memstick.sh <directory tree> <image filename>
 #
@@ -13,11 +13,15 @@
 PATH=/bin:/usr/bin:/sbin:/usr/sbin
 export PATH
 
+BLOCKSIZE=10240
+
 if [ $# -ne 2 ]; then
   echo "make-memstick.sh /path/to/directory /path/to/image/file"
   exit 1
 fi
 
+tempfile="${2}.$$"
+
 if [ ! -d ${1} ]; then
   echo "${1} must be a directory"
   exit 1
@@ -28,21 +32,62 @@ if [ -e ${2} ]; then
   exit 1
 fi
 
-echo '/dev/ufs/FreeBSD_Install / ufs ro,noatime 1 1' > ${1}/etc/fstab
-makefs -B little -o label=FreeBSD_Install ${2} ${1}
+rm -f ${tempfile}
+echo "/dev/gpt/rootfs / ufs ro,noatime 1 1" > ${1}/etc/fstab
+makefs ${tempfile} ${1}
 if [ $? -ne 0 ]; then
   echo "makefs failed"
   exit 1
 fi
 rm ${1}/etc/fstab
 
+#
+# Use $BLOCKSIZE for transfers to improve efficiency.  When calculating
+# how many blocks to transfer "+ 520" is to account for truncation in the
+# division and to provide ample padding.
+#
+
+filesize=`stat -f "%z" ${tempfile}`
+blocks=$(($filesize / ${BLOCKSIZE} + 520 ))
+dd if=/dev/zero of=${2} bs=${BLOCKSIZE} count=${blocks}
+if [ $? -ne 0 ]; then
+  echo "creation of image file failed"
+  exit 1
+fi
+
 unit=`mdconfig -a -t vnode -f ${2}`
 if [ $? -ne 0 ]; then
   echo "mdconfig failed"
   exit 1
 fi
-gpart create -s BSD ${unit}
-gpart bootcode -b ${1}/boot/boot ${unit}
-gpart add -t freebsd-ufs ${unit}
+
+gpart create -s gpt /dev/${unit}
+if [ $? -ne 0 ]; then
+  echo "GPT creation failed"
+  exit 1
+fi
+gpart add -t freebsd-boot -s 512k /dev/${unit}
+if [ $? -ne 0 ]; then
+  echo "Creating boot partition failed"
+  exit 1
+fi
+gpart bootcode -b ${1}/boot/pmbr -p ${1}/boot/gptboot -i 1 /dev/${unit}
+if [ $? -ne 0 ]; then
+  echo "Writing bootcode failed"
+  exit 1
+fi
+gpart add -t freebsd-ufs -l rootfs /dev/${unit}
+if [ $? -ne 0 ]; then
+  echo "Creating UFS partition failed"
+  exit 1
+fi
+
+dd if=${tempfile} of=/dev/${unit}p2 bs=$BLOCKSIZE conv=sync
+if [ $? -ne 0 ]; then
+  echo "copying filesystem into image file failed"
+  exit 1
+fi
+
 mdconfig -d -u ${unit}
 
+rm -f ${tempfile}

Modified: head/release/i386/make-memstick.sh
==============================================================================
--- head/release/i386/make-memstick.sh	Fri Apr 25 01:20:10 2014	(r264906)
+++ head/release/i386/make-memstick.sh	Fri Apr 25 01:38:57 2014	(r264907)
@@ -3,7 +3,7 @@
 # This script generates a "memstick image" (image that can be copied to a
 # USB memory stick) from a directory tree.  Note that the script does not
 # clean up after itself very well for error conditions on purpose so the
-# problem can be diagnosed (full filesystem most likely but ...).
+# problem can be diagnosed.
 #
 # Usage: make-memstick.sh <directory tree> <image filename>
 #
@@ -13,11 +13,15 @@
 PATH=/bin:/usr/bin:/sbin:/usr/sbin
 export PATH
 
+BLOCKSIZE=10240
+
 if [ $# -ne 2 ]; then
   echo "make-memstick.sh /path/to/directory /path/to/image/file"
   exit 1
 fi
 
+tempfile="${2}.$$"
+
 if [ ! -d ${1} ]; then
   echo "${1} must be a directory"
   exit 1
@@ -28,21 +32,62 @@ if [ -e ${2} ]; then
   exit 1
 fi
 
-echo '/dev/ufs/FreeBSD_Install / ufs ro,noatime 1 1' > ${1}/etc/fstab
-makefs -B little -o label=FreeBSD_Install ${2} ${1}
+rm -f ${tempfile}
+echo "/dev/gpt/rootfs / ufs ro,noatime 1 1" > ${1}/etc/fstab
+makefs ${tempfile} ${1}
 if [ $? -ne 0 ]; then
   echo "makefs failed"
   exit 1
 fi
 rm ${1}/etc/fstab
 
+#
+# Use $BLOCKSIZE for transfers to improve efficiency.  When calculating
+# how many blocks to transfer "+ 520" is to account for truncation in the
+# division and to provide ample padding.
+#
+
+filesize=`stat -f "%z" ${tempfile}`
+blocks=$(($filesize / ${BLOCKSIZE} + 520 ))
+dd if=/dev/zero of=${2} bs=${BLOCKSIZE} count=${blocks}
+if [ $? -ne 0 ]; then
+  echo "creation of image file failed"
+  exit 1
+fi
+
 unit=`mdconfig -a -t vnode -f ${2}`
 if [ $? -ne 0 ]; then
   echo "mdconfig failed"
   exit 1
 fi
-gpart create -s BSD ${unit}
-gpart bootcode -b ${1}/boot/boot ${unit}
-gpart add -t freebsd-ufs ${unit}
+
+gpart create -s gpt /dev/${unit}
+if [ $? -ne 0 ]; then
+  echo "GPT creation failed"
+  exit 1
+fi
+gpart add -t freebsd-boot -s 512k /dev/${unit}
+if [ $? -ne 0 ]; then
+  echo "Creating boot partition failed"
+  exit 1
+fi
+gpart bootcode -b ${1}/boot/pmbr -p ${1}/boot/gptboot -i 1 /dev/${unit}
+if [ $? -ne 0 ]; then
+  echo "Writing bootcode failed"
+  exit 1
+fi
+gpart add -t freebsd-ufs -l rootfs /dev/${unit}
+if [ $? -ne 0 ]; then
+  echo "Creating UFS partition failed"
+  exit 1
+fi
+
+dd if=${tempfile} of=/dev/${unit}p2 bs=$BLOCKSIZE conv=sync
+if [ $? -ne 0 ]; then
+  echo "copying filesystem into image file failed"
+  exit 1
+fi
+
 mdconfig -d -u ${unit}
 
+rm -f ${tempfile}



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201404250138.s3P1cvHn050580>