* [RESEND PATCH v2] arm/arm64: kvm: Use virt_to_idmap instead of virt_to_phys for idmap mappings
@ 2013-11-19 19:59 Santosh Shilimkar
2013-12-10 19:45 ` Santosh Shilimkar
0 siblings, 1 reply; 5+ messages in thread
From: Santosh Shilimkar @ 2013-11-19 19:59 UTC (permalink / raw)
To: linux-arm-kernel
KVM initialisation fails on architectures implementing virt_to_idmap()
because virt_to_phys() on such architectures won't fetch you the correct
idmap page.
So update the KVM ARM code to use the virt_to_idmap() to fix the issue.
Since the KVM code is shared between arm and arm64, we create
kvm_virt_to_phys() and handle the redirection in respective headers.
Cc: Christoffer Dall <christoffer.dall@linaro.org>
Cc: Marc Zyngier <marc.zyngier@arm.com>
Cc: Catalin Marinas <catalin.marinas@arm.com>
Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
---
Update the earlier version as per Marc's suggestion
arch/arm/include/asm/kvm_mmu.h | 1 +
arch/arm/kvm/mmu.c | 8 ++++----
arch/arm64/include/asm/kvm_mmu.h | 1 +
3 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/arch/arm/include/asm/kvm_mmu.h b/arch/arm/include/asm/kvm_mmu.h
index 77de4a4..2d122ad 100644
--- a/arch/arm/include/asm/kvm_mmu.h
+++ b/arch/arm/include/asm/kvm_mmu.h
@@ -140,6 +140,7 @@ static inline void coherent_icache_guest_page(struct kvm *kvm, hva_t hva,
}
#define kvm_flush_dcache_to_poc(a,l) __cpuc_flush_dcache_area((a), (l))
+#define kvm_virt_to_phys(x) virt_to_idmap((unsigned long)(x))
#endif /* !__ASSEMBLY__ */
diff --git a/arch/arm/kvm/mmu.c b/arch/arm/kvm/mmu.c
index 5809069..659db0e 100644
--- a/arch/arm/kvm/mmu.c
+++ b/arch/arm/kvm/mmu.c
@@ -916,9 +916,9 @@ int kvm_mmu_init(void)
{
int err;
- hyp_idmap_start = virt_to_phys(__hyp_idmap_text_start);
- hyp_idmap_end = virt_to_phys(__hyp_idmap_text_end);
- hyp_idmap_vector = virt_to_phys(__kvm_hyp_init);
+ hyp_idmap_start = kvm_virt_to_phys(__hyp_idmap_text_start);
+ hyp_idmap_end = kvm_virt_to_phys(__hyp_idmap_text_end);
+ hyp_idmap_vector = kvm_virt_to_phys(__kvm_hyp_init);
if ((hyp_idmap_start ^ hyp_idmap_end) & PAGE_MASK) {
/*
@@ -945,7 +945,7 @@ int kvm_mmu_init(void)
*/
kvm_flush_dcache_to_poc(init_bounce_page, len);
- phys_base = virt_to_phys(init_bounce_page);
+ phys_base = kvm_virt_to_phys(init_bounce_page);
hyp_idmap_vector += phys_base - hyp_idmap_start;
hyp_idmap_start = phys_base;
hyp_idmap_end = phys_base + len;
diff --git a/arch/arm64/include/asm/kvm_mmu.h b/arch/arm64/include/asm/kvm_mmu.h
index 680f74e..7f1f940 100644
--- a/arch/arm64/include/asm/kvm_mmu.h
+++ b/arch/arm64/include/asm/kvm_mmu.h
@@ -136,6 +136,7 @@ static inline void coherent_icache_guest_page(struct kvm *kvm, hva_t hva,
}
#define kvm_flush_dcache_to_poc(a,l) __flush_dcache_area((a), (l))
+#define kvm_virt_to_phys(x) __virt_to_phys((unsigned long)(x))
#endif /* __ASSEMBLY__ */
#endif /* __ARM64_KVM_MMU_H__ */
--
1.7.9.5
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [RESEND PATCH v2] arm/arm64: kvm: Use virt_to_idmap instead of virt_to_phys for idmap mappings
2013-11-19 19:59 [RESEND PATCH v2] arm/arm64: kvm: Use virt_to_idmap instead of virt_to_phys for idmap mappings Santosh Shilimkar
@ 2013-12-10 19:45 ` Santosh Shilimkar
2013-12-11 0:23 ` Christoffer Dall
0 siblings, 1 reply; 5+ messages in thread
From: Santosh Shilimkar @ 2013-12-10 19:45 UTC (permalink / raw)
To: linux-arm-kernel
Christoffer,
On Tuesday 19 November 2013 02:59 PM, Santosh Shilimkar wrote:
> KVM initialisation fails on architectures implementing virt_to_idmap()
> because virt_to_phys() on such architectures won't fetch you the correct
> idmap page.
>
> So update the KVM ARM code to use the virt_to_idmap() to fix the issue.
> Since the KVM code is shared between arm and arm64, we create
> kvm_virt_to_phys() and handle the redirection in respective headers.
>
> Cc: Christoffer Dall <christoffer.dall@linaro.org>
> Cc: Marc Zyngier <marc.zyngier@arm.com>
> Cc: Catalin Marinas <catalin.marinas@arm.com>
> Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
> ---
> Update the earlier version as per Marc's suggestion
>
Have you picked this up for 3.14 ?
> arch/arm/include/asm/kvm_mmu.h | 1 +
> arch/arm/kvm/mmu.c | 8 ++++----
> arch/arm64/include/asm/kvm_mmu.h | 1 +
> 3 files changed, 6 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/include/asm/kvm_mmu.h b/arch/arm/include/asm/kvm_mmu.h
> index 77de4a4..2d122ad 100644
> --- a/arch/arm/include/asm/kvm_mmu.h
> +++ b/arch/arm/include/asm/kvm_mmu.h
> @@ -140,6 +140,7 @@ static inline void coherent_icache_guest_page(struct kvm *kvm, hva_t hva,
> }
>
> #define kvm_flush_dcache_to_poc(a,l) __cpuc_flush_dcache_area((a), (l))
> +#define kvm_virt_to_phys(x) virt_to_idmap((unsigned long)(x))
>
> #endif /* !__ASSEMBLY__ */
>
> diff --git a/arch/arm/kvm/mmu.c b/arch/arm/kvm/mmu.c
> index 5809069..659db0e 100644
> --- a/arch/arm/kvm/mmu.c
> +++ b/arch/arm/kvm/mmu.c
> @@ -916,9 +916,9 @@ int kvm_mmu_init(void)
> {
> int err;
>
> - hyp_idmap_start = virt_to_phys(__hyp_idmap_text_start);
> - hyp_idmap_end = virt_to_phys(__hyp_idmap_text_end);
> - hyp_idmap_vector = virt_to_phys(__kvm_hyp_init);
> + hyp_idmap_start = kvm_virt_to_phys(__hyp_idmap_text_start);
> + hyp_idmap_end = kvm_virt_to_phys(__hyp_idmap_text_end);
> + hyp_idmap_vector = kvm_virt_to_phys(__kvm_hyp_init);
>
> if ((hyp_idmap_start ^ hyp_idmap_end) & PAGE_MASK) {
> /*
> @@ -945,7 +945,7 @@ int kvm_mmu_init(void)
> */
> kvm_flush_dcache_to_poc(init_bounce_page, len);
>
> - phys_base = virt_to_phys(init_bounce_page);
> + phys_base = kvm_virt_to_phys(init_bounce_page);
> hyp_idmap_vector += phys_base - hyp_idmap_start;
> hyp_idmap_start = phys_base;
> hyp_idmap_end = phys_base + len;
> diff --git a/arch/arm64/include/asm/kvm_mmu.h b/arch/arm64/include/asm/kvm_mmu.h
> index 680f74e..7f1f940 100644
> --- a/arch/arm64/include/asm/kvm_mmu.h
> +++ b/arch/arm64/include/asm/kvm_mmu.h
> @@ -136,6 +136,7 @@ static inline void coherent_icache_guest_page(struct kvm *kvm, hva_t hva,
> }
>
> #define kvm_flush_dcache_to_poc(a,l) __flush_dcache_area((a), (l))
> +#define kvm_virt_to_phys(x) __virt_to_phys((unsigned long)(x))
>
> #endif /* __ASSEMBLY__ */
> #endif /* __ARM64_KVM_MMU_H__ */
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [RESEND PATCH v2] arm/arm64: kvm: Use virt_to_idmap instead of virt_to_phys for idmap mappings
2013-12-10 19:45 ` Santosh Shilimkar
@ 2013-12-11 0:23 ` Christoffer Dall
2013-12-11 10:01 ` Marc Zyngier
0 siblings, 1 reply; 5+ messages in thread
From: Christoffer Dall @ 2013-12-11 0:23 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Dec 10, 2013 at 02:45:17PM -0500, Santosh Shilimkar wrote:
> Christoffer,
>
> On Tuesday 19 November 2013 02:59 PM, Santosh Shilimkar wrote:
> > KVM initialisation fails on architectures implementing virt_to_idmap()
> > because virt_to_phys() on such architectures won't fetch you the correct
> > idmap page.
> >
> > So update the KVM ARM code to use the virt_to_idmap() to fix the issue.
> > Since the KVM code is shared between arm and arm64, we create
> > kvm_virt_to_phys() and handle the redirection in respective headers.
> >
> > Cc: Christoffer Dall <christoffer.dall@linaro.org>
> > Cc: Marc Zyngier <marc.zyngier@arm.com>
> > Cc: Catalin Marinas <catalin.marinas@arm.com>
> > Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
> > ---
> > Update the earlier version as per Marc's suggestion
> >
> Have you picked this up for 3.14 ?
>
Actually, I was waiting for Marc to ack this one. Marc?
-Christoffer
> > arch/arm/include/asm/kvm_mmu.h | 1 +
> > arch/arm/kvm/mmu.c | 8 ++++----
> > arch/arm64/include/asm/kvm_mmu.h | 1 +
> > 3 files changed, 6 insertions(+), 4 deletions(-)
> >
> > diff --git a/arch/arm/include/asm/kvm_mmu.h b/arch/arm/include/asm/kvm_mmu.h
> > index 77de4a4..2d122ad 100644
> > --- a/arch/arm/include/asm/kvm_mmu.h
> > +++ b/arch/arm/include/asm/kvm_mmu.h
> > @@ -140,6 +140,7 @@ static inline void coherent_icache_guest_page(struct kvm *kvm, hva_t hva,
> > }
> >
> > #define kvm_flush_dcache_to_poc(a,l) __cpuc_flush_dcache_area((a), (l))
> > +#define kvm_virt_to_phys(x) virt_to_idmap((unsigned long)(x))
> >
> > #endif /* !__ASSEMBLY__ */
> >
> > diff --git a/arch/arm/kvm/mmu.c b/arch/arm/kvm/mmu.c
> > index 5809069..659db0e 100644
> > --- a/arch/arm/kvm/mmu.c
> > +++ b/arch/arm/kvm/mmu.c
> > @@ -916,9 +916,9 @@ int kvm_mmu_init(void)
> > {
> > int err;
> >
> > - hyp_idmap_start = virt_to_phys(__hyp_idmap_text_start);
> > - hyp_idmap_end = virt_to_phys(__hyp_idmap_text_end);
> > - hyp_idmap_vector = virt_to_phys(__kvm_hyp_init);
> > + hyp_idmap_start = kvm_virt_to_phys(__hyp_idmap_text_start);
> > + hyp_idmap_end = kvm_virt_to_phys(__hyp_idmap_text_end);
> > + hyp_idmap_vector = kvm_virt_to_phys(__kvm_hyp_init);
> >
> > if ((hyp_idmap_start ^ hyp_idmap_end) & PAGE_MASK) {
> > /*
> > @@ -945,7 +945,7 @@ int kvm_mmu_init(void)
> > */
> > kvm_flush_dcache_to_poc(init_bounce_page, len);
> >
> > - phys_base = virt_to_phys(init_bounce_page);
> > + phys_base = kvm_virt_to_phys(init_bounce_page);
> > hyp_idmap_vector += phys_base - hyp_idmap_start;
> > hyp_idmap_start = phys_base;
> > hyp_idmap_end = phys_base + len;
> > diff --git a/arch/arm64/include/asm/kvm_mmu.h b/arch/arm64/include/asm/kvm_mmu.h
> > index 680f74e..7f1f940 100644
> > --- a/arch/arm64/include/asm/kvm_mmu.h
> > +++ b/arch/arm64/include/asm/kvm_mmu.h
> > @@ -136,6 +136,7 @@ static inline void coherent_icache_guest_page(struct kvm *kvm, hva_t hva,
> > }
> >
> > #define kvm_flush_dcache_to_poc(a,l) __flush_dcache_area((a), (l))
> > +#define kvm_virt_to_phys(x) __virt_to_phys((unsigned long)(x))
> >
> > #endif /* __ASSEMBLY__ */
> > #endif /* __ARM64_KVM_MMU_H__ */
> >
>
--
Christoffer
^ permalink raw reply [flat|nested] 5+ messages in thread
* [RESEND PATCH v2] arm/arm64: kvm: Use virt_to_idmap instead of virt_to_phys for idmap mappings
2013-12-11 0:23 ` Christoffer Dall
@ 2013-12-11 10:01 ` Marc Zyngier
2013-12-11 20:52 ` Christoffer Dall
0 siblings, 1 reply; 5+ messages in thread
From: Marc Zyngier @ 2013-12-11 10:01 UTC (permalink / raw)
To: linux-arm-kernel
On 11/12/13 00:23, Christoffer Dall wrote:
> On Tue, Dec 10, 2013 at 02:45:17PM -0500, Santosh Shilimkar wrote:
>> Christoffer,
>>
>> On Tuesday 19 November 2013 02:59 PM, Santosh Shilimkar wrote:
>>> KVM initialisation fails on architectures implementing virt_to_idmap()
>>> because virt_to_phys() on such architectures won't fetch you the correct
>>> idmap page.
>>>
>>> So update the KVM ARM code to use the virt_to_idmap() to fix the issue.
>>> Since the KVM code is shared between arm and arm64, we create
>>> kvm_virt_to_phys() and handle the redirection in respective headers.
>>>
>>> Cc: Christoffer Dall <christoffer.dall@linaro.org>
>>> Cc: Marc Zyngier <marc.zyngier@arm.com>
>>> Cc: Catalin Marinas <catalin.marinas@arm.com>
>>> Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
>>> ---
>>> Update the earlier version as per Marc's suggestion
>>>
>> Have you picked this up for 3.14 ?
>>
>
> Actually, I was waiting for Marc to ack this one. Marc?
Acked-by: Marc Zyngier <marc.zyngier@arm.com>
M.
--
Jazz is not dead. It just smells funny...
^ permalink raw reply [flat|nested] 5+ messages in thread
* [RESEND PATCH v2] arm/arm64: kvm: Use virt_to_idmap instead of virt_to_phys for idmap mappings
2013-12-11 10:01 ` Marc Zyngier
@ 2013-12-11 20:52 ` Christoffer Dall
0 siblings, 0 replies; 5+ messages in thread
From: Christoffer Dall @ 2013-12-11 20:52 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Dec 11, 2013 at 10:01:45AM +0000, Marc Zyngier wrote:
> On 11/12/13 00:23, Christoffer Dall wrote:
> > On Tue, Dec 10, 2013 at 02:45:17PM -0500, Santosh Shilimkar wrote:
> >> Christoffer,
> >>
> >> On Tuesday 19 November 2013 02:59 PM, Santosh Shilimkar wrote:
> >>> KVM initialisation fails on architectures implementing virt_to_idmap()
> >>> because virt_to_phys() on such architectures won't fetch you the correct
> >>> idmap page.
> >>>
> >>> So update the KVM ARM code to use the virt_to_idmap() to fix the issue.
> >>> Since the KVM code is shared between arm and arm64, we create
> >>> kvm_virt_to_phys() and handle the redirection in respective headers.
> >>>
> >>> Cc: Christoffer Dall <christoffer.dall@linaro.org>
> >>> Cc: Marc Zyngier <marc.zyngier@arm.com>
> >>> Cc: Catalin Marinas <catalin.marinas@arm.com>
> >>> Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
> >>> ---
> >>> Update the earlier version as per Marc's suggestion
> >>>
> >> Have you picked this up for 3.14 ?
> >>
> >
> > Actually, I was waiting for Marc to ack this one. Marc?
>
> Acked-by: Marc Zyngier <marc.zyngier@arm.com>
>
> M.
Thanks, applied.
-Christoffer
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-12-11 20:52 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-11-19 19:59 [RESEND PATCH v2] arm/arm64: kvm: Use virt_to_idmap instead of virt_to_phys for idmap mappings Santosh Shilimkar
2013-12-10 19:45 ` Santosh Shilimkar
2013-12-11 0:23 ` Christoffer Dall
2013-12-11 10:01 ` Marc Zyngier
2013-12-11 20:52 ` Christoffer Dall
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.