Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 19 Aug 2016 09:11:50 +0000 (UTC)
From:      =?UTF-8?Q?Dag-Erling_Sm=c3=b8rgrav?= <des@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-releng@freebsd.org
Subject:   svn commit: r304457 - releng/11.0/usr.sbin/bsdinstall/partedit
Message-ID:  <201608190911.u7J9Boh4032102@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: des
Date: Fri Aug 19 09:11:50 2016
New Revision: 304457
URL: https://svnweb.freebsd.org/changeset/base/304457

Log:
  MFH (r304142): ensure stripe size is non-zero multiple of 4096
  
  PR:		211361
  Approved by:	re (gjb)

Modified:
  releng/11.0/usr.sbin/bsdinstall/partedit/gpart_ops.c
Directory Properties:
  releng/11.0/   (props changed)

Modified: releng/11.0/usr.sbin/bsdinstall/partedit/gpart_ops.c
==============================================================================
--- releng/11.0/usr.sbin/bsdinstall/partedit/gpart_ops.c	Fri Aug 19 08:44:37 2016	(r304456)
+++ releng/11.0/usr.sbin/bsdinstall/partedit/gpart_ops.c	Fri Aug 19 09:11:50 2016	(r304457)
@@ -795,6 +795,7 @@ gpart_max_free(struct ggeom *geom, intma
 {
 	struct gconfig *gc;
 	struct gprovider *pp, **providers;
+	intmax_t sectorsize, stripesize, offset;
 	intmax_t lastend;
 	intmax_t start, end;
 	intmax_t maxsize, maxstart;
@@ -845,12 +846,25 @@ gpart_max_free(struct ggeom *geom, intma
 
 	pp = LIST_FIRST(&geom->lg_consumer)->lg_provider;
 
-	/* Compute beginning of new partition and maximum available space */
-	if (pp->lg_stripesize > 0 &&
-	    (maxstart*pp->lg_sectorsize % pp->lg_stripesize) != 0) {
-		intmax_t offset = (pp->lg_stripesize -
-		    ((maxstart*pp->lg_sectorsize) % pp->lg_stripesize)) /
-		    pp->lg_sectorsize;
+	/*
+	 * Round the start and size of the largest available space up to
+	 * the nearest multiple of the adjusted stripe size.
+	 *
+	 * The adjusted stripe size is the least common multiple of the
+	 * actual stripe size, or the sector size if no stripe size was
+	 * reported, and 4096.  The reason for this is that contemporary
+	 * disks often have 4096-byte physical sectors but report 512
+	 * bytes instead for compatibility with older / broken operating
+	 * systems and BIOSes.  For the same reasons, virtualized storage
+	 * may also report a 512-byte stripe size, or none at all.
+	 */
+	sectorsize = pp->lg_sectorsize;
+	if ((stripesize = pp->lg_stripesize) == 0)
+		stripesize = sectorsize;
+	while (stripesize % 4096 != 0)
+		stripesize *= 2;
+	if ((offset = maxstart * sectorsize % stripesize) != 0) {
+		offset = (stripesize - offset) / sectorsize;
 		maxstart += offset;
 		maxsize -= offset;
 	}



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