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 C3A3E423EF for ; Thu, 14 Apr 2022 21:38:00 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 94A6D68B4A5; Fri, 15 Apr 2022 00:37:57 +0300 (EEST) Received: from NAM04-BN8-obe.outbound.protection.outlook.com (mail-bn8nam08olkn2098.outbound.protection.outlook.com [40.92.47.98]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id A291168B05A for ; Fri, 15 Apr 2022 00:37:50 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=GWLNPk1h0UlpeVhTIlQxuTejPvdMMhvhJkk6cNSVLbQI84Plr5MOp5vEhAMiGKKAkAr08If2JFSXR7dfi4jYl4N+iMnbCXNDDqtjK9xdV7hh2aJfyea6EqU+I9tsx7WQI/TwAkqXxS9bgxzRE9lZ+mRsiIDR2sGiN7oX+ee9cMvtMYt+1Ni4dMpbGirDDV+R2K4BjGcfTnMyq/4poGwFuJsAOblV+QjGQeiKN6bEJSEDIQNxDBzJA6UAKohZn+1nlZY5ir10/0MXTmegPYrgFkoYXIzzgyUTb6Y5VPzXEw+9PAL7zcDlC6zto2o5/F3Zq9kK4t/bmtMTwMvviF/bKg== 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=9bnM7JK5RsOXM6LIWYIVoXozrEZuJJ1Qaco5j28X3Xo=; b=ZzI1HopgIwNPiE02OgF7OJ0Ejt4h1ySE1hSxf9xI72O8/a63J9/5v7LMHHR/6Z4D7lx7WdzmSQxyTuFydxRUTl3I2RGXNEci4w5fSl+5oX0OxR+5txt5UaE0Bupmjtp+5OhAfXra70lsCxN+x1vjUnxbA1vsG3NNJIhID5io/uFkh5kJMmoVkaMMlgglgxZzmw0bOEcxOZC1WLe00srQz/DxUBjcVD+YpuJKshKpwIbB0RRdHayIEDm2g8SEv+Sg4epKo7y0IbPUuVc0ukJ2eXH43LaO6VVvlvD5uA6A1Pfu/ttQfUpol2N8+ovj/uMtE0P1IZpcgDlQ7krm8DTErg== 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=9bnM7JK5RsOXM6LIWYIVoXozrEZuJJ1Qaco5j28X3Xo=; b=ErUoOjlkrFVuwIGaUWbe+7HQ0bocvhU8/fdufrhENAo1PuF51fKWpiDZE5RMi+5fy79moRmtWdKybvRgPcmPlndOhNtVLg6arYzGNR3DQSppzwr1G4GtvLIqePXXfics/VYZCfj5gWQd9dKLG/AUC+3/BvwtnWJUKhtjkm7Y+p/hAQA6rjkYy0fVDHknG/602aSRKQJ3IrCadqK8s5UB3TPRL2l+gS5jJwJBOhBmrNJZy4O9jf+lrMvpnynCvE0HmotE3d/92NYnKodLaYPW1+fqvTeXZ9aEbzx6dcL0qIhuesICWnQI3zP9ZI98Jn6cPjmuUiPqKUMqEIel1g7seQ== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by BN0P223MB0014.NAMP223.PROD.OUTLOOK.COM (2603:10b6:408:154::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5164.20; Thu, 14 Apr 2022 21:37:46 +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; Thu, 14 Apr 2022 21:37:46 +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/AIAAuNswgAJ1FQCAAMIPIA== Date: Thu, 14 Apr 2022 21:37:46 +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: [8CKGZgP7Bytn+Z6u10e9qAXoj62yT3IO] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 323f637f-0e69-400f-5b86-08da1e5f0415 x-ms-traffictypediagnostic: BN0P223MB0014:EE_ x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: nyChwF6JtK0WDFzPenXs6fR/GhLbG9nWnB7QEG+wgO/7HEaWxxmlvImJ/qAFgpK7cuK0dr9oG+XWZrm9t0rwfnLQ50fBXX2l72CUZvAhUECFda/iLy0XPPPdCYI1Zy7Etot4lAq+6gG58g4EbJJIX62+BU/aFt11I21PagHBIihWUUMm4SU0mZSJL8HraSVtqWJIUQAILxGL1FgyHA5JpkNbP7etcOSkILuQsZRK8GmruhgA3+v6pv8JOGYJc6QPFrL2GdgCfOXf4gnUURWrIiqc5Lg6EdkAo7CQOMEIvsfpmfS367XZ96aQWUBC2M1tBQnk8lDI8sGu9oWfzOCQHB3RlojatIhEbgrnWGQDrqn42xcBx1sFM2A8ophjtHJcWbNSd9ZowvlszksFUvcOxh5+i5Fi8SWwyRJcG4qQB9RF+s2TkUtkly4J+RuQwsLirFPzPzTF1qcQBHjDHUku7XcAd7D3NibV3agpTih5li6QieDHm0BM4LIm2eUfKZ5paMjaK4ws1X33BAVhxkltvwLGv0N1CGgeF7E/0ksuelqur2jw28T50L3OcLekxthtpkyFGQ5hFgXktrD8lt4v/Q== x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?Y1FWTmcyZFJzVE1oMFIvbERwSkY4T2g4UndneVlxcGlnd1orZUIvcXpEVlda?= =?utf-8?B?b3V1WU1kQkhXY1gyQjR5b0p5TEJnczkxVjZMTEw0cFpXcEQ5M2ZoVHQ5R01C?= =?utf-8?B?MXE4TzNrWkt2bnd4UG8zTkRxdkcvNnBFb2hTaVNaZWdLam55d2d5R2dQRW0y?= =?utf-8?B?NVRoVlRKdmRPNDZXSFJjY2dWZGVWOFVUVWtDUk5SZG9zVS8yVEpSM0h4bms4?= =?utf-8?B?VGxOUWlJQ01FS3BSbnlrSlhNN1V6Q3VXWDFnRnk0SmwyeGNDcU0wMExqYzRX?= =?utf-8?B?NzBIeFZCTTJTKzhRSlJJS0xiNXBXekFaOUJnRWR5WFA4ZVVSQjErTjdHa0pT?= =?utf-8?B?OUFEdDliVW4vVVN4OWR1bk5kMUQ1T2lwUnBMa3lXalYra00wdzA2azFIRzNP?= =?utf-8?B?WkJ1SklscnFyUzZ3KzYwUWMzZHZwZk1IMjdFUkt4QlphTFY1eUt2b1U3WXlZ?= =?utf-8?B?YWlUbVI5bGdPZTkwQnFObHo1OVRLZzNJaFNuYy9hd2gyZjBWYUt3L2ZRYkdx?= =?utf-8?B?V1dmclgwQldZUS8wNE9QeFE1SjhCN1NBc2M4MkVuKzNhRHlNbzNVVVhrRG81?= =?utf-8?B?MmZnZjhTbEIwMVhXZ0lyWlg3ekVwUWRVRGt1akhadjRTMStXYjNSRlBoM2RR?= =?utf-8?B?dFZMNHlmekNNc1hiamNQTEkvdDVLVnJuc2h3UUVXZXc5RGQ3Q3p2am4vKzNi?= =?utf-8?B?YjRUYkpRd29HSHlrL1VZbTg5YmgwWmxhTUFhVXZJLzltRGJKSkI3aGlZeDlt?= =?utf-8?B?UUFPM21LNkJnMWRFYlppeWVlNDZ6RG9MTzZoYVFWdnBtdDdFaHExaEFzUVcr?= =?utf-8?B?UWFBM3pSdVNXanpLeWJKWnpDYUFRNHo4K3MrZzRGd0dzQzBVUUtGSXlYSEpk?= =?utf-8?B?SUNIUkxEVmdvZFk0a2o2dFFXUHJKME43RWZaL0RuZStmaHpCTlhDT21vcWpN?= =?utf-8?B?WWVlYkliRU53M1l6R3kyTFJzTndLR21JZUhWYWpYaXlENHkrclBRakt5YnVk?= =?utf-8?B?TjBCaW1NMUUyYjlXMkRwMGM0bmdXV3JPNjhDS3NUM2xmWExETWhpL3EveUhF?= =?utf-8?B?YTl4NXVWRTRzekVRK0pTa0YzcUxnb01JOTFhcjdRSnZ1aEJPbitBcHBwT2M1?= =?utf-8?B?QmpPallUTFgwbzdqUnhFMEFreHlBMVFPWWpLdE5PTFZ3NHV4NUk5WFpGMm9r?= =?utf-8?B?YVlCR2hRNUJEeUk0cG04OGZJQUcwTTB2WWRDMEtvMmowVVlRNStyaGJwVjVE?= =?utf-8?B?VWhKaG9naG1YUVd5OWNYQlljcW54TjAzQTQ1WURnMXpYTEpZeDkxK2tTRUh1?= =?utf-8?B?NnZjTEQ1WVBtcktCQXNON21IdHVSN21ySUczSzNvU05ITzFsVlVtaW1DdVZO?= =?utf-8?B?S2JBMm05cFFSNkhJTHpnVUV1TE1DbSszekpPRm5tYUtUQi93U3hPanBLUnRV?= =?utf-8?B?ZHJsOTN2M25mODVaMDkzMWdiSXFYeHV3ME9xQ3B0NCtITGRnSlRxVmMyMXBj?= =?utf-8?B?L3pCcFNWSGZ3cC9KVGVMUE9oRWFUSUZqMzBEMGhhei8vZkErWTVpTTAyV0Fo?= =?utf-8?B?b2oxMU5HRmppMjRlb1BxT3hXWEVWQlAwQTBwNTZsUTFWTFZIWm51VFJmbmc1?= =?utf-8?B?czJFUm5KRFlJNEFMKzVKclJqdWMxakZub3dnL1RqN2VBOFZGV0kxamNsYkFQ?= =?utf-8?B?TmRPY0dTcWVJRVNVQmNpOU4yN21rOEF3bVd1L2xabFc5eXF0K1dYMTBTOXBN?= =?utf-8?B?RWVpbzdlVHV0bGVZbitIVy9HN3pFejUzTDEyMTlEWTB4cVprVXZYaWtIMDRS?= =?utf-8?B?L0duTjhjMm0vbEd1aEVHUFRJVHg0Tmt0R21ZNDdlZFlyekk2UHBMSm5VcUhw?= =?utf-8?B?c3Y2NUpsSUdvMXNNR2xPbStxMlNMMDlodEkzdzVtUlNtTnY1SlM3bzBTVFpu?= =?utf-8?Q?dWlqoEQDbiQ=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: 323f637f-0e69-400f-5b86-08da1e5f0415 X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Apr 2022 21:37:46.4487 (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: BN0P223MB0014 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: Thursday, April 14, 2022 12:02 PM > To: FFmpeg development discussions and patches devel@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [RFC] Switching ffmpeg.c to a threaded > architecture > > On Wed, Apr 13, 2022 at 12:43 AM Soft Works > wrote: > > > > > > > > -----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. > > > > Please provide current version of your work via link to repository. > > Changing AVFrame is sign that something is not implemented carefully. Yup, will do shortly, please give me a few days. Thanks, sw _______________________________________________ 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".