Commit 157816f3 authored by Jason Gunthorpe's avatar Jason Gunthorpe

mm/hmm: Do not use list*_rcu() for hmm->ranges

This list is always read and written while holding hmm->lock so there is
no need for the confusing _rcu annotations.
Signed-off-by: default avatarJason Gunthorpe <jgg@mellanox.com>
Reviewed-by: default avatarJérôme Glisse <jglisse@redhat.com>
Reviewed-by: default avatarJohn Hubbard <jhubbard@nvidia.com>
Acked-by: default avatarSouptick Joarder <jrdr.linux@gmail.com>
Reviewed-by: default avatarRalph Campbell <rcampbell@nvidia.com>
Reviewed-by: default avatarIra Weiny <iweiny@intel.com>
Reviewed-by: default avatarChristoph Hellwig <hch@lst.de>
Tested-by: default avatarPhilip Yang <Philip.Yang@amd.com>
parent 378a6040
...@@ -912,7 +912,7 @@ int hmm_range_register(struct hmm_range *range, ...@@ -912,7 +912,7 @@ int hmm_range_register(struct hmm_range *range,
range->hmm = hmm; range->hmm = hmm;
kref_get(&hmm->kref); kref_get(&hmm->kref);
list_add_rcu(&range->list, &hmm->ranges); list_add(&range->list, &hmm->ranges);
/* /*
* If there are any concurrent notifiers we have to wait for them for * If there are any concurrent notifiers we have to wait for them for
...@@ -942,7 +942,7 @@ void hmm_range_unregister(struct hmm_range *range) ...@@ -942,7 +942,7 @@ void hmm_range_unregister(struct hmm_range *range)
return; return;
mutex_lock(&hmm->lock); mutex_lock(&hmm->lock);
list_del_rcu(&range->list); list_del(&range->list);
mutex_unlock(&hmm->lock); mutex_unlock(&hmm->lock);
/* Drop reference taken by hmm_range_register() */ /* Drop reference taken by hmm_range_register() */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment