Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757120AbcCURWK (ORCPT ); Mon, 21 Mar 2016 13:22:10 -0400 Received: from mail-ig0-f182.google.com ([209.85.213.182]:37466 "EHLO mail-ig0-f182.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756270AbcCURWI (ORCPT ); Mon, 21 Mar 2016 13:22:08 -0400 MIME-Version: 1.0 X-Originating-IP: [2601:282:702:b960:29be:1b83:7aed:5501] In-Reply-To: <20160321152144.GL8544@smitten> References: <20160321152144.GL8544@smitten> Date: Mon, 21 Mar 2016 11:22:06 -0600 X-Google-Sender-Auth: 9GzNXQmfFsgMhLxBq27dNmsVDA0 Message-ID: Subject: Re: confusing mountinfo output when bind-mounting files From: Chris Murphy To: Tycho Andersen Cc: Btrfs BTRFS , linux-kernel Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1464 Lines: 34 On Mon, Mar 21, 2016 at 9:21 AM, Tycho Andersen wrote: > Hi all, > > I'm seeing some strange behavior when bind mounting files from a btrfs > subvolume. Consider the output below: > > root@criu2:/tmp# mount -o loop /tmp/tester.btrfs /tmp/dir1 > root@criu2:/tmp# touch dir1/file > root@criu2:/tmp# sudo mount --bind dir1/file dir2/file > root@criu2:/tmp# grep "/tmp/dir" /proc/self/mountinfo > 128 24 0:45 / /tmp/dir1 rw,relatime shared:107 - btrfs /dev/loop0 rw,space_cache,subvolid=5,subvol=/ > 129 24 0:45 /file /tmp/dir2/file rw,relatime shared:107 - btrfs /dev/loop0 rw,space_cache,subvolid=5,subvol=/file > root@criu2:/tmp# btrfs --version > btrfs-progs v4.4 > root@criu2:/tmp# uname -a > Linux criu2 4.4.0-8-generic #23-Ubuntu SMP Wed Feb 24 20:45:30 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux > > The issue here is that the "subvol=" mount option for the target of the bind > mount is "/file" when no such subvolume actually exists. Is this > intended? It's confusing to say the least, but seems like a bug to me. Since btrfs mount subvol= is a bind mount behind the scene, I'm not sure the mount info code distinguishes between bind mounts. At the moment, this is something of a secret decoder ring where if you see subvolid=5 first, then anything after that other than / is just not true (can't be). Hence probably why both subvolid and subvol are listed for now; you kinda have to parse them both. -- Chris Murphy