[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 5/6] hw/ppc: Always store the decrementer value
From: |
Nicholas Piggin |
Subject: |
Re: [PATCH 5/6] hw/ppc: Always store the decrementer value |
Date: |
Sun, 30 Jul 2023 19:40:45 +1000 |
On Thu Jul 27, 2023 at 10:26 PM AEST, Cédric Le Goater wrote:
> Hello Nick,
>
> On 7/26/23 20:22, Nicholas Piggin wrote:
> > When writing a value to the decrementer that raises an exception, the
> > irq is raised, but the value is not stored so the store doesn't appear
> > to have changed the register when it is read again.
> >
> > Always store the write value to the register.
>
> This change has a serious performance impact when a guest is run under
> PowerNV. Could you please take a look ?
Yeah, the decrementer load doesn't sign-extend the value correctly as
it should for the large-decrementer option. It makes skiboot detect
the decrementer size as 64 bits instead of 56, and things go bad from
there. KVM seems more affected because it's saving and restoring DEC
frequently.
The fix seems simple but considering the compounding series of bugs
and issues coming up with this, I think it will be better to defer
the decrementer work until 8.2 unfortunately.
Thanks,
Nick
> Thanks,
>
> C.
>
> PS: We should really introduce avocado tests for nested.
Yeah agreed. Both for pseries and powernv, ideally.
Thanks,
Nick
- [PATCH 0/6] ppc fixes possibly for 8.1, Nicholas Piggin, 2023/07/26
- [PATCH 1/6] target/ppc: Implement ASDR register for ISA v3.0 for HPT, Nicholas Piggin, 2023/07/26
- [PATCH 2/6] target/ppc: Fix VRMA page size for ISA v3.0, Nicholas Piggin, 2023/07/26
- [PATCH 3/6] target/ppc: Fix pending HDEC when entering PM state, Nicholas Piggin, 2023/07/26
- [PATCH 4/6] hw/ppc: Avoid decrementer rounding errors, Nicholas Piggin, 2023/07/26
- [PATCH 5/6] hw/ppc: Always store the decrementer value, Nicholas Piggin, 2023/07/26
- [PATCH 6/6] target/ppc: Migrate DECR SPR, Nicholas Piggin, 2023/07/26
- Re: [PATCH 0/6] ppc fixes possibly for 8.1, Daniel Henrique Barboza, 2023/07/28