[Libva] [Libva-intel-driver][PATCH] Set cost for modes used for VP8 encoding

[Sean V Kelley]

On Wed, 2016-08-03 at 16:43 +0800, Xiang, Haihao wrote:
> This is similar to what commit 1cd6795 does

lgtm, applied.

Thanks,

Sean

> > > 
Signed-off-by: Xiang, Haihao <haihao.xiang at intel.com>
Tested-by: Wang, Fei W <fei.w.wang at intel.com>
---
 src/gen6_mfc_common.c | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/src/gen6_mfc_common.c b/src/gen6_mfc_common.c
index c3e4d80..59f7785 100644
--- a/src/gen6_mfc_common.c
+++ b/src/gen6_mfc_common.c
@@ -906,10 +906,16 @@ void intel_vme_vp8_update_mbmv_cost(VADriverContextP ctx,
         qp = mfc_context->bit_rate_control_context[slice_type].QpPrimeY;
 
     lambda = intel_lambda_qp(qp * QP_MAX / VP8_QP_MAX);
+
+    m_cost = lambda;
+    vme_state_message[MODE_CHROMA_INTRA] = intel_format_lutvalue(m_cost, 0x8f);
+
     if (is_key_frame) {
 	vme_state_message[MODE_INTRA_16X16] = 0;
 	m_cost = lambda * 16; 
 	vme_state_message[MODE_INTRA_4X4] = intel_format_lutvalue(m_cost, 0x8f);
+	m_cost = lambda * 3;
+	vme_state_message[MODE_INTRA_NONPRED] = intel_format_lutvalue(m_cost, 0x6f);
     } else {
     	m_cost = 0;
 	vme_state_message[MODE_INTER_MV0] = intel_format_lutvalue(m_cost, 0x6f);
@@ -934,6 +940,7 @@ void intel_vme_vp8_update_mbmv_cost(VADriverContextP ctx,
             vme_state_message[MODE_INTER_16X8] = 0x4a;
             vme_state_message[MODE_INTER_8X8] = 0x4a;
             vme_state_message[MODE_INTER_4X4] = 0x4a;
+            vme_state_message[MODE_INTER_BWD] = 0;
             return;
 	}
 	m_costf = lambda * 10;
@@ -941,6 +948,10 @@ void intel_vme_vp8_update_mbmv_cost(VADriverContextP ctx,
 	m_cost = lambda * 24; 
 	vme_state_message[MODE_INTRA_4X4] = intel_format_lutvalue(m_cost, 0x8f);
             
+        m_costf = lambda * 3.5;
+        m_cost = m_costf;
+        vme_state_message[MODE_INTRA_NONPRED] = intel_format_lutvalue(m_cost, 0x6f);
+
         m_costf = lambda * 2.5;
         m_cost = m_costf;
         vme_state_message[MODE_INTER_16X16] = intel_format_lutvalue(m_cost, 0x8f);

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/libva/attachments/20160804/5bd182d2/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <https://lists.freedesktop.org/archives/libva/attachments/20160804/5bd182d2/attachment.sig>