From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-9.1 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 167B1C43381 for ; Wed, 13 Mar 2019 19:29:39 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id D35292075C for ; Wed, 13 Mar 2019 19:29:38 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1552505378; bh=1c3qADhncljUBQOHqPQD6kj4YkiNWpprGkkuIeWg+6U=; h=From:To:Cc:Subject:Date:In-Reply-To:References:List-ID:From; b=r55GRxMIKpZOzsPflRQjg0e2vHNUst/rUywSvnHmi8pvCigTXunP89TIIxcbrzUP7 x6z+urlq4GdKGqeflhUafx8HHd6iJ+pfXHHPrqKDWgUGRz5TPcQ+y1PzcZLIs4+eDE Raa3qjtoVgOnPB1onFstdJCNbgfV8wWt2DRkq9p4= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727832AbfCMTMu (ORCPT ); Wed, 13 Mar 2019 15:12:50 -0400 Received: from mail.kernel.org ([198.145.29.99]:43420 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727806AbfCMTMt (ORCPT ); Wed, 13 Mar 2019 15:12:49 -0400 Received: from sasha-vm.mshome.net (c-73-47-72-35.hsd1.nh.comcast.net [73.47.72.35]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id E0E7B2075C; Wed, 13 Mar 2019 19:12:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1552504368; bh=1c3qADhncljUBQOHqPQD6kj4YkiNWpprGkkuIeWg+6U=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=0pJVwmRZ8ZiofAzlxgzR1MeYL5BqSLlWmIwE66e8CObkOcpnOu4PJLKz0YcYCJDtS q3ZshFDIeUKresS2Ti3YT4jQxG67bgLNOoDl0OSwpCc/PrSocVWqUT3wHWFVDxW+Tz Y4imLy5qvQLwIFa4gKWCLmfsgggFH+/GgHY2DFtI= From: Sasha Levin To: linux-kernel@vger.kernel.org, stable@vger.kernel.org Cc: Thadeu Lima de Souza Cascardo , David Ahern , "David S . Miller" , Sasha Levin , netdev@vger.kernel.org, linux-kselftest@vger.kernel.org Subject: [PATCH AUTOSEL 4.20 60/60] selftests: fib_tests: sleep after changing carrier. again. Date: Wed, 13 Mar 2019 15:10:21 -0400 Message-Id: <20190313191021.158171-60-sashal@kernel.org> X-Mailer: git-send-email 2.19.1 In-Reply-To: <20190313191021.158171-1-sashal@kernel.org> References: <20190313191021.158171-1-sashal@kernel.org> MIME-Version: 1.0 X-Patchwork-Hint: Ignore Content-Transfer-Encoding: 8bit Sender: stable-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org From: Thadeu Lima de Souza Cascardo [ Upstream commit af548a27b158d548d41e56255e6eaca1658cc3be ] Just like commit e2ba732a1681 ("selftests: fib_tests: sleep after changing carrier"), wait one second to allow linkwatch to propagate the carrier change to the stack. There are two sets of carrier tests. The first slept after the carrier was set to off, and when the second set ran, it was likely that the linkwatch would be able to run again without much delay, reducing the likelihood of a race. However, if you run 'fib_tests.sh -t carrier' on a loop, you will quickly notice the failures. Sleeping on the second set of tests make the failures go away. Cc: David Ahern Signed-off-by: Thadeu Lima de Souza Cascardo Reviewed-by: David Ahern Signed-off-by: David S. Miller Signed-off-by: Sasha Levin --- tools/testing/selftests/net/fib_tests.sh | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/testing/selftests/net/fib_tests.sh b/tools/testing/selftests/net/fib_tests.sh index 802b4af18729..1080ff55a788 100755 --- a/tools/testing/selftests/net/fib_tests.sh +++ b/tools/testing/selftests/net/fib_tests.sh @@ -388,6 +388,7 @@ fib_carrier_unicast_test() set -e $IP link set dev dummy0 carrier off + sleep 1 set +e echo " Carrier down" -- 2.19.1