CS 3733 Operating Systems, Spring 2010 Assignment 5 Comments

Score on this assignment:
parallel code: 16
serial code: 15

• For Part 4, the mutex should only appear in the function searchForTwoStringsThread.
  It should be declared and initialized there.
• The following is not valid:

       int main(argc, char *argv[]) {
           int x[argc -3];
           if (argc < 4)
               return 0;
           ...
  

• The following produces serial code:

        for ( ... ) {
           pthread_create(...);
           pthread_join(...);
        }
  

  If should be:

        for ( ... )
           pthread_create(...);
        for ( ... )
           pthread_join(...);
  

• The following produces serial code (in searchForTwoStringsThread);

       lock mutex
       searchForTwoStrings
       print results
       unlock mutex
  

  If should be:

       searchForTwoStrings
       lock mutex
       print results
       unlock mutex
  

• A number of people did the following:

       for ( ... ) {
          ptr = malloc( ... );
          ...
       }
       ...
       free(ptr);
       exit(0);
  

  Freeing is not necessary if the process is terminating.
  This only frees one of the many buffers that were malloc'd.