diff mbox

[ndctl] ndctl, documentation: fix the manpage of inject-smart

Message ID 20180712054000.18519-1-qi.fuli@jp.fujitsu.com (mailing list archive)
State New, archived
Headers show

Commit Message

QI Fuli July 12, 2018, 5:40 a.m. UTC
The ndctl inject-smart doesn't have a [-H | --health] option.
This patch replaces [-H | --health] in the manpage of inject-smart
with [-f | --fatal].

Signed-off-by: QI Fuli <qi.fuli@jp.fujitsu.com>
---
 Documentation/ndctl/ndctl-inject-smart.txt | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

Comments

Verma, Vishal L July 12, 2018, 4:43 p.m. UTC | #1
On Thu, 2018-07-12 at 14:40 +0900, QI Fuli wrote:
> The ndctl inject-smart doesn't have a [-H | --health] option.
> This patch replaces [-H | --health] in the manpage of inject-smart
> with [-f | --fatal].
> 
> Signed-off-by: QI Fuli <qi.fuli@jp.fujitsu.com>
> ---
>  Documentation/ndctl/ndctl-inject-smart.txt | 12 +++++++-----
>  1 file changed, 7 insertions(+), 5 deletions(-)
> 
> diff --git a/Documentation/ndctl/ndctl-inject-smart.txt
> b/Documentation/ndctl/ndctl-inject-smart.txt
> index 0dc6481..06710ec 100644
> --- a/Documentation/ndctl/ndctl-inject-smart.txt
> +++ b/Documentation/ndctl/ndctl-inject-smart.txt
> @@ -50,6 +50,7 @@ OPTIONS
>  --media-temperature-threshold=::
>  	Set <value> for the smart media temperature threshold.
>  
> +-x::
>  --media-temperature-alarm=::
>  	Enable or disable the smart media temperature alarm. Options
> are
>  	'on' or 'off'.
> @@ -62,6 +63,7 @@ OPTIONS
>  --ctrl-temperature-threshold=::
>  	Set <value> for the smart controller temperature threshold.
>  
> +-y::
>  --ctrl-temperature-alarm=::
>  	Enable or disable the smart controller temperature alarm.
> Options are
>  	'on' or 'off'.
> @@ -74,16 +76,16 @@ OPTIONS
>  --spares-threshold=::
>  	Set <value> for the smart spares threshold.
>  
> +-z::
>  --spares-alarm=::
>  	Enable or disable the smart spares alarm. Options are 'on'
> or 'off'.

Hi Qi,

For the x, y, and z short options, I left them undocumented since they
don't exactly match the actual field being set.

>  
> --H::
> ---health=::
> -	Smart attribute for health status. Provide either 'fatal' or
> 'nominal'
> -	to set the state of the attribute.

This was a leftover from a previous iteration, and does need to be
fixed. However, I am reworking this man page along with the smart
injection utility to also provide a way to 'uninject' or disable
injected values, and go back to reporting actual current status. I have
this fix in that series.

> +-f::
> +--fatal::
> +	Inject fatal for the health state attribute.
>  
>  -U::
> ---unsafe-shutdown=::
> +--unsafe-shutdown::
>  	Set the flag to spoof an unsafe shutdown on the next power
> down.
>  
>  -v::
diff mbox

Patch

diff --git a/Documentation/ndctl/ndctl-inject-smart.txt b/Documentation/ndctl/ndctl-inject-smart.txt
index 0dc6481..06710ec 100644
--- a/Documentation/ndctl/ndctl-inject-smart.txt
+++ b/Documentation/ndctl/ndctl-inject-smart.txt
@@ -50,6 +50,7 @@  OPTIONS
 --media-temperature-threshold=::
 	Set <value> for the smart media temperature threshold.
 
+-x::
 --media-temperature-alarm=::
 	Enable or disable the smart media temperature alarm. Options are
 	'on' or 'off'.
@@ -62,6 +63,7 @@  OPTIONS
 --ctrl-temperature-threshold=::
 	Set <value> for the smart controller temperature threshold.
 
+-y::
 --ctrl-temperature-alarm=::
 	Enable or disable the smart controller temperature alarm. Options are
 	'on' or 'off'.
@@ -74,16 +76,16 @@  OPTIONS
 --spares-threshold=::
 	Set <value> for the smart spares threshold.
 
+-z::
 --spares-alarm=::
 	Enable or disable the smart spares alarm. Options are 'on' or 'off'.
 
--H::
---health=::
-	Smart attribute for health status. Provide either 'fatal' or 'nominal'
-	to set the state of the attribute.
+-f::
+--fatal::
+	Inject fatal for the health state attribute.
 
 -U::
---unsafe-shutdown=::
+--unsafe-shutdown::
 	Set the flag to spoof an unsafe shutdown on the next power down.
 
 -v::