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 07EDF40F2A for ; Tue, 12 Apr 2022 22:43:18 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id EA4BA68B35D; Wed, 13 Apr 2022 01:43:15 +0300 (EEST) Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11olkn2075.outbound.protection.outlook.com [40.92.19.75]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 4725868A34B for ; Wed, 13 Apr 2022 01:43:09 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=YqoIOwxGysNKM2582/BxztpN/4NHxerC8OvLR592yFAynMRSpkvdB2LOiRbqsntOKffRyZGmGsKypPjmXCxXAYWuKSHP+1fONJoP9Jfuq2bKCdHqFSnri5VHoMBc+7Xv9cBRO92+Hwo1g2CqVgVut1apk3kXRIopo2PLB7yjboQDlQJG99UFfwC3uvhkG6HBg8tGBB2+fV5cWNbnZCu0dc0Zufl0PCBHJMtTig8JtfVu+ILilQUFu+PPlOjxA24aAr0kHDDhcKd75SIUZauDOieuyQpMaEV3H9Jk9ySTkLCjAL6ZHiE165QjzG8N/PvNkx1xnbgRn3IzkQzcJrOihQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=Mz8r1kCiRwoI11gOJhtMTmCIw9OIg6V5SPZNK+EiIW8=; b=awAepLZpmCdGHavpYH8Wy3VzgIlBSIKZddsonk8wGhJiVkbV9GAOSKj1xW7GQRqdG3fPphgGrzUjfWjdFGRnMyuhGU1+hlzjNN3S9rVdhAg06py9VyE5YmUx7bYafYP6+bxtoFLz/XCMW8DXRFT7hS4nsI8oMf69MRzxKfJLr//utI3r2ylU9mdOTRCwCzeMCt7NSiKHb+e1A0ZUaQMyZuhrxWulG1wi7hMd4TvJ4bC1YEAqiypaliUCzIAOcU9ytuU5oGJzdbQzwEbLHT7XlKZdWT7LiV1lCQB8PubslCExERzrd4lwCtro1s1tFuwvL+meHHT5atd1b0coUt+3Ig== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=hotmail.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Mz8r1kCiRwoI11gOJhtMTmCIw9OIg6V5SPZNK+EiIW8=; b=qBHc7LtzSNgCyK/s5aoIvtsdPTzvwb5G+ul565Xhxg2fIi0O+wFRvyt18v+PQJrw7wi7Ugt7/PpsZzC+syTekvctd90n8pke6+cg/LxzRft2+/0jHhOid9lZLseMpdrI06lhAxDKRwvt7OTqggx03dN9XFO1LHKQC2RrwPWCNHyOZWKk/IuWTtVc6ou5r3Qsafscmbco1e71y/0tQOtnx0sXACMSapQKjm9GlsvyCPrH2VwjhYsHR7XxT9Gc2+NI3ci9LYhU8tP2xwQb4re0nsI6vnRCTTJWZyVQtIsapefIZcmhJmClCh5yOimSCyoIq0n08SgyOjG2VxaG6uxqXw== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by DM8P223MB0126.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:5::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5144.22; Tue, 12 Apr 2022 22:43:06 +0000 Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::1d14:8778:3a51:ed0]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::1d14:8778:3a51:ed0%6]) with mapi id 15.20.5144.029; Tue, 12 Apr 2022 22:43:06 +0000 From: Soft Works To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [RFC] Switching ffmpeg.c to a threaded architecture Thread-Index: AQHYSBeuHAE+RPuGxESL7bK+MVIG+KzhsuYAgAAIfACAANjoZ4ABj6plgAZIWCCAALN0QIAAHCSAgAAAq4CAANK/AIAAuNsw Date: Tue, 12 Apr 2022 22:43:06 +0000 Message-ID: References: <20220404113037.13070-1-anton@khirnov.net> <20220405191542.GV2829255@pb2> <164918796468.24258.6158464741625303482@lain.red.khirnov.net> <164923451378.24258.12863595879743109558@lain.red.khirnov.net> <164932035193.21047.13588447156789154824@lain.red.khirnov.net> <164966573624.21047.5872909837014503123@lain.red.khirnov.net> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-tmn: [FH+PM8YMNKXc+zSW2xBq0AAIZIDcPjuS] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 13e152c4-d768-48c0-fd0f-08da1cd5cfb8 x-ms-traffictypediagnostic: DM8P223MB0126:EE_ x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: m2rU9pNIy6ur17pe/d94O7JZGj+aw9IK7wlHOhm9cWHSXlF88a6YNEVR3Hg/7QwMsOWX/YA7+Y0w0HAKdaZOTTKvxGNka19ve0CSiCvlxky08dx7dKXGXy39kUA0yd0SAiSTh+1riQmthEzJg2+qG+93aHIVQJdELRu5meXFGmUFvK4ivshv4vaI2WEERI3x2k/lmnfexMaqMxBIlVa5ZoUtUAFaqQmLhxVgMoTV9qegrFrg2ocmTXVoJ4F7MYMWmQWliZZtrMrhgjM2ShKDeYPDFXeUbS+WV2Zhw0a2paHvLEq7NVDvxCc88fC1W/aLxX81ylGnFSZxg9bKI5fC5P68H88MLHY2cqAV62bsNAJpvmamSsbPCGy/VMNy8cQuLd8Mi4OJ0MdPdbgwg6DfS41nRvbW4Iphx33gRW6aGzEMcjhKarAjg/NdQB7gPEqAwT+gsYauwegAQdXAQ3J9xoEAR2JwKJNh2cMYOrERrtQTOAE2/qSllYnpAAmQkG0vMQXTge9uLfwB6HIbjcmGoaTQDLM634T/bFJAJDbbYwyoh4ZN14WWkOv2J/DbDJNnNGoRog1gwsQabEbGfbFECg== x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?Vnk5NVB5OFdUUFNyT1FjUHBWS0Q1VVpvcDR5R2JhVXNJOW05YmFsai9pK3d4?= =?utf-8?B?ZGpZeDJaemllSzUxRmNYVDVXMjg3ZGdTZldubUovK2pzWURqUHFpZFY0R0Zr?= =?utf-8?B?MlBWdG41bVRyUjVkNGtSZFV1YnFUbGNPQUdjUWNkM2RtVENoM1NZa0RMQVU0?= =?utf-8?B?NlFnd20zazlUMjNaZWpRSFpFWjBPcEluaHJWK3RDV2g3MUJFZmxGQWRDRERM?= =?utf-8?B?RWdFbk1ONnk1UDJueTdxc3MwYjRWaDlBeWlYWCtDNVVQSjZrK3p1MllTRUN4?= =?utf-8?B?clJXbjNnWXd5ME4ySHRDbGdQbXpOV0tmdndCMG5LMmh3VE13aDVOY3FaUXQ4?= =?utf-8?B?MGVHc1ZkcXNSZ0EycG5KNkd6OFBZSGl2Wk9yRzdyOXJId0JWWE1sa1owcnhj?= =?utf-8?B?eUJ4VkNJb3ZSc29ZcmhGTjV4cDNJLzhWaXFrb1luakwwRGNzYk53VkU4YUZi?= =?utf-8?B?TnU0V3NLOGpyRlFXc3FoYTlxYzFqa0VoTFNzalBPUUh3c2tMVHJ2eVdiWjhh?= =?utf-8?B?UXJscENKVEU2eEMrVHpiQ1VRaUdZSkF6NU5wVEJhK0VjY2VHYWpNSmtCVTNR?= =?utf-8?B?M2tuQ1JkSERjRHo1djQzdlplMDBUZ1lnMCtKM1ZLVUhFckNLTFBQRWVOa3k4?= =?utf-8?B?SDVBbW80MnloVGxwd2ZIYzJ0WTlIZGxiNVJOcElMSkszWG1vMzI3VVBmQmhx?= =?utf-8?B?eHhOV2hJMURLQklha0NONVFaTDZvZWFRTjZNdzVMQUdldUhNYktMempVVkZW?= =?utf-8?B?cGlTUXlJZVc1TXZQVzVWaTZVOGtSY2lvTmI4SE1XR0hMWi9sRFIvVUlXdFQw?= =?utf-8?B?UVhlTTRaVm9BTHJ2alltSG1aRnoyd0J2T1kyMXJUbHZpbGVMcFZUNGZvdzBs?= =?utf-8?B?bEJOTkxYNDF6OHBhTCtDVWlqNmgyRFBhR1NVeE5OZXpEYXhscXdoeTl1S3Az?= =?utf-8?B?Skg3RXJSckYzTVEyZWpuUkhjSk1VRWM1dFNvMStIN1BUNGFqRDZLbTJiVjBj?= =?utf-8?B?QXdrOWh1S3A4NkhUWE9kZTNTcHFkWFdJdDRQWjdSYzhuUTRWeTQ5OGl0KytL?= =?utf-8?B?TWgwUk9jNXZkaEJnVlh3dXZ3Vk1ldU5qc3JqT2l0QWZQTGt5ZHZYaEZvZklk?= =?utf-8?B?MXpyYlM2ZFRSbVJYL1h2MG5Wckp4S0VZQk1BclNhSXRjTTVXS1V6c0Q1R1pE?= =?utf-8?B?ZjZjazB3QUx5WTVHekhkc1JuK0FwMzNibW9rNWdoNDBrdnFTNkZvdVdlWEMr?= =?utf-8?B?dE54UHBBSzFaRXVZWmZNTFJGemE3WnIxQVAxcFlzZzlRaU9VNW1TeUlkN25S?= =?utf-8?B?N3Z5OGhWSXY4d2RWM0lPT08wN2hySmtPT1Q2SllFR0ZpQUUvRnpSTVB6WG9O?= =?utf-8?B?bGNzaGJzWjExa2xHM3dZUEIrZjJtb1l5ditSZHlwbm9hdVpjREhSV3Vuc1RS?= =?utf-8?B?ZGVpOVJ3dlFDSy9FbFFnS3hxbzJhTjJRcUpDbHFrU3VEWGlNWnp2S2ROWXhl?= =?utf-8?B?MmNxek02c012eVRtekQxb2FTbUFUQ2xrT2ppdDZMUE1HSVErcm1xSGM2SGNi?= =?utf-8?B?V0daUVRyR3liUlpIZ2x2MnArQktYVGhuSzFEQ0d5b2NKLzF1TnE1SGpYK3Jq?= =?utf-8?B?aVpyVDBveTgvL3U0dFBIL3dZMnd0bWtoazl0NkhSYWsvcnYxcWV5dFVkWkY1?= =?utf-8?B?RnB4NGZyWms3TU1aN29xemlHdGE2d3hvTUUxdlozY3pSaDFpYXlkWE9jY3Fx?= =?utf-8?B?Tmtud3ZORDB4QTZoNjRvTUF5bjNHSTY2YkVUY1UrQWt2VWplRnRkNnBCYjNy?= =?utf-8?B?SWFXdUZHNTFJMzhNR1E4TWNZbUlvWnByQmxmSnBtbm1QZFM1YXdjOVNxclVX?= =?utf-8?B?MEZJWnNpem5XQ0pEL3Q1Nm9Gek50VEZhUmZTazJENzJ5TnJ4VWtXdmdHNXpO?= =?utf-8?Q?ExTCxh0zyGU=3D?= MIME-Version: 1.0 X-OriginatorOrg: sct-15-20-4755-11-msonline-outlook-1ff67.templateTenant X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-CrossTenant-Network-Message-Id: 13e152c4-d768-48c0-fd0f-08da1cd5cfb8 X-MS-Exchange-CrossTenant-originalarrivaltime: 12 Apr 2022 22:43:06.4340 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-rms-persistedconsumerorg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM8P223MB0126 Subject: Re: [FFmpeg-devel] [RFC] Switching ffmpeg.c to a threaded architecture 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: > -----Original Message----- > From: ffmpeg-devel On Behalf Of Paul > B Mahol > Sent: Tuesday, April 12, 2022 11:29 AM > To: FFmpeg development discussions and patches devel@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [RFC] Switching ffmpeg.c to a threaded > architecture > > On Mon, Apr 11, 2022 at 10:58 PM Soft Works > wrote: > > > > > > > > -----Original Message----- > > > From: ffmpeg-devel On Behalf Of > Paul > > > B Mahol > > > Sent: Monday, April 11, 2022 10:52 PM > > > To: FFmpeg development discussions and patches > > devel@ffmpeg.org> > > > Subject: Re: [FFmpeg-devel] [RFC] Switching ffmpeg.c to a threaded > > > architecture > > > > > > On Mon, Apr 11, 2022 at 10:10 PM Soft Works > > > > wrote: > > > > > > > > > > > > > > > > -----Original Message----- > > > > > From: ffmpeg-devel On Behalf > Of > > > > > Anton Khirnov > > > > > Sent: Monday, April 11, 2022 10:29 AM > > > > > To: FFmpeg development discussions and patches > > > > devel@ffmpeg.org> > > > > > Subject: Re: [FFmpeg-devel] [RFC] Switching ffmpeg.c to a > threaded > > > > > architecture > > > > > > > > > > Quoting Soft Works (2022-04-08 17:27:10) > > > > > > > Furthermore, remember that this is just the first step. > There > > > will > > > > > be > > > > > > > further patchsets converting the other components. I > intend to > > > > > > > upstream > > > > > > > them gradually one after the other. Your suggestion would > > > require > > > > > me > > > > > > > to > > > > > > > instead write the whole thing at once, fighting rebase > > > conflicts > > > > > all > > > > > > > the > > > > > > > way, and then submit it as a giant utterly unreviewable > > > patchset. > > > > > > > > > > > > That's not what I meant, but anyway it's not worth > discussing > > > when > > > > > > it's a minority opinion. > > > > > > > > > > > > Just a practical question instead for planning purposes: > > > > > > > > > > > > Which timeframe do you expect for the whole process? > > > > > > When do you plan to start > > > > > > > > > > If you mean "start pushing the patches", then I intend to do > that > > > as > > > > > they are reviewed and approved. I hope to send the > upstreamable > > > > > version > > > > > of this set this week, if nobody has strong objectsions then I > > > might > > > > > push it after vacation, i.e. late April/early May. > > > > > > > > > > > and for how long do you think it will take until all further > > > > > patchsets > > > > > > will be submitted/applied? > > > > > > > > > > This is very hard to estimate accurately. A pessimistic guess > > > assuming > > > > > I > > > > > get stuck on every stupid thing would be end of this year, but > I > > > hope > > > > > for things to go much faster. > > > > > > > > Thanks for the reply. I'm asking because I need to decide about > the > > > > way I'm going to proceed with the subtitle filtering patchset. > > > > > > > > I think I will have to keep and continue this in private during > this > > > > procedure as I don't have the resources to regularly adapt and > sync > > > > from my (5.0 based) working branch back to the master branch. > > > > > > > > > > > That is big waste of resource when not implementing thing > properly. > > > > From my point of view, somebody who has never given any detailed > > reviews, didn't state what exactly(!) he would consider to be > "improper" > > and never made any suggestion how the implementation would need to > > be changed to become "proper" - doesn't have the right to make such > > claims. > > > > You never asked kindly. I have always asked you kindly, probably more kindly than many others do (going through history, I just found many very kind questions I've been asking you). For the specific issue about the subtitles patchset, you jumped in on IRC, saying "it's a hack". I had asked you (kindly!) what makes you think that it would be a hack and what you would think needs to be changed. You never answered the question since that time, instead you kept labeling it in all those ways. I think the fundamental problem about the current situation is that there were discussions with several other developers whose arguments were based on theoretical considerations after reviewing the code but without having actually worked with it and gone through all the real-world situations that exist. My code though, is made to provision for all cases by providing a high level of flexibility, which is done by having timing fields that are redundant in some cases but are needed (and non-redundant) in other cases. Full coverage of cases is elementary for me, though; I can't drop it, like somebody had suggested. As a matter of fact, I see two chances: - others believe what I'm telling or - another developer of sufficient reputation and credibility gets to look into the details for confirming my reasoning > > > That is big waste of resources Inclusion in ffmpeg master has always been a secondary objective only with the intention to contribute something useful and keep the diff-to-official at a moderate level, avoiding the effort for adapting to upstream changes. Now that ffmpeg.c is about to undergo changes for the next months, it would really be a "big waste of resources" to regularly keep up with those changes. On the other side, I think exactly those changes would have benefitted from many of the subtitle changes in my patchset, as this eliminates a lot of all the special treatment which is in place for subtitle stream handling. I recognize though, that interest and intentions for improvement in this area are low; otherwise, a disagreement about two fields more or less in AVFrame, wouldn't probably be blocking the story as a whole. softworkz _______________________________________________ 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".