Merge commit 'origin' into net-dev
[akaros.git] / kern / src / env.c
1 /* See COPYRIGHT for copyright information. */
2 #ifdef __DEPUTY__
3 //#pragma nodeputy
4 #pragma noasync
5 #endif
6
7 #include <arch/x86.h>
8 #include <arch/mmu.h>
9 #include <arch/elf.h>
10 #include <arch/apic.h>
11 #include <arch/smp.h>
12
13 #include <atomic.h>
14 #include <string.h>
15 #include <assert.h>
16 #include <env.h>
17 #include <pmap.h>
18 #include <trap.h>
19 #include <monitor.h>
20 #include <manager.h>
21
22 #include <ros/syscall.h>
23 #include <ros/error.h>
24
25 env_t *envs = NULL;             // All environments
26 uint32_t num_envs = 0;          // Number of envs
27 // TODO: make this a struct of info including the pointer and cacheline-align it
28 // This lets the kernel know what process is running on the core it traps into.
29 // A lot of the Env business, including this and its usage, will change when we
30 // redesign the env as a multi-process.
31 env_t* curenvs[MAX_NUM_CPUS] = {[0 ... (MAX_NUM_CPUS-1)] NULL};
32 static env_list_t env_free_list;        // Free list
33
34 #define ENVGENSHIFT     12              // >= LOGNENV
35
36 //
37 // Converts an envid to an env pointer.
38 //
39 // RETURNS
40 //   0 on success, -EBADENV on error.
41 //   On success, sets *env_store to the environment.
42 //   On error, sets *env_store to NULL.
43 //
44 int
45 envid2env(envid_t envid, env_t **env_store, bool checkperm)
46 {
47         env_t *e;
48         env_t* curenv = curenvs[lapic_get_id()];
49
50         // If envid is zero, return the current environment.
51         if (envid == 0) {
52                 *env_store = curenv;
53                 return 0;
54         }
55
56         // Look up the Env structure via the index part of the envid,
57         // then check the env_id field in that env_t
58         // to ensure that the envid is not stale
59         // (i.e., does not refer to a _previous_ environment
60         // that used the same slot in the envs[] array).
61         e = &envs[ENVX(envid)];
62         if (e->env_status == ENV_FREE || e->env_id != envid) {
63                 *env_store = 0;
64                 return -EBADENV;
65         }
66
67         // Check that the calling environment has legitimate permission
68         // to manipulate the specified environment.
69         // If checkperm is set, the specified environment
70         // must be either the current environment
71         // or an immediate child of the current environment.
72         if (checkperm && e != curenv && e->env_parent_id != curenv->env_id) {
73                 *env_store = 0;
74                 return -EBADENV;
75         }
76
77         *env_store = e;
78         return 0;
79 }
80
81 //
82 // Mark all environments in 'envs' as free, set their env_ids to 0,
83 // and insert them into the env_free_list.
84 // Insert in reverse order, so that the first call to env_alloc()
85 // returns envs[0].
86 //
87 void
88 env_init(void)
89 {
90         int i;
91         LIST_INIT(&env_free_list);
92         for (i = NENV-1; i >= 0; i--) {
93                 // these should already be set from when i memset'd the array to 0
94                 envs[i].env_status = ENV_FREE;
95                 envs[i].env_id = 0;
96                 LIST_INSERT_HEAD(&env_free_list, &envs[i], env_link);
97         }
98 }
99
100 //
101 // Initialize the kernel virtual memory layout for environment e.
102 // Allocate a page directory, set e->env_pgdir and e->env_cr3 accordingly,
103 // and initialize the kernel portion of the new environment's address space.
104 // Do NOT (yet) map anything into the user portion
105 // of the environment's virtual address space.
106 //
107 // Returns 0 on success, < 0 on error.  Errors include:
108 //      -ENOMEM if page directory or table could not be allocated.
109 //
110 static int
111 env_setup_vm(env_t *e)
112 WRITES(e->env_pgdir, e->env_cr3, e->env_procinfo, e->env_procdata,
113        e->env_sysbackring)
114 {
115         int i, r;
116         page_t *pgdir = NULL, *pginfo = NULL, *pgdata = NULL;
117
118         // Allocate pages for the page directory, shared info, and shared data pages
119         r = page_alloc(&pgdir);
120         r = page_alloc(&pginfo);
121         r = page_alloc(&pgdata);
122         if (r < 0) {
123                 page_free(pgdir);
124                 page_free(pginfo);
125                 return r;
126         }
127
128         // Now, set e->env_pgdir and e->env_cr3,
129         // and initialize the page directory.
130         //
131         // Hint:
132         //    - The VA space of all envs is identical above UTOP
133         //      (except at VPT and UVPT, which we've set below).
134         //      (and not for UINFO either)
135         //      See inc/memlayout.h for permissions and layout.
136         //      Can you use boot_pgdir as a template?  Hint: Yes.
137         //      (Make sure you got the permissions right in Lab 2.)
138         //    - The initial VA below UTOP is empty.
139         //    - You do not need to make any more calls to page_alloc.
140         //    - Note: pp_ref is not maintained for most physical pages
141         //      mapped above UTOP -- but you do need to increment
142         //      env_pgdir's pp_ref!
143
144         // need to up pgdir's reference, since it will never be done elsewhere
145         pgdir->pp_ref++;
146         e->env_pgdir = page2kva(pgdir);
147         e->env_cr3 = page2pa(pgdir);
148         e->env_procinfo = page2kva(pginfo);
149         e->env_procdata = page2kva(pgdata);
150
151         memset(e->env_pgdir, 0, PGSIZE);
152         memset(e->env_procinfo, 0, PGSIZE);
153         memset(e->env_procdata, 0, PGSIZE);
154
155         // Initialize the generic syscall ring buffer
156         SHARED_RING_INIT((syscall_sring_t *SAFE)e->env_procdata);
157         // Initialize the backend of the ring buffer
158         BACK_RING_INIT(&e->env_sysbackring, (syscall_sring_t *SAFE)e->env_procdata,
159                        PGSIZE);
160
161         // should be able to do this so long as boot_pgdir never has
162         // anything put below UTOP
163         memcpy(e->env_pgdir, boot_pgdir, PGSIZE);
164
165         // something like this.  TODO, if you want
166         //memcpy(&e->env_pgdir[PDX(UTOP)], &boot_pgdir[PDX(UTOP)], PGSIZE - PDX(UTOP));
167         // check with
168         // assert(memcmp(e->env_pgdir, boot_pgdir, PGSIZE) == 0);
169
170         // VPT and UVPT map the env's own page table, with
171         // different permissions.
172         e->env_pgdir[PDX(VPT)]  = e->env_cr3 | PTE_P | PTE_W;
173         e->env_pgdir[PDX(UVPT)] = e->env_cr3 | PTE_P | PTE_U;
174
175         // Insert the per-process info and data pages into this process's pgdir
176         // I don't want to do these two pages later (like with the stack), since
177         // the kernel wants to keep pointers to it easily.
178         // Could place all of this with a function that maps a shared memory page
179         // that can work between any two address spaces or something.
180         r = page_insert(e->env_pgdir, pginfo, (void*SNT)UINFO, PTE_U);
181         r = page_insert(e->env_pgdir, pgdata, (void*SNT)UDATA, PTE_U | PTE_W);
182         if (r < 0) {
183                 // note that we can't currently deallocate the pages created by
184                 // pgdir_walk (inside insert).  should be able to gather them up when
185                 // we destroy environments and their page tables.
186                 page_free(pgdir);
187                 page_free(pginfo);
188                 page_free(pgdata);
189                 return r;
190         }
191
192         /* Shared page for all processes.  Can't be trusted, but still very useful
193          * at this stage for us.  Consider removing when we have real processes.
194          * (TODO).  Note the page is alloced only the first time through
195          */
196         static page_t* shared_page = 0;
197         if (!shared_page)
198                 page_alloc(&shared_page);
199         // Up it, so it never goes away.  One per user, plus one from page_alloc
200         // This is necessary, since it's in the per-process range of memory that
201         // gets freed during page_free.
202         shared_page->pp_ref++;
203
204         // Inserted into every process's address space at UGDATA
205         page_insert(e->env_pgdir, shared_page, (void*SNT)UGDATA, PTE_U | PTE_W);
206
207         return 0;
208 }
209
210 //
211 // Allocates and initializes a new environment.
212 // On success, the new environment is stored in *newenv_store.
213 //
214 // Returns 0 on success, < 0 on failure.  Errors include:
215 //      -ENOFREEENV if all NENVS environments are allocated
216 //      -ENOMEM on memory exhaustion
217 //
218 int
219 env_alloc(env_t **newenv_store, envid_t parent_id)
220 {
221         int32_t generation;
222         int r;
223         env_t *e;
224
225         if (!(e = LIST_FIRST(&env_free_list)))
226                 return -ENOFREEENV;
227         
228         //memset((void*)e + sizeof(e->env_link), 0, sizeof(*e) - sizeof(e->env_link));
229
230     { INITSTRUCT(*e)
231
232         // Allocate and set up the page directory for this environment.
233         if ((r = env_setup_vm(e)) < 0)
234                 return r;
235
236         // Generate an env_id for this environment.
237         generation = (e->env_id + (1 << ENVGENSHIFT)) & ~(NENV - 1);
238         if (generation <= 0)    // Don't create a negative env_id.
239                 generation = 1 << ENVGENSHIFT;
240         e->env_id = generation | (e - envs);
241
242         // Set the basic status variables.
243     e->lock = 0;
244         e->env_parent_id = parent_id;
245         e->env_status = ENV_RUNNABLE;
246         e->env_runs = 0;
247         e->env_refcnt = 1;
248         e->env_flags = 0;
249
250         // Clear out all the saved register state,
251         // to prevent the register values
252         // of a prior environment inhabiting this Env structure
253         // from "leaking" into our new environment.
254         memset(&e->env_tf, 0, sizeof(e->env_tf));
255
256         // Set up appropriate initial values for the segment registers.
257         // GD_UD is the user data segment selector in the GDT, and
258         // GD_UT is the user text segment selector (see inc/memlayout.h).
259         // The low 2 bits of each segment register contains the
260         // Requestor Privilege Level (RPL); 3 means user mode.
261         e->env_tf.tf_ds = GD_UD | 3;
262         e->env_tf.tf_es = GD_UD | 3;
263         e->env_tf.tf_ss = GD_UD | 3;
264         e->env_tf.tf_esp = USTACKTOP;
265         e->env_tf.tf_cs = GD_UT | 3;
266         // You will set e->env_tf.tf_eip later.
267         // set the env's EFLAGSs to have interrupts enabled
268         e->env_tf.tf_eflags |= 0x00000200; // bit 9 is the interrupts-enabled
269
270         // commit the allocation
271         LIST_REMOVE(e, env_link);
272         *newenv_store = e;
273         atomic_inc(&num_envs);
274
275         e->env_tscfreq = system_timing.tsc_freq;
276         // TODO: for now, the only info at procinfo is this env's struct
277         // note that we need to copy this over every time we make a change to env
278         // that we want userspace to see.  also note that we don't even want to
279         // show them all of env, only specific things like PID, PPID, etc
280         memcpy(e->env_procinfo, e, sizeof(env_t));
281
282         env_t* curenv = curenvs[lapic_get_id()];
283
284         printk("[%08x] new env %08x\n", curenv ? curenv->env_id : 0, e->env_id);
285         } // INIT_STRUCT
286         return 0;
287 }
288
289 //
290 // Allocate len bytes of physical memory for environment env,
291 // and map it at virtual address va in the environment's address space.
292 // Does not zero or otherwise initialize the mapped pages in any way.
293 // Pages should be writable by user and kernel.
294 // Panic if any allocation attempt fails.
295 //
296 static void
297 segment_alloc(env_t *e, void *SNT va, size_t len)
298 {
299         void *SNT start, *SNT end;
300         size_t num_pages;
301         int i, r;
302         page_t *page;
303         pte_t *pte;
304
305         start = ROUNDDOWN(va, PGSIZE);
306         end = ROUNDUP(va + len, PGSIZE);
307         if (start >= end)
308                 panic("Wrap-around in memory allocation addresses!");
309         if ((uintptr_t)end > UTOP)
310                 panic("Attempting to map above UTOP!");
311         // page_insert/pgdir_walk alloc a page and read/write to it via its address
312         // starting from pgdir (e's), so we need to be using e's pgdir
313         assert(e->env_cr3 == rcr3());
314         num_pages = PPN(end - start);
315         for (i = 0; i < num_pages; i++, start += PGSIZE) {
316                 // skip if a page is already mapped.  yes, page_insert will page_remove
317                 // whatever page was already there, but if we are seg allocing adjacent
318                 // regions, we don't want to destroy that old mapping/page
319                 // though later on we are told we can ignore this...
320                 pte = pgdir_walk(e->env_pgdir, start, 0);
321                 if (pte && *pte & PTE_P)
322                         continue;
323                 if ((r = page_alloc(&page)) < 0)
324                         panic("segment_alloc: %e", r);
325                 page_insert(e->env_pgdir, page, start, PTE_U | PTE_W);
326         }
327 }
328
329 //
330 // Set up the initial program binary, stack, and processor flags
331 // for a user process.
332 // This function is ONLY called during kernel initialization,
333 // before running the first user-mode environment.
334 //
335 // This function loads all loadable segments from the ELF binary image
336 // into the environment's user memory, starting at the appropriate
337 // virtual addresses indicated in the ELF program header.
338 // At the same time it clears to zero any portions of these segments
339 // that are marked in the program header as being mapped
340 // but not actually present in the ELF file - i.e., the program's bss section.
341 //
342 // All this is very similar to what our boot loader does, except the boot
343 // loader also needs to read the code from disk.  Take a look at
344 // boot/main.c to get ideas.
345 //
346 // Finally, this function maps one page for the program's initial stack.
347 //
348 // load_icode panics if it encounters problems.
349 //  - How might load_icode fail?  What might be wrong with the given input?
350 //
351 static void
352 load_icode(env_t *e, uint8_t *COUNT(size) binary, size_t size)
353 {
354         // Hints:
355         //  Load each program segment into virtual memory
356         //  at the address specified in the ELF section header.
357         //  You should only load segments with ph->p_type == ELF_PROG_LOAD.
358         //  Each segment's virtual address can be found in ph->p_va
359         //  and its size in memory can be found in ph->p_memsz.
360         //  The ph->p_filesz bytes from the ELF binary, starting at
361         //  'binary + ph->p_offset', should be copied to virtual address
362         //  ph->p_va.  Any remaining memory bytes should be cleared to zero.
363         //  (The ELF header should have ph->p_filesz <= ph->p_memsz.)
364         //  Use functions from the previous lab to allocate and map pages.
365         //
366         //  All page protection bits should be user read/write for now.
367         //  ELF segments are not necessarily page-aligned, but you can
368         //  assume for this function that no two segments will touch
369         //  the same virtual page.
370         //
371         //  You may find a function like segment_alloc useful.
372         //
373         //  Loading the segments is much simpler if you can move data
374         //  directly into the virtual addresses stored in the ELF binary.
375         //  So which page directory should be in force during
376         //  this function?
377         //
378         // Hint:
379         //  You must also do something with the program's entry point,
380         //  to make sure that the environment starts executing there.
381         //  What?  (See env_run() and env_pop_tf() below.)
382
383         elf_t *elfhdr = (elf_t *)binary;
384         int i, r;
385
386         // is this an elf?
387         assert(elfhdr->e_magic == ELF_MAGIC);
388         // make sure we have proghdrs to load
389         assert(elfhdr->e_phnum);
390
391         // to actually access any pages alloc'd for this environment, we
392         // need to have the hardware use this environment's page tables.
393         // we can use e's tables as long as we want, since it has the same
394         // mappings for the kernel as does boot_pgdir
395         lcr3(e->env_cr3);
396
397         // TODO: how do we do a runtime COUNT?
398         {TRUSTEDBLOCK
399         proghdr_t* phdr = (proghdr_t*)(binary + elfhdr->e_phoff);
400         for (i = 0; i < elfhdr->e_phnum; i++, phdr++) {
401         // zra: TRUSTEDBLOCK until validation is done.
402                 if (phdr->p_type != ELF_PROG_LOAD)
403                         continue;
404         // TODO: validate elf header fields!
405                 // seg alloc creates PTE_U|PTE_W pages.  if you ever want to change
406                 // this, there will be issues with overlapping sections
407                 segment_alloc(e, (void*SNT)phdr->p_va, phdr->p_memsz);
408                 memcpy((void*)phdr->p_va, binary + phdr->p_offset, phdr->p_filesz);
409                 memset((void*)phdr->p_va + phdr->p_filesz, 0, phdr->p_memsz - phdr->p_filesz);
410         }}
411
412         e->env_tf.tf_eip = elfhdr->e_entry;
413
414         // Now map one page for the program's initial stack
415         // at virtual address USTACKTOP - PGSIZE.
416
417         segment_alloc(e, (void*SNT)(USTACKTOP - PGSIZE), PGSIZE);
418 }
419
420 //
421 // Allocates a new env and loads the named elf binary into it.
422 //
423 env_t* env_create(uint8_t *binary, size_t size)
424 {
425         env_t *e;
426         int r;
427
428         if ((r = env_alloc(&e, 0)) < 0)
429                 panic("env_create: %e", r);
430         load_icode(e, binary, size);
431         return e;
432 }
433
434 //
435 // Frees env e and all memory it uses.
436 //
437 void
438 env_free(env_t *e)
439 {
440         pte_t *pt;
441         uint32_t pdeno, pteno;
442         physaddr_t pa;
443
444         // Note the environment's demise.
445         env_t* curenv = curenvs[lapic_get_id()];
446         cprintf("[%08x] free env %08x\n", curenv ? curenv->env_id : 0, e->env_id);
447
448         // Flush all mapped pages in the user portion of the address space
449         static_assert(UTOP % PTSIZE == 0);
450         for (pdeno = 0; pdeno < PDX(UTOP); pdeno++) {
451
452                 // only look at mapped page tables
453                 if (!(e->env_pgdir[pdeno] & PTE_P))
454                         continue;
455
456                 // find the pa and va of the page table
457                 pa = PTE_ADDR(e->env_pgdir[pdeno]);
458                 pt = (pte_t*COUNT(NPTENTRIES)) KADDR(pa);
459
460                 // unmap all PTEs in this page table
461                 for (pteno = 0; pteno <= PTX(~0); pteno++) {
462                         if (pt[pteno] & PTE_P)
463                                 page_remove(e->env_pgdir, PGADDR(pdeno, pteno, 0));
464                 }
465
466                 // free the page table itself
467                 e->env_pgdir[pdeno] = 0;
468                 page_decref(pa2page(pa));
469         }
470
471         // Moved to page_decref
472         // need a known good pgdir before releasing the old one
473         //lcr3(boot_cr3);
474
475         // free the page directory
476         pa = e->env_cr3;
477         e->env_pgdir = 0;
478         e->env_cr3 = 0;
479         page_decref(pa2page(pa));
480
481         // return the environment to the free list
482         e->env_status = ENV_FREE;
483         LIST_INSERT_HEAD(&env_free_list, e, env_link);
484 }
485
486 /*
487  * This allows the kernel to keep this process around, in case it is being used
488  * in some asynchronous processing.
489  * The refcnt should always be greater than 0 for processes that aren't dying.
490  * When refcnt is 0, the process is dying and should not allow any more increfs.
491  * TODO: Make sure this is never called from an interrupt handler (irq_save)
492  */
493 error_t env_incref(env_t* e)
494 {
495         error_t retval = 0;
496         spin_lock(&e->lock);
497         if (e->env_refcnt)
498                 e->env_refcnt++;
499         else
500                 retval = -EBADENV;
501         spin_unlock(&e->lock);
502         return retval;
503 }
504
505 /*
506  * When the kernel is done with a process, it decrements its reference count.
507  * When the count hits 0, no one is using it and it should be freed.
508  * env_destroy calls this.
509  * TODO: Make sure this is never called from an interrupt handler (irq_save)
510  */
511 void env_decref(env_t* e)
512 {
513         // need a known good pgdir before releasing the old one
514         // sometimes env_free is called on a different core than decref
515         lcr3(boot_cr3);
516
517         spin_lock(&e->lock);
518         e->env_refcnt--;
519         spin_unlock(&e->lock);
520         // if we hit 0, no one else will increment and we can check outside the lock
521         if (e->env_refcnt == 0)
522                 env_free(e);
523 }
524
525
526 //
527 // Frees environment e.
528 // If e was the current env, then runs a new environment (and does not return
529 // to the caller).
530 //
531 void
532 env_destroy(env_t *e)
533 {
534         // TODO: race condition with env statuses, esp when running / destroying
535         e->env_status = ENV_DYING;
536
537         env_decref(e);
538         atomic_dec(&num_envs);
539
540         // for old envs that die on user cores.  since env run never returns, cores
541         // never get back to their old hlt/relaxed/spin state, so we need to force
542         // them back to an idle function.
543         uint32_t id = lapic_get_id();
544         // There is no longer a curenv for this core. (TODO: Think about this.)
545         curenvs[id] = NULL;
546         if (id) {
547                 smp_idle();
548                 panic("should never see me");
549         }
550         // else we're core 0 and can do the usual
551
552         /* Instead of picking a new environment to run, or defaulting to the monitor
553          * like before, for now we'll hop into the manager() function, which
554          * dispatches jobs.  Note that for now we start the manager from the top,
555          * and not from where we left off the last time we called manager.  That
556          * would require us to save some context (and a stack to work on) here.
557          */
558         //manager();
559         //assert(0); // never get here
560
561         schedule();
562 }
563
564 /* ugly, but for now just linearly search through all possible
565  * environments for a runnable one.
566  * the current *policy* is to round-robin the search
567  */
568 void schedule(void)
569 {
570         env_t *e;
571         static int last_picked = 0;
572         
573         for (int i = 0, j = last_picked + 1; i < NENV; i++, j = (j + 1) % NENV) {
574                 e = &envs[ENVX(j)];
575                 // TODO: race here, if another core is just about to start this env.
576                 // Fix it by setting the status in something like env_dispatch when
577                 // we have multi-contexted processes
578                 if (e && e->env_status == ENV_RUNNABLE) {
579                         last_picked = j;
580                         env_run(e);
581                 }
582         }
583
584         cprintf("Destroyed the only environment - nothing more to do!\n");
585         while (1)
586                 monitor(NULL);
587 }
588
589 //
590 // Restores the register values in the Trapframe with the 'iret' instruction.
591 // This exits the kernel and starts executing some environment's code.
592 // This function does not return.
593 //
594 void env_pop_tf(trapframe_t *tf)
595 {
596         asm volatile ("movl %0,%%esp;           "
597                       "popal;                   "
598                       "popl %%es;               "
599                       "popl %%ds;               "
600                       "addl $0x8,%%esp;         "
601                       "iret                     "
602                       : : "g" (tf) : "memory");
603         panic("iret failed");  /* mostly to placate the compiler */
604 }
605
606 /* Return path of sysexit.  See sysenter_handler's asm for details. */
607 void env_pop_tf_sysexit(trapframe_t *tf)
608 {
609         asm volatile ("movl %0,%%esp;           "
610                       "popal;                   "
611                       "popl %%es;               "
612                       "popl %%ds;               "
613                       "addl $0x10, %%esp;       "
614                       "popfl;                   "
615                       "movl %%ebp, %%ecx;       "
616                       "movl %%esi, %%edx;       "
617                       "sysexit                  "
618                       : : "g" (tf) : "memory");
619         panic("sysexit failed");  /* mostly to placate the compiler */
620 }
621
622 //
623 // Context switch from curenv to env e.
624 // Note: if this is the first call to env_run, curenv is NULL.
625 //  (This function does not return.)
626 //
627 void
628 env_run(env_t *e)
629 {
630         // Step 1: If this is a context switch (a new environment is running),
631         //         then set 'curenv' to the new environment,
632         //         update its 'env_runs' counter, and
633         //         and use lcr3() to switch to its address space.
634         // Step 2: Use env_pop_tf() to restore the environment's
635         //         registers and drop into user mode in the
636         //         environment.
637
638         // Hint: This function loads the new environment's state from
639         //      e->env_tf.  Go back through the code you wrote above
640         //      and make sure you have set the relevant parts of
641         //      e->env_tf to sensible values.
642
643         // TODO: race here with env destroy on the status and refcnt
644         // Could up the refcnt and down it when a process is not running
645         e->env_status = ENV_RUNNING;
646         if (e != curenvs[lapic_get_id()]) {
647                 curenvs[lapic_get_id()] = e;
648                 e->env_runs++;
649                 lcr3(e->env_cr3);
650         }
651         /* If the process entered the kernel via sysenter, we need to leave via
652          * sysexit.  sysenter trapframes have 0 for a CS, which is pushed in
653          * sysenter_handler.
654          */
655         if (e->env_tf.tf_cs)
656                 env_pop_tf(&e->env_tf);
657         else
658                 env_pop_tf_sysexit(&e->env_tf);
659 }
660
661 /* This is the top-half of an interrupt handler, where the bottom half is
662  * env_run (which never returns).  Just add it to the delayed work queue,
663  * which isn't really a queue yet.
664  */
665 void run_env_handler(trapframe_t *tf, void* data)
666 {
667         assert(data);
668         per_cpu_info_t *cpuinfo = &per_cpu_info[lapic_get_id()];
669         spin_lock_irqsave(&cpuinfo->lock);
670         { TRUSTEDBLOCK // TODO: how do we make this func_t cast work?
671         cpuinfo->delayed_work.func = (func_t)env_run;
672         cpuinfo->delayed_work.data = data;
673         }
674         spin_unlock_irqsave(&cpuinfo->lock);
675 }