Параллельная qsort с использованием openMP

Question

Параллельная qsort с использованием openMP

0

Я пытаюсь распараллелить функцию qsort() в c++ с помощью openMP, но у меня есть некоторые проблемы. Для всех, для большого количества элементов для сортировки (ex: V = 1000) qsort вообще не работает, я никогда не выхожу из этой функции (бесконечный цикл). Затем я замечаю, что моя параллельная версия qsort намного медленнее, чем серийная. Кто-нибудь может мне помочь? Вот код: https://dpaste.de/EaH0.

здесь среднее время, когда kruscalP является параллельной версией: thanks

Betelgeuse 20 фев. 2014, в 11:00

Источник

1

можешь показать нам какой-нибудь код? мы не можем видеть, что вы делаете неправильно.
Pandrei 20 фев. 2014, в 09:55
0

вот пример кода: dpaste.de/A2p1 этот алгоритм упорядочивает ребра графа относительно их весов
Betelgeuse 20 фев. 2014, в 10:07
0

Мне кажется, что у вас есть состояние гонки, когда вы делаете A[i] = A[i+1] . Вам нужно изменить свой алгоритм или поместить свой своп в критический раздел.
Z boson 20 фев. 2014, в 10:13
0

У вас также есть условие exch0 когда вы пишете в exch0 и exch1 так как эти переменные являются общими, а не частными.
Z boson 20 фев. 2014, в 10:16
1

вы знаете, что с помощью OpenMP вы создаете потоки для параллельного запуска вашего кода. Но у вас нет синхронизации в вашем коде. Из-за этого потоки будут читать неверные данные. Возможно, вам стоит взглянуть на учебник openMP, чтобы понять проблему
Pandrei 20 фев. 2014, в 10:21
1

Пожалуйста, включите код в ваш вопрос. Внешние сайты могут выходить из строя или исчезать, поэтому лучше иметь код здесь
flipchart 20 фев. 2014, в 10:47

Показать ещё 4 комментария

Теги:

c++

parallel-processing

openmp

quicksort

1 ответ

Ещё вопросы

можешь показать нам какой-нибудь код? мы не можем видеть, что вы делаете неправильно.
вот пример кода: dpaste.de/A2p1 этот алгоритм упорядочивает ребра графа относительно их весов
Мне кажется, что у вас есть состояние гонки, когда вы делаете A[i] = A[i+1] . Вам нужно изменить свой алгоритм или поместить свой своп в критический раздел.
У вас также есть условие exch0 когда вы пишете в exch0 и exch1 так как эти переменные являются общими, а не частными.
вы знаете, что с помощью OpenMP вы создаете потоки для параллельного запуска вашего кода. Но у вас нет синхронизации в вашем коде. Из-за этого потоки будут читать неверные данные. Возможно, вам стоит взглянуть на учебник openMP, чтобы понять проблему
Пожалуйста, включите код в ваш вопрос. Внешние сайты могут выходить из строя или исчезать, поэтому лучше иметь код здесь

Z boson · Accepted Answer · 2014-02-20T07-56-00.000Z

В вашем коде есть несколько условий гонки, когда вы пишете A, exch0 и exch1 которые являются общими. Мало того, что это может ухудшить производительность, скорее всего, даст неправильный результат. К сожалению, вы должны использовать критический раздел, чтобы исправить это, что также может повредить производительность, но, по крайней мере, это даст правильный результат.

#pragma omp for    
for (i = 0; i < N-1; i += 2) {
    #pragma critical
    {
        if (A[i].weight > A[i+1].weight) {
            temp = A[i];
            A[i] = A[i+1];
            A[i+1] = temp;
            exch0 = 1;
        }
    }
}

То же самое относится к следующему циклу. Если вы хотите сделать это эффективно, вам придется изменить свой алгоритм. Возможно, рассмотрим сортировку слияния.