On 6/23/20 11:19 AM, Peter Maydell wrote: > On Tue, 23 Jun 2020 at 10:18, Andrew Jones <drjones@xxxxxxxxxx> wrote: >> >> On Tue, Jun 23, 2020 at 11:06:20AM +0200, Philippe Mathieu-Daudé wrote: >>> Since v2: >>> - include Drew test fix (addressed Peter review comments) >>> - addressed Drew review comments >>> - collected R-b/A-b >>> >>> Andrew Jones (1): >>> tests/qtest/arm-cpu-features: Add feature setting tests >>> >>> Philippe Mathieu-Daudé (1): >>> target/arm: Check supported KVM features globally (not per vCPU) >>> >>> target/arm/kvm_arm.h | 21 ++++++++----------- >>> target/arm/cpu.c | 2 +- >>> target/arm/cpu64.c | 10 ++++----- >>> target/arm/kvm.c | 4 ++-- >>> target/arm/kvm64.c | 14 +++++-------- >>> tests/qtest/arm-cpu-features.c | 38 ++++++++++++++++++++++++++++++---- >>> 6 files changed, 56 insertions(+), 33 deletions(-) >>> >>> -- >>> 2.21.3 >>> >>> >> >> Hi Phil, >> >> Thanks for including the test patch. To avoid breaking bisection, if one >> were to use qtest to bisect something, then the order of patches should >> be reversed. Oops :) >> I guess Peter can apply them that way without a repost >> though. > > Yeah, I can just flip the order. Thanks both!