Messages in this thread | | | Date | Wed, 5 Dec 2018 12:52:11 +0200 | From | Mike Rapoport <> | Subject | Re: [RFC PATCH 02/14] mm/hms: heterogenenous memory system (HMS) documentation |
| |
On Mon, Dec 03, 2018 at 06:34:57PM -0500, jglisse@redhat.com wrote: > From: Jérôme Glisse <jglisse@redhat.com> > > Add documentation to what is HMS and what it is for (see patch content). > > Signed-off-by: Jérôme Glisse <jglisse@redhat.com> > Cc: Rafael J. Wysocki <rafael@kernel.org> > Cc: Ross Zwisler <ross.zwisler@linux.intel.com> > Cc: Dan Williams <dan.j.williams@intel.com> > Cc: Dave Hansen <dave.hansen@intel.com> > Cc: Haggai Eran <haggaie@mellanox.com> > Cc: Balbir Singh <balbirs@au1.ibm.com> > Cc: Aneesh Kumar K.V <aneesh.kumar@linux.ibm.com> > Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org> > Cc: Felix Kuehling <felix.kuehling@amd.com> > Cc: Philip Yang <Philip.Yang@amd.com> > Cc: Christian König <christian.koenig@amd.com> > Cc: Paul Blinzer <Paul.Blinzer@amd.com> > Cc: Logan Gunthorpe <logang@deltatee.com> > Cc: John Hubbard <jhubbard@nvidia.com> > Cc: Ralph Campbell <rcampbell@nvidia.com> > Cc: Michal Hocko <mhocko@kernel.org> > Cc: Jonathan Cameron <jonathan.cameron@huawei.com> > Cc: Mark Hairgrove <mhairgrove@nvidia.com> > Cc: Vivek Kini <vkini@nvidia.com> > Cc: Mel Gorman <mgorman@techsingularity.net> > Cc: Dave Airlie <airlied@redhat.com> > Cc: Ben Skeggs <bskeggs@redhat.com> > Cc: Andrea Arcangeli <aarcange@redhat.com> > --- > Documentation/vm/hms.rst | 275 ++++++++++++++++++++++++++++++++++----- > 1 file changed, 246 insertions(+), 29 deletions(-)
This document describes userspace API and it's better to put it into Documentation/admin-guide/mm. The Documentation/vm is more for description of design and implementation.
I've spotted a couple of typos, but I think it doesn't make sense to nitpick about them before v10 or so ;-)
> diff --git a/Documentation/vm/hms.rst b/Documentation/vm/hms.rst > index dbf0f71918a9..bd7c9e8e7077 100644 > --- a/Documentation/vm/hms.rst > +++ b/Documentation/vm/hms.rst
-- Sincerely yours, Mike.
| |