Opened 2 years ago
Last modified 3 months ago
#3571 new bug
torrents get lost
Reported by: | Xuefer | Owned by: | |
---|---|---|---|
Priority: | critical | Milestone: | needs verified |
Component: | libtorrent | Version: | 2.1.0 |
Keywords: | Cc: |
Description
steps to reproduce:
- make sure there're more than 128 torrents added, in any state
- use ltconfig plugin to set alert_queue_size to 100 (or load preset: minimal memory usage, which use 100 for alert_queue_size)
- restart deluge
only about 126~130 torrents get loaded. restart deluge again you'll see size of torrents.state get slim down to a few kbytes from mbytes "Finished loading 427 torrents in 0:00:06.760026" is not appearing in the log. unless the count is slim down
alert_queue_size = 100 is not required for this to happen
Reason: loading existing torrents, rely on alert_queue as callback Suggestion: need a reliable callback mechanic other than alert_queue which is limited by size.
Change History (3)
follow-up: 2 comment:1 by , 20 months ago
comment:2 by , 3 months ago
Replying to Calum:
Yes Deluge has a queue size default of 10000 since the queue is critical to functionality so not unexpected that a very low value would cause issues
Alerts is the main mechanism for libtorrent to report errors and events.
Deluge requires the queue for events but there are potential improvements to alert handling.
Can someone take a look at this? Critical events shouldn't be drop for any reason, otherwise bad things happens. "a very low value" is not required to reproduce the bug, a slow PC can reproduce the bug on a busy deluge too
comment:3 by , 3 months ago
an idea would be to set large enough (larger than normal) alert queue size and mask other alerts allowing only needed for loading/saving torrent list when and only when loading/saving torrent list. can anyone implement this?
alert queue size is set to 100 in ltconfig "minimum memory usage" prest, which will sure trigger this bug
Yes Deluge has a queue size default of 10000 since the queue is critical to functionality so not unexpected that a very low value would cause issues
Deluge requires the queue for events but there are potential improvements to alert handling.