Imperial-MEng/pintos_22
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Temporary fixes for process waiting and stack setup to allow simple user programs to run #24

Merged
sb3923 merged 2 commits from user-programs-temporary-fix into user-programs 2024-11-06 16:41:49 +00:00
Conversation 5 Commits 2 Files Changed 1
+8 -2
sb3923 commented 2024-11-06 15:52:44 +00:00 (Migrated from gitlab.doc.ic.ac.uk)
Copy Link
Copy Source

The temporary fixes are:

  1. process_wait now waits for 1 second just to give enough time for simple user programs to run.
  2. stack setup grows the stack pointer a bit to prevent page faults in programs that do not check their argument list.

Compiles.

The temporary fixes are: 1. process_wait now waits for 1 second just to give enough time for simple user programs to run. 2. stack setup grows the stack pointer a bit to prevent page faults in programs that do not check their argument list. Compiles.
sb3923 commented 2024-11-06 15:52:44 +00:00 (Migrated from gitlab.doc.ic.ac.uk)
Copy Link
Copy Source

requested review from @td1223

requested review from @td1223
sb3923 commented 2024-11-06 15:52:44 +00:00 (Migrated from gitlab.doc.ic.ac.uk)
Copy Link
Copy Source

assigned to @sb3923

assigned to @sb3923
ed1223 commented 2024-11-06 16:41:31 +00:00 (Migrated from gitlab.doc.ic.ac.uk)
Copy Link
Copy Source

approved this merge request

approved this merge request
ed1223 commented 2024-11-06 16:41:49 +00:00 (Migrated from gitlab.doc.ic.ac.uk)
Copy Link
Copy Source

mentioned in commit 2fefdef605

mentioned in commit 2fefdef605b43684038408b7e80c17c60b006b7b
ed1223 (Migrated from gitlab.doc.ic.ac.uk) merged commit 2fefdef605 into user-programs 2024-11-06 16:41:49 +00:00
ed1223 (Migrated from gitlab.doc.ic.ac.uk) approved these changes 2025-10-16 16:43:17 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
ed1223
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
cyclane (Gleb Koval)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Imperial-MEng/pintos_22#24