On Fri, Apr 25, 2025 at 03:05:57PM -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> > --- > scripts/runtime.bash | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/scripts/runtime.bash b/scripts/runtime.bash > index 4b9c7d6b..59d1727c 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 Let's use [[ ! $ACCEL =~ ^$accel(,.*|$) ]] to be a bit more precise. Thanks, drew > print_result "SKIP" $testname "" "$accel only, but ACCEL=$ACCEL" > return 2 > elif [ -n "$ACCEL" ]; then > > base-commit: 0d3cb7dd56ec255a71af867c2d76c8f4b22cd420 > -- > 2.49.0.850.g28803427d3-goog >
