OpenMP: Replace critical section with locks

Question

How can I replace a critical section with omp_locks instead?

My original code with critical section is like this, which works fine:

#pragma omp for
for (int i = 0; i < n; i++){
    // do thread-safe pre-processing

    #pragma omp critical
    {
        // do critical section stuff
    }
}

Now I do the same thing but with locks instead:

omp_lock_t lock;
omp_init_lock(&lock);

#pragma omp for
for (int i = 0; i < n; i++){
    // do thread-safe pre-processing

    omp_set_lock(&lock);

    // do critical section stuff

    omp_unset_lock(&lock);
}
omp_destroy_lock(&lock);

But for some reason, I get the wrong result. Am I doing something wrong here?

Also, when I try to use multiple locks (e.g. for each element of array to write on), it seems to go into deadlock?

@Gyakenji Maybe put the lock initialisation outside of the parallel block? — RmbRT, Mar 10 '19 at 21:58
The whole thing? Including the definition of the lock variable? Yeah, that'd be a problem because that makes them private ones, not shared... — Shawn, Mar 10 '19 at 22:42
Possible duplicate of [How to use lock in OpenMP?](https://stackoverflow.com/questions/2396430/how-to-use-lock-in-openmp) — Dmytro Dadyka, Mar 11 '19 at 04:14

score 2 · Accepted Answer · answered Mar 11 '19 at 06:02

2

Careless mistake: defining lock should be outside parallel block

answered Mar 11 '19 at 06:02

Gyakenji

177
1
6

OpenMP: Replace critical section with locks

1 Answers1