From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH 6/9] xen: arm: Handle CP14 32-bit register accesses from userspace Date: Tue, 10 Feb 2015 03:40:41 +0000 Message-ID: <1423539641.5851.5.camel@citrix.com> References: <1410279730.8217.238.camel@kazak.uk.xensource.com> <1410279788-27167-6-git-send-email-ian.campbell@citrix.com> <540F9136.3080903@linaro.org> <1410342510.8217.275.camel@kazak.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1410342510.8217.275.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: Julien Grall Cc: xen-devel@lists.xen.org, tim@xen.org, stefano.stabellini@eu.citrix.com List-Id: xen-devel@lists.xenproject.org On Wed, 2014-09-10 at 10:48 +0100, Ian Campbell wrote: > On Tue, 2014-09-09 at 16:45 -0700, Julien Grall wrote: > > Hi Ian, > > > > On 09/09/14 09:23, Ian Campbell wrote: > > > Accesses to these from 32-bit userspace would cause a hypervisor exception > > > (host crash) when running a 64-bit kernel, which is worked around by the fix to > > > XSA-102. On 32-bit kernels they would be implemented as RAZ/WI which is > > > incorrect but harmless. > > > > > > Update as follows: > > > - DBGDSCRINT should be R/O. > > > - DBGDSCREXT should be EL1 only. > > > - DBGOSLAR is RO and EL1 only. > > > - DBGVCR, DBGB[VC]R*, DBGW[VC]R*, and DBGOSDLR are EL1 only. > > > > > > DBGDIDR and DBGDSCRINT are accessible from EL0 if DBGDSCRext.UDCCdis. Since we > > > emulate that as RAZ/WI we allow access. > > > > Shall we just set DBGDSCRext.UDCCdis to avoid taking care of EL0 access? > > I'd need to lookup what the acceptable reset states for that bit are, > but perhaps. The AArch32 version of this bit resets to 0, so I think the code is OK as it is, at least for now. I'd like to implement proper handling of dbg registers sooner rather than later, but I think this series should go in first and the the dbg stuff can be built on it later. Ian. > > Ian. > > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xen.org > http://lists.xen.org/xen-devel