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 4D1E742800 for ; Sun, 1 May 2022 05:09:41 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 86E3368B0A4; Sun, 1 May 2022 08:09:39 +0300 (EEST) Received: from NAM11-BN8-obe.outbound.protection.outlook.com (mail-bn8nam11olkn2062.outbound.protection.outlook.com [40.92.20.62]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id DFF5968A2F6 for ; Sun, 1 May 2022 08:09:33 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=UQJ9S4t4FezXYqOWuVdTxqqMLORlo+FEWJlmPJKx0DPpbgsYZXhg6VoSBjPnIGZr69B0BjhOoMD7u9TbG51h74tHz5QmL3TLQE62dcSeZm28l9NooP5qGb3Zz/YEbTRc1pIPlapXMiBcL9dOzqvqMVCph+qilha5F91j0s/8KfIS/isDZ2VhqgmLNpZGr3nZquM8DyOgTu6ZXMwo/akl0OEpyyahXD0Ih+w5pnbcE96y8uppUliPiW20/O4V66exIMsn3zB5D6GkytWxmBZXLu4EWKrU67qFIIFFfxxQR172fz9HWUFu2moloG5Ye0OFFZsxaIXk2SM+oHs69/z+ow== 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=HVmK8NVJVGb/Hrd54QcBhHa/JLNuSfIex+3LZB1JJGo=; b=OaLcqh4lLYDDf5QMoWg2TciO+nh7jRaPEBQcEu7svfosc+O3bhyJsQiNEWfbgkO1WmbdOpbpd3GjWWOxaqM3eAJUnSmq9RNjFFsumm+d3eZ/s8FTZ0woiTzmAvDlp551MZHQh1RVOZyo32+w6acIcTI8vyZ+5g1ceUIlOMXkTwusrrAgrhNNla2eu8JbaFiMT45wOWTzD3nCDCzwBk+EoBX0NDA51qxlfu8gOjLE7iFS3+e1ZVI5TnSdtW7TvumdpFo1NeOa6k1WhoTyR5QxEmEndS7nnzKzgzPVp2lKs0goJPNOZp1Cmud5zhx5n08ZNE9JDNPlNOHRGCs2MWoLig== 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=HVmK8NVJVGb/Hrd54QcBhHa/JLNuSfIex+3LZB1JJGo=; b=edbbywv9wod3eh2ExEOncbxZOJWj26beiTvHbLbrQ1MdOaRhlMKxiWoi113uvrsK3Efc7ZGgZw/JAGAXKJvxqJYBvI76zeGT3B7vj60NVmMqL40fNcKsJxQtluFMVH5Xs0PzzstLjU9ZYagX+aoT34NJ9AkwXbVV7PKWyqy413a0zDwokg4MFF3kLc1YOsnpFcUGOPZS76OBMslZAgUxZsHBs2LcVwVHbiO8spr2NSYx58urpydIIqmzyp471qKmwBfT1XY5NbwHfBOQC5nFd2mneDB5qAT8riFFNbxOBpAz/bcoW17JPLBkN1+BjBdXK6snMk7xOvCF8pcq1L33GA== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by CH0P223MB0057.NAMP223.PROD.OUTLOOK.COM (2603:10b6:610:e9::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5186.14; Sun, 1 May 2022 05:09:31 +0000 Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::7472:6f83:eeb:45e3]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::7472:6f83:eeb:45e3%9]) with mapi id 15.20.5206.013; Sun, 1 May 2022 05:09:31 +0000 From: Soft Works To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [PATCH v2 1/5] avutil/hwcontext_d3d11va: fix the uninitialized texture bindflag Thread-Index: AQHYW7ZG7PhAR+3Ey0KAzg+WBYVY360HcgaAgAEHemCAAPMjgIAAAdfQ Date: Sun, 1 May 2022 05:09:31 +0000 Message-ID: References: <20220429104505.1747-1-tong1.wu@intel.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-tmn: [kbClYkmi42vEZ0CH14nirvolGHZv9oz4] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: c7dd1b69-c1c1-4bfa-d0c9-08da2b30c63c x-ms-traffictypediagnostic: CH0P223MB0057:EE_ x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 8w/3RjaGVl3hS1QB9/nabW3Of8n0DkA3CgygreY4AD8raHgafnSNsh4m6AXAPKillpu5xOwGDvoHbKuJNr8+eChOpB+8Fd0k9mZnVGw3/JdOK/VD96UzTdSfz59InABrxn3nbu96OX34366e/vAI2dFiMKDhIaffXqXIQ+BisS3XaR1kGlwiWxfvzJ6XLnqJ+BLddD0va2YVjPD6oKOVx9pxOVJu/oFCMajo2k9PCF14ZjWaSDhbraTVdil57nsdspK+ctnyYkgcSiJmCPrdDFDZxJ6KDazC69jiyJ9YSzCQbDbZ+gNwQF66QbfnHt3r0v+aV1Za6iIP+atlJeW0HohrECBBsF4Dk5G2BHw5kVYtk4Td+AMfQ7uhbLc9GU/ek4nQW9QPP9t9MTLmcA58VBuncY26iWKG2PD+9ORQMve1FW+pMZvzEOB/2kMaGxCmbQcr9PUEHDLqA+x14rZTHAEbDbI9emfzyBwfM2i0QzL3UZaQRxk3oSpZ4oakVZWkqtEb/aFSKUdWfwQgE7Q5CVhmX22vTp9Qkkt4xQYeOrWYAV9+LZ853oDWmgckeaER8VVWG2ExeGGd202bvqddfg== x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?TU1ha1ppSW1xMGlEaFJyNFdpWXZoODNlVUxRUHVKMUNMU0pESWR0ZVdaSHpz?= =?utf-8?B?VlJmSG84ZTZvd3BEc1dNdCswTnliWnJETUdhV1hSZC9aRitadXhPenV0Q2Z2?= =?utf-8?B?OG0yRjRNaFRER1NocGxiR1E4bmxEWitiSmlTUzErTzEwelhBUHBGQ2RUZkdE?= =?utf-8?B?RGVIYW1jR1NYdUFOVW5mdHFOdlJBdTY0WC9tTmpDcWNjVmtyZ2VQQU92Wmlh?= =?utf-8?B?cUJJZTJUZVJDdWNLWnFIV2U2UFBqdVZ0bFRiaUh3bjNiRmhyeENINktwcFdG?= =?utf-8?B?bU5EV2pXeFFpS3F0Q3F5dS9ReHlDbFNiZEVkb3RWb3lKNnA0d2xDVnRhWnBo?= =?utf-8?B?dkdxd0ptb2hpb21iajdqTGswOEhkalhzWnpRQ0hkcWhEUERNdkZMVllBL1da?= =?utf-8?B?R0ZuZ1VkK3dhN2VLa09MNWh4M0twWUZrODV1MW9tS2Z4WHl1T1JFdncrbHhF?= =?utf-8?B?Q3ZLZGdKbE8xdjFXSlZYMTVsT2hTdmFxMWRsVW82YWxWZWVrdERqdDhCS1ZE?= =?utf-8?B?bWNBUndMNE1jbkFTOWQ1ZktJakFDQ2ErMzRmNXNWRjErUkFKM2ZaTVlsVVRl?= =?utf-8?B?M2dzVHVleS9vaDlQbzI2UkJrY2RIVTNGRHJ2c2NPWXpCeUR5b3RyVENzMzZ6?= =?utf-8?B?WmJrOFRsUWRGYUk1M0dWUUJzcTNzUVp3emVlZjNXR3dxVjZxVzNkL2dTdldm?= =?utf-8?B?dXFFNW82QWN4b3RZT2k0S3k0REtZR3g1dEkzd0tCL0dIREx4eHBheUUxZUJE?= =?utf-8?B?cGs4NmdQWmF4N1B3SEZsTEpoK2NEN2xDWU5wS2cxWGpGRG9FNVZ2UFdHT3Z5?= =?utf-8?B?VXBBUkpBSEFOZDVPdFVmYmdwLy96VURuRjBiV21vZDIyMkhPcEIxS3k5bm5K?= =?utf-8?B?MSswN2xDeXQxaytsU1lFcVZnR3JLT1ptY2lKM3VjV04zRU0wUmVkQ1c5NVh3?= =?utf-8?B?Qk5KTkpJVXdOSk11VXQ4RU1KSEQxaERWbGdkRXFBMEJuMkc4Q1loOHYrRG1E?= =?utf-8?B?TGl0SU05dkwyQzMzRjBUdVpKQnR5UldjMis5MGJQYWpTbThkVXJkVTdUQXB6?= =?utf-8?B?aW9hVUZsdHdJeEI2NWpzMGtqNjkvNU4rSzZDN3RxUTB1SU5tMDFleUkrVEx1?= =?utf-8?B?Y0ZITlBwbTFOb3lPaDlIMlJHSjlkQ2tPa2VLcjFCQkQvNDNNS1J3V2pZRm55?= =?utf-8?B?b1F4T3VWZm9rbXhCNkF2SFNScFNqT200b3h3cG5VYTJpV3NJOFdnVkQ0dDRM?= =?utf-8?B?RFZ3alJ4bU4xUGpOYW5BTjhFcG9yY2FZdFgzbnF6RnlLN3RKeTYyME4reU41?= =?utf-8?B?VHpLL2VqV3FtY1MrRGUvdjVJUlpQSEdzSVJYd2swaWt4b3RBN0VxSXVlU2dl?= =?utf-8?B?SW56b1Q3eEhVd2RKZEpQY1A0cThLMCtCRDh1ZkdBQy9nM0FRZjhTK1BYLzgr?= =?utf-8?B?QjZoVmFLdVFiblYxL0g1Z1lMcjJyaWEwd1B6RUhlYjBKSGNzOG5RemxLTkl1?= =?utf-8?B?aHJ2aldISFZrVSs0dXJ2UWZUUHIzc29DMGw4UXJOcEswSm1TSjZOaGNuYnlX?= =?utf-8?B?VTJGdTQxT0V4THJScit2WWVRR0hMekJTS2kwRkhlcHhvZlo5SXA0RG1zdndX?= =?utf-8?B?ZEw3T3VwcXNlR3RtR2FHUmYwZGdia3JmWGtUUkVTQkczMEVqVVJiYVNGbGtC?= =?utf-8?B?WnpycGJqcUwycWpXV2VVT1ZhQmtyaG5Ed0RYa05zdlRjZlh3ZW9XSlVpRXhR?= =?utf-8?B?S2IwbTBpOFpPSUhUMWI2ZmFhQU1hVWU2eXp0QVFyNjY2Tmhwdy9UUWdvVmNV?= =?utf-8?B?NnpLS3lEWEpNNGVPSEYrZllIbHhLNjVKUndlKytJVlpkZmx2TmtHQkZoTmJ1?= =?utf-8?B?ZlAzVFJDRUNVZnA1b3k2NEpENEFCc09uaDlzajhYbHRyaTgxUUxDbGYrYll4?= =?utf-8?Q?0L5GGwkFAy8=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: c7dd1b69-c1c1-4bfa-d0c9-08da2b30c63c X-MS-Exchange-CrossTenant-originalarrivaltime: 01 May 2022 05:09:31.0177 (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: CH0P223MB0057 Subject: Re: [FFmpeg-devel] [PATCH v2 1/5] avutil/hwcontext_d3d11va: fix the uninitialized texture bindflag 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 > Xiang, Haihao > Sent: Sunday, May 1, 2022 6:15 AM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH v2 1/5] avutil/hwcontext_d3d11va: > fix the uninitialized texture bindflag > > On Sat, 2022-04-30 at 13:59 +0000, Soft Works wrote: > > > -----Original Message----- > > > From: ffmpeg-devel On Behalf Of > > > Hendrik Leppkes > > > Sent: Saturday, April 30, 2022 12:02 AM > > > To: FFmpeg development discussions and patches > > devel@ffmpeg.org> > > > Subject: Re: [FFmpeg-devel] [PATCH v2 1/5] > avutil/hwcontext_d3d11va: > > > fix the uninitialized texture bindflag > > > > > > On Fri, Apr 29, 2022 at 12:45 PM Tong Wu > > > wrote: > > > > > > > > When uploading rawvideos using d3d11va hardware framecontext, > the > > > > > > bindflag > > > > is not initialized and will cause creating texture failure. Now > fix > > > > > > it, > > > > assign it the value of D3D11_BIND_RENDER_TARGET. > > > > > > > > > > As with similar fixes of this nature, this implicit behavior to > fix > > > one particular bug does not seem fitting inside the hwcontext > itself. > > > There can be a large list of usages of the hwcontext that all > require > > > different BindFlags, but we can only define one default - why this > one > > > specifically? > > > > I agree that this change is not ideal. On one side, it is "safe" in > a way > > that a texture is practically unusable for video processing without > having > > at least one of the flags (decoder, encoder or render_target), > > so this wouldn't "hurt" anybody. > > > > > So rather: > > > > > > Where is the context created? > > > > Looking at the command line in the commit message, this is about > > standalone D3D11 context creation. > > > > > Why is a required flag not set there? That would be better, > because > > > that knows what flags it needs. > > > > There doesn't really exist an appropriate "there". I see two options > > > > 1. Add a generic internal device creation parameter to the > dictionary > > in ffmpeg_hw.c like "standalone=1" > > (for all devices created via init_hw_device) > > > > Some time ago, I had another case where I thought this could be > useful. > > Then, this could be used in d3d11va_device_create() to set an > internal > > field 'default_bindflags' which would be used as condition in > > d3d11va_frames_init. The situation would remain similar though, as > that > > when the device is used by a decoder (which sets the decoder flag) > > this needs to override the default. > > > > 2. Use a device parameter specific to the D3D11 > > hwcontextD3D11_BIND_RENDER_TARGET > > > > This would need to be specified in the command line. > > Everything else like in #1 > > > > What do you think? > > There aren't extra parameters for other standalone hwcontext creation. > May we > take BindFlags=0 as the default setting and set texDesc.BindFlags to > D3D11_BIND_RENDER_TARGET directly ? The ffmpeg cli is not the only way how ffmpeg libs are being used. That's why we shouldn't make assumptions which might apply when used in the context of ffmpeg cli. I think that's what Hendrik wanted to point out as far as I understood. Kind regards, 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".