Re: [uml-user] [PATCH v4 2/2] powerpc/mm: Tracking vDSO remap

On 26/03/2015 19:55, Ingo Molnar wrote: > > * Laurent Dufour wrote: > >> +{ >> +unsigned long vdso_end, vdso_start; >> + >> +if (!mm->context.vdso_base) >> +return; >> +vdso_start = mm->context.vdso_base; >> +

Re: [uml-user] [PATCH v3 2/2] powerpc/mm: Tracking vDSO remap

On 26/03/2015 15:17, Ingo Molnar wrote: > > * Laurent Dufour wrote: > >>> I argue we should use the right condition to clear vdso_base: if >>> the vDSO gets at least partially unmapped. Otherwise there's >>> little point in the whole patch: either co

[uml-user] [PATCH v4 2/2] powerpc/mm: Tracking vDSO remap

remapping and unmapping. Moving or unmapping partially the vDSO lead to invalidate it from the kernel point of view. Signed-off-by: Laurent Dufour --- arch/powerpc/include/asm/mmu_context.h | 32 +++- arch/powerpc/kernel/vdso.c | 39

[uml-user] [PATCH v4 1/2] mm: Introducing arch_remap hook

, unsigned long new_end); Signed-off-by: Laurent Dufour --- mm/mremap.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/mm/mremap.c b/mm/mremap.c index 57dadc025c64..bafc234db45c 100644 --- a/mm/mremap.c +++ b/mm/mremap.c @@ -25,6 +25,7 @@ #include #include

[uml-user] [PATCH v4 0/2] Tracking user space vDSO remaping

the second patch. Thanks again, Ingo. Changes in v2: -- - Following the Ingo Molnar's advice, enabling the call to arch_remap through the __HAVE_ARCH_REMAP macro. This reduces considerably the first patch. Laurent Dufour (2): mm: Introducing arch_remap hook powerpc/mm

Re: [uml-user] [PATCH v3 2/2] powerpc/mm: Tracking vDSO remap

On 26/03/2015 10:48, Ingo Molnar wrote: > > * Benjamin Herrenschmidt wrote: > +#define __HAVE_ARCH_REMAP +static inline void arch_remap(struct mm_struct *mm, +unsigned long old_start, unsigned long old_end, +unsigned long new_st

Re: [uml-user] [PATCH v3 2/2] powerpc/mm: Tracking vDSO remap

On 26/03/2015 10:43, Ingo Molnar wrote: > > * Benjamin Herrenschmidt wrote: > >> On Wed, 2015-03-25 at 19:36 +0100, Ingo Molnar wrote: >>> * Ingo Molnar wrote: >>> > +#define __HAVE_ARCH_REMAP > +static inline void arch_remap(struct mm_struct *mm, > + unsigned

[uml-user] [PATCH v3 0/2] Tracking user space vDSO remaping

ing the call to arch_remap through the __HAVE_ARCH_REMAP macro. This reduces considerably the first patch. Laurent Dufour (2): mm: Introducing arch_remap hook powerpc/mm: Tracking vDSO remap arch/powerpc/include/asm/mmu_context.h | 36

[uml-user] [PATCH v3 1/2] mm: Introducing arch_remap hook

, unsigned long new_end); Signed-off-by: Laurent Dufour --- mm/mremap.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/mm/mremap.c b/mm/mremap.c index 57dadc025c64..bafc234db45c 100644 --- a/mm/mremap.c +++ b/mm/mremap.c @@ -25,6 +25,7 @@ #include #include

[uml-user] [PATCH v3 2/2] powerpc/mm: Tracking vDSO remap

remapping and unmapping. Signed-off-by: Laurent Dufour --- arch/powerpc/include/asm/mmu_context.h | 36 +- 1 file changed, 35 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/include/asm/mmu_context.h b/arch/powerpc/include/asm/mmu_context.h index 73382eba02dc

[uml-user] [PATCH v2 1/2] mm: Introducing arch_remap hook

, unsigned long new_end); Signed-off-by: Laurent Dufour --- mm/mremap.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/mm/mremap.c b/mm/mremap.c index 57dadc025c64..bafc234db45c 100644 --- a/mm/mremap.c +++ b/mm/mremap.c @@ -25,6 +25,7 @@ #include #include

Re: [uml-user] [PATCH v2 2/2] powerpc/mm: Tracking vDSO remap

On 25/03/2015 13:11, Ingo Molnar wrote: > > * Laurent Dufour wrote: > >> Some processes (CRIU) are moving the vDSO area using the mremap system >> call. As a consequence the kernel reference to the vDSO base address is >> no more valid and the signal return frame b

[uml-user] [PATCH v2 2/2] powerpc/mm: Tracking vDSO remap

remapping and unmapping. Signed-off-by: Laurent Dufour --- arch/powerpc/include/asm/mmu_context.h | 36 +- 1 file changed, 35 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/include/asm/mmu_context.h b/arch/powerpc/include/asm/mmu_context.h index 73382eba02dc

[uml-user] [PATCH v2 0/2] Tracking user space vDSO remaping

lock still hold. The next patch is adding the vDSO remap and unmap tracking to the powerpc architecture. Changes in v2: -- - Following the Ingo Molnar's advice, enabling the call to arch_remap through the __HAVE_ARCH_REMAP macro. This reduces considerably the first patch. La

Re: [uml-user] [PATCH 1/2] mm: Introducing arch_remap hook

On 23/03/2015 09:52, Ingo Molnar wrote: > > * Laurent Dufour wrote: > >> Some architecture would like to be triggered when a memory area is moved >> through the mremap system call. >> >> This patch is introducing a new arch_remap mm hook which is placed in the

[uml-user] [PATCH 0/2] Tracking user space vDSO remaping

lock still hold. The next patch is adding the vDSO remap and unmap tracking to the powerpc architecture. Laurent Dufour (2): mm: Introducing arch_remap hook powerpc/mm: Tracking vDSO remap arch/powerpc/include/asm/mmu_context.h | 35 +++- arch/s390/include/asm/mmu

[uml-user] [PATCH 2/2] powerpc/mm: Tracking vDSO remap

remapping and unmapping. Signed-off-by: Laurent Dufour --- arch/powerpc/include/asm/mmu_context.h | 35 +- 1 file changed, 34 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/include/asm/mmu_context.h b/arch/powerpc/include/asm/mmu_context.h index 73382eba02dc

[uml-user] [PATCH 1/2] mm: Introducing arch_remap hook

build, this patch adds the empty hook definition to the architectures that were not using the generic hook's definition. Signed-off-by: Laurent Dufour --- arch/s390/include/asm/mmu_context.h | 6 ++ arch/um/include/asm/mmu_context.h| 5 + arch/unicore32/includ