On 30/11/12 14:35, Peter Maydell wrote: > On 30 November 2012 09:31, Claudio Fontana <Claudio.Fontana@xxxxxxxxxx> wrote: >> all the handlers are already there in coproc_a15.c, but the traps >> are not enabled at world switch in __kvm_vcpu_run. >> In particular this was noticed because the guests were trying to >> read the number of CPUs using l2ctlr, and the number of physical >> host CPUs was returned instead of the number of VCPUs. > > L2CTLR should be trapped via HCR.TIDCP, not the HSTR. > Traps for whole swathes of the coprocessor space via HSTR > should be avoided unless absolutely necessary; the functionally > grouped traps in HCR are much better. > > (In fact I thought we already did this? Marc?) We already have HCR_TIDCP set in HCR, which should be enough (see ARM ARM B1.14.3). You should definitely see L2CTLR accesses being trapped with this configuration. What is your hardware? Thanks, M. -- Jazz is not dead. It just smells funny... _______________________________________________ kvmarm mailing list kvmarm@xxxxxxxxxxxxxxxxxxxxx https://lists.cs.columbia.edu/cucslists/listinfo/kvmarm
