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 6A36C461D3 for ; Thu, 7 Sep 2023 08:05:08 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 7E65668C77C; Thu, 7 Sep 2023 11:05:06 +0300 (EEST) Received: from EUR05-DB8-obe.outbound.protection.outlook.com (mail-db8eur05olkn2060.outbound.protection.outlook.com [40.92.89.60]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id ABB0D68BE7B for ; Thu, 7 Sep 2023 11:05:00 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=LesWRh5a2KWgaCxRDH7DVLpF7t3tZ53qPn6wZVbBzKx6B6hpWzVfGuKktSSnHmXkPoeRBEaIrHzMpCaiSNRiO5bLmIOPVrbIojGluxAO7aPwMwj3InwEmXcqdiTE0oqzAQfGPUQESZJpOUwxs8XbLZx3yLUxnWDrYG0GXjsJuztFz1cDmIyF0e516V2u6mMCF0R/RtGU9CP2eMDp6/RPKiYviwPHd2o3zSP0ObJUWtODIhTYx5nHO99uvKxip5gw3xK5RYJdT40qvq8XUL3eBhusMF4IuMO8SV3V9t3bAgxxi4OhqC0jpJO+GO03TgovfKq1Z1dm4fHp7LWiyjOEtA== 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=a1ivyw73Wgn4Z3OYP/LllCu34SVWWUTgrIFcGV2mpaU=; b=AkBYtcPGW6xD3VF+tIU+lq4sF7QeUUDRh9ooKPVheQFOVb9ipibXbCEt7nDFxW7xq1NK/F7VB72PI1VbQUCxUb/KsJi47vqcSm+a4kwMFbXRKmPBaPv05Jt1r+vhCGhb24BqKsupPXpIpHRcOgbfQ2TPsYzjw7rzUpS2zxxbv5/rlTuyKyi8YwBfDibg1UUcni9c+yCrGCFpDHbAmqGog/t+eaM3h3nhYLTb/Euri7JsAOVWo+zpJ1Zhj1FZWt9YI7VFIY1ZHF1tNhjwqrjDrwwTU48EPzUku8dgwYumrn8ucdZQ7oNSzX9HFoKqEl6fO71AuU95Zo0V4dTcMHLK1A== 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=outlook.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=a1ivyw73Wgn4Z3OYP/LllCu34SVWWUTgrIFcGV2mpaU=; b=Eyazkvktx/sK9rBlkIMGl4xbIH+zW1q8SBFJHVrWiokpAS/KRX54ba4tla5MjVQIxkzm3CXV7aJK566iecdV/gr+x8fyWAZpoSPsq/lWpnE19RMnja8ISeDJB9dJNWXkuf9L4UXvyMlGcAjpAM/ic9cocRimQnW57gnoay45FTpGl6hW88+4kAES8y3GUEGLpCZatgznXvo/iiY1qiwsZY0vT4NYbYwAQ0cPulEGHA4A+iRRhsNmR24xyEd862OHAl5I+bzbWo8LwHr0Wje2WqpEvY9Up4KQBfDg9fUsN1YlRKvVrVnZY9cmXBK6B2q345/UTXOBYcrdO/10SawB5w== Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) by AM8P250MB0139.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:322::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6745.36; Thu, 7 Sep 2023 08:04:59 +0000 Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::5e01:aea5:d3a8:cafa]) by AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::5e01:aea5:d3a8:cafa%3]) with mapi id 15.20.6745.034; Thu, 7 Sep 2023 08:04:59 +0000 Message-ID: Date: Thu, 7 Sep 2023 10:06:16 +0200 To: ffmpeg-devel@ffmpeg.org References: <20230907020645.37017-1-jamrial@gmail.com> Content-Language: en-US From: Andreas Rheinhardt In-Reply-To: <20230907020645.37017-1-jamrial@gmail.com> X-TMN: [Bsxnr77ZWqdLfR5gt0i1eOigwNXDKriS] X-ClientProxiedBy: ZR0P278CA0021.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:1c::8) To AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) X-Microsoft-Original-Message-ID: <0e78a4ad-f270-d111-a23f-f1e9ede36cae@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AS8P250MB0744:EE_|AM8P250MB0139:EE_ X-MS-Office365-Filtering-Correlation-Id: f63595c8-95cb-42a2-bf1e-08dbaf79215c X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: IkCqQPkNk9xpImcvfsrQaBdKfXNDUltW4EBWfpxkcI3UuNuFW5WZwglUwUvooSq/NWxkBqA0G07v4F2UKNX96d9nD/+ID2E2DFEKGn+O5xiD2rZjyr7VoW+dIKIGp8JlkFajXcYdZ2BT8R6W2XS1vYFam6LRB6/aKw0JCKhneheVdEqspbJ/akz1Ng8PHBz9v36yYrCDdu+6d+Z7gCgsObp2Y0VaMm3lXASdhmnxHEjXs4TPPGqMakJ9U8MH9uUCsOlBDloDX9I1ePCOwu+qO1wuRi/iipzwezSJUarOHkL6GLotiq14q5SKDIz7SooJw5mw+JX5glOYK7RxR3XQWhsM789aWuGQZRn4SMbIp7tkZQ4scI+Tsegoo71BzB+9K3WyZwGAmY0FCAWS7YlCoLg1Y8g0xhnIJoGH22YBW7BSGY5F2fyeVvC+PHPbqlEOJiW8cWUhQ5zbVX2+zvj0FfTT8YNYE5Kogg2F34MfQtqC/5TaMUQ/1ds+XC3PJQ85FuyDDsmfZWX5P/BQ4NuQsDwajRPFSMiF/psDjK+yDSgcvsQkBvEontvKciCgGgli X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?QytLbmljVjlDQXJFam04NVVQSE1qRldVNm1RVHE4MjJXNjBFc052b3BkR3NK?= =?utf-8?B?RklQQ0d2Q0ZxMThMWS9EV3BxaXdNREszVTluQ2ZBZkx4SllDN0FSd0c4Wkd0?= =?utf-8?B?WlVJUHJRZ3RUeTkzVCtGZ0hJOWtoeXc5Z3hwcjhTTTF2dXg5b0RXYldWT0Nq?= =?utf-8?B?eERWNTJVR0ZxM3l2NWlPYlRsNEJWcEtlSjVWSHFVdkZMdjRqL3BzTyt0ZU5N?= =?utf-8?B?b2pHWVRFbDJTVCtZVzY5cWt0OFF5TmxrUGhqb04waFNWaVIwSnlOdDJ1NC9H?= =?utf-8?B?eDBMWlNNRUJiaUZ2SjM3QTBuOGhwRmlkdjdDS25WNnI1dG52MGRmTHEwNCtl?= =?utf-8?B?NGRvTHRDQktuQUhUMlVSZnNQY2ZLWG5mNUVra3lpQXFzdlVVVWk4aXR2dk93?= =?utf-8?B?NjNNK3NxblZDek80ZkdDSERSVmxjQVZDa25zOHFkekUxaytKL3ZMWkxuTENX?= =?utf-8?B?YjV1UUg1a1NkcE4xM1ZZT2NySm1SbEFRRUh2VzRwVE45d09lZ0kzWjlMY1pG?= =?utf-8?B?eDg1NUNiaTM5NDBJWEx2a1l4Y3lVa0xSZmJ0bVptK1dMcU9vT1U5eVl5WlVq?= =?utf-8?B?Wm5vbGlpTkorMnd0ZE5hVWY4WW9ZT0dPaUVvSUgvMy9ZU0IrRWJDcWhZcGh5?= =?utf-8?B?UzE3RGhxQ0sxeDh6SVl4Q3EzZFA1aU9mdE95cGdkUDJzK3JjSkJRTTYzVUxW?= =?utf-8?B?a2RrV2NzQTE4Q0FpUURPN0NoOUViVmJ2RlFkeEY0WktjM29CYW0wUUNoT2s2?= =?utf-8?B?KzR1VHhqSU9ibGFQeWIzaVRiVkhBS0w1YTg1Tm1DbXgxTkhpT2lJa3luVlNN?= =?utf-8?B?T01RMHFlWmNNRnRPc1g3Mm5DdDRrYWE3aU9iMlpjUnViVGJKSmdwNG54YXhR?= =?utf-8?B?ampvcVZMRkZReWtBSWVTYis0cU0zcitlTlBPT0F2NS9ETFRodU1GRWZQQnNC?= =?utf-8?B?ZCtoZloweC9VSWhPN0FFKzMwWlFtdG5mOXJKUHhuNVhWUmY0RE13T014QnlO?= =?utf-8?B?Ykp1Sm1JUGJ5dmNrczZ4TGFEREw2ZUl3amVJWk1kcjRMVVNMcmR4WExQeXRk?= =?utf-8?B?WmlQVkY3TnNxSnM3ZDJxRkFIc2hXVUN1NGNsOUdvbTRuczhqTmdRcE9ndERZ?= =?utf-8?B?K0UzSzZMUjJyendOL2wvUXNLb1grUnFkeGlmdzZxK2Iwa3JGdFlvcS96cVV1?= =?utf-8?B?WHZlOXZaQzk3VVl2ZjNJWk1JVHRHd0dkTVNTMWREQTQ3L3RvbFp1Y01TdlF6?= =?utf-8?B?MDJEYTg1MXVWaUZhaTVLZkdDZG1sTWY4NWVnMkx1RXJwNW51bmVJVTJmMFlN?= =?utf-8?B?ZTBTeTVqcy93bGJxcGd1NUNXcGdHQXdMTmhvVTZrUkpLTk9QN0dMT2x4dnFJ?= =?utf-8?B?ZTdGa2Jnc0hhTzhvTkxUd2VOTkJyMTlIS2VYRGVuV0pqQllCSUNTalR5alhn?= =?utf-8?B?TzY3UENCY2IyWG81RlJMZFd6TE83eWJ4U2ZKODFtSkx5ZTNLSGRiSzdjaE03?= =?utf-8?B?L0ZKbHZTNkxHZmN6NUNTQThpUHptWVFHbCs4NHppMjAwQWZGM3UzT1Vjbk9v?= =?utf-8?B?KzVTMXN2TGxYSzk2UlpwM3FmbVJrbUE3Y1pQV1RjODNoSWt3TDdUR3hpek5l?= =?utf-8?B?LzU4amdzMFRDSmlUVUczdXVHWjVGbk03czhCRi9kOG5qL1BpdjY5K3NZd0M0?= =?utf-8?Q?YHpUYkS+WDdRa0WYP6yy?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: f63595c8-95cb-42a2-bf1e-08dbaf79215c X-MS-Exchange-CrossTenant-AuthSource: AS8P250MB0744.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Sep 2023 08:04:59.0798 (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: AM8P250MB0139 Subject: Re: [FFmpeg-devel] [PATCH] avcodec/av1dec: export pixel format even if no hardware decoder is present 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: James Almer: > And remove the AVOID_PROBING flag, given it's the last av1 decoder to be tested > either way. > This fixes a regression introduced in 1652f2492f88434010053289d946dab6a57e4d58, > where even if forcing the native av1 decoder, if another decoder was present, > like libdav1d or libaom-av1, they'd be used for probing and some fate tests > would have different results. > > Signed-off-by: James Almer > --- > libavcodec/av1dec.c | 8 ++++---- > tests/fate/lavf-container.mak | 8 ++++---- > tests/ref/fate/av1-annexb-demux | 2 +- > tests/ref/lavf-fate/av1.mkv | 4 ++-- > tests/ref/lavf-fate/av1.mp4 | 4 ++-- > 5 files changed, 13 insertions(+), 13 deletions(-) > > diff --git a/libavcodec/av1dec.c b/libavcodec/av1dec.c > index ec8401f4e0..a1e08185a7 100644 > --- a/libavcodec/av1dec.c > +++ b/libavcodec/av1dec.c > @@ -612,6 +612,9 @@ static int get_pixel_format(AVCodecContext *avctx) > if (ret < 0) > return ret; > > + s->pix_fmt = pix_fmt; > + avctx->pix_fmt = ret; > + > /** > * check if the HW accel is inited correctly. If not, return un-implemented. > * Since now the av1 decoder doesn't support native decode, if it will be > @@ -623,9 +626,6 @@ static int get_pixel_format(AVCodecContext *avctx) > return AVERROR(ENOSYS); Is the log message here actually accurate? The get_format callback choosing the software pixel format does not mean that the hardware this is run on or the lavc binary in use do not support hardware accelerated AV1 decoding. (How is an API user actually supposed to know that decoding will fail if the software pixel format is selected?) > } > > - s->pix_fmt = pix_fmt; > - avctx->pix_fmt = ret; > - > return 0; > } > > @@ -1518,7 +1518,7 @@ const FFCodec ff_av1_decoder = { > .init = av1_decode_init, > .close = av1_decode_free, > FF_CODEC_RECEIVE_FRAME_CB(av1_receive_frame), > - .p.capabilities = AV_CODEC_CAP_DR1 | AV_CODEC_CAP_AVOID_PROBING, > + .p.capabilities = AV_CODEC_CAP_DR1, > .caps_internal = FF_CODEC_CAP_INIT_CLEANUP, > .flush = av1_decode_flush, > .p.profiles = NULL_IF_CONFIG_SMALL(ff_av1_profiles), > diff --git a/tests/fate/lavf-container.mak b/tests/fate/lavf-container.mak > index 0d4a224601..0081b45eea 100644 > --- a/tests/fate/lavf-container.mak > +++ b/tests/fate/lavf-container.mak > @@ -70,9 +70,9 @@ fate-lavf-wtv: CMD = lavf_container "" "-c:a mp2 -threads 1" > FATE_AVCONV += $(FATE_LAVF_CONTAINER) > fate-lavf-container fate-lavf: $(FATE_LAVF_CONTAINER) > > -FATE_LAVF_CONTAINER_FATE-$(call ALLYES, IVF_DEMUXER AV1_PARSER MOV_MUXER) += av1.mp4 > +FATE_LAVF_CONTAINER_FATE-$(call ALLYES, IVF_DEMUXER AV1_DECODER AV1_PARSER MOV_MUXER) += av1.mp4 > +FATE_LAVF_CONTAINER_FATE-$(call ALLYES, IVF_DEMUXER AV1_DECODER AV1_PARSER MATROSKA_MUXER) += av1.mkv > FATE_LAVF_CONTAINER_FATE-$(call ALLYES, EVC_DEMUXER EVC_PARSER MOV_MUXER) += evc.mp4 > -FATE_LAVF_CONTAINER_FATE-$(call ALLYES, IVF_DEMUXER AV1_PARSER MATROSKA_MUXER) += av1.mkv > FATE_LAVF_CONTAINER_FATE-$(call ALLYES, H264_DEMUXER H264_PARSER MOV_MUXER) += h264.mp4 > FATE_LAVF_CONTAINER_FATE-$(call ALLYES, MATROSKA_DEMUXER OGG_MUXER) += vp3.ogg > FATE_LAVF_CONTAINER_FATE-$(call ALLYES, MATROSKA_DEMUXER OGV_MUXER) += vp8.ogg > @@ -86,8 +86,8 @@ FATE_LAVF_CONTAINER_FATE = $(FATE_LAVF_CONTAINER_FATE-yes:%=fate-lavf-fate-%) > $(FATE_LAVF_CONTAINER_FATE): REF = $(SRC_PATH)/tests/ref/lavf-fate/$(@:fate-lavf-fate-%=%) > $(FATE_LAVF_CONTAINER_FATE): $(AREF) $(VREF) > > -fate-lavf-fate-av1.mp4: CMD = lavf_container_fate "av1-test-vectors/av1-1-b8-05-mv.ivf" "" "-c:v copy" > -fate-lavf-fate-av1.mkv: CMD = lavf_container_fate "av1-test-vectors/av1-1-b8-05-mv.ivf" "" "-c:v copy" > +fate-lavf-fate-av1.mp4: CMD = lavf_container_fate "av1-test-vectors/av1-1-b8-05-mv.ivf" "-c:v av1" "-c:v copy" > +fate-lavf-fate-av1.mkv: CMD = lavf_container_fate "av1-test-vectors/av1-1-b8-05-mv.ivf" "-c:v av1" "-c:v copy" > fate-lavf-fate-evc.mp4: CMD = lavf_container_fate "evc/akiyo_cif.evc" "" "-c:v copy" > fate-lavf-fate-h264.mp4: CMD = lavf_container_fate "h264/intra_refresh.h264" "" "-c:v copy" > fate-lavf-fate-vp3.ogg: CMD = lavf_container_fate "vp3/coeff_level64.mkv" "-idct auto" > diff --git a/tests/ref/fate/av1-annexb-demux b/tests/ref/fate/av1-annexb-demux > index 77e0e378ab..139a893ec1 100644 > --- a/tests/ref/fate/av1-annexb-demux > +++ b/tests/ref/fate/av1-annexb-demux > @@ -3,7 +3,7 @@ > #media_type 0: video > #codec_id 0: av1 > #dimensions 0: 300x300 > -#sar 0: 1/1 > +#sar 0: 0/1 > 0, 0, 0, 48000, 12691, 0xf0adcc79 > 0, 48000, 48000, 48000, 4975, 0x1742a45f, F=0x0 > 0, 96000, 96000, 48000, 928, 0x7408be1a, F=0x0 > diff --git a/tests/ref/lavf-fate/av1.mkv b/tests/ref/lavf-fate/av1.mkv > index 84924fe05c..e6db3a814a 100644 > --- a/tests/ref/lavf-fate/av1.mkv > +++ b/tests/ref/lavf-fate/av1.mkv > @@ -1,3 +1,3 @@ > -ba198efa114b4db3c9f772728f84978b *tests/data/lavf-fate/lavf.av1.mkv > -55656 tests/data/lavf-fate/lavf.av1.mkv > +d875b42964dd55c8cea2e2752379d327 *tests/data/lavf-fate/lavf.av1.mkv > +55660 tests/data/lavf-fate/lavf.av1.mkv > tests/data/lavf-fate/lavf.av1.mkv CRC=0x7c27cc15 > diff --git a/tests/ref/lavf-fate/av1.mp4 b/tests/ref/lavf-fate/av1.mp4 > index ef174589ba..53b0cbe637 100644 > --- a/tests/ref/lavf-fate/av1.mp4 > +++ b/tests/ref/lavf-fate/av1.mp4 > @@ -1,3 +1,3 @@ > -d6afef355cf1bcf1a71b9cfb98c32fba *tests/data/lavf-fate/lavf.av1.mp4 > -55976 tests/data/lavf-fate/lavf.av1.mp4 > +194cd796f3d28a890b516b2507f57180 *tests/data/lavf-fate/lavf.av1.mp4 > +55960 tests/data/lavf-fate/lavf.av1.mp4 > tests/data/lavf-fate/lavf.av1.mp4 CRC=0x7c27cc15 _______________________________________________ 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".