Send connman mailing list submissions to
        connman@lists.01.org

To subscribe or unsubscribe via email, send a message with subject or
body 'help' to
        connman-requ...@lists.01.org

You can reach the person managing the list at
        connman-ow...@lists.01.org

When replying, please edit your Subject line so it is more specific
than "Re: Contents of connman digest..."

Today's Topics:

   1. RE: [PATCH] timeserver: Reset synchro on system timeserver update
      (VAUTRIN Emmanuel (Canal Plus Prestataire))
   2. RE: [PATCH] timeserver: Fix time update manual->auto at startup
      (VAUTRIN Emmanuel (Canal Plus Prestataire))
   3. RE: [PATCH] timeserver: Fix time update manual->auto at startup
      (VAUTRIN Emmanuel (Canal Plus Prestataire))


----------------------------------------------------------------------

Date: Wed, 3 Feb 2021 13:04:11 +0000
From: "VAUTRIN Emmanuel (Canal Plus Prestataire)"
        <emmanuel.vaut...@cpexterne.org>
Subject: RE: [PATCH] timeserver: Reset synchro on system timeserver
        update
To: Daniel Wagner <w...@monom.org>
Cc: "connman@lists.01.org" <connman@lists.01.org>
Message-ID:  <db7pr02mb4791101e888593db168ff77793...@db7pr02mb4791.eur
        prd02.prod.outlook.com>
Content-Type: text/plain; charset="iso-8859-1"

It is necessary to reset time synchronization when system timeserver
configuration is updated, moreover when it is not available via current
service (no timeserver, no responses...).
---
 src/timeserver.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/timeserver.c b/src/timeserver.c
index d82fb42d..678d11c9 100644
--- a/src/timeserver.c
+++ b/src/timeserver.c
@@ -506,7 +506,7 @@ int __connman_timeserver_system_set(char **servers)
        save_timeservers(servers);
 
        service = connman_service_get_default();
-       __connman_timeserver_conf_update(service);
+       ts_reset(service);
 
        return 0;
 }
-- 
2.25.1


------------------------------

Date: Wed, 3 Feb 2021 15:36:42 +0000
From: "VAUTRIN Emmanuel (Canal Plus Prestataire)"
        <emmanuel.vaut...@cpexterne.org>
Subject: RE: [PATCH] timeserver: Fix time update manual->auto at
        startup
To: "connman@lists.01.org" <connman@lists.01.org>
Message-ID:  <db7pr02mb4791c52d82216b434e4e155d93...@db7pr02mb4791.eur
        prd02.prod.outlook.com>
Content-Type: text/plain; charset="iso-8859-1"

When resetting the time synchronization, always set the associated
service, even when the timeserver list is empty, to enable future
synchronization, when switching time update from manual to auto.
---
 src/timeserver.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/timeserver.c b/src/timeserver.c
index 678d11c9..2c6c8dd4 100644
--- a/src/timeserver.c
+++ b/src/timeserver.c
@@ -386,6 +386,8 @@ static void ts_reset(struct connman_service *service)
 
        __connman_service_timeserver_changed(service, timeservers_list);
 
+       ts_service = service;
+
        if (!timeservers_list) {
                DBG("No timeservers set.");
                return;
@@ -393,7 +395,6 @@ static void ts_reset(struct connman_service *service)
 
        ts_recheck_enable();
 
-       ts_service = service;
        timeserver_sync_start();
 }
 
-- 
2.25.1

------------------------------

Date: Wed, 3 Feb 2021 17:05:08 +0000
From: "VAUTRIN Emmanuel (Canal Plus Prestataire)"
        <emmanuel.vaut...@cpexterne.org>
Subject: RE: [PATCH] timeserver: Fix time update manual->auto at
        startup
To: KeithG <ys3al...@gmail.com>
Cc: "connman@lists.01.org" <connman@lists.01.org>
Message-ID:  <db7pr02mb4791a43522bf88d18dcae88693...@db7pr02mb4791.eur
        prd02.prod.outlook.com>
Content-Type: text/plain; charset="iso-8859-1"

Hi Keith,

> As a clarification, is this set of patches to 'fix' the systemd-timesyncd 
> problem where when a device is booted, it may not know what time it is and 
> then > stops any communication because the DNSSEC time stamps do not match 
> between it and the DNS server?

I have no idea about the  systemd-timesyncd problem, but, this set (3 patches I 
have sent today on the subject time synchronization)  is effectively to fix 
some corner cases and improve time synchronization. We add some workarounds in 
higher layers to ensure time synchronization, the goal was to remove them and 
rely only on ConnMan.

According to your explanation, if there is a relevant default timestamp 
(/etc/timestamp) in the rootfs, you can encounter this mismatch with DNS server 
when no time synchronization is applied.  So, maybe it solves your problem. The 
best will be to check when integrated. If it is not the case, what is the 
device time (default /etc/timestamp) in your case and the clock properties 
(with the TimeserverSynced new property) when booting?


Best Regards,

Emmanuel

------------------------------

Subject: Digest Footer

_______________________________________________
connman mailing list -- connman@lists.01.org
To unsubscribe send an email to connman-le...@lists.01.org


------------------------------

End of connman Digest, Vol 64, Issue 3
**************************************

Reply via email to