From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f42.google.com (mail-wr1-f42.google.com [209.85.221.42]) (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 0516E3F65E5 for ; Mon, 18 May 2026 11:30:42 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.42 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779103845; cv=none; b=kxjD81d3CxL7mY0bbDcNgNKd5tFLmSNtVRZYW3tIcykUf0fG89OGC94VOZAKMRlg2FkmLwoE/4JIvEoKREOG+SADCb2joikv2OF9dBYw+jKr5KaWx0/2n2BceGLfdR0FAPOPSvekIwS5IxGLp8z5uUd9EjL+mdkQTEdEVN2cS8g= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779103845; c=relaxed/simple; bh=a8pMTZrbCVo8ggCBj+04CNraHKq2UJhbdckV3vw1Iug=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=BnwOjuI5fuOd/l23wjAI5dXyYH3v6bHT7GYOLmPaBTs+VW+mDbUiPHX1Jny3cBmA3AdMP31vp8iDcVBBNIjqUHrZdXYM300WZQ90TcOnJGN4C48cz++6k/g5rTq4yaQMees987xXaDE1chevgfQ4pGYztIxhEwOyRWk4BuL+Ruk= 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=j/w7ML8J; arc=none smtp.client-ip=209.85.221.42 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="j/w7ML8J" Received: by mail-wr1-f42.google.com with SMTP id ffacd0b85a97d-44b330c5cc6so1534779f8f.1 for ; Mon, 18 May 2026 04:30:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1779103840; x=1779708640; 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=cq2IndJkk2Z/xx3Cj2HaRpHci4EMlQjTZVq9ezC+9Ls=; b=j/w7ML8J73W63tCtdkT/b7bwilbqX+p9hoDdVfWb0P9dzWJmRSjZoWLM41bZjAbowc FY3zaxdMtKx7ma7DJY+nxCbYsolsV3+uahYbCNTaEi9v2JaoVnSj73UGf8imubNP062/ R/gOtWyWQSO4Zkla0Tet3ldXO+xVEuxbJlRPs719to5ccOIamfykCcPtBVWk9wb7JZ6W /87XySZcz2mhBiFrMzI4fLvPswOLi/Q3ijCbjZ0TYsAdDpCaPyC7ujKXc0YY8cSC6Vxn 9IeXdMNKall1KQ9t6vBfgulrE6SSWFLbY0jpquojWA2Zh8zI/LX4E41rnQPva9zQu0Dz hlNw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1779103840; x=1779708640; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=cq2IndJkk2Z/xx3Cj2HaRpHci4EMlQjTZVq9ezC+9Ls=; b=UKR5LQXyEqd08tQYJgPwmJdzeuK5uhLTeyTk7XpwDPlPcnMecAollGD/AY+jhOyfA8 Pu+YJukGzyr7YXEdFqwAbx5QukQ6h1lw3Ib+A5rVWWB26XhtR+IA/NrgYbltucBQderq 57+54RFeF5O3a+5fQOyWNhCKhHF15crTKXt5X705isE7WUli+KMqekrQBtGj/VzmSJjv ZJKBsriXLvb8STTlupCfIwuPIZcUV+N3+X2kvoDmSVFz/beCw//CY7Dsh25TemyZWT3a ETn5WcarzwgFHt2Phk94vSt9d/48Ube5rClwtuxzfJhFKASaGzXCRe3vAH7tAg4s3b2j CUaQ== X-Forwarded-Encrypted: i=1; AFNElJ/9WfKYQOKLbFQ6otr0Wx10BPiVJKOXQ+aeVCe8MCFTH62AsAMG7uL6wC1Y3/cMSinksHWZbwECpqLSrd0=@vger.kernel.org X-Gm-Message-State: AOJu0YyxPwZrMixD/typixT0TbGSeD6V2BT4nKAw3FJcACcLMC6Qztfy rEJU6nDmLEhaeIbvlaDvARFRSqtWn0e+oHYELosAfJ7pgqYTinOWhMju X-Gm-Gg: Acq92OF1XhozZBZbAeAckwR4bvY74YnnYtZKEJHmPGs+x94eKakj6i8xy1aTllSWFI4 EguM5bW5JvWvurkbqReGkZyMkoDz3GAK2RaEEK0OAw3sZd0h8r9VknEFGkFRSSy6hjxMZTpm9HI lpi7BjcXUdVAAimQ544pVpe/DN6hwhV5c1UcwS+ekUtl7wBQ5yrN2bPLGfnuf4QegeKYjBYFmt4 Bvet8tdMvcFVArjl4j9klnimGVNXf1dO9i9Y7O2ZGilwRalGdDbW47fMOjvFCHPnLndpbtgASju Aoaxo0PeTobfIYpgn72HuPPnCxYB4x66hOmb8IaYKbyPFi7pmcHEdx8ci7Ny0HMcSw/yTF2sjEl qW9rWhdWCKtCBNpD9NLFDGjBeIIdPl4+h1xc5H8Gs2z0iqzlgqNPORBs9z9gS7lgiv4nbWqjZi/ ecEnTNAQ+XtyERE7MvXv6rG+x4G0Z3TByCSFD681dlDpVNmH+n3sWaJ994tJEpooDXYSEd3kwxf uQg+lWc+LN9EGpBzhQEd6XQxJgOcW9F5lDfVA6Lmp8O70svHlDQESWUnxkfyKYjiJG5e4tHELk5 NAHdE1dYZE+p3USWdjhXy26GOkohtaTdZg1mXSi1HYw= X-Received: by 2002:a05:6000:2dc2:b0:45d:4c20:7285 with SMTP id ffacd0b85a97d-45e5c5bbe46mr24159481f8f.6.1779103840151; Mon, 18 May 2026 04:30:40 -0700 (PDT) Received: from mail.gmail.com (2a01cb0889497e003b74b873959997d2.ipv6.abo.wanadoo.fr. [2a01:cb08:8949:7e00:3b74:b873:9599:97d2]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-45da0fe13b4sm37171571f8f.28.2026.05.18.04.30.37 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 18 May 2026 04:30:38 -0700 (PDT) Date: Mon, 18 May 2026 13:30:36 +0200 From: Paul Chaignon To: Roman Kvasnytskyi Cc: bpf@vger.kernel.org, Alexei Starovoitov , Daniel Borkmann , Andrii Nakryiko , Martin KaFai Lau , Eduard Zingerman , Kumar Kartikeya Dwivedi , Song Liu , Yonghong Song , Jiri Olsa , Shuah Khan , KP Singh , linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH bpf] selftests/bpf: Reject unsupported -k option in vmtest.sh Message-ID: References: <20260516120625.80839-1-roman@kvasnytskyi.net> Precedence: bulk X-Mailing-List: linux-kernel@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: <20260516120625.80839-1-roman@kvasnytskyi.net> On Sat, May 16, 2026 at 02:06:25PM +0200, Roman Kvasnytskyi wrote: > vmtest.sh does not document a -k option and does not handle it in the > getopts case statement. However, the getopts optstring includes k, which > causes the script to accept -k silently instead of reporting it as an > invalid option. > > Remove k from the optstring so unsupported options are rejected through > the existing invalid-option path. > > Fixes: c9709f52386d ("bpf: Helper script for running BPF presubmit tests") > Signed-off-by: Roman Kvasnytskyi I'm unsure this needs to go through the bpf tree, but it currently applies cleanly to both bpf-next and bpf anyway. Acked-by: Paul Chaignon > --- > tools/testing/selftests/bpf/vmtest.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/testing/selftests/bpf/vmtest.sh b/tools/testing/selftests/bpf/vmtest.sh > index 2f869daf8a06..9ca802285393 100755 > --- a/tools/testing/selftests/bpf/vmtest.sh > +++ b/tools/testing/selftests/bpf/vmtest.sh > @@ -382,7 +382,7 @@ main() > local exit_command="poweroff -f" > local debug_shell="no" > > - while getopts ':hskl:id:j:' opt; do > + while getopts ':hsl:id:j:' opt; do > case ${opt} in > l) > LOCAL_ROOTFS_IMAGE="$OPTARG" > > base-commit: a828abbb897657451d96ad7bf20f1893ac983bb9 > -- > 2.54.0 > >