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 D5CE6429A9 for ; Sat, 9 Jul 2022 00:28:24 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 7F60B68BA5A; Sat, 9 Jul 2022 03:28:21 +0300 (EEST) Received: from NAM11-CO1-obe.outbound.protection.outlook.com (mail-co1nam11olkn2090.outbound.protection.outlook.com [40.92.18.90]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 30D7468BA40 for ; Sat, 9 Jul 2022 03:28:14 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=VQIiuLGca0w30PJhMQKFBl1WvFv4q1eE8peGMZu9EiOexi4e9HEUX/aUk1iYYxv2IRvcVGTO5SDNHh1CHzL2yB/ulTXKoOGxdyl7yR+KKnlh/gygVeP1kzyBqbVurd39rMiRAK/ctm9QWjaXJ2uxuTNrQDtjE487StuAcz8xnc+fGP1JOfoMhsWflQmehDp+wyS43wzYdFy3Xq48RIV9AKdqWzObduJhehLC0JEIB08UQam23Px7iCtCUMgGYD/V0pzVjsAtvMZR4ldLC8EbMOGWPyeoXXg2Ijnb/pCoZn9yS9CMO8IZ6MRY3YG8S1XLBc1CeGE6cRWcPiSET4ZnTg== 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=tRNIVJVm4okCCrASW6qRSRu6bRbS2r1nxPXDbPy29zQ=; b=Yyucepd2Dj6JN+0Xt/YbPeMYw4Qg4tpmqgKsu+pVK6oD8I0xtERu2TADIuP064trwtuu1fqY5F0WfQR5q0XMF+W9hWYlfgY1GRhVCvgR9P305O4rR0ou9sFvlglCx4N7MkbsrTK1gOJuLrHFYRIQwznFn+tPVpy7Bh2AezgAHufWzHCFpgBA54esrF3Pxn/Qi+lgckRE6rMqFVK76EhQb8Iekkr+BXrObNmVym9HhUr3ej4wSHzBX+ceFi2HuRxwj4BzbMFEuGIxL8X2ZgxrLNUcPgXabMYkNxBp0IWQvz9RJYLBb9dBJ8P6wP+Mx9KtQ651NWLRrbX6xDkdK2vadA== 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=tRNIVJVm4okCCrASW6qRSRu6bRbS2r1nxPXDbPy29zQ=; b=CO7zTN/v3OOkn+IheqUy5SajK3Bfei/1lPkIoAV9jgeezKBoeDjgaSlgpYaulExYgxvxSwNccBymHob4CUiFUBD5GiI/jcasWI+2k99tal6od0ad/hlqLX+l22MUKt3YR86gWHeGzDnM/v6thwIFFNHAxope4iwsuPHgWlWiG4mOf4hgnYeK7KbSf6I2TOePOIalbc7IoyLkkqiQnQqzjgm+onx4UYQDsNqiaITEcLckiLyN1wZVkMiEsLLDW8ztU26Fzt7p3wsQAdTWAg6MLXM5EqddDrvodWrxHbFuAD8jtwCjbySyE8Pq38hOG73YbXPLPoozF1VEso1zpxfHDA== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by DM4P223MB0447.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b0::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5417.20; Sat, 9 Jul 2022 00:28:10 +0000 Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::d9e4:ced6:ab31:c231]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::d9e4:ced6:ab31:c231%2]) with mapi id 15.20.5417.020; Sat, 9 Jul 2022 00:28:10 +0000 From: Soft Works To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [PATCH v2 0/8] ddagrab source filter, lavfi.c wrapped_avframe and dependent changes Thread-Index: AQHYkx3HI8iZqnwCekKc85fJzHEcoq11IuAwgAABkwCAAAFyIIAABcWAgAACVpA= Date: Sat, 9 Jul 2022 00:28:10 +0000 Message-ID: References: <20220708225404.23748-1-timo@rothenpieler.org> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-tmn: [mTIN28GNMWsFSpH3WPsSyFydsda1Rera] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 2b93aecb-636b-45b8-bbb3-08da6141e6fa x-ms-traffictypediagnostic: DM4P223MB0447:EE_ x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: Q/RiEqJDgXWSx0WSWdbK0kE903RpTxFS47O07CS29IVeGpvWRidm6PHraA/r3QulAIqnj9wUpADDk17uq3glxsEplfR0mG+n3A1O9wnWgNCWJC3aiKs95tgiaEnD57gtwb9NmmwelzXcAVHfrVliy4xYEXBu0Rk7AVzGCciwmM6FrP8XxEAbon4y1qXmk6SgvgwU0vXcvmDxUVsDueNOPPzVY8rxaA63NdFN9FJo42nnpMnin6+DfIEAvi+LT+P3T9BRoq+lRfYtYjsmeRBoteAR0EkqALFIPseOc8I/cyCnXmpJz+aaTGxBVlbwi435V+WbUO34JZLJpExwVwGID28oopL6R5IUCnnTOAQHcjEKDPKJ+mn/k5bntg8wE3XJ28tbUwmTnzpaqcAhCPCXlFgk1ZWqvwVyuVym8ykuVMfr4fTkRs1cyvd5yTldcoDJvJyv8yVRD5io/xWpu4C9TQ+mFOLs6E8pHKfXVIIVcdKCQf304Jo8eRvSD4kPUhBkIRwEE+ZRT0PJe0cixaH7UyOSTEuEHiXcEiNV0MCTQ3vRFKKCF7pb+3fBQUvmOavZBtuNXq4ZK1UgQ5Ha7gDjFDMrQzzs+Pp/oyuAl+WYQ0IFQ4XDgWy1AXueCyyhYC84bpcBgRw9d/PihowFY2gGEYRX8hQZNf7cHMIn3oBLFGup6rImhAOMX2Lpbua5lTJfjonKgyKJpZKGhA3V5hn7SaK1kLwzfi/kMNMxjMIkGsA= x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?N1R0c0tsV0VzNFhaTlZnMy9OK3ZYKy9qbWd6cU14OS93a3luNCt4RVMyd2Vi?= =?utf-8?B?d2xWczVVY3lrOHZrbUdSTGNEOUd6dGRNR21zd2lwN2R6ODBqQzBLV2hSVzVE?= =?utf-8?B?SS90SjR6VlQvSVJ6b1hZSXdweFoxbFZob0RXcDFVcVVHQkNjd25LbnNCMDFJ?= =?utf-8?B?MUNiRG5nQm5IYzJSOEhoZjFTUXVhdEFjS3RhaUwyYjRoYXZIbGpPMXkxTCtk?= =?utf-8?B?SmFuYzZ6MVlVTnRudnM4dlI5U1ZKdm5uVFJha2QxTnQvQm5BUjY5ZjJObFVX?= =?utf-8?B?bFV1UFNOd2JjSnVFbWxKNExhR2l2WDM3bkZvZWlSRC90K0R4UlVMQW1tT0to?= =?utf-8?B?SXFkV29hOXczL200ekJBRDZpdU5XYWZTOGxTMCtMMUlML0I2UDJwWmdXdmc0?= =?utf-8?B?cW5UU1U2K1d1M1E4YlNxdzRLbVZPSnpRd2R6Y0FySS95bUtENW81V1N3c0lV?= =?utf-8?B?YjNFalFWQjIyMjVFcHdTS09FZ1crdVdvbktYOTB5YzFCSVd4QWp1NTZ0NTNM?= =?utf-8?B?eThJdXgvaFNhMnZxdnhKZk84RXJaZGpvMXE2S1o4d0pPSktCb2t5QVZ2dEg5?= =?utf-8?B?SDZmc2thQVczbXI3T3ZQbjlhNnZOWnZtUkVrbkpsM1dud3daZEJYMFNrWFRu?= =?utf-8?B?a3cyUEY0UDJQUEh2ZTVaeDY2NTJpZlp5Qmg2L2QzcU9sL1FKNVVsL21acU5y?= =?utf-8?B?K2labmlhRVl4L2F5K3ZqVUdhVVE3L0xTMzdNTkUwR0F0TWRoNDhpWlRGckc2?= =?utf-8?B?T2tHYytEbDVLNldrbllpYXhPQW9Ca21IbjhWYXQ2cVRvdjdsbldIVk1KOVhr?= =?utf-8?B?ZXBOQ3ZJSjVXWDM0TGhQNG5IVUJjeGhEQjRrWGIrUzVDOGZLcDhVc0dyTGc0?= =?utf-8?B?WFRieHlBZUJxMlpXZ2prVEhiMjR5R2hGWmtnQUxteGVaekNIM1BXaDNvcXdr?= =?utf-8?B?dGxzdmFFaFhFRGZGcDlTMHM2UEJORDAybEQ1Ri9qVG1yaVJDWGRia3ZKU2Z5?= =?utf-8?B?SGpXK1o1YkJVMGRWSWx4YUxSWkJoRFRQd3VvSDgyOFRXNHEzUWNaSnlubGlH?= =?utf-8?B?M0RSMWZsV0J3bnI3M1VRUDdaOGxtRnZYenU3UHd1TURUN2JEb0NHRHBSRXM0?= =?utf-8?B?cERBNEJHbHF6cE02ZEVBU3U3aTB5QWE0WUIvRjA1S0d0SjRMcFZnWXl5T2Zt?= =?utf-8?B?MUgyekowUnQyVEo4cWRUNnQ4RUphQ2NHY21kd2lEYkFUS2kyUVA3a2NkRXIz?= =?utf-8?B?VW92VkdJNGozWFFuV2haeU5PVTZsTitERFlaVTVOY2swOFI1aFRzL0hYT3FT?= =?utf-8?B?WnFtaUJ6ZGVNWkw5VEYvbm5TVGRqZ2JOUGluRjVZS0J6RCswaVI2d2Fzcm5M?= =?utf-8?B?ODJzZXFWZk4xUDVnR255U0U5SzFabk1heTZyN1g4TW5iVDZkRFQyd2F3aCsw?= =?utf-8?B?QU1jb2ZnUndkc1lFenc2N1JTZGFGUEN0SHVkVk54VkFka0MwQ2ZoYXRNVjZU?= =?utf-8?B?aDJHb21pRTZSTU02UUV5Nk14cUhWcjhzZFdIVVdBbnZKaGlpZVBIL2JpVzdM?= =?utf-8?B?RGtrQU1JeTNVMnEwS3JyY0ZMZFlVVmRYaS9UeUpHYkxPQ0dBZ2k2QnNuclM3?= =?utf-8?B?T085WG0rVWJ0cy9ReXdaeHVhYzQ1bDhBekZwNHluSUtleUx5ZkM5SHdnQWFm?= =?utf-8?B?alpucmU1QkVieDV3RFBTL0ZMcjRpdlFoNlZwWFhYc0pRWGhTbFk0WWZBPT0=?= 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: 2b93aecb-636b-45b8-bbb3-08da6141e6fa X-MS-Exchange-CrossTenant-originalarrivaltime: 09 Jul 2022 00:28:10.1837 (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: DM4P223MB0447 Subject: Re: [FFmpeg-devel] [PATCH v2 0/8] ddagrab source filter, lavfi.c wrapped_avframe and dependent changes 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 > Timo Rothenpieler > Sent: Saturday, July 9, 2022 2:15 AM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH v2 0/8] ddagrab source filter, > lavfi.c wrapped_avframe and dependent changes > > On 09.07.2022 02:01, Soft Works wrote: > > I can submit the missing bit of differences as a patch. I thought > > it was no longer needed. The requirements have also changed over > > MSDK versions. The 8bit file mapping that I've shown recently > exists > > in fact because an earlier MSDK version was requesting such > texture. > > > > Anyway, when something is causing trouble, then it should be fixed > > in hwcontext_qsv. > > Settings those flags correctly isn't at all a bad idea though, and > not > fixing any issue I actively experienced. Just an oversight I noticed > while reading the code. > > Fixing the heap overflow is the major issue and can really only be > done > in the d3d11 hwcontext. > Look at the texture_flags array. > It only ever gets initialized to the size of the initial pool size. > With a non-fixed-size pool, that grows over time, that obviously is > an > issue, and will trash whatever comes after the hwcontext on the heap > whenever more frames are requested than initially allocated. > With a initial size of 0, that is... immediately. > > > An entire other issue this does not address at all, but which also > does > not cause any memory corruption at least: > > QSV can't properly deal with a non-fixed-size pool. > What happens if more fresh frames are allocated after the qsv hwctx > has > been derived? > From the looks of it, it iterates and maps all the textures only > once > at init. There's an allocator pattern which filters are using, so when using non-array textures it might be possible to have bigger pools than textures allocated. But that's just a "might". Please give me one or two days to respond with a better and more comprehensive answer. Right now, I'm deeply drowned into something else ;-) 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".