On 12/07/2017 10:52 PM, Austin S. Hemmelgarn wrote:
On 2017-12-07 09:36, Anand Jain wrote:
Add ability to deregister a or all devices. I have named this sub cmd
as deregister, but I am open to your suggestions.
Being a bit picky here, but from the perspective of a native speaker of
American English, I would say that 'deregister' sounds rather synthetic
and somewhat harsh and alien.
Given that, as odd as it sounds, I think 'ignore' might be a more user
friendly name for the sub-command. It accurately describes what the
command is doing (telling the kernel to ignore the device), and it's a
lot less alien sounding than 'deregister'.
If you're set on having it be based on the word 'register', I would
suggest changing it to 'unregister', as I think that sounds more natural
than 'deregister'.
Additionally, if you do continue with 'deregister' or go with
'unregister' as the name though, I would suggest adding 'register' as a
synonym for the 'scan' sub-command to keep things reasonably symmetrical.
A look up on unregister lead me to use deregister as more appropriate.
Anyway I won't bother much, I will be go be suggestions, and how
about unscan, since scan is already there. OR how about purge.
Thanks, Anand.
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
