[PATCH net v3] selftests/net: fin_ack_lat: fix latency threshold typo

[PATCH net v3] selftests/net: fin_ack_lat: fix latency threshold typo

[Brian Grech]

The commit message for af8c8a450bf4 ("selftests: net: Add FIN_ACK
processing order related latency spike test") states: "if the latency
is larger than 1 second (spike), print a message". However the code
uses a threshold of 100000 us (100 ms), not 1000000 us (1 s).

The lower threshold causes false positives on slower hardware where
normal connection latency occasionally exceeds 100 ms but never
approaches the 1 s spike that indicates the actual FIN/ACK race bug.

Fix the threshold to match the documented intent.

Fixes: af8c8a450bf4 ("selftests: net: Add FIN_ACK processing order related latency spike test")
Cc: stable@vger.kernel.org
Reviewed-by: Simon Horman <horms@kernel.org>
Signed-off-by: Brian Grech <bgrech@redhat.com>
---
v3: Added Fixes tag, referenced original commit message wording per reviewer feedback
v2: Added missing maintainer CCs

 tools/testing/selftests/net/fin_ack_lat.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/net/fin_ack_lat.c b/tools/testing/selftests/net/fin_ack_lat.c
index 70187494b57a..4117332eb1a9 100644
--- a/tools/testing/selftests/net/fin_ack_lat.c
+++ b/tools/testing/selftests/net/fin_ack_lat.c
@@ -69,7 +69,7 @@ static void client(int port)
 		lat = timediff(start, end);
 		sum_lat += lat;
 		nr_lat++;
-		if (lat < 100000)
+		if (lat < 1000000)
 			goto close;
 
 		if (getsockname(sock, (struct sockaddr *)&laddr, &len) == -1)
-- 
2.55.0

Re: [PATCH net v3] selftests/net: fin_ack_lat: fix latency threshold typo

[patchwork-bot+netdevbpf]

Hello:

This patch was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Thu,  6 Aug 2026 10:16:45 -0500 you wrote:
> The commit message for af8c8a450bf4 ("selftests: net: Add FIN_ACK
> processing order related latency spike test") states: "if the latency
> is larger than 1 second (spike), print a message". However the code
> uses a threshold of 100000 us (100 ms), not 1000000 us (1 s).
> 
> The lower threshold causes false positives on slower hardware where
> normal connection latency occasionally exceeds 100 ms but never
> approaches the 1 s spike that indicates the actual FIN/ACK race bug.
> 
> [...]

Here is the summary with links:
  - [net,v3] selftests/net: fin_ack_lat: fix latency threshold typo
    https://git.kernel.org/netdev/net-next/c/2373107782a9

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html