Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 31 Jan 2012 13:58:44 -0700 (MST)
From:      Warren Block <wblock@wonkity.com>
To:        Daniel Gerzo <danger@freebsd.org>
Cc:        doc@freebsd.org
Subject:   Re: New Handbook Section for Review - graid3
Message-ID:  <alpine.BSF.2.00.1201311344400.97077@wonkity.com>
In-Reply-To: <20120131104953.GA55314@freefall.freebsd.org>
References:  <20120131104953.GA55314@freefall.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
  This message is in MIME format.  The first part should be readable text,
  while the remaining parts are likely unreadable without MIME-aware tools.

--SUOF0GtieIMvvwua
Content-Type: TEXT/PLAIN; CHARSET=US-ASCII; format=flowed
Content-ID: <alpine.BSF.2.00.1201311344402.97077@wonkity.com>
Content-Disposition: INLINE

On Tue, 31 Jan 2012, Daniel Gerzo wrote:

>  A new Handbook section covering graid3 is now available for review;
>  the patch is attached.
>
>  The text is based on PR 164228. A built version is available at
>  http://people.freebsd.org/~danger/geom-raid3.html.
>
>  Comments are welcome.

Patch with suggested changes attached.  This is a full diff, not a 
diff to your diff.

Some general things I'm not sure about, but will suggest:

Personally, I prefer

   add this setting to configfile
   <programlisting>'blahblah'</programlisting>

to

   Run this command
   <screen>'echo blahblah >> configfile'</screen>

I did not change those, though.


Various wording changes.  The sentence explaining total capacity could 
still be better, I think.  Paragraphs were intentionally not rewrapped, 
but should be.

The paragraph starting "To retain this configuration across reboots" was 
unclear.  I hope it still means what it did.

