Thread (7 messages) 7 messages, 3 authors, 2004-04-21

Re: [PATCH] radeonfb(): memmove() fix -- this one works ;-)

From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Date: 2004-04-19 01:58:53

,On Mon, 2004-04-19 at 11:46, David Eger wrote:
Dear Ben,

Please apply this patch to the radeonfb driver (I think of you as the
maintainer for this driver) and then forward to James and Linus :-)

This fixes the same problem as mentioned before -- overlapping
regions sent to copyarea() get fux0r3d.  This patch corrects my earlier
braindamage of setting up w, h, dx, etc. and then forgetting to use them
later on.  Ah, having two versions of the same code (userspace and kernel
space) -- little details like that just float past the eye...  no problem
with fbcon at all, as i previously speculated.  apologies to james.
BTW, James, did we decide what is the best way to set fbcon accel
mode ? For drivers with copyarea accelerated, we want YMOVE. I'd like
a patch to send Linus/Andrew asap along with some pending radeonfb
fixes for that.

Ben.




-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help