From: "xavier.gnata@gmail.com" <xavier.gnata@gmail.com>
To: Anand Jain <Anand.Jain@oracle.com>,
Johannes Hirte <johannes.hirte@datenkhaos.de>
Cc: linux-btrfs@vger.kernel.org
Subject: Re: "Btrfs: device_list_add() should not update list when mounted" breaks subvol mount
Date: Sun, 14 Sep 2014 13:12:21 +0200 [thread overview]
Message-ID: <54157815.4060006@gmail.com> (raw)
In-Reply-To: <5413D7E5.6080905@oracle.com>
The way an encrypted home is mounted is based on links as described here:
https://help.ubuntu.com/community/EncryptedPrivateDirectory
It used to be ok before 3.17-rc3 and df -h shows:
/dev/sda5 363G 128G 217G 38% /
/home/gnata/.Private 363G 128G 217G 38% /home/gnata
It is suppose to work with a kernel >3.17-rc3 when "home" btrfs is a
subvolume?
Does it make sense to test "[PATCH] btrfs-progs: canonicalize pathnames
for device commands" or it is premature?
Xavier
>
> Hi Xavier,
>
> Thanks for the report.
>
> I got this reproduced: its a very corner case, it depends on the
> device path given in the subsequent subvol mounts, the fix appear
> to be outside of this patch at this moment and I am digging to know
> if we need to normalize the device path before using it in the btrfs
> kernel, just like btrfs-progs did recently.
>
>
> reproducer:
> ls -l /root/dev/sde-link
> /root/dev/sde-link -> /dev/sde
>
> mount -o device=/root/dev/sde-link /dev/sdd /btrfs1
> btrfs fi show
> Label: none uuid: 943bf422-998c-4640-9d7f-d49f17b782ce
> Total devices 2 FS bytes used 272.00KiB
> devid 1 size 1.52GiB used 339.50MiB path /dev/sdd
> devid 2 size 1.52GiB used 319.50MiB path /root/dev/sde-link
>
> mount -o subvol=sv1,device=/dev/sde /dev/sdd /btrfs <-shouldn't fail.
> mount: /dev/sdd already mounted or /btrfs busy
> mount: according to mtab, /dev/sdd is mounted on /btrfs1
>
> mount -o device=/root/dev/sde-link /dev/sdd /btrfs
> echo $?
> 0
>
>
> Xavier, Johannes,
>
> The quickest workaround for you will be to try to match
> the device path as in the btrfs fi show -m </mnt> output to
> your probably fstab/mnttab entry.
>
>
> Anand
>
> On 09/13/2014 04:43 AM, xavier.gnata@gmail.com wrote:
>> Hi,
>>
>> On standard ubuntu 14.04 a with an encrypted (cryptsetup) /home as brtfs
>> subvolume we have the following results:
>> 3.17-rc2 : Ok.
>> 3.17-rc3 and 3.17-rc4 : /home fails to mount on boot. If one try mount
>> -a then the system tells that the partition is already mounted according
>> to matab.
>
>
>>
>> On a 3.17-rc4, btrfs fi sh returns nothing special:
>> "
>> Label: none uuid: f4f554bb-57d9-4647-ab14-ea978c9e7e9f
>> Total devices 1 FS bytes used 131.41GiB
>> devid 1 size 173.31GiB used 134.03GiB path /dev/sda5
>>
>> Btrfs v3.12
>> "
>>
>> I'm not sure if it has something to do with cryptsetup...
>>
>>
>> Xavier
>>
>>> Hi Johannes,
>>>
>>>> I've two more systems with kernel version >3.17-rc3
>>>> running and no problem like this.
>>>
>>> Does this 3.17-rc3 also has the same type of subvol config
>>> and it mount operation/sequence as you mentioned ?
>>>
>>> > - one hdd with btrfs
>>> > - default subvolume (rootfs) is different from subovlid=0
>>> > - at boot, several subvols are mounted at /home/$DIR
>>>
>>> I ran a few tests, on our mainline
>>> ----
>>> mount -o subvol=sv1 /dev/sdh1 /btrfs
>>> mount /dev/sdh1 /btrfs
>>> mount: /dev/sdh1 already mounted or /btrfs busy [*]
>>> mount /dev/sdh1 /btrfs1
>>> echo $?
>>> 0
>>> -----
>>>
>>> [*] hope this isn't the problem you are mentioning.
>>>
>>> Thanks, Anand
prev parent reply other threads:[~2014-09-14 11:12 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-10 22:22 "Btrfs: device_list_add() should not update list when mounted" breaks subvol mount Johannes Hirte
2014-09-10 23:26 ` Anand Jain
2014-09-11 2:37 ` Anand Jain
2014-09-12 20:43 ` xavier.gnata
2014-09-13 5:36 ` Anand Jain
2014-09-13 17:55 ` Johannes Hirte
2014-09-13 21:23 ` Johannes Hirte
2014-09-14 0:45 ` Duncan
2014-09-15 17:13 ` Johannes Hirte
2014-09-15 12:32 ` Anand Jain
2014-09-15 17:14 ` Johannes Hirte
2014-09-15 17:39 ` Anand Jain
2014-09-15 22:17 ` Johannes Hirte
2014-09-16 21:55 ` Anand Jain
2014-09-17 8:49 ` Anand Jain
2014-09-14 11:12 ` xavier.gnata [this message]
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=54157815.4060006@gmail.com \
--to=xavier.gnata@gmail.com \
--cc=Anand.Jain@oracle.com \
--cc=johannes.hirte@datenkhaos.de \
--cc=linux-btrfs@vger.kernel.org \
/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).