[Skiboot] [RFC PATCH v2 0/4] OpenBMC MBOX work

Cyril Bur cyril.bur at au1.ibm.com
Tue Jan 24 11:54:13 AEDT 2017


Hi,

On OpenBMC platforms we currently expect the BMC to have pointed the
LPC bus at the flash where we can do flash read/writes/erases. This
makes controlling access between the host and the BMC difficult.

A better strategy is to say that the BMC owns the flash and the host
must request access. Doing so instantly solves the concurrent access
problem but also allows for some abstraction of the flash by the BMC.
How this is going to work is that skiboot requests access to (a region
of) the flash and the BMC places it in its own RAM and points the LPC
bus at it. The current implementation on the BMC is a copy of the
flash into ram and a write back to the flash when needed but it could
be much better.

These patches attempt to achieve this by using the MBOX registers to
talk to the BMC and request the flash. Once requested the 'flash' can
just be read from the LPC bus with the regular accessors, same for
writing

There is now a reference implementation of the daemon which runs on
the BMC. This is the other side of the protocol used in 3/4. The daemon
will undergo changes but hopefully the core of the protocol will
remain unchanged. https://github.com/openbmc/mboxbridge

It may be worth noting here that the BMC communication is done
synchronously, not exactly fast. Future work may look at making this
interface asynchronous where it makes sense.

Thanks for the review,

Cyril

v2: Split patch 1 into two - the driver and using the driver.
    Check if we can use mbox from the device tree
    Removed a virtually unused queue - we can add it (better) when
    things become asynchronous
    Added checks for using the driver uninitialised
    Various cleanups
    Happy new year Joel


Cyril Bur (4):
  hw/lpc-mbox: Add skiboot driver for the BMC mbox regs
  ast-bmc: Use the mbox driver
  libflash: blocklevel backend for MBOX flash access
  astbmc/pnor: Use mbox-flash for flash accesses

 hw/Makefile.inc           |   2 +-
 hw/ast-bmc/ast-io.c       |  25 +++
 hw/lpc-mbox.c             | 290 ++++++++++++++++++++++++
 include/ast.h             |   4 +
 include/lpc-mbox.h        |  58 +++++
 include/skiboot.h         |   1 +
 libflash/Makefile.inc     |   2 +-
 libflash/mbox-flash.c     | 562 ++++++++++++++++++++++++++++++++++++++++++++++
 libflash/mbox-flash.h     |  24 ++
 platforms/astbmc/common.c |  38 ++++
 platforms/astbmc/pnor.c   |  48 ++--
 11 files changed, 1035 insertions(+), 19 deletions(-)
 create mode 100644 hw/lpc-mbox.c
 create mode 100644 include/lpc-mbox.h
 create mode 100644 libflash/mbox-flash.c
 create mode 100644 libflash/mbox-flash.h

-- 
2.11.0



More information about the Skiboot mailing list