web-dev-qa-db-ja.com

構造体を引数としてpthreadに渡す

わかりました。数値のペアをstructからpthreadのpthread_create関数に渡そうとしています。しかし、私が渡す番号と関数が呼び出されたときに取得する番号は異なり、ランダムです

これがstructです

struct Pairs {
    long i,j;
};

そしてメインの内側

void main()
{
    long thread_cmp_count = (long)n*(n-1)/2;
    long t,index = 0;
    struct Pairs *pair;
    pair = malloc(sizeof(struct Pairs));

    cmp_thread = malloc(thread_cmp_count*sizeof(pthread_t));
    for(thread = 0;(thread < thread_cmp_count); thread++){
        for(t = thread+1; t < n; t++){
            (*pair).i = thread;
            (*pair).j = t;
            pthread_create(&cmp_thread[index++], NULL, Compare, (void*) pair);

        }
    }

    for(thread= 0;(thread<thread_cmp_count); thread++){
        pthread_join(cmp_thread[thread], NULL);
    }

    free(cmp_thread);
}

そして機能比較

void* Compare(void* pair){
    struct Pairs *my_pair = (struct Pairs*)pair;
    printf("\nThread %ld, %ld", (*my_pair).i, (*my_pair).j);
    return NULL;
}

私が得ている数とそれもランダムです。

Thread 0,2
Thread 1,2
Thread 2,3
Thread 2,3
Thread 2,3
Thread 2,3

structを間違って渡していますか？

cmultithreadingstructpthreads
6
Jos

これは、すべてのpthreadに同じポインターを渡しているためです。

pthread_create(..., (void*) pair)を呼び出すと、新しいスレッドへのポインターが渡されますが、次の反復では、そのメモリが上書きされます（新しいスレッドがそれらの値を抽出する前の可能性があります）。

    long thread_cmp_count = (long)n*(n-1)/2;
    long t,index = 0;
    struct Pairs *pair;

    cmp_thread = malloc(thread_cmp_count*sizeof(pthread_t));
    for(thread = 0;(thread < thread_cmp_count); thread++){
        for(t = thread+1; t < n; t++){
            // allocate a separate pair for each thread
            pair = malloc(sizeof(struct Pairs));
            (*pair).i = thread;
            (*pair).j = t;
            pthread_create(&cmp_thread[index++], NULL, Compare, (void*) pair);

        }
    }

    for(thread= 0;(thread<thread_cmp_count); thread++){
        pthread_join(cmp_thread[thread], NULL);
    }

    free(cmp_thread);

。

void* Compare(void* pair){
    struct Pairs *my_pair = (struct Pairs*)pair;
    printf("\nThread %ld, %ld", (*my_pair).i, (*my_pair).j);

    // free that memory after it has been used
    free (pair);
    return NULL;
}
19
Sergey L.

問題が解決しました。問題はオーバーラップにありました。ポインタをタイプarrayペアのstructとして使用すると解決されます

これが正しいコードです

long thread_cmp_count = (long)n*(n-1)/2;
long t,index = 0;
Pair * pair;
pair = malloc(thread_cmp_count*sizeof(Pair));

free(thread_handles);

thread_handles = malloc(thread_cmp_count*sizeof(pthread_t));
for(thread = 0;(thread < n-1); thread++){
    for(t = thread+1; t < n; t++){
        (pair+index)->i = thread;
        (pair+index)->j = t;
        pthread_create(&thread_handles[index], NULL, Compare, (void*) (pair+index));
        index++;
    }
}
for(thread= 0;(thread<thread_cmp_count); thread++){
    pthread_join(thread_handles[thread], NULL);
}

free(thread_handles);

そして関数比較

void* Compare(void* pair){
    long t,i,j;
    Pair *my_pair = (Pair*)pair;
    i = my_pair->i;
    j = my_pair->j;
    printf("\n..................................................................");
        if((x_array[i] < x_array[j])&&(x_array[i] != x_array[j])){
            w_array[i] = 0;
            printf(
                "\nThread T(%ld,%ld)"
                " compares x[%ld] = %ld and x[%ld] = %ld,"
                " and writes 0 to w[%ld]", i, j,
                i,x_array[i],
                j,x_array[j],
                i);
        }
        else if((x_array[i] > x_array[j])&&(x_array[i] != x_array[j])){
            w_array[j] = 0;
            printf(
                "\nThread T(%ld,%ld)"
                " compares x[%ld] = %ld and x[%ld] = %ld,"
                " and writes 0 to w[%ld]", i, j,
                i,x_array[i],
                j,x_array[j],
                j);
        }
        else
            return NULL;
    return NULL;
}
4
Jos