From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([208.118.235.92]:42174)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <andreas.faerber@web.de>) id 1SzUlh-0002DZ-1J
	for qemu-devel@nongnu.org; Thu, 09 Aug 2012 11:35:26 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <andreas.faerber@web.de>) id 1SzUlg-0008LQ-1N
	for qemu-devel@nongnu.org; Thu, 09 Aug 2012 11:35:24 -0400
Message-ID: <5023D8AC.4030801@web.de>
Date: Thu, 09 Aug 2012 17:35:08 +0200
From: =?ISO-8859-1?Q?Andreas_F=E4rber?= <andreas.faerber@web.de>
MIME-Version: 1.0
References: <alpine.DEB.2.00.1205291426350.26786@kaball-desktop>
	<4FD6FCCE.8080807@web.de> <5012C933.5060207@web.de>
In-Reply-To: <5012C933.5060207@web.de>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 8bit
Subject: Re: [Qemu-devel] [Qemu-devel for-1.2] qemu_rearm_alarm_timer: do
 not call rearm if the next deadline is INT64_MAX
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Anthony Liguori <anthony@codemonkey.ws>, Blue Swirl <blauwirbel@gmail.com>
Cc: Stefano Stabellini <stefano.stabellini@eu.citrix.com>, Stefan Hajnoczi <stefanha@gmail.com>, qemu-devel@nongnu.org, qemu-stable@nongnu.org, Stefan Weil <sw@weilnetz.de>, Paolo Bonzini <pbonzini@redhat.com>, Karl-Michael Schindler <karl-michael.schindler@web.de>

Am 27.07.2012 19:00, schrieb Andreas Färber:
> Am 12.06.2012 10:24, schrieb Andreas Färber:
>> Am 29.05.2012 15:35, schrieb Stefano Stabellini:
>>> qemu_rearm_alarm_timer partially duplicates the code in
>>> qemu_next_alarm_deadline to figure out if it needs to rearm the timer.
>>> If it calls qemu_next_alarm_deadline, it always rearms the timer even if
>>> the next deadline is INT64_MAX.
>>>
>>> This patch simplifies the behavior of qemu_rearm_alarm_timer and removes
>>> the duplicated code, always calling qemu_next_alarm_deadline and only
>>> rearming the timer if the deadline is less than INT64_MAX.
>>>
>>> Signed-off-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
>>
>> Tested-by: Andreas Färber <andreas.faerber@web.de>
> 
> Ping! Can the patch please be applied? Note: Patchwork apparently got
> confused by the later follow-up inline patches - only the original patch
> is needed.

Ping^2! This didn't make it into 1.1, would be nice to have 1.2 working.

Correct Patchwork link is: http://patchwork.ozlabs.org/patch/161749/

/-F

> 
> Also cc'ing qemu-stable for stable-1.1.
> 
> Andreas
>