Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Peng Fan <peng.fan@nxp.com>
To:     bjorn.andersson@linaro.org, ohad@wizery.com
Cc:     linux-remoteproc@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-imx@nxp.com, Peng Fan <peng.fan@nxp.com>
Subject: [PATCH 1/2] remoteproc: drop memset when loading elf segments
Date:   Thu,  9 Apr 2020 16:22:51 +0800
Message-Id: <1586420572-28353-1-git-send-email-peng.fan@nxp.com>
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from localhost.localdomain (119.31.174.66) by SG2PR01CA0133.apcprd01.prod.exchangelabs.com (2603:1096:4:8f::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.20.2900.15 via Frontend Transport; Thu, 9 Apr 2020 08:30:46 +0000
X-Mailer: git-send-email 2.7.4
X-Originating-IP: [119.31.174.66]
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-HT: Tenant
X-MS-Office365-Filtering-Correlation-Id: e1a87978-b1a2-4e7f-ae33-08d7dc604e84
X-MS-TrafficTypeDiagnostic: AM0PR04MB6626:|AM0PR04MB6626:
X-MS-Exchange-Transport-Forked: True
X-Microsoft-Antispam-PRVS: <AM0PR04MB662635AC2EB4C4122C119B9788C10@AM0PR04MB6626.eurprd04.prod.outlook.com>
X-MS-Oob-TLC-OOBClassifiers: OLM:7691;
X-Forefront-PRVS: 0368E78B5B
X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR04MB4481.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:(10009020)(4636009)(346002)(376002)(396003)(136003)(366004)(39860400002)(6486002)(86362001)(478600001)(44832011)(316002)(2906002)(6666004)(36756003)(6506007)(66946007)(66556008)(52116002)(66476007)(16526019)(81156014)(26005)(186003)(8936002)(8676002)(956004)(5660300002)(6512007)(81166007)(4326008)(2616005)(69590400007);DIR:OUT;SFP:1101;
Received-SPF: None (protection.outlook.com: nxp.com does not designate
 permitted sender hosts)
X-MS-Exchange-SenderADCheck: 1
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: zIC8FVJKBh4jaBz4esdgjZbu/6tPeQU+9cJcRDMbEZ5czykDXglF1EJ6Tm7cmV0iD0dnWRK9k942AHQkAiEcG9mu9KSm1rzW+51SxLj7mIWc7K25XArue2tbV53hJKFukdmiX0uu67leQ9TW4j/JcZxSAPBvUqvudXz/ZVLwuYDBnZ/5zaTuiDf931Q3SEGgxmXpN1UH6zRr/L9zuPHb/MpD2OsyyNfpGX5mrqVqxq4xRtycbI7y+KclLcJ7YlUOjdDXdFvPzEEa/ymZFaHRDbUahO2pOWKnDYT/IfPx7Se/zN+ojyZjOOYpBpkLM69CDpX5OzQmNSQpw4e2ju3cwN/AAW1uBVE3xqYfWcdj17ZDbiX+HMp6GMMRQ+JVopc9HBiV1/MFXjjusA+ukv43K32h8EBYcu471aFLwEpfCp1LP95fcDKeOAsFlSYr3Vpr5ZAnoU140Zvbd5w/2osbF257m29O0tKXyioE2VqHFvzqfPeq/l7Bfs1ukYBCCrvA
X-MS-Exchange-AntiSpam-MessageData: C0fAcnpNh53lA2gx1OxrGFLMR4cAPLyL2Z5ktLlQd2db+6EpkEUCNTxqG65G6l5BJBYIHX9kjDLC88g0LnAHunNHphXBq64bKvNTPT3gwLiXeBM95ISf9mFojJlheMb/8Qw1T9Py62nPmbZBLZWxqg==
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e1a87978-b1a2-4e7f-ae33-08d7dc604e84
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Apr 2020 08:30:49.1592
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: fu2aQLGtfypnF4wLZR5CcTX6hmrSsAza6vjdCXjj1nfIq7NXdeL2KCrIMNhmXSw10Kt+2NAc73gqpuQ6W064Hg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR04MB6626
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

To arm64, "dc      zva, dst" is used in memset.
Per ARM DDI 0487A.j, chapter C5.3.8 DC ZVA, Data Cache Zero by VA,

"If the memory region being zeroed is any type of Device memory,
this instruction can give an alignment fault which is prioritized
in the same way as other alignment faults that are determined
by the memory type."

On i.MX platforms, when elf is loaded to onchip TCM area, the region
is ioremapped, so "dc zva, dst" will trigger abort.

Since memset is not strictly required, let's drop it.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
 drivers/remoteproc/remoteproc_elf_loader.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/drivers/remoteproc/remoteproc_elf_loader.c b/drivers/remoteproc/remoteproc_elf_loader.c
index 16e2c496fd45..cc50fe70d50c 100644
--- a/drivers/remoteproc/remoteproc_elf_loader.c
+++ b/drivers/remoteproc/remoteproc_elf_loader.c
@@ -238,14 +238,11 @@ int rproc_elf_load_segments(struct rproc *rproc, const struct firmware *fw)
 			memcpy(ptr, elf_data + offset, filesz);
 
 		/*
-		 * Zero out remaining memory for this segment.
+		 * No need zero out remaining memory for this segment.
 		 *
 		 * This isn't strictly required since dma_alloc_coherent already
-		 * did this for us. albeit harmless, we may consider removing
-		 * this.
+		 * did this for us.
 		 */
-		if (memsz > filesz)
-			memset(ptr + filesz, 0, memsz - filesz);
 	}
 
 	if (ret == 0)
-- 
2.16.4