envoyproxy/envoy

[Events] Add an file event wakeup method with socketpair

Open

#14.244 aperta il 2 dic 2020

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
help wantedtech debt

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

Description:

The goal is to create a premitive in source/common/event that wakes up (schedules) an event using a socketpair instead of scheduleCallbackCurrentIteration. The reason for that is that scheduleCallbackCurrentIteration is using event_active(&raw_event_, EV_TIMEOUT, 0); which takes a lock.

See comment in: https://github.com/envoyproxy/envoy/pull/13954#discussion_r533796896

This mechanism could be applied to:

  1. Win32 DirectoryWatcher
  2. Replace the wakeup mechanism in dispatcher_impl.cc -> void DispatcherImpl::post(std::function<void()> callback)

For Win32 it is good to also have #10871 if we are going to build on top of socketpairs

Guida contributor

Direzione di ricerca
Esamina l'implementazione esistente di DispatcherImpl::post e il commento citato nella pull request #13954. Comprendi come funziona il wakeup con socketpair e considera di applicarlo a Win32 DirectoryWatcher e al meccanismo post.
Tech stack
cpp
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++event loop concepts
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30