On Wed, Jun 07, 2023, Jinrong Liang wrote:
> static void __test_fixed_counter_bitmap(struct kvm_vcpu *vcpu, uint8_t idx,
> uint8_t nr_fixed_counters)
> {
> @@ -881,6 +894,25 @@ static void __test_fixed_counter_bitmap(struct kvm_vcpu *vcpu, uint8_t idx,
> bitmap);
> TEST_ASSERT(!!count == !(bitmap & BIT(idx)),
> "Fixed event filter does not work as expected.");
> +
> + /*
> + * Check that setting both events[] and fixed_counter_bitmap
> + * does not affect the consistency of the fixed ctrs' behaviour.
Spell out "counters". And this isn't about "consistency", the fixed_counter_bitmap
should straight up win, i.e.
/*
* Check that fixed_counter_bitmap has higher priority than
* events[] when both are set.
*/
