Opened 13 years ago

Closed 13 years ago

#429 closed defect (fixed)

Encoder creates a different bitstream each time when using wavefront option with 2 substreams

Reported by: sandeepkanumuri Owned by:
Priority: minor Milestone: HM-6.1
Component: HM Version: HM-6.0
Keywords: Cc: fbossen, ksuehring, davidf, jct-vc@…

Description

In HM-6.0-dev branch, if we use wavefronts with 2 substreams, the encoder creates a slightly different bitstream each time it is run.

I run the encoder with the following options:
Release\TAppEncoder.exe -c encoder_lowdelay_main.cfg -c RaceHorses.cfg --FrameToBeEncoded=8 --WaveFrontSynchro=1 --WaveFrontFlush=1 --WaveFrontSubstreams=2

The config files used are attached (should be the same as those in software).

However, if I change WaveFrontSubstreams to 4, I don't see this issue (the bitstream generated does not change with each run).

As far as I can tell, this issue starts with revision 2120.

Attachments (4)

encoder_lowdelay_main.cfg​ (10.3 KB) - added by sandeepkanumuri 13 years ago.
RaceHorses.cfg​ (536 bytes) - added by sandeepkanumuri 13 years ago.
HM-6.0-dev-r2140-WPPEntryPoint.patch​ (1.0 KB) - added by kiranmisra 13 years ago.
HM-6.0-dev-r2140-WPPEntryPoint_r1.patch​ (1.9 KB) - added by hao 13 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 13 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

Changed 13 years ago by sandeepkanumuri

Changed 13 years ago by sandeepkanumuri

Changed 13 years ago by kiranmisra

comment:2 Changed 13 years ago by kiranmisra

Transmitting one more wavefront entry point information than required causes invalid memory access. The attached patch looks to fix the issue.

comment:3 Changed 13 years ago by hao

Decoder also has the similar memory access problem.
In addition, memory allocation is a bit redundant in the patched encoder.
The r1 patch to fix the issues is attached.

Changed 13 years ago by hao

comment:4 Changed 13 years ago by ksuehring

  • Milestone set to HM-6.1
  • Resolution set to fixed
  • Status changed from new to closed

The patch was applied in r2142

Note: See TracTickets for help on using tickets.

This list contains all users that will be notified about changes made to this ticket.

These roles will be notified: Reporter, Owner, Subscriber, Participant

  • David Flynn(Subscriber)
  • Frank Bossen(Subscriber)
  • Hirofumi Aoki(Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Participant, Always)
  • Kiran Misra(Participant)
  • Sandeep Kanumuri(Reporter)