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 F1F9D440E9 for ; Thu, 24 Nov 2022 19:04:54 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 02F1C68B7D4; Thu, 24 Nov 2022 21:04:52 +0200 (EET) Received: from NAM11-BN8-obe.outbound.protection.outlook.com (mail-bn8nam11olkn2010.outbound.protection.outlook.com [40.92.20.10]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id B732A68803C for ; Thu, 24 Nov 2022 21:04:45 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=dEa/PPRikpu4vZOENwwc6Y+AsNxOsR9jjyWOGefMcIqe6jpZje7qWLthVkCcvUshw4L5OBNln/CRNEoNwdGd/WkmOtI+StD15vTLHGjXM4dAi0Vd9xxTbj36O9P8IVGX8uP/rJe908C4glxk1oK8JgD86meXsvqgR8tOn+MxgbR0xOCzBEx+HyRFNOA/TZja1Q4tuFseJGUhr/jBaZrzYZEq5kGIq4MYHwrpUU+UMqBwNX90+YHOLMUZtBVI3KADjxlzf8cIh9RE02Tti+5RARaXihNFQebu+w5Vi6E85BU/f+iS5RrYZuV9D8DrWNZSj4b4Izevm5t5ORNMF3j6PA== 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=md8yVBEMwgOXaizlE8of0hqy37JsnTFttoIyjFu8uxA=; b=gl/XQnMRfaXKopeggT89BCQ1Xx+1MdwHb7mP92YiVJUHvCutxYnx4GawgtANNMCtYU/jK5+qaFt2ujHDrJ9pprx2ojWXXXa3SfaPrpBS9pKxe9NJ26l+d0DuDTItIyA9TTzivyXDOKsaXb87/4sah725kbamLxqC5hsu4MR98ov9MsWuwXhdJD4CZRXtxnZO5LTCtmZ0NSbkxwkWjyChekyFkkJGk2l29ATBTzIe/4hd0usW0bBpDFsN4XaKwwKgJj8pMAenKd/j5LaUnm4zTsrroUanHBj8czyo8xkKd1cKp/l9NVzhyJZZphDLZM4JSNGUIxcKb+FnY8qALyvXZg== 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=md8yVBEMwgOXaizlE8of0hqy37JsnTFttoIyjFu8uxA=; b=ONLxbpE2lZYsAuF7X8Qp20eDhjEY8fufi19UhFqPhicWFzsGK4c9Mr0k/kjaBVmD4ecUxiGCi5FvqVgTAI9WMqQsgsI0fYKsKMufCVbX2Pn6qcKdj6qgAE80JYiuCJPc39Wrqb7vll1KE7HdSqTT74LJ1y/Hc6FDsxQi5PO7JUlVLJDlXvZ7BeOWGV9lIGmtJXWvzI0ko+W7SKdWblsppDnZTGIPfZYCBUcXY3Yz8VDPJSD820uE8Ogphj029xvokk1Tli39CrH5aFIAW1EKvlvw8lhGkGeNdlx9iEZQHwW6Z4oewxNMk38KXw9DuZEj8JOacGzhyNevi2hpVMWlRQ== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by SJ0P223MB0661.NAMP223.PROD.OUTLOOK.COM (2603:10b6:a03:47a::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5857.19; Thu, 24 Nov 2022 19:04:43 +0000 Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::45ea:be80:31df:38c0]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::45ea:be80:31df:38c0%9]) with mapi id 15.20.5857.019; Thu, 24 Nov 2022 19:04:42 +0000 From: Soft Works To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [PATCH 3/4] lavfi/qsvvpp: provide a default framerate if needed Thread-Index: AQHY/+X/jI2Ni3OvxEe/97iLqhKT/a5ObwOQ Date: Thu, 24 Nov 2022 19:04:42 +0000 Message-ID: References: <20221124091930.558046-1-haihao.xiang@intel.com> <20221124091930.558046-3-haihao.xiang@intel.com> In-Reply-To: <20221124091930.558046-3-haihao.xiang@intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-tmn: [9Tu+RH/bVjvG/I7qzmPSg03Zl0N86NZA] x-ms-publictraffictype: Email x-ms-traffictypediagnostic: DM8P223MB0365:EE_|SJ0P223MB0661:EE_ x-ms-office365-filtering-correlation-id: 6e2cd30b-ef30-41ec-3c8c-08dace4ebebd x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: W55OR1aQ5fLlxQM83gLE+kAgnDNNw0swSFExfLcE7zaJnmHV9BfiTp72hvXNWMk1da2fdetB8dNeicJ+hPvoX3SPF2nXDNYEL6qko7S6CvFo6TVnJ4EB1YTvyLD+wDjFYg4drUixqV2UdmDXfUoFbYEnh5l0bHkcxEWUeMWowHcJPjwfkDUAYIOQUMgQHoi8mQoQnOJwDt0clp2NJy3wThD5q1IdZs46yM4Wqe6NuxBpc/aU90MRgjJhLbRs48GJfZfOsnjBN4JX19GoU1sFGYiWniMcloeXNywm2EDgqykHcdAt2ARxW/aZcu2sUlyyr+35ux5vrcpNlBY+dbUc1ka3QGU+KJsQ4zfy20QgjFHH/7xyg2+1AMw2lL5z3XjXISKeSU+WECR6MgUXOVcJZbJLZikcmyQXnMXFDq8pmpBY9wb2LPPZ63FQlmpBEc8tuTmyujHPesf2iX8mny/8jhJhidfRmKMNa0VHCp1mSyDlNXk7+KGIpJTvrgA4AC97b1gNPZAdqBe6rqwqOTd6jIvg1U+INl8WQnWGVJbMrZHSbhlux7mlAWoxYaIdQ+PeffC/XwBxQV0E1BPsxrAFrH/Bm0Tu6ejwnMukmdvtOU+pOgBa3mofQIw4ogZUBe1IF1ByF5jaeg0Z/OE2emrHJgBg3rP8Ci3Ck2U1YT+6Hr4= x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?VnhaSUtUU0ZKbVZvU3ZVb0srOEp1NEVKcDdIa1M0TzlGazBmUjNQaURhMXYv?= =?utf-8?B?ZTFnWm1HaGJjdkZxOTU4cjlJM2c2U2tobXlBdGhlblBrWEdxVFd0TXNtOTNL?= =?utf-8?B?aXZOTC9BMGdzNVJabGE2bFREV293eEQ3bUljRlpkd3V5WWxtRkU1K28xWUxE?= =?utf-8?B?SjJsL2UrdTdjY1dqT2gzV0hvam5NOUFwempBNHo2c1JybzI2YzlUTDFqY0xV?= =?utf-8?B?YXNFdDFoT01zbEtxdGZQdml2UjZiYXMzZlN5Qm1xQS9VZkZ0MmRrOGhPOGRT?= =?utf-8?B?b1BBMDIvc05pTDFsRWVIQndsWDlrQ2tFZE54emp0RmpKU1YyZVorN3l4NlFq?= =?utf-8?B?MXBZNDZaZlpqZCtaTVo0c1lLNlJ4UmtiZkJMMVRVU0tNZzJMM2ExV2dDZTVE?= =?utf-8?B?M0dqSm5WVGc4SlRiQTJpU1FJb1kvR1RYMlZ4NXp4ZXBFUUt6dVcwVnR0Vy91?= =?utf-8?B?by9WbGFTT1N1RlNDMS90eUtNR2JiQTNVdUs5NmFYbTk4ZGZidjVTVXQxSzdo?= =?utf-8?B?R05rd1Vjb0ZxUzc5SnROc1RPMXRoSWRaQ1ZNOUpoTzZaS3FKYkdBOVU2b20v?= =?utf-8?B?dmV6VjYvVXhRamZxMlk1UFlNb0FhNDd1OUE2bnhrQkNqOHI3SkxtdzlqbGNr?= =?utf-8?B?YjZ2RGFZblVjR3lhdjdjazZnVmw0MHN3aU9PK2xrVjQwOFg5Q0pJZ01sWUJT?= =?utf-8?B?NjhXRExnK0ZQVUpLZmdHZGZNbVA3QmxiMmFzSVM5TWNoL2JrWElrT3FwNlJW?= =?utf-8?B?MFAzWUhVNkFFeDJORXBwRmFUOUdhTkpGdVYxeHR2TDZrVTVHcENxTVEraG1s?= =?utf-8?B?MDQ5QTVpSDFza2ZxbXdKdGtFWWMxUzRZRDNUSi9kZEZ3VDRiSGVQdmdoenFL?= =?utf-8?B?eGJpaFlNTXNVclAwTWZpb0Y0SVhsTFdUa3F5RlMxcHVMbHFpZ1hxQnNWRElz?= =?utf-8?B?VkluSDJyMG41aEJCY0VybEpKVSthRk51MjkwS0g3eW5WQXcxWWVsU1c4N0pH?= =?utf-8?B?SXVhWU9LUENDZGJhQ1czYlg4bkUzL3UvcFpEdTlZYmpvZjdqTzArdzJLYjFp?= =?utf-8?B?N2tqeFhmWFdmbmN2MmlXaytpZk9Xb3QzaFdpL0ozQW13MkxiL3EzYWVVd3k4?= =?utf-8?B?cE1DMEFVSFlyaDkrZDBkblRnMzArQlhET0p1R1A1Y2VVVUh1NGZoU05abks0?= =?utf-8?B?RG04R09ybzl5T2Znalh0TWtYZDlXa2hUam5TeVNwQXNCYTBzOUdRM1I4Z2FK?= =?utf-8?B?c1p5SWc5VDB6V1pLSUZMMjlFQ2lySmROblZ0cnVSNnprUTdqYjkrOG9OZGlr?= =?utf-8?B?N2xoMlNGSmNDV0NFZjZISTB1Z2IzR2g2SFZXZURxTjBQZzNFL1BzUGN1YnZ5?= =?utf-8?B?dU9BNmo5R3pqeTgrc3o3T3ZyM1RFK1VsaVdyWTVzOFlUbFZ2b0oxcW9mWHNu?= =?utf-8?B?SlNoNnFvdUFlWmcwSDZDTjFlNXg5UU5ObnY2akplZDl2OTVjL1VOYVFqRC9X?= =?utf-8?B?bXc1QTlNYzVubEhCdjYyZ3BSNTloK01ucjQwNWhsYnErVE1nK21XMUhVTWp6?= =?utf-8?B?OUxIYms0TU85bmdralNoVkpRMHFiQ3BMVS84ek93VTBDZlRqMWN2WFFJOWFR?= =?utf-8?B?U1ZuUUFocEg0bllDekVnOXJSNXNsWWhkdk1RWE1ZSDNORDNVTVZvQW91ZThl?= =?utf-8?B?U0xDV1VmVzA3dnRPL0g5Z24wN3pRS3NmdUxvaXR6MERwZDhVRkVsWDhBPT0=?= 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: 6e2cd30b-ef30-41ec-3c8c-08dace4ebebd X-MS-Exchange-CrossTenant-originalarrivaltime: 24 Nov 2022 19:04:42.8721 (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: SJ0P223MB0661 Subject: Re: [FFmpeg-devel] [PATCH 3/4] lavfi/qsvvpp: provide a default framerate if needed 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 Cc: Haihao Xiang 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: Thursday, November 24, 2022 10:19 AM > To: ffmpeg-devel@ffmpeg.org > Cc: Haihao Xiang > Subject: [FFmpeg-devel] [PATCH 3/4] lavfi/qsvvpp: provide a default > framerate if needed > > From: Haihao Xiang > > VPP in the SDK requires the frame rate to be set to a valid value, > otherwise init will fail, so always set a default framerate when the > input link doesn't have a valid framerate. > > Signed-off-by: Haihao Xiang > --- > libavfilter/qsvvpp.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/libavfilter/qsvvpp.c b/libavfilter/qsvvpp.c > index a088f6b61f..a588a37610 100644 > --- a/libavfilter/qsvvpp.c > +++ b/libavfilter/qsvvpp.c > @@ -324,6 +324,14 @@ static int fill_frameinfo_by_link(mfxFrameInfo > *frameinfo, AVFilterLink *link) > frameinfo->CropH = link->h; > frameinfo->FrameRateExtN = link->frame_rate.num; > frameinfo->FrameRateExtD = link->frame_rate.den; > + > + /* Apparently VPP in the SDK requires the frame rate to be set > to some value, otherwise > + * init will fail */ > + if (frameinfo->FrameRateExtD == 0 || frameinfo->FrameRateExtN == > 0) { > + frameinfo->FrameRateExtN = 25; > + frameinfo->FrameRateExtD = 1; > + } > + > frameinfo->AspectRatioW = link->sample_aspect_ratio.num ? > link->sample_aspect_ratio.num : 1; > frameinfo->AspectRatioH = link->sample_aspect_ratio.den ? > link->sample_aspect_ratio.den : 1; > > -- LGTM. I have this in place for about a year. 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".