mirrors/qemu
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

pseries: More complete WIMG validation in H_ENTER code

Browse Source 
Currently our implementation of the H_ENTER hypercall, which inserts a
mapping in the hash page table assumes that only ordinary memory is ever
mapped, and only permits mapping attribute bits accordingly (WIMG==0010).

However, we intend to start adding emulated IO to the pseries platform
(and real IO with PCI passthrough on kvm) which means this simple test
will no longer suffice.

This patch extends the h_enter validation code to check if the given
address is a RAM address.  If it is it enforces WIMG==0010, otherwise
it assumes that it is an IO mapping and instead enforces WIMG=010x.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Alexander Graf <agraf@suse.de>
This commit is contained in:
David Gibson 2011-08-03 21:02:19 +00:00 committed by Alexander Graf
parent 9dfef5aae4
commit f73a2575a3
3 changed files with 21 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
hw/spapr.c
View File

@ -336,7 +336,8 @@ static void ppc_spapr_init(ram_addr_t ram_size,
} }
/* allocate RAM */ /* allocate RAM */
ram_offset = qemu_ram_alloc(NULL, "ppc_spapr.ram", ram_size); spapr->ram_limit = ram_size;
ram_offset = qemu_ram_alloc(NULL, "ppc_spapr.ram", spapr->ram_limit);
cpu_register_physical_memory(0, ram_size, ram_offset); cpu_register_physical_memory(0, ram_size, ram_offset);
/* allocate hash page table. For now we always make this 16mb, /* allocate hash page table. For now we always make this 16mb,

1
hw/spapr.h
View File

@ -10,6 +10,7 @@ typedef struct sPAPREnvironment {
struct VIOsPAPRBus *vio_bus; struct VIOsPAPRBus *vio_bus;
struct icp_state *icp; struct icp_state *icp;
target_phys_addr_t ram_limit;
void *htab; void *htab;
long htab_size; long htab_size;
target_phys_addr_t fdt_addr, rtas_addr; target_phys_addr_t fdt_addr, rtas_addr;

22
hw/spapr_hcall.c
View File

@ -99,6 +99,8 @@ static target_ulong h_enter(CPUState *env, sPAPREnvironment *spapr,
target_ulong pte_index = args[1]; target_ulong pte_index = args[1];
target_ulong pteh = args[2]; target_ulong pteh = args[2];
target_ulong ptel = args[3]; target_ulong ptel = args[3];
target_ulong page_shift = 12;
target_ulong raddr;
target_ulong i; target_ulong i;
uint8_t *hpte; uint8_t *hpte;
@ -111,6 +113,7 @@ static target_ulong h_enter(CPUState *env, sPAPREnvironment *spapr,
#endif #endif
if ((ptel & 0xff000) == 0) { if ((ptel & 0xff000) == 0) {
/* 16M page */ /* 16M page */
page_shift = 24;
/* lowest AVA bit must be 0 for 16M pages */ /* lowest AVA bit must be 0 for 16M pages */
if (pteh & 0x80) { if (pteh & 0x80) {
return H_PARAMETER; return H_PARAMETER;
@ -120,12 +123,23 @@ static target_ulong h_enter(CPUState *env, sPAPREnvironment *spapr,
} }
} }
/* FIXME: bounds check the pa? */ raddr = (ptel & HPTE_R_RPN) & ~((1ULL << page_shift) - 1);
/* Check WIMG */ if (raddr < spapr->ram_limit) {
if ((ptel & HPTE_R_WIMG) != HPTE_R_M) { /* Regular RAM - should have WIMG=0010 */
return H_PARAMETER; if ((ptel & HPTE_R_WIMG) != HPTE_R_M) {
return H_PARAMETER;
}
} else {
/* Looks like an IO address */
/* FIXME: What WIMG combinations could be sensible for IO?
* For now we allow WIMG=010x, but are there others? */
/* FIXME: Should we check against registered IO addresses? */
if ((ptel & (HPTE_R_W | HPTE_R_I | HPTE_R_M)) != HPTE_R_I) {
return H_PARAMETER;
}
} }
pteh &= ~0x60ULL; pteh &= ~0x60ULL;
if ((pte_index * HASH_PTE_SIZE_64) & ~env->htab_mask) { if ((pte_index * HASH_PTE_SIZE_64) & ~env->htab_mask) {