From: Shuah Khan <shuah@kernel.org>
To: Pintu Agarwal <pintu.ping@gmail.com>,
linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org,
labbott@redhat.com, gregkh@linuxfoundation.org,
dvhart@infradead.org, bamvor.zhangjian@linaro.org,
Shuah Khan <shuahkh@osg.samsung.com>,
Shuah Khan <shuah@kernel.org>
Subject: Re: [PATCHv3 1/1] [tools/selftests]: android/ion: userspace test utility for ion buffer sharing
Date: Tue, 17 Oct 2017 08:18:28 -0600 [thread overview]
Message-ID: <dde862a6-fe72-750b-bba7-5c53c939c244@kernel.org> (raw)
In-Reply-To: <1507981006-7430-1-git-send-email-pintu.ping@gmail.com>
On 10/14/2017 05:36 AM, 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 (with SCM_RIGHTS) as IPC to transfer an FD to
> another process to share the same buffer.
> This utility demonstrates how ION buffer sharing can be implemented between
> two user space processes, using various heap types.
>
> This utility is made to be run as part of kselftest framework in kernel.
> The utility is verified on Ubuntu-32 bit system with Linux Kernel 4.14,
> using ION system heap and CMA heap.
>
> For more information about the utility please check the README file.
>
> Signed-off-by: Pintu Agarwal <pintu.ping@gmail.com>
> ---
> tools/testing/selftests/Makefile | 3 +-
> tools/testing/selftests/android/Makefile | 44 ++++
> tools/testing/selftests/android/ion/.gitignore | 2 +
> tools/testing/selftests/android/ion/Makefile | 16 ++
> tools/testing/selftests/android/ion/README | 132 +++++++++++
> tools/testing/selftests/android/ion/config | 3 +
> tools/testing/selftests/android/ion/ion_test.sh | 61 +++++
> .../testing/selftests/android/ion/ionapp_export.c | 151 ++++++++++++
> .../testing/selftests/android/ion/ionapp_import.c | 88 +++++++
> tools/testing/selftests/android/ion/ionutils.c | 259 +++++++++++++++++++++
> tools/testing/selftests/android/ion/ionutils.h | 55 +++++
> tools/testing/selftests/android/ion/ipcsocket.c | 227 ++++++++++++++++++
> tools/testing/selftests/android/ion/ipcsocket.h | 35 +++
> tools/testing/selftests/android/run.sh | 3 +
> 14 files changed, 1078 insertions(+), 1 deletion(-)
> create mode 100644 tools/testing/selftests/android/Makefile
> create mode 100644 tools/testing/selftests/android/ion/.gitignore
> create mode 100644 tools/testing/selftests/android/ion/Makefile
> create mode 100644 tools/testing/selftests/android/ion/README
> create mode 100644 tools/testing/selftests/android/ion/config
> create mode 100755 tools/testing/selftests/android/ion/ion_test.sh
> create mode 100644 tools/testing/selftests/android/ion/ionapp_export.c
> create mode 100644 tools/testing/selftests/android/ion/ionapp_import.c
> create mode 100644 tools/testing/selftests/android/ion/ionutils.c
> create mode 100644 tools/testing/selftests/android/ion/ionutils.h
> create mode 100644 tools/testing/selftests/android/ion/ipcsocket.c
> create mode 100644 tools/testing/selftests/android/ion/ipcsocket.h
> create mode 100755 tools/testing/selftests/android/run.sh
>
Looks good to me. I am hoping you were able to verify that the following
use-cases work:
make kselftest
make O=dir kselftest
I plan to get this into 4.15-rc1
thanks,
-- Shuah
next prev parent reply other threads:[~2017-10-17 14:18 UTC|newest]
Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-26 18:08 [PATCH 1/1] [tools]: " Pintu Agarwal
2017-09-26 18:50 ` Pintu Kumar
2017-09-26 19:22 ` Laura Abbott
2017-09-27 14:24 ` Pintu Kumar
2017-09-28 18:11 ` Pintu Kumar
2017-09-28 18:16 ` Pintu Kumar
2017-09-30 4:04 ` Pintu Kumar
2017-10-03 16:48 ` [PATCHv2 " Pintu Agarwal
2017-10-04 0:12 ` Laura Abbott
2017-10-04 11:29 ` Pintu Kumar
2017-10-04 22:07 ` Laura Abbott
2017-10-06 14:28 ` Pintu Kumar
2017-10-09 14:17 ` Pintu Kumar
2017-10-09 21:19 ` Laura Abbott
2017-10-04 7:20 ` Greg KH
2017-10-04 10:54 ` Pintu Kumar
2017-10-14 11:36 ` [PATCHv3 1/1] [tools/selftests]: " Pintu Agarwal
2017-10-17 14:18 ` Shuah Khan [this message]
2017-10-17 20:21 ` Laura Abbott
2017-10-17 20:58 ` Shuah Khan
2017-10-18 10:38 ` Pintu Kumar
2017-10-20 14:40 ` Shuah Khan
2017-10-20 15:18 ` Pintu Kumar
2017-10-20 19:21 ` Shuah Khan
2017-10-23 13:41 ` Pintu Kumar
2017-10-23 23:26 ` [PATCHv4 " Pintu Agarwal
2017-10-27 5:34 ` Pintu Kumar
2017-10-27 8:45 ` Greg Kroah-Hartman
2017-10-27 10:00 ` Pintu Kumar
2017-10-29 14:21 ` [lkp-robot] [android/ion] 5fb70554d6: kernel_selftests.android.make_fail kernel test robot
2017-10-30 7:12 ` Pintu Kumar
2017-10-30 21:02 ` Laura Abbott
2017-10-31 10:21 ` Pintu Kumar
2017-10-31 21:58 ` Laura Abbott
2017-11-01 10:12 ` Pintu Kumar
2017-11-01 10:30 ` Greg Kroah-Hartman
2017-11-01 15:04 ` Shuah Khan
2017-11-01 16:26 ` Pintu Kumar
2017-11-01 16:57 ` Shuah Khan
2017-11-01 17:14 ` Pintu Kumar
2017-11-01 22:38 ` Shuah Khan
2017-11-02 8:53 ` Greg Kroah-Hartman
2017-11-02 14:50 ` Shuah Khan
2017-10-30 22:56 ` [PATCHv4 1/1] [tools/selftests]: android/ion: userspace test utility for ion buffer sharing Laura Abbott
2017-10-31 9:49 ` Pintu Kumar
2017-11-01 17:00 ` [PATCHv5 " Pintu Agarwal
2017-11-02 15:07 ` Shuah Khan
2017-11-02 16:15 ` Pintu Kumar
2017-11-02 16:19 ` Shuah Khan
2017-11-02 17:53 ` Shuah Khan
2017-11-02 23:54 ` Laura Abbott
2017-11-03 0:32 ` Shuah Khan
2017-11-03 4:36 ` Pintu Kumar
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=dde862a6-fe72-750b-bba7-5c53c939c244@kernel.org \
--to=shuah@kernel.org \
--cc=bamvor.zhangjian@linaro.org \
--cc=dvhart@infradead.org \
--cc=gregkh@linuxfoundation.org \
--cc=labbott@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-kselftest@vger.kernel.org \
--cc=pintu.ping@gmail.com \
--cc=shuahkh@osg.samsung.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®