MPCVideoDec: Используем функции "direct" только для аппаратных декодеров "copyback".

Для программного декодера это позволит, например, вместо plane_copy_direct_sse4 использовать функцию plane_copy_sse2, которая не требует выравнивания и соответственно не будет сильно проседать по производительности из-за неудачного стечения обстоятельств.

Author: v0lt

src/filters/transform/MPCVideoDec/FormatConverter.cpp

@@ -280,7 +280,7 @@ void CFormatConverter::SetConvertFunc()
 	m_RequiredAlignment = 16;
 
 	// optimized direct function
-	if (CPUInfo::HaveSSE4()) {
+	if (m_bDirect && CPUInfo::HaveSSE4()) {
 		if (m_FProps.pftype == PFType_NV12) {
 			if (m_out_pixfmt == PixFmt_NV12) {
 				m_pConvertFn = &CFormatConverter::plane_copy_direct_nv12_sse4;
@@ -320,6 +320,8 @@ void CFormatConverter::SetConvertFunc()
 		return;
 	}
 
+	m_pConvertFn = &CFormatConverter::ConvertGeneric;
+
 	// optimized function
 	switch (m_out_pixfmt) {
 	case PixFmt_NV12:
@@ -448,13 +450,6 @@ void CFormatConverter::SetConvertFunc()
 		}
 		break;
 	}
-
-	if (m_pConvertFn) {
-		return;
-	}
-
-	m_pConvertFn = &CFormatConverter::ConvertGeneric;
-	m_RequiredAlignment = 16;
 }
 
 void CFormatConverter::UpdateOutput(MPCPixelFormat out_pixfmt, int dstStride, int planeHeight)

src/filters/transform/MPCVideoDec/FormatConverter.h

@@ -148,6 +148,7 @@ class CFormatConverter
 	// Conversion function pointer
 	typedef HRESULT (CFormatConverter::*ConverterFn)(CONV_FUNC_PARAMS);
 	ConverterFn m_pConvertFn = nullptr;
+	BOOL m_bDirect = FALSE;
 
 	// from LAV Filters
 	HRESULT ConvertGeneric(CONV_FUNC_PARAMS);
@@ -197,6 +198,7 @@ class CFormatConverter
 	MPCPixelFormat GetOutPixFormat() { return m_out_pixfmt; }
 
 	bool Converting(BYTE* dst, AVFrame* pFrame);
+	void SetDirect(BOOL bDirect) { m_bDirect = bDirect; }
 
 	void Cleanup();
 

src/filters/transform/MPCVideoDec/MPCVideoDec.cpp

@@ -3615,6 +3615,7 @@ HRESULT CMPCVideoDecFilter::DecodeInternal(AVPacket *avpkt, REFERENCE_TIME rtSta
 
 			if (m_pHWFrame->format == m_HWPixFmt && !DXVAState::GetState()) {
 				SetDXVAState();
+				m_FormatConverter.SetDirect(TRUE);
 			}
 		}