A couple of blank lines had whitespace.  Title capitalization was
changed a bit.  igor (http://www.wonkity.com/~wblock/igor/) can 
proofread those.
--SUOF0GtieIMvvwua
Content-Type: TEXT/PLAIN; charset=US-ASCII; name=geom-wb.diff
Content-Transfer-Encoding: BASE64
Content-ID: <alpine.BSF.2.00.1201311358440.97077@wonkity.com>
Content-Description: 
Content-Disposition: attachment; filename=geom-wb.diff

SW5kZXg6IGNoYXB0ZXIuc2dtbA0KPT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQ0K
UkNTIGZpbGU6IC9ob21lL2RjdnMvZG9jL2VuX1VTLklTTzg4NTktMS9ib29r
cy9oYW5kYm9vay9nZW9tL2NoYXB0ZXIuc2dtbCx2DQpyZXRyaWV2aW5nIHJl
dmlzaW9uIDEuNTENCmRpZmYgLXUgLXIxLjUxIGNoYXB0ZXIuc2dtbA0KLS0t
IGNoYXB0ZXIuc2dtbAkyMSBOb3YgMjAxMSAxODoxMToyNSAtMDAwMAkxLjUx
DQorKysgY2hhcHRlci5zZ21sCTMxIEphbiAyMDEyIDIwOjQ0OjE1IC0wMDAw
DQpAQCAtNDM2LDYgKzQzNiwxNjIgQEANCiAgICAgPC9zZWN0Mj4NCiAgIDwv
c2VjdDE+DQogDQorICA8c2VjdDEgaWQ9IkdFT00tcmFpZDMiPg0KKyAgICA8
c2VjdDFpbmZvPg0KKyAgICAgIDxhdXRob3Jncm91cD4NCisJPGF1dGhvcj4N
CisJICA8Zmlyc3RuYW1lPk1hcms8L2ZpcnN0bmFtZT4NCisJICA8c3VybmFt
ZT5HbGFkbWFuPC9zdXJuYW1lPg0KKwkgIDxjb250cmliPldyaXR0ZW4gYnkg
PC9jb250cmliPg0KKwk8L2F1dGhvcj4NCisJPGF1dGhvcj4NCisJICA8Zmly
c3RuYW1lPkRhbmllbDwvZmlyc3RuYW1lPg0KKwkgIDxzdXJuYW1lPkdlcnpv
PC9zdXJuYW1lPg0KKwk8L2F1dGhvcj4NCisgICAgICA8L2F1dGhvcmdyb3Vw
Pg0KKyAgICAgIDxhdXRob3Jncm91cD4NCisJPGF1dGhvcj4NCisJICA8Zmly
c3RuYW1lPlRvbTwvZmlyc3RuYW1lPg0KKwkgIDxzdXJuYW1lPlJob2Rlczwv
c3VybmFtZT4NCisJICA8Y29udHJpYj5CYXNlZCBvbiBkb2N1bWVudGF0aW9u
IGJ5IDwvY29udHJpYj4NCisJPC9hdXRob3I+DQorCTxhdXRob3I+DQorCSAg
PGZpcnN0bmFtZT5NdXJyYXk8L2ZpcnN0bmFtZT4NCisJICA8c3VybmFtZT5T
dG9rZWx5PC9zdXJuYW1lPg0KKwk8L2F1dGhvcj4NCisgICAgICA8L2F1dGhv
cmdyb3VwPg0KKyAgICA8L3NlY3QxaW5mbz4NCisNCisgICAgPGluZGV4dGVy
bT4NCisgICAgICA8cHJpbWFyeT5HRU9NPC9wcmltYXJ5Pg0KKyAgICA8L2lu
ZGV4dGVybT4NCisgICAgPGluZGV4dGVybT4NCisgICAgICA8cHJpbWFyeT5S
QUlEMzwvcHJpbWFyeT4NCisgICAgPC9pbmRleHRlcm0+DQorDQorICAgIDx0
aXRsZT48YWNyb255bT5SQUlEPC9hY3JvbnltPjMgLSBCeXRlLWxldmVsIFN0
cmlwaW5nIHdpdGggRGVkaWNhdGVkDQorICAgICAgUGFyaXR5PC90aXRsZT4N
CisNCisgICAgPHBhcmE+PGFjcm9ueW0+UkFJRDwvYWNyb255bT4zIGlzIGEg
bWV0aG9kIHVzZWQgdG8gY29tYmluZSBzZXZlcmFsDQorICAgICAgZGlzayBk
cml2ZXMgaW50byBhIHNpbmdsZSB2b2x1bWUgd2l0aCBhIGRlZGljYXRlZCBw
YXJpdHkNCisgICAgICBkaXNrLiAgSW4gYSA8YWNyb255bT5SQUlEPC9hY3Jv
bnltPjMgc3lzdGVtLCBkYXRhIGlzIHNwbGl0IHVwDQorICAgICAgaW50byBh
IG51bWJlciBvZiBieXRlcyB0aGF0IGdldCB3cml0dGVuIGFjcm9zcyBhbGwg
dGhlIGRyaXZlcyBpbg0KKyAgICAgIHRoZSBhcnJheSBleGNlcHQgZm9yIG9u
ZSBkaXNrIHdoaWNoIGFjdHMgYXMgYSBkZWRpY2F0ZWQgcGFyaXR5DQorICAg
ICAgZGlzay4gIFRoaXMgbWVhbnMgdGhhdCByZWFkaW5nIDEwMjRLQiBmcm9t
IGENCisgICAgICA8YWNyb255bT5SQUlEPC9hY3JvbnltPjMgaW1wbGVtZW50
YXRpb24gd2lsbCBhY2Nlc3MgYWxsIGRpc2tzIGluDQorICAgICAgdGhlIGFy
cmF5LiAgUGVyZm9ybWFuY2UgY2FuIGJlIGVuaGFuY2VkIGJ5IHVzaW5nIG11
bHRpcGxlDQorICAgICAgZGlzayBjb250cm9sbGVycy4gIFRoZSA8YWNyb255
bT5SQUlEPC9hY3JvbnltPjMgYXJyYXkgcHJvdmlkZXMgYQ0KKyAgICAgIGZh
dWx0IHRvbGVyYW5jZSBvZiAxIGRyaXZlLCB3aGlsZSBwcm92aWRpbmcgYSBj
YXBhY2l0eSBvZiAxIC0gMS9uIHRpbWVzIHRoZQ0KKyAgICAgIHRvdGFsIGNh
cGFjaXR5IG9mIGFsbCBkcml2ZXMgaW4gdGhlIGFycmF5LCB3aGVyZSBuIGlz
IHRoZQ0KKyAgICAgIG51bWJlciBvZiBoYXJkIGRyaXZlcyBpbiB0aGUgYXJy
YXkuICBTdWNoIGEgY29uZmlndXJhdGlvbiBpcw0KKyAgICAgIG1vc3RseSBz
dWl0YWJsZSBmb3Igc3RvcmluZyBkYXRhIG9mIGxhcmdlciBzaXplcywgZS5n
Lg0KKyAgICAgIG11bHRpbWVkaWEgZmlsZXMuPC9wYXJhPg0KKw0KKyAgICA8
cGFyYT5BdCBsZWFzdCAzIHBoeXNpY2FsIGhhcmQgZHJpdmVzIGFyZSByZXF1
aXJlZCB0byBidWlsZCBhDQorICAgICAgPGFjcm9ueW0+UkFJRDwvYWNyb255
bT4zIGFycmF5LiAgRWFjaCBkaXNrIG11c3QgYmUgb2YgdGhlIHNhbWUNCisg
ICAgICBzaXplLCBzaW5jZSBJL08gcmVxdWVzdHMgYXJlIGludGVybGVhdmVk
IHRvIHJlYWQgb3Igd3JpdGUgdG8NCisgICAgICBtdWx0aXBsZSBkaXNrcyBp
biBwYXJhbGxlbC4gIEFsc28gZHVlIHRvIHRoZSBuYXR1cmUgb2YNCisgICAg
ICA8YWNyb255bT5SQUlEPC9hY3JvbnltPjMsIHRoZSBudW1iZXIgb2YgZHJp
dmVzIG11c3QgYmUNCisgICAgICBlcXVhbCB0byAzLCA1LCA5LCAxNywgZXRj
LiAoMl5uICsgMSkuPC9wYXJhPg0KKw0KKyAgICA8c2VjdDI+DQorICAgICAg
PHRpdGxlPkNyZWF0aW5nIGEgRGVkaWNhdGVkIDxhY3JvbnltPlJBSUQ8L2Fj
cm9ueW0+MyBBcnJheTwvdGl0bGU+DQorDQorICAgICAgPHBhcmE+SW4gJm9z
Oywgc3VwcG9ydCBmb3IgPGFjcm9ueW0+UkFJRDwvYWNyb255bT4zIGlzDQor
CWltcGxlbWVudGVkIGJ5IHRoZSAmbWFuLmdyYWlkMy44OyA8YWNyb255bT5H
RU9NPC9hY3JvbnltPg0KKwljbGFzcy4gIENyZWF0aW5nIGEgZGVkaWNhdGVk
DQorCTxhY3JvbnltPlJBSUQ8L2Fjcm9ueW0+MyBhcnJheSBvbiAmb3M7IHJl
cXVpcmVzIHRoZSBmb2xsb3dpbmcgc3RlcHMuPC9wYXJhPg0KKw0KKyAgICAg
IDxub3RlPg0KKwk8cGFyYT5XaGlsZSBpdCBpcyB0aGVvcmV0aWNhbGx5IHBv
c3NpYmxlIHRvIGJvb3QgZnJvbSBhDQorCSAgPGFjcm9ueW0+UkFJRDwvYWNy
b255bT4zIGFycmF5IG9uICZvczssIHN1Y2ggYSBjb25maWd1cmF0aW9uDQor
CSAgaXMgbm90IGNvbW1vbiBhbmQgaXMgbm90IGFkdmlzZWQuICBBcyBzdWNo
LCB0aGlzIHNlY3Rpb24NCisJICBkb2VzIG5vdCBkZXNjcmliZSBob3cgdG8g
YWNjb21wbGlzaCB0aGF0DQorCSAgY29uZmlndXJhdGlvbi48L3BhcmE+DQor
ICAgICAgPC9ub3RlPg0KKw0KKyAgICAgIDxwcm9jZWR1cmU+DQorCTxzdGVw
Pg0KKwkgIDxwYXJhPkZpcnN0LCBsb2FkIHRoZSA8ZmlsZW5hbWU+Z2VvbV9y
YWlkMy5rbzwvZmlsZW5hbWU+IGtlcm5lbCBtb2R1bGUuPC9wYXJhPg0KKw0K
KwkgIDxzY3JlZW4+JnByb21wdC5yb290OyA8dXNlcmlucHV0PmdyYWlkMyBs
b2FkPC91c2VyaW5wdXQ+PC9zY3JlZW4+DQorDQorCSAgPHBhcmE+QWx0ZXJu
YXRpdmVseSwgaXQgaXMgcG9zc2libGUgdG8gbWFudWFsbHkgbG9hZCB0aGUN
CisJICAgIDxmaWxlbmFtZT5nZW9tX3JhaWQzLmtvPC9maWxlbmFtZT4gbW9k
dWxlOjwvcGFyYT4NCisNCisJICA8c2NyZWVuPiZwcm9tcHQucm9vdDsgPHVz
ZXJpbnB1dD5rbGRsb2FkIGdlb21fcmFpZDMua288L3VzZXJpbnB1dD48L3Nj
cmVlbj4NCisJPC9zdGVwPg0KKw0KKwk8c3RlcD4NCisJICA8cGFyYT5DcmVh
dGUgb3IgZW5zdXJlIHRoYXQgYSBzdWl0YWJsZSBtb3VudCBwb2ludA0KKwkg
ICAgZXhpc3RzOjwvcGFyYT4NCisNCisJICA8c2NyZWVuPiZwcm9tcHQucm9v
dDsgPHVzZXJpbnB1dD5ta2RpciA8cmVwbGFjZWFibGU+L211bHRpbWVkaWEv
PC9yZXBsYWNlYWJsZT48L3VzZXJpbnB1dD48L3NjcmVlbj4NCisJPC9zdGVw
Pg0KKw0KKwk8c3RlcD4NCisJICA8cGFyYT5EZXRlcm1pbmUgdGhlIGRldmlj
ZSBuYW1lcyBmb3IgdGhlIGRpc2tzIHdoaWNoIHdpbGwgYmUNCisJICAgIGFk
ZGVkIHRvIHRoZSBhcnJheSwgYW5kIGNyZWF0ZSB0aGUgbmV3DQorCSAgICA8
YWNyb255bT5SQUlEPC9hY3JvbnltPjMgZGV2aWNlLiAgVGhlIGZpbmFsIGRl
dmljZSBsaXN0ZWQNCisJICAgIHdpbGwgYWN0IGFzIHRoZSBkZWRpY2F0ZWQg
cGFyaXR5IGRpc2suICBUaGlzDQorCSAgICBleGFtcGxlIHVzZXMgdGhyZWUg
dW5wYXJ0aXRpb25lZA0KKwkgICAgPGFjcm9ueW0+QVRBPC9hY3JvbnltPiBk
cml2ZXM6DQorCSAgICA8ZGV2aWNlbmFtZT48cmVwbGFjZWFibGU+YWRhMTwv
cmVwbGFjZWFibGU+PC9kZXZpY2VuYW1lPg0KKwkgICAgYW5kIDxkZXZpY2Vu
YW1lPjxyZXBsYWNlYWJsZT5hZGEyPC9yZXBsYWNlYWJsZT48L2RldmljZW5h
bWU+DQorCSAgICBmb3IgZGF0YSwgYW5kDQorCSAgICA8ZGV2aWNlbmFtZT48
cmVwbGFjZWFibGU+YWRhMzwvcmVwbGFjZWFibGU+PC9kZXZpY2VuYW1lPg0K
KwkgICAgZm9yIHBhcml0eS48L3BhcmE+DQorDQorCSAgPHNjcmVlbj4mcHJv
bXB0LnJvb3Q7IDx1c2VyaW5wdXQ+Z3JhaWQzIGxhYmVsIC12IGdyMCAvZGV2
L2FkYTEgL2Rldi9hZGEyIC9kZXYvYWRhMzwvdXNlcmlucHV0Pg0KK01ldGFk
YXRhIHZhbHVlIHN0b3JlZCBvbiAvZGV2L2FkYTEuDQorTWV0YWRhdGEgdmFs
dWUgc3RvcmVkIG9uIC9kZXYvYWRhMi4NCitNZXRhZGF0YSB2YWx1ZSBzdG9y
ZWQgb24gL2Rldi9hZGEzLg0KK0RvbmUuPC9zY3JlZW4+DQorCTwvc3RlcD4N
CisNCisJPHN0ZXA+DQorCSAgPHBhcmE+UGFydGl0aW9uIHRoZSBuZXdseSBj
cmVhdGVkDQorCSAgICA8ZGV2aWNlbmFtZT5ncjA8L2RldmljZW5hbWU+IGRl
dmljZSBhbmQgcHV0IGEgVUZTIGZpbGUNCisJICAgIHN5c3RlbSBvbiBpdDo8
L3BhcmE+DQorDQorCSAgPHNjcmVlbj4mcHJvbXB0LnJvb3Q7IDx1c2VyaW5w
dXQ+Z3BhcnQgY3JlYXRlIC1zIEdQVCAvZGV2L3JhaWQzL2dyMDwvdXNlcmlu
cHV0Pg0KKyZwcm9tcHQucm9vdDsgPHVzZXJpbnB1dD5ncGFydCBhZGQgLXQg
ZnJlZWJzZC11ZnMgL2Rldi9yYWlkMy9ncjA8L3VzZXJpbnB1dD4NCismcHJv
bXB0LnJvb3Q7IDx1c2VyaW5wdXQ+bmV3ZnMgLWogL2Rldi9yYWlkMy9ncjBw
MTwvdXNlcmlucHV0Pjwvc2NyZWVuPg0KKw0KKwkgIDxwYXJhPk1hbnkgbnVt
YmVycyB3aWxsIGdsaWRlIGFjcm9zcyB0aGUgc2NyZWVuLCBhbmQgYWZ0ZXIg
YQ0KKwkgICAgZmV3IHNlY29uZHMsIHRoZSBwcm9jZXNzIHdpbGwgYmUgY29t
cGxldGUuICBUaGUgdm9sdW1lIGhhcw0KKwkgICAgYmVlbiBjcmVhdGVkIGFu
ZCBpcyByZWFkeSB0byBiZSBtb3VudGVkLjwvcGFyYT4NCisJPC9zdGVwPg0K
Kw0KKwk8c3RlcD4NCisJICA8cGFyYT5UaGUgbGFzdCBzdGVwIGlzIHRvIG1v
dW50IHRoZSBmaWxlIHN5c3RlbTo8L3BhcmE+DQorDQorCSAgPHNjcmVlbj4m
cHJvbXB0LnJvb3Q7IDx1c2VyaW5wdXQ+bW91bnQgL2Rldi9yYWlkMy9ncjBw
MSAvbXVsdGltZWRpYS88L3VzZXJpbnB1dD48L3NjcmVlbj4NCisNCisJICA8
cGFyYT5UaGUgPGFjcm9ueW0+UkFJRDwvYWNyb255bT4zIGFycmF5IGlzIG5v
dyByZWFkeSB0bw0KKwkgICAgdXNlLjwvcGFyYT4NCisJPC9zdGVwPg0KKyAg
ICAgIDwvcHJvY2VkdXJlPg0KKw0KKyAgICAgIDxub3RlPg0KKwk8cGFyYT5U
aGUNCisJICA8ZmlsZW5hbWU+Z2VvbV9yYWlkMy5rbzwvZmlsZW5hbWU+IG1v
ZHVsZSBtdXN0IGJlIGxvYWRlZA0KKwkgIGJlZm9yZSB0aGUgYXJyYXkgY2Fu
IGJlIG1vdW50ZWQuICBUbw0KKwkgIGF1dG9tYXRpY2FsbHkgbG9hZCB0aGUg
a2VybmVsIG1vZHVsZSBkdXJpbmcgc3lzdGVtIGluaXRpYWxpemF0aW9uLA0K
KwkgIGludm9rZSB0aGUgZm9sbG93aW5nIGNvbW1hbmQ6PC9wYXJhPg0KKw0K
Kwk8c2NyZWVuPiZwcm9tcHQucm9vdDsgPHVzZXJpbnB1dD5lY2hvICdnZW9t
X3JhaWQzX2xvYWQ9IllFUyInID4+IC9ib290L2xvYWRlci5jb25mPC91c2Vy
aW5wdXQ+PC9zY3JlZW4+DQorDQorCTxwYXJhPkZvcg0KKwkgIGF1dG9tYXRp
YyBtb3VudGluZyBvZiB0aGUgYXJyYXkncyBmaWxlIHN5c3RlbSBkdXJpbmcg
dGhlIGJvb3QNCisJICBwcm9jZXNzLCBhZGQgdGhlIHZvbHVtZSBpbmZvcm1h
dGlvbiBpbg0KKwkgIDxmaWxlbmFtZT4vZXRjL2ZzdGFiPC9maWxlbmFtZT4g
ZmlsZTo8L3BhcmE+DQorDQorCTxzY3JlZW4+JnByb21wdC5yb290OyA8dXNl
cmlucHV0PmVjaG8gIi9kZXYvcmFpZDMvZ3IwcDEgL211bHRpbWVkaWEgdWZz
IHJ3IDIgMiIgPj4gL2V0Yy9mc3RhYjwvdXNlcmlucHV0Pjwvc2NyZWVuPg0K
KyAgICAgIDwvbm90ZT4NCisgICAgPC9zZWN0Mj4NCisgIDwvc2VjdDE+DQor
DQogICA8c2VjdDEgaWQ9Imdlb20tZ2dhdGUiPg0KICAgICA8dGl0bGU+R0VP
TSBHYXRlIE5ldHdvcmsgRGV2aWNlczwvdGl0bGU+DQogDQo=

--SUOF0GtieIMvvwua--



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