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 0ABC340F46 for ; Mon, 14 Mar 2022 02:15:42 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 8DA5168B178; Mon, 14 Mar 2022 04:15:39 +0200 (EET) Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 821C768A905 for ; Mon, 14 Mar 2022 04:15:32 +0200 (EET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1647224137; x=1678760137; h=from:to:subject:date:message-id:references:in-reply-to: content-id:content-transfer-encoding:mime-version; bh=jlotoi/oW0jKuUNmFK7sZYJgBcX2YusG584cmzx3R0k=; b=FqQnl5dJb687hvaPj4gNOepJgjXTpnKR4gtxY66TXQDMlSCdbLVe8fc2 uD/MH4Xm5ybJTRwDDzaEVusrg5abCDIh9DSFLbOMQCxWy3cXoj1qW9R6n 0OmzQXgwNxRIyPq+itNuEpGxUnts6ldptsYxpmm/bZlI37oIQxOFl6X7p KApbrpE6ucgXGGfXsV+6AptLN9M5Gvdf4IFQFpy9a7khzzFehKKhbvVVk 4U2ONJFQtqb3MQxe+n0rYf4TjwxikUGzYAEV+32xKlrkJZDqHM5yf8j5C 9lfhC/j3Uv8smN9PhVtH+jh9YFTMFr3wqTs2lxxA2bjIq8lexa1Uei0DK A==; X-IronPort-AV: E=McAfee;i="6200,9189,10285"; a="319145685" X-IronPort-AV: E=Sophos;i="5.90,179,1643702400"; d="scan'208";a="319145685" Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 Mar 2022 19:15:29 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.90,179,1643702400"; d="scan'208";a="818563305" Received: from fmsmsx602.amr.corp.intel.com ([10.18.126.82]) by fmsmga005.fm.intel.com with ESMTP; 13 Mar 2022 19:15:29 -0700 Received: from fmsmsx606.amr.corp.intel.com (10.18.126.86) by fmsmsx602.amr.corp.intel.com (10.18.126.82) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.21; Sun, 13 Mar 2022 19:15:28 -0700 Received: from fmsedg602.ED.cps.intel.com (10.1.192.136) by fmsmsx606.amr.corp.intel.com (10.18.126.86) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.21 via Frontend Transport; Sun, 13 Mar 2022 19:15:28 -0700 Received: from NAM04-MW2-obe.outbound.protection.outlook.com (104.47.73.175) by edgegateway.intel.com (192.55.55.71) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2308.21; Sun, 13 Mar 2022 19:15:28 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=oZCL8QVhFkBdfMx6z7G9Oa9s7qDMBfCIWV49qq1oW2h7LFFKucVT5cypfGCCgRmXBe05tXGjapcZgP95Q43TI5ED/ZF9o85Kk84Zq4yoNVWLQ272EwH5dWBu/PXLXZ+I1jDYRcp+fttzornI4dN6ir2VrOKClPKTd2F6bGrbSI60VVDfY5560bTcDAgBlccgcp3boH6iHZnJnfCMKgeU1HOFuoLMQtBmZJNX9Sg+9iT4G8TjUuVDwTVq2TYhPXMXy8NZrHZrQbpGVL914i9Oq7U/0gpTHZ+JHSrUciFYCw9whBlBPeJvsCtuZ/ByTeGBor+FBAoX/xzAo6C7AIs/5g== 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=jlotoi/oW0jKuUNmFK7sZYJgBcX2YusG584cmzx3R0k=; b=aKApXyyY5x4jDcs0rTxrShXvH4qI5c+e5iEdjKw8F6KOsRmIvPuMVjLuxrr72fRANfmsFCKiI9qroqSUWRXkKLOh0mDQdmeqKLZUpDHPBxolklD8ZxCz4sOxXSzyiv7vVK7GaOk2FJ2PQHXjEFtV+ATdqpolMv/+xHnK88qZ3md/qo6krEQ66rqTtAdeuAlSgSKS9bWIGAhgOXntvWU2r7mDuOIj94/D/BfS0rK8cWWYYrAVRgOb9BdzQroWWh93rMmkyfkQweu9TbF/k7x4OetdnmYEfyb7fNlVRGKaJJ/o0dWMUG7PmVG7NaDtcq5x/Gv7XI5XaPSsQYzFg0oAkg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none Received: from BN9PR11MB5515.namprd11.prod.outlook.com (2603:10b6:408:104::8) by CH2PR11MB4391.namprd11.prod.outlook.com (2603:10b6:610:3b::24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5061.21; Mon, 14 Mar 2022 02:15:26 +0000 Received: from BN9PR11MB5515.namprd11.prod.outlook.com ([fe80::7101:6a5c:456c:9fa6]) by BN9PR11MB5515.namprd11.prod.outlook.com ([fe80::7101:6a5c:456c:9fa6%3]) with mapi id 15.20.5061.028; Mon, 14 Mar 2022 02:15:26 +0000 From: "Xiang, Haihao" To: "ffmpeg-devel@ffmpeg.org" Thread-Topic: [FFmpeg-devel] [PATCH v4 1/4] lavc/vaapi_encode_h265: Add GPB frame support for hevc_vaapi Thread-Index: AQHYNSeehFGaue9sDU2z6NtvmoVevay9zFCAgABcGgA= Date: Mon, 14 Mar 2022 02:15:26 +0000 Message-ID: References: <20220311090046.262777-1-fei.w.wang@intel.com> In-Reply-To: Accept-Language: en-AS, zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-mailer: Evolution 3.28.5-0ubuntu0.18.04.2 authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 3c7e1d69-268b-42ed-aca8-08da056080f5 x-ms-traffictypediagnostic: CH2PR11MB4391:EE_ x-microsoft-antispam-prvs: x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: oRHJQEcY2RReQrzMkAwaibLg4nKiRtaDdXWi7FtS1jpO9u8Y+sBBc3KZX/4/j8V6f9vcBVWrsjicNGj3+slXy1kDORLXsBWl36XGEFt7fXkfmw5dCrK1j7MTTNlodu2LPZVehvcccnpsVlbLfqepWki7DAxDyOQ6JQ8iExbsvrtFoFSgtd8avIwuwvRMMdECJHMTfy1TicCJhgDzrWOXOhqQxjorZdydKJAC6JhjLr6eRTM/fh2FxPzmUTV8le2PFYmq+uFoGXjZAEqaiT31J8nv9Sb55edEOD463lzUqZmbPz/G5CmhKfpad6JCOu6tYh6EZEN4oWZcLYoes9e6FOW4HjErqtDJ188iyPerA4mUmcF8OIlFq0rjwkcAnx2NDgnA9uf5ZuFV9QAuMoSwypDE2v/R4fNGgcGnJKUAIlcAVJV33zO8H/JNpep4Iy62Vf9cBZ/CjrB9T3VUyIDRYhNlgOiolCTmaVlcq7FZvg9dsW5LyLsq+4sR5gP3+royU8O7GLwCFBRdZ7uX6Jliu29e/qQX6YxTMk7fpCYK3HtK4Ptf3srOx8NgKrTh5E/Zm4Ygh70lJbH+p/IA71D+9nh45886NMABxomVK3D9B3Thu6hs7BpZUps/9DuDqILXfOmHqGGWntIwH6pQuOWWRgcB0iix00/IrDCqGvbiDRpJ+bQ/O0jBUD+V2raLH/u5QOCJQEEA7VwvQeZwm0OUH5E8rIjvReBDtwHbZMEWgbcqwLAxQ/TeIhlheAu+J0NWF0HFUYCnw8b29gCZSZC12F0bhxqgtDedpB9bIKuUeZqQU1JirXPwurZSpCPs9/Lh x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:BN9PR11MB5515.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230001)(366004)(83380400001)(316002)(2906002)(86362001)(38070700005)(91956017)(8676002)(66946007)(76116006)(38100700002)(66556008)(66446008)(66476007)(6506007)(53546011)(508600001)(8936002)(64756008)(2616005)(6916009)(6512007)(966005)(122000001)(36756003)(6486002)(26005)(5660300002)(186003)(82960400001)(71200400001)(99106002); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?WHBvMmU0azRraU92TzdFcW95SkdLV1BDclJVM2tMMWJXMXBVR0MrMTZsakkz?= =?utf-8?B?RFIyQjE5TWduSUdXMzU3dEZRTkg0Uk14ZU1tSWV1MzcyejhYVE9keUY4OHBN?= =?utf-8?B?eSt6ZkxpT3BGRktrT1k4Z2ZDb2JyYTFPTmJrb003M0ozaWdJam9HcFlrRWx5?= =?utf-8?B?QmU1Y05nS0Nia2ljZU9YWGpVVTlMcjBxVHdVME52NGcwNjd6MC9LaTVjZzlM?= =?utf-8?B?T04yOFYzZDBubE5weUZyaUxSVk8wZkVWR2V3eUZlSWhLNXBPL3J6dlJKQ3lk?= =?utf-8?B?cURkdTFQRkJZWHo2ZVFReDZ0QnVySWlwcGR3T1ZhMnI3TEJ0Q210MGdmZ0hk?= =?utf-8?B?WkNwT3d3bVRUQ1hJSFhMbnRFL1BFN0ErWU5vN3J1cU1OTnBXU2owTW5CejJr?= =?utf-8?B?end6a2w5MGJ0Z1BaOVZ6OTlucmRoRFJYeHgvZFUvalZQSEJSRVQ4T0pMY1hj?= =?utf-8?B?MlN3a0lsK0lzdlp3Q2FmaFN6S0E3UzBLUGtTb2tkQVdTU3AyU29ueUdBNHFF?= =?utf-8?B?cWlpanRpcHJCQk5rS3dEMDhaV1ZFRHBrMTdqSTZZeEtmdG85L28wM2ZITlN4?= =?utf-8?B?MG14VExJeUxXemw3UXIrYlp3K0dyQ082Qnd6bk9NcWhWblFVbDJhTFQrYWM3?= =?utf-8?B?aWtyUFBGUFB0TXNHLzA5M3k1N1g0NVBTMTAwaDliczU1aHV3UEdHaXNnUnE1?= =?utf-8?B?UXB6YisxZ25hQkpOOWV3K0ltUDRkWVlXSExLOWVvRkZWY3ZxZDU3ZThONHkw?= =?utf-8?B?bXdpdERTWmV1MTBPODFlck05V1V6WjFxSC9uemEyOXJjaERGMkxwREh0cmEz?= =?utf-8?B?NXl5ZFhhakliR3FtK3pDdUQzNlVCUGtjcDdIUmxuRnJYS201WXh5b2p4ZTRr?= =?utf-8?B?a0taOHVaWHA5OUE2UGRGUExFS1lLbjliQ0RxNTduL2pHK2tHTGw0VWVLb0hs?= =?utf-8?B?ZHFEbmwvK3JVNWdCYTVSdDNpTUxyUkNOMjB5SEg2MURUS2x4QlFoN2Z3WHNw?= =?utf-8?B?anRsbWNtL0o1WUdzQmRiVzZFZ3lUS0dPb08wQzVuaWRwMzdYOFU1bEZpZDFP?= =?utf-8?B?OFl0QlJOb0hDWTIxRlUreWcyQmx5eFlRaTAvL2VKbmFzd25DYXZNMHlNM3Y5?= =?utf-8?B?MUZMU0l5aDZYeVFWNXUwSkRnSTN3TDREN0R5b2tGYXpKdmNVbVVhVHQxVnVZ?= =?utf-8?B?SDBYc1ZOQ00wbTFYNW1sektqenBwRHA3NnRrRCtHdGQyZVU0czJsZGtpTWp4?= =?utf-8?B?bm9OSDF4UXFFTHpMMXJ4dUxCZ3dFKzFvMkNpazBnNlBZU0x2L2RYK09ocUF5?= =?utf-8?B?eVA1cDJEbXlHckRZK1hrQTlXc3U5SktmakRrc0x6MEFBOEpQOWZzNXE0djhX?= =?utf-8?B?a2YyWjBSM1VCZkhKWGNtUmxDVmgycjR3aUhUNWpPRW01QnUvcFgxQlRsdTVW?= =?utf-8?B?YnA2M0xuZVovT3Mvd2R0SHNCcElaV3M2TCtza1U3Yy9lenpnc2lhQVlsRnAz?= =?utf-8?B?ell5OWRKM3pTcVpNR2pJYUhzOE1DcFIyZlY1VWJzbkwreHc1dWJKMmRRazJv?= =?utf-8?B?ZUptYngxVThhVzB4MnBveXpkcEZ6allrRmlTU3hmUkVDSEh6bXdPV0lQaDJz?= =?utf-8?B?RHhFVXN5VU40eVRqS2R0WitHOG9WTlFhelFmZGQ0Rnc4bENJcDZyYndIU3Bp?= =?utf-8?B?V3Zoclc5T1g0eGVSUG8ySTd1Rm5mTG9haGhIajhqUjJtZ0JoRlBqWGVaMG1S?= =?utf-8?B?bklHaGhqTGhFMzNnelA4UGRwcjBocDBwTTcxeGNtSURBWmsxeGhRcXNYMDRT?= =?utf-8?B?Z1dFamcwOEM2ZVBnanRsK1pWVjVMcVZYcGFITmg0MG1EOHF3Q0VPallCUVov?= =?utf-8?B?ZXhzMjRWRHB3Y2l1UUlvZTllS2lsaU52TE1EaUx5TW1oeGRmL1RqdUtiRWxQ?= =?utf-8?B?aWloYVM0M2lmTkNWdmhvUStPWkJ4RHlzdkR1a0ZTWkJwNUFqQmVZaWNqempY?= =?utf-8?B?UWlwQ2hkRXN3PT0=?= Content-ID: <73391904D2F3924DA65241E86A6B1186@namprd11.prod.outlook.com> MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: BN9PR11MB5515.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 3c7e1d69-268b-42ed-aca8-08da056080f5 X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Mar 2022 02:15:26.3680 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: PMCnTYZY2a1CkCWV2wyxgd7EdCjkkENIEsyC8qFuf3hwGievg9bXtxyjXyjIDXb/NEM9XSdjHWxNXbTjYzn+5Q== X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH2PR11MB4391 X-OriginatorOrg: intel.com Subject: Re: [FFmpeg-devel] [PATCH v4 1/4] lavc/vaapi_encode_h265: Add GPB frame support for hevc_vaapi 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: On Sun, 2022-03-13 at 20:45 +0000, Mark Thompson wrote: > On 11/03/2022 09:00, Fei Wang wrote: > > From: Linjie Fu > > > > Use GPB frames to replace regular P/B frames if backend driver does not > > support it. > > > > - GPB: > > Generalized P and B picture. Regular P/B frames replaced by B > > frames with previous-predict only, L0 == L1. Normal B frames > > still have 2 different ref_lists and allow bi-prediction > > > > Signed-off-by: Linjie Fu > > Signed-off-by: Fei Wang > > --- > > update: > > 1. Add b to gpb. > > 2. Optimise debug message. > > > > libavcodec/vaapi_encode.c | 74 +++++++++++++++++++++++++++++++--- > > libavcodec/vaapi_encode.h | 2 + > > libavcodec/vaapi_encode_h265.c | 24 ++++++++++- > > 3 files changed, 93 insertions(+), 7 deletions(-) > > > > diff --git a/libavcodec/vaapi_encode.c b/libavcodec/vaapi_encode.c > > index 3bf379b1a0..bdba9726b2 100644 > > --- a/libavcodec/vaapi_encode.c > > +++ b/libavcodec/vaapi_encode.c > > @@ -848,9 +848,13 @@ static void vaapi_encode_set_b_pictures(AVCodecContext > > *avctx, > > pic->b_depth = current_depth; > > > > vaapi_encode_add_ref(avctx, pic, start, 1, 1, 0); > > - vaapi_encode_add_ref(avctx, pic, end, 1, 1, 0); > > vaapi_encode_add_ref(avctx, pic, prev, 0, 0, 1); > > > > + if (!ctx->b_to_gpb) > > + vaapi_encode_add_ref(avctx, pic, end, 1, 1, 0); > > + else > > + vaapi_encode_add_ref(avctx, pic, end, 0, 1, 0); > > This is doing something extremely dubious. If b-to-gpb is set, then don't use > the future reference? According to https://github.com/intel/media-driver/blob/master/media_driver/agnostic/common/codec/hal/codechal_vdenc_hevc.cpp#L3072-L3087 , L0 and L1 should be the same for vdenc hevc on some platforms, so user can't use past and future reference together, which is why you experienced the failure after applying version 2 Thanks Haihao > That means these pictures will only have the past reference, and the coding > efficiency will often be much worse. > > E.g. if you have the default structure (max_b_frames = 2, max_b_depth = 1) > then in a sequence of four pictures you get: > > 1 referring to something previous > 4 referring to 1 > 2 referring to 1 and 4 > 3 referring to 1 and 4 > > and this change means you lose the 2 -> 4 and 3 -> 4 references. Therefore, a > change in the picture between 1 and 2 will end up coded three times in 2, 3 > and 4 rather than just being coded in 4 and then referred to by the others. > > > + > > for (ref = end->refs[1]; ref; ref = ref->refs[1]) > > vaapi_encode_add_ref(avctx, pic, ref, 0, 1, 0); > > } > > @@ -871,8 +875,11 @@ static void vaapi_encode_set_b_pictures(AVCodecContext > > *avctx, > > > > vaapi_encode_add_ref(avctx, pic, pic, 0, 1, 0); > > vaapi_encode_add_ref(avctx, pic, start, 1, 1, 0); > > - vaapi_encode_add_ref(avctx, pic, end, 1, 1, 0); > > vaapi_encode_add_ref(avctx, pic, prev, 0, 0, 1); > > + if (!ctx->b_to_gpb) > > + vaapi_encode_add_ref(avctx, pic, end, 1, 1, 0); > > + else > > + vaapi_encode_add_ref(avctx, pic, end, 0, 1, 0); > > > > for (ref = end->refs[1]; ref; ref = ref->refs[1]) > > vaapi_encode_add_ref(avctx, pic, ref, 0, 1, 0); > > @@ -1845,6 +1852,51 @@ static av_cold int > > vaapi_encode_init_gop_structure(AVCodecContext *avctx) > > ref_l1 = attr.value >> 16 & 0xffff; > > } > > > > + ctx->p_to_gpb = 0; > > + ctx->b_to_gpb = 0; > > + > > +#if VA_CHECK_VERSION(1, 9, 0) > > + if (!(ctx->codec->flags & FLAG_INTRA_ONLY || > > + avctx->gop_size <= 1)) { > > + attr = (VAConfigAttrib) { VAConfigAttribPredictionDirection }; > > + vas = vaGetConfigAttributes(ctx->hwctx->display, > > + ctx->va_profile, > > + ctx->va_entrypoint, > > + &attr, 1); > > + if (vas != VA_STATUS_SUCCESS) { > > + av_log(avctx, AV_LOG_WARNING, "Failed to query prediction > > direction " > > + "attribute: %d (%s).\n", vas, vaErrorStr(vas)); > > + return AVERROR_EXTERNAL; > > + } else if (attr.value == VA_ATTRIB_NOT_SUPPORTED) { > > + av_log(avctx, AV_LOG_VERBOSE, "Driver does not report any > > additional " > > + "prediction constraints.\n"); > > + } else { > > + if (((ref_l0 > 0 || ref_l1 > 0) && !(attr.value & > > VA_PREDICTION_DIRECTION_PREVIOUS)) || > > + ((ref_l1 == 0) && (attr.value & > > (VA_PREDICTION_DIRECTION_FUTURE | VA_PREDICTION_DIRECTION_BI_NOT_EMPTY)))) { > > + av_log(avctx, AV_LOG_ERROR, "Driver report incorrect > > prediction " > > + "direction attribute.\n"); > > + return AVERROR_EXTERNAL; > > + } > > + > > + if (!(attr.value & VA_PREDICTION_DIRECTION_FUTURE)) { > > + if (ref_l0 > 0 && ref_l1 > 0) { > > + ctx->b_to_gpb = 1; > > + av_log(avctx, AV_LOG_VERBOSE, "Driver support previous > > prediction " > > + "only for B-frames.\n"); > > + } > > + } > > Please note that the PREDICTION_DIRECTION_FUTURE/PREVIOUS options don't mean > anything for H.265. > > The driver isn't told which direction the prediction is in, it's only told > about some reference frames and how to refer to them. Whether those frames > are in the past or future is unspecified and irrelevant - a P frame can refer > to a single future frame if it wants. > > (I tried to argue this when it was added, but given that they are meaningless > I didn't argue very hard.) > > I suspect you are trying to use this as a test of something else. Perhaps you > could explain what the test you actually want is? > > > + > > + if (attr.value & VA_PREDICTION_DIRECTION_BI_NOT_EMPTY) { > > + if (ref_l0 > 0 && ref_l1 > 0) { > > + ctx->p_to_gpb = 1; > > + av_log(avctx, AV_LOG_VERBOSE, "Driver does not support > > P-frames, " > > + "replacing them with previous prediction only B- > > frames.\n"); > > + } > > + } > > + } > > + } > > +#endif > > ... > > - Mark > _______________________________________________ > 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". _______________________________________________ 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".