# HG changeset patch # User mru # Date 1227572449 0 # Node ID c5276ad92ff83a497e842fd698529a1acd239f1c # Parent 93423820d82f595a07824a4d6d2a4a42c4c20205 snow: move scratch buffer from stack to malloced buffer in context This is required to ensure 16-byte alignment. diff -r 93423820d82f -r c5276ad92ff8 snow.c --- a/snow.c Mon Nov 24 21:31:01 2008 +0000 +++ b/snow.c Tue Nov 25 00:20:49 2008 +0000 @@ -488,6 +488,8 @@ slice_buffer sb; MpegEncContext m; // needed for motion estimation, should not be used for anything else, the idea is to eventually make the motion estimation independent of MpegEncContext, so this will be removed then (FIXME/XXX) + + uint8_t *scratchbuf; }SnowContext; typedef struct { @@ -2423,7 +2425,7 @@ BlockNode *rb= lb+1; uint8_t *block[4]; int tmp_step= src_stride >= 7*MB_SIZE ? MB_SIZE : MB_SIZE*src_stride; - uint8_t tmp[src_stride*7*MB_SIZE]; //FIXME align + uint8_t *tmp = s->scratchbuf; uint8_t *ptmp; int x,y; @@ -2785,7 +2787,7 @@ uint8_t *dst= s->current_picture.data[plane_index]; uint8_t *src= s-> input_picture.data[plane_index]; IDWTELEM *pred= (IDWTELEM*)s->m.obmc_scratchpad + plane_index*block_size*block_size*4; - uint8_t cur[ref_stride*2*MB_SIZE]; //FIXME alignment + uint8_t *cur = s->scratchbuf; uint8_t tmp[ref_stride*(2*MB_SIZE+HTAPS_MAX-1)]; const int b_stride = s->b_width << s->block_max_depth; const int b_height = s->b_height<< s->block_max_depth; @@ -3703,6 +3705,7 @@ scale_mv_ref[i][j] = 256*(i+1)/(j+1); s->avctx->get_buffer(s->avctx, &s->mconly_picture); + s->scratchbuf = av_malloc(s->mconly_picture.linesize[0]*7*MB_SIZE); return 0; } @@ -4438,6 +4441,7 @@ av_freep(&s->m.obmc_scratchpad); av_freep(&s->block); + av_freep(&s->scratchbuf); for(i=0; iref_mvs[i]);