From 7b30a3e98def6f9f158da0e315b7077655acfd20 Mon Sep 17 00:00:00 2001 From: Jeroen Bakker Date: Fri, 11 Jun 2021 15:55:09 +0200 Subject: Performance: Use parallel range for ImBuf scanline processor. Scanline processor did its own heurestic what didn't scale well when having a multiple cores. In stead of using our own code this patch will leave it to TBB to determine how to split the scanlines over the available threads. Performance of the IMB_transform before this change was 0.002123s, with this change 0.001601s. This change increases performance in other areas as well including color management conversions. Reviewed By: zeddb Differential Revision: https://developer.blender.org/D11578 --- source/blender/imbuf/intern/rectop.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) (limited to 'source/blender/imbuf/intern/rectop.c') diff --git a/source/blender/imbuf/intern/rectop.c b/source/blender/imbuf/intern/rectop.c index 6ae93def50f..4b5d68b9c13 100644 --- a/source/blender/imbuf/intern/rectop.c +++ b/source/blender/imbuf/intern/rectop.c @@ -988,8 +988,9 @@ typedef struct RectBlendThreadData { bool accumulate; } RectBlendThreadData; -static void rectblend_thread_do(void *data_v, int start_scanline, int num_scanlines) +static void rectblend_thread_do(void *data_v, int scanline) { + const int num_scanlines = 1; RectBlendThreadData *data = (RectBlendThreadData *)data_v; IMB_rectblend(data->dbuf, data->obuf, @@ -999,11 +1000,11 @@ static void rectblend_thread_do(void *data_v, int start_scanline, int num_scanli data->texmask, data->mask_max, data->destx, - data->desty + start_scanline, + data->desty + scanline, data->origx, - data->origy + start_scanline, + data->origy + scanline, data->srcx, - data->srcy + start_scanline, + data->srcy + scanline, data->width, num_scanlines, data->mode, -- cgit v1.2.3