openbmc.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: "Velumani T-ERS,HCLTech" <velumanit@hcl.com>
To: Patrick Williams <patrick@stwcx.xyz>,
	Richard Hanley <rhanley@google.com>
Cc: "openbmc@lists.ozlabs.org" <openbmc@lists.ozlabs.org>,
	Patrick Williams <patrickw3@fb.com>
Subject: RE: adding sync method in phosphor-time-manager
Date: Wed, 7 Oct 2020 17:18:06 +0000	[thread overview]
Message-ID: <PU1PR04MB22481EAF83C8BD20C42B4799A70A0@PU1PR04MB2248.apcprd04.prod.outlook.com> (raw)
In-Reply-To: <20201007013824.GF6152@heinlein>

Classification: HCL Internal

Time owner module was a different module which decides who owns the time BMC/host and set the time using NTP mode or Manual Mode.
in Time owner module ,user set the host time using the below command manually by providing  the time.
busctl set-property xyz.openbmc_project.Time.Manager \
       /xyz/openbmc_project/time/host xyz.openbmc_project.Time.EpochTime \
                   Elapsed t <value-in-microseconds>

we are now actually not repeating the time owner implementation instead we are adding one more mode(HostSync) to time manager setting.
    a.NTP
    b.Manual
    c.HostSync

    How HostSync intent to work:
                1.host sync mode will get time from the host.
                   1.1 Add an empty system service file in this repo and spawn this service when HostSync enabled
                   1.2 This service file can be override by the platform machine layer.
                   1.3 The platform machine layer install the service file and the shell script to get the time from host.
                 2.it will set the time to BMC using the shell script.
Regards,
Velu
-----Original Message-----
From: Patrick Williams <patrick@stwcx.xyz>
Sent: Wednesday, October 7, 2020 7:08 AM
To: Richard Hanley <rhanley@google.com>
Cc: Velumani T-ERS,HCLTech <velumanit@hcl.com>; openbmc@lists.ozlabs.org; Patrick Williams <patrickw3@fb.com>
Subject: Re: adding sync method in phosphor-time-manager

On Tue, Oct 06, 2020 at 02:41:29PM -0700, Richard Hanley wrote:
> While we're on the subject, has anyone ever taken a look at using
> roughtime on a BMC? I imagine it could a really valuable extension to
> phosphor-time-manager some time in the future.
> https://blog.cloudflare.com/roughtime/

One of the problems I see is that there are a few different proposals for "next generation time" protocols beyond NTP.  Unless a project member has an explicit requirement for one of them, it doesn't seem like we should go out of our way to pick one of them.  If someone is especially interested in implementing one of them in their spare time though, they're more than welcome.

--
Patrick Williams
::DISCLAIMER::
________________________________
The contents of this e-mail and any attachment(s) are confidential and intended for the named recipient(s) only. E-mail transmission is not guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or may contain viruses in transmission. The e mail and its contents (with or without referred errors) shall therefore not attach any liability on the originator or HCL or its affiliates. Views or opinions, if any, presented in this email are solely those of the author and may not necessarily reflect the views or opinions of HCL or its affiliates. Any form of reproduction, dissemination, copying, disclosure, modification, distribution and / or publication of this message without the prior written consent of authorized representative of HCL is strictly prohibited. If you have received this email in error please delete it and notify the sender immediately. Before opening any email and/or attachments, please check them for viruses and other defects.
________________________________

  reply	other threads:[~2020-10-07 17:19 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-06 17:38 adding sync method in phosphor-time-manager Velumani T-ERS,HCLTech
2020-10-06 21:41 ` Richard Hanley
2020-10-07  1:38   ` Patrick Williams
2020-10-07 17:18     ` Velumani T-ERS,HCLTech [this message]
2020-10-07  1:34 ` Patrick Williams
2021-04-08 18:08   ` Alexander Amelkin
2021-04-08 19:36     ` Ed Tanous
2021-04-09  7:26     ` Lei Yu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=PU1PR04MB22481EAF83C8BD20C42B4799A70A0@PU1PR04MB2248.apcprd04.prod.outlook.com \
    --to=velumanit@hcl.com \
    --cc=openbmc@lists.ozlabs.org \
    --cc=patrick@stwcx.xyz \
    --cc=patrickw3@fb.com \
    --cc=rhanley@google.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).