From mboxrd@z Thu Jan 1 00:00:00 1970 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 kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id E5B87C6FD1F for ; Sun, 12 Mar 2023 11:27:11 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 853C68E0012; Sun, 12 Mar 2023 07:27:11 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 804DA8E000E; Sun, 12 Mar 2023 07:27:11 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 67ED38E0012; Sun, 12 Mar 2023 07:27:11 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id 4F44B8E000E for ; Sun, 12 Mar 2023 07:27:11 -0400 (EDT) Received: from smtpin26.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay06.hostedemail.com (Postfix) with ESMTP id 18BA9AAD18 for ; Sun, 12 Mar 2023 11:27:11 +0000 (UTC) X-FDA: 80560019862.26.B92612C Received: from mga06.intel.com (mga06b.intel.com [134.134.136.31]) by imf05.hostedemail.com (Postfix) with ESMTP id EF90310000D for ; Sun, 12 Mar 2023 11:27:08 +0000 (UTC) Authentication-Results: imf05.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=OEvsBgI2; spf=none (imf05.hostedemail.com: domain of kirill.shutemov@linux.intel.com has no SPF policy when checking 134.134.136.31) smtp.mailfrom=kirill.shutemov@linux.intel.com; dmarc=pass (policy=none) header.from=intel.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1678620429; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=OZUQfYUiRxDD4yajdfiAuzpHqN59m/bqizk0pX5fkM8=; b=O1fFvoOhTdbgTFwI+Z4Y4+w9RjHyzfegMf+5CQbIVrl3PdTgBDsk5qlAciECY5gRlJBu8i 00AAXPh2EDtPqg9sOLdPpI+jvCJAFtr1yJCS171oKeXxD0wFK8vpAVYC9o5U521aZzvCaZ hGsu6VcWsJk8a+DkSxUzjRq1lPAN7kU= ARC-Authentication-Results: i=1; imf05.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=OEvsBgI2; spf=none (imf05.hostedemail.com: domain of kirill.shutemov@linux.intel.com has no SPF policy when checking 134.134.136.31) smtp.mailfrom=kirill.shutemov@linux.intel.com; dmarc=pass (policy=none) header.from=intel.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1678620429; a=rsa-sha256; cv=none; b=ZRX6jhzk3UdY3jbkSpbyqwLPZJicnNQYwjY6YzriU8NHO0U9KmxYIklJSyxjsagRwEo4bO 74psi1WfzCTBsGnaiU0sprtEOupTN0lCrIm8qQZxPpjyo9DBmEcxwITvWT/Mz1EZ/r0okR 01KEhQdCVMAWeYjADawKlP4sReRSDPc= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1678620429; x=1710156429; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=oTU4sxTXYKtAIVLwPW8AIWDYdktjaL52nQvpCjw3OeY=; b=OEvsBgI2NNCX85in7SUlmH/UlqaRkrFjcjESMM/JVqHz2sFAo004V8RA tmtr6TsFWyMuiN1p/yJTshGUlxdGcejETgfC452O2Hzmj0y9dj8mONaAt 6oKPGiLIKyTMDKiud0e6onLoCaS5tTFdDCRonYIHYVt0pp1zFj9zmzfDB 6P8QSFuzLEwv34Ny6FhXWtDTlYaLbzS+hW1lOf7FQuM4YfB2FEPX98IqH 6Prplb041En3QWAxeqb2V6XFutzu1ogEQ3z1JAyDK/4s/nvRxInA4vAhT Y9jqgvtzyPZSwB8bslc2tjaGUhQxTYATiOa+QiTOEBS4zqftAW3PZOGf1 A==; X-IronPort-AV: E=McAfee;i="6500,9779,10646"; a="399577658" X-IronPort-AV: E=Sophos;i="5.98,254,1673942400"; d="scan'208";a="399577658" Received: from orsmga007.jf.intel.com ([10.7.209.58]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 12 Mar 2023 04:26:43 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6500,9779,10646"; a="671580755" X-IronPort-AV: E=Sophos;i="5.98,254,1673942400"; d="scan'208";a="671580755" Received: from nmoazzen-mobl1.amr.corp.intel.com (HELO box.shutemov.name) ([10.251.219.215]) by orsmga007-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 12 Mar 2023 04:26:38 -0700 Received: by box.shutemov.name (Postfix, from userid 1000) id 0BE2910D7BF; Sun, 12 Mar 2023 14:26:20 +0300 (+03) From: "Kirill A. Shutemov" To: Dave Hansen , Andy Lutomirski , Peter Zijlstra Cc: x86@kernel.org, Kostya Serebryany , Andrey Ryabinin , Andrey Konovalov , Alexander Potapenko , Taras Madan , Dmitry Vyukov , "H . J . Lu" , Andi Kleen , Rick Edgecombe , Bharata B Rao , Jacob Pan , Ashok Raj , Linus Torvalds , linux-mm@kvack.org, linux-kernel@vger.kernel.org, "Kirill A. Shutemov" Subject: [PATCHv16 17/17] selftests/x86/lam: Add test cases for LAM vs thread creation Date: Sun, 12 Mar 2023 14:26:12 +0300 Message-Id: <20230312112612.31869-18-kirill.shutemov@linux.intel.com> X-Mailer: git-send-email 2.39.2 In-Reply-To: <20230312112612.31869-1-kirill.shutemov@linux.intel.com> References: <20230312112612.31869-1-kirill.shutemov@linux.intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Stat-Signature: bfmuwowt8npqri9pfzdafs91qgmjq8uf X-Rspam-User: X-Rspamd-Queue-Id: EF90310000D X-Rspamd-Server: rspam06 X-HE-Tag: 1678620428-543920 X-HE-Meta: U2FsdGVkX19aMLaoEM6PR//XKJScuCOHFjP5WfuIrpp0a7zRZI2EWhlPHWvtPAk4OQgv/jk7HRpM5TMmBIr5Tgfsa+9Z3F6ePmOrQF3nsE8D2SwW4BVudVoOonO6ziOTx1W5CcH4/4neFnRSsuh9ghu9ehvfy2K1h4LPa4Q6WUU+ipv3KBZ7nIZ5EQeRyym7/Y5NHSIMzNPMe2S7Yf3Fqlyk63zpFEGCMiHTm8Q73qL5FS0TPXZlB8K3pS8pX7ezLsApVA4aYSG/gLKGD8+eCgIIfFcc0t2gyXQLqKgrZurTNZdqyHRYrRnQweUN+y/HyVH9NQrEPBfXWJ7BFPqLsKvot5Y/QFULulW8LX9p8myJb6xNo5EXAlFEmaChbl0pM3eaaeYhYqiS78+3lYBfbvSc59F4t/VYX01JQGFsvhxEv3sIXPP9PDeXFtvzbvpbgCxEzZcwQWpTifaY+KGrkagod8rn3uirAhZ7OBQppFvuTOPQ9rzSTUsq+5q+NQMtsW00t48QVUepWASWljs9zICtWTorZS9BaAdnWDBxQyZm4ObHbN9/A+2vQdg05TxbQ4pjSyqQAeaqxPE67QX5PdyRvt6IR79JXkhWkWrmQYSRk/ZvYdxrHBUIOsE6pMcrDrlXlfVY2Aw17AdtvMa4ZlxPbjSnZFlFzHveVD1QXD8xw6I+A7yCiW0XeUv/D4szqjU0EcDdjaZprEt8/HqruZ6F7JH/ZmJT9iy0Qi393p3ashjW6RNjZacpw/4SLxxfSvqX77YucV3oa8cyU9C4db/RoOKp9K6XNXhf+CLsRtgyMRawDiM2hTlBLADqGlOeKt4xS8oFLvp5Q6KeRlNHpssDvqpyq+e1yDk7UYby/vti7Yl7Ub7Obj1pj7a9s9/AFsBFlV9C3zOLzEYTosXETF6wukFYVRr2TBLe6TPm8dXa7PwZ7ST3b37v6ovbqaheST0nJgwcBKaiKqsPIYi v3+MUhZA JljkJRrPTV3AKUx5zUm+uQOiJKh1uKFYg5001XDEw2CGQ/bS2p3R60W4IxUvGbW20PT4zF9A2FlKa3oPujwMy2f39JjhevAh1hUjfiPEtNJu93QHcNumN7iZpNFkPJIPjQDDTIvz2REodHLKSXdrHGnzMGZ4kNw0eGM05T8AS0BiAf6J9luWQ35ltUN+CniXDXu2RmXOMjxmgvhl/YSGhu62FNrbxVOhleXEa2ZuYQbVXkd089KIqGgyzeOrPNCpAncUt35s9EmDQHFUFcwy+KiS7/SRlOE8KyHDd X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: LAM enabling is only allowed when the process has single thread. LAM mode is inherited into child thread. Trying to enable LAM after spawning a thread has to fail. Signed-off-by: Kirill A. Shutemov Acked-by: Peter Zijlstra (Intel) --- tools/testing/selftests/x86/lam.c | 92 +++++++++++++++++++++++++++++++ 1 file changed, 92 insertions(+) diff --git a/tools/testing/selftests/x86/lam.c b/tools/testing/selftests/x86/lam.c index a8c91829b616..eb0e46905bf9 100644 --- a/tools/testing/selftests/x86/lam.c +++ b/tools/testing/selftests/x86/lam.c @@ -1,4 +1,5 @@ // SPDX-License-Identifier: GPL-2.0 +#define _GNU_SOURCE #include #include #include @@ -12,6 +13,7 @@ #include #include #include +#include #include #include @@ -50,6 +52,8 @@ #define PAGE_SIZE (4 << 10) +#define STACK_SIZE 65536 + #define barrier() ({ \ __asm__ __volatile__("" : : : "memory"); \ }) @@ -731,6 +735,75 @@ static int handle_inheritance(struct testcases *test) return 0; } +static int thread_fn_get_lam(void *arg) +{ + return get_lam(); +} + +static int thread_fn_set_lam(void *arg) +{ + struct testcases *test = arg; + + return set_lam(test->lam); +} + +static int handle_thread(struct testcases *test) +{ + char stack[STACK_SIZE]; + int ret, child_ret; + int lam = 0; + pid_t pid; + + /* Set LAM mode in parent process */ + if (!test->later) { + lam = test->lam; + if (set_lam(lam) != 0) + return 1; + } + + pid = clone(thread_fn_get_lam, stack + STACK_SIZE, + SIGCHLD | CLONE_FILES | CLONE_FS | CLONE_VM, NULL); + if (pid < 0) { + perror("Clone failed."); + return 1; + } + + waitpid(pid, &child_ret, 0); + ret = WEXITSTATUS(child_ret); + + if (lam != ret) + return 1; + + if (test->later) { + if (set_lam(test->lam) != 0) + return 1; + } + + return 0; +} + +static int handle_thread_enable(struct testcases *test) +{ + char stack[STACK_SIZE]; + int ret, child_ret; + int lam = test->lam; + pid_t pid; + + pid = clone(thread_fn_set_lam, stack + STACK_SIZE, + SIGCHLD | CLONE_FILES | CLONE_FS | CLONE_VM, test); + if (pid < 0) { + perror("Clone failed."); + return 1; + } + + waitpid(pid, &child_ret, 0); + ret = WEXITSTATUS(child_ret); + + if (lam != ret) + return 1; + + return 0; +} static void run_test(struct testcases *test, int count) { int i, ret = 0; @@ -846,6 +919,25 @@ static struct testcases inheritance_cases[] = { .test_func = handle_inheritance, .msg = "FORK: LAM_U57, child process should get LAM mode same as parent\n", }, + { + .expected = 0, + .lam = LAM_U57_BITS, + .test_func = handle_thread, + .msg = "THREAD: LAM_U57, child thread should get LAM mode same as parent\n", + }, + { + .expected = 1, + .lam = LAM_U57_BITS, + .test_func = handle_thread_enable, + .msg = "THREAD: [NEGATIVE] Enable LAM in child.\n", + }, + { + .expected = 1, + .later = 1, + .lam = LAM_U57_BITS, + .test_func = handle_thread, + .msg = "THREAD: [NEGATIVE] Enable LAM in parent after thread created.\n", + }, { .expected = 0, .lam = LAM_U57_BITS, -- 2.39.2