From owner-svn-src-all@FreeBSD.ORG Wed Nov 13 10:44:25 2013 Return-Path: Delivered-To: svn-src-all@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 ECCFCAFE; Wed, 13 Nov 2013 10:44:24 +0000 (UTC) Received: from mail105.syd.optusnet.com.au (mail105.syd.optusnet.com.au [211.29.132.249]) by mx1.freebsd.org (Postfix) with ESMTP id 5A13B2607; Wed, 13 Nov 2013 10:44:24 +0000 (UTC) Received: from c122-106-156-23.carlnfd1.nsw.optusnet.com.au (c122-106-156-23.carlnfd1.nsw.optusnet.com.au [122.106.156.23]) by mail105.syd.optusnet.com.au (Postfix) with ESMTPS id F083E104224F; Wed, 13 Nov 2013 21:43:32 +1100 (EST) Date: Wed, 13 Nov 2013 21:43:31 +1100 (EST) From: Bruce Evans X-X-Sender: bde@besplex.bde.org To: John Baldwin Subject: Re: svn commit: r258053 - head/usr.sbin/mfiutil In-Reply-To: <201311121322.46733.jhb@freebsd.org> Message-ID: <20131113211029.O1034@besplex.bde.org> References: <201311121710.rACHAuCI027408@svn.freebsd.org> <201311121322.46733.jhb@freebsd.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed X-Optus-CM-Score: 0 X-Optus-CM-Analysis: v=2.1 cv=bpB1Wiqi c=1 sm=1 tr=0 a=ebeQFi2P/qHVC0Yw9JDJ4g==:117 a=PO7r1zJSAAAA:8 a=iPal_rS6wjAA:10 a=kj9zAlcOel0A:10 a=JzwRw_2MAAAA:8 a=JUldux9VRoEA:10 a=6_aYeVEia8bRkXMAfHIA:9 a=CjuIK1q_8ugA:10 Cc: svn-src-head@FreeBSD.org, Sean Bruno , src-committers@FreeBSD.org, svn-src-all@FreeBSD.org X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.16 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: Wed, 13 Nov 2013 10:44:25 -0000 On Tue, 12 Nov 2013, John Baldwin wrote: > On Tuesday, November 12, 2013 12:10:56 pm Sean Bruno wrote: >> Author: sbruno >> Log: >> Noted that the stripe_size argument was not being displayed in the usage >> message > > Does this need some wrapping to fit in 80 cols now? Of course. The output style is horrible enough without inconsostencies in it. It doesn't have to be as bad as the source. Previous output: % mptutil: illegal option -- ? % usage: mptutil [-u unit] ... Oops, wrong command. It has slightly different bugs. E.g., the man page says "-s stripe_size" while the usage message says "-s stripe". There are many more. Perhaps this doesn't need further description, but verbose descriptions are part of the non-KNF style here. See the man page for splitting this after the addition. But this and the man page have other bugs: - v and q are sorted backwards here. The man page sorts them forwards, but doesn't group them % mfiutil: illegal option -- ? % usage: mfiutil [-de] [-u unit] ... % % Commands include: % version % show adapter - display controller information % show battery - display battery information % show config - display RAID configuration % show drives - list physical drives % show events - display event log % show firmware - list firmware images % show foreign - display detected foreign volumes % show logstate - display event log sequence numbers % show volumes - list logical volumes % show patrol - display patrol read status % show progress - display status of active operations % fail - fail a physical drive % good - set a failed/SYSPD drive as UNCONFIGURED % rebuild - mark failed drive ready for rebuild % syspd - set drive into use as SYSPD JBOD % drive progress - display status of active operations % drive clear - clear a drive with all 0x00 mptutil is more careful about formatting. It starts a new line for the description if necessary to align it. % start rebuild % abort rebuild Some commands are missing descriptions. % locate - toggle drive LED % cache [command [setting]] % name % volume progress - display status of active operations % clear - clear volume configuration % create [-v] [,[,...]] [[,[,...]] This command needs a description more than the simple ones. See the man page for (not very good) formatting of the line splitting. It aligns with the option before "create", but should align with the option after "create". The man page doesn't use angle brackets around parameters like . The syntax for the 1 or 2 drive lists is complicated and perhaps not worth giving in detaul here. % delete % add [volume] - add a hot spare % remove - remove a hot spare % patrol [interval [start]] % start patrol - start a patrol read % stop patrol - stop a patrol read The order of at least the patrol-related commands is different to the man page. % foreign scan - scan for foreign configurations % foreign clear [volume] - clear foreign configurations (default all) % foreign diag [volume] - diagnostic display foreign configurations (default all) This was already too long. % foreign preview [volume] - preview foreign configurations (default all) % foreign import [volume] - import foreign configurations (default all) % flash % start learn - start a BBU relearn % bbu - set BBU properties Bruce