Hi,
I noticed that btrfs_getattr() is filling stat->dev with an anonymous device
(for per-snapshot root?):
stat->dev = BTRFS_I(inode)->root->anon_super.s_dev;
but /proc/pid/maps uses the "real" block device:
dev = inode->i_sb->s_dev;
This results in some unfortunate behavior for lsof as it reports some
duplicate paths (except on different block devices). The easiest way to see
this (if your root partition is btrfs):
$ lsof | grep lsof
<snip>
lsof 9238 root txt REG 0,19 139736
14478 /usr/bin/lsof
lsof 9238 root mem REG 0,17
14478 /usr/bin/lsof (path dev=0,19)
Ultimately, this breaks existing software. In my case, "zypper ps" gets
really unhappy (which may partially also be due to a zypper bug, hooray!)
I'm not really quite sure how this should be handled though. Do we have
/proc/pid/maps report the subvolumes device (via some callback I suppose)?
Another alternative of course is to return the true block device in
btrfs_getattr() but that has some obvious downsides too.
Thanks and best regards,
--Mark
--
Mark Fasheh
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
