[opensource-dev] Review Request: STORM-1122 Linux viewer sucks up file descriptors, stops loading content and crashes

Discrete Dreamscape discrete.dreamscape at gmail.com
Sun Apr 3 20:54:38 PDT 2011



> On April 3, 2011, 8:29 p.m., Merov Linden wrote:
> > indra/llwindow/llwindow.cpp, line 210
> > <http://codereview.secondlife.com/r/245/diff/1/?file=1369#file1369line210>
> >
> >     Hmmm, this is basically doing for Linux what is already done for Mac and Win32. If that's the case, I'd rather have your hack move to llwindowsdl.cpp. It might be time to retire that getDynamicFallbackFontList() code entirely then.

After speaking to a friend and looking a little deeper, it appears that the function does have some reasonable purpose.. the font files included in the viewer do not have substantial Unicode support, which I discovered when trying to input Japanese. On Windows/Mac, there are sections clearly defining fonts that provide this support, but no such fonts exist for Linux, perhaps because it's not certain that they're on the system. I recall from the past that Arial Unicode isn't a completely free font and can't be included by default.. So I guess the proper solution would actually first be to either add Linux (or universal) fonts that provide proper support, or dig deeper into the fallback font function to figure out why it's going descriptor-crazy. It's a pretty hairy-looking thing..


- Discrete


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/245/#review538
-----------------------------------------------------------


On March 30, 2011, 11:49 a.m., Discrete Dreamscape wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/245/
> -----------------------------------------------------------
> 
> (Updated March 30, 2011, 11:49 a.m.)
> 
> 
> Review request for Viewer.
> 
> 
> Summary
> -------
> 
> Resolved Linux file descriptor greediness by removing obsolete fallback font searching (call to LLWindowSDL::getDynamicFallbackFontList()), as this seems to be obsolete unless your skin's configuration references font files that are not packaged with the viewer, which is not the default case. Would like to know if this solves the instability described in STORM-1122 for Linux users, particularly those with lower than average file descriptor limits set (find out by running `ulimit -a`, it's the value 'open files', mine is 1024 on Ubuntu 10.10 and I'd have extreme problems prior to the patch).
> 
> 
> This addresses bug STORM-1122.
>     http://jira.secondlife.com/browse/STORM-1122
> 
> 
> Diffs
> -----
> 
>   doc/contributions.txt a8f868007986 
>   indra/llwindow/llwindow.cpp a8f868007986 
> 
> Diff: http://codereview.secondlife.com/r/245/diff
> 
> 
> Testing
> -------
> 
> Useful ways to examine file descriptor usage for the viewer
> 
> lsof -c do-not | less
> 
> This should be much, much less than 1024
> lsof -c do-not | wc -l
> 
> This shows all descriptors containing the word 'font' along with the number of each (there are tons of duplicates)
> lsof -c do-not | egrep -o '[^ ]*font[^ ]*' | sort | uniq -c | less
> 
> 
> Thanks,
> 
> Discrete
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110404/1803baed/attachment-0001.htm 


More information about the opensource-dev mailing list