Mesa (9.1): nvc0: allow frame dropping in h264

Maarten Lankhorst mlankhorst at kemper.freedesktop.org
Mon Jul 1 06:49:13 UTC 2013


Module: Mesa
Branch: 9.1
Commit: 9975e7c223cc05dd58e6ca23a93360863d6f294c
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=9975e7c223cc05dd58e6ca23a93360863d6f294c

Author: Maarten Lankhorst <maarten.lankhorst at canonical.com>
Date:   Mon Jul  1 08:47:49 2013 +0200

nvc0: allow frame dropping in h264

The only reason the checks existed were paranoia, when I first
wrote the code I wasn't sure it was correct. Now that I am,
the asserts triggered when XBMC was dropping frames, so remove it.

NOTE: This is a candidate for the 9.1 branch.
(cherry picked from commit bf95ca7de0c02a00a75d1d5e2693a33b68ff784d)

---

 src/gallium/drivers/nvc0/nvc0_video_vp.c |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/src/gallium/drivers/nvc0/nvc0_video_vp.c b/src/gallium/drivers/nvc0/nvc0_video_vp.c
index 84af0d6..12047ac 100644
--- a/src/gallium/drivers/nvc0/nvc0_video_vp.c
+++ b/src/gallium/drivers/nvc0/nvc0_video_vp.c
@@ -185,8 +185,6 @@ nvc0_decoder_handle_references(struct nvc0_decoder *dec, struct nvc0_video_buffe
               (!dec->refs[idx].decoded_bottom || !dec->refs[idx].decoded_top)));
       if (target == refs[i])
          empty_spot = 0;
-      assert(!h264 ||
-             dec->refs[idx].last_used == seq - 1);
 
       if (dec->refs[idx].vidbuf != refs[i]) {
          debug_printf("%p is not a real ref\n", refs[i]);
@@ -338,7 +336,6 @@ nvc0_decoder_fill_picparm_h264_vp(struct nvc0_decoder *dec,
    unsigned ring, i, j = 0;
    assert(offsetof(struct h264_picparm_vp, u224) == 0x224);
    *is_ref = d->is_reference;
-   assert(!d->frame_num || dec->last_frame_num + 1 == d->frame_num || dec->last_frame_num == d->frame_num);
    dec->last_frame_num = d->frame_num;
 
    h->width = mb(dec->base.width);




More information about the mesa-commit mailing list