[PATCH] micro-optimize RADEONCopySwap in radeon_accel.c for powerpc
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.
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