[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250701224353.9062-1-bijan311@gmail.com>
Date: Tue, 1 Jul 2025 17:43:30 -0500
From: Bijan Tabatabai <bijan311@...il.com>
To: SeongJae Park <sj@...nel.org>
Cc: Bijan Tabatabai <bijan311@...il.com>,
Andrew Morton <akpm@...ux-foundation.org>,
Bijan Tabatabai <bijantabatab@...ron.com>,
damon@...ts.linux.dev,
kernel-team@...a.com,
linux-kernel@...r.kernel.org,
linux-mm@...ck.org
Subject: Re: [RFC PATCH 1/4] mm/damon: add struct damos_migrate_dest
On Sat, 21 Jun 2025 10:31:28 -0700 SeongJae Park <sj@...nel.org> wrote:
> Introduce a new struct, namely damos_migrate_dest, for specifying
> multiple DAMOS' migration destination nodes and their weights.
>
> Signed-off-by: SeongJae Park <sj@...nel.org>
> ---
> include/linux/damon.h | 16 ++++++++++++++++
> 1 file changed, 16 insertions(+)
>
> diff --git a/include/linux/damon.h b/include/linux/damon.h
> index bb58e36f019e..d60addd0b7c8 100644
> --- a/include/linux/damon.h
> +++ b/include/linux/damon.h
> @@ -447,6 +447,22 @@ struct damos_access_pattern {
> unsigned int max_age_region;
> };
>
> +/**
> + * struct damos_migrate_dest - Migration destination nodes and their weights.
Nit: Can this be renamed to damos_migrate_dests?
I think plural fits better because it stores a list of destinations.
Thanks,
Bijan
> + * @node_id_arr: Array of migration destination node ids.
> + * @weight_arr: Array of migration weights for @node_id_arr.
> + * @nr_dests: Length of the @node_id_arr and @weight_arr arrays.
> + *
> + * @node_id_arr is an array of the ids of migration destination nodes.
> + * @weight_arr is an array of the weights for those. The weights in
> + * @weight_arr are for nodes in @node_id_arr of same array index.
> + */
> +struct damos_migrate_dest {
> + unsigned int *node_id_arr;
> + unsigned int *weight_arr;
> + size_t nr_dests;
> +};
> +
> /**
> * struct damos - Represents a Data Access Monitoring-based Operation Scheme.
> * @pattern: Access pattern of target regions.
> --
> 2.39.5
Powered by blists - more mailing lists