Skip to content

Commit

Permalink
Eliminate USERTOP; no use for it.
Browse files Browse the repository at this point in the history
  • Loading branch information
Frans Kaashoek committed Aug 24, 2011
1 parent 194f8bf commit 6f23275
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
1 change: 0 additions & 1 deletion memlayout.h
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@

// Key addresses for address space layout (see kmap in vm.c for the layout)
#define KERNBASE 0xF0000000 // First kernel virtual address
#define USERTOP (KERNBASE-PGSIZE) // Highest user virtual address
#define KERNLINK (KERNBASE+EXTMEM) // Address where kernel is linked

#ifndef __ASSEMBLER__
Expand Down
6 changes: 3 additions & 3 deletions vm.c
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ mappages(pde_t *pgdir, void *va, uint size, uint pa, int perm, char* (*alloc)(vo
// than its memory.
//
// setupkvm() and exec() set up every page table like this:
// 0..USERTOP : user memory (text, data, stack, heap), mapped to some unused phys mem
// 0..KERNBASE : user memory (text, data, stack, heap), mapped to some unused phys mem
// KERNBASE..KERNBASE+EXTMEM: mapped to 0..EXTMEM (below extended memory)
// KERNBASE+EXTMEM..KERNBASE+end : mapped to EXTMEM..end (mapped without write permission)
// KERNBASE+end..KERBASE+PHYSTOP : mapped to end..PHYSTOP (rw data + free memory)
Expand Down Expand Up @@ -223,7 +223,7 @@ allocuvm(pde_t *pgdir, uint oldsz, uint newsz)
char *mem;
uint a;

if(newsz > USERTOP)
if(newsz >= KERNBASE)
return 0;
if(newsz < oldsz)
return oldsz;
Expand Down Expand Up @@ -279,7 +279,7 @@ freevm(pde_t *pgdir)

if(pgdir == 0)
panic("freevm: no pgdir");
deallocuvm(pgdir, USERTOP, 0);
deallocuvm(pgdir, KERNBASE, 0);
for(i = 0; i < NPDENTRIES; i++){
if(pgdir[i] & PTE_P) {
char * v = p2v(PTE_ADDR(pgdir[i]));
Expand Down

0 comments on commit 6f23275

Please sign in to comment.