diff options
author | Jérôme Glisse <jglisse@redhat.com> | 2015-05-01 16:03:52 -0400 |
---|---|---|
committer | Jérôme Glisse <jglisse@redhat.com> | 2015-10-20 15:48:07 -0400 |
commit | d9e16c86bf1b6bbbac33566c4d74491f541af1cf (patch) | |
tree | 02386e919ff795291a8394af0d62fd6f04e8f885 | |
parent | b83eacd1b4f27708b33f81fd1d53f89331ef9070 (diff) |
HMM: fork copy migrated memory into system memory for child process.
When forking if process being fork had any memory migrated to some
device memory, we need to make a system copy for the child process.
Latter patches can revisit this and use the same COW semantic for
device memory.
Signed-off-by: Jérôme Glisse <jglisse@redhat.com>
-rw-r--r-- | mm/hmm.c | 38 |
1 files changed, 37 insertions, 1 deletions
@@ -480,7 +480,37 @@ int hmm_mm_fork(struct mm_struct *src_mm, unsigned long start, unsigned long end) { - return -ENOMEM; + unsigned long npages = (end - start) >> PAGE_SHIFT; + struct hmm_event event; + dma_addr_t *dst; + struct hmm *hmm; + pte_t *new_pte; + int ret; + + hmm = hmm_ref(src_mm->hmm); + if (!hmm) + return -EINVAL; + + + dst = kcalloc(npages, sizeof(*dst), GFP_KERNEL); + if (!dst) { + hmm_unref(hmm); + return -ENOMEM; + } + new_pte = kcalloc(npages, sizeof(*new_pte), GFP_KERNEL); + if (!new_pte) { + kfree(dst); + hmm_unref(hmm); + return -ENOMEM; + } + + hmm_event_init(&event, hmm, start, end, HMM_FORK); + ret = hmm_migrate_back(hmm, &event, dst_mm, dst_vma, new_pte, + dst, start, end); + hmm_unref(hmm); + kfree(new_pte); + kfree(dst); + return ret; } EXPORT_SYMBOL(hmm_mm_fork); @@ -656,6 +686,12 @@ static void hmm_mirror_update_pte(struct hmm_mirror *mirror, } if (hmm_pte_test_valid_dev(hmm_pte)) { + /* + * On fork device memory is duplicated so no need to write + * protect it. + */ + if (event->etype == HMM_FORK) + return; *hmm_pte &= event->pte_mask; if (!hmm_pte_test_valid_dev(hmm_pte)) hmm_pt_iter_directory_unref(iter); |