(
Jump to the original submission)
Thu Feb 24 12:23:36 2011, comment #9:
More on the DNS thing:
https://savannah.gnu.org/bugs/?31235#comment2
|
Thu Feb 24 12:18:09 2011, comment #8:
I belive it was an attempt to drop random sleeps, considered a waste of time.
From the "sleeps are bad" dept.
BTW, I've found the thread about the DNS issue.
The reporter also suggest to "sleep" (while the current code,
in case of DNS resolving lag, doesn't even select...)
http://marc.info/?l=gnash-dev&m=128289903628873
|
Thu Feb 24 11:44:50 2011, comment #7:
I don't remember any more. If the commit messages or discussion on the list at that point don't give any information the reason is probably lost.
|
Thu Feb 24 11:40:53 2011, comment #6:
What was the reason for introducing select()?
|
Thu Feb 24 10:36:26 2011, comment #5:
If a fixed-time sleep is better, use it.
|
Thu Feb 24 10:29:49 2011, comment #4:
`cause you replaced the fixed-time sleep with the select:
4006e6d610421d15a65e92019587512526583bf3
|
Thu Feb 24 10:21:35 2011, comment #3:
Why is this assigned to me?
|
Wed Feb 23 23:04:43 2011, comment #2:
URL in submission is supposed to be http://cdn.optmd.com/V2/73952/203826/mor_13fifty_v1_728x90_cas.swf
|
Wed Feb 23 22:53:48 2011, comment #1:
basically forcing longer sleeps we guarantee to have less calls to ::fillCache, with bigger chunks of data to crunch.
If this is desiderable, there might be no point in doing the select
|
Wed Feb 23 22:51:14 2011, original submission:
STR:
(Trickle is a bandwidth limiter. Alternatively, if you have a router utterly devoid of QoS, like mine, you can simply start an upload rather than run trickle.)
You'll notice that CPU rises to 100% on at least two cores and stays there for quite some time.
After some debugging I've pinned the issue down to fillCache() busy-looping when data is available continuously. In a low-bandwidth situation, data trickles in for a long time, leading to little to no time spent waiting in select(). With this SWF, a bunch of threads are started at the same time, exacerbating the problem.
Sandro suggests that this could be fixed by reintroducing a sleep removed by 4006e6d610421d15a65e92019587512526583bf3.
|
(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)
Attach File(s):
Comment:
No files currently attached
Depends on the following items: None found
Items that depend on this one: None found
Follows 1 latest change.