public inbox for util-linux@vger.kernel.org
 help / color / mirror / Atom feed
From: Karel Zak <kzak@redhat.com>
To: Ruediger Meier <sweet_f_a@gmx.de>
Cc: util-linux@vger.kernel.org
Subject: Re: tests: hwclock questions
Date: Thu, 5 Jun 2014 09:48:05 +0200	[thread overview]
Message-ID: <20140605074805.GV9000@x2.net.home> (raw)
In-Reply-To: <201406050032.24233.sweet_f_a@gmx.de>

On Thu, Jun 05, 2014 at 12:32:24AM +0200, Ruediger Meier wrote:
> But before doing this I have two questions.

 Maybe we can remove the test at all (or use --force to enable the test). It 
 was always very problematic test and I have doubts it's still necessary as 
 relevant hwclock code is completely different and more robust now.

> 1. I wonder what is the original purpose of this test. Are we really 
> testing hwclock or the kernel or hardware? I mean setting and reading

 The purpose is to test how precisely is able hwclock to set HW time.
 Long time ago we had problem that each iteration added extra 0.5s to
 the time.

> cock 10 times ... could this really discover a bug in our hwclock code?

 Yes.

> 2. Why do we _set_ the time from ntp server at the beginning? Wouldnt it 
> be enough to check whether the offset to ntp is the same before and 
> after the test?

 Probably yes.

    Karel

-- 
 Karel Zak  <kzak@redhat.com>
 http://karelzak.blogspot.com

  reply	other threads:[~2014-06-05  7:48 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-04 22:32 tests: hwclock questions Ruediger Meier
2014-06-05  7:48 ` Karel Zak [this message]
2014-06-05 12:45   ` Ruediger Meier
2014-06-06 13:06     ` [PATCH] " Ruediger Meier

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=20140605074805.GV9000@x2.net.home \
    --to=kzak@redhat.com \
    --cc=sweet_f_a@gmx.de \
    --cc=util-linux@vger.kernel.org \
    /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