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

On 11/08/2012 06:31 PM, Stefan Behrens wrote:

On Thu, 8 Nov 2012 13:50:19 +0100, Goffredo Baroncelli wrote:

[...]

quoted

I think that so "replace" would be the natural 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").

Yes, but scrub was a a new command. Instead I see "replace" as an
extension of "btrfs device add/del" (from an user interface POV).

If someone would extend the "btrfs device delete" command to support
status/pause/resume, how could do it ?
May be we need a new series of command which handle the "background
process" (like btrfs replace, btrfs device delete, btrfs subvolume
delete....) to status/stop/suspend/resume these processes ?

I am doing a bit of brain-storming...

-- 
gpg @keyserver.linux.it: Goffredo Baroncelli (kreijackATinwind.it>
Key fingerprint BBF5 1610 0B64 DAC6 5F7D  17B2 0EDA 9B37 8B82 E0B5