From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id 2B00244CB9 for ; Sat, 14 Jan 2023 14:28:28 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 3245B6808C9; Sat, 14 Jan 2023 16:28:25 +0200 (EET) Received: from mail-ed1-f47.google.com (mail-ed1-f47.google.com [209.85.208.47]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id B636568037B for ; Sat, 14 Jan 2023 16:28:18 +0200 (EET) Received: by mail-ed1-f47.google.com with SMTP id v10so33735710edi.8 for ; Sat, 14 Jan 2023 06:28:18 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=to:subject:message-id:date:from:in-reply-to:references:mime-version :from:to:cc:subject:date:message-id:reply-to; bh=7NNGbEDiSw2AV0+t0yPTv+9F4Nuzz/YOZ/ev91dAYxc=; b=W5ob596Wu5vGZYyWFJvpgFFiLVS2cyroCxmzy6jDUWZORihgHBuKRIeRfMYT80rvTU UIaW9FVSXBfK0kAMoM5/gxEqRVPZzIaUO3tsPvicRPdozyG523riD+WSgOU0a2F/OpEC UZxmQmcu23eM5/XcIaP4Cl5rsItuq+z0PXW1d8+bWlvlzIshwyRvYUtaSMxKn0fXwIcI 3QVrjJ9p9ZTLbfk/S5mzNPAMFFt5P3Tn9V2hPHzHYojsk9mVJYgi9E7CT//nhGajr5FM vbqOll5aq3zUI7wWKKtXwg9OQSX7zbh0xlgLf3auZvcF9SxDyFThadd9gTd8FCWwY+jf fJxQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=to:subject:message-id:date:from:in-reply-to:references:mime-version :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=7NNGbEDiSw2AV0+t0yPTv+9F4Nuzz/YOZ/ev91dAYxc=; b=cUAdtjBhN36aZXX2wH9P+2IKFEV5M+XxNYnx3xo52ztisrqb/stVZ8X+CsQNnnPOfc xMV7OUJ+Jd/ATgXgyjvkfQEd4ubI/jr/EPDifin5B2TdMlslCFpUoCXyrakHC2NUJelL 9oEGUExl1s7Br3+GIcDbw95aQjJBzboyJTsg3tvWAQUixSF0pA2/Y69bJGwSRXOhDaJ7 upIzPw70RVwCw86P/IWp2udINatvD2Am6PRvg5QEI9ShgZEdbCNPIfErKYkShEKpLFAa e/oomwrXNGhxf9uie9vcouYV8l8O1fzbMB29sDlWDlMvADVCQna7jit0U3ukDG/+hcmW V3DQ== X-Gm-Message-State: AFqh2ko432MLDFoviQhT/5MlHCRE8JVKeCmUuF03Mq7l/sg9FfKGFzVw Uu8mYrIjCzyV0LIJo0eQi9ExKzFCv4gz0UpiUU/LgNIL2R0= X-Google-Smtp-Source: AMrXdXvwzmqTEu9Yupllvec823d/GrLo1NkPt9Z/85OqvWtXUxFo82YDC+uXx9XbFzMk4+a5h9NXvNU1bO50IfzS8ug= X-Received: by 2002:a05:6402:4cd:b0:49b:53d6:ca38 with SMTP id n13-20020a05640204cd00b0049b53d6ca38mr1379860edw.221.1673706497606; Sat, 14 Jan 2023 06:28:17 -0800 (PST) MIME-Version: 1.0 References: In-Reply-To: From: "Ronald S. Bultje" Date: Sat, 14 Jan 2023 09:28:05 -0500 Message-ID: To: FFmpeg development discussions and patches X-Content-Filtered-By: Mailman/MimeDel 2.1.29 Subject: Re: [FFmpeg-devel] Let us review and collebrate on vvc native decoder. 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 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: Hi, On Sat, Jan 14, 2023 at 8:32 AM Nuo Mi wrote: > On Sat, Jan 14, 2023 at 9:13 PM Nuo Mi wrote: > > I am happy to send out the first draft of vvc decoder. > > It's not ready for upstream yet, but it's a good base ground for review > > and future improvement. > > > > It has the following features: > > * C only > > + Fast. On a 4 cores laptop, it can get 30~35+ fps for 10bits 1080P. > > + Support traditional features I, P B frames, 8/10 bits, chroma 400,420, > > 422, and 444, > > + Support VVC new tools like MIP, CCLM, AFFINE, GPM, DMVR, PROF, BDOF, > > LMCS, ALF > > - Not support RPR, PCM, IBC, PALETTE, and other minor features yet. > > > > Please help review the attached patch and provide your valuable comment. > The custom threading framework (Executor, Task) - can you elaborate on this further? If this is meant to be used outside (since it lacks VVC prefix), it should be properly namespaced (FF, AV). It seems like you're trying to combine row/col/frame threading from vvc_thread.c. These (frame, col/row) are fairly common in FFmpeg and there already is existing API for that (although you can't combine them; ask kurosu who IIRC has a private branch doing this for hevc :-) ). This probably needs some further discussion depending on what the rest of the community wants. Right now, this adds VVC private functionality to enable combination of "slice" (row/col) and "frame" threading, which other codecs wouldn't have. This is probably not a desirable end state. Some functions contain av_assert calls, e.g. chroma_mc_bi or get_luma_weight. Are these not yet implemented or impossible because of bitstream coding constraints? If former, please use avpriv_report_missing_feature() so the user is informed of the incomplete decoding. If the latter, please add a comment and remove #if 0 code. For other cases of missing features (e.g. PCM), don't silently return 0, but use avpriv_report_missing_feature() so the user is informed. How come vvcdsp has only 8/10 bits/component code but vvcpred has 8/9/10/12 bits/component code? Not a lot of comments other than references to the spec... Is any of the code (e.g. tools such as sao dsp) shareable with HEVC? This would reduce the total implementation requirement for arch-specific code. Ronald _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".