Message ID | 20230825133546.249683-1-marpagan@redhat.com (mailing list archive) |
---|---|
State | New |
Headers | show |
Series | docs/mm: remove references to hmm_mirror ops and clean typos | expand |
On Fri, Aug 25, 2023 at 03:35:46PM +0200, Marco Pagani wrote: > Clean typos and remove the reference to the sync_cpu_device_pagetables() > callback since all hmm_mirror ops have been removed. > > Fixes: a22dd506400d ("mm/hmm: remove hmm_mirror and related") > Signed-off-by: Marco Pagani <marpagan@redhat.com> > Reviewed-by: Mika Penttilä <mpenttil@redhat.com> > --- > Documentation/mm/hmm.rst | 11 +---------- > 1 file changed, 1 insertion(+), 10 deletions(-) Reviewed-by: Jason Gunthorpe <jgg@nvidia.com> Jason
Marco Pagani <marpagan@redhat.com> writes: > Clean typos and remove the reference to the sync_cpu_device_pagetables() > callback since all hmm_mirror ops have been removed. > > Fixes: a22dd506400d ("mm/hmm: remove hmm_mirror and related") > Signed-off-by: Marco Pagani <marpagan@redhat.com> > Reviewed-by: Mika Penttilä <mpenttil@redhat.com> > --- > Documentation/mm/hmm.rst | 11 +---------- > 1 file changed, 1 insertion(+), 10 deletions(-) Applied, thanks. jon
diff --git a/Documentation/mm/hmm.rst b/Documentation/mm/hmm.rst index 9aa512c3a12c..1e0f2b2f58fb 100644 --- a/Documentation/mm/hmm.rst +++ b/Documentation/mm/hmm.rst @@ -163,16 +163,7 @@ use:: It will trigger a page fault on missing or read-only entries if write access is requested (see below). Page faults use the generic mm page fault code path just -like a CPU page fault. - -Both functions copy CPU page table entries into their pfns array argument. Each -entry in that array corresponds to an address in the virtual range. HMM -provides a set of flags to help the driver identify special CPU page table -entries. - -Locking within the sync_cpu_device_pagetables() callback is the most important -aspect the driver must respect in order to keep things properly synchronized. -The usage pattern is:: +like a CPU page fault. The usage pattern is:: int driver_populate_range(...) {