Re: [PATCH] pm2fb: fb_sync added

["Krzysztof Helt" <krzysztof.h1@wp.pl>]

Dnia 8-05-2007 o godz. 14:32 Antonino A. Daplas napisa³(a):
> On Tue, 2007-05-08 at 09:26 +0200, Krzysztof Helt wrote:

> > Current testing shows that fb_sync is not
> > required

> 
> Actually, creating an fb_sync() method would be better in terms of
> maximizing the thoroughput of the GPU in comparison with
indiscriminate
> calls to sync waits. 


I was not clear in the first email. The pm2fb_sync is required to
avoid corruption of display (to define it as fb_sync) during
scrolling. It is not required to be called inside the
pm2fb_block_op() explicitly.

The change is to remove the sync call from the pm2fb_block_op().

> It would have been better if imageblit is also
> hardware assisted.
> 

I know, but I have only as much time as my work and family allow
 (there was plenty of it last week = holiday).

I see a major problem with using copyarea acceleration comparing
to redraw function. The copyarea always moves the whole
framebuffer (milions of pixels) while the redraw works very smart
to redraw only things which has to be redrawn. I am working to
use something smarter for copyarea. No success yet.

Regards,
Krzysztof

----------------------------------------------------
Zobacz, jak± walkê stoczy superbohater, by ocaliæ siebie
i swych bliskich. A najwiêksza bitwa rozegra siê w jego duszy.
SPIDER-MAN 3 w kinach od 4 maja.
http://klik.wp.pl/?adr=http%3A%2F%2Fadv.reklama.wp.pl%2Fas%2Fspiderman3.html&sid=1122



-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/