X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=TODO;h=7d6ba406ed2365122dc213cb8b30da257f582db2;hb=31711e65718e8334dbb672eca64b6b4b906d1a03;hp=d4afd5c0ad8b2782c3403364270b50bdc385c903;hpb=12399ec5a71cb2fde72835f6abde1cef8e6057e6;p=pintos-anon diff --git a/TODO b/TODO index d4afd5c..7d6ba40 100644 --- a/TODO +++ b/TODO @@ -1,37 +1,38 @@ -*- text -*- -* The tests in tests/ don't apply the grading patches. +* Get rid of rox--causes more trouble than it's worth -* We need better and more example programs. +* Reconsider command line arg style--confuses everyone. - - Need an mmap example program as a replacement for the crappy mmap FAQ - question. +* pintos script doesn't (always?) delete temp disks - - How about `diff' and `cmp' programs? +* Finish writing tour. -* Make it clear that the students own their code, because there was some - confusion on that point. +via Godmar Back: -* Threads: +* Project 3 solution needs FS lock. - - join-invalid doesn't compile if tid_t is not scalar type. +* Get rid of mmap syscall, add sbrk. + +* hash_destroy() should take function arg. + +* Make backtrace program accept multiple object file arguments, + e.g. add -u option to allow backtracing user program also. + +* VM projects do not use enough memory to force eviction. + Should reduce user limit or increase memory consumption. - - mlfqs tests suck. They aren't even correct, e.g. the amarv - submission from win0405 is graded incorrectly. -* Userprog project: - - Don't emphasize that stuff needs to be copied from user space to - kernel space. Instead, emphasize validation and suggest that - copying is a common solution and that it will be necessary in - project 3 and in real OSes. Also revise the grading criteria to - match. - - Move `join' implementation here, from `threads' project, to help - normalize the project difficulties. - - The semantics of the join system call should change so that it - only returns the exit code once. +* process_death test needs improvement + +* Internal tests. + +* Improve automatic interpretation of exception messages. + +* Userprog project: - Mark read-only pages as actually read-only in the page table. Or, since this was consistently rated as the easiest project by the @@ -43,43 +44,15 @@ Alternately we could just remove the synchronization on pid selection and check that students fix it. -* VM project: - - - Discuss the perils of mixing dirty bits between kernel and user virtual - memory. - - - Sample solution. - - - Update grading/vm to reflect new mmap, munmap forms. - * Filesys project: - - Increase maximum disk size from 8 MB to something that actually - requires doubly indirect nodes. There is a negative pressure here - from the bitmap object--perhaps we need a specialized bitmap that - doesn't have to be all in-memory at once. - - Alternatively, shrink the inode size. - - - Add mkdir and ls example user programs. - - - Add option to disable buffer cache. - - - Get rid of "dump" commands--they're not really useful. - - - Sample solution. - - Need a better way to measure performance improvement of buffer cache. Some students reported that their system was slower with cache--likely, Bochs doesn't simulate a disk with a realistic speed. - - Clarify effect of remove(cwd). - * Documentation: - - Finish writing tour. - - Add "Digging Deeper" sections that describe the nitty-gritty x86 details for the benefit of those interested. @@ -99,27 +72,3 @@ . opendir/readdir/closedir . everything needed for getcwd() - - - Add src/testcases/vm, src/testcases/filesys and make it clear to use - them? - -* Tests: - - - Release some of them. - - - The threads, userprog, vm test source files could use - factorization and cleanup along the lines of fslib in the filesys - tests. - - - The p1-4.c testcase needs significant tuning. Currently it takes - too long (especially when SHOW_PROGRESS is turned on) and doesn't - show significant improvement. - -* Code: - - - Rewrite quick_sort() to use heap sort, for O(1) stack usage. - - - Rewrite list_sort() to use merge sort, for O(1) heap usage. - - - Make list_elem and hash_elem a struct, by encapsulating a struct - list_elem inside a struct hash_elem as a member.