From owner-svn-src-head@FreeBSD.ORG Fri Apr 25 01:38:57 2014 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id DD790A02; Fri, 25 Apr 2014 01:38:57 +0000 (UTC) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:1900:2254:2068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id C9C5F1951; Fri, 25 Apr 2014 01:38:57 +0000 (UTC) Received: from svn.freebsd.org ([127.0.1.70]) by svn.freebsd.org (8.14.8/8.14.8) with ESMTP id s3P1cvSF050582; Fri, 25 Apr 2014 01:38:57 GMT (envelope-from gjb@svn.freebsd.org) Received: (from gjb@localhost) by svn.freebsd.org (8.14.8/8.14.8/Submit) id s3P1cvHn050580; Fri, 25 Apr 2014 01:38:57 GMT (envelope-from gjb@svn.freebsd.org) Message-Id: <201404250138.s3P1cvHn050580@svn.freebsd.org> From: Glen Barber Date: Fri, 25 Apr 2014 01:38:57 +0000 (UTC) To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r264907 - in head/release: amd64 i386 X-SVN-Group: head MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.17 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 25 Apr 2014 01:38:57 -0000 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 # @@ -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 # @@ -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}