#2379 closed patch (Fixed)
Move cleanup of prev_status from torrent.py to torrentmanager.py
| Reported by: | Bro | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | 2.0.0 |
| Component: | Core | Version: | develop |
| Keywords: | Cc: |
Description
Each torrent instance has a LoopingCall that cleans up the prev_status dictionary regularly. With a lot of torrents the number of LoopingCalls starts to generate a lot of CPU usage.
This patch replaces the LoopingCalls for each torrent with one LoopingCall in torrentmanager.py. I tested on my desktop copmuter with the daemon running idle (not downloading/seeding or with clients connected).
With 5000 torrents the CPU usage was reduced from 26-40% to 3-5%. With 10000 it was reduced from 50-60% to 5-6%.
Change History (7)
comment:1 by , 11 years ago
comment:4 by , 11 years ago
| Resolution: | → Fixed |
|---|---|
| Status: | new → closed |
Committed to develop: [feaeee0379]
comment:5 by , 11 years ago
| Milestone: | performance → 1.4.0 |
|---|
comment:6 by , 9 years ago
| Milestone: | 2.0.x → 2.0 |
|---|
Note:
See TracTickets
for help on using tickets.
Patch: https://github.com/bendikro/deluge/commit/98386c43dca6021bb832295f4fba14bbab3f881b