Re: [PATCH 00/26] Btrfs: Add device replace code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 8 Nov 2012 13:50:19 +0100, Goffredo Baroncelli wrote:
> great work. However I have a suggestion: what about putting all the
> command under 'device' sub commands: something like:
> 
> - btrfs device replace <old> <new> </path>
> 
> - btrfs device status </path>
> 
> Where "btrfs device status" would show only the status of the
> "replacing" operation; but in the future it could show also the status
> of the "delete" command (which it is the only other "device
> sub-command" which needs time to complete).
> 
> Of course I am not asking to complete the "btrfs device status" part
> for the "btrfs device delete" command. This could be implemented in a
> second time.
> 
> I think that so "replace" would be the natral extension to the "add"
> and "delete" subcommands.

"btrfs device replace <old> <new> <path>"
was also my first idea. It used to be like this initially.

"btrfs device replace cancel <path>"
was the point when I gave up putting it below the "device" commands. IMO
that's just too long, too much to type.

Now it has the same look and feel as the "scrub" commands ("scrub
start", "scrub status" and "scrub cancel").

--
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


[Index of Archives]     [Linux Filesystem Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux