From mboxrd@z Thu Jan 1 00:00:00 1970 From: Aurelien Jarno Subject: Re: [PATCH v2 04/10] target-mips: get_physical_address: Add KVM awareness Date: Sat, 21 Dec 2013 16:34:17 +0100 Message-ID: <20131221153417.GA9151@ohm.rr44.fr> References: <1387203165-5553-1-git-send-email-james.hogan@imgtec.com> <1387203165-5553-5-git-send-email-james.hogan@imgtec.com> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Cc: Paolo Bonzini , Gleb Natapov , qemu-devel@nongnu.org, kvm@vger.kernel.org, Sanjay Lal To: James Hogan Return-path: Content-Disposition: inline In-Reply-To: <1387203165-5553-5-git-send-email-james.hogan@imgtec.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org Sender: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org List-Id: kvm.vger.kernel.org On Mon, Dec 16, 2013 at 02:12:39PM +0000, James Hogan wrote: > MIPS KVM trap & emulate mode (which is currently the only supported > mode) has to add an extra kseg0/kseg1 at 0x40000000 and an extra > kseg2/kseg3 at 0x60000000. Take this into account in > get_physical_address() so that debug memory access works. > > This is done by translating the address to a standard kseg0 or kseg2 > address before doing the normal address translation. The real virtual > address is still used for TLB lookups. > > Signed-off-by: James Hogan > Cc: Aurelien Jarno > --- > target-mips/helper.c | 33 ++++++++++++++++++++++++++------- > 1 file changed, 26 insertions(+), 7 deletions(-) > > diff --git a/target-mips/helper.c b/target-mips/helper.c > index 2e96655..c4be887 100644 > --- a/target-mips/helper.c > +++ b/target-mips/helper.c > @@ -24,6 +24,7 @@ > #include > > #include "cpu.h" > +#include "sysemu/kvm.h" > > enum { > TLBRET_DIRTY = -4, > @@ -100,7 +101,7 @@ int r4k_map_address (CPUMIPSState *env, hwaddr *physical, int *prot, > } > > static int get_physical_address (CPUMIPSState *env, hwaddr *physical, > - int *prot, target_ulong address, > + int *prot, target_ulong real_address, > int rw, int access_type) > { > /* User mode can only access useg/xuseg */ > @@ -113,6 +114,8 @@ static int get_physical_address (CPUMIPSState *env, hwaddr *physical, > int KX = (env->CP0_Status & (1 << CP0St_KX)) != 0; > #endif > int ret = TLBRET_MATCH; > + /* effective address (modified for KVM T&E kernel segments) */ > + target_ulong address = real_address; > > #if 0 > qemu_log("user mode %d h %08x\n", user_mode, env->hflags); > @@ -124,19 +127,35 @@ static int get_physical_address (CPUMIPSState *env, hwaddr *physical, > #define KSEG2_BASE 0xC0000000UL > #define KSEG3_BASE 0xE0000000UL > > +#define KVM_KSEG0_BASE 0x40000000UL > +#define KVM_KSEG2_BASE 0x60000000UL > + > + if (kvm_enabled()) { > + /* KVM T&E adds guest kernel segments in useg */ > + if (real_address >= KVM_KSEG0_BASE) { > + if (real_address < KVM_KSEG2_BASE) { > + /* kseg0 */ > + address += KSEG0_BASE - KVM_KSEG0_BASE; > + } else if (real_address <= USEG_LIMIT) { > + /* kseg2/3 */ > + address += KSEG2_BASE - KVM_KSEG2_BASE; > + } > + } > + } > + > if (address <= USEG_LIMIT) { > /* useg */ > if (env->CP0_Status & (1 << CP0St_ERL)) { > *physical = address & 0xFFFFFFFF; > *prot = PAGE_READ | PAGE_WRITE; > } else { > - ret = env->tlb->map_address(env, physical, prot, address, rw, access_type); > + ret = env->tlb->map_address(env, physical, prot, real_address, rw, access_type); > } > #if defined(TARGET_MIPS64) > } else if (address < 0x4000000000000000ULL) { > /* xuseg */ > if (UX && address <= (0x3FFFFFFFFFFFFFFFULL & env->SEGMask)) { > - ret = env->tlb->map_address(env, physical, prot, address, rw, access_type); > + ret = env->tlb->map_address(env, physical, prot, real_address, rw, access_type); > } else { > ret = TLBRET_BADADDR; > } > @@ -144,7 +163,7 @@ static int get_physical_address (CPUMIPSState *env, hwaddr *physical, > /* xsseg */ > if ((supervisor_mode || kernel_mode) && > SX && address <= (0x7FFFFFFFFFFFFFFFULL & env->SEGMask)) { > - ret = env->tlb->map_address(env, physical, prot, address, rw, access_type); > + ret = env->tlb->map_address(env, physical, prot, real_address, rw, access_type); > } else { > ret = TLBRET_BADADDR; > } > @@ -161,7 +180,7 @@ static int get_physical_address (CPUMIPSState *env, hwaddr *physical, > /* xkseg */ > if (kernel_mode && KX && > address <= (0xFFFFFFFF7FFFFFFFULL & env->SEGMask)) { > - ret = env->tlb->map_address(env, physical, prot, address, rw, access_type); > + ret = env->tlb->map_address(env, physical, prot, real_address, rw, access_type); > } else { > ret = TLBRET_BADADDR; > } > @@ -185,7 +204,7 @@ static int get_physical_address (CPUMIPSState *env, hwaddr *physical, > } else if (address < (int32_t)KSEG3_BASE) { > /* sseg (kseg2) */ > if (supervisor_mode || kernel_mode) { > - ret = env->tlb->map_address(env, physical, prot, address, rw, access_type); > + ret = env->tlb->map_address(env, physical, prot, real_address, rw, access_type); > } else { > ret = TLBRET_BADADDR; > } > @@ -193,7 +212,7 @@ static int get_physical_address (CPUMIPSState *env, hwaddr *physical, > /* kseg3 */ > /* XXX: debug segment is not emulated */ > if (kernel_mode) { > - ret = env->tlb->map_address(env, physical, prot, address, rw, access_type); > + ret = env->tlb->map_address(env, physical, prot, real_address, rw, access_type); > } else { > ret = TLBRET_BADADDR; > } Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG: 1024D/F1BCDB73 aurelien@aurel32.net http://www.aurel32.net From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40150) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VuOZk-00005a-BJ for qemu-devel@nongnu.org; Sat, 21 Dec 2013 10:34:53 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VuOZZ-00037k-Fw for qemu-devel@nongnu.org; Sat, 21 Dec 2013 10:34:48 -0500 Received: from hall.aurel32.net ([2001:bc8:30d7:101::1]:46172) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VuOZZ-00033l-6W for qemu-devel@nongnu.org; Sat, 21 Dec 2013 10:34:37 -0500 Date: Sat, 21 Dec 2013 16:34:17 +0100 From: Aurelien Jarno Message-ID: <20131221153417.GA9151@ohm.rr44.fr> References: <1387203165-5553-1-git-send-email-james.hogan@imgtec.com> <1387203165-5553-5-git-send-email-james.hogan@imgtec.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Content-Disposition: inline In-Reply-To: <1387203165-5553-5-git-send-email-james.hogan@imgtec.com> Subject: Re: [Qemu-devel] [PATCH v2 04/10] target-mips: get_physical_address: Add KVM awareness List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: James Hogan Cc: Paolo Bonzini , Gleb Natapov , qemu-devel@nongnu.org, kvm@vger.kernel.org, Sanjay Lal On Mon, Dec 16, 2013 at 02:12:39PM +0000, James Hogan wrote: > MIPS KVM trap & emulate mode (which is currently the only supported > mode) has to add an extra kseg0/kseg1 at 0x40000000 and an extra > kseg2/kseg3 at 0x60000000. Take this into account in > get_physical_address() so that debug memory access works. > > This is done by translating the address to a standard kseg0 or kseg2 > address before doing the normal address translation. The real virtual > address is still used for TLB lookups. > > Signed-off-by: James Hogan > Cc: Aurelien Jarno > --- > target-mips/helper.c | 33 ++++++++++++++++++++++++++------- > 1 file changed, 26 insertions(+), 7 deletions(-) > > diff --git a/target-mips/helper.c b/target-mips/helper.c > index 2e96655..c4be887 100644 > --- a/target-mips/helper.c > +++ b/target-mips/helper.c > @@ -24,6 +24,7 @@ > #include > > #include "cpu.h" > +#include "sysemu/kvm.h" > > enum { > TLBRET_DIRTY = -4, > @@ -100,7 +101,7 @@ int r4k_map_address (CPUMIPSState *env, hwaddr *physical, int *prot, > } > > static int get_physical_address (CPUMIPSState *env, hwaddr *physical, > - int *prot, target_ulong address, > + int *prot, target_ulong real_address, > int rw, int access_type) > { > /* User mode can only access useg/xuseg */ > @@ -113,6 +114,8 @@ static int get_physical_address (CPUMIPSState *env, hwaddr *physical, > int KX = (env->CP0_Status & (1 << CP0St_KX)) != 0; > #endif > int ret = TLBRET_MATCH; > + /* effective address (modified for KVM T&E kernel segments) */ > + target_ulong address = real_address; > > #if 0 > qemu_log("user mode %d h %08x\n", user_mode, env->hflags); > @@ -124,19 +127,35 @@ static int get_physical_address (CPUMIPSState *env, hwaddr *physical, > #define KSEG2_BASE 0xC0000000UL > #define KSEG3_BASE 0xE0000000UL > > +#define KVM_KSEG0_BASE 0x40000000UL > +#define KVM_KSEG2_BASE 0x60000000UL > + > + if (kvm_enabled()) { > + /* KVM T&E adds guest kernel segments in useg */ > + if (real_address >= KVM_KSEG0_BASE) { > + if (real_address < KVM_KSEG2_BASE) { > + /* kseg0 */ > + address += KSEG0_BASE - KVM_KSEG0_BASE; > + } else if (real_address <= USEG_LIMIT) { > + /* kseg2/3 */ > + address += KSEG2_BASE - KVM_KSEG2_BASE; > + } > + } > + } > + > if (address <= USEG_LIMIT) { > /* useg */ > if (env->CP0_Status & (1 << CP0St_ERL)) { > *physical = address & 0xFFFFFFFF; > *prot = PAGE_READ | PAGE_WRITE; > } else { > - ret = env->tlb->map_address(env, physical, prot, address, rw, access_type); > + ret = env->tlb->map_address(env, physical, prot, real_address, rw, access_type); > } > #if defined(TARGET_MIPS64) > } else if (address < 0x4000000000000000ULL) { > /* xuseg */ > if (UX && address <= (0x3FFFFFFFFFFFFFFFULL & env->SEGMask)) { > - ret = env->tlb->map_address(env, physical, prot, address, rw, access_type); > + ret = env->tlb->map_address(env, physical, prot, real_address, rw, access_type); > } else { > ret = TLBRET_BADADDR; > } > @@ -144,7 +163,7 @@ static int get_physical_address (CPUMIPSState *env, hwaddr *physical, > /* xsseg */ > if ((supervisor_mode || kernel_mode) && > SX && address <= (0x7FFFFFFFFFFFFFFFULL & env->SEGMask)) { > - ret = env->tlb->map_address(env, physical, prot, address, rw, access_type); > + ret = env->tlb->map_address(env, physical, prot, real_address, rw, access_type); > } else { > ret = TLBRET_BADADDR; > } > @@ -161,7 +180,7 @@ static int get_physical_address (CPUMIPSState *env, hwaddr *physical, > /* xkseg */ > if (kernel_mode && KX && > address <= (0xFFFFFFFF7FFFFFFFULL & env->SEGMask)) { > - ret = env->tlb->map_address(env, physical, prot, address, rw, access_type); > + ret = env->tlb->map_address(env, physical, prot, real_address, rw, access_type); > } else { > ret = TLBRET_BADADDR; > } > @@ -185,7 +204,7 @@ static int get_physical_address (CPUMIPSState *env, hwaddr *physical, > } else if (address < (int32_t)KSEG3_BASE) { > /* sseg (kseg2) */ > if (supervisor_mode || kernel_mode) { > - ret = env->tlb->map_address(env, physical, prot, address, rw, access_type); > + ret = env->tlb->map_address(env, physical, prot, real_address, rw, access_type); > } else { > ret = TLBRET_BADADDR; > } > @@ -193,7 +212,7 @@ static int get_physical_address (CPUMIPSState *env, hwaddr *physical, > /* kseg3 */ > /* XXX: debug segment is not emulated */ > if (kernel_mode) { > - ret = env->tlb->map_address(env, physical, prot, address, rw, access_type); > + ret = env->tlb->map_address(env, physical, prot, real_address, rw, access_type); > } else { > ret = TLBRET_BADADDR; > } Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG: 1024D/F1BCDB73 aurelien@aurel32.net http://www.aurel32.net