On 30/10/18 13:19, Christoffer Dall wrote: > On Mon, Sep 03, 2018 at 02:25:50PM +0200, Andrew Jones wrote: >> On Mon, Sep 03, 2018 at 12:58:45PM +0200, Christoffer Dall wrote: >>> The current MAX_SMP check launches QEMU for every value of the SMP >>> parameter to check if the SMP parameter based on the native is higher >>> than the QEMU maximum supported configuration. >>> >>> On something like a TX2, which has 224 threads, this takes a very long >>> time where the test script just sits there with no output. >>> >>> Reduce the wait time by taking the log2 of MAX_SMP in each iteration of >>> the loop instead. >>> >>> Signed-off-by: Christoffer Dall <christoffer.dall@xxxxxxx> >>> --- >>> scripts/runtime.bash | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/scripts/runtime.bash b/scripts/runtime.bash >>> index a31ae91..1e2c288 100644 >>> --- a/scripts/runtime.bash >>> +++ b/scripts/runtime.bash >>> @@ -140,5 +140,5 @@ function run() >>> # just remove it... >>> while $RUNTIME_arch_run _NO_FILE_4Uhere_ -smp $MAX_SMP \ >>> |& grep -qi 'exceeds max CPUs'; do >>> - ((--MAX_SMP)) >>> + MAX_SMP=$((MAX_SMP >> 1)) >>> done >>> -- >>> 2.7.4 >>> >> >> Reviewed-by: Andrew Jones <drjones@xxxxxxxxxx> >> >> > > Paolo, if you're ok with this patch, could you apply it? > > (I failed to CC you the first time around, only had the kvm list.) > > Thanks, > > Christoffer > Queued now. Paolo
