Re: [PATCH] overlay: test hardlink breakage on copy up

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Dec 13, 2016 at 5:35 AM, Eryu Guan <eguan@xxxxxxxxxx> wrote:
> On Mon, Dec 12, 2016 at 12:29:21PM +0200, Amir Goldstein wrote:
>> Introduce a new test to demonstrate a known issue with overlayfs:
>> - file A and B are hardlinked in lower
>> - modify A to trigger copy up
>> - file A is no longer a hardlink of file B
>>
>> There is no fix for this issue at this time.
>
> Is this something we want to fix eventually?

This is a question for Miklos to answer, but here is a quote from him
on the stable inode POC thread:
https://marc.info/?l=linux-unionfs&m=148045641305234&w=3

"...
The following issues are left:
 - performance of readdir;
 - what to do if not all layers are on the same fs;
 - hard link copy ups.
"

> If not, do we want a test
> that is failing forever? I ask because I heard that this is something
> not easy/worth fixing, at least low priority. But things may change now.
>

And another quote from Miklos on an answer to similar question on
the overlay/016 tests thread:
https://marc.info/?l=fstests&m=148041025721866&w=3

"...
> What is the preferred timing for merging these sort of tests?
> Is it productive to have these tests merged before a fix is merged to master?
> Before a fix is queued for next?
> Before a fix is available?

IMO adding a test doesn't hurt, it'll just indicate that the current
version is broken.  It doesn't have to have any synchronization with
the actual fix.
"


>>
>> Signed-off-by: Amir Goldstein <amir73il@xxxxxxxxx>
>> ---
>>  tests/overlay/019     | 88 +++++++++++++++++++++++++++++++++++++++++++++++++++
>>  tests/overlay/019.out |  2 ++
>>  tests/overlay/group   |  1 +
>>  3 files changed, 91 insertions(+)
>>  create mode 100755 tests/overlay/019
>>  create mode 100644 tests/overlay/019.out
>>
>> diff --git a/tests/overlay/019 b/tests/overlay/019
>> new file mode 100755
>> index 0000000..382e4ee
>> --- /dev/null
>> +++ b/tests/overlay/019
>> @@ -0,0 +1,88 @@
>> +#! /bin/bash
>> +# FSQA Test No. 019
>> +#
>> +# Test hardlink breakage
>> +#
>> +# This simple test demonstrates a known issue with overlayfs:
>> +# - file A and B are hardlinked in lower
>> +# - modify A to trigger copy up
>> +# - file A is no longer a hardlink of file B
>> +#
>> +#-----------------------------------------------------------------------
>> +#
>> +# Copyright (C) 2016 CTERA Networks. All Rights Reserved.
>> +# Author: Amir Goldstein <amir73il@xxxxxxxxx>
>> +#
>> +# This program is free software; you can redistribute it and/or
>> +# modify it under the terms of the GNU General Public License as
>> +# published by the Free Software Foundation.
>> +#
>> +# This program is distributed in the hope that it would be useful,
>> +# but WITHOUT ANY WARRANTY; without even the implied warranty of
>> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
>> +# GNU General Public License for more details.
>> +#
>> +# You should have received a copy of the GNU General Public License
>> +# along with this program; if not, write the Free Software Foundation,
>> +# Inc.,  51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
>> +#-----------------------------------------------------------------------
>> +#
>> +
>> +seq=`basename $0`
>> +seqres=$RESULT_DIR/$seq
>> +echo "QA output created by $seq"
>> +
>> +tmp=/tmp/$$
>> +status=1     # failure is the default!
>> +trap "_cleanup; exit \$status" 0 1 2 3 15
>> +
>> +_cleanup()
>> +{
>> +     cd /
>> +     rm -f $tmp.*
>> +}
>> +
>> +# get standard environment, filters and checks
>> +. ./common/rc
>> +. ./common/filter
>> +
>> +# real QA test starts here
>> +_supported_fs overlay
>> +_supported_os Linux
>> +_require_scratch
>> +
>> +rm -f $seqres.full
>> +
>> +_scratch_mkfs >>$seqres.full 2>&1
>> +
>> +# Create 2 hardlinked files in lower
>> +lowerdir=$SCRATCH_DEV/$OVERLAY_LOWER_DIR
>> +mkdir -p $lowerdir
>> +echo "patient zero" >> $lowerdir/foo
>> +ln $lowerdir/foo $lowerdir/bar
>> +
>> +
>> +_scratch_mount
>> +
>> +cd $SCRATCH_MNT
>
> It's uncommon to cd to $SCRATCH_MNT in fstests, if it can be done
> without doing so. Usually we do
>
> testfile1=$SCRATCH_MNT/foo
> testfile2=$SCRATCH_MNT/bar
>
> <do tests on $testfile1 and $testfile2>
>
> Can this test be updated in this way? And patch "overlay: test unstable
> inode number"? Sorry that I forgot about this yesterday..
>

No problem.

> Thanks!
>
> Eryu
>
>> +
>> +rm -f $tmp.before $tmp.after
>> +
>> +# Record inode number and nlink before copy up
>> +ls -li foo bar | awk '{ print $1, $3 }' > $tmp.before
>> +
>> +# Modify content of one of the hardlinks
>> +echo "mutated" >> foo
>> +
>> +# Record inode number and nlink after copy up
>> +ls -li foo bar | awk '{ print $1, $3 }' > $tmp.after
>> +
>> +# Compare ino/nlink before..after - expect silence
>> +diff $tmp.before $tmp.after
>> +
>> +# Compare content of files - expect silence
>> +diff foo bar
>> +
>> +echo "Silence is golden"
>> +status=0
>> +exit
>> diff --git a/tests/overlay/019.out b/tests/overlay/019.out
>> new file mode 100644
>> index 0000000..163484b
>> --- /dev/null
>> +++ b/tests/overlay/019.out
>> @@ -0,0 +1,2 @@
>> +QA output created by 019
>> +Silence is golden
>> diff --git a/tests/overlay/group b/tests/overlay/group
>> index fb6610d..f266b0a 100644
>> --- a/tests/overlay/group
>> +++ b/tests/overlay/group
>> @@ -20,3 +20,4 @@
>>  015 auto quick whiteout
>>  016 auto quick copyup
>>  018 auto quick copyup
>> +019 auto quick copyup
>> --
>> 2.7.4
>>
--
To unsubscribe from this list: send the line "unsubscribe linux-unionfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Filesystems Devel]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux