Re: [PATCH] x86/mm/pti: remove dead logic during user pagetable population

2018-01-07 Thread Jike Song
On Sun, Jan 7, 2018 at 5:48 PM, Thomas Gleixner wrote: > On Sun, 7 Jan 2018, Jike Song wrote: >> On Sun, Jan 7, 2018 at 3:33 AM, Thomas Gleixner wrote: >> > On Sun, 7 Jan 2018, Jike Song wrote: >> > >> > Care to explain why you think this is not needed? >>

Re: [PATCH] x86/mm/pti: remove dead logic during user pagetable population

2018-01-07 Thread Jike Song
On Sun, Jan 7, 2018 at 5:48 PM, Thomas Gleixner wrote: > On Sun, 7 Jan 2018, Jike Song wrote: >> On Sun, Jan 7, 2018 at 3:33 AM, Thomas Gleixner wrote: >> > On Sun, 7 Jan 2018, Jike Song wrote: >> > >> > Care to explain why you think this is not needed? >> > >> >> Hi Thomas, >> >> Look at one of

Re: [PATCH] x86/mm/pti: remove dead logic during user pagetable population

2018-01-07 Thread Thomas Gleixner
On Sun, 7 Jan 2018, Jike Song wrote: > On Sun, Jan 7, 2018 at 3:33 AM, Thomas Gleixner wrote: > > On Sun, 7 Jan 2018, Jike Song wrote: > > > > Care to explain why you think this is not needed? > > > > Hi Thomas, > > Look at one of the original code snippets: > > 162

Re: [PATCH] x86/mm/pti: remove dead logic during user pagetable population

2018-01-07 Thread Thomas Gleixner
On Sun, 7 Jan 2018, Jike Song wrote: > On Sun, Jan 7, 2018 at 3:33 AM, Thomas Gleixner wrote: > > On Sun, 7 Jan 2018, Jike Song wrote: > > > > Care to explain why you think this is not needed? > > > > Hi Thomas, > > Look at one of the original code snippets: > > 162 if

Re: [PATCH] x86/mm/pti: remove dead logic during user pagetable population

2018-01-06 Thread Jike Song
On Sun, Jan 7, 2018 at 4:03 AM, Willy Tarreau wrote: > On Sun, Jan 07, 2018 at 01:50:59AM +0800, Jike Song wrote: >> Signed-off-by: Jike Song > > It would be nice to have a commit message, particularly in this quite > sensitive series... Yes that's useful, will

Re: [PATCH] x86/mm/pti: remove dead logic during user pagetable population

2018-01-06 Thread Jike Song
On Sun, Jan 7, 2018 at 4:03 AM, Willy Tarreau wrote: > On Sun, Jan 07, 2018 at 01:50:59AM +0800, Jike Song wrote: >> Signed-off-by: Jike Song > > It would be nice to have a commit message, particularly in this quite > sensitive series... Yes that's useful, will add it in v2 :) > > Willy --

Re: [PATCH] x86/mm/pti: remove dead logic during user pagetable population

2018-01-06 Thread Jike Song
On Sun, Jan 7, 2018 at 3:33 AM, Thomas Gleixner wrote: > On Sun, 7 Jan 2018, Jike Song wrote: > > Care to explain why you think this is not needed? > Hi Thomas, Look at one of the original code snippets: 162 if (pgd_none(*pgd)) { 163 unsigned

Re: [PATCH] x86/mm/pti: remove dead logic during user pagetable population

2018-01-06 Thread Jike Song
On Sun, Jan 7, 2018 at 3:33 AM, Thomas Gleixner wrote: > On Sun, 7 Jan 2018, Jike Song wrote: > > Care to explain why you think this is not needed? > Hi Thomas, Look at one of the original code snippets: 162 if (pgd_none(*pgd)) { 163 unsigned long new_p4d_page =

Re: [PATCH] x86/mm/pti: remove dead logic during user pagetable population

2018-01-06 Thread Willy Tarreau
On Sun, Jan 07, 2018 at 01:50:59AM +0800, Jike Song wrote: > Signed-off-by: Jike Song It would be nice to have a commit message, particularly in this quite sensitive series... Willy

Re: [PATCH] x86/mm/pti: remove dead logic during user pagetable population

2018-01-06 Thread Willy Tarreau
On Sun, Jan 07, 2018 at 01:50:59AM +0800, Jike Song wrote: > Signed-off-by: Jike Song It would be nice to have a commit message, particularly in this quite sensitive series... Willy

Re: [PATCH] x86/mm/pti: remove dead logic during user pagetable population

2018-01-06 Thread Thomas Gleixner
On Sun, 7 Jan 2018, Jike Song wrote: Care to explain why you think this is not needed? Thanks, tglx

Re: [PATCH] x86/mm/pti: remove dead logic during user pagetable population

2018-01-06 Thread Thomas Gleixner
On Sun, 7 Jan 2018, Jike Song wrote: Care to explain why you think this is not needed? Thanks, tglx

[PATCH] x86/mm/pti: remove dead logic during user pagetable population

2018-01-06 Thread Jike Song
Signed-off-by: Jike Song --- arch/x86/mm/pti.c | 28 1 file changed, 4 insertions(+), 24 deletions(-) diff --git a/arch/x86/mm/pti.c b/arch/x86/mm/pti.c index 43d4a4a29037..dc611d039bd5 100644 --- a/arch/x86/mm/pti.c +++ b/arch/x86/mm/pti.c @@

[PATCH] x86/mm/pti: remove dead logic during user pagetable population

2018-01-06 Thread Jike Song
Signed-off-by: Jike Song --- arch/x86/mm/pti.c | 28 1 file changed, 4 insertions(+), 24 deletions(-) diff --git a/arch/x86/mm/pti.c b/arch/x86/mm/pti.c index 43d4a4a29037..dc611d039bd5 100644 --- a/arch/x86/mm/pti.c +++ b/arch/x86/mm/pti.c @@ -164,12 +164,7 @@