xfs
[Top] [All Lists]

Re: Bug (?) : cumulative xfsrestore does not restore files and folders i

To: Damien Gombault <damien.gombault@xxxxxxxx>
Subject: Re: Bug (?) : cumulative xfsrestore does not restore files and folders in a directory which was renamed
From: Dave Chinner <david@xxxxxxxxxxxxx>
Date: Sat, 25 Jun 2016 09:03:46 +1000
Cc: xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <eacabe85-cdff-7fbc-8bb6-c8eb51fc0dac@xxxxxxxx>
References: <a08ae222-7654-4a6f-d9ba-f70017784f74@xxxxxxxx> <20160622220959.GV12670@dastard> <20160622222328.GW12670@dastard> <20160623014217.GZ12670@dastard> <eacabe85-cdff-7fbc-8bb6-c8eb51fc0dac@xxxxxxxx>
User-agent: Mutt/1.5.21 (2010-09-15)
On Fri, Jun 24, 2016 at 03:51:57PM +0200, Damien Gombault wrote:
> Hi.
> 
> Thank you for the patch.
> 
> The patch fixes simple cases (like my reproducer) but it does not fix
> restoration of more complex directory structures : I have tested the
> patch on my
> real data, it fixes some but not all warnings (a user renamed/moved
> a lots of
> files and directory between the 2 dumps).
> 
> Here is another testcase which fails (with the patch) :
> 
> mkdir dira
> mkdir dira/dirc
> touch dira/dirc/filea
> mkdir dirb
> 
> .
> âââ dira
> â   âââ dirc
> â       âââ filea
> âââ dirb
> 
> Make a level 0 dump.
> 
> mv dirb dira/dirB
> mv dira/dirc/ dira/dirB/dirC
> touch dira/dirB/dirC/fileb
> 
> .
> âââ dira
>     âââ dirB
>         âââ dirC
>             âââ filea
>             âââ fileb

Well, yes. I specifically said in the patch description that it
doesn't fix those nested dependency cases and that much deeper
surgery is needed to handle those cases.

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx

<Prev in Thread] Current Thread [Next in Thread>