From nobody Sun Feb 8 20:35:34 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 73954C001DC for ; Wed, 19 Jul 2023 06:33:27 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230348AbjGSGdZ (ORCPT ); Wed, 19 Jul 2023 02:33:25 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55902 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230143AbjGSGdX (ORCPT ); Wed, 19 Jul 2023 02:33:23 -0400 Received: from APC01-PSA-obe.outbound.protection.outlook.com (mail-psaapc01on2118.outbound.protection.outlook.com [40.107.255.118]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9A4611FCB; Tue, 18 Jul 2023 23:33:21 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=lFaoU1KckSwsbkAZXevH0XK34kzdsMYq4pQFJL52hdGZLZozdeyjGN9ChmQ5xLJt0emUF42TpP/33cdemZ4stU7ZL+6Az7WqsAKpPS+5Rn0wYeEZABXUl48au1ePw1A62Kw3xSdpN/txFaDBgCldviLGWLhoasie49YVrNvfhChD39MmQ2993OJGaVD14z0xv266XEwVtsbag5kuM6gurkyWmb5qUCRMyesmd+jj2AqcMc0ERyyMYm0jyUMHBc5OXMMaUdlMD/gtCPu+3xLw5cxO0p6GYBB3Yw/jM34i8nL4IrYZhNv0y35zMwPsefUlr6YRqza4OCOD+5jDiAdCcA== 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=pQbB9AeJxrvjKq2Af2sz4lX/ZqNpRW1aGmgojZJpt7E=; b=b+sBq5reIA2kGvfbrIHHRIGxKzn4ebqSAqvNRhNJEI2MC/dkowuHnC0gNPSrf9hUqFYoFOgszEaeucn3fYRoBmfWrLmEhCvvHSsarCZcJ1nKLkDFsY/1y1hf0IlDnzXKXyN7TDtOc5z4Mw/nQQdTbeN0Wu7Y8EPDR6vPnX2OmlOKbgcdgalaYLK/+FUsUIiwuE2ep28zMzqm8ZYZFSRqPxdLtRn2X8L0D5hH7sN2qQALdtaWdalz99jlCMUXVkKnA+JzI5BDwvnFGNo93QO68JyHAsrrdReCli7HN4+L1zeEeZ2rnhkhuN5ThteIPASgWCW426ErCqmsGQQK+9bYwQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=aspeedtech.com; dmarc=pass action=none header.from=aspeedtech.com; dkim=pass header.d=aspeedtech.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=aspeedtech.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=pQbB9AeJxrvjKq2Af2sz4lX/ZqNpRW1aGmgojZJpt7E=; b=cddBzSLcZfykKYNwjserqse2JvLHIds6bWyrFNGJj5drgFy9WFeFOthoRguP3uY/88LuHKGbpOA6AKLKBgnNVKycqFZvnJwE+NIlO3ghplPoZlVa0c1qlTzdXYXw4KWn1JluG1cDJSEczkAaAyz2gwU3tTmZVAy/yv7LUXYMhQS5T1hqsJbbUf4yxmZzJCz+ZJIwJdKur3L6CChSsJwV4TCIo0bIs2xXcFsLv8d0OdhURPFqCnPtehLATAJnblqrobcpowRezxrjX3C+03+O5+hgVldv0m5w6m+diqFiW9ItsUVFCyrwxMjcSLkLmzLXfwEg5hL/mSznWJR8N5nElA== Received: from TYZPR06MB6568.apcprd06.prod.outlook.com (2603:1096:400:45f::6) by SI2PR06MB4410.apcprd06.prod.outlook.com (2603:1096:4:15e::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6588.33; Wed, 19 Jul 2023 06:33:18 +0000 Received: from TYZPR06MB6568.apcprd06.prod.outlook.com ([fe80::72f5:eab8:7757:7c49]) by TYZPR06MB6568.apcprd06.prod.outlook.com ([fe80::72f5:eab8:7757:7c49%6]) with mapi id 15.20.6588.031; Wed, 19 Jul 2023 06:33:18 +0000 From: Jammy Huang To: Hans Verkuil , "eajames@linux.ibm.com" , "mchehab@kernel.org" , "joel@jms.id.au" , "andrew@aj.id.au" , "linux-media@vger.kernel.org" , "openbmc@lists.ozlabs.org" , "linux-arm-kernel@lists.infradead.org" , "linux-aspeed@lists.ozlabs.org" , "linux-kernel@vger.kernel.org" Subject: [PATCH RESEND v2] media: aspeed: Fix memory overwrite if timing is 1600x900 Thread-Topic: [PATCH RESEND v2] media: aspeed: Fix memory overwrite if timing is 1600x900 Thread-Index: AQHZugrnPAu3uBx5akaz+7fGPVJhRg== Date: Wed, 19 Jul 2023 06:33:18 +0000 Message-ID: References: <20230717095111.1957-1-jammy_huang@aspeedtech.com> <21e13f35-5caf-6d60-c388-3386e3540834@xs4all.nl> In-Reply-To: Accept-Language: zh-TW, en-US Content-Language: zh-TW X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=aspeedtech.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: TYZPR06MB6568:EE_|SI2PR06MB4410:EE_ x-ms-office365-filtering-correlation-id: 0642e748-6873-4f04-223b-08db88220a26 x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: NjgC0TbLCCTdYVUm7lxRZNtKNBrHAwEsLnpvX57rof3NfzeZsRK+0K+wCfFTGRX3hCFN4H5lsf0VizKrSwfUXkklysO/RvWT082u1W707t9XU3Xdaq1XjE4xeCiWdykRQKoKhToRagbDmnM0XFiAZzOvxEovyFkSPTqzXkfZ0u1DmVXaPn5wigr/hk3S7t3p4jl34rz9Xnl+SLBmBoKa/kzRlSCTE4cMkQhF6K/g28T4CpJEqbd1bRVWxvyBEorPvN6cxMoDVZGRMgeeQxNo2KRep6KCzuTld5sfQ4PGLYFNAZqxEegVXOMKm+wbwJQ4MylbxyXYWehR4BAnoAihgxEFjXRpOVo3DxxYFTEvSvJZw85l+qBCC9o1nUY/E+xeWaNWoiHF/N0xEdB6muFp2OAQrcLglNt9mhzagb6syOoFegd2urJXdVtrdniVyJum4fhR2h8pV2icIbP9UxyxXOsildn+Yo5tHXd1ILnuyRfQQhmP2Id472+FZXI59rwitbEsKca/jGA5Fzr+3V2XKARqeKlxyL3xlPMxPdAyNsywnTFcS71+C3icPZW+asN5bxiEQdIffDgaSnSQdxrwGHg2ugyEe4O85rlgfpl3Ex+ErYxgNaHKeovDjZwJPFxYEF5S+9dpkvZ3JJblEpzokw== x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:TYZPR06MB6568.apcprd06.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(366004)(376002)(136003)(39850400004)(396003)(346002)(451199021)(38100700002)(7416002)(8676002)(5660300002)(8936002)(41300700001)(52536014)(7696005)(2906002)(316002)(110136005)(478600001)(71200400001)(83380400001)(2940100002)(186003)(6506007)(26005)(9686003)(76116006)(66556008)(66476007)(64756008)(66946007)(66446008)(55016003)(38070700005)(33656002)(86362001)(921005)(122000001);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?R0YveFNiaTM1M0s0QTQvcjFXUWRsRjdMcWh4cktwZnJVeXQ4TEZQT1VkK1FG?= =?utf-8?B?Vkh3YkNTb2M5THYrQ3ZBdW11WC81M20zVUR5WVZVVmQvVVpGOTc3VlBXeXJH?= =?utf-8?B?T0EzdWhyWGhCcWZOQXZWeFVjRVJyOVVJMFh5S0VIYVdoM0VkWlZvSFFUdVgz?= =?utf-8?B?WmR1TzltamdrSVQrdGl5RzV6ck5SM3l6aHZ6ZlhNY29OZUNwc1NzWVM0UmVq?= =?utf-8?B?ZmVTOUI3cG9rMUs4b3lTQVFNY1FwczNjckVlblErNW92emFZOFBJSVZUMTlL?= =?utf-8?B?WUphNklQUXhYMjlUUTE5a09wVEFUZVY3SGpYVnNpbENBaHltUGYrOVM0eEMv?= =?utf-8?B?N2xNM2YrVktLaExrOWloMDdCenJwYTlBWjJja3ZJck1IdTZrRUVYb2pnQmFq?= =?utf-8?B?ZStoVS85NmJXcE42RzRmdGZTcTFzT1QzdEMreU85MWhuMWc2UnYvZENWdk1F?= =?utf-8?B?eFhhcVkyYkNlaDd2QkRWeXU3ZXdBbDZJcjVnNGZaWGN4NytFamtobStqV0M0?= =?utf-8?B?MFh6Z1R0L0ZJZ08xV0FtWTYzN0J5bWw1VlZveEtRUFkrWW4yOVRIWnFHdENw?= =?utf-8?B?VlMzWnQrVGZHWS9FSCtsWkxyUEIrbklhNjNRYWEzQ05WcFVyVEJkVi9VYXN5?= =?utf-8?B?SmZHRTNMY1d4bjErQnpYV0gwSVczZVdxcG1NRTZ4cC8xZzZObFF6RjBkSFNC?= =?utf-8?B?MGt0dFpUc1BwSEhCTU9XWU44Y2lTRWlVY3p4K3NzM3FTdjFmWW84VHIrNFdE?= =?utf-8?B?QWJHRGtSekhMMzBmTkFUeTYvZnU0cmN1S2hQZmxZc3dSZzU3ZzlkSEJJelhr?= =?utf-8?B?MmVkc2kxd2FpdWVLajl0QU9CV2k3VXcxWkZOSkhsQlFNQUFqZWRwZU9jbkNR?= =?utf-8?B?TUtCWml5dXhMa0xMc1VIeHJiamVockZZdjlCZ2ZVVWh1TWJHV1pwcXF4Um0w?= =?utf-8?B?aG9ldndDdUdzbS9mUVBlNXJQOTg5SlN3VGpQTk5BakZEc08rSmdDQkRTc3Jk?= =?utf-8?B?RzNuMlhZdytDVGRIWEk4Uzd2ZGdVR0RFTTdwc25JNWhvRG5zWVZ2b2orbDVP?= =?utf-8?B?Y2NUYVorUy9xcCt3R2xzVk94QnM3RjB1QnJqb0VjVHIxUDRjVU0rckZzRTlh?= =?utf-8?B?aEhBRTVCWUNnUXBPQ3R2U0RMZGZPL3VFaXdFVWZQMlBmSk96L09sRmxoSEZT?= =?utf-8?B?QmdUL00vKzBxZXJxQjNZQVJHaGlGYld2cmFtSzFDV0lqYkpzc2lPcFd4T3Ev?= =?utf-8?B?OERuR1NCSFNlcFN1MGFrWTZJYWpoZWhaYnZkWGYyQzMwN0VWODEybFFNMVlt?= =?utf-8?B?bVZvZGJZbzFobzh3dXFxM283NFB4QkhJbHhseGRydDRtbDlIZ050V05rUmUy?= =?utf-8?B?eTJvQ0lENk92RDd3SHpwQlhlTTBqc3p2OWxaNTRlQThIcGdtZUlTbjV1S1Qz?= =?utf-8?B?LzBBUWdxWEN5akk1aFFWa3pUWUhGcnh4RitOWFo5L2FJZFVNdER2RFY3QzYr?= =?utf-8?B?N2c5K1V2UjNPMm9hbWxtMC8yMkljVUxkalg5ZHoySTVpeFR6SFY0VXNOVllM?= =?utf-8?B?RmpnNWZxQ2x5WDl2ZDlwMVZYZ3RPb2x6Q1RhT1BPRHBnL3NMUEpaR3JQVzZq?= =?utf-8?B?QVJhY2FJWEFCc2hKWGhWS1RUdGtvSXhIZDJJVzQrTW4yd0VNbEV0VjZ2T094?= =?utf-8?B?WkZFb3pRcXd1bUdHNHBuYUEzcUViR2I2bkRJUVRuQkY5RHpFYWNJZWNLMjBL?= =?utf-8?B?OGcrQ1VkV0tsWEd2VkpYNm01ejRBOHNRZEk4YThoWjlza0xwS1Y5UG5GSG0x?= =?utf-8?B?T1lmWWhrMmU4bnpMYStiQUJlNVM4QXlQbzYvOUJQV0hnY3hDSlhkTFFSY0xW?= =?utf-8?B?WmlxTElaMFVkOC91TWp5TDlEdkxKVXJBNXJyR2U3b0lTdXNhWHRIQXZxQnUx?= =?utf-8?B?Vlk1dGV3WmlRYkhzSVlwVEFmazdydy9JSkk5dEpueGRjV3BsRjk4S0VuNzZl?= =?utf-8?B?UTJweTQrNnl1Y3RMY3lXTjVtWm9UeFp6NnBSSTV0WGRyeEUySy9aZUN4VWVj?= =?utf-8?B?N1dZcFNVRU1OYXF6ZGxpcW1UVWs3ZzJMblQva0pHUlhVUlNkYlR5SDBEWk40?= =?utf-8?B?NGJjcEphMnR2V3A3Vzd3Ymg0bVpuQnNnK3VIclFpbjQ4RWwzMGRhZHI1OWRq?= =?utf-8?B?TUE9PQ==?= Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: aspeedtech.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: TYZPR06MB6568.apcprd06.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 0642e748-6873-4f04-223b-08db88220a26 X-MS-Exchange-CrossTenant-originalarrivaltime: 19 Jul 2023 06:33:18.3402 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 43d4aa98-e35b-4575-8939-080e90d5a249 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: YGLpzfaGfyjJqJIavt8zmKS5P+m/PnumZmX+ij2Q6r23JwvBTI6UFT5fIXgnuFNps0FtF9wmONy5qSrtA3QvIh3T8fdgEfEFNWp+ut4cwiY= X-MS-Exchange-Transport-CrossTenantHeadersStamped: SI2PR06MB4410 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org When capturing 1600x900, system could crash when system memory usage is tight. The way to reproduce this issue: 1. Use 1600x900 to display on host 2. Mount ISO through 'Virtual media' on OpenBMC's web 3. Run script as below on host to do sha continuously #!/bin/bash while [ [1] ]; do find /media -type f -printf '"%h/%f"\n' | xargs sha256sum done 4. Open KVM on OpenBMC's web The size of macro block captured is 8x8. Therefore, we should make sure the height of src-buf is 8 aligned to fix this issue. Signed-off-by: Jammy Huang --- v2 changes - Add how to reproduce this issue. --- drivers/media/platform/aspeed/aspeed-video.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform/aspeed/aspeed-video.c b/drivers/media/p= latform/aspeed/aspeed-video.c index 374eb7781936..14594f55a77f 100644 --- a/drivers/media/platform/aspeed/aspeed-video.c +++ b/drivers/media/platform/aspeed/aspeed-video.c @@ -1130,7 +1130,7 @@ static void aspeed_video_get_resolution(struct aspeed= _video *video) static void aspeed_video_set_resolution(struct aspeed_video *video) { struct v4l2_bt_timings *act =3D &video->active_timings; - unsigned int size =3D act->width * act->height; + unsigned int size =3D act->width * ALIGN(act->height, 8); =20 /* Set capture/compression frame sizes */ aspeed_video_calc_compressed_size(video, size); @@ -1147,7 +1147,7 @@ static void aspeed_video_set_resolution(struct aspeed= _video *video) u32 width =3D ALIGN(act->width, 64); =20 aspeed_video_write(video, VE_CAP_WINDOW, width << 16 | act->height); - size =3D width * act->height; + size =3D width * ALIGN(act->height, 8); } else { aspeed_video_write(video, VE_CAP_WINDOW, act->width << 16 | act->height); base-commit: 2605e80d3438c77190f55b821c6575048c68268e --=20 2.25.1