On Fri, May 02, 2025 at 12:56:18PM -0700, Sean Christopherson wrote: > Search the entire ACCEL string for the required accelerator as searching > for an exact match incorrectly rejects ACCEL when additional accelerator > specific options are provided, e.g. > > SKIP pmu (kvm only, but ACCEL=kvm,kernel_irqchip=on) > > Signed-off-by: Sean Christopherson <seanjc@xxxxxxxxxx> > --- > > v2: Require the accelerator string to match the start and end, with an > allowance for comma-seperate options. [Drew] > > scripts/runtime.bash | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/scripts/runtime.bash b/scripts/runtime.bash > index 4b9c7d6b..ee229631 100644 > --- a/scripts/runtime.bash > +++ b/scripts/runtime.bash > @@ -126,7 +126,7 @@ function run() > machine="$MACHINE" > fi > > - if [ -n "$accel" ] && [ -n "$ACCEL" ] && [ "$accel" != "$ACCEL" ]; then > + if [ -n "$accel" ] && [ -n "$ACCEL" ] && [[ ! "$ACCEL" =~ ^$accel(,|$) ]]; then > print_result "SKIP" $testname "" "$accel only, but ACCEL=$ACCEL" > return 2 > elif [ -n "$ACCEL" ]; then > > base-commit: abdc5d02a7796a55802509ac9bb704c721f2a5f6 > -- > 2.49.0.906.g1f30a19c02-goog > Merged. Thanks, drew
