Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit fef093be authored by Alexander Graf's avatar Alexander Graf Committed by Avi Kivity
Browse files

KVM: PPC: Make use of hash based Shadow MMU 



We just introduced generic functions to handle shadow pages on PPC.
This patch makes the respective backends make use of them, getting
rid of a lot of duplicate code along the way.

Signed-off-by: default avatarAlexander Graf <agraf@suse.de>
Signed-off-by: default avatarMarcelo Tosatti <mtosatti@redhat.com>
parent 7741909b

arch/powerpc/include/asm/kvm_book3s.h

+9 −0
Original line number Diff line number Diff line
@@ -115,6 +115,15 @@ extern void kvmppc_mmu_book3s_32_init(struct kvm_vcpu *vcpu); 
extern int kvmppc_mmu_map_page(struct kvm_vcpu *vcpu, struct kvmppc_pte *pte);

extern int kvmppc_mmu_map_segment(struct kvm_vcpu *vcpu, ulong eaddr);

extern void kvmppc_mmu_flush_segments(struct kvm_vcpu *vcpu);



extern void kvmppc_mmu_hpte_cache_map(struct kvm_vcpu *vcpu, struct hpte_cache *pte);

extern struct hpte_cache *kvmppc_mmu_hpte_cache_next(struct kvm_vcpu *vcpu);

extern void kvmppc_mmu_hpte_destroy(struct kvm_vcpu *vcpu);

extern int kvmppc_mmu_hpte_init(struct kvm_vcpu *vcpu);

extern void kvmppc_mmu_invalidate_pte(struct kvm_vcpu *vcpu, struct hpte_cache *pte);

extern int kvmppc_mmu_hpte_sysinit(void);

extern void kvmppc_mmu_hpte_sysexit(void);



extern int kvmppc_ld(struct kvm_vcpu *vcpu, ulong *eaddr, int size, void *ptr, bool data);

extern int kvmppc_st(struct kvm_vcpu *vcpu, ulong *eaddr, int size, void *ptr, bool data);

extern void kvmppc_book3s_queue_irqprio(struct kvm_vcpu *vcpu, unsigned int vec);

arch/powerpc/include/asm/kvm_host.h

+14 −3
Original line number Diff line number Diff line
@@ -38,7 +38,13 @@ 
#define KVM_NR_PAGE_SIZES	1

#define KVM_PAGES_PER_HPAGE(x)	(1UL<<31)



#define HPTEG_CACHE_NUM 1024

#define HPTEG_CACHE_NUM			(1 << 15)

#define HPTEG_HASH_BITS_PTE		13

#define HPTEG_HASH_BITS_VPTE		13

#define HPTEG_HASH_BITS_VPTE_LONG	5

#define HPTEG_HASH_NUM_PTE		(1 << HPTEG_HASH_BITS_PTE)

#define HPTEG_HASH_NUM_VPTE		(1 << HPTEG_HASH_BITS_VPTE)

#define HPTEG_HASH_NUM_VPTE_LONG	(1 << HPTEG_HASH_BITS_VPTE_LONG)



struct kvm;

struct kvm_run;
@@ -151,6 +157,9 @@ struct kvmppc_mmu { 
};



struct hpte_cache {

	struct hlist_node list_pte;

	struct hlist_node list_vpte;

	struct hlist_node list_vpte_long;

	u64 host_va;

	u64 pfn;

	ulong slot;
@@ -282,8 +291,10 @@ struct kvm_vcpu_arch { 
	unsigned long pending_exceptions;



#ifdef CONFIG_PPC_BOOK3S

	struct hpte_cache hpte_cache[HPTEG_CACHE_NUM];

	int hpte_cache_offset;

	struct hlist_head hpte_hash_pte[HPTEG_HASH_NUM_PTE];

	struct hlist_head hpte_hash_vpte[HPTEG_HASH_NUM_VPTE];

	struct hlist_head hpte_hash_vpte_long[HPTEG_HASH_NUM_VPTE_LONG];

	int hpte_cache_count;

#endif

};

arch/powerpc/kvm/Makefile

+2 −0
Original line number Diff line number Diff line
@@ -45,6 +45,7 @@ kvm-book3s_64-objs := \ 
	book3s.o \

