From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ot1-f52.google.com (mail-ot1-f52.google.com [209.85.210.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 584433C3442 for ; Wed, 5 Aug 2026 03:09:43 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.52 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785899384; cv=none; b=U+lyFhyIC8SnGq8YsfBZHhpfX/Ixyi4kezS/0lKodaXT0Ru3dGsUQ0XxFOTHvgv5tLa9ysgPIGkg0gUVO7WDVHIyGZhigTz6GJcS/NbalLRajqDyw5qZwxj40M255IMhiYIqMHwowvPKyKL3yuMwu9xs+mgk5Y0MW2AY7iwzCsE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785899384; c=relaxed/simple; bh=bFLblOr7/KRtDWGaNUQ/oxXl5YkPdtI2jyGgXHP9qko=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=t8FIcxHdz5h9aJDq9Z5Qaj0xHBpG8NQPjZd1tNlU5QwIplnlPPN/bWRumK52MA/CZhHRkI0A7SjshF0mJ3z4LLfhLSirRmTpKrmzqvtZuZrTEsjRTDb/+dpYWNpCwy4e2rWjBLI0Ka1Xxpgt5t4w/AaSjpHNKZYfHEpYg0do5bc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=rO+WoNmf; arc=none smtp.client-ip=209.85.210.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="rO+WoNmf" Received: by mail-ot1-f52.google.com with SMTP id 46e09a7af769-7e9ecd7216cso354515a34.3 for ; Tue, 04 Aug 2026 20:09:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1785899382; x=1786504182; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=OtqhiQPtfLx478uCqIbccXvcoISzUjC+Hx+PXZFH8QM=; b=rO+WoNmfbg0eSFcqarVT07ZnGLu6pD6yC1N0OTL/t0F3Y7jdtHE0In3ZtCtdToQO4P KQXJOiuep2BUB0C/a0TWfV5+ZIgyLbq2qlWKCD6Da3sYkZWgysGzj++RuX4OSMtHLxjD 3SlsVzmiFm7FGjN4v4BVY1l3KdXLpCVe1C0iaVmEtB2cRwyVPQxSmU+bYwJwLSyJmKb3 3y1XgzI8eCQHC90I7kGHygNGyx/N5H4FSczJaBt4rh8fHK99PdoQlVSdAl3OMKi/XJJB kO/zdJ6rllJeOFL8QVFmkt7POl9/Ngx0m68ETKxeKHR+c9/fAhH4XxQeSTT+M9W7mzdg gF0w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1785899382; x=1786504182; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=OtqhiQPtfLx478uCqIbccXvcoISzUjC+Hx+PXZFH8QM=; b=pDmk9Nn7hNARwpPTsVmgjTseCnXxlrbddqSSgXizQgYMotQ71k03fJG+W73pUtZUG2 5a7ObnUHtMMjdiHnsF33zlJFV2jlfgr5GmsT87Ij+6sZiZIj8A+TI4mQziGFioBOlr9Z jRFzI8M7JJMBGk/8H0PC6QjgD9tnvq7Z9qPjbO1a9CSJ3RWqOmly2NYHBDYipu5/ZFBx K6N9yjZhq+MUxKeJZVdJPmJpp2AURalwKvHBU6eKxZeVcqiHUwMMrPQhUoXbWarqWbb3 +kr2yvYFz9MJuz3iOhdUpHZFr1wJ0HDJIS4DE960Rz5LbF/1vCngHHZIQcVWNn+Ykfpc 8+CA== X-Forwarded-Encrypted: i=1; AHgh+Rr2+Qn15aZqUgJU5msEJtbwJWkASnVnuc+SGwzrxGrgNMnsAxmviNk+jg6nOvD75iJaTL/ICxu+j6pItdo=@vger.kernel.org X-Gm-Message-State: AOJu0YyG8zoBkcMOvzp/SxZMJyTt1aCF6t/+T505Rg34GTOO8dgZUF9G dMuGI+iyTKk54Xck17V8dMbhCxAvJPKuTe9jgJIGGmrubchyb+naW9rb X-Gm-Gg: AR+sD11N18Wjbul5d8Wm9BK5v7/Oyiei45Ki17JgIRTlXyROF396g99QUvV2C8AHncd VPJBMFZwyZf50GabTH8xaJyC8dCGqVWylxoEThtsU3nZkmG4ka+cPwtKRiX0r5ztWxz9qoq9jfn 8rYVyCNlS+ffM3bkBM27zoQn2zzqblj6gi8DZx53McetEPGn8LF8psHJ/oLK7XoBHjx6Ru+cE1/ mrVDZkug2cOMul/2d7qi91nicpSoHiRh2OdqIl4r1QEmCKFkZBWfMBR8Hwe1sLRVG/MECXCKqXU A8D//mSkei10Mmi8ZylGYM0q5cF5ZfjUel76qP2wu3mSThWyNi4ANCDh5UcKm4NWcFK3ZaWsW4k JbflBnKHhNWk8AUjfFFQ9fRunhs8gJwLKHaWYtyCiyUezCv2l3xnAcWmNNa8yH7K/p/NMH4tjQO DywNHueMpBazscm+V1rmnnRK/rKpoDQE0zbKEsd0WgPsLm3aO6BFZY5k6x X-Received: by 2002:a05:6830:2116:b0:7d7:ed69:81b2 with SMTP id 46e09a7af769-7f1e5d0f2e8mr2759082a34.5.1785899382203; Tue, 04 Aug 2026 20:09:42 -0700 (PDT) Received: from localhost ([2a03:2880:31ff:a::]) by smtp.gmail.com with ESMTPSA id 46e09a7af769-7f1df2fd11dsm2032300a34.5.2026.08.04.20.09.40 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 04 Aug 2026 20:09:41 -0700 (PDT) From: Mohsin Bashir To: netdev@vger.kernel.org Cc: andrew+netdev@lunn.ch, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org, pabeni@redhat.com, shuah@kernel.org, alexander.duyck@gmail.com Subject: [PATCH net-next 01/14] selftests: net: shaper: Drop redundant command timeouts Date: Tue, 4 Aug 2026 20:09:23 -0700 Message-ID: <20260805030936.1092907-2-mohsin.bashr@gmail.com> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260805030936.1092907-1-mohsin.bashr@gmail.com> References: <20260805030936.1092907-1-mohsin.bashr@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: Mohsin Bashir Commit 57bb59ab6fa3 ("selftests: net: bump default cmd() timeout to 20 seconds") raised the default cmd() timeout to 20 seconds, so the explicit timeout=10 passed to the ethtool channel commands in queue_update() is now redundant and, in fact, shorter than the default. Drop it and rely on the default timeout. Signed-off-by: Jakub Kicinski Signed-off-by: Mohsin Bashir --- tools/testing/selftests/drivers/net/shaper.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tools/testing/selftests/drivers/net/shaper.py b/tools/testing/selftests/drivers/net/shaper.py index e39d270e688d..c80a4bf8cc05 100755 --- a/tools/testing/selftests/drivers/net/shaper.py +++ b/tools/testing/selftests/drivers/net/shaper.py @@ -388,7 +388,7 @@ def queue_update(cfg, nl_shaper) -> None: 'bw-max': (i + 1) * 1000}) # Delete a channel, with no shapers configured on top of the related # queue: no changes expected - cmd(f"ethtool -L {cfg.dev['ifname']} {cfg.rx_type} 3", timeout=10) + cmd(f"ethtool -L {cfg.dev['ifname']} {cfg.rx_type} 3") shapers = nl_shaper.get({'ifindex': cfg.ifindex}, dump=True) ksft_eq(shapers, [{'ifindex': cfg.ifindex, 'parent': {'scope': 'netdev'}, @@ -408,7 +408,7 @@ def queue_update(cfg, nl_shaper) -> None: # Delete a channel, with a shaper configured on top of the related # queue: the shaper must be deleted, too - cmd(f"ethtool -L {cfg.dev['ifname']} {cfg.rx_type} 2", timeout=10) + cmd(f"ethtool -L {cfg.dev['ifname']} {cfg.rx_type} 2") shapers = nl_shaper.get({'ifindex': cfg.ifindex}, dump=True) ksft_eq(shapers, [{'ifindex': cfg.ifindex, @@ -423,7 +423,7 @@ def queue_update(cfg, nl_shaper) -> None: 'bw-max': 2000}]) # Restore the original channels number, no expected changes - cmd(f"ethtool -L {cfg.dev['ifname']} {cfg.rx_type} {cfg.nr_queues}", timeout=10) + cmd(f"ethtool -L {cfg.dev['ifname']} {cfg.rx_type} {cfg.nr_queues}") shapers = nl_shaper.get({'ifindex': cfg.ifindex}, dump=True) ksft_eq(shapers, [{'ifindex': cfg.ifindex, 'parent': {'scope': 'netdev'}, -- 2.53.0-Meta