Re: [PATCH v3 4/7] device-dax/kmem: Set node state as N_DEMOTION_TARGETS

From: Wei Xu <weixugc@google.com>
To: Jagdish Gediya <jvgediya@linux.ibm.com>
Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org,
	 akpm@linux-foundation.org, baolin.wang@linux.alibaba.com,
	 dave.hansen@linux.intel.com, ying.huang@intel.com,
	aneesh.kumar@linux.ibm.com,  shy828301@gmail.com,
	gthelen@google.com, dan.j.williams@intel.com
Subject: Re: [PATCH v3 4/7] device-dax/kmem: Set node state as N_DEMOTION_TARGETS
Date: Fri, 22 Apr 2022 13:34:31 -0700	[thread overview]
Message-ID: <CAAPL-u83O5W8B-G9xqZFWGuJNNc5nNkQ6iTVtJ-Gn01o4uFoPA@mail.gmail.com> (raw)
In-Reply-To: <20220422195516.10769-5-jvgediya@linux.ibm.com>

On Fri, Apr 22, 2022 at 12:55 PM Jagdish Gediya <jvgediya@linux.ibm.com> wrote:
>
> Set dax-device node as N_DEMOTION_TARGETS so that it
> can be used as demotion target.
>
> In future, support should be added to distinguish the
> dax-devices which are not preferred as demotion target
> e.g. HBM, for such devices, node shouldn't be set to
> N_DEMOTION_TARGETS.
>
> Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.ibm.com>
> Signed-off-by: Jagdish Gediya <jvgediya@linux.ibm.com>
> ---

Acked-by: Wei Xu <weixugc@google.com>

>  drivers/dax/kmem.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/dax/kmem.c b/drivers/dax/kmem.c
> index a37622060fff..f42ab9d04bdf 100644
> --- a/drivers/dax/kmem.c
> +++ b/drivers/dax/kmem.c
> @@ -147,6 +147,8 @@ static int dev_dax_kmem_probe(struct dev_dax *dev_dax)
>
>         dev_set_drvdata(dev, data);
>
> +       node_set_state(numa_node, N_DEMOTION_TARGETS);
> +
>         return 0;
>
>  err_request_mem:
> --
> 2.35.1
>