	book3s_emulate.o \

	book3s_interrupts.o \

	book3s_mmu_hpte.o \

	book3s_64_mmu_host.o \

	book3s_64_mmu.o \

	book3s_32_mmu.o
@@ -57,6 +58,7 @@ kvm-book3s_32-objs := \ 
	book3s.o \

	book3s_emulate.o \

	book3s_interrupts.o \

	book3s_mmu_hpte.o \

	book3s_32_mmu_host.o \

	book3s_32_mmu.o

kvm-objs-$(CONFIG_KVM_BOOK3S_32) := $(kvm-book3s_32-objs)

arch/powerpc/kvm/book3s.c

+12 −2
Original line number Diff line number Diff line
@@ -1384,12 +1384,22 @@ int __kvmppc_vcpu_run(struct kvm_run *kvm_run, struct kvm_vcpu *vcpu) 


static int kvmppc_book3s_init(void)

{

	return kvm_init(NULL, sizeof(struct kvmppc_vcpu_book3s), 0,

	int r;



	r = kvm_init(NULL, sizeof(struct kvmppc_vcpu_book3s), 0,

		     THIS_MODULE);



	if (r)

		return r;



	r = kvmppc_mmu_hpte_sysinit();



	return r;

}



static void kvmppc_book3s_exit(void)

{

	kvmppc_mmu_hpte_sysexit();

	kvm_exit();

}

arch/powerpc/kvm/book3s_32_mmu_host.c

+10 −94
Original line number Diff line number Diff line
@@ -58,105 +58,19 @@ 
static ulong htab;

static u32 htabmask;



static void invalidate_pte(struct kvm_vcpu *vcpu, struct hpte_cache *pte)

void kvmppc_mmu_invalidate_pte(struct kvm_vcpu *vcpu, struct hpte_cache *pte)

{

	volatile u32 *pteg;



	dprintk_mmu("KVM: Flushing SPTE: 0x%llx (0x%llx) -> 0x%llx\n",

		    pte->pte.eaddr, pte->pte.vpage, pte->host_va);



	/* Remove from host HTAB */

	pteg = (u32*)pte->slot;



	pteg[0] = 0;



	/* And make sure it's gone from the TLB too */

	asm volatile ("sync");

	asm volatile ("tlbie %0" : : "r" (pte->pte.eaddr) : "memory");

	asm volatile ("sync");

	asm volatile ("tlbsync");



	pte->host_va = 0;



	if (pte->pte.may_write)

		kvm_release_pfn_dirty(pte->pfn);

	else

		kvm_release_pfn_clean(pte->pfn);

}



void kvmppc_mmu_pte_flush(struct kvm_vcpu *vcpu, ulong guest_ea, ulong ea_mask)

{

	int i;



	dprintk_mmu("KVM: Flushing %d Shadow PTEs: 0x%x & 0x%x\n",

		    vcpu->arch.hpte_cache_offset, guest_ea, ea_mask);

	BUG_ON(vcpu->arch.hpte_cache_offset > HPTEG_CACHE_NUM);



	guest_ea &= ea_mask;

	for (i = 0; i < vcpu->arch.hpte_cache_offset; i++) {

		struct hpte_cache *pte;



		pte = &vcpu->arch.hpte_cache[i];

		if (!pte->host_va)

			continue;



		if ((pte->pte.eaddr & ea_mask) == guest_ea) {

			invalidate_pte(vcpu, pte);

		}

	}



	/* Doing a complete flush -> start from scratch */

	if (!ea_mask)

		vcpu->arch.hpte_cache_offset = 0;

}



void kvmppc_mmu_pte_vflush(struct kvm_vcpu *vcpu, u64 guest_vp, u64 vp_mask)

{

	int i;



	dprintk_mmu("KVM: Flushing %d Shadow vPTEs: 0x%llx & 0x%llx\n",

		    vcpu->arch.hpte_cache_offset, guest_vp, vp_mask);

	BUG_ON(vcpu->arch.hpte_cache_offset > HPTEG_CACHE_NUM);



	guest_vp &= vp_mask;

	for (i = 0; i < vcpu->arch.hpte_cache_offset; i++) {

		struct hpte_cache *pte;



		pte = &vcpu->arch.hpte_cache[i];

		if (!pte->host_va)

			continue;



		if ((pte->pte.vpage & vp_mask) == guest_vp) {

			invalidate_pte(vcpu, pte);

		}

	}

}



void kvmppc_mmu_pte_pflush(struct kvm_vcpu *vcpu, ulong pa_start, ulong pa_end)

{

	int i;



	dprintk_mmu("KVM: Flushing %d Shadow pPTEs: 0x%llx & 0x%llx\n",

		    vcpu->arch.hpte_cache_offset, pa_start, pa_end);

	BUG_ON(vcpu->arch.hpte_cache_offset > HPTEG_CACHE_NUM);



	for (i = 0; i < vcpu->arch.hpte_cache_offset; i++) {

		struct hpte_cache *pte;



		pte = &vcpu->arch.hpte_cache[i];

		if (!pte->host_va)

			continue;



		if ((pte->pte.raddr >= pa_start) &&

		    (pte->pte.raddr < pa_end)) {

			invalidate_pte(vcpu, pte);

		}

	}

}



static int kvmppc_mmu_hpte_cache_next(struct kvm_vcpu *vcpu)

{

	if (vcpu->arch.hpte_cache_offset == HPTEG_CACHE_NUM)

		kvmppc_mmu_pte_flush(vcpu, 0, 0);



	return vcpu->arch.hpte_cache_offset++;

}



/* We keep 512 gvsid->hvsid entries, mapping the guest ones to the array using
@@ -230,7 +144,6 @@ int kvmppc_mmu_map_page(struct kvm_vcpu *vcpu, struct kvmppc_pte *orig_pte) 
	register int rr = 0;

	bool primary = false;

	bool evict = false;

	int hpte_id;

	struct hpte_cache *pte;



	/* Get host physical address for gpa */
@@ -315,8 +228,7 @@ int kvmppc_mmu_map_page(struct kvm_vcpu *vcpu, struct kvmppc_pte *orig_pte) 


	/* Now tell our Shadow PTE code about the new page */



	hpte_id = kvmppc_mmu_hpte_cache_next(vcpu);

	pte = &vcpu->arch.hpte_cache[hpte_id];

	pte = kvmppc_mmu_hpte_cache_next(vcpu);



	dprintk_mmu("KVM: %c%c Map 0x%llx: [%lx] 0x%llx (0x%llx) -> %lx\n",

		    orig_pte->may_write ? 'w' : '-',
@@ -329,6 +241,8 @@ int kvmppc_mmu_map_page(struct kvm_vcpu *vcpu, struct kvmppc_pte *orig_pte) 
	pte->pte = *orig_pte;

	pte->pfn = hpaddr >> PAGE_SHIFT;



	kvmppc_mmu_hpte_cache_map(vcpu, pte);



	return 0;

}
@@ -413,7 +327,7 @@ void kvmppc_mmu_flush_segments(struct kvm_vcpu *vcpu) 


void kvmppc_mmu_destroy(struct kvm_vcpu *vcpu)

{

	kvmppc_mmu_pte_flush(vcpu, 0, 0);

	kvmppc_mmu_hpte_destroy(vcpu);

	preempt_disable();

	__destroy_context(to_book3s(vcpu)->context_id);

	preempt_enable();
@@ -453,5 +367,7 @@ int kvmppc_mmu_init(struct kvm_vcpu *vcpu) 
	htabmask = ((sdr1 & 0x1FF) << 16) | 0xFFC0;

	htab = (ulong)__va(sdr1 & 0xffff0000);



	kvmppc_mmu_hpte_init(vcpu);



	return 0;

}