commit 765ddfd5046c9498b0e53d4d52e6887f65bb4340
author walter-bai <walter.bai@intel.com> Wed Feb 24 21:58:17 2021 +0800
committer intel-mediadev <mediadev@intel.com> Thu Feb 25 06:04:34 2021 +0800
tree 3c062e67ad3f52105f74f67d151334f52e4ef565
parent ee193b5e004696d5dcf34233d361eae456ca03da

[Encode] Fix MSDK random assert issue

* [Encode] Fix MSDK random assert issue

Fix an issue that for multi-packet case, status report counter is updated per packet level, which can cause issue in getting status report.

media_driver/media_driver_next/agnostic/common/shared/statusreport/media_status_report.cpp

diff --git a/media_driver/media_driver_next/agnostic/common/shared/statusreport/media_status_report.cpp b/media_driver/media_driver_next/agnostic/common/shared/statusreport/media_status_report.cpp
index fd1648b..f7b766a 100644
--- a/media_driver/media_driver_next/agnostic/common/shared/statusreport/media_status_report.cpp
+++ b/media_driver/media_driver_next/agnostic/common/shared/statusreport/media_status_report.cpp
@@ -54,7 +54,6 @@
 {
     MOS_STATUS eStatus = MOS_STATUS_SUCCESS;
 
-    uint32_t completedCount = *m_completedCount;
     uint32_t reportedCount = m_reportedCount;
     uint32_t reportedCountOrigin = m_reportedCount;
     uint32_t availableCount = m_submittedCount - reportedCount;
@@ -62,10 +61,10 @@
     uint32_t reportIndex = 0;
     bool reverseOrder = (requireNum > 1);
 
-    while (reportedCount != completedCount && generatedReportCount < requireNum){
+    while (reportedCount != *m_completedCount && generatedReportCount < requireNum){
 
         // Get reverse order index to temporally fix application get status report size bigger than 2 case.
-        reportIndex = reverseOrder ? CounterToIndex(completedCount + reportedCountOrigin - reportedCount -1) :
+        reportIndex = reverseOrder ? CounterToIndex(*m_completedCount + reportedCountOrigin - reportedCount - 1) :
                                      CounterToIndex(reportedCount);
         // m_reportedCount is used by component. Need to assign actual index before call ParseStatus
         m_reportedCount = reportIndex;