Re: [PATCH 2/5] android: binder: Add allocator selftest

2017-09-10 Thread Sherry Yang
Hi Geert, This selftest can't run until userspace has called mmap. Since it is hooked into ioctl, it's probably not worth the effort of making it a module, since ioctl will have to check at runtime whether the selftest module has been loaded, which would add overhead even when the test is not

Re: [PATCH 2/5] android: binder: Add allocator selftest

2017-09-10 Thread Sherry Yang
Hi Geert, This selftest can't run until userspace has called mmap. Since it is hooked into ioctl, it's probably not worth the effort of making it a module, since ioctl will have to check at runtime whether the selftest module has been loaded, which would add overhead even when the test is not

Re: [PATCH 2/5] android: binder: Add allocator selftest

2017-09-10 Thread Geert Uytterhoeven
Hi Sherry, On Wed, Aug 16, 2017 at 2:25 AM, Sherry Yang wrote: > binder_alloc_selftest tests that alloc_new_buf handles page allocation and > deallocation properly when allocate and free buffers. The test allocates 5 > buffers of various sizes to cover all possible page

Re: [PATCH 2/5] android: binder: Add allocator selftest

2017-09-10 Thread Geert Uytterhoeven
Hi Sherry, On Wed, Aug 16, 2017 at 2:25 AM, Sherry Yang wrote: > binder_alloc_selftest tests that alloc_new_buf handles page allocation and > deallocation properly when allocate and free buffers. The test allocates 5 > buffers of various sizes to cover all possible page alignment cases, and >

[PATCH 2/5] android: binder: Add allocator selftest

2017-08-15 Thread Sherry Yang
binder_alloc_selftest tests that alloc_new_buf handles page allocation and deallocation properly when allocate and free buffers. The test allocates 5 buffers of various sizes to cover all possible page alignment cases, and frees the buffers using a list of exhaustive freeing order. Test: boot the

[PATCH 2/5] android: binder: Add allocator selftest

2017-08-15 Thread Sherry Yang
binder_alloc_selftest tests that alloc_new_buf handles page allocation and deallocation properly when allocate and free buffers. The test allocates 5 buffers of various sizes to cover all possible page alignment cases, and frees the buffers using a list of exhaustive freeing order. Test: boot the