Em Fri, 12 Feb 2016 18:39:54 -0700
Shuah Khan <[email protected]> escreveu:

> This test opens user specified Media Device and calls
> MEDIA_IOC_DEVICE_INFO ioctl in a loop once every 10
> seconds. This test is for detecting errors in device
> removal path.
> 
> Usage:
>     sudo ./media_devkref_test -d /dev/mediaX
> 
> While test is running, remove the device and
> ensure there are no use after free errors and
> other Oops in the dmesg. Enable KaSan kernel
> config option for use-after-free error detection.
> 
> Signed-off-by: Shuah Khan <[email protected]>

Looks good to me.

Reviewed-by: Mauro Carvalho Chehab <[email protected]>

> ---
>  tools/testing/selftests/media_tests/.gitignore     |  1 +
>  tools/testing/selftests/media_tests/Makefile       |  7 ++
>  .../selftests/media_tests/media_device_test.c      | 94 
> ++++++++++++++++++++++
>  3 files changed, 102 insertions(+)
>  create mode 100644 tools/testing/selftests/media_tests/.gitignore
>  create mode 100644 tools/testing/selftests/media_tests/Makefile
>  create mode 100644 tools/testing/selftests/media_tests/media_device_test.c
> 
> diff --git a/tools/testing/selftests/media_tests/.gitignore 
> b/tools/testing/selftests/media_tests/.gitignore
> new file mode 100644
> index 0000000..1c07117
> --- /dev/null
> +++ b/tools/testing/selftests/media_tests/.gitignore
> @@ -0,0 +1 @@
> +media_device_test
> diff --git a/tools/testing/selftests/media_tests/Makefile 
> b/tools/testing/selftests/media_tests/Makefile
> new file mode 100644
> index 0000000..7071bcc
> --- /dev/null
> +++ b/tools/testing/selftests/media_tests/Makefile
> @@ -0,0 +1,7 @@
> +TEST_PROGS := media_device_test
> +all: $(TEST_PROGS)
> +
> +include ../lib.mk
> +
> +clean:
> +     rm -fr media_device_test
> diff --git a/tools/testing/selftests/media_tests/media_device_test.c 
> b/tools/testing/selftests/media_tests/media_device_test.c
> new file mode 100644
> index 0000000..a47880b
> --- /dev/null
> +++ b/tools/testing/selftests/media_tests/media_device_test.c
> @@ -0,0 +1,94 @@
> +/*
> + * media_devkref_test.c - Media Controller Device Kref API Test
> + *
> + * Copyright (c) 2016 Shuah Khan <[email protected]>
> + * Copyright (c) 2016 Samsung Electronics Co., Ltd.
> + *
> + * This file is released under the GPLv2.
> + */
> +
> +/*
> + * This file adds a test for Media Controller API.
> + * This test should be run as root and should not be
> + * included in the Kselftest run. This test should be
> + * run when hardware and driver that makes use Media
> + * Controller API are present in the system.
> + *
> + * This test opens user specified Media Device and calls
> + * MEDIA_IOC_DEVICE_INFO ioctl in a loop once every 10
> + * seconds.
> + *
> + * Usage:
> + *   sudo ./media_devkref_test -d /dev/mediaX
> + *
> + *   While test is running, remove the device and
> + *   ensure there are no use after free errors and
> + *   other Oops in the dmesg. Enable KaSan kernel
> + *   config option for use-after-free error detection.
> +*/
> +
> +#include <stdio.h>
> +#include <unistd.h>
> +#include <stdlib.h>
> +#include <errno.h>
> +#include <string.h>
> +#include <fcntl.h>
> +#include <sys/ioctl.h>
> +#include <sys/stat.h>
> +#include <linux/media.h>
> +
> +int main(int argc, char **argv)
> +{
> +     int opt;
> +     char media_device[256];
> +     int count = 0;
> +     struct media_device_info mdi;
> +     int ret;
> +     int fd;
> +
> +     if (argc < 2) {
> +             printf("Usage: %s [-d </dev/mediaX>]\n", argv[0]);
> +             exit(-1);
> +     }
> +
> +     /* Process arguments */
> +     while ((opt = getopt(argc, argv, "d:")) != -1) {
> +             switch (opt) {
> +             case 'd':
> +                     strncpy(media_device, optarg, sizeof(media_device) - 1);
> +                     media_device[sizeof(media_device)-1] = '\0';
> +                     break;
> +             default:
> +                     printf("Usage: %s [-d </dev/mediaX>]\n", argv[0]);
> +                     exit(-1);
> +             }
> +     }
> +
> +     if (getuid() != 0) {
> +             printf("Please run the test as root - Exiting.\n");
> +             exit(-1);
> +     }
> +
> +     /* Open Media device and keep it open */
> +     fd = open(media_device, O_RDWR);
> +     if (fd == -1) {
> +             printf("Media Device open errno %s\n", strerror(errno));
> +             exit(-1);
> +     }
> +
> +     printf("\nNote:\n"
> +            "While test is running, remove the device and\n"
> +            "ensure there are no use after free errors and\n"
> +            "other Oops in the dmesg. Enable KaSan kernel\n"
> +            "config option for use-after-free error detection.\n\n");
> +
> +     while (count < 100) {
> +             ret = ioctl(fd, MEDIA_IOC_DEVICE_INFO, &mdi);
> +             if (ret < 0)
> +                     printf("Media Device Info errno %s\n", strerror(errno));
> +             printf("Media device model %s driver %s\n",
> +                     mdi.model, mdi.driver);
> +             sleep(10);
> +             count++;
> +     }
> +}


-- 
Thanks,
Mauro

Reply via email to