[Skiboot] [PATCH v8 1/2] phb4: set PHB CMPM registers for tunneled operations

Stewart Smith stewart at linux.vnet.ibm.com
Fri Mar 2 15:03:51 AEDT 2018


Philippe Bergheaud <felix at linux.vnet.ibm.com> writes:
> P9 supports PCI tunneled operations (atomics and as_notify) that require
> setting the PHB ASN Compare/Mask register with a 16-bit indication.
>
> This register is currently initialized by enable_capi_mode(). But, as
> tunneled operations may also work in PCI mode, the ASN Compare/Mask
> register should rather be initialized in phb4_init_ioda3().
>
> This patch also adds "ibm,phb-indications" to the device tree, to tell
> Linux the values of CAPI, ASN, and NBW indications, when supported.
>
> Tunneled operations tested by IBM in CAPI mode, by Mellanox Technologies
> in PCI mode.
>
> Signed-off-by: Philippe Bergheaud <felix at linux.vnet.ibm.com>
> Reviewed-by: Frederic Barrat <fbarrat at linux.vnet.ibm.com>
> ---
> Changelog:
>
> v2: Do not let Linux choose indication values.
>     Let skiboot tell Linux via the device tree.
>     Initialize the PHB ASN indication in PCI mode.
>
> v3: No functional change.
>     Drop cosmetic fixes in comments.
>
> v4: No change.
>
> v5: Set bit 59 of ASN indication and mask, as required for as_notify,
>     in order to prevent address translation on lpid,pid,tid messages,
>     that are not real addresses.
>
> v6: No change.
>
> v7: Explain those indicator and mask values, with comments kindly
>     supplied by Frederic Barrat.
>     Remove tests for dd1 in new code, as nobody in the capi galaxy
>     is using dd1 any more.
>
> v8: No change.
> ---
>  hw/phb4.c           | 49 ++++++++++++++++++++++++++++++++++++++++++-------
>  include/phb4-regs.h |  4 ++--
>  2 files changed, 44 insertions(+), 9 deletions(-)

Thanks! Series merged to master as of 0f3584d846629d5bc1832a1dc4663a1befe8c4b0

-- 
Stewart Smith
OPAL Architect, IBM.



More information about the Skiboot mailing list