在 5/23/2025 10:32 PM, Zorro Lang 写道:
> [You don't often get email from zl...@redhat.com. Learn why this is important 
> at https://aka.ms/LearnAboutSenderIdentification ]
>
> On Thu, May 22, 2025 at 01:57:06AM -0600, Chunhai Guo wrote:
>> This is a regression test case to verify whether the CP_TRIMMED_FLAG is
>> properly set after performing the following steps:
>> 1. mount the f2fs filesystem
>> 2. create a file, write data to it, then delete the file
>> 3. unmount the filesystem
>> 4. verify that the 'trimmed' flag is set in the checkpoint state
>>
>> We should apply the commit ("f2fs: fix missing discard for active
>> segments") to resolve the issue where the 'trimmed' flag is missing.
>>
>> Signed-off-by: Chunhai Guo <guochun...@vivo.com>
>> ---
> This test case looks good to me, just a few picky review points as below:
>
>> v2:
>>    - Skip this test case if discard is not supported.
>>    - Renumber this test case from 012 to 014.
>> v1:
>>    
>> https://lore.kernel.org/linux-f2fs-devel/20250521064952.1373690-1-guochun...@vivo.com/
>> ---
>>   tests/f2fs/014     | 41 +++++++++++++++++++++++++++++++++++++++++
>>   tests/f2fs/014.out |  2 ++
>>   2 files changed, 43 insertions(+)
>>   create mode 100755 tests/f2fs/014
>>   create mode 100644 tests/f2fs/014.out
>>
>> diff --git a/tests/f2fs/014 b/tests/f2fs/014
>> new file mode 100755
>> index 000000000000..f72d53e65292
>> --- /dev/null
>> +++ b/tests/f2fs/014
>> @@ -0,0 +1,41 @@
>> +#! /bin/bash
>> +# SPDX-License-Identifier: GPL-2.0
>> +# Copyright (c) 2025 Guo Chunhai.  All Rights Reserved.
> You're using a enterprise mail address -- @vivo.com, so I'm just wondering
> your the Copyright should be yourself or your company?


Thanks for the reminder. I'll change the Copyright to vivo in the V3 
patch update.


>> +#
>> +# FS QA Test No. f2fs/014
>> +#
>> +# This is a regression test case to verify whether the CP_TRIMMED_FLAG is
>> +# properly set after performing the following steps:
>> +# 1. mount the f2fs filesystem
>> +# 2. create a file, write data to it, then delete the file
>> +# 3. unmount the filesystem
>> +# 4. verify that the 'trimmed' flag is set in the checkpoint state
>> +# We should apply the commit ("f2fs: fix missing discard for active
>> +# segments") to resolve the issue where the 'trimmed' flag is missing.
>> +#
>> +. ./common/preamble
>> +_begin_fstest auto quick
> Do we need to add this case to "trim" test group, as you always metion
> "trimmed" in comment :)


Yes, I do need to add this case to the "trim" test group.


>
>> +
>> +_fixed_by_kernel_commit 21263d035ff2 \
>> +     "f2fs: fix missing discard for active segments"
>> +
>> +_require_scratch
>> +_require_xfs_io_command "pwrite"
> This line isn't wrong, but I think you can save this line, due to the "pwrite"
> is always there.
>
> Others look good to me,

Got it. I will remove this line.

Thank you for all the reminders.

>
> Reviewed-by: Zorro Lang <zl...@redhat.com>
>
>> +_require_command "$DUMP_F2FS_PROG" dump.f2fs
>> +
>> +_scratch_mkfs >> $seqres.full
>> +_scratch_mount >> $seqres.full
>> +
>> +_require_batched_discard $SCRATCH_MNT
>> +
>> +foo=$SCRATCH_MNT/foo
>> +$XFS_IO_PROG -c "pwrite 0 1m" -c "fsync" -f $foo >> $seqres.full
>> +sync
>> +
>> +rm $foo
>> +_scratch_unmount >> $seqres.full 2>&1
>> +
>> +$DUMP_F2FS_PROG $SCRATCH_DEV | grep "checkpoint state" | grep -o trimmed
>> +
>> +status=0
>> +exit
>> diff --git a/tests/f2fs/014.out b/tests/f2fs/014.out
>> new file mode 100644
>> index 000000000000..cb0c9447d675
>> --- /dev/null
>> +++ b/tests/f2fs/014.out
>> @@ -0,0 +1,2 @@
>> +QA output created by 014
>> +trimmed
>> --
>> 2.34.1
>>
>>


_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to