Regression in OCL_ImgProc/CvtColor8u.BGR5652GRAY/0 (Bug #3754)


Added by Tom Stellard over 10 years ago. Updated over 10 years ago.


Status:Cancelled Start date:2014-06-16
Priority:Normal Due date:
Assignee:Ilya Lavrenov % Done:

0%

Category:ocl
Target version:-
Affected version:2.4.9 (latest release) Operating System:Linux
Difficulty: HW Platform:x64
Pull request:

Description

Commit 27c1bd27624f0a9c248cd05d6779cc6859d86892 in the 2.4 branch causes a regression in the OCL_ImgProc/CvtColor8u.BGR5652GRAY/0. The problem is that the value bidx is defined as -1, which leads to coeffs array in the RGB2YUV kernel being accessed by a negative array index in several places.


Associated revisions

Revision ac8045b9
Added by Vadim Pisarevsky about 10 years ago

Merge pull request #3754 from eliteraspberries:unsigned-mask

History

Updated by Ilya Lavrenov over 10 years ago

Hi Tom,
It's very strange how CvtColor8u.BGR5652GRAY can affect to RGB2YUV. BGR5652GRAY uses your own kernel and bidx is useless for it, so bidx is defined as -1.
And what do you mean under `regression`? Is it performance regression or something works wrong?

  • Assignee changed from Vadim Pisarevsky to Ilya Lavrenov

Updated by Tom Stellard over 10 years ago

It looks like this is a bug in our compiler, we crash on the negative indices, but this kernel is compiled but not used so it has no impact on the test as you point out.

  • Status changed from New to Cancelled

Also available in: Atom PDF