On Tue, May 21, 2019 at 11:21:18AM +0300, Mike Rapoport wrote:
> On Mon, May 20, 2019 at 02:24:01PM -0700, Randy Dunlap wrote:
> > From: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> >
> > Fix Sphinx warnings in Documentation/vm/hmm.rst by using "::"
> > notation and inserting a blank line. Also add a missing ';'.
> >
> > Documentation/vm/hmm.rst:292: WARNING: Unexpected indentation.
> > Documentation/vm/hmm.rst:300: WARNING: Unexpected indentation.
> >
> > Fixes: 023a019a9b4e ("mm/hmm: add default fault flags to avoid the need to pre-fill pfns arrays")
> >
> > Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> > Cc: Jérôme Glisse <jglisse@xxxxxxxxxx>
> > Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
>
> Reviewed-by: Mike Rapoport <rppt@xxxxxxxxxxxxx>
Reviewed-by: Jérôme Glisse <jglisse@xxxxxxxxxx>
>
> > ---
> > Documentation/vm/hmm.rst | 8 +++++---
> > 1 file changed, 5 insertions(+), 3 deletions(-)
> >
> > --- lnx-52-rc1.orig/Documentation/vm/hmm.rst
> > +++ lnx-52-rc1/Documentation/vm/hmm.rst
> > @@ -288,15 +288,17 @@ For instance if the device flags for dev
> > WRITE (1 << 62)
> >
> > Now let say that device driver wants to fault with at least read a range then
> > -it does set:
> > - range->default_flags = (1 << 63)
> > +it does set::
> > +
> > + range->default_flags = (1 << 63);
> > range->pfn_flags_mask = 0;
> >
> > and calls hmm_range_fault() as described above. This will fill fault all page
> > in the range with at least read permission.
> >
> > Now let say driver wants to do the same except for one page in the range for
> > -which its want to have write. Now driver set:
> > +which its want to have write. Now driver set::
> > +
> > range->default_flags = (1 << 63);
> > range->pfn_flags_mask = (1 << 62);
> > range->pfns[index_of_write] = (1 << 62);
> >
> >
>
> --
> Sincerely yours,
> Mike.
>
