nvme: improve selftest option for nvmecontrol

[wanpengqian_gmail.com]

Summary

selftest now can specify -a to abort a selftest operation, -s to start a short selftest operation, -e to start a extended selftest operation.
also can speccify special namespace, such as 0xFFFFFFFF for all active namespaces.

Test Plan

execute selftest to check the result.

[imp]

These changes look good to me.

[wanpengqian_gmail.com]

fix a typo.

[rpokala]

sbin/nvmecontrol/nvmecontrol.8

532

In English, "An nsid" sounds better than "A nsid".

While the rule is usually "an :vowel::alpha:*" / "a :consonant::alpha:*", when the letter "n" is pronounced as a standalone letter (as in "N S I D" or "N SID"), the first *sound* is vowel-forward "en".

536

"specifies that the device self-test operation be run on the controller, independent of any namespaces."

541

"specifies that the device self-test operation be run on all active namespaces."

sbin/nvmecontrol/selftest.c

114

"Abort option"

152

"Execute short self-test"

154

"Execute extended self-test"

[wanpengqian_gmail.com]

Fix grammer/comments as reviewer suggested.

[wanpengqian_gmail.com]

sbin/nvmecontrol/nvmecontrol.8

532

I seems OK for me, I copy this sentence from identify section. do I need to modify that one?

-n      The namespace ⟨nsid⟩ to use instead of the namespace associated
        with the device.  A nsid of “0” is used to retrieve the
        IDENTIFY_CONTROLLER data associated with that drive.

[rpokala]

sbin/nvmecontrol/nvmecontrol.8

511

"an extended"

532

Yes, please update that one too.

[wanpengqian_gmail.com]

Fix grammer/comments as reviewer suggested.

[wanpengqian_gmail.com]

update others too. (An nsid)

[pauamma_gundo.com]

Manual page LGTM at this point; will rereview when it's no longer a WIP.