tags:

views:

221

answers:

1

Hello, I'm getting error "undefined reference to `pthread_attr_init'", even though that should be in pthread.h. This is in a UNIX environment that should be set up for Pthreads. Also, is a void* a good way to point to the current matrix? Here's my code, any ideas?

#include <cstdlib>
#include <iostream>
#include <sstream>
#include <string>
#include <vector>
#include <time.h>
#include <limits.h>
#include <pthread.h>
#include <semaphore.h>

#define MAX_MATRIX_SIZE  256
#define MAX_THREADS   64

using namespace std;

void *StripProcessor(void *);
void Barrier();
void InitMatrix(int rows, int cols);

int main(int argc, char *argv[])
{
    /* Get command line params */
    if (argc != 3) {
     cout << "Error: need 2 command line params." << endl;
     getchar();
     return 1;
    }
    int numThreads = atoi(argv[1]);
    double epsilon = atof(argv[2]);


    /* Locals hang here */
    pthread_mutex_t barrier;
    pthread_cond_t leave;
    int numWaiting = 0;

    int rows;
    int cols;
    int stripSize;
    int lastStripSize;
    int iterations;

    double matrix1[MAX_MATRIX_SIZE][MAX_MATRIX_SIZE];
    double matrix2[MAX_MATRIX_SIZE][MAX_MATRIX_SIZE];
    void *currentMatrix = matrix1; //????????????????
    double stripMaxDiff[MAX_THREADS];

    clock_t start;
    clock_t finish;


    /* Get rows and cols from standard in */
    string buffer;
    getline(cin, buffer);
    istringstream myStream(buffer);
    myStream >> rows;
    myStream >> cols;
    cout << rows << " " << cols << endl;
    getchar();


    /* Set locals */
    if (rows % numThreads == 0) {
     stripSize = rows / numThreads;
     lastStripSize = 0;
    }
    else {
     stripSize = rows / numThreads + 1;
     if (rows % stripSize == 0) {
      numThreads = rows / stripSize;
      lastStripSize = 0;
     }
     else {
      numThreads = rows / stripSize + 1;
      lastStripSize = rows - (stripSize * (numThreads - 1));
     }
    }


    /* Set up threads */
    vector<pthread_t> threadID;
    pthread_attr_t attr;
    pthread_attr_init(&attr);
    pthread_attr_setscope(&attr, PTHREAD_SCOPE_SYSTEM);


    /* Init mutex, CV, and matrix */
    pthread_mutex_init(&barrier, NULL);
    pthread_cond_init(&leave, NULL);
    //InitMatrix(rows, cols);


    /* Create threads and wait for completion */
    start = clock();

    /*for (unsigned int i = 0; i < threadID.size; i++)
     pthread_create(&threadID[i], &attr, StripProcessor, (void *) i);
    for (unsigned int i = 0; i < threadID.size; i++)
     pthread_join(threadID[i], NULL);*/

    finish = clock();


    /* Print results */

}
+4  A: 

It will depend on your platform, but I would try adding -lpthread to your link command as that's what is required in Linux and a few others. Your program compiles fine here as g++ foo.cc -lpthread.

asveikau
For pthreads (specifically), isn't the compiler switch -pthread, or has that changed recently?
Suppressingfire
That does work on some compilers/platforms... for example Sun's cc used to work like this. I think gcc supports it also to be compatible with others.
asveikau