From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-yx1-f46.google.com (mail-yx1-f46.google.com [74.125.224.46]) (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 0C5B9322DA8 for ; Wed, 5 Nov 2025 16:11:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.224.46 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762359096; cv=none; b=l5Bxb7pEOL+blDT9jOpNQPPwJ0ufbhnYpAB4bBGmdgVvBwy4CaiGpcHlfnC+DOVUzdox2WE4MvWvLqDVjOUoWAuahvaoqgYoggrFGEm/+1YlpvAnC/7kmf0JEM8RnxqXLI1pnHGQaSnWsBEX2K4boDEYrQEwIA3OEVWu9vncIoo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762359096; c=relaxed/simple; bh=W3rIEqDAjFHWAwm+pwB74Wz+AVL3IfQgKO4Y4lo7W2o=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=nY3EzZNLUt5k3GnyU3IqXv0gV0PaAcf+UewI2iFP3X24snOVXMRB1CF5CQfTtj9ZemCk5PcOclhx+jFmut/dYeWt4S379k4Yi0WbR1SA8Cfhqnoc8q7yVA2psluOmeOWw+OCTT/YhldggUFdQVcnVdc4O9PzSfn/Q076MYMCcPU= 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=YHbDbLIt; arc=none smtp.client-ip=74.125.224.46 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="YHbDbLIt" Received: by mail-yx1-f46.google.com with SMTP id 956f58d0204a3-63fc8c337f2so22923d50.0 for ; Wed, 05 Nov 2025 08:11:33 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1762359093; x=1762963893; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=ymZ6xdXrqo7vWAgf0QChueJMpmkC1T045PA4GVzRwIo=; b=YHbDbLItjw4+eMXKEAHsY2z6+TDNrzIu7JCue8TYKGBW5rNDBDA4JyyM/UgPjyAf4M 5MqkGFSQxKclN7aJD+qGLJCYV8vCTg6YPemfsPJrI3AV/XQJqcMzkaKNo0t8msTKlFda e0pVYvImharsbSWHL60w2PP/PvzOKgwAoZzx3jTNCWMoPWQBzVE8Htct92I7j5p4HMZE BhFEFcR3GXgmVF4F6OGJ1AgDCO4C0Dk3QKjpQlQJvUykVFNLhQIEiM9DB7tiQ78fkYeL 41nBDihiXKZ3pD+aMvDrHd4Kdyc90F8YhaprnNZCz7GEXfN1+UFPvh2qp+b46wLn+QxX /s6Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1762359093; x=1762963893; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=ymZ6xdXrqo7vWAgf0QChueJMpmkC1T045PA4GVzRwIo=; b=hByMAAO5bRADCZWTheJq3Xj6Czd/YPhq4SiZeayA5eehdHMWEcP0cH1gY/W7n+yYLR 75KFUiBJQZvZq0MnX7JOMCsJL0xy5jd+jyvnGjvl9nBedu+7mkvf/r0AVCBxgnLAymKl KV8NCP6i1mP2ROwBSX6hcLsU61KGIyNP88XWIefojKwltRsKIn8P62NsTnhHm9a/K7uL cJsKOYdgbMx3f9yB/UlnqpwNh75lKtfKSxILgYVRfYUJgktZJQ8gC0mL98JWhHvI/C9/ 3aWrofSiZ3o5CysgEJJdOl85NnYvLHCn6pgA4fMUDnR+Bz6lqvKovT5n6ajmSUFVkE5l /Kow== X-Forwarded-Encrypted: i=1; AJvYcCUIw3V37GS+ynmct4iHRPq1k5XdSdMsw4b+NQNDPxT7GKnEoAjlu+m/WQrVu87LLuaaQ0R0KbE=@vger.kernel.org X-Gm-Message-State: AOJu0YyXsdvQ+CIj9DsnVjan0AvGKIf7+wcGihdihvV6x1yssEKsMFTK jw2mF7hebaYp74FhNe05Pk/hre31r+x8XR2hybBDuvWp8CZtE1JIi7cD X-Gm-Gg: ASbGncsDteHp0Z/FrTW/P0K3FJlgPfp7VotclBDRaYGyL11RkmT7qspP11mlY0eWkJ3 /fR+KhJV4F7O7oQBE5B4AIAMJjxNJojtvYmDQyM2r6oSbRtWgQDSpkxSz0CTaQo8B9oPn5DKBQd fAnt3vFaAAZ4HH8kt/GIW0fGRwIOcZoD6hXb01QjPEBRqeK+tz9UHUinYcWZVQsV6zvdx89nE1q ddnHU+E/WWoIk26AZ1tAz+dxme+fqr2owGkRaj8diPca6OHuuBEOjSMOHU1EQk3XJEKl223oFTC yxVsVv9gTR9KGg5wuEZFWUq+bZZJdfmhz7r5e4VJe3EK7Jhf6amWoC74674lakuXobfz4+Ci23h vK27cnU+6SdUBN553F72bYMWL3tC1i0w0Fe1XDvu5OtrT2qAWM+vN4r4DbZTg7UF8yE2k7uhxY7 WHulgXIPbSBVN1pZDeItJEvtft5vLSgjvfZEc= X-Google-Smtp-Source: AGHT+IFO1Aj58ygyCOlI2zA+lUSTMddFikFOwBkJL3a+he6zkxfPMBnlF0h5B3IACmlOERHv2GNjEQ== X-Received: by 2002:a05:690c:9a0a:b0:786:581d:f24b with SMTP id 00721157ae682-786a41e29cemr57871067b3.47.1762359092678; Wed, 05 Nov 2025 08:11:32 -0800 (PST) Received: from devvm11784.nha0.facebook.com ([2a03:2880:25ff:7::]) by smtp.gmail.com with ESMTPSA id 00721157ae682-78691d8f0b4sm20313777b3.4.2025.11.05.08.11.32 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 05 Nov 2025 08:11:32 -0800 (PST) Date: Wed, 5 Nov 2025 08:11:26 -0800 From: Bobby Eshleman To: Stefano Garzarella Cc: Shuah Khan , Jakub Kicinski , virtualization@lists.linux.dev, netdev@vger.kernel.org, linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org, Bobby Eshleman , Simon Horman Subject: Re: [PATCH net] selftests/vsock: avoid false-positives when checking dmesg Message-ID: References: <20251104-vsock-vmtest-dmesg-fix-v1-1-80c8db3f5dfe@meta.com> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Wed, Nov 05, 2025 at 12:16:42PM +0100, Stefano Garzarella wrote: > On Tue, Nov 04, 2025 at 01:50:50PM -0800, Bobby Eshleman wrote: > > From: Bobby Eshleman > > > > Sometimes VMs will have some intermittent dmesg warnings that are > > unrelated to vsock. Change the dmesg parsing to filter on strings > > containing 'vsock' to avoid false positive failures that are unrelated > > to vsock. The downside is that it is possible for some vsock related > > warnings to not contain the substring 'vsock', so those will be missed. > > > > Fixes: a4a65c6fe08b ("selftests/vsock: add initial vmtest.sh for vsock") > > Reviewed-by: Simon Horman > > Signed-off-by: Bobby Eshleman > > --- > > Previously was part of the series: > > https://lore.kernel.org/all/20251022-vsock-selftests-fixes-and-improvements-v1-0-edeb179d6463@meta.com/ > > --- > > tools/testing/selftests/vsock/vmtest.sh | 8 ++++---- > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > > diff --git a/tools/testing/selftests/vsock/vmtest.sh b/tools/testing/selftests/vsock/vmtest.sh > > index edacebfc1632..e1732f236d14 100755 > > --- a/tools/testing/selftests/vsock/vmtest.sh > > +++ b/tools/testing/selftests/vsock/vmtest.sh > > @@ -389,9 +389,9 @@ run_test() { > > local rc > > > > host_oops_cnt_before=$(dmesg | grep -c -i 'Oops') > > - host_warn_cnt_before=$(dmesg --level=warn | wc -l) > > + host_warn_cnt_before=$(dmesg --level=warn | grep -c -i 'vsock') > > vm_oops_cnt_before=$(vm_ssh -- dmesg | grep -c -i 'Oops') > > - vm_warn_cnt_before=$(vm_ssh -- dmesg --level=warn | wc -l) > > + vm_warn_cnt_before=$(vm_ssh -- dmesg --level=warn | grep -c -i 'vsock') > > > > name=$(echo "${1}" | awk '{ print $1 }') > > eval test_"${name}" > > @@ -403,7 +403,7 @@ run_test() { > > rc=$KSFT_FAIL > > fi > > > > - host_warn_cnt_after=$(dmesg --level=warn | wc -l) > > + host_warn_cnt_after=$(dmesg --level=warn | grep -c -i vsock) > > In the previous hunk we quoted 'vsock', but here and in the next we did > not. Can we be consistent at least in the same patch ? > > The rest LGTM. > > Stefano Just sent the update, sorry for the oversight. Best, Bobby