linux-kernel - RE: [PATCH 1/1] staging: hv: Add Time Sync feature to hv

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 6 May 2010 01:42:01 +0000
From:	Haiyang Zhang <haiyangz@...rosoft.com>
To:	Joe Perches <joe@...ches.com>
CC:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"devel@...verdev.osuosl.org" <devel@...verdev.osuosl.org>,
	"virtualization@...ts.osdl.org" <virtualization@...ts.osdl.org>,
	"gregkh@...e.de" <gregkh@...e.de>,
	Hank Janssen <hjanssen@...rosoft.com>
Subject: RE: [PATCH 1/1] staging: hv: Add Time Sync feature to hv_utils
 module

> Why a maximum of 50 samples?
After reboot the flag ICTIMESYNCFLAG_SYNC is included in the 
first time message after the timesync channel is opened. Since the 
hv_utils module is loaded after hv_vmbus, the first message is usually
missed. The other thing is, systime is automatically set to emulated 
hardware clock which may not be UTC time or the same time 
zone. So, to override these effects, we use the first 50 time samples 
for initial system time setting.

> It might be better to do something like this
> so the ns_to_timespec isn't performe when unnecessary.
Thanks for the optimization, I will put it into the code.

- Haiyang--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/