From patchwork Thu Jun 1 19:17:20 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Dwivedi, Avaneesh Kumar (avani)" X-Patchwork-Id: 9760911 X-Patchwork-Delegate: agross@codeaurora.org Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id CBD5660375 for ; Thu, 1 Jun 2017 19:18:31 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id B9D2028504 for ; Thu, 1 Jun 2017 19:18:31 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id AE80028518; Thu, 1 Jun 2017 19:18:31 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=2.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_HI,T_DKIM_INVALID autolearn=unavailable version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id DF4F328504 for ; Thu, 1 Jun 2017 19:18:29 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751284AbdFATRp (ORCPT ); Thu, 1 Jun 2017 15:17:45 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:45046 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751097AbdFATRn (ORCPT ); Thu, 1 Jun 2017 15:17:43 -0400 Received: by smtp.codeaurora.org (Postfix, from userid 1000) id BC8C9607DF; Thu, 1 Jun 2017 19:17:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1496344662; bh=V5FmTVhZ4wF31vvPU64tARH9uQE0lRSeDAlpRWqiBKw=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=GO86kNfjvn/xWqG5s9/u0oV858FU61B+RypFr5AzsPbh2iNqJLj0y9hJfidlCIvE0 yvz7i2A3fmtjQEXsp5g52x7nlmmS3uCkF+hcNME7sEapugmdbdAg5EdXypPAIhCpnA RL4XL3/M0lcWgSA5L+WLQXhoOaUbPUX94R3RSAdE= Received: from akdwived-linux.qualcomm.com (blr-c-bdr-fw-01_globalnat_allzones-outside.qualcomm.com [103.229.19.19]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: akdwived@smtp.codeaurora.org) by smtp.codeaurora.org (Postfix) with ESMTPSA id 2F1C060710; Thu, 1 Jun 2017 19:17:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1496344662; bh=V5FmTVhZ4wF31vvPU64tARH9uQE0lRSeDAlpRWqiBKw=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=GO86kNfjvn/xWqG5s9/u0oV858FU61B+RypFr5AzsPbh2iNqJLj0y9hJfidlCIvE0 yvz7i2A3fmtjQEXsp5g52x7nlmmS3uCkF+hcNME7sEapugmdbdAg5EdXypPAIhCpnA RL4XL3/M0lcWgSA5L+WLQXhoOaUbPUX94R3RSAdE= DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 2F1C060710 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=akdwived@codeaurora.org From: Avaneesh Kumar Dwivedi To: bjorn.andersson@linaro.org Cc: sboyd@codeaurora.org, agross@codeaurora.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, linux-remoteproc@vger.kernel.org, Avaneesh Kumar Dwivedi Subject: [PATCH v5 3/4] remoteproc: qcom: Make secure world call for mem ownership switch Date: Fri, 2 Jun 2017 00:47:20 +0530 Message-Id: <1496344641-6291-4-git-send-email-akdwived@codeaurora.org> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1496344641-6291-1-git-send-email-akdwived@codeaurora.org> References: <1496344641-6291-1-git-send-email-akdwived@codeaurora.org> Sender: linux-arm-msm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP MSS proc on msm8996 can not access fw loaded region without stage second translation of memory pages where mpss image are loaded. This patch in order to enable mss boot on msm8996 invoke scm call to switch or share ownership between apps and modem. Signed-off-by: Avaneesh Kumar Dwivedi --- drivers/remoteproc/qcom_q6v5_pil.c | 81 +++++++++++++++++++++++++++++++++++++- 1 file changed, 79 insertions(+), 2 deletions(-) diff --git a/drivers/remoteproc/qcom_q6v5_pil.c b/drivers/remoteproc/qcom_q6v5_pil.c index f5f8850..266efad 100644 --- a/drivers/remoteproc/qcom_q6v5_pil.c +++ b/drivers/remoteproc/qcom_q6v5_pil.c @@ -110,6 +110,7 @@ struct rproc_hexagon_res { struct qcom_mss_reg_res *active_supply; char **proxy_clk_names; char **active_clk_names; + bool need_mem_protection; }; struct q6v5 { @@ -151,6 +152,7 @@ struct q6v5 { phys_addr_t mpss_reloc; void *mpss_region; size_t mpss_size; + bool need_mem_protection; struct qcom_rproc_subdev smd_subdev; }; @@ -288,6 +290,40 @@ static struct resource_table *q6v5_find_rsc_table(struct rproc *rproc, return &table; } +static int q6v5_xfer_mem_ownership(struct q6v5 *qproc, + int image, phys_addr_t addr, + size_t size) +{ + static int current_owner[3][1] = {{BIT(QCOM_SCM_VMID_HLOS)}, + {BIT(QCOM_SCM_VMID_HLOS)}, + {BIT(QCOM_SCM_VMID_HLOS)} }; + struct qcom_scm_vmperm next[] = {{0} }; + int ret; + + if (!qproc->need_mem_protection) + return 0; + + if (current_owner[image][0] == BIT(QCOM_SCM_VMID_HLOS)) { + next->vmid = QCOM_SCM_VMID_MSS_MSA; + next->perm = QCOM_SCM_PERM_RW; + } else { + next->vmid = QCOM_SCM_VMID_HLOS; + next->perm = QCOM_SCM_PERM_RWX; + } + + ret = qcom_scm_assign_mem(addr, ALIGN(size, SZ_4K), + current_owner[image][0], next, 1); + if (ret < 0) { + pr_err("Failed to assign %s memory access in range %p to %p ret = %d\n", + (image == 0 ? "MDT" : image == 1 ? "MBA" : "MPSS"), + (void *)addr, (void *)(addr + size), ret); + return ret; + } + + current_owner[image][0] = ret; + return 0; +} + static int q6v5_load(struct rproc *rproc, const struct firmware *fw) { struct q6v5 *qproc = rproc->priv; @@ -450,6 +486,7 @@ static int q6v5_mpss_init_image(struct q6v5 *qproc, const struct firmware *fw) { unsigned long dma_attrs = DMA_ATTR_FORCE_CONTIGUOUS; dma_addr_t phys; + int xferop_ret; void *ptr; int ret; @@ -461,6 +498,10 @@ static int q6v5_mpss_init_image(struct q6v5 *qproc, const struct firmware *fw) memcpy(ptr, fw->data, fw->size); + /* Hypervisor mapping to access metadata by modem */ + ret = q6v5_xfer_mem_ownership(qproc, 0, phys, fw->size); + if (ret) + return -EAGAIN; writel(phys, qproc->rmb_base + RMB_PMI_META_DATA_REG); writel(RMB_CMD_META_DATA_READY, qproc->rmb_base + RMB_MBA_COMMAND_REG); @@ -470,6 +511,11 @@ static int q6v5_mpss_init_image(struct q6v5 *qproc, const struct firmware *fw) else if (ret < 0) dev_err(qproc->dev, "MPSS header authentication failed: %d\n", ret); + /* Metadata authentication done, remove modem access */ + xferop_ret = q6v5_xfer_mem_ownership(qproc, 0, phys, fw->size); + if (xferop_ret) + dev_warn(qproc->dev, + "mdt buffer not reclaimed system may become unstable\n"); dma_free_attrs(qproc->dev, fw->size, ptr, phys, dma_attrs); return ret < 0 ? ret : 0; @@ -579,6 +625,10 @@ static int q6v5_mpss_load(struct q6v5 *qproc) /* Transfer ownership of modem ddr region with q6*/ boot_addr = relocate ? qproc->mpss_phys : min_addr; + ret = q6v5_xfer_mem_ownership(qproc, 2, + qproc->mpss_phys, qproc->mpss_size); + if (ret) + return -EAGAIN; writel(boot_addr, qproc->rmb_base + RMB_PMI_CODE_START_REG); writel(RMB_CMD_LOAD_READY, qproc->rmb_base + RMB_MBA_COMMAND_REG); writel(size, qproc->rmb_base + RMB_PMI_CODE_LENGTH_REG); @@ -598,6 +648,7 @@ static int q6v5_mpss_load(struct q6v5 *qproc) static int q6v5_start(struct rproc *rproc) { struct q6v5 *qproc = (struct q6v5 *)rproc->priv; + int xfermemop_ret; int ret; ret = q6v5_regulator_enable(qproc, qproc->proxy_regs, @@ -633,6 +684,11 @@ static int q6v5_start(struct rproc *rproc) goto assert_reset; } + /* Assign MBA image access in DDR to q6 */ + xfermemop_ret = q6v5_xfer_mem_ownership(qproc, 1, + qproc->mba_phys, qproc->mba_size); + if (xfermemop_ret) + goto assert_reset; writel(qproc->mba_phys, qproc->rmb_base + RMB_MBA_IMAGE_REG); ret = q6v5proc_reset(qproc); @@ -654,16 +710,21 @@ static int q6v5_start(struct rproc *rproc) ret = q6v5_mpss_load(qproc); if (ret) - goto halt_axi_ports; + goto reclaim_mem; ret = wait_for_completion_timeout(&qproc->start_done, msecs_to_jiffies(5000)); if (ret == 0) { dev_err(qproc->dev, "start timed out\n"); ret = -ETIMEDOUT; - goto halt_axi_ports; + goto reclaim_mem; } + xfermemop_ret = q6v5_xfer_mem_ownership(qproc, 1, + qproc->mba_phys, qproc->mba_size); + if (xfermemop_ret) + dev_err(qproc->dev, + "Failed to reclaim mba buffer system may become unstable\n"); qproc->running = true; q6v5_clk_disable(qproc->dev, qproc->proxy_clks, @@ -673,12 +734,21 @@ static int q6v5_start(struct rproc *rproc) return 0; +reclaim_mem: + xfermemop_ret = q6v5_xfer_mem_ownership(qproc, 2, + qproc->mpss_phys, qproc->mpss_size); + WARN_ON(xfermemop_ret); halt_axi_ports: q6v5proc_halt_axi_port(qproc, qproc->halt_map, qproc->halt_q6); q6v5proc_halt_axi_port(qproc, qproc->halt_map, qproc->halt_modem); q6v5proc_halt_axi_port(qproc, qproc->halt_map, qproc->halt_nc); q6v5_clk_disable(qproc->dev, qproc->active_clks, qproc->active_clk_count); + xfermemop_ret = q6v5_xfer_mem_ownership(qproc, 1, + qproc->mba_phys, qproc->mba_size); + if (xfermemop_ret) + dev_err(qproc->dev, "Failed to reclaim mba buffer, system may become unstable\n"); + assert_reset: reset_control_assert(qproc->mss_restart); disable_vdd: @@ -698,6 +768,7 @@ static int q6v5_stop(struct rproc *rproc) { struct q6v5 *qproc = (struct q6v5 *)rproc->priv; int ret; + u32 val; qproc->running = false; @@ -715,6 +786,9 @@ static int q6v5_stop(struct rproc *rproc) q6v5proc_halt_axi_port(qproc, qproc->halt_map, qproc->halt_modem); q6v5proc_halt_axi_port(qproc, qproc->halt_map, qproc->halt_nc); + ret = q6v5_xfer_mem_ownership(qproc, 2, + qproc->mpss_phys, qproc->mpss_size); + WARN_ON(ret); reset_control_assert(qproc->mss_restart); q6v5_clk_disable(qproc->dev, qproc->active_clks, qproc->active_clk_count); @@ -1012,6 +1086,7 @@ static int q6v5_probe(struct platform_device *pdev) if (ret) goto free_rproc; + qproc->need_mem_protection = desc->need_mem_protection; ret = q6v5_request_irq(qproc, pdev, "wdog", q6v5_wdog_interrupt); if (ret < 0) goto free_rproc; @@ -1087,6 +1162,7 @@ static int q6v5_remove(struct platform_device *pdev) "mem", NULL }, + .need_mem_protection = false, }; static const struct rproc_hexagon_res msm8974_mss = { @@ -1124,6 +1200,7 @@ static int q6v5_remove(struct platform_device *pdev) "mem", NULL }, + .need_mem_protection = false, }; static const struct of_device_id q6v5_of_match[] = {