Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 418039 - <net-p2p/transmission-2.61: leaks file descriptors and stalls as soon as the open file limit is hit
Summary: <net-p2p/transmission-2.61: leaks file descriptors and stalls as soon as the ...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Peter Volkov (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-28 20:08 UTC by masc
Modified: 2012-07-30 09:26 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description masc 2012-05-28 20:08:05 UTC
it's described in more detail here.
https://trac.transmissionbt.com/ticket/3504#comment:38



Reproducible: Always

Steps to Reproduce:
1. emerge curl with USE="threads -ares"
2. start transmission-daemon
Actual Results:  
file descriptors will leak over time, the open file limit will be a hit sooner or later, so transmission will stall and has to be restarted.

Expected Results:  
file descriptors are not supposed to leak.

the issue does not occur when compiling curl with USE="-threads ares".

maybe it's a good idea to have these curl use flags as a requirement when emerging transmission.
Comment 1 nzqr 2012-07-20 13:31:45 UTC
I had faced with this problem too, but it disappeared somehow not long ago. Only thing that changed apart of ~arch upgrades, was that i added to curl's configure options --without-zlib.
Comment 2 Samuli Suominen (RETIRED) gentoo-dev 2012-07-25 09:11:18 UTC
ebuild of curl-7.26.0 has:

# ares must be disabled for threads
REQUIRED_USE="threads? ( !ares )

and from ChangeLog:

20 Feb 2011; Tomáš Chvátal <scarabeus@gentoo.org> curl-7.21.4.ebuild:
Update REQUIRED_USE to allow disabling of both ares and threads useflag.
Thanks to Brian for hint.

since the bug was filed this May and the bug was fixed already back in February, closing as INVALID ->
Comment 3 masc 2012-07-25 09:24:40 UTC
this useflag constraint does not fix the issue.
as mentioned in my initial description.

Steps to Reproduce:
1. emerge curl with USE="threads -ares"
Comment 4 Samuli Suominen (RETIRED) gentoo-dev 2012-07-25 09:32:42 UTC
(In reply to comment #3)
> this useflag constraint does not fix the issue.
> as mentioned in my initial description.
> 
> Steps to Reproduce:
> 1. emerge curl with USE="threads -ares"

https://trac.transmissionbt.com/ticket/3504#comment:44

"... I'm using threads enabled, and c-ares enabled, which is the one that should give the best performance."

Our curl ebuild doesn't even seem to allow this combination. CCing curl maintainers for this. Seems like the commit made by scarabeus mentioned in Comment #2 can't be correct and, is missing a bug # explaining the change anyways.

And I'm definately not going to block USE="threads" of curl from transmission ebuild.
Comment 5 Samuli Suominen (RETIRED) gentoo-dev 2012-07-25 09:33:50 UTC
Also have you retried with transmission-2.61?
Comment 6 masc 2012-07-25 09:58:45 UTC
> "... I'm using threads enabled, and c-ares enabled, which is the one that
> should give the best performance."

he also mentions that +threads -ares is the only combination which causes a problem (while even +threads +ares would work).

I didn't check against 2.61 yet, but I will (also using curl 7.26).

the point of my suggestion was that 2.52 is marked stable but is very much unstable when used with curl USE="+threads -ares".
which alternative is there to stabilize?

unstable ebuilds should certainly not contain this constraint, but a warning may still be appropriate as long as the issue is not resolved.
Comment 7 masc 2012-07-30 09:23:09 UTC
I have been running transmission 2.61 with curl 7.26.0 for a few days now and it seems okay.
Comment 8 Samuli Suominen (RETIRED) gentoo-dev 2012-07-30 09:26:14 UTC
(In reply to comment #7)
> I have been running transmission 2.61 with curl 7.26.0 for a few days now
> and it seems okay.

phew. thank you for testing.