On Tue, Jan 05, 2016 at 05:17:07PM +0100, Lukasz Fiedorowicz wrote:
> +static void open_guc_status(void)
> +{
> +     guc_status_fd = igt_debugfs_open("i915_guc_load_status", O_RDONLY);
> +     igt_assert_f(guc_status_fd >= 0, "Can't open i915_guc_load_status\n");

igt_require(). So that the test doesn't explode if the kernel changes.

> +
> +static enum guc_status get_guc_status(void)
> +{
> +     char buf[LOAD_STATUS_BUF_SIZE];
> +
> +     FILE *fp = fdopen(guc_status_fd, "r");

If you just wanted the FILE*, use igt_debugfs_fopen.

> +     igt_assert_f(fp != NULL, "Can't open i915_guc_load_status file\n");
> +
> +     while (fgets(buf, LOAD_STATUS_BUF_SIZE, fp))
> +             if ((strstr(buf, "\tload: SUCCESS\n")))
> +                     return GUC_ENABLED;

Testing FILE* leak handling?
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to