2
0
Gráfico de cometimentos

71 Cometimentos

Autor(a) SHA1 Mensagem Data
jeffqjiangNew 82d328b627 * rocDecode/HEVC: Fixed the decode failure of the conformance stream BUMPING_A_ericsson_1.bit. (#133)
- This stream tests output order conformance, in particular the bumping process. Four temporal layers are used and IRAP pictures with no_output_of_prior_pics_flag equal to 1 are present in the bitstream.

[ROCm/rocdecode commit: 42f109369e]
2023-12-13 09:48:15 -05:00
jeffqjiangNew 7cd5d6c0ee * rocDecode/HEVC: Added frame rate info report when it is available, from the parser. (#130)
[ROCm/rocdecode commit: 7240ee8973]
2023-12-12 13:47:19 -05:00
Aryan Salmanpour 5a9f519761 Don't call DestroyDataBuffers during the reconfiguration - this function will be called during SubmitDecode (#131)
[ROCm/rocdecode commit: b8a8b66b91]
2023-12-11 17:10:10 -05:00
Aryan Salmanpour b62bf4a6a1 Fix perf drop on some MI250 nodes - create VABuffers for every submission, and don't use vaMapBuffer/vaUnmapBuffer (#129)
* create VABuffers for every submission, and don't use vaMapBuffer/vaUnmapBuffer

* code clean up

* address review comments

[ROCm/rocdecode commit: b77d00e38e]
2023-12-11 15:40:50 -05:00
jeffqjiangNew a265b9a3da * rocDecode: Fix for the performance regression introduced by PR#114. (#128)
- Skip the additional buffer bumping specified in process C.5.2.2., to avoid synchronous job submission in some cases.
  - Increase DPB buffer size now by 2 to avoid overwriting pictures to be output.

[ROCm/rocdecode commit: a2dfa8bc26]
2023-12-08 17:53:01 -05:00
Lakshmi Kumar 6c3f0095da adding support for MI300A A1 - gfx942 (#124)
[ROCm/rocdecode commit: f221d78258]
2023-12-07 09:31:44 -05:00
jeffqjiangNew 64d7f1c37f * rocDecode/HEVC: In flush DPB function, added a check for any buffers that need for output before any actions. This prevents any potential issues. (#121)
[ROCm/rocdecode commit: fcd63b9358]
2023-12-06 09:39:49 -05:00
Aryan Salmanpour 08c99da4fa Add nullptr parameters check for all decoder/parser APIs and some code clean up (#120)
[ROCm/rocdecode commit: 9257cd6fcf]
2023-12-05 13:06:34 -05:00
Aryan Salmanpour 70a18314d5 Add support for rocDecReconfigureDecoder API (#116)
* Add support for rocDecReconfigureDecoder API

* adrress reviewer's comemnts

* Add doxygen comment for ReconfigureDecoder function

[ROCm/rocdecode commit: 875994bb4f]
2023-12-05 11:16:32 -05:00
jeffqjiangNew dd3b609a09 * rocDecode/HEVC: Fix for decode errors on a few conformance streams. (#114)
- Re-wrote short term RPS parsing function. The previous code generates incorrect results on some conformance streams.
  - No longer skip the additional buffer bumping specified in process C.5.2.2. This avoids the situation where the current frame overwrites the previously decoded buffer that is being output/displayed.

[ROCm/rocdecode commit: 5e65788bdd]
2023-12-04 11:53:26 -05:00
Aryan Salmanpour bf9ca09a9a Chamne the return type of FillSeqCallbackFn to int and check to see if it returns any error (#112)
[ROCm/rocdecode commit: ae63fbacab]
2023-12-01 14:50:33 -05:00
jeffqjiangNew 10545bcd94 * rocDecode/HEVC: When video size changes, set new SPS activation flag. This takes care of the case where a new SPS replaces the old SPS with the same id but with different dimensions. (#110)
[ROCm/rocdecode commit: a878a106fb]
2023-11-30 18:32:25 -05:00
Aryan Salmanpour 7358b1b595 use std::max to avoid compilation issue (#107)
[ROCm/rocdecode commit: c1a8ac48b4]
2023-11-30 11:29:39 -05:00
Aryan Salmanpour 1bbbe808f5 clean up - use google style guide for the RocDecoder class (#101)
* clean up - use google style guide for the RocDecoder class

* make other files/classes to follow the google style guides

[ROCm/rocdecode commit: 48474fdad9]
2023-11-29 13:50:38 -05:00
jeffqjiangNew f89b8e0d80 * rocDecode/HEVC: Fixed corruptions in RASL (Random Access Skipped Leading) pictures of an associated CRA picture. (#96)
- It appears that the root cause of the corruption is the loss of some reference info of the RASL pictures at VA-API driver level.
  - For reasons that are not documented in VA-API, or simply implementation limitations, the DPB buffer status when a CRA picture is decoded, needs to be sent to VA-API driver to the correct decoding of the associated RASL pictures. The info is stored in PocStFoll and PocLtFoll and is not needed for CRA picture decode, which is an intra picture. Without this info, the following RASL picture decode will run into problem even when its reference picture info is correctly specified.
  - Note this appears to be a VA-API specific issue because it did not occur on other platforms.

[ROCm/rocdecode commit: 2d14eb61b2]
2023-11-29 08:33:00 -05:00
Aryan Salmanpour ae3ec857fe Improve logging errors for VAAPI and HIP by including the API names and the status in the logs (#95)
* Improve logging errors for VAAPI and HIP by including the API names and the status in the logs

* clean up

[ROCm/rocdecode commit: 3625829546]
2023-11-28 16:52:27 -05:00
jeffqjiangNew b4216ccb63 * rocDecode/HEVC: Fixed a method name typo. (#92)
[ROCm/rocdecode commit: 27d28dd5f4]
2023-11-28 14:03:13 -05:00
jeffqjiangNew ab36eb705a * rocDecode/HEVC: Fixed the missing output picture issue on certain streams. (#89)
- When we hit IRAP (Intra Random Access Point) picture and need to bump all remaining decoded pictures from DPB, call display callback immediately, instead of delaying the callback to the IRAP decode process.

[ROCm/rocdecode commit: 09da68790c]
2023-11-27 19:54:49 -05:00
Aryan Salmanpour 729a6db0cb Return an error if the ExportSurface function fails. Add extra error logging in the destructor of the VaapiVideoDecoder class (#87)
[ROCm/rocdecode commit: e5d5bd4abb]
2023-11-27 13:08:34 -05:00
jeffqjiangNew 7035965722 End of stream flush (#86)
* * rocDecode/HEVC: Added picture output operation in DPB and display callback function.
  - To avoid serialization of decode submissions and/or display callback in certain cases, increased DPB buffer size by 1 and tweaked picture bumping procedure.

* * rocDecode/HEVC: Added flushing of the remaining pictures from DPB at the end of decode session.

* * rocDecode/HEVC: Changed output_pic_num to num_output_pics to void some confusion. Set top_field_first to 1.

* * rocDecode/HEVC: Added a few return error checks.

[ROCm/rocdecode commit: 624c3c0f09]
2023-11-27 13:03:26 -05:00
jeffqjiangNew 8b432bce65 Added picture output operation in DPB and display callback function. (#83)
* * rocDecode/HEVC: Added picture output operation in DPB and display callback function.
  - To avoid serialization of decode submissions and/or display callback in certain cases, increased DPB buffer size by 1 and tweaked picture bumping procedure.

* * rocDecode/HEVC: Added a few error checks and minor format changes.

[ROCm/rocdecode commit: 57c040eff8]
2023-11-27 11:04:01 -05:00
jeffqjiangNew 67c9ea1983 Re-enabled SEI message processing. (#79)
* * rocDecode/HEVC: Re-enabled SEI message processing.
  - Added "-sei" command option to extract SEI messages from stream.
  - Added support for SEI info parsing for multiple SEI NAL units with multiple payloads.
  - Fixed a SEI syntax parsing issue and several procssing issues.
  - Fixed a double free memory issue in SEI data buffer handling in HandlePictureDisplay().

* * rocDecode/HEVC: Coding format changes.

* * rocDecode/HEVC: Minor format changes.

[ROCm/rocdecode commit: ad186a90da]
2023-11-21 14:33:15 -05:00
Rajy Rawther 6918fb07c4 Rr/check sei function pointer (#78)
* commending out ParseSeiMessage because it is causing segfault

* CMake changes for valgrind

* added function pointer validity for sei callback

* fix review comments

[ROCm/rocdecode commit: 012f96c8c5]
2023-11-21 11:23:47 -05:00
Rajy Rawther e925faf9fe commending out ParseSeiMessage because it is causing segfault (#76)
[ROCm/rocdecode commit: 210976616f]
2023-11-17 20:56:05 -05:00
Rajy Rawther 13654ccef6 Rr/parser clean up (#75)
* minor clean_up on parser struct allocation

* minor clean_up

* address review comments

[ROCm/rocdecode commit: cbf6857f74]
2023-11-17 20:55:53 -05:00
Rajy Rawther d77c19c254 Rr/vaapi decoder mem check (#74)
* add some checks for buffer requirements

* address review comments

[ROCm/rocdecode commit: 81abe58fb4]
2023-11-17 17:33:12 -05:00
jeffqjiangNew 43a70df640 * rocDecode/HEVC: Removed parsing of entry_point_offset_minus1[] and beyond in slice header since these syntax elements are not used by HW decode. (#73)
[ROCm/rocdecode commit: c528f46843]
2023-11-17 15:59:29 -05:00
jeffqjiangNew 6ec565f17b * rocDecode/HEVC: Fixed the segmentation fault at the end of decode session on stream chimei_demo_1080p_h265_60fps.mp4 or similar streams. (#72)
Root cause: num_entry_point_offsets in the slice header of frame #2164 is 674, exceeding the entry_point_offset_minus1[440] array size. Parser writes beyond the array boundary, corrupting memory. Frame #2164 is not in conformance of the HEVC spec.

We need to put the constraint from the spec on the num_entry_point_offsets parsed from the stream. Also need to change the array size to the max possible for 8K.

[ROCm/rocdecode commit: b6d0c41a59]
2023-11-17 12:57:19 -05:00
Aryan Salmanpour f62b0fa617 Add additional debug info if we couldn't find the matching pic in reference list and return error (#69)
* Add additional debug info if we couldn't find the matching pic in refrence list and return error

* Correct the debug info message

[ROCm/rocdecode commit: 5d5db9a017]
2023-11-16 12:39:23 -05:00
jeffqjiangNew b961ebfe19 IRAP POC calculation fix. (#66)
* * rocDecode/HEVC: Fixed a bug in IRAP picture order count calculation. Added a few picture type check utilities.

* * rocDecode/HEVC: Minor coding style changes. No functional changes.

[ROCm/rocdecode commit: 69c2243c9c]
2023-11-15 16:07:18 -05:00
jeffqjiangNew 463ed05a63 Do not set up slice ref list for intra frames for VAAPI calls. (#62)
* * rocDecode/HEVC: Do not set up slice ref list for intra frames.

* *rocDecode/HEVC: Condensed slice type check code. No functionaly changes.

[ROCm/rocdecode commit: 1d4f038695]
2023-11-14 15:44:35 -05:00
jeffqjiangNew 9d9ac34a91 * rocDecode/HEVC: Fixed a few issues in reference list setup. Also looks like we need to use VA surface IDs on all buffers including the reference (need to confirm with VA driver team). (#61)
[ROCm/rocdecode commit: 3357aba104]
2023-11-14 11:42:13 -05:00
Aryan Salmanpour e86c9487b5 Add support for the rocDecUnMapVideoFrame API (#58)
[ROCm/rocdecode commit: eb5614d250]
2023-11-13 12:54:32 -05:00
Aryan Salmanpour 77481cf7e0 Close fds after importing the surVAAPI face to HIP memory (#55)
[ROCm/rocdecode commit: 730b80b600]
2023-11-10 18:24:21 -05:00
jeffqjiangNew 30c4df3af1 Added picture decode submission to VA-API driver. (#54)
* * rocDeocde/HEVC: Added several "reserved" fields to structs to match the corresponding VA-API structs.

* * rocDecode/HEVC: Added decode submission to VA-API driver.

* * rocDecode/HEVC: Corrected data buffer id type.

[ROCm/rocdecode commit: 3c7f29dab4]
2023-11-10 17:15:47 -05:00
Aryan Salmanpour 9ac1a081de Add support for rocDecMapVideoFrame API (#53)
[ROCm/rocdecode commit: 7a90c0c054]
2023-11-10 17:02:57 -05:00
jeffqjiangNew 0b39cc1c2c * rocDecode/HEVC: Fixed a regression introduced by PR#30. We should not abort parsing when we see NAL unit types that we are not interested in. Instead, we should just ignore and continue. (#48)
[ROCm/rocdecode commit: 80845837af]
2023-11-09 14:57:22 -05:00
jeffqjiangNew 7db33e7837 Decode callback implementation step 3: completion. (#47)
* * rocDecode/HEVC: Filled HEVC picture parameter structures for decode callback. Added some missing inferences in tile size and weighted prediction table parsing.

* * rocDecode/HEVC: Changed CurrPicIdx assignment from POC to pic_idx.

* * rocDecode/HEVC: Rmoved one TODO comment, which has been done.

* * rocDecode/HEVC: Corrected two typos in comment.

* * rocDecode/HEVC: No need to set up scaling lists in decode callback when scaling list is not enabled.

[ROCm/rocdecode commit: 8ed03b5ad0]
2023-11-09 13:52:02 -05:00
Aryan Salmanpour d44d320717 Add support for rocDecGetDecodeStatus API (#46)
[ROCm/rocdecode commit: 6ab31ce40e]
2023-11-09 08:47:58 -05:00
Aryan Salmanpour b27fe23cd1 Return an error if the video parser creation fails (#45)
[ROCm/rocdecode commit: 9afec26af0]
2023-11-08 11:54:35 -05:00
Aryan Salmanpour d72a1ca3c5 Add support to GetDecoderCaps to correctly return decoder caps for gfx arch names with a suffix (#43)
[ROCm/rocdecode commit: 3a5fc3608b]
2023-11-08 10:25:45 -05:00
Aryan Salmanpour d9c139d2ab Add nullptr check for parser APIs before dereferencing the parser's handle (#44)
[ROCm/rocdecode commit: ecbd787e4a]
2023-11-08 10:18:17 -05:00
Aryan Salmanpour b459dc6f33 Add support for creating a vaapi context and surfaces for decoding (#41)
[ROCm/rocdecode commit: 2c940dd9d0]
2023-11-07 13:47:53 -05:00
Aryan Salmanpour e25ad867c0 Create the correct render node names on MI300A/MI300X (#38)
[ROCm/rocdecode commit: ba484a91a9]
2023-11-07 11:08:20 -05:00
jeffqjiangNew 6a37f8bbb2 Decode callback step 1: Added skeleton decode callback to get the flow going. (#39)
* * rocDecode/HEVC: Added skeleton decode callback to get the flow going.

* *rocDecode/HEVC: Added error handling for decode call.

[ROCm/rocdecode commit: f56a64d832]
2023-11-06 22:04:25 -05:00
Lakshmi Kumar 295091beb8 Parser - sei message (#30)
* sei mesaage aprser

* sei callback fn

* copy payload and add condition for callback

* compile error fix

* review comments

* review  comment

[ROCm/rocdecode commit: 213ffc517e]
2023-11-06 09:25:09 -05:00
jeffqjiangNew 97f1b39d98 * rocDecode/HEVC: Fixed a bug in slice header parsing where short term RPS is not properly assigned when SPS short term RPS is referred and the number of these RPSs are 1. Also added error handling in ref list construction to avoid infinite looping. (#36)
[ROCm/rocdecode commit: f5e5d981e1]
2023-11-06 08:22:31 -05:00
Aryan Salmanpour 0c7e31bbfa Add support for gfx1100 - update the vcn lookup table (#34)
[ROCm/rocdecode commit: f47fd89f62]
2023-11-02 22:05:17 -04:00
jeffqjiangNew 170cc43cfb Added all preparing steps in decode process before calling VA-API. (#32)
* rocDecode/HEVC parsing: Implemented correct logic to set active VPS, SPS and PPS: start from slice header and up. Added video image size change detection. Fixed slice_segment_address bits calculation in slice segment header parsing.

* rocDecode/HEVC: Changed variable naming to Google style.

* rocDecode/HEVC: Added VPS, SPS, PPS, and slice segment header content logging functions for debug purposes.

* rocDecode/HEVC: Compacted opening bracket lines to keep coding style consistency.

* * rocDecode/HEVC: Fixed an issue in profile-tier-level parsing where 44 bits are skipped in an incorrect way using ReadBits() which can only consume 32 bits max. Also formated SPS parsing code by using pointer instead of array element.

* * rocDecode/HEVC: Fixed the incorrect array size for sps_max_dec_pic_buffering_minus1, sps_max_num_reorder_pics and sps_max_num_reorder_pics in SPS structure define.

* * rocDecode/HEVC: Modified scaling list parsing to make sure we explicitly follow the spec.

* * rocDecode/HEVC: Added a few syntax elements to short team RPS structure, updated RPS parsing function and logging function.

* * rocDecode/HEVC: In PPS parsing function, use structure pointer instead of structure array element to improve coding clarity.

* * rocDecode/HEVC: Compacted the opening bracket lines. No functional changes.

* * rocDecode/HEVC: Added full parsing of slice segment header. Added PPS range extension parsing.

* * rocDecode/HEVC: Fixed a build error in debug mode due to merging from master branch: removing a redundant debug function.

* * rocDecode/HEVC: Changed the condition for sequence call back from picture size change to more general case where a new SPS is activated. Also moved the call from slice header parsing to upper level where frame data is processed.

* * rocDecode: Compacted opening bracket line. No functional changes.

* * rocDecode/HEVC: Moved POC calculation from slice header parsing to picture level, since we only need to do the calculation once per picture.

* * rocDecode/HEVC: Added RPS decode function.

* *rocDecode/HEVC: Added reference picture list construction.

* * rocDecode/HEVC: Added current picture marking in DPB.

* * rocDecode/HEVC: Removed VA surface id from DPB buffer info structure.

* * rocDecode/HEVC: Do not construct ref list for I pictures.

[ROCm/rocdecode commit: 7a537bbb88]
2023-11-02 18:08:24 -04:00
Aryan Salmanpour a227d5fbcf Fix a bug for the video sequence callback function (#31)
[ROCm/rocdecode commit: 874bd59d89]
2023-11-02 14:34:52 -04:00