linux-test-project / kirk

Kirk application aims to merge multiple Linux testing frameworks in one tool.
GNU General Public License v2.0
6 stars 7 forks source link

Remove epoll() usage in LTX communication #11

Closed acerv closed 11 months ago

acerv commented 11 months ago

Using epoll() is a bad idea when asyncio is involved. The reason is that asyncio itself is using epoll() to implement concurrency, so we end up having 2 epoll() instances running in parallel. When we record a new file descriptor into an asyncio application, we supposed to use the asyncio.BaseEventLoop.add_reader() method which is, unfortunately, not working (by experience) with plain files.

For this reason, the current solution is to use threaded read() syscall which permits to execute reading operations, as well as keeping asyncio loop running in the background without killing UI or data acquisition.