[Skiboot] [PATCH 1/2] Revert "Revert "astbmc: Try IPMI HIOMAP for P8""

Andrew Jeffery andrew at aj.id.au
Mon Feb 18 18:05:52 AEDT 2019


On Mon, 18 Feb 2019, at 17:31, Oliver wrote:
> On Mon, Feb 18, 2019 at 5:58 PM Lei YU <mine260309 at gmail.com> wrote:
> >
> > On Mon, Feb 18, 2019 at 2:27 PM Andrew Jeffery <andrew at aj.id.au> wrote:
> > >
> > > On Mon, 18 Feb 2019, at 16:56, Lei YU wrote:
> > > > This reverts commit 23470f10d0b1e120dc2d2f1606444fb6fc07b506.
> > >
> > > NAK - it might work for Palmetto with OpenBMC, but it broke a bunch of
> > > AMI machines. I need to sort out what I did wrong.
> >
> > Oh I thought AMI BMC is ready to use mbox.
> > If it's not the case, then please hold the merge.
> 
> Is there any way to detect if mbox/hiomap is available at runtime?
> It'd be nice if we could keep older BMC firmware working with newer
> host firmware.

That's what the patch is doing, except just not very well. It seems to break
AMI machines that don't yet support hiomap (all of them). As mentioned,
I need to look into what's going wrong. It's next on my todo list.

Andrew


More information about the Skiboot mailing list