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 E8F6BC433EF for ; Wed, 29 Dec 2021 13:10:38 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 006886B0075; Wed, 29 Dec 2021 08:10:36 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id EAD376B0078; Wed, 29 Dec 2021 08:10:35 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id CFCA46B007B; Wed, 29 Dec 2021 08:10:35 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0037.hostedemail.com [216.40.44.37]) by kanga.kvack.org (Postfix) with ESMTP id AC7306B0078 for ; Wed, 29 Dec 2021 08:10:35 -0500 (EST) Received: from smtpin13.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id 6FFEB82499B9 for ; Wed, 29 Dec 2021 13:10:35 +0000 (UTC) X-FDA: 78970866030.13.315E74A Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf22.hostedemail.com (Postfix) with ESMTP id 57E83C0005 for ; Wed, 29 Dec 2021 13:10:33 +0000 (UTC) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 4989E61416; Wed, 29 Dec 2021 13:10:33 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5916AC36AF0; Wed, 29 Dec 2021 13:10:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1640783432; bh=Tqc33p9GRUSgtrjHIQAmSM+HAYm5FH9FB0Llx5PwAqc=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=LPhi3v/kMUPXv96TQTW22OOl8z+JEHoDL1BPsEH8asi285cOnODkyp5IjDv8+9DPU Z2CB0VeK0gOamZH4joyCUv+w4HgJA/QhErfStJJ6rz3DQ4dF1DPEkAWcUU+meSEbnX iKEc0CYSIqe3NX6G0S8GxwF6BurG7ASPRlyekHHB8rXSrVzxO8vmykLeJWzNSUobTU lGHFBM3KMTj9wKKyZXPb4LJmeMh2lL9jnJPjl7QWbLrvcDcc5SIP5xzYrFzUnmdmzH KALCME/O8ZfLPh48LfI5bC/2+oAj5Y7uHSsBMMLzDfmpuxjO/e5QwBQPL0U9cZORfj VW9fkQv3VY+Ug== From: SeongJae Park To: akpm@linux-foundation.org Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org, SeongJae Park Subject: [PATCH 3/4] mm/damon/vaddr: Hide kernel pointer from damon_va_three_regions() failure log Date: Wed, 29 Dec 2021 13:10:15 +0000 Message-Id: <20211229131016.23641-4-sj@kernel.org> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20211229131016.23641-1-sj@kernel.org> References: <20211229131016.23641-1-sj@kernel.org> Authentication-Results: imf22.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b="LPhi3v/k"; dmarc=pass (policy=none) header.from=kernel.org; spf=pass (imf22.hostedemail.com: domain of sj@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=sj@kernel.org X-Rspamd-Server: rspam12 X-Rspamd-Queue-Id: 57E83C0005 X-Stat-Signature: oekjx3ttjzwyhnaqfyzxhmf9kedkor84 X-HE-Tag: 1640783433-267351 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: The failure log message for 'damon_va_three_regions()' prints the target id, which is a 'struct pid' pointer in the case. To avoid exposing the kernel pointer via the log, this commit makes the log to use the index of the target in the context's targets list instead. Signed-off-by: SeongJae Park --- mm/damon/vaddr.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/mm/damon/vaddr.c b/mm/damon/vaddr.c index 7997e1c000ed..73ee9719f8ba 100644 --- a/mm/damon/vaddr.c +++ b/mm/damon/vaddr.c @@ -232,13 +232,19 @@ static int damon_va_three_regions(struct damon_target *t, static void __damon_va_init_regions(struct damon_ctx *ctx, struct damon_target *t) { + struct damon_target *ti; struct damon_region *r; struct damon_addr_range regions[3]; unsigned long sz = 0, nr_pieces; - int i; + int i, tidx = 0; if (damon_va_three_regions(t, regions)) { - pr_debug("Failed to get three regions of target %lu\n", t->id); + damon_for_each_target(ti, ctx) { + if (ti == t) + break; + tidx++; + } + pr_debug("Failed to get three regions of %dth target\n", tidx); return; } -- 2.17.1