envoyproxy/envoy

[Events] Add an file event wakeup method with socketpair

Open

#14.244 geöffnet am 2. Dez. 2020

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
help wantedtech debt

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuchen Sie die vorhandene Implementierung von DispatcherImpl::post und den zitierten Kommentar in Pull Request #13954. Verstehen Sie, wie das Aufwecken mit Socketpair funktioniert, und erwägen Sie die Anwendung auf den Win32 DirectoryWatcher und den Post Mechanismus.
Tech Stack
cpp
Domain
backend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++event loop concepts
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30