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

2014-10-23 Thread yan . wang
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@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/beignet


___
Beignet mailing list
Beignet@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet


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

2014-10-23 Thread Zhigang Gong
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.w...@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@lists.freedesktop.org
  http://lists.freedesktop.org/mailman/listinfo/beignet
 
 
 ___
 Beignet mailing list
 Beignet@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/beignet
___
Beignet mailing list
Beignet@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet


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

2014-10-23 Thread yan . wang
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.w...@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@lists.freedesktop.org
  http://lists.freedesktop.org/mailman/listinfo/beignet
 

 ___
 Beignet mailing list
 Beignet@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/beignet
 ___
 Beignet mailing list
 Beignet@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/beignet


___
Beignet mailing list
Beignet@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet


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

2014-10-23 Thread Zhigang Gong
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.w...@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.w...@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@lists.freedesktop.org
   http://lists.freedesktop.org/mailman/listinfo/beignet
  
 
  ___
  Beignet mailing list
  Beignet@lists.freedesktop.org
  http://lists.freedesktop.org/mailman/listinfo/beignet
  ___
  Beignet mailing list
  Beignet@lists.freedesktop.org
  http://lists.freedesktop.org/mailman/listinfo/beignet
 
 
 ___
 Beignet mailing list
 Beignet@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/beignet
___
Beignet mailing list
Beignet@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet


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

2014-10-23 Thread yan . wang
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.w...@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.w...@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@lists.freedesktop.org
   http://lists.freedesktop.org/mailman/listinfo/beignet
  
 
  ___
  Beignet mailing list
  Beignet@lists.freedesktop.org
  http://lists.freedesktop.org/mailman/listinfo/beignet
  ___
  Beignet mailing list
  Beignet@lists.freedesktop.org
  http://lists.freedesktop.org/mailman/listinfo/beignet
 

 ___
 Beignet mailing list
 Beignet@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/beignet
 ___
 Beignet mailing list
 Beignet@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/beignet


___
Beignet mailing list
Beignet@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet


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

2014-10-23 Thread yan . wang
Sure. I will try again and file it.

 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-boun...@lists.freedesktop.org] On Behalf
 Of
 yan.w...@linux.intel.com
 Sent: Thursday, October 23, 2014 5:05 PM
 To: Zhigang Gong
 Cc: yan.w...@linux.intel.com; beignet@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.w...@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.w...@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@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet
   
  
   ___
   Beignet mailing list
   Beignet@lists.freedesktop.org
   http://lists.freedesktop.org/mailman/listinfo/beignet
   ___
   Beignet mailing list
   Beignet@lists.freedesktop.org
   http://lists.freedesktop.org/mailman/listinfo/beignet
  
 
  ___
  Beignet mailing list
  Beignet@lists.freedesktop.org
  http://lists.freedesktop.org/mailman/listinfo/beignet
  ___
  Beignet mailing list
  Beignet@lists.freedesktop.org
  http://lists.freedesktop.org/mailman/listinfo/beignet
 

 ___
 Beignet mailing list
 Beignet@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/beignet

 ___
 Beignet mailing list
 Beignet@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/beignet


___
Beignet mailing list
Beignet@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet


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

2014-10-23 Thread yan . wang
Done.

 Sure. I will try again and file it.

 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-boun...@lists.freedesktop.org] On Behalf
 Of
 yan.w...@linux.intel.com
 Sent: Thursday, October 23, 2014 5:05 PM
 To: Zhigang Gong
 Cc: yan.w...@linux.intel.com; beignet@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.w...@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.w...@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@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet
   
  
   ___
   Beignet mailing list
   Beignet@lists.freedesktop.org
   http://lists.freedesktop.org/mailman/listinfo/beignet
   ___
   Beignet mailing list
   Beignet@lists.freedesktop.org
   http://lists.freedesktop.org/mailman/listinfo/beignet
  
 
  ___
  Beignet mailing list
  Beignet@lists.freedesktop.org
  http://lists.freedesktop.org/mailman/listinfo/beignet
  ___
  Beignet mailing list
  Beignet@lists.freedesktop.org
  http://lists.freedesktop.org/mailman/listinfo/beignet
 

 ___
 Beignet mailing list
 Beignet@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/beignet

 ___
 Beignet mailing list
 Beignet@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/beignet


 ___
 Beignet mailing list
 Beignet@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/beignet


___
Beignet mailing list
Beignet@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet


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

2014-10-23 Thread Zhigang Gong
Thanks.

On Thu, Oct 23, 2014 at 02:59:14AM -0700, yan.w...@linux.intel.com wrote:
 Done.
 
  Sure. I will try again and file it.
 
  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-boun...@lists.freedesktop.org] On Behalf
  Of
  yan.w...@linux.intel.com
  Sent: Thursday, October 23, 2014 5:05 PM
  To: Zhigang Gong
  Cc: yan.w...@linux.intel.com; beignet@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.w...@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.w...@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@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/beignet

   
___
Beignet mailing list
Beignet@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet
___
Beignet mailing list
Beignet@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet
   
  
   ___
   Beignet mailing list
   Beignet@lists.freedesktop.org
   http://lists.freedesktop.org/mailman/listinfo/beignet
   ___
   Beignet mailing list
   Beignet@lists.freedesktop.org
   http://lists.freedesktop.org/mailman/listinfo/beignet
  
 
  ___
  Beignet mailing list
  Beignet@lists.freedesktop.org
  http://lists.freedesktop.org/mailman/listinfo/beignet
 
  ___
  Beignet mailing list
  Beignet@lists.freedesktop.org
  http://lists.freedesktop.org/mailman/listinfo/beignet
 
 
  ___
  Beignet mailing list
  Beignet@lists.freedesktop.org
  http://lists.freedesktop.org/mailman/listinfo/beignet
 
 
 ___
 Beignet mailing list
 Beignet@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/beignet
___
Beignet mailing list
Beignet@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet