| To: | David Chinner <dgc@xxxxxxx> |
|---|---|
| Subject: | Re: ADD 977766 - mkfs.xfs man page needs the default settings updated. [REVIEW TAKE 3] |
| From: | Eric Sandeen <sandeen@xxxxxxxxxxx> |
| Date: | Mon, 10 Mar 2008 20:45:57 -0500 |
| Cc: | Niv Sardi <xaiki@xxxxxxxxxxxxx>, Niv Sardi <xaiki@xxxxxxx>, sgi.bugs.xfs@xxxxxxxxxxxxxxxxx, xfs-dev@xxxxxxx, xfs@xxxxxxxxxxx |
| In-reply-to: | <20080311011514.GE155407@sgi.com> |
| References: | <incident_977766@bugworks.engr.sgi.com> <20080222003514.8D88E2C3@toolshop.engr.sgi.com> <nccskzlzg9f.fsf_-_@sgi.com> <nccablbf8ra.fsf_-_@sgi.com> <20080310060751.GY155407@sgi.com> <416c461f0803092315m7ae6f55ek9b64058c3793aaa7@mail.gmail.com> <20080311011514.GE155407@sgi.com> |
| Sender: | xfs-bounce@xxxxxxxxxxx |
| User-agent: | Thunderbird 2.0.0.12 (Macintosh/20080213) |
David Chinner wrote: >> The change was motivated by Eric's comments on OSS that it is not >> clear why one should pick log v1 or v2, and I believe he is right. > > If you don't understand - use the default. In most cases v2 logs are the > right thing to use and no amount of text in the man page is going to be > able to explain the corner cases where you'd want to use v1 logs.... I think the only problem, Dave, is that there are maybe 2 people on the face of this earth who DO understand ;) (and I don't count myself among them). Just saying that v1 logs are still there for corner cases & specialized workloads which may perform better is probably fine, don't you think? That way those people who kill for such things can test both flavors. Without that, people won't know if v1 is broken, deprecated, dangerous, or what. If you say nothing at all about the differences, then don't even bother to document the log version option at all, IMHO. -Eric |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH, RFC] - remove mounpoint UUID code, Eric Sandeen |
|---|---|
| Next by Date: | Re: Final call for review of sb_bad_features2 in userspace, Josef 'Jeff' Sipek |
| Previous by Thread: | Re: ADD 977766 - mkfs.xfs man page needs the default settings updated. [REVIEW TAKE 3], David Chinner |
| Next by Thread: | Re: XFS internal error, Andreas Kotes |
| Indexes: | [Date] [Thread] [Top] [All Lists] |