linux-btrfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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


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