[PATCH] micro-optimize RADEONCopySwap in radeon_accel.c for powerpc

Jochen Rollwagen joro-2013 at t-online.de
Fri Nov 4 08:25:02 UTC 2016


Am 02.11.2016 um 04:28 schrieb Michel Dänzer:
> On 02/11/16 02:39 AM, Jochen Rollwagen wrote:
>> So as far as i’m concerned the last version of the patch i sent is the
>> final one (i’m attaching it).
> Please re-send the patch you want to be applied generated by git
> format-patch, ideally with your Signed-off-by tag.
>
>
>> On my machine I’m seeing quite noticeable performance improvements with
>> the patch.
> I'm afraid that seems unlikely, since AFAICT there are no callers
> passing RADEON_HOST_DATA_SWAP_HDW to RADEONCopySwap. Did you verify that
> you're really hitting this code?
>
>
> P.S. These days we're reviewing xf86-video-ati patches on the amd-gfx
> mailing list.
>
Hi,

thanks for your reply.

I've sent two patches to the amd-gfx mailing list, one to just remove 
that case (because it's indeed never called and most probably never will 
be) and one to fix a bug re big-endian handling in RadeonCopyData.

Thanks for your efforts.



More information about the xorg-driver-ati mailing list