Re: [PATCH v6 2/7] arm64: Introduce VA_BITS and translation level options

From: Jungseok Lee
Date: Wed Jul 16 2014 - 10:23:56 EST


On Jul 16, 2014, at 6:44 AM, Catalin Marinas wrote:
> On 15 Jul 2014, at 15:53, Jungseok Lee <jungseoklee85@xxxxxxxxx> wrote:
>> On Jul 15, 2014, at 7:41 AM, Catalin Marinas wrote:
>>> On Mon, Jul 14, 2014 at 09:38:59PM +0100, Joel Schopp wrote:
>>>> I agree that these patches would be very useful. I just rebased my fix
>>>> for a VTTBR_BADDR_MASK bug on one of these patches that could be pulled
>>>> out independently. See
>>>> https://lists.cs.columbia.edu/pipermail/kvmarm/2014-July/010480.html
>>>>
>>>> The original author Jungseok Lee is no longer available to work on
>>>> future versions of these patches. I was thinking that if they didn't
>>>> get picked up as they are that with the original author's blessing I
>>>> would pick them up and keep them forward ported/resubmitted. I have an
>>>> SOC to test them on.
>>>
>>> The patches are pretty good. I'll give them a try tomorrow and if there
>>> isn't something fundamental missing I'll consider taking them for 3.17.
>>
>> Hi All,
>>
>> If only stage1 side is taken and merged, KVM should be disabled under 4 level
>> lookups with the following configuration adjustment. I've tested it on top of
>> arm64/for-next/core branch and it works fine.
>>
>> --- a/arch/arm64/kvm/Kconfig
>> +++ b/arch/arm64/kvm/Kconfig
>> @@ -18,6 +18,7 @@ if VIRTUALIZATION
>>
>> config KVM
>> bool "Kernel-based Virtual Machine (KVM) support"
>> + depends on !ARM64_4_LEVELS
>> select MMU_NOTIFIER
>> select PREEMPT_NOTIFIERS
>> select ANON_INODES
>>
>> However, I don't know whether it does make sense or not.
>
> I added some patches on top of your series, I hope to be able to post
> them tomorrow. My interim approach was to disable 4-levels if KVM is
> enabled.

Hi Catalin,

Thanks for the work!
I will take a close look at them.

- Jungseok Lee--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/