From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34445) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YGP3F-00056T-WC for qemu-devel@nongnu.org; Wed, 28 Jan 2015 04:36:46 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YGP39-000278-FD for qemu-devel@nongnu.org; Wed, 28 Jan 2015 04:36:45 -0500 Received: from mx1.redhat.com ([209.132.183.28]:57519) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YGP39-00026b-7v for qemu-devel@nongnu.org; Wed, 28 Jan 2015 04:36:39 -0500 From: Paolo Bonzini Date: Wed, 28 Jan 2015 10:36:32 +0100 Message-Id: <1422437793-14843-2-git-send-email-pbonzini@redhat.com> In-Reply-To: <1422437793-14843-1-git-send-email-pbonzini@redhat.com> References: <1422437793-14843-1-git-send-email-pbonzini@redhat.com> Subject: [Qemu-devel] [PATCH 1/2] cpu-exec: simplify align_clocks List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Sebastian Tanase sc->diff_clk is already equal to sleep_delay (split in a second and a nanosecond part). If you subtract sleep_delay - rem_delay, the result is exactly rem_delay. Cc: Sebastian Tanase Signed-off-by: Paolo Bonzini --- cpu-exec.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/cpu-exec.c b/cpu-exec.c index a4f0eff..ef0f12c 100644 --- a/cpu-exec.c +++ b/cpu-exec.c @@ -61,8 +61,7 @@ static void align_clocks(SyncClocks *sc, const CPUState *cpu) sleep_delay.tv_sec = sc->diff_clk / 1000000000LL; sleep_delay.tv_nsec = sc->diff_clk % 1000000000LL; if (nanosleep(&sleep_delay, &rem_delay) < 0) { - sc->diff_clk -= (sleep_delay.tv_sec - rem_delay.tv_sec) * 1000000000LL; - sc->diff_clk -= sleep_delay.tv_nsec - rem_delay.tv_nsec; + sc->diff_clk = rem_delay.tv_sec * 1000000000LL + rem_delay.tv_nsec; } else { sc->diff_clk = 0; } -- 2.1.0