fsx: fix infinite/too long loops when generating ranges for copy_file_range
authorFilipe Manana <fdmanana@suse.com>
Mon, 20 Apr 2020 17:09:17 +0000 (18:09 +0100)
committerEryu Guan <guaneryu@gmail.com>
Sun, 10 May 2020 12:33:47 +0000 (20:33 +0800)
commite5faa1ef15e48e428b0a62e8aae5e5fd57324c38
treeb319e09d441104c239ec19297497666e6229bda7
parentc8b5a737509b386790ca712b175229dc77863747
fsx: fix infinite/too long loops when generating ranges for copy_file_range

While running generic/521 I've had fsx taking a lot of CPU time and not
making any progress for several hours. Attaching gdb to the fsx process
revealed that fsx was in the loop that generates the ranges for a
copy_file_range operation, in particular the loop seemed to never end
because the range defined by 'offset2' kept overlapping with the range
defined by 'offset'.
So far this happened one time only in one of my test VMs with generic/521.

Fix this by breaking out of the loop after trying 30 times, like we
currently do for dedupe operations, which results in logging the operation
as skipped.

Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: Brian Foster <bfoster@redhat.com>
Signed-off-by: Eryu Guan <guaneryu@gmail.com>
ltp/fsx.c