Re: Performance regression

2011-12-01 Thread Thomas Martitz
Am 02.12.2011 07:27, schrieb Thomas Martitz: Am 01.12.2011 22:47, schrieb Michael Sevakis: Is that a size we still can live with? I forgot to ask: were you proposing to increase it for just maemo or for all targets? I wouldn't recommend doing the latter. In fact it probably wouldn't compile

Re: Performance regression

2011-12-01 Thread Thomas Martitz
Am 01.12.2011 22:47, schrieb Michael Sevakis: Is that a size we still can live with? I forgot to ask: were you proposing to increase it for just maemo or for all targets? I wouldn't recommend doing the latter. In fact it probably wouldn't compile on many targets anyway and also scrollwheels

Re: Performance regression

2011-12-01 Thread Michael Sevakis
Am Donnerstag, 1. Dezember 2011, 22:07:33 schrieb Michael Sevakis: I forgot something else too. The including of optimized mixer assembly for App targets is disabled. Something needs to be done there to get it to compile. Almost fixed, see FS #12421. There's something else I left open for w

Re: Performance regression

2011-12-01 Thread Michael Sevakis
Is that a size we still can live with? I forgot to ask: were you proposing to increase it for just maemo or for all targets? I wouldn't recommend doing the latter. In fact it probably wouldn't compile on many targets anyway and also scrollwheels wouldn't be able to click frequently enough.

Re: Performance regression

2011-12-01 Thread Michael Sevakis
Actually I did a test build back then (still floating around on my phone) and everything was working fine. I didn't notice the performance drop. Maemo doesn't need pcm_play_lock() as the locking is done on the gstreamer object level. It for locking out the callback for the rest of the system, l

Re: Performance regression

2011-12-01 Thread Thomas Jarosch
Am Donnerstag, 1. Dezember 2011, 22:07:33 schrieb Michael Sevakis: > I forgot something else too. The including of optimized mixer assembly for > App targets is disabled. Something needs to be done there to get it to > compile. Almost fixed, see FS #12421. Thomas

Re: Performance regression

2011-12-01 Thread Thomas Jarosch
Am Donnerstag, 1. Dezember 2011, 19:02:28 schrieb Michael Sevakis: > One more thing. You mentioned maemo and really, I did not work on that so > much. I just took my best guess at what to do with it in r30097 and noone > was conveniently around to check it. I also noticed the lack of > pcm_play_loc

Re: Performance regression

2011-12-01 Thread Michael Sevakis
I forgot something else too. The including of optimized mixer assembly for App targets is disabled. Something needs to be done there to get it to compile.

Re: Performance regression

2011-12-01 Thread Michael Sevakis
On Thursday, 1. December 2011 08:45:36 Michael Sevakis wrote: I'll try if I can reproduce it with the SDL RaaA target, debugging/bisecting on the workstation is much easier. One more thing. You mentioned maemo and really, I did not work on that so much. I just took my best guess at what to do

Re: Performance regression

2011-12-01 Thread Thomas Jarosch
On Thursday, 1. December 2011 08:45:36 Michael Sevakis wrote: > On RaaA, I have no idea what the impact was. Some > things have a cost afterall. Thanks for the detailed explanation, Mike. I'll try if I can reproduce it with the SDL RaaA target, debugging/bisecting on the workstation is much easie