From: Anand Jain <anand.jain@oracle.com>
To: "Misono, Tomohiro" <misono.tomohiro@jp.fujitsu.com>,
linux-btrfs@vger.kernel.org
Cc: Duncan <1i5t5.duncan@cox.net>
Subject: Re: [PATCH v2 3/3] btrfs-progs: device: add remove missing-all
Date: Mon, 16 Oct 2017 14:46:37 +0800 [thread overview]
Message-ID: <bf44fc1a-4a33-5dc7-1d11-5c987643552d@oracle.com> (raw)
In-Reply-To: <649452a3-908d-670d-8bfd-b0e248b32b32@jp.fujitsu.com>
On 10/16/2017 12:35 PM, Misono, Tomohiro wrote:
>
>
> On 2017/10/16 12:30, Anand Jain wrote:
>>
>>
>> On 10/13/2017 01:27 PM, Duncan wrote:
>>> Misono, Tomohiro posted on Wed, 11 Oct 2017 11:18:50 +0900 as excerpted:
>>>
>>>> Add 'btrfs remove missing-all' to remove all the missing devices at once
>>>> for improving usability.
>>>>
>>>> Example:
>>>> sudo mkfs.btrfs -f -d raid1 /dev/sdb1 /dev/sdb2 /dev/sdb3 /dev/sdb4
>>>> sudo wipefs -a /dev/sdb1 /dev/sdb3
>>>> sudo mount -o degraded /dev/sdb2 /mnt <--
>>
>>
>> I agree with Duncan here. This step itself will fail even with RO
>> option.
Correction. RO mount is possible based on which disk you choose to mount.
Thanks - Anand
>> Do you have any patch that is not in the ML which will
>> make this step a success in the first place ?
>>
>> Thanks, Anand
>>
>
> commit 21634a19f646 ("btrfs: Introduce a function to check if all chunks a OK
> for degraded rw mount") allow this from 4.14 (I checked on 4.14-rc4).
> But I will withdraw this patch as Duncan suggests.
>
> Thanks,
> Tomohiro
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
next prev parent reply other threads:[~2017-10-16 6:46 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-11 2:14 [PATCH v2 0/3] btrfs-progs: device: update btrfs device remove missing Misono, Tomohiro
2017-10-11 2:16 ` [PATCH v2 1/3] btrfs-progs: device: add description of alias to help message Misono, Tomohiro
2017-10-11 2:17 ` [PATCH v2 2/3] btrfs-progs: doc: add description of missing and example of device remove Misono, Tomohiro
2017-10-11 2:18 ` [PATCH v2 3/3] btrfs-progs: device: add remove missing-all Misono, Tomohiro
2017-10-13 5:27 ` Duncan
2017-10-16 3:30 ` Anand Jain
2017-10-16 4:35 ` Misono, Tomohiro
2017-10-16 6:46 ` Anand Jain [this message]
2017-10-16 14:16 ` Duncan
2017-10-16 4:32 ` Misono, Tomohiro
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=bf44fc1a-4a33-5dc7-1d11-5c987643552d@oracle.com \
--to=anand.jain@oracle.com \
--cc=1i5t5.duncan@cox.net \
--cc=linux-btrfs@vger.kernel.org \
--cc=misono.tomohiro@jp.fujitsu.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).