Thread (290 messages) flat view 290 messages, 13 authors, 2022-04-12

Re: [dpdk-dev] [EXTERNAL] Re: [PATCH v6 06/10] eal: add thread lifetime management

From: Dmitry Kozlyuk <hidden>
Date: 2021-04-30 17:51:15

2021-04-30 20:22 (UTC+0300), Dmitry Kozlyuk:
2021-04-29 21:31 (UTC+0000), Dmitry Malloy:
quoted
[...]
How about we defer fixing this to another follow-up change?  
I'm strictly against accepting code with known severe bugs.

Alternative:

1. Don't add rte_thread_cancel() in public API.
2. Implement rte_thread_cancel() as internal API with cancellation tokens.
3. Rewrite DPDK code to use rte_thread_cancel().
[...]
Or just don't export this function on Windows for now.
None of the drivers using it is soon to be enabled for Windows,
and portable apps don't need it anyway.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help