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 8360C407C4 for ; Wed, 2 Mar 2022 07:23:40 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 8016868B22F; Wed, 2 Mar 2022 09:23:38 +0200 (EET) Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 77F8268B1C7 for ; Wed, 2 Mar 2022 09:23:31 +0200 (EET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1646205816; x=1677741816; h=from:to:subject:date:message-id:references:in-reply-to: content-id:content-transfer-encoding:mime-version; bh=DCqkFmQa+Ly/A/t5d/oczcc6ZmdFfaiQuhY3rynQYA8=; b=ah5yZDjZkVtrVaaFVM9BDmycNAPko2OI0uD3Cd7NSaurqAg1ug5CKjwd 29YDJBgZgTKeaFoE4BsRLCRhDVU2TfjD7u/cA7M6rwErPyxuGP65pAUTd sInZjgh/vv92x5mYBF0rWcUcNULmwi9lNG95TYnJflnNwRJ/g0CyFNJN3 Rlea8vmbsAHDUmTT8fArLbD0YiLiEw/DpLVJRXBe+CfXMiGQoFTvYqnuM 8y6idgNcp02XxX1fSbVpzEdukiCuFQQHmupAOuDjhcJv9mdKyuVqEsI26 T4KhnrRJTueCXE2CP9M33OS1Jc8Jf9i2jHJcg9qtj+2MpD6FdNQmcKvjo Q==; X-IronPort-AV: E=McAfee;i="6200,9189,10273"; a="240743331" X-IronPort-AV: E=Sophos;i="5.90,148,1643702400"; d="scan'208";a="240743331" Received: from orsmga006.jf.intel.com ([10.7.209.51]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 01 Mar 2022 23:23:19 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.90,148,1643702400"; d="scan'208";a="510896960" Received: from orsmsx605.amr.corp.intel.com ([10.22.229.18]) by orsmga006.jf.intel.com with ESMTP; 01 Mar 2022 23:23:19 -0800 Received: from orsmsx610.amr.corp.intel.com (10.22.229.23) by ORSMSX605.amr.corp.intel.com (10.22.229.18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.21; Tue, 1 Mar 2022 23:23:19 -0800 Received: from ORSEDG602.ED.cps.intel.com (10.7.248.7) by orsmsx610.amr.corp.intel.com (10.22.229.23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.21 via Frontend Transport; Tue, 1 Mar 2022 23:23:19 -0800 Received: from NAM11-DM6-obe.outbound.protection.outlook.com (104.47.57.172) by edgegateway.intel.com (134.134.137.103) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2308.20; Tue, 1 Mar 2022 23:23:10 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=I0pSr9MPmhcZnqBHZdkwhP5I3ivub2p9ElC82/x3cN4SjCFelznfhCyo3gRC91RjagkmOAzS+cA1M/8SnQfZGInUNPxeIHxjSJ4zJmTN4wNK7wEEvKHuHWzn49jMRklIvpDFdSacy+j8wU3TA+IHfu9nVBdhWBYpDS0iN7RAh2zysIDeiRW9kR9A9zV7j9SyV6sLrYb7gr2CmJ8qYv5oIR6vOThd8TDhrSa9Ak69DoMV6b3lfrHb3qm5ILvCnBP8zQdQY7Isi4RdIN4JKhlQHXztR4DRwdVuaQNNSZha1bp87MyiTSaDtEDu8nhA1yyx8d9ULdSIx4Il57ylL/zxfA== 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=DCqkFmQa+Ly/A/t5d/oczcc6ZmdFfaiQuhY3rynQYA8=; b=Pt5iFdPAAwDDkBcc1WH65JR4YlvFI5wEO+M33VZZ+dlPq77xAxGe77qD//IxEFK3SQj0Saszp63CTJgkX1Tj9yXK922V/KoSJOmJFNkAGBgAxiTktqx65cwNEjwEZlrnCWf9aO8hCoe+0weMILYcjcxwyF87pnCroA+CJEKYr6Bq2nbQ0gs6ogndM/K0jjimk4ocD8wiV4RLkVrxhjHzXy+QcmcQzHR+6ywSrL/tjvCXF70e9v4BGZ5BNJyA16GjzJchJD5x4iKfmc1FC80YN/UltCe/xDVoKyutac9Ok8GEW8dXbUwZ/AAGivyT5Z/TBg40ORXCmAuKKJug9gawCA== 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 BN9PR11MB5417.namprd11.prod.outlook.com (2603:10b6:408:11e::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5017.25; Wed, 2 Mar 2022 07:23:08 +0000 Received: from BN9PR11MB5515.namprd11.prod.outlook.com ([fe80::9b4:2af2:de92:517f]) by BN9PR11MB5515.namprd11.prod.outlook.com ([fe80::9b4:2af2:de92:517f%5]) with mapi id 15.20.5017.027; Wed, 2 Mar 2022 07:23:08 +0000 From: "Xiang, Haihao" To: "ffmpeg-devel@ffmpeg.org" Thread-Topic: [FFmpeg-devel] [PATCH] avfilter/framepool: fix alignment requirements for audio and video filters Thread-Index: AQHYJx40jOa06WIGykaZIS12USnm7KyqKTuAgABtHYCAASgmAA== Date: Wed, 2 Mar 2022 07:23:08 +0000 Message-ID: References: <20220221122715.424475-1-onemda@gmail.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: 840b26b5-b565-42a8-9cff-08d9fc1d8072 x-ms-traffictypediagnostic: BN9PR11MB5417: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: skAb+1qYd+5M+J6RPh+Bc0FV0nORT+aOYC73KEvYjlKmO8E+7nBSo36Ghl6tk7Axp6/rQozVjtHjdSjd6vYGjUeijqizx5+RR0jgo30T3AM9uFexqBr2wCDI/PAFGOUGnDAHGyKyb04g4GV3RpzLsny9HRWDzT+VXQFWQNK5u10wYc+EUwYsBsQLpQKncYb05DJAj4gAedNCYmrxpyJty66HjUPOGppAaRXT0p1gndPwW3pE6EoLnHT5K54bFSD14IKNzA1cfVvKZXE7abHp6rV3nPviXTBgQDjhMJ3s0TIHInKKAN4E1ciXGZvoAw47Wn2EQBdzIUcbyr/l3Ypu423LWK+g5EuRiP87yiu+qDQbzoL1zlpUElWWmrcN71pYzPF2Tnz2n/8uH2SKxCAy1GocT6pqNup5wq34XnUGHnSSE0ypG966B+z/D2FZ/0biwSZrRy/oIVog0AaD1hVauqbmnRYHRi5zGZa2Iu5g8x5ch9US0zNzM+YdYfpoeOIFhuCTQfHx74loNuKGiyFZBVC87O3uw1KehgiwEJAF6xE97hbx3RXfumCeaQU7wy4baBSi8oSr20rtp9h8saCl/TMubkc0CGmhhMkaHjBe9foPKZ1rjmbAF/zN+UfHkGigdU03830JQuKWZub/eH642O8VS94hrJtTBEEXcGyMqTqdTT3m8mxp3IZ0JRZD8Tp+CeAvX/ND5lldth5vo/U0UCnmtnzu+mEhkK7V5uE5QyI= 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)(6512007)(6506007)(53546011)(6916009)(6486002)(316002)(71200400001)(508600001)(38100700002)(122000001)(4001150100001)(8936002)(36756003)(2906002)(8676002)(26005)(186003)(2616005)(82960400001)(5660300002)(38070700005)(91956017)(66556008)(66476007)(86362001)(76116006)(66946007)(66446008)(64756008)(99106002); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?Vk54aHJ4MmlSNkNmVHNjMFBISnF2Z2NPK2UxUVhJRlBiK2l5TFAzZmRVMjJM?= =?utf-8?B?TVQxODhkV3Roak95TFBJUENxMnVVdFAvdjA3bFphRlBXRlk4MFk0OUJxb1lM?= =?utf-8?B?YkRwSmJQMWh4RjlSNEtPNCtYOUVLczBVUktkeDk2TnFGSDlIbFNCZXpZSUF2?= =?utf-8?B?bUNCeERVVUhUOHhvYW0yVzdCcHNNVFJPVDc1UTFQRmR2emNBMDZvY2ptd2ta?= =?utf-8?B?RzlpQkZJd2phSlR1d0dHcGR3cFhnc3BRWkM1WHFvY0hEVU9UQXJqdThhUVRn?= =?utf-8?B?Qy8rVmpMaVFIRTdQTFk4MTc4emZ4TWsvK1p5UmNKQzd4OVJpSWdDbDg3alRU?= =?utf-8?B?R1lUU2xVTlVNS21USUwrcXVtYlA1RVdVMWVFWU4rOUhKR2VZRXhHNDZFOHBK?= =?utf-8?B?SXZ2c3lpd1pjVXRkNVhDd0tkK0xWeEVoNlZuU3FBeWozaEJieW5XWm90OXJT?= =?utf-8?B?UWxtVW82dmUwbGtiS0tySXkzelhRSWpaVU1mclF3N2FtTDhacm5tYUdQWXVw?= =?utf-8?B?Z1Fadjc4MjVDZXY5RjBXc1VqaVZ4ZCtQbytGRHRLRWJqL3NBVm10Ym9IRXp4?= =?utf-8?B?UnlMRnk3UUhyVGJRYkkydmpUNXJhc3IrL1FINjYyYnpUeGZtbDRCYWZ5TWhx?= =?utf-8?B?VXorbDVtVWtXNmZWN0tTUHNBWWxVelIrM1VSSi9nQWVoR2NwMk9BcGJFQ1RC?= =?utf-8?B?cm5ydzFET0t0TW9MaW9ldHNlbStnSEdzaC8wZWMyVjZmS3hiM3IwbFh3WUhu?= =?utf-8?B?OW9JWXdLTG1pQkozK212K2sxaTQydkJ5ZjJuZEJyTkhvcllVZjlVVUtHb0hu?= =?utf-8?B?UXNJMm9pS2l6N3htWlV1QnljZS9sbWRDYmtWOXM1UUp0Ni93S0E0ZFBqM2J6?= =?utf-8?B?MnRvdGJGZHIzNDZTWjlHZjNtczlIRWw1Nld5Z0VVNldyWGNOb2Y2MHdpY3Q0?= =?utf-8?B?WDJxdU1pMVMyVXVtcWRKSk9keXZPeDVBRDg1L2hHb2kzTXNBSFR0bFFoT2ha?= =?utf-8?B?TGNBa01tdTlha0JuVmlNZWlIZkJRVTY0V1FDeHExcGI4dENQeHAzT0xDMkpH?= =?utf-8?B?K2dXa3ZjbXpMWUdwc2IySXY0bGI0NncycHF6QzZGRVhBM3NSRmdUVGtlc20z?= =?utf-8?B?VXgzMXUyRDNDeXArUVpRSXlKZFg4NS9KZmpRbExNYWdoRHN6c29hajZaVWtF?= =?utf-8?B?Y3dpOHJyU2kzcHFQSTFtWGZ5U3NrYkc3TE0vYkVKUWRRWWRJQmhJTzNkZHRn?= =?utf-8?B?dUpyVGgvMnhhdlptVlR5SUs1b1V3RGplWHhVcU9Rb3FJcEQrWXJZcThyekdu?= =?utf-8?B?SFI2dDUxT3o5aWQ1b1VQWTY2WWZ3d1NteHBxdXZtZFZPRUNyL1BVNTNwWXFC?= =?utf-8?B?QnFXdzNwUEdFTDdPZmkxQ3hIMEg1RTUxdm02Q21uSFMvSW5TOWhnUGl1eDV6?= =?utf-8?B?NWhjRzV3TVl2NVFOSCtnYjgyYitNRHZSMVh0OURtVWxLV0tQai8zVS9ZYVB3?= =?utf-8?B?bSs1b1d2QVpyWjNMNS9ZSjVvQzVBUDRheExuWnFVb1pFdW93THkyY0t4Tjkw?= =?utf-8?B?L3N1T0NLRTNvRXJqYVJrNlBTNjhJOStWUDFOMERqTlNJYnpadFp2b3JYZ2tt?= =?utf-8?B?ZTQwTkRWOE1NV2NuanJoU1I3SlROWjFENUdOVFJGdkk3dGRjY1c5WUZuN3lo?= =?utf-8?B?L1RneWFkcnVVNjRIRzRXMDhPQ0dvUCtsV1VOK1ZEZzRTUTE2NUxRTGFVd0hM?= =?utf-8?B?alBpSFNtWWJMQU1FT01vVkJJNktkcVZLZUNTalZzZFY2Z05KSmY0N2o2Ni9p?= =?utf-8?B?ZVdmUzlySURFc2NWVzBIaldTbndQamJEaHdQZytEd2lNa3lIVy9DL1hrN1Ji?= =?utf-8?B?cmM2aUZ2TkY2WGVwL3dZTGJWa053YWF3WjJiSlYvTU5sbEo3a3UyYmorYnd0?= =?utf-8?B?Q1JJL0FqK3RvZTFJVGpTZkljUFBFWmRZdTRHcm1PZWV3T1JibjYrT3VnaDA1?= =?utf-8?B?WDlheTFCcmgwYjRkRjI4U0xYT1NWeDFxVUVKbk5najF6VG9rVU9ySHcxa2px?= =?utf-8?B?YlpBWmIza1ZXOEFRb2h6QXFEM21XMW9SSUdBZ2Izd214ZXB3MjY2ajdGaUZ4?= =?utf-8?B?RjdFaXQ2eEY4amUzTktrMzVURFlQZE14ZGZaNnQvQVpEQjdiL1YzVFRwcExG?= =?utf-8?Q?I787mMZPuIgmUB+XBoPiDfk=3D?= Content-ID: <6DC218D774209249A9607476C4322603@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: 840b26b5-b565-42a8-9cff-08d9fc1d8072 X-MS-Exchange-CrossTenant-originalarrivaltime: 02 Mar 2022 07:23:08.7944 (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: 8LS1STVb7N32gi29e0obLmqtUctD/HRLvcX7E0/AAZ+Pz4kMZNmiW0jwMBICycSk5atB9er+IfHWnZSqo22HXQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN9PR11MB5417 X-OriginatorOrg: intel.com Subject: Re: [FFmpeg-devel] [PATCH] avfilter/framepool: fix alignment requirements for audio and video filters 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 Tue, 2022-03-01 at 10:43 -0300, James Almer wrote: > On 3/1/2022 4:12 AM, Xiang, Haihao wrote: > > Hi, > > > > fate-vsynth1-mpeg2-422 is broken after this patch (commit 17a59a6). > > > > --- ./tests/ref/vsynth/vsynth1-mpeg2-422 2021-11-22 > > 12:59:25.013517219 +0800 > > +++ tests/data/fate/vsynth1-mpeg2-422 2022-03-01 13:30:48.446378312 +0800 > > @@ -1,4 +1,4 @@ > > -6e135a1a27235a320311a932147846b4 *tests/data/fate/vsynth1-mpeg2- > > 422.mpeg2video > > -730780 tests/data/fate/vsynth1-mpeg2-422.mpeg2video > > -0273cd8463d1fc115378748239951560 *tests/data/fate/vsynth1-mpeg2- > > 422.out.rawvideo > > -stddev: 10.27 PSNR: 27.90 MAXDIFF: 162 bytes: 7603200/ 7603200 > > +ac2bfd738a93180bd3cb285367236d85 *tests/data/fate/vsynth1-mpeg2- > > 422.mpeg2video > > +702035 tests/data/fate/vsynth1-mpeg2-422.mpeg2video > > +b8db8de89c37fab3b9a0248c2820a5ed *tests/data/fate/vsynth1-mpeg2- > > 422.out.rawvideo > > +stddev: 10.30 PSNR: 27.87 MAXDIFF: 167 bytes: 7603200/ 7603200 > > > > configuration: --prefix=/usr --libdir=/usr/lib/x86_64-linux-gnu -- > > shlibdir=/usr/lib/x86_64-linux-gnu --disable-static --enable-shared -- > > enable-gpl > > > > It works with option --disable-ssse3 > > > > Below is the info for my CPU > > > > $ cat /proc/cpuinfo | grep 'model name' | head -1 > > model name : Intel(R) Core(TM) i5-1035G1 CPU @ 1.00GHz > > > > Thanks > > Haihao > > Does this help? > > > diff --git a/libavfilter/framepool.c b/libavfilter/framepool.c > > index 5b510c9af9..8258a79845 100644 > > --- a/libavfilter/framepool.c > > +++ b/libavfilter/framepool.c > > @@ -90,7 +90,7 @@ FFFramePool *ff_frame_pool_video_init(AVBufferRef* > > (*alloc)(size_t size), > > } > > > > for (i = 0; i < 4 && pool->linesize[i]; i++) { > > - int h = pool->height; > > + int h = FFALIGN(pool->height, align); > > if (i == 1 || i == 2) > > h = AV_CEIL_RSHIFT(h, desc->log2_chroma_h); > > It's the only behavior change i could find in the patch that could make > a difference. > The above change doesn't work for me. Thanks Haihao _______________________________________________ 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".