1

pthread プログラミングは初めてで、C++ と C の混合コードで作業しているときにこのエラーが発生しました。

私が行ったことは、c++ コードによって作成されたスレッドで c コードを呼び出すことです。スレッドで使用される静的ブール ポインターがありis_center、スレッドが終了すると解放されるはずです。

ただし、プログラムが c 関数に処理されるたびに、ブール ポインターの値が変更され、free() が原因でセグメンテーション違反が発生することに気付きました。問題は、c コードを使用した場合にのみ発生します。C コードを削除すると、マルチスレッドの C++ 部分がうまく機能します。

詳細コードは次のとおりです。

static bool *is_center;

// omit other codes in between ...

void streamCluster( PStream* stream)
{
    // some code here ...
    while(1){
        // some code here ...
        is_center = (bool*)calloc(points.num,sizeof(bool));

        // start the parallel thread here.
        // the c code is invoked in this function.
        localSearch(&points,kmin, kmax,&kfinal); // parallel

        free(is_center);
    }

そして、並列を使用する関数は次のとおりです (私の C コードは各スレッドで呼び出されます)。

void localSearch( Points* points, long kmin, long kmax, long* kfinal ) {
    pthread_barrier_t barrier;
    pthread_t* threads = new pthread_t[nproc];
    pkmedian_arg_t* arg = new pkmedian_arg_t[nproc];

    pthread_barrier_init(&barrier,NULL,nproc);

    for( int i = 0; i < nproc; i++ ) {
            arg[i].points = points;
            arg[i].kmin = kmin;
            arg[i].kmax = kmax;
            arg[i].pid = i;
            arg[i].kfinal = kfinal;
            arg[i].barrier = &barrier;

            pthread_create(threads+i,NULL,localSearchSub,(void*)&arg[i]);
    }

    for ( int i = 0; i < nproc; i++) {
        pthread_join(threads[i],NULL);
    }

    delete[] threads;
    delete[] arg;
    pthread_barrier_destroy(&barrier);
}

最後に、私のCコードを呼び出す関数:

void* localSearchSub(void* arg_) {                                                                                                                                                        

    int eventSet = PAPI_NULL;                                                                                                                                                                                                                                                                                                                                     
    begin_papi_thread(&eventSet);                                                                                                                                                         

    pkmedian_arg_t* arg= (pkmedian_arg_t*)arg_;                                                                                                                                             
    pkmedian(arg->points,arg->kmin,arg->kmax,arg->kfinal,arg->pid,arg->barrier);                                                                                                            

    end_papi_thread(&eventSet);                                                                                                                                                                                                                                                                                                                                                 

    return NULL;                                                                                                                                                                            
}   

そしてgdbから、私が得たものis_centerは次のとおりです。

Breakpoint 2, localSearchSub (arg_=0x600000000000bc40) at streamcluster.cpp:1711
1711      end_papi_thread(&eventSet);
(gdb) s

Hardware watchpoint 1: is_center

Old value = (bool *) 0x600000000000bba0
New value = (bool *) 0xa93f3
0x400000000000d8d1 in localSearchSub (arg_=0x600000000000bc40) at streamcluster.cpp:1711
1711      end_papi_thread(&eventSet);

助言がありますか?前もって感謝します!

コードに関するいくつかの新しい情報: C コードについては、PAPI パッケージを使用しています。システムカウンターを初期化して読み取るために、独自の papi ラッパーを作成します。コードは次のとおりです。

void begin_papi_thread(int* eventSet)                                                                                                                                                     
{                                                                                                                                                                                         
    int thread_id = pthread_self();                                                                                                                                                       
    // Events                                                                                                                                                                             
    if (PAPI_create_eventset(eventSet)) {                                                                                                                                                 
        PAPI_perror(return_value, error_string, PAPI_MAX_STR_LEN);                                                                                                                        
        printf("*** ERROR *** Failed to create event set for thread %d: %s\n.", thread_id, error_string);                                                                                 
    }                                                                                                                                                                                     
    if((return_value = PAPI_add_events(*eventSet, event_code, event_num)) != PAPI_OK)                                                                                                     
    {                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                        
        printf("*** ERROR *** Failed to add event for thread %d: %d.\n", thread_id, return_value);                                                                                                                                                                                                                                                                         
    }                                                                                                                                                                                     
    // Start counting                                                                                                                                                                     
    if ((return_value = PAPI_start(*eventSet)) != PAPI_OK) {                                                                                                                                                                                                                                                                                                              
        PAPI_perror(return_value, error_string, PAPI_MAX_STR_LEN);                                                                                                                                                                                                                                                                                                         
        printf("*** ERROR *** PAPI failed to start the event for thread %d: %s.\n", thread_id, error_string);
    }                                                                                                                                                                                                                                                                                                                                                                             
}  
void end_papi_thread(int* eventSet)                                                                                                                                                       
{                                                                                                                                                                                         
    int thread_id = pthread_self();                                                                                                                                                       
    int i;                                                                                                                                                                                

    long long * count_values = (long long*)malloc(sizeof(long long) * event_num);                                                                                                         
    if (PAPI_read(*eventSet, count_values) != PAPI_OK)                                                                                                                                    
        printf("*** ERROR *** Failed to load count values.\n");                                                                                                                           

    if (PAPI_stop(*eventSet, &dummy_values) != PAPI_OK) {
        PAPI_perror(return_value, error_string, PAPI_MAX_STR_LEN);
        printf("*** ERROR *** PAPI failed to stop the event for thread %d: %s.\n", thread_id, error_string);
        return;
    }
    if(PAPI_cleanup_eventset(*eventSet) != PAPI_OK)
        printf("*** ERROR *** Clean up failed for the thread %d.\n", thread_id);                                                                                                          
} 
4

1 に答える 1

2

問題を本当に理解するのに十分なコードを投稿していないと思いますが、is_centerグローバルと宣言したのは疑わしいようです。おそらく複数のスレッドによって、複数の場所で使用していると思います(localSearchSubこれは、ワーカースレッド関数です)。

is_center複数のスレッドによって読み取りまたは書き込みが行われている場合は、おそらくpthreadmutexで保護する必要があります。「スレッドが終了すると解放される」と言いますが、スレッドがあり、それらはすべて一連のboolnprocsで動作しているように見えることに注意する必要があります。is_center[points]もしそうならpoints != nproc、これは悪いことかもしれません[1]。各スレッドはおそらく独自の配列で動作しlocalSearch、結果を集約する必要があります。

関数はxxx_papi_threadGoogleでヒットしないので、私はそれがあなた自身のものであると想像することしかできません...問題がそこにある場合、私たちがあなたを助けることができるとは思われません:)

[1]:たとえpoints == nproc、複数のスレッドから配列の異なる要素に書き込むことは必ずしもOKではありません(コンパイラーとプロセッサーに依存します)。安全を確保し、ミューテックスを使用してください。

また、これはタグ付けされてC++います。calloc動的配列(を使用new)をvectorsに置き換えることはできますか?デバッグが容易になる可能性があり、保守も確実に容易になります。なぜあなたはあなたのコードの読者を嫌い、罰したいのですか?;)

于 2010-06-06T01:48:17.920 に答える