Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Zi Yan <ziy@nvidia.com>
To:     Dave Hansen <dave.hansen@intel.com>
CC:     Shakeel Butt <shakeelb@google.com>,
        Dave Hansen <dave.hansen@linux.intel.com>,
        LKML <linux-kernel@vger.kernel.org>,
        Linux MM <linux-mm@kvack.org>,
        Yang Shi <yang.shi@linux.alibaba.com>,
        David Rientjes <rientjes@google.com>,
        Huang Ying <ying.huang@intel.com>,
        Dan Williams <dan.j.williams@intel.com>
Subject: Re: [RFC] [PATCH 0/8] Migrate Pages in lieu of discard
Date:   Wed, 1 Jul 2020 10:24:33 -0400
Message-ID: <8927EC7B-76F2-4DFC-9AD8-EC2F15E3CEF9@nvidia.com>
In-Reply-To: <fdfeb902-e363-f8b5-4c63-df224b9817fa@intel.com>
References: <20200629234503.749E5340@viggo.jf.intel.com>
 <CALvZod55OFzOozsDbyTkUh1uZEobo4CZ=+8JgrJJHw8QuWh0hw@mail.gmail.com>
 <654d785f-3fe5-d8bd-86bf-bf7431527184@intel.com>
 <CALvZod5-YuMC8b6rs=a_ahh0WF14wgKJBW18CARJtGa_bYUn0Q@mail.gmail.com>
 <fdfeb902-e363-f8b5-4c63-df224b9817fa@intel.com>
MIME-Version: 1.0
Content-Type: multipart/signed;
        boundary="=_MailMate_ED97B0CF-ABF2-4CF2-859F-B952D1167A95_=";
        micalg=pgp-sha512; protocol="application/pgp-signature"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

--=_MailMate_ED97B0CF-ABF2-4CF2-859F-B952D1167A95_=
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On 30 Jun 2020, at 15:31, Dave Hansen wrote:

>
>
>> BTW is this proposal only for systems having multi-tiers of memory?
>> Can a multi-node DRAM-only system take advantage of this proposal? For=

>> example I have a system with two DRAM nodes running two jobs
>> hardwalled to each node. For each job the other node is kind of
>> low-tier memory. If I can describe the per-job demotion paths then
>> these jobs can take advantage of this proposal during occasional
>> peaks.
>
> I don't see any reason it could not work there.  There would just need
> to be a way to set up a different demotion path policy that what was
> done here.

We might need a different threshold (or GFP flag) for allocating new page=
s
in remote node for demotion. Otherwise, we could
see scenarios like: two nodes in a system are almost full and Node A is
trying to demote some pages to Node B, which triggers page demotion from
Node B to Node A. Then, we might be able to avoid a demotion cycle by not=

allowing Node A to demote pages again but swapping pages to disk when Nod=
e B
is demoting its pages to Node A, but this still leads to a long reclaim p=
ath
compared to making Node A swapping to disk directly. In such cases, Node =
A
should just swap pages to disk without bothering Node B at all.

Maybe something like GFP_DEMOTION flag for allocating pages for demotion =
and
the flag requires more free pages available in the destination node to
avoid the situation above?


=E2=80=94
Best Regards,
Yan Zi

--=_MailMate_ED97B0CF-ABF2-4CF2-859F-B952D1167A95_=
Content-Description: OpenPGP digital signature
Content-Disposition: attachment; filename="signature.asc"
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQJDBAEBCgAtFiEEh7yFAW3gwjwQ4C9anbJR82th+ooFAl78nKEPHHppeUBudmlk
aWEuY29tAAoJEJ2yUfNrYfqKE1AQAIVa+5JqVcHM2uFniFUSGzVgzIka+D1JSM6W
p5gxj3U6mRrFNrSbimOhZUP6qsZEiYlPIR0CCwxAZdkx68CUOFnQvC1pmCljm50/
9rjmttuqcEjeW+vh7CWgmln/gVm250hyIKj726i8C8xnNueMxl52++Z9KNtUBuHy
PFnnpkuWbL2e0XqpwJYmCQpvAchnzGFRhk9xiqz73HirnzOaSgGfzhQJFRs7u1uz
gvIy2rOGX/4n/Ia+4LHdFgp3FwHP4gXzN6v3uoW5tOGtv1qXER99RRly3oIGG2jw
BfQK3rYvfGT7h5sLsuQ16HMXpOrDsy/myHJVzCyO6pnczD6gUzsFHWBGI5YcVzKQ
vxDpJsl3Iz+x5ue3Ppm/hI/Tud5BmDHCS7p4XyXbn3cP7ijDB4eGj+F4j5rDiVTC
Xi6VBAn7gKpAOF+Iwr2DFUAjkwleUn8tpGK4Tuuf2niWu/yQ5A0oskcuJkKoHP9W
5yLgjWnv7KYn0J1zyKw5g76oopU3WzpYHN7iVve7EYz8YmKKRlv3R1gpQi/pwo6x
bgtvVk/MyIcwkMl75EA0elFe98i8M/y65G4ZZ22Ak8wxR/2v7FciSoOkrSbiXVSm
Rqyh6fuwlJ8jG7YeDpvNCVgYpQDubDHAyyTwDrNi1pc+z2VaghbgbJUKHAyJe0tY
AnJGE+T4
=hhli
-----END PGP SIGNATURE-----

--=_MailMate_ED97B0CF-ABF2-4CF2-859F-B952D1167A95_=--