From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [PATCH 3/4] xen/arm: Implement a dummy debug monitor for ARM32 Date: Fri, 02 May 2014 14:39:46 +0100 Message-ID: <5363A022.4000401@linaro.org> References: <1398379556-1132-1-git-send-email-julien.grall@linaro.org> <1398379556-1132-4-git-send-email-julien.grall@linaro.org> <1399028953.32736.42.camel@kazak.uk.xensource.com> <53639561.3070303@linaro.org> <1399037211.6043.7.camel@kazak.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta4.messagelabs.com ([85.158.143.247]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1WgDgr-0002Jz-7v for xen-devel@lists.xenproject.org; Fri, 02 May 2014 13:39:49 +0000 Received: by mail-ee0-f50.google.com with SMTP id c13so3217658eek.9 for ; Fri, 02 May 2014 06:39:47 -0700 (PDT) In-Reply-To: <1399037211.6043.7.camel@kazak.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell Cc: xen-devel@lists.xenproject.org, tim@xen.org, stefano.stabellini@citrix.com List-Id: xen-devel@lists.xenproject.org On 05/02/2014 02:26 PM, Ian Campbell wrote: > On Fri, 2014-05-02 at 13:53 +0100, Julien Grall wrote: >> I don't plan to more spend time to write a correct emulation (i.e >> context switching) to support HW debug. > > I'm not going to ack a patch which causes arm32 to diverge from arm64 in > this area, especially not when the correct solution (more critical on > arm64 than arm32) is to properly context switch these registers. We don't diverge... The Linux HW debug arm32 implementation doesn't permit to use RAZ/WI on some registers. Currently arm64 HW debug may or may not work but it won't crash the guest. It's not the case on arm32. So the current Xen already diverge. As said earlier, the HW debug is not essential. Writing a proper emulation will take some time and I don't have time for writing and testing it correctly. IHMO, this solution is perfect for Xen 4.4, otherwise Xen 4.4.1 will breaks support with lots of distribution. For Xen 4.5, it's intermediate solution to allow guest working correctly and people playing their shiny distribution on top of Xen. When someone will care about HW debug, then we will have to support it correctly. Regards, -- Julien Grall