diff options
| -rw-r--r-- | mm/rmap.c | 13 |
1 files changed, 8 insertions, 5 deletions
| @@ -405,6 +405,7 @@ out: | |||
| 405 | struct anon_vma *page_lock_anon_vma(struct page *page) | 405 | struct anon_vma *page_lock_anon_vma(struct page *page) |
| 406 | { | 406 | { |
| 407 | struct anon_vma *anon_vma = NULL; | 407 | struct anon_vma *anon_vma = NULL; |
| 408 | struct anon_vma *root_anon_vma; | ||
| 408 | unsigned long anon_mapping; | 409 | unsigned long anon_mapping; |
| 409 | 410 | ||
| 410 | rcu_read_lock(); | 411 | rcu_read_lock(); |
| @@ -415,13 +416,15 @@ struct anon_vma *page_lock_anon_vma(struct page *page) | |||
| 415 | goto out; | 416 | goto out; |
| 416 | 417 | ||
| 417 | anon_vma = (struct anon_vma *) (anon_mapping - PAGE_MAPPING_ANON); | 418 | anon_vma = (struct anon_vma *) (anon_mapping - PAGE_MAPPING_ANON); |
| 418 | if (mutex_trylock(&anon_vma->root->mutex)) { | 419 | root_anon_vma = ACCESS_ONCE(anon_vma->root); |
| 420 | if (mutex_trylock(&root_anon_vma->mutex)) { | ||
| 419 | /* | 421 | /* |
| 420 | * If we observe a !0 refcount, then holding the lock ensures | 422 | * If the page is still mapped, then this anon_vma is still |
| 421 | * the anon_vma will not go away, see __put_anon_vma(). | 423 | * its anon_vma, and holding the mutex ensures that it will |
| 424 | * not go away, see __put_anon_vma(). | ||
| 422 | */ | 425 | */ |
| 423 | if (!atomic_read(&anon_vma->refcount)) { | 426 | if (!page_mapped(page)) { |
| 424 | anon_vma_unlock(anon_vma); | 427 | mutex_unlock(&root_anon_vma->mutex); |
| 425 | anon_vma = NULL; | 428 | anon_vma = NULL; |
| 426 | } | 429 | } |
| 427 | goto out; | 430 | goto out; |
