-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Failing Usertests - Big Files #11
Comments
Is it common for students to have lots of non-default user programs and significantly inflated kernel binaries for any given assignment? Just trying to sus-out the user story here. On the undergrad side students are working on a distinct fork of the template for each assignment, so we never run into the problem where such an accounting would be valuable. |
Or is this bug simply saying that the user test crashes instead of fails? Ya spo'se we should fix that |
I'm trying to get one of our TAs to work on it |
I have a lot of students that make multiple intermediate copies of user binaries with different behaviors/tests. We typically model assignments after an existing user binary, so copy-paste is a common first step. We don't recommend that, but what can you do The current assignment they're working on required 3 programs for us, too, so when they had even one other up it would fail that test |
xv6/user/usertests.c
Line 188 in 87f3902
There's a bug in this printf statement that causes a balloc crash if the test was going to fail (could not write the full 140 blocks)
Additionally, this test in general needs to change to account for current User space disk space usage vs. the current setting of the Filesystem size in param.h (if students have more than a few binaries in their root, and/or the sum total of their user + kernel binaries is greater than FSSIZE - 140 * BSIZE, the "large file" test will fail inherently)
The text was updated successfully, but these errors were encountered: