Re: RFC: new design of phosphor-time-manager on sdbusplus

[Patrick Williams <patrick@stwcx.xyz>]

[-- Attachment #1: Type: text/plain, Size: 925 bytes --]

On Thu, Jan 19, 2017 at 05:54:02PM +0530, vishwa wrote:
 
> I thought and partially agree to their concern. For changing the owner 
> *away from SPLIT*, we can actually consume that readily. Its only the 
> change * TO SPLIT* that needs to delayed until off.

Actually the opposite.  You can safely switch to split while running,
but switch from split or host needs to be deferred.

> >      2. If the process restarts we need it to go back into the "current
> >         state" and not the "requested state".  How do we make this
> >         happen?  The current implementation might also have a flaw here
> >         so maybe we log it as an issue for follow up.
> 
> 
> The current implementation only sets the values based on what was set 
> prior to reset.
> It does that by consuming the saved data in /var/lib/obmc/saved_*. So 
> there is not an issue there.

Sounds good.

-- 
Patrick Williams

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 801 bytes --]