Ru-MaNGOS

Ru-MaNGOS (http://mangos.ytdb.ru/index.php)
-   Патчи (http://mangos.ytdb.ru/forumdisplay.php?f=6)
-   -   [a bit of research] ACE Reactor usage. (http://mangos.ytdb.ru/showthread.php?t=1593)

Astellar 24.06.2010 21:30

[a bit of research] ACE Reactor usage.
 
Дело было вечером, делать было нечего...

Решил я подробнее посмотреть на их использование в ядре. И не зря, сразу нашёл к чему придраться.
Код:

#if defined (ACE_HAS_EVENT_POLL) || defined (ACE_HAS_DEV_POLL)
    ACE_Reactor::instance(new ACE_Reactor(new ACE_Dev_Poll_Reactor(ACE::max_handles(), 1), 1), true);
#else
    ACE_Reactor::instance(new ACE_Reactor(new ACE_TP_Reactor(), true), true);
#endif

Это код из realmd. Вопрос один. Почему здесь используется ACE_TP_Reactor? Он специально разрабатывался для работы в пуле потоков, т.е. способен выполнять функцию run_reactor_event_loop параллельно, без боязни возникновения коллизий. Для поддержания подобного механизма работы, этот реактор накормлен дополнительными блокировками, в результате чего стал более неповоротливым.

И как мы его используем? Мы не создаем пул потоков. Мы продолжаем использовать его как обычный ACE_Select_Reactor, впустую тратя ресурсы на блокировки и прочее.

Использование ACE_TP_Reactor "по ГОСТу" выглядит следующим образом (внимание, не полный код).
Код:

static ACE_THR_FUNC_RETURN event_loop(void *arg)
{
    ACE_Reactor *reactor = static_cast<ACE_Reactor*>(arg);

    // No-op for TP reactor, but useful for others
    reactor->owner(ACE_Thread::self());

    reactor->run_reactor_event_loop();
}

void main(int, char **)
{
    ACE_Reactor::instance
        (new ACE_Reactor(new ACE_TP_Reactor, 1), 1);

    ACE_Thread_Manager::instance()->spawn_n
        (reactor_thread_num, event_loop, ACE_Reactor::instance());

    return ACE_Thread_Manager::instance()->wait();
}

P.S. Вообще я проводил сравнение производительности реакторов под Linux. По итогам этого сравнения лидирует ACE_Select_Reactor с приличным отрывом, а отнюдь не Dev Poll. Могу выложить в эту тему результаты, но там реально много буковок получится.

KiriX 24.06.2010 21:39

Выкладывай тогда уж патч, если уж стал так усиленно копать в этом направлении ;)

Astellar 24.06.2010 21:44

Патч подождет пока что, ибо он до неприличия прост, но может смысл есть какой-то именно так реактор использовать.

zhenya 25.06.2010 08:40

А это не связанно с тем что реалм кроме авторизации делает еще отправку патчей если они есть?

Astellar 25.06.2010 09:14

Нет. PatchHandler наследуется от ACE_Svc_Handler, но судя по всему действует вообще в обход реактора (reactor(NULL); в конструкторе) в отдельном потоке, используя напрямую дескриптор сокета. Вопроса по поводу ACE_TP_Reactor это не снимает :)


Текущее время: 11:52. Часовой пояс GMT +3.

ru-mangos.ru - Русское сообщество MaNGOS