<div dir="ltr">In the future, please just send the text inline instead of as an attachment.<div><br></div><div>I don't understand the setting hierarchy.  To me, /org/openbmc/settings/Host/host0 implies I am modifying something related to the host CPU, not the BMC.  TimeMode only applies to the BMC so having it under host0 feels weird.</div><div><br></div><div>1. I like the clear separation of NTP vs manual.</div><div><br></div><div>2.1. "Both" seems like what we have today which doesn't really work at all.  I suggest omitting it.  "Split" feels like it should work for all cases.  If TimeMode is NTP, an offset is recorded.  If TimeMode is Manual, the BMC time is set.</div><div><br></div><div>APIs:</div><div>1. What time zone does SetTime assume?  If UTC, make sure to add tests for a valid leap second and leap year.</div><div>2. I really dislike APIs that change behavior.  Provide separate GetBmcTime and GetHostTime APIs if you must.</div><div>3. SetNTP is a very limiting name.  if this really changes TimeMode, call it SetTimeMode.  That way we can support things like 1588, GPS, etc later.</div><div><br></div><div>Changes to NetworkManager:</div><div>- I don't see the point of UseNTP for SetDHCP.  Configuring an NTP address is different from using NTP as a time source.  It's up to the DHCP server to provide NTP options.  Whether the BMC uses them is controlled by TimeMode.</div><div>- Add a SetNtpServer API instead of adding to SetAddress4.  NTP is entirely separate from IPv4 address configuration.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 18, 2016 at 4:08 AM, vishwa <span dir="ltr"><<a href="mailto:vishwa@linux.vnet.ibm.com" target="_blank">vishwa@linux.vnet.ibm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Team,<br>
<br>
Please help look into this document that describes what I think the TimeManager on openBMC systems should look like.<br>
<br>
Please weigh in your thoughts.<br>
<br>
Thanks.<br>
<br>
Vishwanath.<br>
<br>______________________________<wbr>_________________<br>
openbmc mailing list<br>
<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a><br>
<a href="https://lists.ozlabs.org/listinfo/openbmc" rel="noreferrer" target="_blank">https://lists.ozlabs.org/<wbr>listinfo/openbmc</a><br>
<br></blockquote></div><br></div>