* [kvm-unit-tests PATCH] x86: pmu: fix failures on 32-bit due to wrong masks
@ 2020-06-19 19:39 Nadav Amit
2020-06-25 16:33 ` Nadav Amit
2020-06-25 16:41 ` Jim Mattson
0 siblings, 2 replies; 4+ messages in thread
From: Nadav Amit @ 2020-06-19 19:39 UTC (permalink / raw)
To: Paolo Bonzini; +Cc: kvm, Nadav Amit
Some mask computation are using long constants instead of long long
constants, which causes test failures on x86-32.
Signed-off-by: Nadav Amit <namit@vmware.com>
---
x86/pmu.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/x86/pmu.c b/x86/pmu.c
index 91a6fb4..5a3d55b 100644
--- a/x86/pmu.c
+++ b/x86/pmu.c
@@ -324,11 +324,11 @@ static void check_counter_overflow(void)
cnt.count = 1 - count;
if (gp_counter_base == MSR_IA32_PMC0)
- cnt.count &= (1ul << eax.split.bit_width) - 1;
+ cnt.count &= (1ull << eax.split.bit_width) - 1;
if (i == num_counters) {
cnt.ctr = fixed_events[0].unit_sel;
- cnt.count &= (1ul << edx.split.bit_width_fixed) - 1;
+ cnt.count &= (1ull << edx.split.bit_width_fixed) - 1;
}
if (i % 2)
@@ -456,7 +456,7 @@ static void check_running_counter_wrmsr(void)
count = -1;
if (gp_counter_base == MSR_IA32_PMC0)
- count &= (1ul << eax.split.bit_width) - 1;
+ count &= (1ull << eax.split.bit_width) - 1;
wrmsr(gp_counter_base, count);
@@ -488,7 +488,7 @@ static void check_gp_counters_write_width(void)
{
u64 val_64 = 0xffffff0123456789ull;
u64 val_32 = val_64 & ((1ull << 32) - 1);
- u64 val_max_width = val_64 & ((1ul << eax.split.bit_width) - 1);
+ u64 val_max_width = val_64 & ((1ull << eax.split.bit_width) - 1);
int i;
/*
--
2.20.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [kvm-unit-tests PATCH] x86: pmu: fix failures on 32-bit due to wrong masks
2020-06-19 19:39 [kvm-unit-tests PATCH] x86: pmu: fix failures on 32-bit due to wrong masks Nadav Amit
@ 2020-06-25 16:33 ` Nadav Amit
2020-06-29 15:27 ` Paolo Bonzini
2020-06-25 16:41 ` Jim Mattson
1 sibling, 1 reply; 4+ messages in thread
From: Nadav Amit @ 2020-06-25 16:33 UTC (permalink / raw)
To: Paolo Bonzini; +Cc: kvm
> On Jun 19, 2020, at 12:39 PM, Nadav Amit <namit@vmware.com> wrote:
>
> Some mask computation are using long constants instead of long long
> constants, which causes test failures on x86-32.
>
> Signed-off-by: Nadav Amit <namit@vmware.com>
>
Paolo,
As you were so quick to respond to the other patches that I recent sent
(despite some defections), I presume you missed this one.
(Otherwise, no rush).
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [kvm-unit-tests PATCH] x86: pmu: fix failures on 32-bit due to wrong masks
2020-06-19 19:39 [kvm-unit-tests PATCH] x86: pmu: fix failures on 32-bit due to wrong masks Nadav Amit
2020-06-25 16:33 ` Nadav Amit
@ 2020-06-25 16:41 ` Jim Mattson
1 sibling, 0 replies; 4+ messages in thread
From: Jim Mattson @ 2020-06-25 16:41 UTC (permalink / raw)
To: Nadav Amit; +Cc: Paolo Bonzini, kvm list
On Fri, Jun 19, 2020 at 12:41 PM Nadav Amit <namit@vmware.com> wrote:
>
> Some mask computation are using long constants instead of long long
> constants, which causes test failures on x86-32.
>
> Signed-off-by: Nadav Amit <namit@vmware.com>
Reviewed-by: Jim Mattson <jmattson@google.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [kvm-unit-tests PATCH] x86: pmu: fix failures on 32-bit due to wrong masks
2020-06-25 16:33 ` Nadav Amit
@ 2020-06-29 15:27 ` Paolo Bonzini
0 siblings, 0 replies; 4+ messages in thread
From: Paolo Bonzini @ 2020-06-29 15:27 UTC (permalink / raw)
To: Nadav Amit; +Cc: kvm
On 25/06/20 18:33, Nadav Amit wrote:
>> On Jun 19, 2020, at 12:39 PM, Nadav Amit <namit@vmware.com> wrote:
>>
>> Some mask computation are using long constants instead of long long
>> constants, which causes test failures on x86-32.
>>
>> Signed-off-by: Nadav Amit <namit@vmware.com>
>>
>
> Paolo,
>
> As you were so quick to respond to the other patches that I recent sent
> (despite some defections), I presume you missed this one.
>
> (Otherwise, no rush).
>
Yes, indeed. Applied now, thanks!
Paolo
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-06-29 19:29 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-19 19:39 [kvm-unit-tests PATCH] x86: pmu: fix failures on 32-bit due to wrong masks Nadav Amit
2020-06-25 16:33 ` Nadav Amit
2020-06-29 15:27 ` Paolo Bonzini
2020-06-25 16:41 ` Jim Mattson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).