| Message ID | 1424094405-6314-5-git-send-email-tjakobi@math.uni-bielefeld.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On 16/02/15 13:46, Tobias Jakobi wrote: > v2: Move the commit description into the patch itself. > Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de> > --- > tests/exynos/exynos_fimg2d_test.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/tests/exynos/exynos_fimg2d_test.c b/tests/exynos/exynos_fimg2d_test.c > index aa140e5..55d2970 100644 > --- a/tests/exynos/exynos_fimg2d_test.c > +++ b/tests/exynos/exynos_fimg2d_test.c > @@ -788,11 +788,19 @@ int main(int argc, char **argv) > > getchar(); > > + /* The blend test uses the userptr functionality of exynos-drm, which * > + * is currently not safe to use. If the kernel hasn't been build with * > + * exynos-iommu support, then the blend test is going to produce (kernel) * > + * memory corruption, eventually leading to a system crash. * > + * * > + * Disable the test for now, until the kernel code has been sanitized. */ > +#if 0 I cannot see a part of libdrm that uses this commenting format. Perhaps use the more common: /* * Some comment */ Cheers, Emil -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/tests/exynos/exynos_fimg2d_test.c b/tests/exynos/exynos_fimg2d_test.c index aa140e5..55d2970 100644 --- a/tests/exynos/exynos_fimg2d_test.c +++ b/tests/exynos/exynos_fimg2d_test.c @@ -788,11 +788,19 @@ int main(int argc, char **argv) getchar(); + /* The blend test uses the userptr functionality of exynos-drm, which * + * is currently not safe to use. If the kernel hasn't been build with * + * exynos-iommu support, then the blend test is going to produce (kernel) * + * memory corruption, eventually leading to a system crash. * + * * + * Disable the test for now, until the kernel code has been sanitized. */ +#if 0 ret = test_case.blend(dev, src, bo, G2D_IMGBUF_USERPTR); if (ret < 0) fprintf(stderr, "failed to test blend operation.\n"); getchar(); +#endif err_free_src: if (src)
v2: Move the commit description into the patch itself. Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de> --- tests/exynos/exynos_fimg2d_test.c | 8 ++++++++ 1 file changed, 8 insertions(+)