mbox series

[net-next,v3,0/2] tools/net/ynl: rework async notification handling

Message ID 20241113090843.72917-1-donald.hunter@gmail.com (mailing list archive)
Headers show
Series tools/net/ynl: rework async notification handling | expand

Message

Donald Hunter Nov. 13, 2024, 9:08 a.m. UTC 
Revert patch 1bf70e6c3a53 which modified check_ntf() and instead add a
new poll_ntf() with async notification semantics. See patch 2 for a
detailed description.

v2 -> v3:
 - Simplify timeout logic - thanks Jakub!

v1 -> v2:
 - Use python selector module (select / epoll)
 - Remove interval parameter from poll_ntf()
 - Handle KeyboardInterrupt in cli.py instead of lib code

Donald Hunter (2):
  Revert "tools/net/ynl: improve async notification handling"
  tools/net/ynl: add async notification handling

 tools/net/ynl/cli.py     | 20 +++++++-------
 tools/net/ynl/lib/ynl.py | 57 ++++++++++++++++++++++++----------------
 2 files changed, 44 insertions(+), 33 deletions(-)

Comments

Jakub Kicinski Nov. 15, 2024, 3:06 a.m. UTC | #1 
On Wed, 13 Nov 2024 09:08:41 +0000 Donald Hunter wrote:
> Revert patch 1bf70e6c3a53 which modified check_ntf() and instead add a
> new poll_ntf() with async notification semantics. See patch 2 for a
> detailed description.

Applied, thanks!