On Tue, Nov 08, 2016 at 10:59:56AM -0800, Mark Fasheh
wrote:> On Mon, Nov 7, 2016 at 6:17 PM, Darrick J. Wong <darrick.wong at
oracle.com> wrote:
> > On Mon, Nov 07, 2016 at 09:54:09PM +0100, Adam Borowski wrote:
> >> Mark has already included XFS in documentation of duperemove, all
that looks
> >> amiss is btrfs-extent-same having an obsolete name. But then, I
never did
> >> any non-superficial tests on XFS, beyond "seems to
work".
>
> I'd actually be ok dropping btrfs-extent-same completely at this point
> but I'm concerned that it would leave some users behind.
>
>
> > /me wonders if ocfs2 will ever catch up to the reflink/dedupe party.
;)
>
> Hey, Ocfs2 started the reflink party! But yeah it's fallen behind
> since then with respect to cow and dedupe. More importantly though I'd
> like to see some extra extent tracking in there like XFS did with the
> reflink b+tree.
Perhaps this should move to the ocfs2 list, but...
...as I understand ocfs2, each inode can point to the head of a refcount
tree that maintains refcounts for all the physical blocks that are
mapped by any of the files that share that refcount tree. It wouldn't
be difficult to hook up this existing refcount structure to the reflink
and dedupe vfs ioctls, with the huge caveat that both inodes will end up
belonging to the same refcount tree (or the call fails). This might not
be such a huge issue for reflink since we're generally only using it
during a file copy anyway, but for dedupe this could have disastrous
consequences if someone does an fs-wide dedupe and every file in the fs
ends up with the same refcount tree.
So I guess you could give each block group its own refcount tree or
something so that all the writes in the fs don't end up contending for a
single data structure.
--D
> --Mark
>
> --
> "When the going gets weird, the weird turn pro."
> Hunter S. Thompson
> --
> To unsubscribe from this list: send the line "unsubscribe
linux-btrfs" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html