[Beignet] opencv_test_imgproc crashs when test OCL_Imgproc/HoughLines.RealImage/0

[Zhigang Gong]

I just tried on my IVB machine, and it could be reproduced easily, just need
to unset the environment variable.
And this is a platform independent issue. The backend is just a compiler.
The compiler could complain warnings or
errors but should never crash. If you met a crash in gcc or clang internal,
no matter which source code it was compiling,
it's definitely a compiler bug.

If possible, please submit this bug to fd.o. Thanks for reporting this.

> -----Original Message-----
> From: Beignet [mailto:beignet-bounces at lists.freedesktop.org] On Behalf Of
> yan.wang at linux.intel.com
> Sent: Thursday, October 23, 2014 5:05 PM
> To: Zhigang Gong
> Cc: yan.wang at linux.intel.com; beignet at lists.freedesktop.org
> Subject: Re: [Beignet] opencv_test_imgproc crashs when test
> OCL_Imgproc/HoughLines.RealImage/0
> 
> I am not sure whether it is platform issue. So far I have only Baytail T
platform
> no others. The assertion will not be hit now. Still need me submit bug? I
could
> do it.
> 
> It may be the bug of OpenCL implementation of opecv hough lines for fail
case.
> Need more investigation.
> 
> > Yan,
> >
> > No matter you set or not set the test data. It should never crash
> > beignet in such an assertion. That's the point. Anyway, we will look
into it.
> >
> > On Thu, Oct 23, 2014 at 01:40:28AM -0700, yan.wang at linux.intel.com
> wrote:
> >> My previous mail may misunderstand you. Sorry.
> >> After I cloned opencv_extra git and set path rightly, this assertion
> >> will not be hit But 1 case of 8 will failed because dst size should
> >> be 1x60 and is 1x61 in fact. I am trying to debug it. The other 7
> >> cases passed.
> >>
> >>
> >> > Yan,
> >> >
> >> > It's still an valid bug. Beignet should never hit such an assertion
> >> > if everything works well. Could you open a bug on fd.o for beignet
> >> module?
> >> > You may need to specify the git-hash of the beignet and the opencv
> >> > you
> >> are
> >> > using. And you need to specify that don't set the opencv test data
> >> > environment variable to reproduce it.
> >> >
> >> > Thanks,
> >> > Zhigang Gong.
> >> >
> >> > On Thu, Oct 23, 2014 at 01:16:13AM -0700, yan.wang at linux.intel.com
> >> wrote:
> >> >> After cloned related test data, it will not crashed.
> >> >> Sorry.
> >> >>
> >> >> > Hi, All,
> >> >> >   This case will crash as the following:
> >> >> >   [ RUN      ] OCL_Imgproc/HoughLines.RealImage/0
> >> >> > ASSERTION FAILED: defIt != bbIt->second->end() && defIt->second
> >> >> > !=
> >> >> NULL
> >> >> >   at file /home/yanwang/beignet/backend/src/ir/value.cpp,
> >> >> > function gbe::ir::LiveOutSet::RegDefSet&
> >> gbe::ir::LiveOutSet::getDefSet(const
> >> >> > gbe::ir::BasicBlock*, gbe::ir::Register), line 91
> >> >> >   Thanks.
> >> >> >
> >> >> > Yan Wang
> >> >> > _______________________________________________
> >> >> > Beignet mailing list
> >> >> > Beignet at lists.freedesktop.org
> >> >> > http://lists.freedesktop.org/mailman/listinfo/beignet
> >> >> >
> >> >>
> >> >> _______________________________________________
> >> >> Beignet mailing list
> >> >> Beignet at lists.freedesktop.org
> >> >> http://lists.freedesktop.org/mailman/listinfo/beignet
> >> > _______________________________________________
> >> > Beignet mailing list
> >> > Beignet at lists.freedesktop.org
> >> > http://lists.freedesktop.org/mailman/listinfo/beignet
> >> >
> >>
> >> _______________________________________________
> >> Beignet mailing list
> >> Beignet at lists.freedesktop.org
> >> http://lists.freedesktop.org/mailman/listinfo/beignet
> > _______________________________________________
> > Beignet mailing list
> > Beignet at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/beignet
> >
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet