[Decode] AVC Indirect BSD Data Length setting fix

RoyZhuIntel · intel-mediadev · commit 3673d208dbf9 · 2025-11-03T11:50:27.000+08:00
AVC Indirect BSD Data Length setting incorrect in MFD_AVC_SLICEADDR command impact high bitrate HW timing
diff --git a/media_common/agnostic/common/codec/shared/codec_def_common_avc.h b/media_common/agnostic/common/codec/shared/codec_def_common_avc.h
@@ -43,6 +43,8 @@
 
 #define CODECHAL_ENCODE_VDENC_BRC_CONST_BUFFER_NUM (NUM_PIC_TYPES + 1)  //!< For each frame type + 1 for ref B
 
+#define CODEC_AVC_SLICE_MIN_NUM_BYTES_LOAD   10  // 10 bytes
+
 //!
 //! \enum     CODEC_AVC_WEIGHT_SCALE_SIZE
 //! \brief    Codec AVC weight scale size
diff --git a/media_softlet/agnostic/common/codec/hal/dec/avc/packet/decode_avc_slice_packet.cpp b/media_softlet/agnostic/common/codec/hal/dec/avc/packet/decode_avc_slice_packet.cpp
@@ -426,19 +426,19 @@ MOS_STATUS AvcDecodeSlcPkt::AddCmd_AVC_SLICE_Addr(MOS_COMMAND_BUFFER &cmdBuffer,
     parSliceAddr.decodeInUse = true;
     if (parSlice.fullFrameData)
     {
-        parSliceAddr.IndirectBsdDataLength       = parSlice.nextLength;
+        parSliceAddr.IndirectBsdDataLength       = MOS_MIN(parSlice.nextLength, CODEC_AVC_SLICE_MIN_NUM_BYTES_LOAD);
         parSliceAddr.IndirectBsdDataStartAddress = parSlice.nextOffset;
     }
     else
     {
         if (m_avcBasicFeature && m_avcBasicFeature->m_hwStartCodeSupportEnabled)
         {
-            parSliceAddr.IndirectBsdDataLength       = parSlice.nextLength;
+            parSliceAddr.IndirectBsdDataLength       = MOS_MIN(parSlice.nextLength, CODEC_AVC_SLICE_MIN_NUM_BYTES_LOAD);
             parSliceAddr.IndirectBsdDataStartAddress = parSlice.nextOffset;
         }
         else
         {
-            parSliceAddr.IndirectBsdDataLength       = parSlice.nextLength + 1 - m_osInterface->dwNumNalUnitBytesIncluded;
+            parSliceAddr.IndirectBsdDataLength       = MOS_MIN(parSlice.nextLength + 1 - m_osInterface->dwNumNalUnitBytesIncluded, CODEC_AVC_SLICE_MIN_NUM_BYTES_LOAD);
             parSliceAddr.IndirectBsdDataStartAddress = parSlice.nextOffset - 1 + m_osInterface->dwNumNalUnitBytesIncluded;
         }
         DECODE_VERBOSEMESSAGE("HwStartCodeSupportEnabled %d, parSlice nextOffset %d, parSlice nextLength %d, IndirectBsdDataStartAddress %d, IndirectBsdDataLength %d",

Original file line number	Diff line number	Diff line change
`@@ -426,19 +426,19 @@ MOS_STATUS AvcDecodeSlcPkt::AddCmd_AVC_SLICE_Addr(MOS_COMMAND_BUFFER &cmdBuffer,`
`426`	`426`	`parSliceAddr.decodeInUse = true;`
`427`	`427`	`if (parSlice.fullFrameData)`
`428`	`428`	`{`
`429`		`- parSliceAddr.IndirectBsdDataLength = parSlice.nextLength;`
	`429`	`+ parSliceAddr.IndirectBsdDataLength = MOS_MIN(parSlice.nextLength, CODEC_AVC_SLICE_MIN_NUM_BYTES_LOAD);`
`430`	`430`	`parSliceAddr.IndirectBsdDataStartAddress = parSlice.nextOffset;`
`431`	`431`	`}`
`432`	`432`	`else`
`433`	`433`	`{`
`434`	`434`	`if (m_avcBasicFeature && m_avcBasicFeature->m_hwStartCodeSupportEnabled)`
`435`	`435`	`{`
`436`		`- parSliceAddr.IndirectBsdDataLength = parSlice.nextLength;`
	`436`	`+ parSliceAddr.IndirectBsdDataLength = MOS_MIN(parSlice.nextLength, CODEC_AVC_SLICE_MIN_NUM_BYTES_LOAD);`
`437`	`437`	`parSliceAddr.IndirectBsdDataStartAddress = parSlice.nextOffset;`
`438`	`438`	`}`
`439`	`439`	`else`
`440`	`440`	`{`
`441`		`- parSliceAddr.IndirectBsdDataLength = parSlice.nextLength + 1 - m_osInterface->dwNumNalUnitBytesIncluded;`
	`441`	`+ parSliceAddr.IndirectBsdDataLength = MOS_MIN(parSlice.nextLength + 1 - m_osInterface->dwNumNalUnitBytesIncluded, CODEC_AVC_SLICE_MIN_NUM_BYTES_LOAD);`
`442`	`442`	`parSliceAddr.IndirectBsdDataStartAddress = parSlice.nextOffset - 1 + m_osInterface->dwNumNalUnitBytesIncluded;`
`443`	`443`	`}`
`444`	`444`	`DECODE_VERBOSEMESSAGE("HwStartCodeSupportEnabled %d, parSlice nextOffset %d, parSlice nextLength %d, IndirectBsdDataStartAddress %d, IndirectBsdDataLength %d",`