From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43650) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bGW7N-0007RW-Tv for qemu-devel@nongnu.org; Fri, 24 Jun 2016 14:46:22 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bGVh5-0002Nd-HL for qemu-devel@nongnu.org; Fri, 24 Jun 2016 14:19:10 -0400 Received: from mail-lf0-x244.google.com ([2a00:1450:4010:c07::244]:33639) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bGVh5-0002M6-8R for qemu-devel@nongnu.org; Fri, 24 Jun 2016 14:19:07 -0400 Received: by mail-lf0-x244.google.com with SMTP id l188so22615496lfe.0 for ; Fri, 24 Jun 2016 11:19:06 -0700 (PDT) References: <1465224465-21998-1-git-send-email-peter.maydell@linaro.org> <1465224465-21998-18-git-send-email-peter.maydell@linaro.org> <8a6e05fc-92e3-d679-3970-83ec1f67f8ac@ilande.co.uk> From: Dmitry Osipenko Message-ID: Date: Fri, 24 Jun 2016 21:19:02 +0300 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PULL 17/28] hw/ptimer: Perform counter wrap around if timer already expired List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell , Mark Cave-Ayland Cc: QEMU Developers , Artyom Tarasenko On 24.06.2016 19:02, Peter Maydell wrote: > On 24 June 2016 at 16:58, Mark Cave-Ayland > wrote: >> On 06/06/16 15:47, Peter Maydell wrote: >> >>> From: Dmitry Osipenko >>> >>> ptimer_get_count() might be called while QEMU timer already been expired. >>> In that case ptimer would return counter = 0, which might be undesirable >>> in case of polled timer. Do counter wrap around for periodic timer to keep >>> it distributed. In order to achieve more accurate emulation behaviour of >>> certain hardware, don't perform wrap around when in icount mode and return >>> counter = 0 in that case (that doesn't affect polled counter >>> distribution). >>> > >> Whilst testing Artyom's qemu-system-sparc patch today, I noticed another >> regression which I've bisected down to the above commit. >> >> Booting my NetBSD/OpenBSD test images with current git master causes the >> following warning to appear on the console: "WARNING: negative runtime; >> monotonic clock has gone backwards". >> >> Could this be a regression or does qemu-system-sparc make an incorrect >> assumption as to how the timer should work in this scenario? > > I'm not sure -- Dmitry ? > > thanks > -- PMM > The problem could be due to the IRQ being raised after the poll of the wrapped around counter in non-icount mode, so CPU "sees the future". In that case patch should be reverted and reworked. During the review of the patch we decided that it shouldn't be an issue. I'll take a closer look at it and try to reproduce the issue. -- Dmitry