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 B0E1F452E6 for ; Fri, 17 Feb 2023 16:35:39 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 1F3E768BE51; Fri, 17 Feb 2023 18:35:36 +0200 (EET) Received: from w4.tutanota.de (w4.tutanota.de [81.3.6.165]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id B0E6568BA2D for ; Fri, 17 Feb 2023 18:35:29 +0200 (EET) Received: from tutadb.w10.tutanota.de (unknown [192.168.1.10]) by w4.tutanota.de (Postfix) with ESMTP id 0CA751060158 for ; Fri, 17 Feb 2023 16:35:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1676651728; s=s1; d=lynne.ee; h=From:From:To:To:Subject:Subject:Content-Description:Content-ID:Content-Type:Content-Type:Content-Transfer-Encoding:Content-Transfer-Encoding:Cc:Date:Date:In-Reply-To:In-Reply-To:MIME-Version:MIME-Version:Message-ID:Message-ID:Reply-To:References:References:Sender; bh=PP3dbyFewiL3aHPst544yIxdFk6KDnCzOX0ZsGyyUko=; b=EbpkTCwLrkJvMLsKFmylcrD6klRHVUOzNU/G968YuNdcoBHDyqEiZKQF3eJtbnO5 VqmnnYSQnL21bZ3rhTftQb9HHHB0RZXq2bqGrVPUqr08hy9ve0o99DhkV2ZKiPlt7mz 7hJykVHJfQlRRstgAywyTyQ3UGlUOeNLWXyav81yVCA+0JPHtmMXnAUEtkkOQuOsAOx RPzlvO90L8bMVQJHvOeunVSCOqTzYiF7itcBT1xMDmHZTywE9Nf6Z9u96SbD0LMy+9e XAiO4dlAzUY5LS1P6j/MvHMxdBidDFIky4wPLxjfui9xUCDJaxoVjSYzRLklXCQPwcw jQgVS3+yMw== Date: Fri, 17 Feb 2023 17:35:28 +0100 (CET) From: Lynne To: FFmpeg development discussions and patches Message-ID: In-Reply-To: <20230217154501.GA4538@pb2> References: <8d23d0d4-51bd-4ae4-ba00-3ec328978647@betaapp.fastmail.com> <20230217154501.GA4538@pb2> MIME-Version: 1.0 Subject: Re: [FFmpeg-devel] [PATCH 00/72] Implement support for Vulkan multiplane images and video decoding 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: Feb 17, 2023, 16:45 by michael@niedermayer.cc: > On Fri, Feb 17, 2023 at 12:52:21PM +0100, Lynne wrote: > >> Feb 17, 2023, 12:05 by kierank@obe.tv: >> >> > On Fri, 17 Feb 2023 at 09:09, Jean-Baptiste Kempf wrote: >> > >> >> Hello, >> >> >> >> On Fri, 17 Feb 2023, at 04:43, Lynne wrote: >> >> > This small patchset mostly rewrites Vulkan to enable using multiplane >> >> images, >> >> >> >> This is not small. We're talking about thousands of lines of code; >> >> >> >> And this changes numerous h264 and hevc headers, and adds callback to the >> >> make AVHWAccel. >> >> >> >> And that is besides the full rewrite of some Vulkan files... >> >> >> >> This will take a long time to review. >> >> >> > >> > I agree. >> > >> >> The codec changes are simple and separate, so it's not that bad to read. >> There's no more than a 100 lines or so added to the H264 parser. >> The 72 patch figure may be scary, but that's because I didn't want to >> squash my vulkan changes behind large rewrite commits. Otherwise, >> this would be a 30-commit patchset. >> > > some of these patches are very simple, yes. Others probably not (i only looked at > the first few) > That said i was never a friend of large last minute patch-sets before releases. > I think the real deciding factor is the communities opinion and > 3 Developers already are against this going in before the branching. So > unless theres a shift in oppinion i think there are more people for applying > this after branching release/6.0 than before. > > Is it an issue if its only in 6.1 in 6months ? > It's quite a long ways away, and this is by far the most complete implementation of the spec which everyone is testing against. This is the first vendor-independent, OS-independent and hardware-independent video decoding API, and a lot of important users like browsers and media players want to use it. Not having it in this release would mean that all those users will use Gstreamer instead, or nothing at all, instead choosing proprietary libraries. 6.0 will still have at least a few days of testing after tagging, which is enough to discover and fix major issues, if they're found. A single dev so far has really suggested leaving it out of 6.0, with a few others just saying that it'll take time to review, but I think reviews are done by looking at code rather than speculating, and at least one person has said they'll be able to look at it tomorrow. _______________________________________________ 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".