RE: [PATCH v7 09/10] x86, mpx: cleanup unused bound tables

2014-07-23 Thread Ren, Qiaowei
On 2014-07-24, Hansen, Dave wrote: > On 07/23/2014 05:49 PM, Ren, Qiaowei wrote: >> I can check a lot of debug information when one VMA and related >> bounds tables are allocated and freed through adding a lot of >> print() like log into kernel/runtime. Do you think this is enough? > > I thought

Re: [PATCH v7 09/10] x86, mpx: cleanup unused bound tables

2014-07-23 Thread Dave Hansen
On 07/23/2014 05:49 PM, Ren, Qiaowei wrote: > I can check a lot of debug information when one VMA and related > bounds tables are allocated and freed through adding a lot of print() > like log into kernel/runtime. Do you think this is enough? I thought the entire reason we grabbed a VM_ flag was t

RE: [PATCH v7 09/10] x86, mpx: cleanup unused bound tables

2014-07-23 Thread Ren, Qiaowei
On 2014-07-24, Hansen, Dave wrote: > On 07/20/2014 10:38 PM, Qiaowei Ren wrote: >> Since the kernel allocated those tables on-demand without userspace >> knowledge, it is also responsible for freeing them when the >> associated mappings go away. >> >> Here, the solution for this issue is to hook

Re: [PATCH v7 09/10] x86, mpx: cleanup unused bound tables

2014-07-23 Thread Dave Hansen
On 07/20/2014 10:38 PM, Qiaowei Ren wrote: > Since the kernel allocated those tables on-demand without userspace > knowledge, it is also responsible for freeing them when the associated > mappings go away. > > Here, the solution for this issue is to hook do_munmap() to check > whether one process

RE: [PATCH v7 09/10] x86, mpx: cleanup unused bound tables

2014-07-21 Thread Zhang, Tianfei
.kernel.org; linux...@kvack.org; Ren, > Qiaowei > Subject: [PATCH v7 09/10] x86, mpx: cleanup unused bound tables > > Since the kernel allocated those tables on-demand without userspace > knowledge, it is also responsible for freeing them when the associated > mappings go away. > > He

[PATCH v7 09/10] x86, mpx: cleanup unused bound tables

2014-07-20 Thread Qiaowei Ren
Since the kernel allocated those tables on-demand without userspace knowledge, it is also responsible for freeing them when the associated mappings go away. Here, the solution for this issue is to hook do_munmap() to check whether one process is MPX enabled. If yes, those bounds tables covered in