From owner-svn-src-all@freebsd.org Fri Aug 19 15:54:01 2016 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 87252BBDFA0; Fri, 19 Aug 2016 15:54:01 +0000 (UTC) (envelope-from allanjude@freebsd.org) Received: from mx1.scaleengine.net (mx1.scaleengine.net [209.51.186.6]) (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 4D0241202; Fri, 19 Aug 2016 15:54:01 +0000 (UTC) (envelope-from allanjude@freebsd.org) Received: from [10.1.1.2] (unknown [10.1.1.2]) (Authenticated sender: allanjude.freebsd@scaleengine.com) by mx1.scaleengine.net (Postfix) with ESMTPSA id 9C6101D60; Fri, 19 Aug 2016 15:53:54 +0000 (UTC) Subject: Re: svn commit: r304142 - head/usr.sbin/bsdinstall/partedit To: Warner Losh References: <201608150930.u7F9UL1V069576@repo.freebsd.org> <861t1n6749.fsf@desk.des.no> <581c856c-826b-529e-c9c6-a397fb679708@freebsd.org> <86wpjf4eun.fsf@desk.des.no> <8cb3fa1a-50cb-e238-d006-b98a628d446d@freebsd.org> <86k2ff4cxs.fsf@desk.des.no> <86fuq24d8s.fsf@desk.des.no> <2f9fb04c-7ec4-be40-8fcb-0cf74bb56859@freebsd.org> Cc: =?UTF-8?Q?Dag-Erling_Sm=c3=b8rgrav?= , Nathan Whitehorn , src-committers , "svn-src-all@freebsd.org" , "svn-src-head@freebsd.org" From: Allan Jude Message-ID: <2c942f49-b90c-58eb-159f-4c4e62624bac@freebsd.org> Date: Fri, 19 Aug 2016 11:53:42 -0400 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="3vJ9Ixf52qiPb3wJHq6RTu3p9IpOuxluR" X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 19 Aug 2016 15:54:01 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --3vJ9Ixf52qiPb3wJHq6RTu3p9IpOuxluR Content-Type: multipart/mixed; boundary="sxpS8TNfnWpFWDHAo0LUfWPwPtRHSisr1" From: Allan Jude To: Warner Losh Cc: =?UTF-8?Q?Dag-Erling_Sm=c3=b8rgrav?= , Nathan Whitehorn , src-committers , "svn-src-all@freebsd.org" , "svn-src-head@freebsd.org" Message-ID: <2c942f49-b90c-58eb-159f-4c4e62624bac@freebsd.org> Subject: Re: svn commit: r304142 - head/usr.sbin/bsdinstall/partedit References: <201608150930.u7F9UL1V069576@repo.freebsd.org> <861t1n6749.fsf@desk.des.no> <581c856c-826b-529e-c9c6-a397fb679708@freebsd.org> <86wpjf4eun.fsf@desk.des.no> <8cb3fa1a-50cb-e238-d006-b98a628d446d@freebsd.org> <86k2ff4cxs.fsf@desk.des.no> <86fuq24d8s.fsf@desk.des.no> <2f9fb04c-7ec4-be40-8fcb-0cf74bb56859@freebsd.org> In-Reply-To: --sxpS8TNfnWpFWDHAo0LUfWPwPtRHSisr1 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 2016-08-19 00:15, Warner Losh wrote: >> Which makes more sense: >> >> A) If stripesize =3D=3D 0, use some sane value like 4096 >=20 > I don't like this. >=20 >> B) Some other combination that uses the reported stripe size, unless i= t >> is 0, in which case it uses 4096 (or some other value controlled by a >> different new sysctl) >=20 > Don't like this so much. >=20 >> C) create kern.geom.min_stripe_size with a default of 512, but users c= an >> set 4096 if they use only 4k devices. (doesn't really solve the proble= m >> for the installer) >=20 > Default it to 4k, and allow users to set it to 512. If the drive > reports < this value > report this value instead. You'll need to make this a tunable. Then the= upper > layers wouldn't care. There's a small chance that some SD cards might b= e > reporting values that are too large. But I think it is confined to SD c= ards and > if I see too many more I'll do something specific in the SD driver. >=20 > Warner >=20 I think I mentioned this earlier in the thread,, but I do have a USB thumbstick that reports an 8mb stripe size. But I agree, I think a sysctl where you set the minimum (default 4096, but settable to 512), that is a tunable that can be overwritten in loader.conf What do we want for the logic as to what values it can be set to? Any power of 2 greater than or equal to 512? --=20 Allan Jude --sxpS8TNfnWpFWDHAo0LUfWPwPtRHSisr1-- --3vJ9Ixf52qiPb3wJHq6RTu3p9IpOuxluR Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (MingW32) iQIcBAEBAgAGBQJXtyuSAAoJEBmVNT4SmAt+iUEP/Raxai9knKnFs6fsTxwaTKDF T1PeDFetotiNtYqQMlvbOVyEissp1HfYfhMSF0ON8Q6Pk4PzLA/eBveIzMB9dMqo /7jc66atNRP7WX7WB1zK/bbAxk7TQ1HbRztk7bTnHlTJNJbxfbAo6yfUwwzJQOYQ Lm8IMD2gURcWek0qVs0hFfjhZ73wi5ZiHbcHOzQ9Z4b6M0g/buU7/DwLkLFxGsOy UKdQQmi32BRqiFoQRfeRq7urF48yQHzYcKy3b8b0DoltYZjR3KYJrIoZX9qF/qOt rTyTHJeYWwLeZQ6+SGa1YtTU4omb1JLRDAFa7EDpbML4Rl9xHWVjQ2OMXcN2GUHH QHV+Iug+FitVIL5y9KiGYTw0N4xspVc6qfkbiNOkBb/9Xr8SX1Fj8W1ssqffOhp0 4Ve2upnbDuEJvHz+31CwkFP2M4C0uf8/7B4dA+P1gk1TwJXoXbX1gaVFemsVx0lk k4PhMh2y9wRchKvpS4hDv7zmYxxdD3QEtgzwRa2G5oYKvJwB9EHsP+MGhlGFBUsr L7YGwGC0UwCBm81SUYt5un7fO/cE0LiXLc2AF1g6yysoJmQjc6F2h+/mApxy3Qjz T4PyY6vrdr8mJGL0jmmKhZ/Mc87cEjmUDqlAyF/9+29IBEodfQmrIP+9jNRk1wlY 7Cb9OEHWn2SbxaGuaSFM =rTGD -----END PGP SIGNATURE----- --3vJ9Ixf52qiPb3wJHq6RTu3p9IpOuxluR--