Отладка рекурсивного вызова потока в C - PullRequest
1 голос
/ 17 ноября 2009

Я пытался отлаживать свой код всякий раз, когда у меня было свободное время последние полтора дня, и я не знаю, что не так с моим кодом. Когда я добавляю функцию close() к рекурсивному вызову, программа выдает мне неверный указатель. Но когда я удаляю вызов функции close(), программа работает нормально, за исключением того, что она не делает то, что должна делать, а именно:

  • добавить все размеры файлов в пользователя входной каталог
  • открыть подкаталоги, если они есть, и добавить все файлы внутри подкаталог

Вместо этого он суммирует все размеры файлов во входном каталоге и может открыть последний подкаталог и добавить файлы в этом каталоге к общему количеству файлов.

Я пытаюсь сделать это с темами. Функция main() создает один основной поток из пользовательского каталога ввода и запускает opendirectory() с нуля.

/*
 * Iterates through given directory
 */
void *opendirectory(void *t)
{
 pthread_mutex_lock(&dirlock);
 DIR *dpntr;
 struct dirent *dentry;
 char new_directory[512], dir = t;

 printf("OPENING DIRECTORY ... %s\n", t);

 /* Checks if given directory can be opened */
 if((dpntr = opendir(t)) == NULL) {
  printf("DIRECTORY FAILED ...%s\n",t);
  perror("ERROR -- COULD NOT OPEN DIR");
  pthread_exit(NULL);
 }

 printf("DIRECTORY OPENED: %s\n", t);

 /* Read each file in current directory */
 while ((dentry = readdir(dpntr)) != NULL ) {
  /* Ignore special directories */
  if(strcmp(dentry -> d_name, ".") == 0 || strcmp(dentry -> d_name, "..") == 0) {
   continue;
  } else {
   compilelist( t, dentry->d_name );
  }
 }

 pthread_mutex_unlock(&dirlock);
 /* Checks if directory can be closed */
 if(closedir(dpntr) < 0)
  printf("ERROR CLOSING %s.\n", t);

}

Это функция, которая определяет, должен ли быть создан новый поток и должен ли он выполняться рекурсивно.

/*
 * Determines if current file is a directory
 * Creates a new thread if true
 */
void compilelist (const char* dirname, const char *filename)
{
    pthread_mutex_lock(&filelock);
    struct stat statdata;
    char *filepathname, *dpntr;

    /* Allocate memory for filepathname */
    if((filepathname = (char *) malloc(sizeof(char) * strlen(dirname))) == NULL)
    {
        printf("CANNOT ALLOCATE MEMORY FOR FILE PATH NAME.");
        pthread_exit(NULL);
    }

    /* Concats directory name with file name */
    if(dirname[strlen(dirname) -1] == '/')
    {
        pthread_mutex_lock(&pathlock);
        sprintf(filepathname, "%s%s", dirname, filename);
        pthread_mutex_unlock(&pathlock);
    }else
    {
        pthread_mutex_lock(&pathlock);
        sprintf(filepathname, "%s/%s", dirname, filename);
        pthread_mutex_unlock(&pathlock);
    }

    lstat(filepathname, &statdata);

    /* Calls print_statdata() if current item is a file */
    if(!(S_ISDIR(statdata.st_mode)))
    {
        printf("FILE: %s\n", filepathname);
        if(!stat( filepathname, &statdata))
        {
            print_statdata( filename, &statdata );
        }
        else {
            fprintf (stderr, "GETTING STAT FOR %s", filepathname);
            perror( "ERROR IN STATDATA WHILE GETTING STAT");
        }
    }
    /* Recursive call to opendirectory() */
    else {
        pthread_mutex_lock(&dircountlock);
        dirCount++;
        pthread_mutex_unlock(&dircountlock);
        dpntr = filepathname;
        free(filepathname);
        printf("SUB-DIRECTORY THREAD: %s\nTHREAD ID NUMBER: %d\n", dpntr, dirCount);
        pthread_attr_setdetachstate(&attr, PTHREAD_CREATE_JOINABLE);
        pthread_create(&threads[dirCount-1], &attr, opendirectory, (void *)dpntr);
    }

    pthread_mutex_unlock(&filelock);

}

Вот это main()

/*
 * Main function prompts user for a directory
 */
int main(int argc, char *argv[])
{
    int i;
    char *dPtr;
    // pthread_attr_t attr;

    printf("ENTER A DIRECTORY:\n\t");
    scanf("%s", directory);
    dPtr = directory;

    /* Initialize mutex and condition variable objects */
    pthread_mutex_init(&mutex, NULL);
    pthread_mutex_init(&filelock, NULL);
    pthread_mutex_init(&dirlock, NULL);
    pthread_mutex_init(&dircountlock, NULL);
    pthread_cond_init (&count_threshold_cv, NULL);

    /* For portability, explicitly create threads in a joinable state */
    pthread_attr_init(&attr);
    pthread_attr_setdetachstate(&attr, PTHREAD_CREATE_JOINABLE);
    pthread_create(&threads[0], &attr, opendirectory, (void *)dPtr);

    /* Wait for all threads to complete */
    for (i = 0; i < dirCount; i++) {
        pthread_join(threads[i], NULL);
    }

    printf("TOTAL DIRECTORY SIZE: %d\n", dirSize);

    /* Clean up and exit */
    pthread_attr_destroy(&attr);
    pthread_mutex_destroy(&mutex);
    pthread_mutex_destroy(&filelock);
    pthread_mutex_destroy(&dirlock);
    pthread_mutex_destroy(&dircountlock);
    pthread_cond_destroy(&count_threshold_cv);
    pthread_exit (NULL);

}

И глобальные переменные ...

pthread_mutex_t mutex;
pthread_mutex_t dirlock;
pthread_mutex_t filelock;
pthread_mutex_t dircountlock;
pthread_mutex_t threadlock;
pthread_cond_t count_threshold_cv;
pthread_attr_t attr;
pthread_t threads[128]; // handles up to 128 threads (i.e. 128 directories, change accordingly)
char directory[512];
int dirSize = 0;
int dirCount = 1; // user's input directory

Мне кажется, что pthread_create(), вызываемый в нижней части функции compilelist(), не работает должным образом. Thread [] ссылается на глобальный массив потоков, размер которого по умолчанию равен 20, при условии, что всего будет не более 20 каталогов. Значение dirCount начинается с 1 из-за входного каталога пользователя и увеличивается при обнаружении новых каталогов.

Ответы [ 3 ]

2 голосов
/ 17 ноября 2009

Ваш код:

dpntr = opendir(t)

...

if(closedir(t) < 0)

должно быть:

if(closedir(dpntr) < 0)
1 голос
/ 17 ноября 2009

Здесь я нашел 2 проблемы вашего кода:

  1. Как и было сказано, закрытый (t) выводит сегфо.

  2. "char filepathname [512];" of compilelist () - это буфер локальной памяти, но вы передаете его в свой поток (opendirectory) и используете его постоянно Вместо этого вы должны использовать копирование или динамическое выделение.

Effo Upd @ 2009nov17: После исправления выше 2 пунктов, на моем FC9 x86_64 он пока работает нормально. Кстати: темы номер 20 действительно недостаточно.

0 голосов
/ 17 ноября 2009

Первая проблема:

всякий раз, когда у меня было свободное время последние полтора дня

Не делай этого, твой мозг не создан для этого. Выделите время, скажите своим коллегам / жене и детям, что, если они будут беспокоить вас в это время, будут выстрелы и участие полиции: -)

Другие проблемы: не знаю (отсюда и вики сообщества).

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...