[PATCH v2 4/5] fbdev: Improve performance of cfb_imageblit()

Thomas Zimmermann tzimmermann at suse.de
Tue Feb 22 18:48:19 UTC 2022


Hi

Am 22.02.22 um 14:01 schrieb Pekka Paalanen:
> On Mon, 21 Feb 2022 20:54:09 +0100
> Thomas Zimmermann <tzimmermann at suse.de> wrote:
> 
>> Improve the performance of sys_imageblit() by manually unrolling
> 
> sys?
> 
>> the inner blitting loop and moving some invariants out. The compiler
>> failed to do this automatically. This change keeps cfb_imageblit()
>> in sync with sys_imagebit().
> 
> This is correct here.
> 
>>
>> A microbenchmark measures the average number of CPU cycles
>> for sys_imageblit() after a stabilizing period of a few minutes
> 
> sys?
> 
>> (i7-4790, FullHD, simpledrm, kernel with debugging).
>>
>> sys_imageblit(), new: 15724 cycles
> 
> sys?
> 
>> cfb_imageblit(): old: 30566 cycles
>>
>> In the optimized case, cfb_imageblit() is now ~2x faster than before.
>>
>> Signed-off-by: Thomas Zimmermann <tzimmermann at suse.de>
>> ---
>>   drivers/video/fbdev/core/cfbimgblt.c | 51 +++++++++++++++++++++++-----
>>   1 file changed, 42 insertions(+), 9 deletions(-)
> 
> Just noticed some confusion in the commit message.

I copied some of the text from the other commit and I could have sworn I 
updated it. But apparently not.

Best regards
Thomas

> 
> 
> Thanks,
> pq

-- 
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
(HRB 36809, AG Nürnberg)
Geschäftsführer: Ivo Totev
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 840 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20220222/332a58ab/attachment-0001.sig>


More information about the dri-devel mailing list