From nobody Fri Nov 29 02:45:04 2024 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id C8D6C158552; Wed, 25 Sep 2024 19:54:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1727294042; cv=none; b=lv4AxWTmdiQlb2+So90SXl1W46cPN+Qz5a2b50uZeS6NfB/ZUxUZms055vah4GfrM/cTUhViP7ChK1M1q3tbHS7D0mazgBpAS7BCygmrma464ihrep9eY7E8LkU/tFCOuSgMm6wrby9p5Q+tkd+kzU08xGUCjc5AajdBK6CtoPc= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1727294042; c=relaxed/simple; bh=CHzNKVXu1RqZbmV2PfE2l0lyp9I7aLbzynLWgxBwRO4=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=VeDcmTPGEhk9wrby78Y40nhY8TCPFSeO+zJf0i3rs0Y0LGqz2Y2Xx9wyLdWdKd09rGxPmlyBr36D0zMx2o7WRmUrltfLMYg+Vzd+dR2NIHgSNrZvknIaMyfFdTP8CPCrt7nuGYIk7HPZIfm7OnAE+kqWgUM2OaHGjvsDjVEmyM0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id B3BE3150C; Wed, 25 Sep 2024 12:54:29 -0700 (PDT) Received: from e132581.cambridge.arm.com (e132581.arm.com [10.2.76.71]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id 7C2213F64C; Wed, 25 Sep 2024 12:53:58 -0700 (PDT) From: Leo Yan To: Arnaldo Carvalho de Melo , Adrian Hunter , Namhyung Kim , Mark Rutland , Alexander Shishkin , Jiri Olsa , Ian Rogers , "Liang, Kan" , James Clark , linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Leo Yan Subject: [PATCH v1 4/5] perf cpumap: Add more tests for CPU map merging Date: Wed, 25 Sep 2024 20:53:24 +0100 Message-Id: <20240925195325.426533-5-leo.yan@arm.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20240925195325.426533-1-leo.yan@arm.com> References: <20240925195325.426533-1-leo.yan@arm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Add additional tests for CPU map merging to cover more cases. These tests include different types of arguments, such as when one CPU map is a subset of another, as well as cases with or without overlap between the two maps. Signed-off-by: Leo Yan --- tools/perf/tests/cpumap.c | 34 +++++++++++++++++++++++++++++----- 1 file changed, 29 insertions(+), 5 deletions(-) diff --git a/tools/perf/tests/cpumap.c b/tools/perf/tests/cpumap.c index ed07ef6d7e33..d5b6c450f5c9 100644 --- a/tools/perf/tests/cpumap.c +++ b/tools/perf/tests/cpumap.c @@ -156,22 +156,46 @@ static int test__cpu_map_print(struct test_suite *tes= t __maybe_unused, int subte return 0; } =20 -static int test__cpu_map_merge(struct test_suite *test __maybe_unused, int= subtest __maybe_unused) +static int __test__cpu_map_merge(const char *lhs, const char *rhs, int nr,= const char *expected) { - struct perf_cpu_map *a =3D perf_cpu_map__new("4,2,1"); - struct perf_cpu_map *b =3D perf_cpu_map__new("4,5,7"); + struct perf_cpu_map *a =3D perf_cpu_map__new(lhs); + struct perf_cpu_map *b =3D perf_cpu_map__new(rhs); struct perf_cpu_map *c =3D perf_cpu_map__merge(a, b); char buf[100]; =20 - TEST_ASSERT_VAL("failed to merge map: bad nr", perf_cpu_map__nr(c) =3D=3D= 5); + TEST_ASSERT_VAL("failed to merge map: bad nr", perf_cpu_map__nr(c) =3D=3D= nr); cpu_map__snprint(c, buf, sizeof(buf)); - TEST_ASSERT_VAL("failed to merge map: bad result", !strcmp(buf, "1-2,4-5,= 7")); + TEST_ASSERT_VAL("failed to merge map: bad result", !strcmp(buf, expected)= ); perf_cpu_map__put(a); perf_cpu_map__put(b); perf_cpu_map__put(c); return 0; } =20 +static int test__cpu_map_merge(struct test_suite *test __maybe_unused, + int subtest __maybe_unused) +{ + int ret; + + ret =3D __test__cpu_map_merge("4,2,1", "4,5,7", 5, "1-2,4-5,7"); + if (ret) + return ret; + ret =3D __test__cpu_map_merge("1-8", "6-9", 9, "1-9"); + if (ret) + return ret; + ret =3D __test__cpu_map_merge("1-8,12-20", "6-9,15", 18, "1-9,12-20"); + if (ret) + return ret; + ret =3D __test__cpu_map_merge("4,2,1", "1", 3, "1-2,4"); + if (ret) + return ret; + ret =3D __test__cpu_map_merge("1", "4,2,1", 3, "1-2,4"); + if (ret) + return ret; + ret =3D __test__cpu_map_merge("1", "1", 1, "1"); + return ret; +} + static int __test__cpu_map_intersect(const char *lhs, const char *rhs, int= nr, const char *expected) { struct perf_cpu_map *a =3D perf_cpu_map__new(lhs); --=20 2.34.1