Mercurial > libavcodec.hg
annotate mpegvideo_xvmc.c @ 8916:bd643af669df libavcodec
Check all critical xvmc struct fields in ff_xvmc_field_start()
and log error if they are not correct. All other functions
are supposedly called after that one, so use assert() for them.
| author | iive |
|---|---|
| date | Sun, 15 Feb 2009 10:06:23 +0000 |
| parents | 9dd34068523e |
| children | 422a9dbebb53 |
| rev | line source |
|---|---|
| 8831 | 1 /* |
| 2 * XVideo Motion Compensation | |
| 3 * Copyright (c) 2003 Ivan Kalvachev | |
| 4 * | |
| 5 * This file is part of FFmpeg. | |
| 6 * | |
| 7 * FFmpeg is free software; you can redistribute it and/or | |
| 8 * modify it under the terms of the GNU Lesser General Public | |
| 9 * License as published by the Free Software Foundation; either | |
| 10 * version 2.1 of the License, or (at your option) any later version. | |
| 11 * | |
| 12 * FFmpeg is distributed in the hope that it will be useful, | |
| 13 * but WITHOUT ANY WARRANTY; without even the implied warranty of | |
| 14 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | |
| 15 * Lesser General Public License for more details. | |
| 16 * | |
| 17 * You should have received a copy of the GNU Lesser General Public | |
| 18 * License along with FFmpeg; if not, write to the Free Software | |
| 19 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA | |
| 20 */ | |
| 21 | |
| 22 #include <limits.h> | |
| 23 | |
| 24 #include "avcodec.h" | |
| 25 #include "dsputil.h" | |
| 26 #include "mpegvideo.h" | |
| 27 | |
| 28 #undef NDEBUG | |
| 29 #include <assert.h> | |
| 30 | |
| 31 #include "xvmc.h" | |
|
8881
f8042554d4c8
Add xvmc_internal.h that contains all internal xvmc function declarations.
diego
parents:
8880
diff
changeset
|
32 #include "xvmc_internal.h" |
| 8831 | 33 |
|
8902
aa49d3e97b70
Doxygen comment/explanation for ff_xvmc_init_block().
diego
parents:
8897
diff
changeset
|
34 /** |
|
aa49d3e97b70
Doxygen comment/explanation for ff_xvmc_init_block().
diego
parents:
8897
diff
changeset
|
35 * Initializes the block field of the MpegEncContext pointer passed as |
|
aa49d3e97b70
Doxygen comment/explanation for ff_xvmc_init_block().
diego
parents:
8897
diff
changeset
|
36 * parameter after making sure that the data is not corrupted. |
|
aa49d3e97b70
Doxygen comment/explanation for ff_xvmc_init_block().
diego
parents:
8897
diff
changeset
|
37 */ |
| 8855 | 38 void ff_xvmc_init_block(MpegEncContext *s) |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
39 { |
|
8914
20da0a97b95f
Rename xvmc_render_state to something with pixfmt in its name.
iive
parents:
8912
diff
changeset
|
40 struct xvmc_pixfmt_render *render = (struct xvmc_pixfmt_render*)s->current_picture.data[2]; |
|
8916
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
41 assert(render && render->magic_id == AV_XVMC_RENDER_MAGIC); |
|
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
42 |
| 8860 | 43 s->block = (DCTELEM *)(render->data_blocks + render->next_free_data_block_num * 64); |
| 8831 | 44 } |
| 45 | |
| 8855 | 46 void ff_xvmc_pack_pblocks(MpegEncContext *s, int cbp) |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
47 { |
| 8889 | 48 int i, j = 0; |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
49 const int mb_block_count = 4 + (1 << s->chroma_format); |
| 8831 | 50 |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
51 cbp <<= 12-mb_block_count; |
|
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
52 for (i = 0; i < mb_block_count; i++) { |
|
8861
da23b389e856
cosmetics: Consistently format all if/else statements in K&R style
diego
parents:
8860
diff
changeset
|
53 if (cbp & (1 << 11)) |
| 8856 | 54 s->pblocks[i] = (short *)(&s->block[j++]); |
|
8861
da23b389e856
cosmetics: Consistently format all if/else statements in K&R style
diego
parents:
8860
diff
changeset
|
55 else |
| 8834 | 56 s->pblocks[i] = NULL; |
| 8831 | 57 cbp+=cbp; |
| 58 } | |
| 59 } | |
| 60 | |
| 8893 | 61 /** |
| 62 * This function should be called for every new field and/or frame. | |
| 63 * It should be safe to call the function a few times for the same field. | |
| 64 */ | |
| 8855 | 65 int ff_xvmc_field_start(MpegEncContext*s, AVCodecContext *avctx) |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
66 { |
|
8914
20da0a97b95f
Rename xvmc_render_state to something with pixfmt in its name.
iive
parents:
8912
diff
changeset
|
67 struct xvmc_pixfmt_render *last, *next, *render = (struct xvmc_pixfmt_render*)s->current_picture.data[2]; |
|
8916
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
68 const int mb_block_count = 4 + (1 << s->chroma_format); |
| 8831 | 69 |
|
8844
9ea7195b2ed4
Replace all (x == NULL) or (x != NULL) in assert and if conditions by !x and x.
diego
parents:
8842
diff
changeset
|
70 assert(avctx); |
|
8916
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
71 if (!render || render->magic_id != AV_XVMC_RENDER_MAGIC || |
|
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
72 !render->data_blocks || !render->mv_blocks){ |
|
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
73 av_log(avctx, AV_LOG_ERROR, |
|
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
74 "Render token doesn't look as expected.\n"); |
|
8870
c5112df7f8b8
cosmetics: grammar/spelling/wording fixes in comments
diego
parents:
8869
diff
changeset
|
75 return -1; // make sure that this is a render packet |
|
8916
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
76 } |
| 8831 | 77 |
| 78 render->picture_structure = s->picture_structure; | |
| 8860 | 79 render->flags = s->first_field ? 0 : XVMC_SECOND_FIELD; |
| 8831 | 80 |
| 8885 | 81 if (render->filled_mv_blocks_num) { |
| 8884 | 82 av_log(avctx, AV_LOG_ERROR, |
|
8916
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
83 "Rendering surface contains %i unprocessed blocks.\n", |
| 8885 | 84 render->filled_mv_blocks_num); |
| 8888 | 85 return -1; |
| 8884 | 86 } |
|
8916
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
87 if (render->total_number_of_mv_blocks < 1 || |
|
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
88 render->total_number_of_data_blocks < mb_block_count) { |
|
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
89 av_log(avctx, AV_LOG_ERROR, |
|
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
90 "Rendering surface doesn't provide enough block structures to work with.\n"); |
|
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
91 return -1; |
|
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
92 } |
|
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
93 if (render->total_number_of_mv_blocks < 1 || |
|
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
94 render->total_number_of_data_blocks < mb_block_count) { |
|
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
95 av_log(avctx, AV_LOG_ERROR, |
|
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
96 "Rendering surface doesn't provide enough block structures to work with.\n"); |
|
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
97 return -1; |
|
bd643af669df
Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents:
8915
diff
changeset
|
98 } |
| 8831 | 99 |
| 100 render->p_future_surface = NULL; | |
| 8837 | 101 render->p_past_surface = NULL; |
| 8831 | 102 |
| 8871 | 103 switch(s->pict_type) { |
| 8831 | 104 case FF_I_TYPE: |
|
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
105 return 0; // no prediction from other frames |
| 8831 | 106 case FF_B_TYPE: |
|
8914
20da0a97b95f
Rename xvmc_render_state to something with pixfmt in its name.
iive
parents:
8912
diff
changeset
|
107 next = (struct xvmc_pixfmt_render*)s->next_picture.data[2]; |
|
8844
9ea7195b2ed4
Replace all (x == NULL) or (x != NULL) in assert and if conditions by !x and x.
diego
parents:
8842
diff
changeset
|
108 if (!next) |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
109 return -1; |
|
8910
a11771712f48
Give a little better name to the magic(_id) field in xvmc struct
iive
parents:
8909
diff
changeset
|
110 if (next->magic_id != AV_XVMC_RENDER_MAGIC) |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
111 return -1; |
| 8831 | 112 render->p_future_surface = next->p_surface; |
|
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
113 // no return here, going to set forward prediction |
| 8831 | 114 case FF_P_TYPE: |
|
8914
20da0a97b95f
Rename xvmc_render_state to something with pixfmt in its name.
iive
parents:
8912
diff
changeset
|
115 last = (struct xvmc_pixfmt_render*)s->last_picture.data[2]; |
| 8872 | 116 if (!last) |
|
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
117 last = render; // predict second field from the first |
|
8910
a11771712f48
Give a little better name to the magic(_id) field in xvmc struct
iive
parents:
8909
diff
changeset
|
118 if (last->magic_id != AV_XVMC_RENDER_MAGIC) |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
119 return -1; |
| 8831 | 120 render->p_past_surface = last->p_surface; |
| 121 return 0; | |
| 122 } | |
| 123 | |
| 124 return -1; | |
| 125 } | |
| 126 | |
|
8897
fa198e44b199
Add Doxygen comment similar to ff_xvmc_field_start() to ff_xvmc_field_end().
diego
parents:
8896
diff
changeset
|
127 /** |
|
fa198e44b199
Add Doxygen comment similar to ff_xvmc_field_start() to ff_xvmc_field_end().
diego
parents:
8896
diff
changeset
|
128 * This function should be called for every new field and/or frame. |
|
fa198e44b199
Add Doxygen comment similar to ff_xvmc_field_start() to ff_xvmc_field_end().
diego
parents:
8896
diff
changeset
|
129 * It should be safe to call the function a few times for the same field. |
|
fa198e44b199
Add Doxygen comment similar to ff_xvmc_field_start() to ff_xvmc_field_end().
diego
parents:
8896
diff
changeset
|
130 */ |
| 8855 | 131 void ff_xvmc_field_end(MpegEncContext *s) |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
132 { |
|
8914
20da0a97b95f
Rename xvmc_render_state to something with pixfmt in its name.
iive
parents:
8912
diff
changeset
|
133 struct xvmc_pixfmt_render *render = (struct xvmc_pixfmt_render*)s->current_picture.data[2]; |
|
8844
9ea7195b2ed4
Replace all (x == NULL) or (x != NULL) in assert and if conditions by !x and x.
diego
parents:
8842
diff
changeset
|
134 assert(render); |
| 8831 | 135 |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
136 if (render->filled_mv_blocks_num > 0) |
| 8877 | 137 ff_draw_horiz_band(s, 0, 0); |
| 8831 | 138 } |
| 139 | |
| 8855 | 140 void ff_xvmc_decode_mb(MpegEncContext *s) |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
141 { |
| 8871 | 142 XvMCMacroBlock *mv_block; |
|
8914
20da0a97b95f
Rename xvmc_render_state to something with pixfmt in its name.
iive
parents:
8912
diff
changeset
|
143 struct xvmc_pixfmt_render *render; |
| 8871 | 144 int i, cbp, blocks_per_mb; |
| 8831 | 145 |
| 8834 | 146 const int mb_xy = s->mb_y * s->mb_stride + s->mb_x; |
| 8831 | 147 |
| 148 | |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
149 if (s->encoding) { |
| 8831 | 150 av_log(s->avctx, AV_LOG_ERROR, "XVMC doesn't support encoding!!!\n"); |
| 151 return; | |
| 152 } | |
| 153 | |
|
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
154 // from MPV_decode_mb(), update DC predictors for P macroblocks |
| 8831 | 155 if (!s->mb_intra) { |
| 156 s->last_dc[0] = | |
| 157 s->last_dc[1] = | |
| 158 s->last_dc[2] = 128 << s->intra_dc_precision; | |
| 159 } | |
| 160 | |
|
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
161 // MC doesn't skip blocks |
| 8831 | 162 s->mb_skipped = 0; |
| 163 | |
| 164 | |
|
8866
09d0ff5cdc81
whitespace cosmetics: Correctly indent all comments.
diego
parents:
8865
diff
changeset
|
165 // Do I need to export quant when I could not perform postprocessing? |
|
09d0ff5cdc81
whitespace cosmetics: Correctly indent all comments.
diego
parents:
8865
diff
changeset
|
166 // Anyway, it doesn't hurt. |
| 8831 | 167 s->current_picture.qscale_table[mb_xy] = s->qscale; |
| 168 | |
|
8870
c5112df7f8b8
cosmetics: grammar/spelling/wording fixes in comments
diego
parents:
8869
diff
changeset
|
169 // start of XVMC-specific code |
|
8914
20da0a97b95f
Rename xvmc_render_state to something with pixfmt in its name.
iive
parents:
8912
diff
changeset
|
170 render = (struct xvmc_pixfmt_render*)s->current_picture.data[2]; |
|
8844
9ea7195b2ed4
Replace all (x == NULL) or (x != NULL) in assert and if conditions by !x and x.
diego
parents:
8842
diff
changeset
|
171 assert(render); |
|
8910
a11771712f48
Give a little better name to the magic(_id) field in xvmc struct
iive
parents:
8909
diff
changeset
|
172 assert(render->magic_id == AV_XVMC_RENDER_MAGIC); |
| 8831 | 173 assert(render->mv_blocks); |
| 174 | |
|
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
175 // take the next free macroblock |
| 8831 | 176 mv_block = &render->mv_blocks[render->start_mv_blocks_num + |
| 8896 | 177 render->filled_mv_blocks_num]; |
| 8831 | 178 |
| 8837 | 179 mv_block->x = s->mb_x; |
| 180 mv_block->y = s->mb_y; | |
|
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
181 mv_block->dct_type = s->interlaced_dct; // XVMC_DCT_TYPE_FRAME/FIELD; |
|
8861
da23b389e856
cosmetics: Consistently format all if/else statements in K&R style
diego
parents:
8860
diff
changeset
|
182 if (s->mb_intra) { |
|
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
183 mv_block->macroblock_type = XVMC_MB_TYPE_INTRA; // no MC, all done |
|
8861
da23b389e856
cosmetics: Consistently format all if/else statements in K&R style
diego
parents:
8860
diff
changeset
|
184 } else { |
| 8831 | 185 mv_block->macroblock_type = XVMC_MB_TYPE_PATTERN; |
| 186 | |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
187 if (s->mv_dir & MV_DIR_FORWARD) { |
|
8836
4c175e9d492e
whitespace cosmetics: another round of formatting consistency fixes
diego
parents:
8835
diff
changeset
|
188 mv_block->macroblock_type |= XVMC_MB_TYPE_MOTION_FORWARD; |
|
8873
02c59b74ba71
Make one comment slightly clearer and more readable.
diego
parents:
8872
diff
changeset
|
189 // PMV[n][dir][xy] = mv[dir][n][xy] |
| 8831 | 190 mv_block->PMV[0][0][0] = s->mv[0][0][0]; |
| 191 mv_block->PMV[0][0][1] = s->mv[0][0][1]; | |
| 192 mv_block->PMV[1][0][0] = s->mv[0][1][0]; | |
| 193 mv_block->PMV[1][0][1] = s->mv[0][1][1]; | |
| 194 } | |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
195 if (s->mv_dir & MV_DIR_BACKWARD) { |
|
8836
4c175e9d492e
whitespace cosmetics: another round of formatting consistency fixes
diego
parents:
8835
diff
changeset
|
196 mv_block->macroblock_type |= XVMC_MB_TYPE_MOTION_BACKWARD; |
| 8831 | 197 mv_block->PMV[0][1][0] = s->mv[1][0][0]; |
| 198 mv_block->PMV[0][1][1] = s->mv[1][0][1]; | |
| 199 mv_block->PMV[1][1][0] = s->mv[1][1][0]; | |
| 200 mv_block->PMV[1][1][1] = s->mv[1][1][1]; | |
| 201 } | |
| 202 | |
| 8877 | 203 switch(s->mv_type) { |
| 8831 | 204 case MV_TYPE_16X16: |
| 205 mv_block->motion_type = XVMC_PREDICTION_FRAME; | |
| 206 break; | |
| 207 case MV_TYPE_16X8: | |
| 208 mv_block->motion_type = XVMC_PREDICTION_16x8; | |
| 209 break; | |
| 210 case MV_TYPE_FIELD: | |
| 211 mv_block->motion_type = XVMC_PREDICTION_FIELD; | |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
212 if (s->picture_structure == PICT_FRAME) { |
|
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
213 mv_block->PMV[0][0][1] <<= 1; |
|
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
214 mv_block->PMV[1][0][1] <<= 1; |
|
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
215 mv_block->PMV[0][1][1] <<= 1; |
|
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
216 mv_block->PMV[1][1][1] <<= 1; |
| 8831 | 217 } |
| 218 break; | |
| 219 case MV_TYPE_DMV: | |
| 220 mv_block->motion_type = XVMC_PREDICTION_DUAL_PRIME; | |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
221 if (s->picture_structure == PICT_FRAME) { |
| 8831 | 222 |
| 8876 | 223 mv_block->PMV[0][0][0] = s->mv[0][0][0]; // top from top |
|
8875
266bd7d64985
whitespace cosmetics: Put some spaces around operators for better readability.
diego
parents:
8874
diff
changeset
|
224 mv_block->PMV[0][0][1] = s->mv[0][0][1] << 1; |
| 8831 | 225 |
| 8876 | 226 mv_block->PMV[0][1][0] = s->mv[0][0][0]; // bottom from bottom |
|
8875
266bd7d64985
whitespace cosmetics: Put some spaces around operators for better readability.
diego
parents:
8874
diff
changeset
|
227 mv_block->PMV[0][1][1] = s->mv[0][0][1] << 1; |
| 8831 | 228 |
| 8876 | 229 mv_block->PMV[1][0][0] = s->mv[0][2][0]; // dmv00, top from bottom |
|
8875
266bd7d64985
whitespace cosmetics: Put some spaces around operators for better readability.
diego
parents:
8874
diff
changeset
|
230 mv_block->PMV[1][0][1] = s->mv[0][2][1] << 1; // dmv01 |
| 8831 | 231 |
| 8876 | 232 mv_block->PMV[1][1][0] = s->mv[0][3][0]; // dmv10, bottom from top |
|
8875
266bd7d64985
whitespace cosmetics: Put some spaces around operators for better readability.
diego
parents:
8874
diff
changeset
|
233 mv_block->PMV[1][1][1] = s->mv[0][3][1] << 1; // dmv11 |
| 8831 | 234 |
|
8861
da23b389e856
cosmetics: Consistently format all if/else statements in K&R style
diego
parents:
8860
diff
changeset
|
235 } else { |
| 8876 | 236 mv_block->PMV[0][1][0] = s->mv[0][2][0]; // dmv00 |
| 237 mv_block->PMV[0][1][1] = s->mv[0][2][1]; // dmv01 | |
| 8831 | 238 } |
| 239 break; | |
| 240 default: | |
| 241 assert(0); | |
| 242 } | |
| 243 | |
| 244 mv_block->motion_vertical_field_select = 0; | |
| 245 | |
|
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
246 // set correct field references |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
247 if (s->mv_type == MV_TYPE_FIELD || s->mv_type == MV_TYPE_16X8) { |
| 8862 | 248 mv_block->motion_vertical_field_select |= s->field_select[0][0]; |
| 8871 | 249 mv_block->motion_vertical_field_select |= s->field_select[1][0] << 1; |
| 250 mv_block->motion_vertical_field_select |= s->field_select[0][1] << 2; | |
| 251 mv_block->motion_vertical_field_select |= s->field_select[1][1] << 3; | |
| 8831 | 252 } |
|
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
253 } // !intra |
|
8870
c5112df7f8b8
cosmetics: grammar/spelling/wording fixes in comments
diego
parents:
8869
diff
changeset
|
254 // time to handle data blocks |
| 8831 | 255 mv_block->index = render->next_free_data_block_num; |
| 256 | |
| 257 blocks_per_mb = 6; | |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
258 if (s->chroma_format >= 2) { |
| 8856 | 259 blocks_per_mb = 4 + (1 << s->chroma_format); |
| 8831 | 260 } |
| 261 | |
|
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
262 // calculate cbp |
| 8831 | 263 cbp = 0; |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
264 for (i = 0; i < blocks_per_mb; i++) { |
|
8836
4c175e9d492e
whitespace cosmetics: another round of formatting consistency fixes
diego
parents:
8835
diff
changeset
|
265 cbp += cbp; |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
266 if (s->block_last_index[i] >= 0) |
| 8831 | 267 cbp++; |
| 268 } | |
| 269 | |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
270 if (s->flags & CODEC_FLAG_GRAY) { |
| 8876 | 271 if (s->mb_intra) { // intra frames are always full chroma blocks |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
272 for (i = 4; i < blocks_per_mb; i++) { |
| 8886 | 273 memset(s->pblocks[i], 0, sizeof(short)*64); // so we need to clear them |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
274 if (!render->unsigned_intra) |
|
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
275 s->pblocks[i][0] = 1 << 10; |
| 8831 | 276 } |
|
8861
da23b389e856
cosmetics: Consistently format all if/else statements in K&R style
diego
parents:
8860
diff
changeset
|
277 } else { |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
278 cbp &= 0xf << (blocks_per_mb - 4); |
| 8876 | 279 blocks_per_mb = 4; // luminance blocks only |
| 8831 | 280 } |
| 281 } | |
| 282 mv_block->coded_block_pattern = cbp; | |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
283 if (cbp == 0) |
| 8831 | 284 mv_block->macroblock_type &= ~XVMC_MB_TYPE_PATTERN; |
| 285 | |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
286 for (i = 0; i < blocks_per_mb; i++) { |
|
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
287 if (s->block_last_index[i] >= 0) { |
| 8831 | 288 // I do not have unsigned_intra MOCO to test, hope it is OK. |
|
8868
2534e44a9472
Restore one set of parentheses to avoid the warning:
diego
parents:
8867
diff
changeset
|
289 if (s->mb_intra && (render->idct || (!render->idct && !render->unsigned_intra))) |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
290 s->pblocks[i][0] -= 1 << 10; |
|
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
291 if (!render->idct) { |
| 8831 | 292 s->dsp.idct(s->pblocks[i]); |
|
8870
c5112df7f8b8
cosmetics: grammar/spelling/wording fixes in comments
diego
parents:
8869
diff
changeset
|
293 /* It is unclear if MC hardware requires pixel diff values to be |
|
c5112df7f8b8
cosmetics: grammar/spelling/wording fixes in comments
diego
parents:
8869
diff
changeset
|
294 * in the range [-255;255]. TODO: Clipping if such hardware is |
|
c5112df7f8b8
cosmetics: grammar/spelling/wording fixes in comments
diego
parents:
8869
diff
changeset
|
295 * ever found. As of now it would only be an unnecessary |
|
c5112df7f8b8
cosmetics: grammar/spelling/wording fixes in comments
diego
parents:
8869
diff
changeset
|
296 * slowdown. */ |
| 8831 | 297 } |
|
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
298 // copy blocks only if the codec doesn't support pblocks reordering |
|
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
299 if (s->avctx->xvmc_acceleration == 1) { |
| 8856 | 300 memcpy(&render->data_blocks[render->next_free_data_block_num*64], |
| 8880 | 301 s->pblocks[i], sizeof(short)*64); |
| 8831 | 302 } |
| 303 render->next_free_data_block_num++; | |
| 304 } | |
| 305 } | |
| 306 render->filled_mv_blocks_num++; | |
| 307 | |
| 8912 | 308 assert(render->filled_mv_blocks_num <= render->total_number_of_mv_blocks); |
| 8831 | 309 assert(render->next_free_data_block_num <= render->total_number_of_data_blocks); |
| 8912 | 310 /* The above conditions should not be able to fail as long as this function |
| 8887 | 311 * is used and the following 'if ()' automatically calls a callback to free |
| 312 * blocks. */ | |
| 8831 | 313 |
| 314 | |
| 8915 | 315 if (render->filled_mv_blocks_num == render->total_number_of_mv_blocks) |
| 8871 | 316 ff_draw_horiz_band(s, 0, 0); |
| 8831 | 317 } |
