[PATCH] KVM: arm/arm64: drop pr_info() of IDMAP page
Ard Biesheuvel
ard.biesheuvel at linaro.org
Thu Mar 1 13:50:22 PST 2018
On 1 March 2018 at 21:10, Marc Zyngier <marc.zyngier at arm.com> wrote:
> On Thu, 01 Mar 2018 19:14:06 +0000,
> Ard Biesheuvel wrote:
>>
>> The KVM IDMAP is a mapping of a statically allocated kernel structure,
>> and so printing its physical address leaks the physical placement of
>> the kernel when physical KASLR is in effect. Just remove it, nobody
>> except Marc needs this anyway.
>
> Bah, even I don't really need it anymore...
>
>> Signed-off-by: Ard Biesheuvel <ard.biesheuvel at linaro.org>
>> ---
>> CC -stable?
>
> In this day and age, probably.
>
>>
>> virt/kvm/arm/mmu.c | 1 -
>> 1 file changed, 1 deletion(-)
>>
>> diff --git a/virt/kvm/arm/mmu.c b/virt/kvm/arm/mmu.c
>> index ec62d1cccab7..c07f88cb6dcf 100644
>> --- a/virt/kvm/arm/mmu.c
>> +++ b/virt/kvm/arm/mmu.c
>> @@ -1810,7 +1810,6 @@ int kvm_mmu_init(void)
>> */
>> BUG_ON((hyp_idmap_start ^ (hyp_idmap_end - 1)) & PAGE_MASK);
>>
>> - kvm_info("IDMAP page: %lx\n", hyp_idmap_start);
>> kvm_info("HYP VA range: %lx:%lx\n",
>> kern_hyp_va(PAGE_OFFSET), kern_hyp_va(~0UL));
>
> Instead of removing this particular line, how about turning all these
> kvm_info into kvm_debug (see [1])?
>
Yeah makes sense. Of the complete output
[ 2.046066] kvm [1]: 8-bit VMID
[ 2.046072] kvm [1]: IDMAP page: d20e35000
[ 2.046077] kvm [1]: HYP VA range: 800000000000:ffffffffffff
[ 2.046888] kvm [1]: vgic-v2 at 2c020000
[ 2.046944] kvm [1]: GIC system register CPU interface enabled
[ 2.047585] kvm [1]: vgic interrupt IRQ1
[ 2.047603] kvm [1]: virtual timer IRQ4
[ 2.048282] kvm [1]: Hyp mode initialized successfully
I think only the first and the last one are useful, and perhaps the
one about the sysreg interface?
More information about the linux-arm-kernel
mailing list