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 2F23242C16 for ; Mon, 1 Aug 2022 02:14:36 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id EC92868BA96; Mon, 1 Aug 2022 05:14:33 +0300 (EEST) Received: from NAM12-BN8-obe.outbound.protection.outlook.com (mail-bn8nam12olkn2035.outbound.protection.outlook.com [40.92.21.35]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 7F92968BA47 for ; Mon, 1 Aug 2022 05:14:27 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=BnqaDPifZFXIc3nX8sb8d7NgQvQdL8dwudnL+bmhgR6Bg7TeSSANcIa085+nwN0Jhx0+SUizjU2g2nlbdLTjTAzVkdl8tdWqsXcqPEPvCop8PFY/yLCh8YNewoqPpLcDUZxXMkvxJWfuZs9NfooO1mort2zeMusEaJgLAguLsA+wwoyS4/YHBKZp1Zr7acGG5dacDIQC8KEHBch2abxqXfZR9BaiBpE9iU8sQCbXqSLRv83+8Ju1896AuwMyiyXwThxoPJ9XLJLAERYv2tfmbhkpMeqmZLC3TLRklmhXItk4UiAMqH9YhXWGfySOP2d5oSV8YNwOZ/h98aphYkMsHg== 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=+Xj8bTRNqoALj/Sg5Nwj0fCZDHmismuENk23LwPorkk=; b=UJLpZfcAc3znUUz7B8QbhLjkKEZVRsgDhVEQyQMcPeYa2CZ54ypd9BdToCYFiMi914exADYZqIKC3/d492UdbLdaEoJG4kKpy1HzHxkXCzT6RgJ/2jghSSNqj+pY0WJgixpej7wbusiiwml/OcIho1XcKsw+2QBc9/dyYdaST6rStsv5Q9AHV+mnHtTg8fq0bZkI+jDgg3Z9m8cFtMEpUmXxXsNoKjLlscL5VdCuKz1TNH8ZrhLINrvAp6+wH2aDEs8uB8vgOj4H6/qLFPzne40SMWxAj7kivy2KSfR86xy02ZE9ITbMAQ6RrY9WCBsEWllXPWYEGr9AlCIWdabdag== 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=+Xj8bTRNqoALj/Sg5Nwj0fCZDHmismuENk23LwPorkk=; b=WYYeDOJdvO37vs9v8LAc09UzU5lUOVZXIdQcLppzBJLYZjk3YuS28S1S8nyAXpzLoeaV7d96u6MX6lMBXRzngKFxgpTzDdr8v8rCGHo2cFgnhbCvZjC3l5icYSHI4THhPLSe0uXWBODDzANIteoDUZWVQXZGxFQM9cRj/khvPUfsfsPP/Ykiyd/5I09xzQctMWRIRiaXhqbJ7JIcPx01Qb6iwY/8atra1qBJoPPCWZLxPgPP/kz0hqFvAe+RC/Lrf67lyqKNLOMQXs7xCrzSNtOw8AjfolNVPRDEbY6/S3YldNMkQond4C3X7e6zmHjWyUiUdBckIuZpvQb17tnqlA== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by IA1P223MB0449.NAMP223.PROD.OUTLOOK.COM (2603:10b6:208:3ef::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5482.16; Mon, 1 Aug 2022 02:14:25 +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.5482.016; Mon, 1 Aug 2022 02:14:24 +0000 From: Soft Works To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [PATCH v3 2/2] ffmpeg_opt: consider HW acceleration method when selecting decoder Thread-Index: AQHYn99rPzCesIz4xk6pALjT7RdzBK2VDT2AgAADgYCAAAp0wIAEN8YAgAABvbA= Date: Mon, 1 Aug 2022 02:14:24 +0000 Message-ID: References: <20220725043051.9692-1-haihao.xiang@intel.com> <20220725043051.9692-2-haihao.xiang@intel.com> <2ab2f6906d871909652d1ca224054c06d02ca4a0.camel@intel.com> In-Reply-To: <2ab2f6906d871909652d1ca224054c06d02ca4a0.camel@intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-tmn: [E2rd1JD6wUMUqen7kSd3dS+vFCblj7Af] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 9138b3d2-a1d7-4add-3e0e-08da73638e1b x-ms-traffictypediagnostic: IA1P223MB0449:EE_ x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: Sx7uIBjKuq+iN6GbmKargtS60VoNkB2oHc/nMZnpg7Hsiqb+gRtT+P+rAh3OcNHbmsJtc74Tv/cxltYg/TcRcyvUWELEMb1C9G1y471BD04Fe9XouqZjpzYloLmyzRJtpQAINWGXv+JHziwJkIXb6V5Bp2NRBJEFCAANXaWjp1tGAA6Qvtw+I03EZDUPmMBkQEHyK9d5LSVuuG0lYL+MUfgX0lf9jvydoQfD07SpgQr4daonF34Y2d3QQd3f19elotPp0Z6qhBv5uIXj7LFIMpcwWTWKig6qd+5Vcgw8zm9YcPOQ3ftvGlJZeoJHogz5+24DXwTyxPnWvu19pyTxQlRwX5SRmHETKzFFOenvRGnIPHSeA9A42H6laY/oCTCLjrl9sXcnw5pc+Qhaj2U6EhgzvxoVA+FICiV3Dp5cRpfaMbk5q7XMCbAa/ClKt1PixFjkP8ThJ/XvsAk4Mlc3go0MrQyKCRkJURz0zigtXcpPP3sORBxX93Iu6u7PIGZ/7/H0FKqfDjalqIqtWNVcV8AJLUhVV4EgJrSAGJPnusEU21mKES3ehCkz3srA+EERtCsr/Swnzvmpq1dh8DIn+sAD6Ip1kYXQLrFlNE0XoGF6HZozG67VhHN+AsW8GS2qoPZNs+nPCd2fe0SDTAFIL0zud4D1K1LP5j1IyCgMZzmcREwNQ2Vl3f+iWUz0Byp6 x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?dmtEL2FNMWttdHpLWnZacS9HUHlLYWlDUjJsQkluRkN2dUpzQzd0RXZDWGdM?= =?utf-8?B?OXdwdzByOGhFN2ViS0ZsVWhSM0JuL1N1RE9QUFphZ2E0K3BvZ2htKzR2aHgw?= =?utf-8?B?Y05ONitCaisvQmQxMmFpcDRkbEEvMHBIMUN0V0RoTDV1QlRsMllwUFFtTGNN?= =?utf-8?B?blBjZEw5TGNsL0ZLZEhJcTcvbDVnSmVyQ1ZOZStUNlUwb2RNbkNvZXBIcXVH?= =?utf-8?B?NDc3SlZZdk1Pelo1SUF6YllTR0V6V0lDYVBxNFhMNVovVnI2NHFESUxyZGxF?= =?utf-8?B?aWF2cEJNeXJPdEpZUjhuMVN0c0h5ZFRQYUF6eHZ2NVBiN1BJMmRlR1RYWG1p?= =?utf-8?B?YzJhVjQ4NlpJTXcrOFIybG4xaW1HQVFUUXVsUjhTRksyQmNHbUtEcFBNNWdP?= =?utf-8?B?M0xFNDc3eS9nU3gya3dGbXo5WTJzQ0lkUGhpa3haNWl6RHRjZmloWFpVbjU0?= =?utf-8?B?dUZ3S09hMi9hRXU4Q0JPQ08rVFRpQ2JVbHZ4bEdiM1E0d1lRY1FUdDB0dTJN?= =?utf-8?B?RzlVTE1OdXd2Q2ZCUExRaUp0WUFFcENKQVFmTXdoRU15c0UvRGxGc2xBK2I2?= =?utf-8?B?Q0VZeHpjK3ZJRVBsZnpCeDJtQ2M3VlJRRDB6QllWTnl6ZGZXa2E4RkJDVjE1?= =?utf-8?B?NE9FYThKS0hkMHFHWHNiVlRpNmd2a0dWL1VEYXd1KytQOXNDTGtZZzZTZzd3?= =?utf-8?B?VUJvYzh5YloyNHdIZFBVY1JZY3htcVNPRFRWMmt0dHJGdWk5N0Vub1EvRzcr?= =?utf-8?B?a2NLbVFCUFhaazE3WTZabjViY0lPNFk2ckRMTHFhVlBUd0g3aXlmNjhZNXBS?= =?utf-8?B?cTZKd0NWNG1FcWk4djJYWnk5VnNkYUwxZUtJWXorWFMrcUpyVjRJanhHN2tI?= =?utf-8?B?dWxuTDZldWdKWE51Z3NmcHJldG5rQy8vUTExSlRvMTBGakZzY3dUSElZSnpD?= =?utf-8?B?RTI4SDZackw0dytVYUk2bndLWFlaVXRQR0YzUzAxSmI4M1JYOXloZGpuTW0y?= =?utf-8?B?MklNSU9iYzdEY3gxQ0hRekRKQVRzZjFMUUdoTENLb1BmUitCb044QTFkWXpD?= =?utf-8?B?cEpoVXc0Uy91OHh4RU1NL08vdTZGYkM2VUZRdVJobzdhUkdlMzdHbEUyREN5?= =?utf-8?B?L3NPWE1zWVhDdG5RRWNSZ2RSc2VXa3FZYXY2Nm44cEZQdWRQV2NxUVBQV3Zq?= =?utf-8?B?dHJuQUJWY2tranZ5OEJrbUZBaEVoUjBCaXM0cTNTb3owU1JnQnNGUHMvNFdX?= =?utf-8?B?d1VGd3dYQzNNNkxuUXdsQWlyL1FnYmlZcndPTXovc0M1N0tiOEpMUWNSU3lK?= =?utf-8?B?V2R3cmE2Y0hPODgxL2JtOE1QZFBSSGNSaGIrb0pKVExwMjh4amtwSFdDUyth?= =?utf-8?B?Qm5mRUNlZVkxRG5HamNQVUdqcWZWTENOSmViK2NWQ29HK2JiMGRPNitTVUJN?= =?utf-8?B?WGl2SDJwVU4rcHYwVUwwMXdkS1F1N1FPaWhseWFwNWlXd0FRUVk5VGM4dnVQ?= =?utf-8?B?WXV3am1Ubmh3K2xBN3N5eGc3dHh1OEhCV2JSZG9FSm55MFZZb2N6Sm1YRHFw?= =?utf-8?B?cVdxV0JzSEkxOExKTkdxRHdWU0VYYlBodStmZXJnZjN0U0Y2cXZjUkNhQit6?= =?utf-8?B?enlzVk1WZU9WRitkWXhjaXM5Uk1qVXREUXlKT1psVHYxZVliTHFXU1d1eTN3?= =?utf-8?B?Ui94bW1DY1BGSE0rU1BjdHNKVEJXckdEUGtFU1BrWGZyT0l3d25VNVh3PT0=?= 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: 9138b3d2-a1d7-4add-3e0e-08da73638e1b X-MS-Exchange-CrossTenant-originalarrivaltime: 01 Aug 2022 02:14:24.9141 (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: IA1P223MB0449 Subject: Re: [FFmpeg-devel] [PATCH v3 2/2] ffmpeg_opt: consider HW acceleration method when selecting decoder 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: Monday, August 1, 2022 3:53 AM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH v3 2/2] ffmpeg_opt: consider HW > acceleration method when selecting decoder > > On Fri, 2022-07-29 at 09:46 +0000, Soft Works wrote: > > > -----Original Message----- > > > From: ffmpeg-devel On Behalf Of > > > Ronald S. Bultje > > > Sent: Friday, July 29, 2022 10:50 AM > > > To: FFmpeg development discussions and patches > > devel@ffmpeg.org> > > > Subject: Re: [FFmpeg-devel] [PATCH v3 2/2] ffmpeg_opt: consider > HW > > > acceleration method when selecting decoder > > > > > > Hi, > > > > > > On Fri, Jul 29, 2022 at 4:38 PM Xiang, Haihao < > > > haihao.xiang-at-intel.com@ffmpeg.org> wrote: > > > > > > > libdav1d is the preferred AV1 decoder in FFmpeg, libdav1d is > always > > > > > > used > > > > when > > > > running the command below even if user expects vaapi or other > HW > > > > acceleration > > > > methods. > > > > > > > > > > I think that is a pretty serious issue. Don't we always want to > > > prefer a hw > > > decoder by default? I agree there should also be ways to force- > select > > > specific hw/sw decoders, but shouldn't hw be the default? > > > > I don't think that this would be reasonably possible in any way. > > There are a lot of questions which ffmpeg cannot answer, e.g.: > > > > - Which hwaccel to choose? > > - Which hwaccel is available? > > - Which parameters are required for selecting a device > > that is working? > > - Is the auto-selected device even capable to decode a certain > > input? > > (pixel format, bit depth, codec profile, frame size, ...) > > > > For the user who is creating the command line, it is important to > > be able to rely on what is going to happen. If they can't command > > lines will fail: > > > > - The outputs of hw decoders vary. Some output to hw format, some > > to sw format by default. You may need to specify > hwaccel_output_format > > or use hwdownload > > - Depending on the hwaccel, a totally different set of filters > > may be required (not to speak of encoders) > > - How would you specify that you want a sw decoder instead? > > > > IMO, predictability is the topmost important behavioral aspect > > for users (be it humans of machines.. ;-) > > > I agree with you it would be better not to make hw the default if > user doesn't > ask for. However '-hwaccel arg' is used to specify hw acceleration > method in the > examples. According to doc ( > https://github.com/FFmpeg/FFmpeg/blob/master/doc/ffmpeg.texi#L1260- > L1262), a hwdecoder is expected. > > "@item -hwaccel[:@var{stream_specifier}] @var{hwaccel} > (@emph{input,per-stream}) > Use hardware acceleration to decode the matching stream(s). " > > This patch is to fix the issue that a SW decoder is chosen even if > user is > expecting a hw method, not to make hw the default in any case. > > Thanks > Haihao > Oh, I'm in no way opposed to your patch, I just talked about the idea of a general automatic selection of hardware decoders. I agree that the behavior for AV1 should be the same as for the other decoders. Best wishes, 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".