In all the variants of the basic test for part 1-1, release the lock

when we're done with it.  This shouldn't be necessary, but not
releasing the lock is a corner case that we don't necessarily want the
students to have to handle.

diff --git a/grading/threads/alarm-multiple.c b/grading/threads/alarm-multiple.c
index aaa3cc84225fc47ab18a1727ded478e0c598260c..6e7c3d4079adc638ef593a35f437a7bf2ea1206c 100644 (file)
--- a/grading/threads/alarm-multiple.c
+++ b/grading/threads/alarm-multiple.c
@@ -130,6 +130,7 @@ test_sleep (int thread_cnt, int iterations)
   
   printf ("Test complete.\n");
 
+  lock_release (&test.output_lock);
   free (output);
   free (threads);
 }

diff --git a/grading/threads/alarm-single.c b/grading/threads/alarm-single.c
index 4681d8983d6bc10755dbd56bc142b63b2f501ba6..aff206467e771242c1becf9ebe67f376946c5faf 100644 (file)
--- a/grading/threads/alarm-single.c
+++ b/grading/threads/alarm-single.c
@@ -130,6 +130,7 @@ test_sleep (int thread_cnt, int iterations)
   
   printf ("Test complete.\n");
 
+  lock_release (&test.output_lock);
   free (output);
   free (threads);
 }

diff --git a/src/tests/threads/p1-1.c b/src/tests/threads/p1-1.c
index 2aeb91001aaa5351d396874a6ac7051d3e0da9c0..627bba5fc2e5109dff8a44b6e1fa99d92e5126fb 100644 (file)
--- a/src/tests/threads/p1-1.c
+++ b/src/tests/threads/p1-1.c
@@ -134,6 +134,7 @@ test_sleep (int thread_cnt, int iterations)
   
   printf ("Test complete.\n");
 
+  lock_release (&test.output_lock);
   free (output);
   free (threads);
 }

diff --git a/src/threads/test.c b/src/threads/test.c
index 2aeb91001aaa5351d396874a6ac7051d3e0da9c0..627bba5fc2e5109dff8a44b6e1fa99d92e5126fb 100644 (file)
--- a/src/threads/test.c
+++ b/src/threads/test.c
@@ -134,6 +134,7 @@ test_sleep (int thread_cnt, int iterations)
   
   printf ("Test complete.\n");
 
+  lock_release (&test.output_lock);
   free (output);
   free (threads);
 }