skiboot-5.5.0-rc1

skiboot-5.5.0-rc1 was released on Tuesday March 28th 2017. It is the first release candidate of skiboot 5.5, which will become the new stable release of skiboot following the 5.4 release, first released November 11th 2016.

skiboot-5.5.0-rc1 contains all bug fixes as of skiboot-5.4.3 and skiboot-5.1.19 (the currently maintained stable releases).

For how the skiboot stable releases work, see Skiboot stable tree rules and releases for details.

The current plan is to cut the final 5.5.0 by April 8th, with skiboot 5.5.0 being for all POWER8 and POWER9 platforms in op-build v1.16 (Due April 12th). This is a short cycle as this release is mainly targetted towards POWER9 bringup efforts.

Following skiboot-5.5.0, we will move to a regular six week release cycle, similar to op-build, but slightly offset to allow for a short stabilisation period. Expected release dates and contents are tracked using GitHub milestone and issues: https://github.com/open-power/skiboot/milestones

Over skiboot-5.4, we have the following changes:

New Platforms

  • SuperMicro’s (SMC) P8DNU: An astbmc based POWER8 platform

  • Add a generic platform to help with bringup of new systems.

  • Four POWER9 based systems (NOTE: All POWER9 systems should be considered for bringup use only at this point):

    • Romulus

    • Witherspoon (a POWER9 system with NVLink2 attached GPUs)

    • Zaius (OpenCompute platform, also known as “Barreleye 2”)

    • ZZ (FSP based system)

New features

  • System reset IPI facility and Mambo implementation Add an opal call OPAL_SIGNAL_SYSTEM_RESET which allows system reset exceptions to be raised on other CPUs and act as an NMI IPI. There is an initial simple Mambo implementation, but allowances are made for a more complex hardware implementation.

    The Mambo implementation is based on the RFC implementation for POWER8 hardware (see https://patchwork.ozlabs.org/patch/694794/) which we hope makes it into a future release.

    This implements an in-band NMI equivalent.

  • add CONTRIBUTING.md, ensuring that people new to the project have a one-stop place to find out how to get started.

  • interrupts: Add optional name for OPAL interrupts

    This adds the infrastructure for an interrupt source to provide a name for an interrupt directed toward OPAL. Those names will be put into an “opal-interrupts-names” property which is a standard DT string list corresponding 1:1 with the “opal-interrupts” property. PSI interrupts get names, and this is visible in Linux through /proc/interrupts

  • platform: add OPAL_REBOOT_FULL_IPL reboot type

    There may be circumstances in which a user wants to force a full IPL reboot rather than using fast reboot. Add a new reboot type, OPAL_REBOOT_FULL_IPL, that disables fast reboot. On platforms which don’t support fast reboot, this will be equivalent to a normal reboot.

  • phb3: Trick to allow control of the PCIe link width and speed

    This implements a hook inside OPAL that catches 16 and 32 bit writes to the link status register of the PHB.

    It allows you to write a new speed or a new width, and OPAL will then cause the PHB to renegociate.

    Example:

    First read the link status on PHB4:

    setpci -s 0004:00:00.0 0x5a.w
    a103
    

    It’s at x16 Gen3 speed (8GT/s)

    bits 0x0ff0 are the width and 0x000f the speed. The width can be 1 to 16 and the speed 1 to 3 (2.5, 5 and 8GT/s)

    Then try to bring it down to 1x Gen1 :

    setpci -s 0004:00:00.0 0x5a.w=0xa011
    

    Observe the result in the PHB:

    / # lspci -s 0004:00:00.0 -vv
    0004:00:00.0 PCI bridge: IBM Device 03dc (prog-if 00 [Normal decode])
    .../...
    LnkSta: Speed 2.5GT/s, Width x1, TrErr- Train- SlotClk- DLActive+ BWMgmt- ABWMgmt+
    

    And in the device:

    / # lspci -s 0004:01:00.0 -vv
    .../...
    LnkSta: Speed 2.5GT/s, Width x1, TrErr- Train- SlotClk+ DLActive- BWMgmt- ABWMgmt-
    
  • core/init: Add hdat-map property to OPAL node.

    Exports the HDAT heap to the OS. This allows the OS to view the HDAT heap directly. This allows us to view the HDAT area without having to use getmemproc.

  • Add a generic platform: If /bmc in device tree, attempt to init one For the most part, this gets us somewhere on some OpenPOWER systems before there’s a platform file for that machine.

    Useful in bringup only, and marked as such with scary looking log messages.

Core

  • asm: Don’t try to set LPCR:LPES1 on P8 and P9, the bit doesn’t exist.

  • pci: Add a framework for quirks

    In future we may want to be able to do fixups for specific PCI devices in skiboot, so add a small framework for doing this.

    This is not intended for the same purposes as quirks in the Linux kernel, as the PCI devices that quirks can match for in skiboot are not properly configured. This is intended to enable having a custom path to make changes that don’t directly interact with the PCI device, for example adding device tree entries.

  • hw/slw: fix possible NULL dereference

  • slw: Print enabled stop states on boot

  • uart: Fix Linux pass-through policy, provide NVRAM override option

  • libc/stdio/vsnprintf.c: add explicit fallthrough, this silences a recent (GCC 7.x) warning

  • init: print the FDT blob size in decimal

  • init: Print some more info before booting linux

    The kernel command line from nvram and the stdout-path are useful to know when debugging console related problems.

  • Makefile: Disable stack protector due to gcc problems

    Depending on how it was built, gcc will use the canary from a global (works for us) or from the TLS (doesn’t work for us and accesses random stuff instead).

    Fixing that would be tricky. There are talks of adding a gcc option to force use of globals, but in the meantime, disable the stack protector.

  • Stop using 3-operand cmp[l][i] for latest binutils Since a5721ba270, binutils does not support 3-operand cmp[l][i]. This adds (previously optional) parameter L.

  • buddy: Add a simple generic buddy allocator

  • stack: Don’t recurse into __stack_chk_fail

  • Makefile: Use -ffixed-r13 We use r13 for our own stuff, make sure it’s properly fixed

  • Always set ibm,occ-functional-state correctly

  • psi: fix the xive registers initialization on P8, which seems to be fine for real HW but causes a lof of pain under qemu

  • slw: Set PSSCR value for idle states

  • Limit number of “Poller recursion detected” errors to display

    In some error conditions, we could spiral out of control on this and spend all of our time printing the exact same backtrace.

    Limit it to 16 times, because 16 is a nice number.

  • slw: do SLW timer testing while holding xscom lock

    We add some routines that let a caller get the xscom lock once and then do a bunch of xscoms while holding it. In some situations without this, it could take long enough to get the xscom lock that the 1ms timeout would expire and we’d falsely think the SLW timer didn’t work when in fact it did.

  • wait_for_resource_loaded: don’t needlessly sleep for 5ms

  • run pollers in cpu_process_local_jobs() if running job synchonously

  • fsp: Don’t recurse pollers in ibm_fsp_terminate

  • chiptod: More hardening against -1 chip ID

  • interrupts: Rewrite/correct doc for opal_set/get_xive

  • cpu: Don’t enable nap mode/PM mode on non-P8

  • platform: Call generic platform probe and init UART there

  • psi: Don’t register more interrupts than the HW supports

  • psi: Add DT option to disable LPC interrupts

I2C and TPM

  • p8i2c: Use calculated poll_interval when booting OPAL Otherwise we’d default to 2seconds (TIMER_POLL) during boot on chips with a functional i2c interrupt, leading to slow i2c during boot (or hitting timeouts instead).

  • i2c: Add i2c_run_req() to crank the state machine for a request

  • tpm_i2c_nuvoton: work out the polling time using mftb()

  • tpm_i2c_nuvoton: handle errors after reading the tpm fifo

  • tpm_i2c_nuvoton: cleanup variables in tpm_read_fifo()

  • tpm_i2c_nuvoton: handle errors after writting the tpm fifo

  • tpm_i2c_nuvoton: cleanup variables in tpm_write_fifo()

  • tpm_i2c_nuvoton: handle errors after writing sts.commandReady in step 5

  • tpm_i2c_nuvoton: handle errors after writing sts.go

  • tpm_i2c_nuvoton: handle errors after checking the tpm fifo status

  • tpm_i2c_nuvoton: return burst_count in tpm_read_burst_count()

  • tpm_i2c_nuvoton: isolate the code that handles the TPM_TIMEOUT_D timeout

  • tpm_i2c_nuvoton: handle errors after reading sts.commandReady

  • tpm_i2c_nuvoton: add tpm_status_read_byte()

  • tpm_i2c_nuvoton: add tpm_check_status()

  • tpm_i2c_nuvoton: rename defines to shorter names

  • tpm_i2c_interface: decouple rc from being done with i2c request

  • tpm_i2c_interface: set timeout before each request

  • i2c: Add nuvoton quirk, disallowing i2cdetect as it locks TPM

    p8-i2c reset things manually in some error conditions

  • stb: create-container and wrap skiboot in Secure/Trusted Boot container

    We produce UNSIGNED skiboot.lid.stb and skiboot.lid.xz.stb as build artifacts.

    These are suitable blobs for flashing onto Trusted Boot enabled op-build builds WITH the secure boot jumpers ON (i.e. NOT in secure mode). It’s just enough of the Secure and Trusted Boot container format to make Hostboot behave.

PCI

  • core/pci: Support SRIOV VFs

    Currently, skiboot can’t see SRIOV VFs. It introduces some troubles as I can see: The device initialization logic (phb->ops->device_init()) isn’t applied to VFs, meaning we have to maintain same and duplicated mechanism in kernel for VFs only. It introduces difficulty to code maintaining and prone to lose sychronization.

    This was motivated by bug reported by Carol: The VF’s Max Payload Size (MPS) isn’t matched with PF’s on Mellanox’s adapter even kernel tried to make them same. It’s caused by readonly PCIECAP_EXP_DEVCTL register on VFs. The skiboot would be best place to emulate this bits to eliminate the gap as I can see.

    This supports SRIOV VFs. When the PF’s SRIOV capability is populated, the number of maximal VFs (struct pci_device) are instanciated, but but not usable yet. In the mean while, PCI config register filter is registered against PCIECAP_SRIOV_CTRL_VFE to capture the event of enabling or disabling VFs. The VFs are initialized, put into the PF’s children list (pd->children), populate its PCI capabilities, and register PCI config register filter against PCICAP_EXP_DEVCTL. The filter’s handler caches what is written to MPS field and returns the cached value on read, to eliminate the gap mentioned as above.

  • core/pci: Avoid hreset after freset

    Commit 5ac71c9 (“pci: Avoid hot resets at boot time”) missed to avoid hot reset after fundamental reset for PCIe common slots.

    This fixes it.

  • core/pci: Enforce polling PCIe link in hot-add path

    In surprise hot-add path, the power state isn’t changed on hardware. Instead, we set the cached power state (@slot->power_state) and return OPAL_SUCCESS. The upper layer starts the PCI probing immediately when receiving OPAL_SUCCESS. However, the PCIe link behind the PCI slot is likely down. Nothing will be probed from the PCI slot even we do have PCI adpater connected to the slot.

    This fixes the issue by returning OPAL_ASYNC_COMPLETION to force upper layer to poll the PCIe link before probing the PCI devices behind the slot in surprise and managed hot-add paths.

  • hw/phb3: fix error handling in complete reset

    During a complete reset, when we get a timeout waiting for pending transaction in state PHB3_STATE_CRESET_WAIT_CQ, we mark the PHB as permanently broken.

    Set the state to PHB3_STATE_FENCED so that the kernel can retry the complete reset.

  • phb3: Lock the PHB on set_xive callbacks

p8dnu platform

  • astbmc/p8dnu: Enable PCI slot’s power supply on PEX9733 in hot-add path

  • astbmc/p8dnu: Enable PCI slot’s power supply on PEX8718 in hot-add path

  • core/pci: Mark broken PDC on slots without surprise hotplug capability

    We has to support surprise hotplug on PCI slots that don’t support it on hardware. So we’re fully utilizing the PCIe link state change event to detect the events (hot-remove and hot-add). The PDC (Presence Detection Change) event isn’t reliable for the purpose. For example, PEX8718 on superMicro’s machines.

    This adds another PCI slot property “ibm,slot-broken-pdc” in the device-tree, to indicate the PDC isn’t reliable on those (software claimed) surprise pluggable slots.

  • core/pci: Fix PCIe slot’s presence

    According to PCIe spec, the presence bit is hardcoded to 1 if PCIe switch downstream port doesn’t support slot capability. The register used for the check in pcie_slot_get_presence_state() is wrong. It should be PCIe capability register instead of PCIe slot capability register. Otherwise, we always have present bit on the PCI topology. The issue is found on Supermicro’s p8dtu2u machine:

     # lspci -t
     -+-[0022:00]---00.0-[01-08]----00.0-[02-08]--+-01.0-[03]----00.0
      |                                           \-02.0-[04-08]--
     # cat /sys/bus/pci/slots/S002204/adapter
     1
     # lspci -vvs 0022:02:02.0
     # lspci -vvs 0022:02:02.0
     0022:02:02.0 PCI bridge: PLX Technology, Inc. PEX 8718 16-Lane, \
     5-Port PCI Express Gen 3 (8.0 GT/s) Switch (rev ab) (prog-if 00 [Normal decode])
        :
     Capabilities: [68] Express (v2) Downstream Port (Slot+), MSI 00
        :
        SltSta:    Status: AttnBtn- PowerFlt- MRL- CmdCplt- PresDet- Interlock-
                   Changed: MRL- PresDet- LinkState-
    
    This fixes the issue by checking the correct register (PCIe capability).
    Also, the register's value is cached in advance as we did for slot and
    link capability.
    
  • core/pci: More reliable way to update PCI slot power state

    The power control bit (SLOT_CTL, offset: PCIe cap + 0x18) isn’t reliable enough to reflect the PCI slot’s power state. Instead, the power indication bits are more reliable comparatively. This leads to mismatch between the cached power state and PCI slot’s presence state, resulting in the hotplug driver in kernel refuses to unplug the devices properly on the request. The issue was found on below NVMe card on “supermicro,p8dtu2u” machine. We don’t have this issue on the integrated PLX 8718 switch.

     # lspci
     0022:01:00.0 PCI bridge: PLX Technology, Inc. PEX 9733 33-lane, \
                  9-port PCI Express Gen 3 (8.0 GT/s) Switch (rev aa)
     0022:02:01.0 PCI bridge: PLX Technology, Inc. PEX 9733 33-lane, \
                  9-port PCI Express Gen 3 (8.0 GT/s) Switch (rev aa)
     0022:02:04.0 PCI bridge: PLX Technology, Inc. PEX 9733 33-lane, \
                  9-port PCI Express Gen 3 (8.0 GT/s) Switch (rev aa)
     0022:02:05.0 PCI bridge: PLX Technology, Inc. PEX 9733 33-lane, \
                  9-port PCI Express Gen 3 (8.0 GT/s) Switch (rev aa)
     0022:02:06.0 PCI bridge: PLX Technology, Inc. PEX 9733 33-lane, \
                  9-port PCI Express Gen 3 (8.0 GT/s) Switch (rev aa)
     0022:02:07.0 PCI bridge: PLX Technology, Inc. PEX 9733 33-lane, \
                  9-port PCI Express Gen 3 (8.0 GT/s) Switch (rev aa)
     0022:17:00.0 Non-Volatile memory controller: Device 19e5:0123 (rev 45)
    
    This updates the cached PCI slot's power state using the power
    indication bits instead of power control bit, to fix above issue.
    

Utilities

  • opal-prd: Direct systemd to always restart opal-prd Always restart the opal-prd daemon, irrespective of why it stopped.

  • external/ffspart: Simple C program to be able to make an FFS partition

  • getscom: Add chip info for P9.

  • gard: Fix make dist target

  • pflash/libflash: arch_flash_arm: Don’t assume mtd labels are short

libffs

  • libffs: Understand how to create FFS partition TOCs and entries.

BMC Based systems

  • platforms/astbmc: Support PCI slots for palmetto

  • habanero/slottable: Remove Network Mezz(2, 0) from PHB1.

  • BMC/PCI: Check slot tables against detected devices On BMC machines, we have slot tables of built in PHBs, slots and devices that are physically present in the system (such as the BMC itself). We can use these tables to check what we detected against what should be in the system and throw an error if they differ.

    We have seen this occur a couple of times while still booting, giving the user just an empty petitboot screen and not much else to go on. This patch helps in that we get a skiboot error message, and at some point in the future when we pump them up to the OS we could get a big friendly error message telling you you’re having a bad day.

  • pci/quirk: Populate device tree for AST2400 VGA

    Adding these properties enables the kernel to function in the same way that it would if it could no longer access BMC configuration registers through a backdoor, which may become the default in future.

    The comments describe how isolating the host from the BMC could be achieved in skiboot, assuming all kernels that the system boots support this. Isolating the BMC and the host from each other is important if they are owned by different parties; for example, a cloud provider renting machines “bare metal”.

  • astbmc/pnor: Use mbox-flash for flash accesses

    If the BMC is MBOX protocol aware, request flash reads/writes over the MBOX regs. This inits the blocklevel for pnor access with mbox-flash.

  • ast: Account for differences between 2400 vs 2500

  • platform: set default bmc_platform The bmc_platform pointer is set to NULL by default and on non-AMI BMC platforms. As a result a few places in hw/ipmi/ipmi-sel.c will blindly dereference a NULL pointer.

POWER9

  • external: Update xscom utils for type 1 indirect accesses

  • xscom: Harden indirect writes

  • xscom: Add POWER9 scom reset

  • homer : Enable HOMER region reservation for POWER9

  • slw: Define stop idle states for P9 DD1

  • slw: Fix parsing of supported STOP states

  • slw: only enable supported STOP states

  • dts: add support for p9 cores

  • asm: Add POWER9 case to init_shared_sprs

    For now, setup the HID and HMEER. We’ll add more as we get good default values from HW.

  • xive/psi/lpc: Handle proper clearing of LPC SerIRQ latch on POWER9 DD1

  • lpc: Mark the power9 LPC bus as compatible with power8

  • Fix typo in PIR mask for POWER9. Fixes booting multi-chip.

  • vpd: add vpd_valid() to check keyword VPD blobs

    Adds a function to check whether a blob is a valid IBM ASCII keyword VPD blob. This allows us to recognise when we do and do not have a VPD blob and act accordingly.

  • core/cpu.c: Use a device-tree node to detect nest mmu presence The nest mmu address scom was hardcoded which could lead to boot failure on POWER9 systems without a nest mmu. For example Mambo doesn’t model the nest mmu which results in failure when calling opal_nmmu_set_ptcr() during kernel load.

  • psi: Fix P9 BAR setup on multi-chips

PHB4:

  • phb4: Fix TVE encoding for start address

  • phb4: Always assign powerbus BARs

    HostBoot configure them with weird values that confuse us, instead let’s just own the assignment. This is temporary, I will centralize memory map management next but this gets us going.

  • phb4: Fix endian issue with link control2/status2 registers Fixes training at larger than PCIe Gen1 speeds.

  • phb4: Add ability to log config space access Useful for debugging

  • phb4: Change debug prints Currently we print “PHB4” and mean either “PHB version 4” or “PHB number 4” which can be quite confusing.

  • phb4: Fix config space enable bits on DD1

  • phb4: Fix location of EEH enable bits

  • phb4: Fix setting of max link speed

  • phb4: Updated inits as of PHB4 spec 0.52

HDAT fixes:

  • hdat: Parse BMC nodes much earlier

    This moves the parsing of the BMC and LPC details to the start of the HDAT parsing. This allows us to enable the Skiboot log console earlier so we can get debug output while parsing the rest of the HDAT.

  • astbmc: Don’t do P8 PSI or DT fixups on P9

    Previously the HDAT format was only ever used with IBM hardware so it would store vital product data (VPD) blobs in the IBM ASCII Keyword VPD format. With P9 HDAT is used on OpenPower machines which use Industry Standard DIMMs that provide their product data through a “Serial Present Detect” EEPROM mounted on the DIMM.

    The SPD blob has a different format and is exported in the device-tree under the “spd” property rather than the “ibm,vpd” property. This patch adds support for recognising these blobs and placing them in the appropriate DT property.

  • hdat: Add __packed to all HDAT structures and workaround HB reserve

    Some HDAT structures aren’t properly aligned. We were using __packed on some but not others and got at least one wrong (HB reserve). This adds it everywhere to avoid such problems.

    However this then triggers another problem where HB gives us a crazy range (0.256M) to reserve with no label, which triggers an assertion failure later on in mem_regions.c.

    So also add a test to skip any region starting at 0 until we can undertand that better and have it fixed one way or another.

  • hdat: Ignore broken memory reserves

    Ignore HDAT memory reserves > 512MB. These are considered bogus and workaround known HDAT bugs.

  • hdat: Add BMC device-tree node for P9 OpenPOWER systems

  • hdat: Fix interrupt & device_type of UART node

    The interrupt should use a standard “interrupts” property. The UART node also need a device_type=”serial” property for historical reasons otherwise Linux won’t pick it up.

  • parse and export STOP levels

  • add new sppcrd_chip_info fields

  • add radix-AP-encodings

  • stop using proc_int_line in favor of pir

  • rename add_icp() to add_xics_icp()

  • Add support for PHB4

  • create XIVE nodes under each xscom node

  • Add P9 compatible property

  • Parse hostboot memory reservations from HDAT

  • Add new fields to IPL params structure and update sys family for p9.

  • Fix ibm,pa-features for all CPU types

  • Fix XSCOM nodes for P9

  • Remove deprecated ‘ibm, mem-interleave-scope’ from DT on POWER9

  • Grab system model name from HDAT when available

  • Grab vendor information from HDAT when available

  • SPIRA-H/S changes for P9

  • Add BMC and LPC IOPATH support

  • handle ISDIMM SPD blobs

  • make HDIF_child() print more useful errors

  • Add PSI HB xscom details

  • Add new fields to proc_init_data structure

  • Add processor version check for hs service ntuple

  • add_iplparams_serial - Validate HDIF_get_iarray_size() return value

XIVE:

The list of XIVE fixes and updates is extensive. Below is only a portion of the changes that have gone into skiboot 5.5.0-rc1 for the new XIVE hardware that is present in POWER9:

  • xive: Enable backlog on queues

  • xive: Use for_each_present_cpu() for setting up XIVE

  • xive: Fix logic in opal_xive_get_xirr()

  • xive: Properly initialize new VP and EQ structures

  • xive: Improve/fix EOI of LSIs

  • xive: Add FIXME comments about mask/umask races

  • xive: Fix memory barrier in opal_xive_get_xirr()

  • xive: Don’t try to find a target EQ for prio 0xff

  • xive: Bump table sizes in direct mode

  • xive: Properly register escalation interrupts

  • xive: Split the OPAL irq flags from the internal ones

  • xive: Don’t touch ESB masks unless masking/unmasking

  • xive: Fix xive_get_ir_targetting()

  • xive: Cleanup escalation PQ on queue change

  • xive: Add any chip for allocating interrupts

  • xive: Add chip_id to get_vp_info

  • xive: Add opal_xive_get/set_vp_info

  • xive: Add VP alloc/free OPAL functions

  • xive: Workaround for bad DD1 checker

  • xive: Add more checks for exploitation mode

  • xive: Add support for EOIs via OPAL

  • xive/phb4: Work around broken LSI control on P9 DD1

  • xive: Forward interrupt names callback

  • xive: Export opal_xive_reset() arguments in OPAL API

  • xive: Add interrupt allocator

  • xive: Implement xive_reset

  • xive: Don’t assert if xive_get_vp() fails

  • xive: Expose exploitation mode DT properties

  • xive: Use a constant for max# of chips

  • xive: Keep track of which interrupts were ever enabled In order to speed up xive reset

  • xive: Implement internal VP allocator

  • xive: Add xive_get/set_queue_info

  • xive: Add helpers to encode and decode VP numbers

  • xive: Add API to donate pages in indirect mode

  • xive: Add asynchronous cache updates and update irq targetting

  • xive: Split xive_provision_cpu() and use cache watch for VP

  • xive: Add cache scrub to push watch updates to memory

  • xive: Mark XIVE owned EQs with a specific flag

  • xive: Use an allocator for EQDs

  • xive: Break assumption that block ID == chip ID

  • xive/phb4: Handle bad ESB offsets in PHB4 DD1

  • xive: Implement get/set_irq_config APIs

  • xive: Rework xive_set_eq_info() to store all info even when masking

  • xive: Implement cache watch and use it for EQs

  • xive: Add locking to some API calls

  • xive: Add opal_xive_get_irq_info()

  • xive: Add CPU node “interrupts” properties representing the IPIs

  • xive: Add basic opal_xive_reset() call and exploitation mode

  • xive: Add support for escalation interrupts

  • xive: OPAL API update

  • xive: Add some dump facility for debugging

  • xive: Document exploitation mode (Pretty much work in progress)

  • xive: Indirect table entries must have top bits “type” set

  • xive: Remove unused field and clarify comment

  • xive: Provide a way to override some IPI sources

  • xive: Add helper to retrieve an IPI trigger port

  • xive: Fix IPI EOI logic in opal_xive_eoi()

  • xive: Don’t try to EOI a masked source

  • xive: Fix comments in xive_source_set_xive()

  • xive: Fix comments in xive_get_ive()

  • xive: Configure forwarding ports

  • xive: Fix mangling of interrupt server# in opal_get/set_xive()

  • xive: Fix interrupt number mangling

Fast-reboot

  • fast-reboot: creset PHBs on fast reboot On fast reboot, perform a creset of all PHBs. This ensures that any PHBs that are fenced will be working after the reboot.

  • fast-reboot: Enable fast reboot with CAPI adapters in CAPI mode CAPI mode is disabled as part of OPAL_SYNC_HOST_REBOOT.

  • opal/fast-reboot: set fw_progress sensor status with IPMI_FW_PCI_INIT.

CAPI

  • hmi: Print CAPP FIR information when handling CAPP malfunction alerts

FSP based systems

  • hw/fsp: Do not queue SP and SPCN class messages during reset/reload This could cause soft lockups if FSP reset reload was done while in OPAL During FSP R/R, the FSP is inaccessible and will lose state. Messages to the FSP are generally queued for sending later.

Tests

  • core/test/run-trace: Reduce number of samples when running under valgrind

    This reduces ‘make check’ run time by ~10 seconds on my laptop, and just the run-trace test itself takes 15 seconds less (under valgrind).

  • test/sreset_world: Kind of like Hello World, but from the SRESET vector. A regression test for the mambo implementation of OPAL_SIGNAL_SYSTEM_RESET.

  • nvram-format: Fix endian issues

    NVRAM formats are always BE, so let’s use the sparse annotation to catch any issues (and correct said issues).

    On LE platforms, the test was erroneously passing as with building the nvram-format code on LE we were produces an incorrect NVRAM image.

  • test/hello_world: use P9MAMBO to differentiate from P8

  • hdata_to_dt: Specify PVR on command line

  • hdata/test: Add DTS output for the test cases

  • hdata/test: strip blobs from the DT output

  • mambo: add mprintf()

    mprintf() is printf(), but it goes straight to the mambo console. This allows it to be independent of Skiboot’s actual console infrastructure so it can be used for debugging the console drivers and for debugging code that runs before the console is setup.

  • generate-fwts-olog: add support for parsing prerror()

  • Add bitmap test

    The worst test suite ever

  • mambo_utils: add ascii output to hexdump

  • mambo_utils: add p_str <addr> [limit]

  • mambo_utils: make p return a value

  • hello_world: print out full path of missing MAMBO_BINARY

  • print-stb-container: Fix build on centos7

  • Travis-ci improvements: - install expect on ubuntu 12.04, disable qemu on 16.04/latest - build and test more on centos7 - hello_world: run p9 mambo tests - install systemsim-p8 on centos7 - install systemsim-p8 on centos6 - install systemsim-p9 - enable fedora25 - always pull new docker image - add fedora rawhide

  • Add fwts annotation for duplicate DT node entries.

  • external/fwts: Add ‘last-tag’ to FWTS olog output This isn’t so useful at the moment, but this will make cleaning out crufty old error definitions much easier.

  • external/fwts: Add FWTS olog merge script A script to merge olog error definitions from multiple skiboot versions into a single olog JSON file. Will prompt when conflicting patterns are found to update the pattern, or add both.

  • mambo: fake NVRAM support

  • mambo: Add Fake NVRAM driver

  • external/mambo: add shortcut to print all GPRs

Contributors

Processed 363 csets from 28 developers. A total of 18105 lines added, 16499 removed (delta 1606)

Developers with the most changesets

Developer

#

%

Benjamin Herrenschmidt

138

(38.0%)

Stewart Smith

56

(15.4%)

Oliver O’Halloran

47

(12.9%)

Michael Neuling

18

(5.0%)

Gavin Shan

15

(4.1%)

Claudio Carvalho

14

(3.9%)

Vasant Hegde

11

(3.0%)

Cyril Bur

11

(3.0%)

Andrew Donnellan

11

(3.0%)

Ananth N Mavinakayanahalli

5

(1.4%)

Cédric Le Goater

5

(1.4%)

Pridhiviraj Paidipeddi

5

(1.4%)

Shilpasri G Bhat

4

(1.1%)

Nicholas Piggin

4

(1.1%)

Russell Currey

3

(0.8%)

Alistair Popple

2

(0.6%)

Jack Miller

2

(0.6%)

Chris Smart

2

(0.6%)

Matt Brown

1

(0.3%)

Michael Ellerman

1

(0.3%)

Frederic Barrat

1

(0.3%)

Hank Chang

1

(0.3%)

Willie Liauw

1

(0.3%)

Werner Fischer

1

(0.3%)

Jeremy Kerr

1

(0.3%)

Patrick Williams

1

(0.3%)

Joel Stanley

1

(0.3%)

Alexey Kardashevskiy

1

(0.3%)

Developers with the most changed lines

Developer

#

%

Oliver O’Halloran

17961

(56.7%)

Benjamin Herrenschmidt

5509

(17.4%)

Cyril Bur

2801

(8.8%)

Stewart Smith

1649

(5.2%)

Gavin Shan

653

(2.1%)

Claudio Carvalho

489

(1.5%)

Willie Liauw

361

(1.1%)

Ananth N Mavinakayanahalli

340

(1.1%)

Andrew Donnellan

315

(1.0%)

Michael Neuling

240

(0.8%)

Shilpasri G Bhat

228

(0.7%)

Nicholas Piggin

219

(0.7%)

Vasant Hegde

207

(0.7%)

Russell Currey

158

(0.5%)

Jack Miller

127

(0.4%)

Cédric Le Goater

126

(0.4%)

Chris Smart

95

(0.3%)

Hank Chang

56

(0.2%)

Pridhiviraj Paidipeddi

47

(0.1%)

Alistair Popple

39

(0.1%)

Matt Brown

29

(0.1%)

Michael Ellerman

3

(0.0%)

Alexey Kardashevskiy

2

(0.0%)

Frederic Barrat

1

(0.0%)

Werner Fischer

1

(0.0%)

Jeremy Kerr

1

(0.0%)

Patrick Williams

1

(0.0%)

Joel Stanley

1

(0.0%)

Developers with the most lines removed

Developer

#

%

Oliver O’Halloran

8810

(53.4%)

Ananth N Mavinakayanahalli

98

(0.6%)

Alistair Popple

9

(0.1%)

Michael Ellerman

3

(0.0%)

Werner Fischer

1

(0.0%)

Developers with the most signoffs

Total 322

Developer

#

%

Stewart Smith

307

(95.3%)

Michael Neuling

6

(1.9%)

Oliver O’Halloran

3

(0.9%)

Benjamin Herrenschmidt

2

(0.6%)

Vaidyanathan Srinivasan

1

(0.3%)

Hank Chang

1

(0.3%)

Jack Miller

1

(0.3%)

Gavin Shan

1

(0.3%)

Developers with the most reviews

Total: 45

Developer

#

%

Vasant Hegde

10

(22.2%)

Andrew Donnellan

9

(20.0%)

Russell Currey

6

(13.3%)

Cédric Le Goater

5

(11.1%)

Oliver O’Halloran

4

(8.9%)

Gavin Shan

3

(6.7%)

Vaidyanathan Srinivasan

2

(4.4%)

Alistair Popple

2

(4.4%)

Frederic Barrat

2

(4.4%)

Mahesh Salgaonkar

1

(2.2%)

Cyril Bur

1

(2.2%)

Developers with the most test credits

Total 11

Developer

#

%

Willie Liauw

4

(36.4%)

Claudio Carvalho

3

(27.3%)

Gavin Shan

1

(9.1%)

Michael Neuling

1

(9.1%)

Pridhiviraj Paidipeddi

1

(9.1%)

Chris Smart

1

(9.1%)

Developers who gave the most tested-by credits

Total 11

Developer

#

%

Gavin Shan

4

(36.4%)

Stewart Smith

4

(36.4%)

Chris Smart

1

(9.1%)

Oliver O’Halloran

1

(9.1%)

Ananth N Mavinakayanahalli

1

(9.1%)

Developers with the most report credits

Total 7

Developer

#

%

Hank Chang

4

(57.1%)

Guilherme G. Piccoli

1

(14.3%)

Colin Ian King

1

(14.3%)

Pradipta Ghosh

1

(14.3%)

Developers who gave the most report credits

Total 7

Developer

#

%

Gavin Shan

5

(71.4%)

Andrew Donnellan

1

(14.3%)

Jeremy Kerr

1

(14.3%)