From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42119) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gP788-0008ES-5R for qemu-devel@nongnu.org; Tue, 20 Nov 2018 09:35:58 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gP787-0002mE-9u for qemu-devel@nongnu.org; Tue, 20 Nov 2018 09:35:56 -0500 Received: from mail-oi1-x243.google.com ([2607:f8b0:4864:20::243]:36749) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1gP787-0002ln-4Y for qemu-devel@nongnu.org; Tue, 20 Nov 2018 09:35:55 -0500 Received: by mail-oi1-x243.google.com with SMTP id x23so1652783oix.3 for ; Tue, 20 Nov 2018 06:35:55 -0800 (PST) MIME-Version: 1.0 In-Reply-To: <20181116212154.GG23658@quinoa.localdomain> References: <20181105185046.2802-1-aaron@os.amperecomputing.com> <20181105185046.2802-13-aaron@os.amperecomputing.com> <20181116212154.GG23658@quinoa.localdomain> From: Peter Maydell Date: Tue, 20 Nov 2018 14:35:33 +0000 Message-ID: Content-Type: text/plain; charset="UTF-8" Subject: Re: [Qemu-devel] [PATCH v7 12/12] target/arm: Send interrupts on PMU counter overflow List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Aaron Lindsay Cc: "qemu-devel@nongnu.org" , Michael Spradling , Digant Desai , Aaron Lindsay , Alistair Francis , Wei Huang , Peter Crosthwaite , Richard Henderson , "qemu-arm@nongnu.org" On 16 November 2018 at 21:22, Aaron Lindsay wrote: > On Nov 05 13:52, Aaron Lindsay wrote: >> Setup a QEMUTimer to get a callback when we expect counters to next >> overflow and trigger an interrupt at that time. > > Peter, > > It looks like there's probably going to be at least a v8 in this series > since you've made a few comments which require changes. But I don't > think this final patch has ever received any review comments from > anyone, and I suspect it's not because it's perfect... > > Would you mind taking a look at this one during your review if you can > find time? I think I'd rather let Richard review this one... thanks -- PMM