Re: [PATCH 02/18] vchecker: introduce the valid access checker

2017-12-01 Thread Joonsoo Kim
On Fri, Dec 01, 2017 at 01:08:13PM +0800, kbuild test robot wrote: > Hi Joonsoo, > > I love your patch! Yet something to improve: Thanks! I will fix all the error from kbuild bot on next spin. Thanks.

Re: [PATCH 02/18] vchecker: introduce the valid access checker

2017-12-01 Thread Joonsoo Kim
On Fri, Dec 01, 2017 at 01:08:13PM +0800, kbuild test robot wrote: > Hi Joonsoo, > > I love your patch! Yet something to improve: Thanks! I will fix all the error from kbuild bot on next spin. Thanks.

Re: [PATCH 02/18] vchecker: introduce the valid access checker

2017-11-30 Thread kbuild test robot
Hi Joonsoo, I love your patch! Yet something to improve: [auto build test ERROR on mmotm/master] [also build test ERROR on v4.15-rc1 next-20171130] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH 02/18] vchecker: introduce the valid access checker

2017-11-30 Thread kbuild test robot
Hi Joonsoo, I love your patch! Yet something to improve: [auto build test ERROR on mmotm/master] [also build test ERROR on v4.15-rc1 next-20171130] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH 02/18] vchecker: introduce the valid access checker

2017-11-28 Thread Joonsoo Kim
On Tue, Nov 28, 2017 at 11:41:08AM -0800, Andi Kleen wrote: > js1...@gmail.com writes: > > > From: Joonsoo Kim > > Looks useful. Essentially unlimited hardware break points, combined > with slab. Thanks!!! > > Didn't do a full review, but noticed some things below. >

Re: [PATCH 02/18] vchecker: introduce the valid access checker

2017-11-28 Thread Joonsoo Kim
On Tue, Nov 28, 2017 at 11:41:08AM -0800, Andi Kleen wrote: > js1...@gmail.com writes: > > > From: Joonsoo Kim > > Looks useful. Essentially unlimited hardware break points, combined > with slab. Thanks!!! > > Didn't do a full review, but noticed some things below. > > + > > + buf =

Re: [PATCH 02/18] vchecker: introduce the valid access checker

2017-11-28 Thread Andi Kleen
js1...@gmail.com writes: > From: Joonsoo Kim Looks useful. Essentially unlimited hardware break points, combined with slab. Didn't do a full review, but noticed some things below. > + > + buf = kmalloc(PAGE_SIZE, GFP_KERNEL); > + if (!buf) > + return

Re: [PATCH 02/18] vchecker: introduce the valid access checker

2017-11-28 Thread Andi Kleen
js1...@gmail.com writes: > From: Joonsoo Kim Looks useful. Essentially unlimited hardware break points, combined with slab. Didn't do a full review, but noticed some things below. > + > + buf = kmalloc(PAGE_SIZE, GFP_KERNEL); > + if (!buf) > + return -ENOMEM; > + > + if