Revert "[Decode] [PDVT-SH] VP9 CRC Output Enable"

This reverts commit intel-innersource/drivers.gpu.unified#f220b5e59

Author: luleilei1

media_softlet/agnostic/common/codec/hal/dec/vp9/packet/decode_vp9_debug_packet.cpp

@@ -64,17 +64,6 @@ MOS_STATUS Vp9DecodePkt::Init()
     m_allocator = m_vp9Pipeline->GetDecodeAllocator();
     DECODE_CHK_NULL(m_allocator);
 
-#if (_DEBUG || _RELEASE_INTERNAL)
-    // Initialize debug packet
-    DecodeSubPacket* subPacket = m_vp9Pipeline->GetSubPacket(
-        DecodePacketId(m_vp9Pipeline, vp9DebugSubPacketId));
-    m_debugPkt = dynamic_cast<Vp9DecodeDebugPkt*>(subPacket);
-    if (m_debugPkt != nullptr)
-    {
-        DECODE_CHK_STATUS(m_debugPkt->Init());
-    }
-#endif
-
     return MOS_STATUS_SUCCESS;
 }
 
@@ -94,14 +83,6 @@ MOS_STATUS Vp9DecodePkt::Prepare()
 
 MOS_STATUS Vp9DecodePkt::Destroy()
 {
-#if (_DEBUG || _RELEASE_INTERNAL)
-    // Destroy debug packet
-    if (m_debugPkt != nullptr)
-    {
-        DECODE_CHK_STATUS(m_debugPkt->Destroy());
-    }
-#endif
-
     m_statusReport->UnregistObserver(this);
     return MOS_STATUS_SUCCESS;
 }
@@ -199,14 +180,6 @@ MOS_STATUS Vp9DecodePkt::Completed(void *mfxStatus, void *rcsStatus, void *statu
     DECODE_VERBOSEMESSAGE("Index = %d", statusReportData->currDecodedPic.FrameIdx);
     DECODE_VERBOSEMESSAGE("FrameCrc = 0x%x", statusReportData->frameCrc);
 
-#if (_DEBUG || _RELEASE_INTERNAL)
-    // Call debug packet completion handling for HCP debug functionality
-    if (m_debugPkt != nullptr)
-    {
-        DECODE_CHK_STATUS(m_debugPkt->Completed());
-    }
-#endif
-
     return MOS_STATUS_SUCCESS;
 }
 
@@ -304,13 +277,6 @@ MOS_STATUS Vp9DecodePkt::EndStatusReport(uint32_t srType, MOS_COMMAND_BUFFER* cm
     DECODE_FUNC_CALL();
     DECODE_CHK_NULL(cmdBuffer);
     DECODE_CHK_STATUS(ReadHcpStatus(m_statusReport, *cmdBuffer));
-#if (_DEBUG || _RELEASE_INTERNAL)
-    // Execute debug packet for HCP debug functionality
-    if (m_debugPkt != nullptr)
-    {
-        DECODE_CHK_STATUS(m_debugPkt->Execute(*cmdBuffer, m_statusReport));
-    }
-#endif
     DECODE_CHK_STATUS(MediaPacket::EndStatusReportNext(srType, cmdBuffer));
 
     MediaPerfProfiler *perfProfiler = MediaPerfProfiler::Instance();

media_softlet/agnostic/common/codec/hal/dec/vp9/packet/decode_vp9_debug_packet.h

@@ -35,15 +35,8 @@
 #include "decode_vp9_picture_packet.h"
 #include "decode_vp9_slice_packet.h"
 
-#if (_DEBUG || _RELEASE_INTERNAL)
-#include "decode_vp9_debug_packet.h"
-#endif
-
 namespace decode
 {
-
-class Vp9DecodeDebugPkt;
-
 class Vp9DecodePkt : public CmdPacket, public MediaStatusReportObserver
 {
 public:
@@ -122,10 +115,6 @@ class Vp9DecodePkt : public CmdPacket, public MediaStatusReportObserver
     uint32_t m_sliceStatesSize      = 0;
     uint32_t m_slicePatchListSize   = 0;
 
-#if (_DEBUG || _RELEASE_INTERNAL)
-    Vp9DecodeDebugPkt*              m_debugPkt = nullptr;
-#endif
-
 MEDIA_CLASS_DEFINE_END(decode__Vp9DecodePkt)
 };
 

media_softlet/agnostic/common/codec/hal/dec/vp9/packet/decode_vp9_packet.cpp

@@ -29,7 +29,6 @@ set(SOFTLET_DECODE_VP9_SOURCES_
     ${CMAKE_CURRENT_LIST_DIR}/decode_vp9_picture_packet.cpp
     ${CMAKE_CURRENT_LIST_DIR}/decode_vp9_slice_packet.cpp
     ${CMAKE_CURRENT_LIST_DIR}/decode_vp9_downsampling_packet.cpp
-    ${CMAKE_CURRENT_LIST_DIR}/decode_vp9_debug_packet.cpp
 )
 
 set(SOFTLET_DECODE_VP9_HEADERS_
@@ -42,7 +41,6 @@ set(SOFTLET_DECODE_VP9_HEADERS_
     ${CMAKE_CURRENT_LIST_DIR}/decode_vp9_picture_packet.h
     ${CMAKE_CURRENT_LIST_DIR}/decode_vp9_slice_packet.h
     ${CMAKE_CURRENT_LIST_DIR}/decode_vp9_downsampling_packet.h
-    ${CMAKE_CURRENT_LIST_DIR}/decode_vp9_debug_packet.h
 )
 source_group( CodecHalNext\\Shared\\Decode FILES ${SOFTLET_DECODE_VP9_SOURCES_} ${SOFTLET_DECODE_VP9_HEADERS_} )
 

media_softlet/agnostic/common/codec/hal/dec/vp9/packet/decode_vp9_packet.h

@@ -32,9 +32,6 @@
 #include "decode_vp9_feature_manager.h"
 #include "decode_vp9_buffer_update.h"
 #include "media_debug_fast_dump.h"
-#if (_DEBUG || _RELEASE_INTERNAL)
-#include "decode_vp9_debug_packet.h"
-#endif
 
 namespace decode
 {
@@ -175,19 +172,6 @@ MOS_STATUS Vp9Pipeline::CreateSubPackets(DecodeSubPacketManager &subPacketManage
 
     DECODE_CHK_STATUS(DecodePipeline::CreateSubPackets(subPacketManager, codecSettings));
 
-#if (_DEBUG || _RELEASE_INTERNAL)
-    // Create VP9 debug packet
-    Vp9DecodeDebugPkt *debugPkt = MOS_New(Vp9DecodeDebugPkt, this, m_hwInterface);
-    DECODE_CHK_NULL(debugPkt);
-    MOS_STATUS status = subPacketManager.Register(
-                        DecodePacketId(this, vp9DebugSubPacketId), *debugPkt);
-    if (status != MOS_STATUS_SUCCESS)
-    {
-        MOS_Delete(debugPkt);
-        return status;
-    }
-#endif
-
     return MOS_STATUS_SUCCESS;
 }
 

media_softlet/agnostic/common/codec/hal/dec/vp9/packet/media_srcs.cmake

@@ -68,7 +68,6 @@ class Vp9Pipeline : public DecodePipeline
     DeclareDecodePacketId(vp9PictureSubPacketId);
     DeclareDecodePacketId(vp9SliceSubPacketId);
     DeclareDecodePacketId(vp9TileSubPacketId);
-    DeclareDecodePacketId(vp9DebugSubPacketId);
 
 protected:
     //!