All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Krzysztof Helt" <krzysztof.h1@wp.pl>
To: "Antonino A.Daplas" <adaplas@gmail.com>
Cc: linux-fbdev-devel <linux-fbdev-devel@lists.sourceforge.net>
Subject: Re: [PATCH] smart blitter usage for scrolling
Date: Thu, 10 May 2007 07:40:59 +0200	[thread overview]
Message-ID: <4642b06be589b@wp.pl> (raw)
In-Reply-To: <464172cc491d5@wp.pl> <1178720358.4861.27.camel@daplas>

Dnia 9-05-2007 o godz. 16:19 Antonino A. Daplas napisa³(a):
> This sounds quite logical to me.  However, I would rather that 
you
> create a new scrolling mode (ie SCROLL_MOVE_REDRAW and
> SCROLL_WRAP_REDRAW). Then create a new FBINFO_HWACCEL flag that 
will
> enable SCROLL_PAN/WRAP_MOVE_REDRAW and perhaps another one that 
will
> enable MERGE_BLIT.
> 

I understand that you want creating a new flag instead of ifdefs 
but this new method should not be much slower in the worst case 
and much faster on average. I would rather consider replacing the 
existing SCROLL_MOVE mode (so no ifdefs after some 
experimental/test period).
Also, the capabilities of hardware for the new method are exactly 
the same as for the old SCROLL_MOVE. Every time the 
HWACCEL_COPYAREA is selected the new scroll method should also be 
selected. The new HWACCEL flag seems to be alwyas a copy of 
HWACCEL_COPYAREA flag.

The MERGE_BLITS is a different story but my suspection is that 
does not give any real benefit (in the few % range plus or minus) 
and could be dropped completely. I left it to allow members of 
this list testing and return a feedback. If my supection is true 
the whole MERGE_BLITS can be dropped to simplify the code.

Kind regards,
Krzzysztof


----------------------------------------------------
Morderstwa i krew ofiar na ¶cianach. Mroczne miasto 
pogr±¿one w chaosie. Sprawd¼ kto jest Przybywaj±cym!
Przybywaj±cy - thriller metafizyczny ju¿ w ksiêgarniach.
http://klik.wp.pl/?adr=http%3A%2F%2Fksiazki.wp.pl%2Fkatalog%2Fksiazki%2Fksiazka.html%3Fkw%3D10414&sid=1128



-------------------------------------------------------------------------
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/

  reply	other threads:[~2007-05-10  5:41 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-09  7:05 [PATCH] smart blitter usage for scrolling Krzysztof Helt
2007-05-09 14:19 ` Antonino A. Daplas
2007-05-10  5:40   ` Krzysztof Helt [this message]
2007-05-10  5:49     ` Antonino A. Daplas
2007-05-10  5:56       ` Antonino A. Daplas

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4642b06be589b@wp.pl \
    --to=krzysztof.h1@wp.pl \
    --cc=adaplas@gmail.com \
    --cc=linux-fbdev-devel@lists.sourceforge.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.