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 706BF44180 for ; Wed, 31 Aug 2022 01:20:22 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 619A368BB02; Wed, 31 Aug 2022 04:20:20 +0300 (EEST) Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 1EB6A68BA94 for ; Wed, 31 Aug 2022 04:20:12 +0300 (EEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1661908818; x=1693444818; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-id:content-transfer-encoding: mime-version; bh=nC0RoAcSCbajbgcWe7G+jVtbo9GdYlh4avUaJyEdmU4=; b=iUl+vmOpWFxXmdOw9wFlYoDrPiDucavGiChyEauJBZsFvcNbQB00up/k mK0nC8KR4y/ce8rIZBg/L3nm16qziPCCqBjb+lpg6i4ehlDiJ9VMOnIvI DpOxLmQqWPU1GHzMwB3mJi4CQKFHZODNkeRpEDToPSfgcOqhUdB1vWwwq fHPVs2Fq6lbPdoGd/ADNGUO+N338+jJQh7lpC+Joec0X00Bt4wD+pOIMt 2ysl72kA23YnjzeWVSescK08n+x0pd+O0L3mxbWqJCbDFdz44YwhkRDKI Q+CokiZmO6hTYHnW1xfKgZhB/g+MvOEynHHQ1xNSR6XMgbkgkO0RqpRU/ Q==; X-IronPort-AV: E=McAfee;i="6500,9779,10455"; a="294085664" X-IronPort-AV: E=Sophos;i="5.93,276,1654585200"; d="scan'208";a="294085664" Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by fmsmga104.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 30 Aug 2022 18:20:09 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.93,276,1654585200"; d="scan'208";a="940260760" Received: from fmsmsx603.amr.corp.intel.com ([10.18.126.83]) by fmsmga005.fm.intel.com with ESMTP; 30 Aug 2022 18:20:09 -0700 Received: from fmsmsx607.amr.corp.intel.com (10.18.126.87) by fmsmsx603.amr.corp.intel.com (10.18.126.83) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.31; Tue, 30 Aug 2022 18:20:09 -0700 Received: from fmsmsx612.amr.corp.intel.com (10.18.126.92) by fmsmsx607.amr.corp.intel.com (10.18.126.87) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.31; Tue, 30 Aug 2022 18:20:08 -0700 Received: from FMSEDG603.ED.cps.intel.com (10.1.192.133) by fmsmsx612.amr.corp.intel.com (10.18.126.92) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.31 via Frontend Transport; Tue, 30 Aug 2022 18:20:08 -0700 Received: from NAM10-MW2-obe.outbound.protection.outlook.com (104.47.55.107) by edgegateway.intel.com (192.55.55.68) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2375.31; Tue, 30 Aug 2022 18:20:08 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=beJDqobuICK/NaIkPfihDeYoxcaZFcr9npVKB9dUad+Gt2qhmwZ0HWsdbsROAAJia7KKrtCaIXMrDPDMv6yoKdWlmFoqS+zunb7KgpvwyBfhlv661CtkUdygJ3CtNzPCIeLEGYa03dHLqNhm1J9Muc8yAKbWaU/+ZVeQXsP7nBX/jFf75O1EpEMjKoz2nGIadgkgDftS45l/nui02/f2IWt7voONo7cPi2IEhxPMOpVdAR25kyei4sE9CaACna6qapLECrtmmXmtjvWlhT9MAzILj96quDnmWy2XHzZvDwOnVBdlcg1qx5ePYoCtJamMVrDhNuzN7dfNKlphWvdLpg== 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=nC0RoAcSCbajbgcWe7G+jVtbo9GdYlh4avUaJyEdmU4=; b=bHIl1ETsPmI2ql6XhCugT8+c9Tam9c5w21SFH6wpAkMoApejqKiKxCnCRndKc64w0OQY7hXWV1H0MhsjoM6yM9rkjdpmKadqCXFWJh3UwovgQjqAWE0sy86WMHGaIdABjTUUT4ev5n5K59vDhsM7a3tYRyjKZhXh0dEiSf84e0cVGscfFSm7DALJgCRHVfNPX+zYFCkzgzraR1pFhkdy5NWiGhIV0UJSsVOCguqK0kfF5LG70fHltpjWS7pUVDCR1VK8OjlWeRLS7J/ii+8ey47CRDhCMYZpz18WnPtCbX9AyXeS0dMsk/sqg9OT/wR0D2TNzhbPo3O1QveUUCGFEQ== 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 PH0PR11MB5030.namprd11.prod.outlook.com (2603:10b6:510:41::5) by SJ1PR11MB6227.namprd11.prod.outlook.com (2603:10b6:a03:45a::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5566.21; Wed, 31 Aug 2022 01:20:07 +0000 Received: from PH0PR11MB5030.namprd11.prod.outlook.com ([fe80::bd7e:47b5:e014:2c0f]) by PH0PR11MB5030.namprd11.prod.outlook.com ([fe80::bd7e:47b5:e014:2c0f%9]) with mapi id 15.20.5566.021; Wed, 31 Aug 2022 01:20:07 +0000 From: "Wang, Fei W" To: "ffmpeg-devel@ffmpeg.org" Thread-Topic: [PATCH v3 1/3] lavc/decode: Warp get_hw_config function Thread-Index: AQHYtskmqwATgBiot0aHmLpUWEUQO63IQd+A Date: Wed, 31 Aug 2022 01:20:07 +0000 Message-ID: <7f8cdfa93aac92c3ef889548c2540cf74033cc3b.camel@intel.com> References: <20220823081929.413947-1-fei.w.wang@intel.com> In-Reply-To: <20220823081929.413947-1-fei.w.wang@intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Evolution 3.36.5-0ubuntu1 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: a11935b3-583f-4746-a1fd-08da8aeef0ba x-ms-traffictypediagnostic: SJ1PR11MB6227:EE_ x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: nf3BEuI3txFj98eX7CoIytWStZJ4Ej6OB0T0PKUQvAUynXoPbrEx1kWbP/1wI5KG3umK+9OwzUx3VcnLLP1BI2g4ifgRb0OjXZLlTcAz5CZPbC8MEsd5DoAtOJVbLXofHqReSnJdbv06jkv2ppcxtVklNpXOlMTVDvYSHY049jtDhQd9wRvnD/iUvc44tQchNsvYzEWi91aMbIYhTqKBRvaUP1/uPJHFcvtsgZZJSIQGdKla5nWm8QBjiL1lZN263hegioYZOU/4oYgiL+ENx9QoeobQnVqlGVki9idEK37ihcdqkvV4odwd8ctzppHIPvl67Pe+4OZUBnum4D9aceWZwK1Vevd13quRRYy9tv33R64Z/ygHWI8qslAMwUUV8Hw+/ojCqvDW+ES0SNDzZzhnUcW+DHaEbkbaSBxdJwnDUU4F5cdszZxSgc3wlpHF3iNqi2xvVAqk19eISzASji7KQx1Ze6iqEEr2P/6gl6UNlbPeOEhWdQX67BnxWJ4QchajdtmxZ23s59jz+W5r7r5ooIqNBSr9OYXM39Sku3F4CQpgM52babSV0NlIW4xuBCclEmP2J3M6eUgNc1NHK7OSpDk99AWZNXSFNnJL0bqH9BE7scFwlG4ZAhfr3q6dIHbTQG5dusdWr9BcEkE8kXqnQkleAQx4JczTa+/h4/dXd/Uaxl1rJjfrgqVi2lFajsIRNFeGB7me0FeOS/z6JuQrdJHDux+BZw7JdlTCFaoA6ikO+QwvmMHUw7/5E4A3DamP0Jnd/cwfQeB/dsPDoA== x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PH0PR11MB5030.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230016)(39860400002)(376002)(366004)(346002)(396003)(136003)(6486002)(4326008)(8676002)(66446008)(2906002)(91956017)(66476007)(66946007)(66556008)(76116006)(64756008)(6512007)(6506007)(316002)(6916009)(26005)(71200400001)(41300700001)(478600001)(107886003)(8936002)(38070700005)(83380400001)(2616005)(186003)(122000001)(36756003)(5660300002)(38100700002)(82960400001)(86362001); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?RWFnbkQ1L2hJMXFpSEtPVzVWajErdGJBNTlCbS9hbitPbGNqaUlxUWJWaXdw?= =?utf-8?B?MzlvYkcrbHY3TDhkUEkwMWhGRTdtVUlHRnB5YkRrRXZOZE1mZ25LbVUwTTZZ?= =?utf-8?B?ZFZPcXppdVJyak9LZXFvZytTZlZsWDhjWXg3ditYR0xINm9IYWtzaGtRSUta?= =?utf-8?B?cTNKYWZLUlZVSU4zdzF1bGMzb1dKTmVnalc4T2ZCRVJmd2dvbE1ubk1Zc20y?= =?utf-8?B?RGVmclFiSXRsQUxaWXgrQ3E5Ty9JSkMwcTVUaGJ4YmRCbC9hakkwYmRoN1lY?= =?utf-8?B?NGhKRmtyQ2c2WFlyZTRnTUtZM1JvRFh4V2k5eUticXEreW11RHRSaXhiMmZ5?= =?utf-8?B?RXpoSzJxVXFOOGtoOTh2T2JDanVKb0NuZ2NCTlRaMFRvdjlPK0Zjdm9BZ01r?= =?utf-8?B?UTJ4UUFkc05VOUp5aUhIWXg0Yk5jcW11L0d6Y3JwcStRYzAra2pQbnpJeTZx?= =?utf-8?B?OWExUGcyOFBTTDg1VXg2YVdnTmU3ZmdIaGZaSFVKdEMwbmRVL3BMNEgzTktT?= =?utf-8?B?Y01Ld1NCcmplQ3ozOElYZVZ6Ni9mbEJXcS9TWkljQ2ZvWnlMN05mRERqbU9p?= =?utf-8?B?a0pucHZ0d3pHWnBFYnpDQjc5NXhCam03a2NkVEZBMndEUzBLUXR1b1JvNjdl?= =?utf-8?B?Zk0vZkxmQlRLYVVDNDlwMGJZb0hsd1JJUmFuZitFV0Ezb2pRZ1lacjJKVFpT?= =?utf-8?B?d3dqclFJWTJ6eUs5SEpHeStnVTlaQTBPVTVKZk1RRytuT0tQUHFiYlNuRjg1?= =?utf-8?B?TGhhK1BDRzZDYk54WXVIWlZ3bXV4WE9IMDlxYWYxbVpOR3BtUjFud0JNK1B6?= =?utf-8?B?YXlUMkFpdGxDb2M2MkYxQnFoZXZmRllyS3J2cHFIVDlHemFhSFZPdzFLbWVz?= =?utf-8?B?dzV3WStkcmdHaUpqS3hlVmFrK2Rra0Jpdkd5RUY5L0xBUFlGY0Z1NDAzZHpL?= =?utf-8?B?VDY3eThQa1dVenVlR0NuemlwT05HYnBQaEM1cFRwVU10aVJORTlKY1VvQ1Zs?= =?utf-8?B?NTFuZ1Y5d21MaGNpL3NuZ0RGWW1SaXVSUzVVeit6Ym85V1IxUjFUZlI1amly?= =?utf-8?B?d0RmNnRZdmpjZS9KQU1KaUxpS1pxa0VpOTVIUm9UdjFvQVpWeU8zUlVUQXhC?= =?utf-8?B?YjhuMkhSbGhMaldCdnFEVk9rQ0RMSDBLdTgwNWdIakRFQUFKY0M5a094L2Zj?= =?utf-8?B?OWVuVGlzL05FK3ZsUHVneFV5cWhpU0NMdTNMSUlTS3RtMVVXa2h3cHhoYXBU?= =?utf-8?B?ZTcyY21ZeXZqakNQTnZuelVJQXp5Zi9WaXRYR2txbmFSakNMNXBsV2xwakhC?= =?utf-8?B?blA4cERQeHIyWGo3YTBxWTFTWHhsdTFSV1F5dlBJemgzVklnbzFRZmFPVFhO?= =?utf-8?B?ZkZLWktCNkl1dmNkRi9adVF2aHUyR2p0M2VoQUx2UU85LzNQTUdyQ0hDOE42?= =?utf-8?B?OTdBb0tWa1FIWDBva3BCbmZpK3Bnbnl2RFYxT3RzK1BVS0dCclpwR0tWRks2?= =?utf-8?B?MHNpMzA0TSs3ZjNZU1MxSnFvMTVpMGRmbXlCS2ZPTmdSM3R0TTB5VFFSU0Zu?= =?utf-8?B?cmIwR1FSS2w1cFQ0QmNIdytmdXo0TkdQRTVWbjhrUTN6Zk5vOFprM1AzcEZK?= =?utf-8?B?YWZSYURxSWlkdVMwbmZlL1RPekdFbmVGU2hMRWpOZGRNUEJHNDV4aU1Samsy?= =?utf-8?B?U3pFYzhqRzMrekw3UlByRjdLYVBoTmkzVndyVWlhd1U2TFBQYktvWUg3OUJD?= =?utf-8?B?WENQTisvMGRQcG5TV1pqUUx2ZVpKY01jZ3YyM2p4a0Jrc1ZPMHhhQlNLVEFH?= =?utf-8?B?WG5wOEdVNVBHVWJyanRoNlpLY0I0UHF3YkF0dmxhVnAwUXpQbG5HcDRYY3d5?= =?utf-8?B?TVAxQUcvTU5Id1RXZFJkSVpwcElhU0EzQUNDejBYN1NwTldONkEyVkg3dFNY?= =?utf-8?B?YlArVkl4K0VjVHhiWDArYWh6cHhQNFgzN3ZUaE9VejdlMU1FR01zMUFNaGFt?= =?utf-8?B?WFIyclduZ2xpOFFOL0ZnNE1rRXJCWmNvbGM3UkZUYjBWbGFUN2xyU3VYazMy?= =?utf-8?B?UitrMUhUMExRb2NPTzlvVkI5UlZ0OGhMK1F6QnhXU0g4NjAxbmxDUzJreVpx?= =?utf-8?Q?2IF5Ab2m7fg1uKctY5QZg5Gu3?= Content-ID: <0A9B8CDF947B8045B3C4109DDFF09F53@namprd11.prod.outlook.com> MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: PH0PR11MB5030.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: a11935b3-583f-4746-a1fd-08da8aeef0ba X-MS-Exchange-CrossTenant-originalarrivaltime: 31 Aug 2022 01:20:07.1646 (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: XjgWVD7EJnJNgn1QRuhhJr8jTB8j2Ngh4PnoSg31S7elU7AA9DcF/5vKS1bwW6JphicC5DvS2c7NPHfJgAGqPA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ1PR11MB6227 X-OriginatorOrg: intel.com Subject: Re: [FFmpeg-devel] [PATCH v3 1/3] lavc/decode: Warp get_hw_config function 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: "linjie.fu@intel.com" 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 Tue, 2022-08-23 at 16:19 +0800, Fei Wang wrote: > From: Linjie Fu > > Wrap the procedure of getting the hardware config from a pixel format > into a function. > > Signed-off-by: Linjie Fu > Signed-off-by: Fei Wang > --- > libavcodec/decode.c | 31 +++++++++++++++++++------------ > 1 file changed, 19 insertions(+), 12 deletions(-) > > diff --git a/libavcodec/decode.c b/libavcodec/decode.c > index 75373989c6..3b69426c09 100644 > --- a/libavcodec/decode.c > +++ b/libavcodec/decode.c > @@ -1156,6 +1156,24 @@ static void hwaccel_uninit(AVCodecContext > *avctx) > av_buffer_unref(&avctx->hw_frames_ctx); > } > > +static const AVCodecHWConfigInternal *get_hw_config(AVCodecContext > *avctx, enum AVPixelFormat fmt) > +{ > + const AVCodecHWConfigInternal *hw_config; > + > + if (!ffcodec(avctx->codec)->hw_configs) > + return NULL; > + > + for (int i = 0;; i++) { > + hw_config = ffcodec(avctx->codec)->hw_configs[i]; > + if (!hw_config) > + return NULL; > + if (hw_config->public.pix_fmt == fmt) > + return hw_config; > + } > + > + return NULL; > +} > + > int ff_get_format(AVCodecContext *avctx, const enum AVPixelFormat > *fmt) > { > const AVPixFmtDescriptor *desc; > @@ -1213,18 +1231,7 @@ int ff_get_format(AVCodecContext *avctx, const > enum AVPixelFormat *fmt) > break; > } > > - if (ffcodec(avctx->codec)->hw_configs) { > - for (i = 0;; i++) { > - hw_config = ffcodec(avctx->codec)->hw_configs[i]; > - if (!hw_config) > - break; > - if (hw_config->public.pix_fmt == user_choice) > - break; > - } > - } else { > - hw_config = NULL; > - } > - > + hw_config = get_hw_config(avctx, user_choice); > if (!hw_config) { > // No config available, so no extra setup required. > ret = user_choice; Ping, any more comments on V3? Thanks Fei _______________________________________________ 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".