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: Sat, 14 Jun 2014 18:16:33 +0100 Message-ID: <539C8371.4070506@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> <1399036479.5226.6.camel@kazak.uk.xensource.com> <53639DC9.2060007@linaro.org> <1402660975.19520.6.camel@kazak.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta5.messagelabs.com ([195.245.231.135]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1WvrZF-0003BQ-8P for xen-devel@lists.xenproject.org; Sat, 14 Jun 2014 17:16:37 +0000 Received: by mail-wg0-f43.google.com with SMTP id b13so3841794wgh.26 for ; Sat, 14 Jun 2014 10:16:35 -0700 (PDT) In-Reply-To: <1402660975.19520.6.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 Hi Ian, On 13/06/14 13:02, Ian Campbell wrote: > Looking at it with fresh eyes this morning I see what you mean now, the > essential difference is that with arm32 DBGDIDR is trapped by > MDCR_EL2.TDA, whereas the the arm64 equivalent (IDAA64DFR*) are not > (it's trapped as part of the ID register group, which we don't bother > trapping). The rest of the registers are RAZ/WI which is consistent with > arm64. Right, this patch is mostly here to let Linux think there is not debug hardware registers. As you said on another mail, the long term goal is to context switch correctly those register to allow perf and debug working in the guest. > So in the end you've convinced me that this is the right thing to do for > now and to backport to 4.4. > > Acked-by: Ian Campbell > > I've committed this and the previous (perfc) one but not the next one > (useful debug for coproc traps) which had comments. > > I did s/DBGCR/DBGBCR/ to match the name used in both the v7 and v8 ARM > ARMs, I think it was just a typo? Hope that's ok. Yes, I forgot the B by mistake. > > Also DBGOSLAR is supposed to be WO but you've implemented it as RAZ/WI, > I didn't think that mattered enough to bother with though. I was lazy to add 2 more lines to handle this register WO. I'm not sure what is behavior when a guest is trying to read a WO register. I guess an undefined instruction. I can send a follow-up to use this behavior for Xen 4.5. Regards, -- Julien Grall