From patchwork Mon Jun 24 13:02:11 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Zhao Zhili X-Patchwork-Id: 50124 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a59:ae71:0:b0:482:c625:d099 with SMTP id w17csp2000269vqz; Mon, 24 Jun 2024 06:12:46 -0700 (PDT) X-Forwarded-Encrypted: i=2; AJvYcCXKewbgc1HLxR7SfYWFbFYqRDkkQTxd21pjUS7vBsEjbgC6BSJg1fZe2YiANA+FKjArhYD6yHOkBRprtUijQQ66NUBBRM6wQrhugw== X-Google-Smtp-Source: AGHT+IFmYsX2+eY6usBbaNjvj1/pZnqJtsCVJs2Du8XBQIvpbdMvbYsFICapXGKbneOlkMbpvciU X-Received: by 2002:a17:907:6a0e:b0:a6f:e05b:b1f5 with SMTP id a640c23a62f3a-a7245c84f13mr334957766b.6.1719234766573; Mon, 24 Jun 2024 06:12:46 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1719234766; cv=none; d=google.com; s=arc-20160816; b=ymy39vnC7Flth0A3Jn4doyk/DcALWSC893lWkn1GeoC92gH3D/cOfoW2J8KG4msJLt HfUXPtN0NDEAr6Qv+C1wMyyref09Uh5vs5EnlaTxiKpTC6YzYfU9bLTpf5Vdi6BsHY7B XLdDyNrS0Mu3Y937b78USy3EPStteGEhCJ5AMv3uK3ApGAHghiF1msCJdQu4gIj/BMiG 9zMY4WUnyj2Hu8pEeUj//g+hgQnsfZPYNVTjujLH0I+Z28lJp92iEDMJGec4hgtiR6gC TXyXNI09u3BS6VL8WaeOZxxJKUn4IXEVw4Xw1kvoZDeQvGDKl9sewgnooN2W6y+wdDLJ w2cQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:cc:reply-to :list-subscribe:list-help:list-post:list-archive:list-unsubscribe :list-id:precedence:subject:mime-version:references:in-reply-to:date :to:from:message-id:dkim-signature:delivered-to; bh=cu6MwSTzFUlSCOBjCyv1qEKly6/g+ptBMMT3ErPRcR0=; fh=HnHYuZ9XgUo86ZRXTLWWmQxhslYEI9B9taZ5X1DLFfc=; b=kh8wYktNTqdZq7ir3f68qf4RfmzLzfP9NbPeqjNdQUCOEYwD8TqQ3iN2Q/wfc7Qk68 tzxqXAZ0teZ/t/v6rPaYiloBI36chggqvuKlJMFu9BOpDKnf4FtVQgxvAy2ZHaqPnMNP 7zqVR+dFT8NVdkpEzsdPgKTUi89lYXNMAEV/YVfYNesq7i4trYqZucR2n/44jK6VVgCQ 0jRqx8sMG/R29VC20NtX+pwPMhS/4LITVr5eEGforWUUwoK1IS07VPL+SdzMzfNeQ+67 8cVrTAhnMSAgvTcg2vAi3Dhjv2szvnqB7dLZPr1DRr01EIxtPsg289TKZE0UtxJ1F53k IvWg==; dara=google.com ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@foxmail.com header.s=s201512 header.b=lTuIPRym; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=foxmail.com Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id a640c23a62f3a-a6fcf579dcbsi383388266b.877.2024.06.24.06.12.46; Mon, 24 Jun 2024 06:12:46 -0700 (PDT) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@foxmail.com header.s=s201512 header.b=lTuIPRym; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=foxmail.com Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 1C6BC68D62E; Mon, 24 Jun 2024 16:02:35 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from out203-205-221-242.mail.qq.com (out203-205-221-242.mail.qq.com [203.205.221.242]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id EFE5568D0E4 for ; Mon, 24 Jun 2024 16:02:26 +0300 (EEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=foxmail.com; s=s201512; t=1719234137; bh=8WYXPGxnKhEGtYyFmoqHbV07jZjAb6611Iimmve4s/s=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=lTuIPRymMW0eDIKMHErvLFxQeRQpoj9Bf7spTCSUimhFoRJQNAexKVR4UGDcg6NNj j/O+Ouf7qBm5zBb8NthoU/8g5MAmEtdZr0hhLiDLE0KNDODXhDZ+Gl0S2YIvggeFLU gfofo4o1KUyjmSvmIDQ/ZJa79yqbm0qSWIaoUN3s= Received: from ZHILIZHAO-MB1.tencent.com ([119.147.10.191]) by newxmesmtplogicsvrszb9-1.qq.com (NewEsmtp) with SMTP id 903582A; Mon, 24 Jun 2024 21:02:16 +0800 X-QQ-mid: xmsmtpt1719234136t474dzsvx Message-ID: X-QQ-XMAILINFO: NGZp1yYNf7Y+F1vsZ8ABarUf7w3+Sd4cKl9q+y5ecmzfSwavFr1GQVIEPf8yFk xLWbTG77+vOBSy5QChyKgdlj7zTBdOF5XtLyLNxAisgCdbTjJjSiBhuJwIJfxn+2e1jo0l1jAaOO 2e4K0Mw9p5+2EmUH6T9h92sKV1wr/ASzaxkS4XuwtlYiwf7B3xTuDhChUXOId2OdsaW0Ty9d6aZI 8+R4cxNvlXLIK3wbe5e7LgzDPb7ffsBjZ3UdoCZZWJkde/sQgNtADbjaDI5ntThPhNCY136Gij1n aP+0wn2qVtLfFM7s2BaUNCkTPT/xtGUk+7MZsqRch3lxuAkzPD33JBuHZIwCMurwDQsgsNyUeV7U uwmSPUZzU58TOa6Bnl3ptyKVMop56s7clSo8LTi9OzfBbj9rusryZT9jk+jOxGEpT53V12T90aIg 04yyF3L7r7qAdlFVPxi8nYesA1cLFoifC7fMUOANdW+0Pb/NKuqH/NP4PK08eoTksUX95S5tHEay cH5xahiz23X+0cxD763SkG0SR2aIDt4tucE54env4jXURAIu4AkE0FaLJw77FU5RNJYFc/JRFzrV RBTdIlAomAd/x2SHqt0SfYmnPPbPdEUlIP9yu2mwrUdrWPw68xKu0NzYfVAirCazb7cFRrwYcB9Q TfaPD6yZofOJ7n75vRstbhIJA/H7+CwzfBRO0Tx/8hypR9O6b731l7GcZRaZP8rbNGvavpui5kHG rMl4b5cPbtgEfVrSeRHbGhg2MXH+KzWiWltXl0RycV8Uq8/vFog9oA1YGIQjmFbt0sWrNwa9lPFE qF0p4cH7CJQ2QwIIDwaXnnWsreSIahJzWPRgXJDvTOiIG3a/fZcIriyhu4Q0EM7NmN+3SImmeJsM mHrUD47CqaS2/6+hL97hqzhXl3rwJ/TPTZ2lz9Sn3gQjngVt45eJz2iI+7Z1a6QUdkNGOWGVOc1g oUngUKLQ5DQ1LT41myzIUnc5PE1paFdsvi/3beAmJCwinLyUc7AkOaaaZzfVECHOj5FMv824ofJz nSn4hgwPiCmGISxtaBi0Ks6B4X43cPC1Mk7qzmN52u2EOJUJcDyT/XoDNdj3Q= X-QQ-XMRINFO: Nq+8W0+stu50PRdwbJxPCL0= From: Zhao Zhili To: ffmpeg-devel@ffmpeg.org Date: Mon, 24 Jun 2024 21:02:11 +0800 X-OQ-MSGID: <20240624130213.71634-1-quinkblack@foxmail.com> X-Mailer: git-send-email 2.42.0 In-Reply-To: References: MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH v4 1/3] swscale/aarch64: Add bgr24 to yuv X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Cc: Zhao Zhili Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" X-TUID: UcBkTDyKJiJ0 From: Zhao Zhili Test on Apple M1 with kperf : -O3 : -O3 -fno-vectorize bgr24_to_uv_8_c : 28.5 : 52.5 bgr24_to_uv_8_neon : 54.5 : 59.7 bgr24_to_uv_128_c : 294.0 : 830.7 bgr24_to_uv_128_neon : 99.7 : 112.0 bgr24_to_uv_1080_c : 965.0 : 6624.0 bgr24_to_uv_1080_neon : 751.5 : 754.7 bgr24_to_uv_1920_c : 1693.2 : 11554.5 bgr24_to_uv_1920_neon : 1292.5 : 1307.5 bgr24_to_uv_half_8_c : 54.2 : 37.0 bgr24_to_uv_half_8_neon : 27.2 : 22.5 bgr24_to_uv_half_128_c : 127.2 : 392.5 bgr24_to_uv_half_128_neon : 63.0 : 52.0 bgr24_to_uv_half_1080_c : 880.2 : 3329.0 bgr24_to_uv_half_1080_neon : 401.5 : 390.7 bgr24_to_uv_half_1920_c : 1585.7 : 6390.7 bgr24_to_uv_half_1920_neon : 694.7 : 698.7 bgr24_to_y_8_c : 21.7 : 22.5 bgr24_to_y_8_neon : 797.2 : 25.5 bgr24_to_y_128_c : 88.0 : 280.5 bgr24_to_y_128_neon : 63.7 : 55.0 bgr24_to_y_1080_c : 616.7 : 2208.7 bgr24_to_y_1080_neon : 900.0 : 452.0 bgr24_to_y_1920_c : 1093.2 : 3894.7 bgr24_to_y_1920_neon : 777.2 : 767.5 --- libswscale/aarch64/input.S | 71 ++++++++++++++++++++++++++---------- libswscale/aarch64/swscale.c | 32 +++++++++------- 2 files changed, 71 insertions(+), 32 deletions(-) diff --git a/libswscale/aarch64/input.S b/libswscale/aarch64/input.S index 33afa34111..2cfec4cb6a 100644 --- a/libswscale/aarch64/input.S +++ b/libswscale/aarch64/input.S @@ -20,7 +20,7 @@ #include "libavutil/aarch64/asm.S" -.macro rgb24_to_yuv_load_rgb, src +.macro rgb_to_yuv_load_rgb src ld3 { v16.16b, v17.16b, v18.16b }, [\src] uxtl v19.8h, v16.8b // v19: r uxtl v20.8h, v17.8b // v20: g @@ -30,7 +30,7 @@ uxtl2 v24.8h, v18.16b // v24: b .endm -.macro rgb24_to_yuv_product, r, g, b, dst1, dst2, dst, coef0, coef1, coef2, right_shift +.macro rgb_to_yuv_product r, g, b, dst1, dst2, dst, coef0, coef1, coef2, right_shift mov \dst1\().16b, v6.16b // dst1 = const_offset mov \dst2\().16b, v6.16b // dst2 = const_offset smlal \dst1\().4s, \coef0\().4h, \r\().4h // dst1 += rx * r @@ -43,12 +43,20 @@ sqshrn2 \dst\().8h, \dst2\().4s, \right_shift // dst_higher_half = dst2 >> right_shift .endm +function ff_bgr24ToY_neon, export=1 + cmp w4, #0 // check width > 0 + ldp w12, w11, [x5] // w12: ry, w11: gy + ldr w10, [x5, #8] // w10: by + b.gt 4f + ret +endfunc + function ff_rgb24ToY_neon, export=1 cmp w4, #0 // check width > 0 ldp w10, w11, [x5] // w10: ry, w11: gy ldr w12, [x5, #8] // w12: by b.le 3f - +4: mov w9, #256 // w9 = 1 << (RGB2YUV_SHIFT - 7) movk w9, #8, lsl #16 // w9 += 32 << (RGB2YUV_SHIFT - 1) dup v6.4s, w9 // w9: const_offset @@ -59,9 +67,9 @@ function ff_rgb24ToY_neon, export=1 dup v2.8h, w12 b.lt 2f 1: - rgb24_to_yuv_load_rgb x1 - rgb24_to_yuv_product v19, v20, v21, v25, v26, v16, v0, v1, v2, #9 - rgb24_to_yuv_product v22, v23, v24, v27, v28, v17, v0, v1, v2, #9 + rgb_to_yuv_load_rgb x1 + rgb_to_yuv_product v19, v20, v21, v25, v26, v16, v0, v1, v2, #9 + rgb_to_yuv_product v22, v23, v24, v27, v28, v17, v0, v1, v2, #9 sub w4, w4, #16 // width -= 16 add x1, x1, #48 // src += 48 cmp w4, #16 // width >= 16 ? @@ -85,10 +93,7 @@ function ff_rgb24ToY_neon, export=1 ret endfunc -.macro rgb24_load_uv_coeff half - ldp w10, w11, [x6, #12] // w10: ru, w11: gu - ldp w12, w13, [x6, #20] // w12: bu, w13: rv - ldp w14, w15, [x6, #28] // w14: gv, w15: bv +.macro rgb_set_uv_coeff half .if \half mov w9, #512 movk w9, #128, lsl #16 // w9: const_offset @@ -105,12 +110,26 @@ endfunc dup v6.4s, w9 .endm +function ff_bgr24ToUV_half_neon, export=1 + cmp w5, #0 // check width > 0 + b.le 3f + + ldp w12, w11, [x6, #12] + ldp w10, w15, [x6, #20] + ldp w14, w13, [x6, #28] + b 4f +endfunc + function ff_rgb24ToUV_half_neon, export=1 cmp w5, #0 // check width > 0 b.le 3f + ldp w10, w11, [x6, #12] // w10: ru, w11: gu + ldp w12, w13, [x6, #20] // w12: bu, w13: rv + ldp w14, w15, [x6, #28] // w14: gv, w15: bv +4: cmp w5, #8 - rgb24_load_uv_coeff half=1 + rgb_set_uv_coeff half=1 b.lt 2f 1: ld3 { v16.16b, v17.16b, v18.16b }, [x3] @@ -118,8 +137,8 @@ function ff_rgb24ToUV_half_neon, export=1 uaddlp v20.8h, v17.16b // v20: g uaddlp v21.8h, v18.16b // v21: b - rgb24_to_yuv_product v19, v20, v21, v22, v23, v16, v0, v1, v2, #10 - rgb24_to_yuv_product v19, v20, v21, v24, v25, v17, v3, v4, v5, #10 + rgb_to_yuv_product v19, v20, v21, v22, v23, v16, v0, v1, v2, #10 + rgb_to_yuv_product v19, v20, v21, v24, v25, v17, v3, v4, v5, #10 sub w5, w5, #8 // width -= 8 add x3, x3, #48 // src += 48 cmp w5, #8 // width >= 8 ? @@ -158,19 +177,33 @@ function ff_rgb24ToUV_half_neon, export=1 ret endfunc +function ff_bgr24ToUV_neon, export=1 + cmp w5, #0 // check width > 0 + b.le 3f + + ldp w12, w11, [x6, #12] + ldp w10, w15, [x6, #20] + ldp w14, w13, [x6, #28] + b 4f +endfunc + function ff_rgb24ToUV_neon, export=1 cmp w5, #0 // check width > 0 b.le 3f + ldp w10, w11, [x6, #12] // w10: ru, w11: gu + ldp w12, w13, [x6, #20] // w12: bu, w13: rv + ldp w14, w15, [x6, #28] // w14: gv, w15: bv +4: cmp w5, #16 - rgb24_load_uv_coeff half=0 + rgb_set_uv_coeff half=0 b.lt 2f 1: - rgb24_to_yuv_load_rgb x3 - rgb24_to_yuv_product v19, v20, v21, v25, v26, v16, v0, v1, v2, #9 - rgb24_to_yuv_product v22, v23, v24, v27, v28, v17, v0, v1, v2, #9 - rgb24_to_yuv_product v19, v20, v21, v25, v26, v18, v3, v4, v5, #9 - rgb24_to_yuv_product v22, v23, v24, v27, v28, v19, v3, v4, v5, #9 + rgb_to_yuv_load_rgb x3 + rgb_to_yuv_product v19, v20, v21, v25, v26, v16, v0, v1, v2, #9 + rgb_to_yuv_product v22, v23, v24, v27, v28, v17, v0, v1, v2, #9 + rgb_to_yuv_product v19, v20, v21, v25, v26, v18, v3, v4, v5, #9 + rgb_to_yuv_product v22, v23, v24, v27, v28, v19, v3, v4, v5, #9 sub w5, w5, #16 add x3, x3, #48 // src += 48 cmp w5, #16 diff --git a/libswscale/aarch64/swscale.c b/libswscale/aarch64/swscale.c index e4ea3309ba..c6594944c3 100644 --- a/libswscale/aarch64/swscale.c +++ b/libswscale/aarch64/swscale.c @@ -201,19 +201,18 @@ void ff_yuv2plane1_8_neon( default: break; \ } -void ff_rgb24ToY_neon(uint8_t *_dst, const uint8_t *src, const uint8_t *unused1, - const uint8_t *unused2, int width, - uint32_t *rgb2yuv, void *opq); - -void ff_rgb24ToUV_neon(uint8_t *_dstU, uint8_t *_dstV, const uint8_t *unused0, - const uint8_t *src1, - const uint8_t *src2, int width, uint32_t *rgb2yuv, - void *opq); - -void ff_rgb24ToUV_half_neon(uint8_t *_dstU, uint8_t *_dstV, const uint8_t *unused0, - const uint8_t *src1, - const uint8_t *src2, int width, uint32_t *rgb2yuv, - void *opq); +#define NEON_INPUT(name) \ +void ff_##name##ToY_neon(uint8_t *dst, const uint8_t *src, const uint8_t *, \ + const uint8_t *, int w, uint32_t *coeffs, void *); \ +void ff_##name##ToUV_neon(uint8_t *, uint8_t *, const uint8_t *, \ + const uint8_t *, const uint8_t *, int w, \ + uint32_t *coeffs, void *); \ +void ff_##name##ToUV_half_neon(uint8_t *, uint8_t *, const uint8_t *, \ + const uint8_t *, const uint8_t *, int w, \ + uint32_t *coeffs, void *) + +NEON_INPUT(bgr24); +NEON_INPUT(rgb24); void ff_lumRangeFromJpeg_neon(int16_t *dst, int width); void ff_chrRangeFromJpeg_neon(int16_t *dstU, int16_t *dstV, int width); @@ -247,6 +246,13 @@ av_cold void ff_sws_init_swscale_aarch64(SwsContext *c) c->yuv2planeX = ff_yuv2planeX_8_neon; } switch (c->srcFormat) { + case AV_PIX_FMT_BGR24: + c->lumToYV12 = ff_bgr24ToY_neon; + if (c->chrSrcHSubSample) + c->chrToYV12 = ff_bgr24ToUV_half_neon; + else + c->chrToYV12 = ff_bgr24ToUV_neon; + break; case AV_PIX_FMT_RGB24: c->lumToYV12 = ff_rgb24ToY_neon; if (c->chrSrcHSubSample)