Message ID | 1422971645-30621-5-git-send-email-tjakobi@math.uni-bielefeld.de (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Hi, On 02/03/2015 10:53 PM, Tobias Jakobi wrote: > 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. > > Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de> > --- > tests/exynos/exynos_fimg2d_test.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/tests/exynos/exynos_fimg2d_test.c b/tests/exynos/exynos_fimg2d_test.c > index aa140e5..48dfe97 100644 > --- a/tests/exynos/exynos_fimg2d_test.c > +++ b/tests/exynos/exynos_fimg2d_test.c > @@ -788,11 +788,13 @@ int main(int argc, char **argv) > > getchar(); > > +#if 0 How about add some comments about why need disable? Thanks. > 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) >
Hello Joonyoung! Joonyoung Shim wrote: > Hi, > > On 02/03/2015 10:53 PM, Tobias Jakobi wrote: >> 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. >> >> Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de> >> --- >> tests/exynos/exynos_fimg2d_test.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/tests/exynos/exynos_fimg2d_test.c b/tests/exynos/exynos_fimg2d_test.c >> index aa140e5..48dfe97 100644 >> --- a/tests/exynos/exynos_fimg2d_test.c >> +++ b/tests/exynos/exynos_fimg2d_test.c >> @@ -788,11 +788,13 @@ int main(int argc, char **argv) >> >> getchar(); >> >> +#if 0 > > How about add some comments about why need disable? You mean the current patch description is not enough, or that I should put that description also in the sourcecode? With best wishes, Tobias
Hi, On 02/11/2015 10:39 AM, Tobias Jakobi wrote: > Hello Joonyoung! > > Joonyoung Shim wrote: >> Hi, >> >> On 02/03/2015 10:53 PM, Tobias Jakobi wrote: >>> 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. >>> >>> Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de> >>> --- >>> tests/exynos/exynos_fimg2d_test.c | 2 ++ >>> 1 file changed, 2 insertions(+) >>> >>> diff --git a/tests/exynos/exynos_fimg2d_test.c b/tests/exynos/exynos_fimg2d_test.c >>> index aa140e5..48dfe97 100644 >>> --- a/tests/exynos/exynos_fimg2d_test.c >>> +++ b/tests/exynos/exynos_fimg2d_test.c >>> @@ -788,11 +788,13 @@ int main(int argc, char **argv) >>> >>> getchar(); >>> >>> +#if 0 >> >> How about add some comments about why need disable? > You mean the current patch description is not enough, or that I should > put that description also in the sourcecode? > I mean just to add on source code, if anyone meets #if 0 without any comments, he will have question. Thanks.
diff --git a/tests/exynos/exynos_fimg2d_test.c b/tests/exynos/exynos_fimg2d_test.c index aa140e5..48dfe97 100644 --- a/tests/exynos/exynos_fimg2d_test.c +++ b/tests/exynos/exynos_fimg2d_test.c @@ -788,11 +788,13 @@ int main(int argc, char **argv) getchar(); +#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)
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. Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de> --- tests/exynos/exynos_fimg2d_test.c | 2 ++ 1 file changed, 2 insertions(+)