[Skiboot] [PATCH v5 5/5] SBE: Add timer support

Benjamin Herrenschmidt benh at kernel.crashing.org
Tue Jul 25 07:00:40 AEST 2017


On Mon, 2017-07-24 at 19:21 +0530, Vasant Hegde wrote:
> 
> > Can't they fix their interface to match what we need instead ?
> 
> We had asked them .. but they told they are not supporting multiple timer.
> 
> So if new timer request is < inflight timer then we have to cancel and schedule 
> new timer.

Why can't they have a "modify timer" request instead of requiring us to
send 2 requests ? That sounds brain dead. We should push, it's a
requirement from us. After all, afaik, we are the only user of that
facility.
> 
> > 
> > >   - SBE expects timeout value in milliseconds. We track timeout value in TB.
> > >     Hence we convert tb to milliseconds before sending request to SBE.
> > 
> > This is WAYYYYYY to coarse resolution. We want microseconds. They need
> > to fix their stuff.
> 
> We will check with FW folks on this.

Ben.



More information about the Skiboot mailing list