[OpenPower-Firmware] skiboot 5.9.5 released
Stewart Smith
stewart at linux.vnet.ibm.com
Wed Dec 13 16:05:21 AEDT 2017
skiboot-5.9.5
*************
skiboot 5.9.5 was released on Wednesday December 13th, 2017. It
replaces skiboot-5.9.4 as the current stable release in the 5.9.x
series.
Over skiboot-5.9.4, we have a few bug fixes, they are:
* Fix *extremely* rare race in timer code.
* xive: Ensure VC informational FIRs are masked
Some HostBoot versions leave those as checkstop, they are harmless
and can sometimes occur during normal operations.
* xive: Fix occasional VC checkstops in xive_reset
The current workaround for the scrub bug described in
__xive_cache_scrub() has an issue in that it can leave dirty invalid
entries in the cache.
When cleaning up EQs or VPs during reset, if we then remove the
underlying indirect page for these entries, the XIVE will checkstop
when trying to flush them out of the cache.
This replaces the existing workaround with a new pair of workarounds
for VPs and EQs:
* The VP one does the dummy watch on another entry than the one we
scrubbed (which does the job of pushing old stores out) using an
entry that is known to be backed by a permanent indirect page.
* The EQ one switches to a more efficient workaround which
consists of doing a non-side-effect ESB load from the EQ’s ESe
control bits.
* io: Add load_wait() helper
This uses the standard form twi/isync pair to ensure a load is
consumed by the core before continuing. This can be necessary under
some circumstances for example when having the following sequence:
* Store reg A
* Load reg A (ensure above store pushed out)
* delay loop
* Store reg A
IE, a mandatory delay between 2 stores. In theory the first store is
only guaranteed to rach the device after the load from the same
location has completed. However the processor will start executing
the delay loop without waiting for the return value from the load.
This construct enforces that the delay loop isn’t executed until the
load value has been returned.
* xive: Do not return a trigger page for an escalation interrupt
This is bogus, we don’t support them. (Thankfully the callers didn’t
actually try to use this on escalation interrupts).
* xive: Mark a freed IRQ’s IVE as valid and masked
Removing the valid bit means a FIR will trip if it’s accessed
inadvertently. Under some circumstances, the XIVE will speculatively
access an IVE for a masked interrupt and trip it. So make sure that
freed entries are still marked valid (but masked).
* hw/nx: Fix NX BAR assignments
The NX rng BAR is used by each core to source random numbers for the
DARN instruction. Currently we configure each core to use the NX rng
of the chip that it exists on. Unfortunately, the NX can be
deconfigured by hostboot and in this case we need to use the NX of a
different chip.
This patch moves the BAR assignments for the NX into the normal nx-
rng init path. This lets us check if the normal (chip local) NX is
active when configuring which NX a core should use so that we can
fallback gracefully.
--
Stewart Smith
OPAL Architect, IBM.
More information about the OpenPower-Firmware
mailing list