topic UMC: MPEG2 Decoder Crash in Intel® Integrated Performance Primitives

UMC: MPEG2 Decoder Crash

Scott_Danahy — Wed, 27 Oct 2010 17:55:28 GMT

Hi,

I've encountered a case where the MPEG2 decoder crashes. I havenumThreads set to 8 and there appears to be a problem correctly setting up the internal worker threads. One of theMPEG2VideoDecoderBase::ThreadWorkingRoutine instances will callDecodeSlices with null buffer pointers (IppVideoContext::bs_* values are all NULL or zero).

This seems to happen whenm_nNumberOfThreads andcurr_thread do not match after the thread initialization loop inMPEG2VideoDecoderBase::DecodePicture.

I can reproduce this with simple_player like this:

simple_player.exeD:\\crash.m2v -vnul -t 8

This only happens for certain MPEG2 bistreams. Has anyone else seen this issue? Any suggestions on how to work around it?

Scott

UMC: MPEG2 Decoder Crash

Vladimir_Dudnik — Wed, 27 Oct 2010 18:11:07 GMT

Hi Scott,

what version of IPP and sample do you use, what is your target cpu, OS? Is it possible to attach problem stream here to help us reproduce and investigate the problem?

Regards,
Vladimir

UMC: MPEG2 Decoder Crash

Scott_Danahy — Wed, 27 Oct 2010 18:16:56 GMT

Hi,

I'm using IPP 6.1 and samples 6.1.6.065. I reproduced this with Windows 7 running on two Xeon X5550processors. I'll have to check and see if I can share the stream in question.

Thank you,

Scott

UMC: MPEG2 Decoder Crash

Scott_Danahy — Thu, 28 Oct 2010 15:31:09 GMT

Hi Vladimir,

I've also found a couple of files that show bad visual errors in the decoded frames. Two other MPEG2 decoders do not show these errors and I am fairly confident that the file itself is good. Can I share this clip with you as well?

Scott

UMC: MPEG2 Decoder Crash

Vladimir_Dudnik — Thu, 28 Oct 2010 17:17:20 GMT

yes, of course that will be useful

Thanks,
Vladimir

UMC: MPEG2 Decoder Crash

Scott_Danahy — Tue, 02 Nov 2010 13:49:41 GMT

Hi Vladimir,

Do you know when I might get some more info on the files I provided?

Thank you,

Scott

UMC: MPEG2 Decoder Crash

Vladimir_Dudnik — Wed, 03 Nov 2010 10:42:07 GMT

Hi Scott,

we are still looking at your issue.

Regards,
Vladimir

UMC: MPEG2 Decoder Crash

Alexander_Modenov__I — Tue, 09 Nov 2010 11:12:59 GMT

Hi Scott,

Looks like we've the solution.

Please make the following changes in the codec/mpeg2_dec/src/umc_mpeg2_dec_pic.cpp file :

Ipp32s curr_thread;-----------> should be moved from line 925 to line 918

Lines 981,982 should be written as :

if (curr_thread < m_nNumberOfThreads)

m_nNumberOfThreads = curr_thread;

Regards,

Alexander

UMC: MPEG2 Decoder Crash

Scott_Danahy — Tue, 09 Nov 2010 14:56:04 GMT

Thank you very much, I'll apply thisimmediately.

Do you have any more information on the other 'bad frame' issue I reported? I attached the 'bad-frames-intel.m2v' sample file in an earlier private response toVladimir.

UMC: MPEG2 Decoder Crash

Alexander_Modenov__I — Wed, 10 Nov 2010 12:37:52 GMT

What do you mean under "bad frames" - horizontal color bands on two or more frames ?

Regards,

Alexander

UMC: MPEG2 Decoder Crash

Scott_Danahy — Wed, 10 Nov 2010 14:51:59 GMT

Hi Alexander,

Yes, I'm talking about the horizontal color bands. Those are not present with other decoders.

Thanks for looking into this.

Scott

UMC: MPEG2 Decoder Crash

Scott_Danahy — Mon, 15 Nov 2010 19:31:32 GMT

Hi Vladimir andAlexander,

Is there any update on the horizontal color band issue?

Scott

UMC: MPEG2 Decoder Crash

Scott_Danahy — Mon, 22 Nov 2010 15:09:17 GMT

Hi Guys,

Any update on this issue?

Scott

UMC: MPEG2 Decoder Crash

Alexander_Modenov__I — Tue, 23 Nov 2010 06:50:33 GMT

Hi Scott,

Thanks for the stream. We've reproduced the problem on IPP 7.0 too. The problem is under investigation now.

We'll inform you about our progress.

Regards,

Alexander

UMC: MPEG2 Decoder Crash

Scott_Danahy — Tue, 23 Nov 2010 15:20:18 GMT

Hi Alexander,

Thanks for the update, please let me know if I can provide any more information.

Scott

UMC: MPEG2 Decoder Crash

Alexander_Modenov__I — Fri, 17 Dec 2010 14:46:27 GMT

Hi Scott,

I've just found the problem case in MPEG2 decoder.

The solution is the following :

file umc_mpeg2_dec_blk.cpp line 212, column 45 0xab shall be written instead of 0xbb

After this change everything should be OK.

Regards,
Alexander

UMC: MPEG2 Decoder Crash

Scott_Danahy — Sat, 18 Dec 2010 02:16:02 GMT

Thank you! I'll test it and get back to you.

UMC: MPEG2 Decoder Crash

Scott_Danahy — Mon, 20 Dec 2010 16:09:50 GMT

Hi Alexander,

I applied the fix, and it resolves the issue with the specific segment I sent you, however I see many more similar errors in the complete file. I will send another sample.

Thank you,

Scott

UMC: MPEG2 Decoder Crash

Scott_Danahy — Thu, 13 Jan 2011 16:35:09 GMT

Hi Alexander,

Happy new year. Any update on the latest visual errors?

Thanks,

Scott

UMC: MPEG2 Decoder Crash

Alexander_Modenov__I — Tue, 18 Jan 2011 08:13:17 GMT

Hi Scott,

Happy new Year.

I've just found the problem root case.

Please add the following changes:

File umc_mpeg2_dec_bstream.h

write

#define SHOW_HI9BITS(video, CODE) \

CODE = (((video##_curr_ptr[0] << 24) | (video##_curr_ptr[1] << 16) | (video##_curr_ptr[2] << 8)) << video##_bit_offset);

#define SHOW_HI9BITS(video, CODE) \CODE = (((video##_curr_ptr[0] << 24) | (video##_curr_ptr[1] << 16) | (video##_curr_ptr[2] << 8)) << video##_bit_offset);

instead of

#define SHOW_HI9BITS(video, CODE) \

CODE = (((video##_curr_ptr[0] << 24) | (video##_curr_ptr[1] << 16)) << video##_bit_offset);

Thanks for testing.

Regards,

Alexander