]> granicus.if.org Git - libvpx/commitdiff
Fix row tiling.
authorRonald S. Bultje <rbultje@google.com>
Wed, 12 Jun 2013 17:30:06 +0000 (10:30 -0700)
committerRonald S. Bultje <rbultje@google.com>
Wed, 12 Jun 2013 17:42:59 +0000 (13:42 -0400)
Change-Id: I57be4eeaea6e4402f6a0cc04f5c6b7a5d9aedf9b

vp9/encoder/vp9_bitstream.c
vp9/encoder/vp9_encodeframe.c
vp9/encoder/vp9_onyx_int.h

index 86cd81031273d77b08dba4db1b03a275855365c8..fafe4a46b83e362e774463f6a11c3e6dc202e580 100644 (file)
@@ -967,8 +967,6 @@ static void write_modes(VP9_COMP *cpi, vp9_writer* const bc,
   int mi_row, mi_col;
 
   m_ptr += c->cur_tile_mi_col_start + c->cur_tile_mi_row_start * mis;
-  vpx_memset(c->above_seg_context, 0, sizeof(PARTITION_CONTEXT) *
-             mi_cols_aligned_to_sb(c));
 
   for (mi_row = c->cur_tile_mi_row_start;
        mi_row < c->cur_tile_mi_row_end;
@@ -1728,24 +1726,34 @@ void vp9_pack_bitstream(VP9_COMP *cpi, uint8_t *dest, unsigned long *size) {
   {
     int tile_row, tile_col, total_size = 0;
     unsigned char *data_ptr = cx_data + header_bc.pos;
-    TOKENEXTRA *tok[1 << 6], *tok_end;
+    TOKENEXTRA *tok[4][1 << 6], *tok_end;
 
-    tok[0] = cpi->tok;
-    for (tile_col = 1; tile_col < pc->tile_columns; tile_col++)
-      tok[tile_col] = tok[tile_col - 1] + cpi->tok_count[tile_col - 1];
+    vpx_memset(cpi->common.above_seg_context, 0, sizeof(PARTITION_CONTEXT) *
+               mi_cols_aligned_to_sb(&cpi->common));
+    tok[0][0] = cpi->tok;
+    for (tile_row = 0; tile_row < pc->tile_rows; tile_row++) {
+      if (tile_row) {
+        tok[tile_row][0] = tok[tile_row - 1][pc->tile_columns - 1] +
+                           cpi->tok_count[tile_row - 1][pc->tile_columns - 1];
+      }
+      for (tile_col = 1; tile_col < pc->tile_columns; tile_col++) {
+        tok[tile_row][tile_col] = tok[tile_row][tile_col - 1] +
+                                  cpi->tok_count[tile_row][tile_col - 1];
+      }
+    }
 
     for (tile_row = 0; tile_row < pc->tile_rows; tile_row++) {
       vp9_get_tile_row_offsets(pc, tile_row);
-      tok_end = cpi->tok + cpi->tok_count[0];
-      for (tile_col = 0; tile_col < pc->tile_columns;
-           tile_col++, tok_end += cpi->tok_count[tile_col]) {
+      for (tile_col = 0; tile_col < pc->tile_columns; tile_col++) {
         vp9_get_tile_col_offsets(pc, tile_col);
+        tok_end = tok[tile_row][tile_col] + cpi->tok_count[tile_row][tile_col];
 
         if (tile_col < pc->tile_columns - 1 || tile_row < pc->tile_rows - 1)
           vp9_start_encode(&residual_bc, data_ptr + total_size + 4);
         else
           vp9_start_encode(&residual_bc, data_ptr + total_size);
-        write_modes(cpi, &residual_bc, &tok[tile_col], tok_end);
+        write_modes(cpi, &residual_bc, &tok[tile_row][tile_col], tok_end);
+        assert(tok[tile_row][tile_col] == tok_end);
         vp9_stop_encode(&residual_bc);
         if (tile_col < pc->tile_columns - 1 || tile_row < pc->tile_rows - 1) {
           // size of this tile
@@ -1757,11 +1765,6 @@ void vp9_pack_bitstream(VP9_COMP *cpi, uint8_t *dest, unsigned long *size) {
       }
     }
 
-    assert((unsigned int)(tok[0] - cpi->tok) == cpi->tok_count[0]);
-    for (tile_col = 1; tile_col < pc->tile_columns; tile_col++)
-      assert((unsigned int)(tok[tile_col] - tok[tile_col - 1]) ==
-                  cpi->tok_count[tile_col]);
-
     *size += total_size;
   }
 }
index 213a9c72a89005296d1f6ff345947393a46e4056..bb784db1221fb4235d0ee4ffd1476bcab366b58d 100644 (file)
@@ -1572,7 +1572,7 @@ static void encode_frame_internal(VP9_COMP *cpi) {
                mi_row < cm->cur_tile_mi_row_end;
                mi_row += 8)
             encode_sb_row(cpi, mi_row, &tp, &totalrate);
-          cpi->tok_count[tile_col] = (unsigned int)(tp - tp_old);
+          cpi->tok_count[tile_row][tile_col] = (unsigned int)(tp - tp_old);
           assert(tp - cpi->tok <=
                  get_token_alloc(cm->mb_rows, cm->mb_cols));
         }
index 2b20f009c008b1c385b24bddcbf6063905c961fe..f5f1c077255f376a5044af3cc45d607c6fbb02d0 100644 (file)
@@ -296,7 +296,7 @@ typedef struct VP9_COMP {
   YV12_BUFFER_CONFIG last_frame_uf;
 
   TOKENEXTRA *tok;
-  unsigned int tok_count[1 << 6];
+  unsigned int tok_count[4][1 << 6];
 
 
   unsigned int frames_since_key;