Re: [Xen-devel] [PATCH XEN v8 24/29] tools/libs/call: linux: touch newly allocated pages after madvise lockdown

2016-01-19 Thread Ian Campbell
On Tue, 2016-01-19 at 15:49 +, Wei Liu wrote: >  > I see. I wonder why we didn't use mlock(2) in Linux too in the first > place. We did, but mlock(2) on Linux doesn't guarantee that there will be no page faults, it only guarantees that there will be no page faults which require I/O to satisfy

Re: [Xen-devel] [PATCH XEN v8 24/29] tools/libs/call: linux: touch newly allocated pages after madvise lockdown

2016-01-19 Thread Wei Liu
On Tue, Jan 19, 2016 at 03:03:31PM +, Ian Campbell wrote: > On Tue, 2016-01-19 at 14:58 +, Wei Liu wrote: > > On Tue, Jan 19, 2016 at 03:54:54PM +0100, Roger Pau Monné wrote: > > > El 19/01/16 a les 14.24, Wei Liu ha escrit: > > > > On Fri, Jan 15, 2016 at 01:23:03PM +, Ian Campbell wro

Re: [Xen-devel] [PATCH XEN v8 24/29] tools/libs/call: linux: touch newly allocated pages after madvise lockdown

2016-01-19 Thread Ian Campbell
On Tue, 2016-01-19 at 14:58 +, Wei Liu wrote: > On Tue, Jan 19, 2016 at 03:54:54PM +0100, Roger Pau Monné wrote: > > El 19/01/16 a les 14.24, Wei Liu ha escrit: > > > On Fri, Jan 15, 2016 at 01:23:03PM +, Ian Campbell wrote: > > > > This avoids a potential issue with a fork after allocation

Re: [Xen-devel] [PATCH XEN v8 24/29] tools/libs/call: linux: touch newly allocated pages after madvise lockdown

2016-01-19 Thread Wei Liu
On Tue, Jan 19, 2016 at 03:54:54PM +0100, Roger Pau Monné wrote: > El 19/01/16 a les 14.24, Wei Liu ha escrit: > > On Fri, Jan 15, 2016 at 01:23:03PM +, Ian Campbell wrote: > >> This avoids a potential issue with a fork after allocation but before > >> madvise. > >> > >> Signed-off-by: Ian Camp

Re: [Xen-devel] [PATCH XEN v8 24/29] tools/libs/call: linux: touch newly allocated pages after madvise lockdown

2016-01-19 Thread Roger Pau Monné
El 19/01/16 a les 14.24, Wei Liu ha escrit: > On Fri, Jan 15, 2016 at 01:23:03PM +, Ian Campbell wrote: >> This avoids a potential issue with a fork after allocation but before >> madvise. >> >> Signed-off-by: Ian Campbell >> --- >> v7: New, replacing "tools/libs/call: linux: avoid forking bet

Re: [Xen-devel] [PATCH XEN v8 24/29] tools/libs/call: linux: touch newly allocated pages after madvise lockdown

2016-01-19 Thread Wei Liu
On Tue, Jan 19, 2016 at 01:40:55PM +, Ian Campbell wrote: > On Tue, 2016-01-19 at 13:24 +, Wei Liu wrote: > > On Fri, Jan 15, 2016 at 01:23:03PM +, Ian Campbell wrote: > > > This avoids a potential issue with a fork after allocation but before > > > madvise. > > > > > > Signed-off-by:

Re: [Xen-devel] [PATCH XEN v8 24/29] tools/libs/call: linux: touch newly allocated pages after madvise lockdown

2016-01-19 Thread Ian Campbell
On Tue, 2016-01-19 at 13:24 +, Wei Liu wrote: > On Fri, Jan 15, 2016 at 01:23:03PM +, Ian Campbell wrote: > > This avoids a potential issue with a fork after allocation but before > > madvise. > > > > Signed-off-by: Ian Campbell > > --- > > v7: New, replacing "tools/libs/call: linux: avoi

Re: [Xen-devel] [PATCH XEN v8 24/29] tools/libs/call: linux: touch newly allocated pages after madvise lockdown

2016-01-19 Thread Wei Liu
On Fri, Jan 15, 2016 at 01:23:03PM +, Ian Campbell wrote: > This avoids a potential issue with a fork after allocation but before > madvise. > > Signed-off-by: Ian Campbell > --- > v7: New, replacing "tools/libs/call: linux: avoid forking between mmap > and madvise". > --- > tools/libs/c

[Xen-devel] [PATCH XEN v8 24/29] tools/libs/call: linux: touch newly allocated pages after madvise lockdown

2016-01-15 Thread Ian Campbell
This avoids a potential issue with a fork after allocation but before madvise. Signed-off-by: Ian Campbell --- v7: New, replacing "tools/libs/call: linux: avoid forking between mmap and madvise". --- tools/libs/call/linux.c | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-)