Skip to content

tunnel_service: Fix busy-wait on win32 #1418

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jun 24, 2025
Merged

Conversation

doronz88
Copy link
Owner

For community

⬇️ Please click the 👍 reaction instead of leaving a +1 or 👍 comment

@doronz88
Copy link
Owner Author

Depends on doronz88/pytun-pmd3#17

@doronz88 doronz88 marked this pull request as draft June 22, 2025 18:58
@doronz88 doronz88 force-pushed the bugfix/wintun-high-cpu branch from 8fb75bf to c9848da Compare June 22, 2025 19:00
@AnNEDoMini
Copy link
Contributor

AnNEDoMini commented Jun 23, 2025

INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     Uvicorn running on http://0.0.0.0:54269 (Press CTRL+C to quit)
INFO:     127.0.0.1:58572 - "GET /favicon.ico HTTP/1.1" 404 Not Found
INFO:     127.0.0.1:58573 - "GET / HTTP/1.1" 200 OK
INFO:     127.0.0.1:58572 - "GET /favicon.ico HTTP/1.1" 404 Not Found
Traceback (most recent call last):
  File "C:\Users\a.zhuravlov\AppData\Roaming\Python\Python313\site-packages\pymobiledevice3\utils.py", line 47, in wrapper
    return await f(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\a.zhuravlov\AppData\Roaming\Python\Python313\site-packages\pymobiledevice3\remote\tunnel_service.py", line 171, in tun_read_task
    packet = await self.tun.async_read()
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\a.zhuravlov\AppData\Roaming\Python\Python313\site-packages\pytun_pmd3\wintun.py", line 354, in async_read
    raise_last_error()
    ~~~~~~~~~~~~~~~~^^
  File "C:\Users\a.zhuravlov\AppData\Roaming\Python\Python313\site-packages\pytun_pmd3\wintun.py", line 189, in raise_last_error
    raise Exception(get_error_message(get_last_error()))
Exception: The operation completed successfully.
INFO:     127.0.0.1:58581 - "GET /start-tunnel?udid=00008130-0019186434D1001C HTTP/1.1" 200 OK

@doronz88 doronz88 force-pushed the bugfix/wintun-high-cpu branch from c9848da to e0e5b20 Compare June 23, 2025 21:18
@doronz88 doronz88 marked this pull request as ready for review June 24, 2025 20:22
@doronz88 doronz88 force-pushed the bugfix/wintun-high-cpu branch from 305fd9e to 83e148d Compare June 24, 2025 20:50
@doronz88 doronz88 force-pushed the bugfix/wintun-high-cpu branch from 83e148d to ab92316 Compare June 24, 2025 20:52
@doronz88 doronz88 merged commit 02f68d1 into master Jun 24, 2025
17 checks passed
@doronz88 doronz88 deleted the bugfix/wintun-high-cpu branch June 24, 2025 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants