我正在使用OpenMP,但我遇到了错误结果的问题。
以下是代码:
#pragma omp parallel shared(L,nthreads,chunk) private(tid,i,j){
tid = omp_get_thread_num();
if (tid == 0)
{
nthreads = omp_get_num_threads();
printf("Starting matrix multiple example with %d threads\n",nthreads);
printf("Initializing matrices...\n");
}
#pragma omp for schedule (static, chunk)
for( i=0; i<SIZE_A;i++){
for( j=0; j<SIZE_B;j++){
if(A[i]==B[j]){
if(i==0 || j==0)
L[i][j]=1;
else
L[i][j] = L[i-1][j-1] + 1;
}
// or reset the matching score to 0
else
L[i][j]=0;
}
}
}
你怎么想,为什么我得到了结果? 我应该改变什么?
非常感谢!
答案 0 :(得分:7)
您有循环数据依赖:
L[i][j] = L[i-1][j-1] + 1;
此处如果已将i
和i-1
交互分配给不同的线程,则无法保证第一个线程在第二个线程开始之前就已完成,因此第二个线程读取不正确(仍未更新)L[i-1][j-1]
的值。您可以通过将ordered
子句赋予omp for
工作共享指令来执行命令,但这会导致并行化。
由于依赖关系是对角线的,你可以重新考虑你的算法以某种方式对角地走L
而不是按行走。