From patchwork Fri Sep 13 16:13:32 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Fu, Linjie" X-Patchwork-Id: 15058 Return-Path: X-Original-To: patchwork@ffaux-bg.ffmpeg.org Delivered-To: patchwork@ffaux-bg.ffmpeg.org Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by ffaux.localdomain (Postfix) with ESMTP id 6B38244A741 for ; Fri, 13 Sep 2019 19:13:43 +0300 (EEST) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 494A76881CA; Fri, 13 Sep 2019 19:13:43 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id EDA6D688045 for ; Fri, 13 Sep 2019 19:13:36 +0300 (EEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga104.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 13 Sep 2019 09:13:35 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,501,1559545200"; d="scan'208";a="215428647" Received: from fmsmsx103.amr.corp.intel.com ([10.18.124.201]) by fmsmga002.fm.intel.com with ESMTP; 13 Sep 2019 09:13:35 -0700 Received: from fmsmsx125.amr.corp.intel.com (10.18.125.40) by FMSMSX103.amr.corp.intel.com (10.18.124.201) with Microsoft SMTP Server (TLS) id 14.3.439.0; Fri, 13 Sep 2019 09:13:34 -0700 Received: from shsmsx152.ccr.corp.intel.com (10.239.6.52) by FMSMSX125.amr.corp.intel.com (10.18.125.40) with Microsoft SMTP Server (TLS) id 14.3.439.0; Fri, 13 Sep 2019 09:13:34 -0700 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.113]) by SHSMSX152.ccr.corp.intel.com ([169.254.6.132]) with mapi id 14.03.0439.000; Sat, 14 Sep 2019 00:13:32 +0800 From: "Fu, Linjie" To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [PATCH, v2 1/6] lavu/pixfmt: add new pixel format ayuv/y210/y410 Thread-Index: AQHVakV+DdBDS+iCUEy6jGGcWpmJB6cpxg0w Date: Fri, 13 Sep 2019 16:13:32 +0000 Message-ID: References: <1568219321-20955-1-git-send-email-linjie.fu@intel.com> <20190913151055.GR3219@michaelspb> In-Reply-To: <20190913151055.GR3219@michaelspb> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action x-ctpclassification: CTP_NT x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiNTgyMzg2ZmYtY2VkMS00Nzc2LWE2MzMtMmRlMDgxNGU1MGZiIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiZXdnT0pwNUR5c1wvVTFXWlFqSU05eDBRTUlHUzFGSFRiTzFjaXp1VExOXC9DaEVpMDVCYjFBY3NSd0lNd1pFMDhtIn0= x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [FFmpeg-devel] [PATCH, v2 1/6] lavu/pixfmt: add new pixel format ayuv/y210/y410 X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.20 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: Michael Niedermayer Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" > -----Original Message----- > From: ffmpeg-devel [mailto:ffmpeg-devel-bounces@ffmpeg.org] On Behalf > Of Michael Niedermayer > Sent: Friday, September 13, 2019 23:11 > To: FFmpeg development discussions and patches devel@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH, v2 1/6] lavu/pixfmt: add new pixel > format ayuv/y210/y410 > > On Thu, Sep 12, 2019 at 12:28:41AM +0800, Linjie Fu wrote: > > Previously, media driver provided planar format(like 420 8 bit), > > but for HEVC Range Extension (422/444 8/10 bit), the decoded image > > is produced in packed format because Windows expects it. > > > > Add some packed pixel formats for hardware decode support in VAAPI > > and QSV: > > > > 4:2:2 10 bit: Y210 > > 4:4:4 8 bit: AYUV > > 4:4:4 10 bit: Y410 > > > > Signed-off-by: Linjie Fu > > --- > > libavutil/pixdesc.c | 62 > +++++++++++++++++++++++++++++++++++++++++++ > > libavutil/pixfmt.h | 9 +++++++ > > libavutil/tests/pixfmt_best.c | 1 + > > libavutil/version.h | 2 +- > > 4 files changed, 73 insertions(+), 1 deletion(-) > > breaks fate > > --- ./tests/ref/fate/pixfmt_best 2019-09-11 20:36:09.435012759 +0200 > +++ tests/data/fate/pixfmt_best 2019-09-13 17:09:53.430392453 +0200 > @@ -1 +1 @@ > -73 tests passed, 0 tests failed. > +74 tests passed, 0 tests failed. > Test pixfmt_best failed. Look at tests/data/fate/pixfmt_best.err for details. > make: *** [fate-pixfmt_best] Error 1 > If I didn't get anything missed in the upstream patch, this patch set has passed the pre-patch check in https://github.com/intel-media-ci/ffmpeg/pull/88, which includes the fate check. Above fate breaking will gone after applying patch 3/6 in the patch set: https://patchwork.ffmpeg.org/patch/15007/ Thanks for the kindly verify. - linjie diff --git a/tests/ref/fate/pixfmt_best b/tests/ref/fate/pixfmt_best index 5f51e2d..1da1846 100644 --- a/tests/ref/fate/pixfmt_best +++ b/tests/ref/fate/pixfmt_best @@ -1 +1 @@ -73 tests passed, 0 tests failed. +74 tests passed, 0 tests failed.