170 похожих чатов

Вопрос по lockfree-очереди. Реализовал её по алгоритму Майкла-Скотта + с

оглядкой на бустовскую и примеры в интернете.
Как и полагается, спустя месяц пришли к выводу, что время от времени падает именно она.

Как дебажить?
Санитайзеров на платформе нет, имеющиеся средства анализа не вылавливают ошибку, однако удалось создать +- повторяемые условия её проявления.

22 ответов

20 просмотров

А вы ошибку уверенно локализовали? Тестик сделали там на hardware_max потоков, которые пытаются её перегружать?

Dmitriy-[Отпуск] Автор вопроса
Constantine Drozdov
А вы ошибку уверенно локализовали? Тестик сделали ...

Можно считать, что да. При > 32 потоков падает в 1 из 100 циклов теста практически гарантированно

Вообще, я бы при хорошей плотности воспроизведения делал примерно так: 0. Поднять все барьеры до seq_cst. Ошибка сохраняется? Да. Добавить целенаправленную ошибку в каждую редкую ветку и сопоставить статистику ошибок. Нет. Понятно, в чём проблема.

Dmitriy-[Отпуск] Автор вопроса
Constantine Drozdov
А исходный код не можете показать?

Да. Но круг поиска может быть шире, чем кажется :) https://github.com/DymOK93/KTL/blob/lockfree_queue_fix/modules/lockfree/queue.hpp

Dmitriy-[Отпуск] Автор вопроса
Constantine Drozdov
Вообще, я бы при хорошей плотности воспроизведения...

Проверил с seq_cst (в моей реализации для x86-64 - на основе MSVC-интринсиков _InterlockedExchange() для store(), _InterlockedCompareExchange для load()) Ошибка сохранилась. Спасибо - буду проверять отдельные ветви исполнения...

Dmitriy [Отпуск]
Проверил с seq_cst (в моей реализации для x86-64 -...

https://github.com/DymOK93/KTL/blob/lockfree_queue_fix/modules/lockfree/queue.hpp#L122 начните с вот этой, тут, кажется, что-то очень странное написано

Dmitriy-[Отпуск] Автор вопроса
Constantine Drozdov
https://github.com/DymOK93/KTL/blob/lockfree_queue...

Хм, вроде не странное... Но перепроверю

Dmitriy [Отпуск]
Хм, вроде не странное... Но перепроверю

ладно, вот это-то точно повод для паранойи https://github.com/DymOK93/KTL/blob/lockfree_queue_fix/modules/lockfree/queue.hpp#L154 мы не захватили владение узлом, почему во время этого присваивания rhs не умер?

Dmitriy-[Отпуск] Автор вопроса
Constantine Drozdov
ладно, вот это-то точно повод для паранойи https:/...

1) head == current_head => "голова" на месте 2) !(head==tail) => очередь не пуста 3) value = next_ptr->value => считали значение 4) cas_weak_helper(m_head.get_ptr(), head, new_head) => если true, то голова все еще наша, считанное значение валидно Если предположить, что между 2) и 4) другой поток прошел тот же путь и "снял голову", то вопрос лишь в том, кто раньше придет в 4) Так это вижу я... И не видел иного рассмотрения в источниках

Dmitriy [Отпуск]
1) head == current_head => "голова" на месте 2) !(...

ну вопрос что поток (А) выполняет это присваивание, а два соседа просто съели элемент и вызвали его деаллокацию

Класс!

Один вопрос: зачем вам lock free queue?

Dmitriy-[Отпуск] Автор вопроса
Ilya Zviagin
Один вопрос: зачем вам lock free queue?

Честно? Больше развлечения ради)) Бутылочным горлышком в том коде очередь не была

Dmitriy [Отпуск]
Честно? Больше развлечения ради)) Бутылочным горлы...

Спасибо за честный ответ! Но зато у вас теперь жизнь такая интересная!

Dmitriy-[Отпуск] Автор вопроса
Ilya Zviagin
Спасибо за честный ответ! Но зато у вас теперь жиз...

Забавнее то, что сама очередь оказалась вообще не при делах - косяк по невнимательности в другом месте

Dmitriy-[Отпуск] Автор вопроса
Андрей Руссков
а почему не std::atomic?

Потому что его у меня нет :)

И чё, реально улучшило перформанс относительно очереди с синхронизацией?

Dmitriy-[Отпуск] Автор вопроса
Roman
И чё, реально улучшило перформанс относительно оче...

В диалоге ниже есть ответы на все вопросы)

написать на TLA+ её сначала. (сорри что поздно)

Dmitriy-[Отпуск] Автор вопроса
Konstantin Osipov
написать на TLA+ её сначала. (сорри что поздно)

Во-первых, поздно, во-вторых, дело вообще не в ней было)

Похожие вопросы

Обсуждают сегодня

Всем привет! Имеется функция: function IsValidChar(ch: UTF8Char): Boolean; var i: Integer; ValidChars: AnsiString; begin ValidChars := 'abcdefghijklmnopqrstuvwxyzABCDE...
Евгений
44
Коллеги, я тут для личных нужд пошел ставить MQTT сервер, пощупал mosquitto, но ужаснулся отсутствию такой банальности, как HTTP API для посмотреть список топиков. А тут что,...
Maksim Lapshin
9
#include <stdio.h> #include <stdlib.h> #include <time.h> void mass_first_generate(int mass[5][7]) {     for (int N = 0; N < 5; N++) {         for (int A = 0; A < 7; A++) {   ...
Чувак
6
Всем привет! Решаю 99 OCaml Problems и столкнулся со следующей проблемой (прошу палками не забивать, я OCaml практически не трогал до этого момента): open OUnit2 let create_...
К|/|pи/\/\ 6е3yглbIи
2
https://www.linkedin.com/posts/ugama-benedicta-kelechi-codergirl-103041300_mobiledevelopment-fluttertraining-handsonlearning-activity-7263445699227254784-IdHB?utm_source=share...
CoderGirl
16
Ну вот просто даже давайте вот как. Какой нибудь конкретный кейс, можете в пример привести, где бч работает и приносит прикладную пользу, а не просто что бы было? Не крипту.
Alexander Andreev
22
Точно, оно. У тебя там имена потоков выставляются?
Александр (Rouse_) Багель
11
возможно ли как-то передать в электрон или таури медиа поток с рендера 2д движка? двиг запускается как dll, а дальше надо как-то отправлять рендер кодировать не подходит, зр...
Kyle Nekto
7
Помогите пожалуйста. Делаю систему плагинов. Проблема сейчас в такая: плагины загружаются в основном потоке. FLibHandle := SafeLoadLibrary(FFileName) Но нужно еще выполнить фу...
Илья 🤣
10
объясните пожалуйста, почему функция не работает должным образом? вроде должно брать активное окно сравнивать его размер с размером экрана, и если есть совпадение = true прове...
JF
12
Карта сайта