提交 962fe7a6 编写于 作者: S SeongJae Park 提交者: Linus Torvalds

mm/damon/vaddr: hide kernel pointer from damon_va_three_regions() failure log

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 makes the log to use the index of the
target in the context's targets list instead.

Link: https://lkml.kernel.org/r/20211229131016.23641-4-sj@kernel.orgSigned-off-by: NSeongJae Park <sj@kernel.org>
Signed-off-by: NAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: NLinus Torvalds <torvalds@linux-foundation.org>
上级 251403f1
...@@ -232,13 +232,19 @@ static int damon_va_three_regions(struct damon_target *t, ...@@ -232,13 +232,19 @@ static int damon_va_three_regions(struct damon_target *t,
static void __damon_va_init_regions(struct damon_ctx *ctx, static void __damon_va_init_regions(struct damon_ctx *ctx,
struct damon_target *t) struct damon_target *t)
{ {
struct damon_target *ti;
struct damon_region *r; struct damon_region *r;
struct damon_addr_range regions[3]; struct damon_addr_range regions[3];
unsigned long sz = 0, nr_pieces; unsigned long sz = 0, nr_pieces;
int i; int i, tidx = 0;
if (damon_va_three_regions(t, regions)) { 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; return;
} }
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册