From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51062) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aSoBo-00088H-SF for qemu-devel@nongnu.org; Mon, 08 Feb 2016 10:57:25 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aSoBn-0005Wn-Ts for qemu-devel@nongnu.org; Mon, 08 Feb 2016 10:57:24 -0500 Received: from mail-vk0-x230.google.com ([2607:f8b0:400c:c05::230]:35637) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aSoBn-0005WZ-P6 for qemu-devel@nongnu.org; Mon, 08 Feb 2016 10:57:23 -0500 Received: by mail-vk0-x230.google.com with SMTP id e6so97767454vkh.2 for ; Mon, 08 Feb 2016 07:57:23 -0800 (PST) MIME-Version: 1.0 In-Reply-To: <1454256948-10485-1-git-send-email-serge.fdrv@gmail.com> References: <1454256948-10485-1-git-send-email-serge.fdrv@gmail.com> From: Peter Maydell Date: Mon, 8 Feb 2016 15:57:03 +0000 Message-ID: Content-Type: text/plain; charset=UTF-8 Subject: Re: [Qemu-devel] [PATCH v3 0/2] Architectural watchpoint check List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Sergey Fedorov Cc: Peter Crosthwaite , QEMU Developers , qemu-arm , Paolo Bonzini , =?UTF-8?Q?Andreas_F=C3=A4rber?= , Richard Henderson On 31 January 2016 at 16:15, Sergey Fedorov wrote: > This series is intended to fix ARM watchpoint emulation misbehavior. > QEMU hangs when QEMU watchpoint fires but it does not pass additional > architectural checks in ARM CPU debug exception handler. For details, > please see individual patches. The most relevant parts of the original > discussion about ARM breakpoint and watchpoint emulation misbehavior can be > found at: > https://lists.gnu.org/archive/html/qemu-devel/2015-08/msg02715.html > https://lists.gnu.org/archive/html/qemu-devel/2015-09/msg00527.html > > Changes in v2: > * Check moved before setting cpu->watchpoint_hit > * Pointer to watchpoint being checked passed to debug_check_watchpoint() > callback > * BP_WATCHPOINT_HIT flag cleared from flags from wp->flags in no-fire case > * Comment for debug_check_watchpoint() callback improved Applied to target-arm.next, thanks. -- PMM