From patchwork Tue Nov 14 09:52:16 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jens Wiklander X-Patchwork-Id: 164798 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:b909:0:b0:403:3b70:6f57 with SMTP id t9csp1752001vqg; Tue, 14 Nov 2023 01:52:43 -0800 (PST) X-Google-Smtp-Source: AGHT+IHv/fh/8r0fMZbsTNs/6MkPRcSVIHR1PQPRX4+uerR1U4KoWyUXfA98Krty0LNE0Ua3ICvD X-Received: by 2002:a17:90b:1b0f:b0:27d:2108:af18 with SMTP id nu15-20020a17090b1b0f00b0027d2108af18mr9003692pjb.25.1699955562994; Tue, 14 Nov 2023 01:52:42 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1699955562; cv=none; d=google.com; s=arc-20160816; b=ZU7LOQMJOELcxKFko3N0oFBXpeby+C2CawBifTK9+7Myd0tjsuaw1sd0vUZRPyK4u6 pM9pgT3as0THP+ZOPx+LeBg62r/uehXbFHiKvyClXc/dm0mjZdB3fInUMlEaxPIbnSen YvGEjNDdSggOyQm1cAIvz5yFX81UA1IO7wftt1vpAs+tNm8hzqYpikvm0SHQgjhG8mxm r3ewQSEMEDnbgN1oQUU1hNkoztmRSZsBUxfWA2GF8kCZdtyqHA5EbWH7a7sWVeNz0uEr GLjbGFoldKmatyyMRLy5vnijtsJedEsW6PqQULas4K9oKaBQW1nH+9XhAiMCQKr2ngdS DG9Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=wLRgBSnIsahghCgCHXz9rGutSrnAaUdwkEkt9d4hS6A=; fh=SSmA14+Y2ZA2PO07peKpbI6W7vxD8MB28Dbup/soyB4=; b=AAXkdoHohQkinPp8qcuqhQnD5nf5S3WYO1Yf0du+84sQdOcGXIN2EnB0iSI9GYWZOY Tnlk4lArMCQBqx8lZt+X7G3Q2iU6zG+bncF1qKyr55dBo9EK21dt/4r1q+Sv5xo2e2lz tLvYGbnVDTXXGDb4hjLztD0o3d+MInwa0ZZWSErSxM/6EtcCaAiziNfW84VfKIAo5lGJ bPK0bG02zFyNDP1y4ijDzFtpuJHpmRkLLjQbrLjk0o+LFxBaaPN5KQ5eD1pcpUltuW72 aNvM8stwprKc68c8d5mfha1cOeaYFXUbfJeJy0gtRDGxxbf1RXJ/LRj6HWsnRazZcBfk CV0A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=cKr0w1Ga; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from snail.vger.email (snail.vger.email. [23.128.96.37]) by mx.google.com with ESMTPS id y5-20020a17090aa40500b0027d1ce4ce41si423165pjp.0.2023.11.14.01.52.42 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 14 Nov 2023 01:52:42 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=cKr0w1Ga; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by snail.vger.email (Postfix) with ESMTP id 9F11880408D9; Tue, 14 Nov 2023 01:52:41 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at snail.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232538AbjKNJwi (ORCPT + 30 others); Tue, 14 Nov 2023 04:52:38 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60204 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229585AbjKNJwg (ORCPT ); Tue, 14 Nov 2023 04:52:36 -0500 Received: from mail-pf1-x436.google.com (mail-pf1-x436.google.com [IPv6:2607:f8b0:4864:20::436]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E4D37191 for ; Tue, 14 Nov 2023 01:52:32 -0800 (PST) Received: by mail-pf1-x436.google.com with SMTP id d2e1a72fcca58-6c115026985so5570288b3a.1 for ; Tue, 14 Nov 2023 01:52:32 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1699955552; x=1700560352; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=wLRgBSnIsahghCgCHXz9rGutSrnAaUdwkEkt9d4hS6A=; b=cKr0w1GaXhCXLGiy4d9VuzTCICQlUl6+Gzg1WfkR7HaCBFBjA1NgEZVPWWIjc2UyGl 5n1jqmPi5yih+aPlXVMK76SF0DKwu9RXulOsw+mBt5Tt/PX+S70xZJu7vLg/6S7yAtiH 1HmeapNBl5h6TcDOZMD20lfQ4sYPQEXrCbIfXOJf2vBKIlI5tiHp77a0qwV557fFgINN rOJxsJnwz8uxoamYEKzMA4tpurUienvc4cIyNZ4wUwFsrWiUEe7TuAtC/VaLNso1JYpW yxQWkJaretPt7jpheMRPcHsQZgw6sElCkWGmDzc+4V1v5rD1MZ+d4gukWOUTs2hvyzc2 bnbw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1699955552; x=1700560352; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=wLRgBSnIsahghCgCHXz9rGutSrnAaUdwkEkt9d4hS6A=; b=o1dAGDEiR1FXAf5iOWKZ22KtsyQwFvHb7j53q3U5iopXogjYXN9+KbOmJWm5NiFqbF PeBa1TN4AyyPusScPdWZcEXNyWGqOUgrSRCoF1gmRzRxOVV6K+2ljxsvzaUu1TYgxQAI koHXICuhxcnkm6Q4Obwmj3/8PJRtjUEF6W1L4ATKTe0kLxySU5HXfi19Bf6uHS7PGkh0 Y32Kuh4yfvGU3+bTodief6lkWrvZb2l9i0+BNxHKKC95GhyjY3OToWfwE1HsgRrHDAWt Pwnl5DlMctdi3mimc3IM0GDVOFNuInGFtvpdYQzhCxtFc8OH6VPxhMBZzcqDwny/zGzG AQtA== X-Gm-Message-State: AOJu0YzWAvQnM4TpAY/taWuWkolsuOFltNlJiU9Z9ddONE5117Hr/5GW i+FhIAPZgomaqs/pHyQI91C5Y3XbD2InGTessf8= X-Received: by 2002:a05:6a20:1447:b0:15d:b243:6131 with SMTP id a7-20020a056a20144700b0015db2436131mr11396876pzi.44.1699955551955; Tue, 14 Nov 2023 01:52:31 -0800 (PST) Received: from rayden.urgonet (h-217-31-164-171.A175.priv.bahnhof.se. [217.31.164.171]) by smtp.gmail.com with ESMTPSA id h10-20020a056a00218a00b006c34015a8f2sm874749pfi.146.2023.11.14.01.52.27 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 14 Nov 2023 01:52:31 -0800 (PST) From: Jens Wiklander To: linux-kernel@vger.kernel.org, op-tee@lists.trustedfirmware.org Cc: Sumit Garg , Jerome Forissier , Shyam Saini , Jens Wiklander Subject: [PATCH v3 1/2] optee: add page list to kernel private shared memory Date: Tue, 14 Nov 2023 10:52:16 +0100 Message-Id: <20231114095217.1142360-2-jens.wiklander@linaro.org> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20231114095217.1142360-1-jens.wiklander@linaro.org> References: <20231114095217.1142360-1-jens.wiklander@linaro.org> MIME-Version: 1.0 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (snail.vger.email [0.0.0.0]); Tue, 14 Nov 2023 01:52:41 -0800 (PST) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1782532604396521223 X-GMAIL-MSGID: 1782532604396521223 Until now has kernel private shared memory allocated as dynamic shared memory (not from the static shared memory pool) been returned without a list of physical pages on allocations via RPC. To support allocations larger than one page add a list of physical pages. Signed-off-by: Jens Wiklander Reviewed-by: Sumit Garg --- drivers/tee/optee/core.c | 28 +++++++++++++---------- drivers/tee/optee/smc_abi.c | 45 +++++++++++++++++-------------------- 2 files changed, 37 insertions(+), 36 deletions(-) diff --git a/drivers/tee/optee/core.c b/drivers/tee/optee/core.c index 2a258bd3b6b5..38ea2fecfc2e 100644 --- a/drivers/tee/optee/core.c +++ b/drivers/tee/optee/core.c @@ -27,7 +27,10 @@ int optee_pool_op_alloc_helper(struct tee_shm_pool *pool, struct tee_shm *shm, unsigned long start)) { unsigned int order = get_order(size); + unsigned int nr_pages = 1 << order; + struct page **pages; struct page *page; + unsigned int i; int rc = 0; /* @@ -42,30 +45,29 @@ int optee_pool_op_alloc_helper(struct tee_shm_pool *pool, struct tee_shm *shm, shm->paddr = page_to_phys(page); shm->size = PAGE_SIZE << order; - if (shm_register) { - unsigned int nr_pages = 1 << order, i; - struct page **pages; + pages = kcalloc(nr_pages, sizeof(*pages), GFP_KERNEL); + if (!pages) { + rc = -ENOMEM; + goto err; + } - pages = kcalloc(nr_pages, sizeof(*pages), GFP_KERNEL); - if (!pages) { - rc = -ENOMEM; - goto err; - } + for (i = 0; i < nr_pages; i++) + pages[i] = page + i; - for (i = 0; i < nr_pages; i++) - pages[i] = page + i; + shm->pages = pages; + shm->num_pages = nr_pages; + if (shm_register) { rc = shm_register(shm->ctx, shm, pages, nr_pages, (unsigned long)shm->kaddr); - kfree(pages); if (rc) goto err; } return 0; - err: free_pages((unsigned long)shm->kaddr, order); + shm->kaddr = NULL; return rc; } @@ -77,6 +79,8 @@ void optee_pool_op_free_helper(struct tee_shm_pool *pool, struct tee_shm *shm, shm_unregister(shm->ctx, shm); free_pages((unsigned long)shm->kaddr, get_order(shm->size)); shm->kaddr = NULL; + kfree(shm->pages); + shm->pages = NULL; } static void optee_bus_scan(struct work_struct *work) diff --git a/drivers/tee/optee/smc_abi.c b/drivers/tee/optee/smc_abi.c index d5b28fd35d66..b69410c7cc0a 100644 --- a/drivers/tee/optee/smc_abi.c +++ b/drivers/tee/optee/smc_abi.c @@ -678,10 +678,11 @@ static void handle_rpc_func_cmd_shm_alloc(struct tee_context *ctx, struct optee_msg_arg *arg, struct optee_call_ctx *call_ctx) { - phys_addr_t pa; struct tee_shm *shm; size_t sz; size_t n; + struct page **pages; + size_t page_count; arg->ret_origin = TEEC_ORIGIN_COMMS; @@ -716,32 +717,23 @@ static void handle_rpc_func_cmd_shm_alloc(struct tee_context *ctx, return; } - if (tee_shm_get_pa(shm, 0, &pa)) { - arg->ret = TEEC_ERROR_BAD_PARAMETERS; - goto bad; - } - - sz = tee_shm_get_size(shm); - - if (tee_shm_is_dynamic(shm)) { - struct page **pages; + /* + * If there are pages it's dynamically allocated shared memory (not + * from the reserved shared memory pool) and needs to be + * registered. + */ + pages = tee_shm_get_pages(shm, &page_count); + if (pages) { u64 *pages_list; - size_t page_num; - - pages = tee_shm_get_pages(shm, &page_num); - if (!pages || !page_num) { - arg->ret = TEEC_ERROR_OUT_OF_MEMORY; - goto bad; - } - pages_list = optee_allocate_pages_list(page_num); + pages_list = optee_allocate_pages_list(page_count); if (!pages_list) { arg->ret = TEEC_ERROR_OUT_OF_MEMORY; goto bad; } call_ctx->pages_list = pages_list; - call_ctx->num_entries = page_num; + call_ctx->num_entries = page_count; arg->params[0].attr = OPTEE_MSG_ATTR_TYPE_TMEM_OUTPUT | OPTEE_MSG_ATTR_NONCONTIG; @@ -752,17 +744,22 @@ static void handle_rpc_func_cmd_shm_alloc(struct tee_context *ctx, arg->params[0].u.tmem.buf_ptr = virt_to_phys(pages_list) | (tee_shm_get_page_offset(shm) & (OPTEE_MSG_NONCONTIG_PAGE_SIZE - 1)); - arg->params[0].u.tmem.size = tee_shm_get_size(shm); - arg->params[0].u.tmem.shm_ref = (unsigned long)shm; - optee_fill_pages_list(pages_list, pages, page_num, + optee_fill_pages_list(pages_list, pages, page_count, tee_shm_get_page_offset(shm)); } else { + phys_addr_t pa; + + if (tee_shm_get_pa(shm, 0, &pa)) { + arg->ret = TEEC_ERROR_BAD_PARAMETERS; + goto bad; + } + arg->params[0].attr = OPTEE_MSG_ATTR_TYPE_TMEM_OUTPUT; arg->params[0].u.tmem.buf_ptr = pa; - arg->params[0].u.tmem.size = sz; - arg->params[0].u.tmem.shm_ref = (unsigned long)shm; } + arg->params[0].u.tmem.size = tee_shm_get_size(shm); + arg->params[0].u.tmem.shm_ref = (unsigned long)shm; arg->ret = TEEC_SUCCESS; return; From patchwork Tue Nov 14 09:52:17 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jens Wiklander X-Patchwork-Id: 164799 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:b909:0:b0:403:3b70:6f57 with SMTP id t9csp1752041vqg; Tue, 14 Nov 2023 01:52:49 -0800 (PST) X-Google-Smtp-Source: AGHT+IFOp3RzW3a86Z/Ss5PcStBhOwV6cWlJp2OTbCILfC83XBZdbqvC9VeSsJpuBDOLjc/nz2yD X-Received: by 2002:a05:6871:4105:b0:1e9:8e22:24c6 with SMTP id la5-20020a056871410500b001e98e2224c6mr11545276oab.44.1699955569256; Tue, 14 Nov 2023 01:52:49 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1699955569; cv=none; d=google.com; s=arc-20160816; b=RAiXU/p+RWdPT3Aw3C6/1CdRyhIwA0NKTIw9gXmzpkALZIhmK+cj6LXUErS0i1ZfgA 3hi9fhOt2bGrsnFZsqEG3HxIbGJQseERsnVtTGpyzy95W1dQW87LQqk/LYsnSInnZKf9 9PvIRMrpGAo8bV91TZzr1KWxvPtjxo2KVjFXfPqrCsmY1gdP2Du8W+5K8o5l/ng2pRO5 QoTTzqdj4Hnb/TQc46XJW5WD+L6FvoNc3QaUrjIC4Y9dlxFUAOqTm1GU0t1Pt2RdGv4R MnhQ7FMGw9LfxgnD7J73GOIMyN+aYTuGKlPxWeJhJH+Q6BFrFbeaLBoXSsNmopr+BhAO z+eA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=sYDsu5lJwL/Ln9te5CEC5b5SypRoh7tDXJlqgzKOaW4=; fh=SSmA14+Y2ZA2PO07peKpbI6W7vxD8MB28Dbup/soyB4=; b=p3LyV0ouLY6N5zWQag3Az6mvWkTmk1h38VnNDuFUiotGcDnYnG6nBZB3oqTOm/+ViU qcqer7d59urU1uSk92+aIgloItzSLaPHV/Ykab5COpbhsAXUuui6RU2JCWKzVtojZe2X H6Z+oJNwTOtaGHe9epH+bGcO9Ne8aFvKEFW8xJbEkFZ+CswtbyVuncCxikd61wGsD0r/ RCHL5Gkf0XKM2fuO5Rpjj/7CwALGHz3uaKq9IwzICHILEQZgQY7qPv0im1bo3lbRkwKV aAtfBActHwwfPbO3EMvj8g6sFpN+1i6Jq9ZoJJT2/jOouO5MoEiXnxBQj5MxRa8x20Jw UmHQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b="p7S5E0/x"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from snail.vger.email (snail.vger.email. [23.128.96.37]) by mx.google.com with ESMTPS id t186-20020a6381c3000000b005be1f94cae8si7414014pgd.378.2023.11.14.01.52.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 14 Nov 2023 01:52:49 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b="p7S5E0/x"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by snail.vger.email (Postfix) with ESMTP id 7F4C0801C029; Tue, 14 Nov 2023 01:52:48 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at snail.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232590AbjKNJwo (ORCPT + 30 others); Tue, 14 Nov 2023 04:52:44 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59194 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232533AbjKNJwn (ORCPT ); Tue, 14 Nov 2023 04:52:43 -0500 Received: from mail-pf1-x429.google.com (mail-pf1-x429.google.com [IPv6:2607:f8b0:4864:20::429]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id ADBFE1B3 for ; Tue, 14 Nov 2023 01:52:37 -0800 (PST) Received: by mail-pf1-x429.google.com with SMTP id d2e1a72fcca58-692c02adeefso4815960b3a.3 for ; Tue, 14 Nov 2023 01:52:37 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1699955557; x=1700560357; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=sYDsu5lJwL/Ln9te5CEC5b5SypRoh7tDXJlqgzKOaW4=; b=p7S5E0/xyHL560T8PZxR6GUvC5Qz6jr6VfE+oADFGKTafqZyZSB+wDhSmtQiVK54YS /8Jp0tcz7c7cTZlVUBAZrE5BZ8p00FRehPCsw3llPxbmFpDEtv5qRltpPPsLDLmRz4z6 e1TPaPfs/S/X6tqDE5Rfzr65DBL2C/qbImzWoieoZFuCyvnzvng8mDx6GwUyB/KlyUUS 8I8hWspJE6U/6mwgvJlL2BpiM+pa6zVGxkF3n/yB5/u6HOnRw8ndCNXfP3ZyRLH+iaNe XyXHdCL2PB94C5goY27dFizAL7sG+c2TMwBvbGYn/MaE+wkqIt3L2GlEknVUN0xTIU3N PQIg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1699955557; x=1700560357; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=sYDsu5lJwL/Ln9te5CEC5b5SypRoh7tDXJlqgzKOaW4=; b=q6qadnRVZv1OxIh6CSSPCuPueGi1hd2Rl23tr9tOv7GlIWciq1aYpPOwfyYLMtZh5W QiUJ6UmP1ank/OJMtzcpoPapoUJP0WFsQTwaJA2XaoHANeCD9YE/RInGO85L+6V6xSCm KKI59wZPiyqvgrYQYgEvAS02j9YC3ZWQY54pWDeKaRPbeH81n7r4LFT/kLyLSn9YGBqj foA1J7FmU54iIFebJHNGJroxnWDqrFsQ1pO41wSHa/ztG9BDTRTgLvveBI7LgtVgfsu4 N5LfhM0wDOwr2BC0fSmrMdJCfPaQJ2N+U6kQTeEdxHb/LciiuMmo3//tKU87uSEUt+xJ GMLg== X-Gm-Message-State: AOJu0YxTfSvChTa/o+MhhwNbEfYt7TvvaSrQT2/CXtahGCE0VR+k1iW6 unZYA35tzlXi41dbq22gJKszYkDYWnyiN4QYHio= X-Received: by 2002:a05:6a20:7d81:b0:186:69ed:d9c6 with SMTP id v1-20020a056a207d8100b0018669edd9c6mr4495586pzj.48.1699955556878; Tue, 14 Nov 2023 01:52:36 -0800 (PST) Received: from rayden.urgonet (h-217-31-164-171.A175.priv.bahnhof.se. [217.31.164.171]) by smtp.gmail.com with ESMTPSA id h10-20020a056a00218a00b006c34015a8f2sm874749pfi.146.2023.11.14.01.52.32 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 14 Nov 2023 01:52:36 -0800 (PST) From: Jens Wiklander To: linux-kernel@vger.kernel.org, op-tee@lists.trustedfirmware.org Cc: Sumit Garg , Jerome Forissier , Shyam Saini , Jens Wiklander Subject: [PATCH v3 2/2] optee: allocate shared memory with alloc_pages_exact() Date: Tue, 14 Nov 2023 10:52:17 +0100 Message-Id: <20231114095217.1142360-3-jens.wiklander@linaro.org> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20231114095217.1142360-1-jens.wiklander@linaro.org> References: <20231114095217.1142360-1-jens.wiklander@linaro.org> MIME-Version: 1.0 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (snail.vger.email [0.0.0.0]); Tue, 14 Nov 2023 01:52:48 -0800 (PST) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1782532611408502061 X-GMAIL-MSGID: 1782532611408502061 Allocate memory to share with the secure using alloc_pages_exact() instead of alloc_pages() for more efficient memory usage. Signed-off-by: Jens Wiklander Reviewed-by: Sumit Garg --- drivers/tee/optee/core.c | 20 +++++++++----------- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/drivers/tee/optee/core.c b/drivers/tee/optee/core.c index 38ea2fecfc2e..4a4b03b4fc7d 100644 --- a/drivers/tee/optee/core.c +++ b/drivers/tee/optee/core.c @@ -26,10 +26,8 @@ int optee_pool_op_alloc_helper(struct tee_shm_pool *pool, struct tee_shm *shm, size_t num_pages, unsigned long start)) { - unsigned int order = get_order(size); - unsigned int nr_pages = 1 << order; + size_t nr_pages = roundup(size, PAGE_SIZE) / PAGE_SIZE; struct page **pages; - struct page *page; unsigned int i; int rc = 0; @@ -37,13 +35,13 @@ int optee_pool_op_alloc_helper(struct tee_shm_pool *pool, struct tee_shm *shm, * Ignore alignment since this is already going to be page aligned * and there's no need for any larger alignment. */ - page = alloc_pages(GFP_KERNEL | __GFP_ZERO, order); - if (!page) + shm->kaddr = alloc_pages_exact(nr_pages * PAGE_SIZE, + GFP_KERNEL | __GFP_ZERO); + if (!shm->kaddr) return -ENOMEM; - shm->kaddr = page_address(page); - shm->paddr = page_to_phys(page); - shm->size = PAGE_SIZE << order; + shm->paddr = virt_to_phys(shm->kaddr); + shm->size = nr_pages * PAGE_SIZE; pages = kcalloc(nr_pages, sizeof(*pages), GFP_KERNEL); if (!pages) { @@ -52,7 +50,7 @@ int optee_pool_op_alloc_helper(struct tee_shm_pool *pool, struct tee_shm *shm, } for (i = 0; i < nr_pages; i++) - pages[i] = page + i; + pages[i] = virt_to_page((u8 *)shm->kaddr + i * PAGE_SIZE); shm->pages = pages; shm->num_pages = nr_pages; @@ -66,7 +64,7 @@ int optee_pool_op_alloc_helper(struct tee_shm_pool *pool, struct tee_shm *shm, return 0; err: - free_pages((unsigned long)shm->kaddr, order); + free_pages_exact(shm->kaddr, shm->size); shm->kaddr = NULL; return rc; } @@ -77,7 +75,7 @@ void optee_pool_op_free_helper(struct tee_shm_pool *pool, struct tee_shm *shm, { if (shm_unregister) shm_unregister(shm->ctx, shm); - free_pages((unsigned long)shm->kaddr, get_order(shm->size)); + free_pages_exact(shm->kaddr, shm->size); shm->kaddr = NULL; kfree(shm->pages); shm->pages = NULL;