Messages in this thread | | | From | Pintu Kumar <> | Date | Wed, 4 Oct 2017 16:24:33 +0530 | Subject | Re: [PATCHv2 1/1] [tools]: android/ion: userspace test utility for ion buffer sharing |
| |
On Wed, Oct 4, 2017 at 12:50 PM, Greg KH <gregkh@linuxfoundation.org> wrote: > On Tue, Oct 03, 2017 at 12:48:59PM -0400, Pintu Agarwal wrote: >> This is a test utility to verify ION buffer sharing in user space >> between 2 independent processes. >> It uses unix domain socket as IPC to transfer an FD to another process >> and install it. >> >> This utility demonstrates how ION buffer sharing can be implemented between >> two user space processes, using various heap ids. >> >> This utility is verified on Ubuntu 32-bit machine using 2 independent >> process such as: ionapp_export (server) and ionapp_import (client). >> First the server needs to be run to export FD to the client. >> This utility works only if /dev/ion interface is present. >> >> Here is a sample demo example: >> >> linux/tools/android/ion$ sudo ./ionapp_export.out -i 1 -s 10 >> heap_type: 2, heap_size: 10 >> Fill buffer content: >> 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd >> Sharing fd: 6, Client fd: 5 >> <ion_close_buffer_fd>: buffer release successfully.... >> >> linux/tools/android/ion$ sudo ./ionapp_import.out >> Received buffer fd: 4 >> Read buffer content: >> 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd >> 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 >> 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 0x0 >> Fill buffer content: >> 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd >> 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd >> 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd 0xfd >> <ion_close_buffer_fd>: buffer release successfully.... > > Can you tie this into the kselftest framework, to make automated tests > of this kernel feature easier? > Ok, I can explore more about kselftest framework and make it part of it in the next version. I can move the whole thing later into: tools/testing/selftests/<android>/<ion>/ Thanks for your feedback.
> thanks, > > greg k-h
| |