From patchwork Sun May 27 20:27:39 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jerome Borsboom X-Patchwork-Id: 9111 Delivered-To: ffmpegpatchwork@gmail.com Received: by 2002:a02:11c:0:0:0:0:0 with SMTP id c28-v6csp1229611jad; Sun, 27 May 2018 13:27:51 -0700 (PDT) X-Google-Smtp-Source: ADUXVKI3Cqol8Uw0Qc1odS6XGBMpMMAofQsPa98+bQPwS/LiSXqHCtAvl2gB4YgI2Fj5kQOcxSZB X-Received: by 2002:a1c:3313:: with SMTP id z19-v6mr6385837wmz.48.1527452871279; Sun, 27 May 2018 13:27:51 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1527452871; cv=none; d=google.com; s=arc-20160816; b=jZD8xP/dYiYccm0DKFJadm4h3MdLpjQACO5eHJw8New8u0ryznEzZP1NX3EMhCnpc9 /Ze44V1N2j4exZ91VQHyEC2oOeT4SEI425/u+duquVywj2YMc4w8sSw2mChAgFgUn1XM 6cBVK4kaSIwltfbfzb/zG+AbiygO75180OPB9dnWALRTeAD7RDBioJRTNRWiEVzADm3w z9NOn3KAf1ot+koObPeUiuCVutt3h07aqPs5ktOq9rSwUIAnAiyY+WP4nKEsdZl9b0sr AvX+e0XfaVkVab4n/FxnQe9G6a9xwjqD0nHYqwWcrhCbJAwZsjiLY9kiEhpScp71Afwt /Xlg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:reply-to:list-subscribe :list-help:list-post:list-archive:list-unsubscribe:list-id :precedence:subject:content-language:mime-version:user-agent:date :message-id:from:to:dkim-signature:delivered-to :arc-authentication-results; bh=IaMR3GQFeK1fT7MDrd4jW5kWcfTOZwLWfIXC7UqZe7Y=; b=afdMeAw5djCz/3/M+BhpYzUaBCw/VMA/SEwr8uPEBOhnlVFtCV+eOE/lK1s3OJXs4/ HDEdlFewiQ+MZAqb9O7i+agEMSes/kyQWo16aZuq9CDsiyu8squwvvOoph+xAQKdRnXR 7cm4kUc3oUOTczdPn0SsXeTWkl2LV5NzNycWY43464DuWN+gslbC25nTXQWGAZA0n3yn YrMATu7ssBSXnmGUpV5WeO9ysi8r3gorHPoCE4rGlDOnRyv5bitZWc89eYeYtRvQIv3b 6rWM9igko89sJJguUchhnuOBda2vx7U/QCw+W7QH18T4iBDMsi1G4J/xEGxp6gJRF5fM RxAw== ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@carpalis.nl header.s=default header.b=ZwAnK0+C; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id j191-v6si8725018wmb.226.2018.05.27.13.27.50; Sun, 27 May 2018 13:27:51 -0700 (PDT) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@carpalis.nl header.s=default header.b=ZwAnK0+C; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 07CA8689CDA; Sun, 27 May 2018 23:27:05 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from kyoto.xs4all.nl (kyoto.xs4all.nl [83.161.153.34]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id D1524688393 for ; Sun, 27 May 2018 23:26:58 +0300 (EEST) Received: from [IPv6:2001:980:9507:0:8e70:5aff:fec6:83fc] ([IPv6:2001:980:9507:0:8e70:5aff:fec6:83fc]) (authenticated bits=0) by kyoto.xs4all.nl (8.14.7/8.14.7) with ESMTP id w4RKRdFt000913 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO) for ; Sun, 27 May 2018 22:27:40 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=carpalis.nl; s=default; t=1527452860; bh=6y+rkuRdr71U1sQxrBBKI4bHwo7j587RmB3xHKbArAM=; h=To:From:Subject:Date; b=ZwAnK0+C32J2ZWaEwiiEz0r+flCNZapC1zNqjVKO0hu7YazoDrQCXC7/y4wx1BIps gWShtwi5VesFEBLs98tYxtENgJWMmQzyGmSA6HIHB/nXaiXdJLshHO9RiQvNqnI3v+ A3T6I43M+ogwi9seKTFh1wGYW7/HcfzoHuxDvzhHJBvy1c/CUbkijWHIlNd+mJCrlp v8Qxliljx+Tnkp4/dSx9JbpK1Ju5rFe/pFy4nQGjyNL3/0MlRFr8Bq9wXICZJ6aOwq IhGhkw1ubQIM6IcmH+hK4oU67pIhLa25Q+0sVnBDurJtrGdt7SNzc3rsOXQrMPRU5o IBdDiSQoEG64Q== To: ffmpeg-devel@ffmpeg.org From: Jerome Borsboom Message-ID: <85bd0478-73c1-100f-2376-ef24b12992b5@carpalis.nl> Date: Sun, 27 May 2018 22:27:39 +0200 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0 MIME-Version: 1.0 Content-Language: nl Subject: [FFmpeg-devel] [PATCH] avcodec/vc1: fix out-of-bounds reference pixel replication X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Out-of-bounds reference pixel replication should take into account the frame coding mode of the reference frame(s), not the frame coding mode of the current frame. Signed-off-by: Jerome Borsboom --- This should fix the remaining issue with the SA10180.vc1 test file. libavcodec/vc1_mc.c | 659 ++++++++++++++++++++++++++++++---------------------- 1 file changed, 379 insertions(+), 280 deletions(-) diff --git a/libavcodec/vc1_mc.c b/libavcodec/vc1_mc.c index 04b359204c..b0f246eb4d 100644 --- a/libavcodec/vc1_mc.c +++ b/libavcodec/vc1_mc.c @@ -179,12 +179,17 @@ void ff_vc1_mc_1mv(VC1Context *v, int dir) int i; uint8_t (*luty)[256], (*lutuv)[256]; int use_ic; + int interlace; + int linesize, uvlinesize; if ((!v->field_mode || (v->ref_field_type[dir] == 1 && v->cur_field_type == 1)) && !v->s.last_picture.f->data[0]) return; + linesize = s->current_picture_ptr->f->linesize[0]; + uvlinesize = s->current_picture_ptr->f->linesize[1]; + mx = s->mv[dir][0][0]; my = s->mv[dir][0][1]; @@ -220,6 +225,7 @@ void ff_vc1_mc_1mv(VC1Context *v, int dir) luty = v->curr_luty; lutuv = v->curr_lutuv; use_ic = *v->curr_use_ic; + interlace = 1; } else { srcY = s->last_picture.f->data[0]; srcU = s->last_picture.f->data[1]; @@ -227,6 +233,7 @@ void ff_vc1_mc_1mv(VC1Context *v, int dir) luty = v->last_luty; lutuv = v->last_lutuv; use_ic = v->last_use_ic; + interlace = s->last_picture.f->interlaced_frame; } } else { srcY = s->next_picture.f->data[0]; @@ -235,6 +242,7 @@ void ff_vc1_mc_1mv(VC1Context *v, int dir) luty = v->next_luty; lutuv = v->next_lutuv; use_ic = v->next_use_ic; + interlace = s->next_picture.f->interlaced_frame; } if (!srcY || !srcU) { @@ -269,9 +277,9 @@ void ff_vc1_mc_1mv(VC1Context *v, int dir) srcV += uvsrc_y * s->uvlinesize + uvsrc_x; if (v->field_mode && v->ref_field_type[dir]) { - srcY += s->current_picture_ptr->f->linesize[0]; - srcU += s->current_picture_ptr->f->linesize[1]; - srcV += s->current_picture_ptr->f->linesize[2]; + srcY += linesize; + srcU += uvlinesize; + srcV += uvlinesize; } /* for grayscale we should not try to read from unknown area */ @@ -289,112 +297,104 @@ void ff_vc1_mc_1mv(VC1Context *v, int dir) const int k = 17 + s->mspel * 2; srcY -= s->mspel * (1 + s->linesize); - if (v->fcm == ILACE_FRAME) { - if (src_y - s->mspel & 1) { - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, - srcY, - 2 * s->linesize, - 2 * s->linesize, - k, - k + 1 >> 1, - src_x - s->mspel, - src_y - s->mspel >> 1, - s->h_edge_pos, - v_edge_pos + 1 >> 1); - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + s->linesize, - srcY + s->linesize, - 2 * s->linesize, - 2 * s->linesize, - k, - k >> 1, - src_x - s->mspel, - src_y - s->mspel + 1 >> 1, - s->h_edge_pos, - v_edge_pos >> 1); - } else { - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, - srcY, - 2 * s->linesize, - 2 * s->linesize, - k, - k + 1 >> 1, - src_x - s->mspel, - src_y - s->mspel >> 1, - s->h_edge_pos, - v_edge_pos >> 1); - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + s->linesize, - srcY + s->linesize, - 2 * s->linesize, - 2 * s->linesize, + if (interlace) { + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, + srcY, + linesize << 1, + linesize << 1, + k, + v->field_mode ? k : k + 1 >> 1, + src_x - s->mspel, + src_y - s->mspel >> !v->field_mode, + s->h_edge_pos, + s->v_edge_pos >> 1); + if (!v->field_mode) + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + linesize, + srcY + linesize, + linesize << 1, + linesize << 1, k, k >> 1, src_x - s->mspel, src_y - s->mspel + 1 >> 1, s->h_edge_pos, - v_edge_pos + 1 >> 1); - } + s->v_edge_pos >> 1); } else - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, srcY, - s->linesize, s->linesize, - k, k, - src_x - s->mspel, src_y - s->mspel, - s->h_edge_pos, v_edge_pos); + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, + srcY, + linesize, + linesize, + k, + v->field_mode ? (k << 1) - 1 : k, + src_x - s->mspel, + src_y - s->mspel, + s->h_edge_pos, + s->v_edge_pos >> v->field_mode); srcY = s->sc.edge_emu_buffer; - if (v->fcm == ILACE_FRAME) { - if (uvsrc_y & 1) { - s->vdsp.emulated_edge_mc(ubuf, srcU, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 8 + 1, 5, - uvsrc_x, uvsrc_y >> 1, - s->h_edge_pos >> 1, (v_edge_pos >> 1) + 1 >> 1); - s->vdsp.emulated_edge_mc(ubuf + s->uvlinesize, srcU + s->uvlinesize, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 8 + 1, 4, - uvsrc_x, uvsrc_y + 1 >> 1, - s->h_edge_pos >> 1, (v_edge_pos >> 1) >> 1); - s->vdsp.emulated_edge_mc(vbuf, srcV, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 8 + 1, 5, - uvsrc_x, uvsrc_y >> 1, - s->h_edge_pos >> 1, (v_edge_pos >> 1) + 1 >> 1); - s->vdsp.emulated_edge_mc(vbuf + s->uvlinesize, srcV + s->uvlinesize, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 8 + 1, 4, - uvsrc_x, uvsrc_y + 1 >> 1, - s->h_edge_pos >> 1, (v_edge_pos >> 1) >> 1); - } else { - s->vdsp.emulated_edge_mc(ubuf, srcU, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 8 + 1, 5, - uvsrc_x, uvsrc_y >> 1, - s->h_edge_pos >> 1, (v_edge_pos >> 1) >> 1); - s->vdsp.emulated_edge_mc(ubuf + s->uvlinesize, srcU + s->uvlinesize, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 8 + 1, 4, - uvsrc_x, uvsrc_y + 1 >> 1, - s->h_edge_pos >> 1, (v_edge_pos >> 1) + 1 >> 1); - s->vdsp.emulated_edge_mc(vbuf, srcV, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 8 + 1, 5, - uvsrc_x, uvsrc_y >> 1, - s->h_edge_pos >> 1, (v_edge_pos >> 1) >> 1); - s->vdsp.emulated_edge_mc(vbuf + s->uvlinesize, srcV + s->uvlinesize, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 8 + 1, 4, - uvsrc_x, uvsrc_y + 1 >> 1, - s->h_edge_pos >> 1, (v_edge_pos >> 1) + 1 >> 1); + if (interlace) { + s->vdsp.emulated_edge_mc(ubuf, + srcU, + uvlinesize << 1, + uvlinesize << 1, + 9, + v->field_mode ? 9 : 5, + uvsrc_x, + uvsrc_y >> !v->field_mode, + s->h_edge_pos >> 1, + s->v_edge_pos >> 2); + s->vdsp.emulated_edge_mc(vbuf, + srcV, + uvlinesize << 1, + uvlinesize << 1, + 9, + v->field_mode ? 9 : 5, + uvsrc_x, + uvsrc_y >> !v->field_mode, + s->h_edge_pos >> 1, + s->v_edge_pos >> 2); + if (!v->field_mode) { + s->vdsp.emulated_edge_mc(ubuf + uvlinesize, + srcU + uvlinesize, + uvlinesize << 1, + uvlinesize << 1, + 9, + 4, + uvsrc_x, + uvsrc_y + 1 >> 1, + s->h_edge_pos >> 1, + s->v_edge_pos >> 2); + s->vdsp.emulated_edge_mc(vbuf + uvlinesize, + srcV + uvlinesize, + uvlinesize << 1, + uvlinesize << 1, + 9, + 4, + uvsrc_x, + uvsrc_y + 1 >> 1, + s->h_edge_pos >> 1, + s->v_edge_pos >> 2); } } else { - s->vdsp.emulated_edge_mc(ubuf, srcU, - s->uvlinesize, s->uvlinesize, - 8 + 1, 8 + 1, - uvsrc_x, uvsrc_y, - s->h_edge_pos >> 1, v_edge_pos >> 1); - s->vdsp.emulated_edge_mc(vbuf, srcV, - s->uvlinesize, s->uvlinesize, - 8 + 1, 8 + 1, - uvsrc_x, uvsrc_y, - s->h_edge_pos >> 1, v_edge_pos >> 1); + s->vdsp.emulated_edge_mc(ubuf, + srcU, + uvlinesize, + uvlinesize, + 9, + v->field_mode ? 17 : 9, + uvsrc_x, + uvsrc_y, + s->h_edge_pos >> 1, + s->v_edge_pos >> 1 + v->field_mode); + s->vdsp.emulated_edge_mc(vbuf, + srcV, + uvlinesize, + uvlinesize, + 9, + v->field_mode ? 17 : 9, + uvsrc_x, + uvsrc_y, + s->h_edge_pos >> 1, + s->v_edge_pos >> 1 + v->field_mode); } srcU = ubuf; srcV = vbuf; @@ -458,12 +458,16 @@ void ff_vc1_mc_4mv_luma(VC1Context *v, int n, int dir, int avg) int v_edge_pos = s->v_edge_pos >> v->field_mode; uint8_t (*luty)[256]; int use_ic; + int interlace; + int linesize; if ((!v->field_mode || (v->ref_field_type[dir] == 1 && v->cur_field_type == 1)) && !v->s.last_picture.f->data[0]) return; + linesize = s->current_picture_ptr->f->linesize[0]; + mx = s->mv[dir][n][0]; my = s->mv[dir][n][1]; @@ -472,15 +476,18 @@ void ff_vc1_mc_4mv_luma(VC1Context *v, int n, int dir, int avg) srcY = s->current_picture.f->data[0]; luty = v->curr_luty; use_ic = *v->curr_use_ic; + interlace = 1; } else { srcY = s->last_picture.f->data[0]; luty = v->last_luty; use_ic = v->last_use_ic; + interlace = s->last_picture.f->interlaced_frame; } } else { srcY = s->next_picture.f->data[0]; luty = v->next_luty; use_ic = v->next_use_ic; + interlace = s->next_picture.f->interlaced_frame; } if (!srcY) { @@ -547,7 +554,7 @@ void ff_vc1_mc_4mv_luma(VC1Context *v, int n, int dir, int avg) srcY += src_y * s->linesize + src_x; if (v->field_mode && v->ref_field_type[dir]) - srcY += s->current_picture_ptr->f->linesize[0]; + srcY += linesize; if (v->rangeredfrm || use_ic || s->h_edge_pos < 13 || v_edge_pos < 23 @@ -557,36 +564,39 @@ void ff_vc1_mc_4mv_luma(VC1Context *v, int n, int dir, int avg) srcY -= s->mspel * (1 + (s->linesize << fieldmv)); /* check emulate edge stride and offset */ - if (v->fcm == ILACE_FRAME) { - if (src_y - (s->mspel << fieldmv) & 1) { - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, srcY, - 2 * s->linesize, 2 * s->linesize, - k, (k << fieldmv) + 1 >> 1, - src_x - s->mspel, src_y - (s->mspel << fieldmv) >> 1, - s->h_edge_pos, v_edge_pos + 1 >> 1); - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + s->linesize, srcY + s->linesize, - 2 * s->linesize, 2 * s->linesize, - k, (k << fieldmv) >> 1, - src_x - s->mspel, src_y - (s->mspel << fieldmv) + 1 >> 1, - s->h_edge_pos, v_edge_pos >> 1); - } else { - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, srcY, - 2 * s->linesize, 2 * s->linesize, - k, (k << fieldmv) + 1 >> 1, - src_x - s->mspel, src_y - (s->mspel << fieldmv) >> 1, - s->h_edge_pos, v_edge_pos >> 1); - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + s->linesize, srcY + s->linesize, - 2 * s->linesize, 2 * s->linesize, - k, (k << fieldmv) >> 1, - src_x - s->mspel, src_y - (s->mspel << fieldmv) + 1 >> 1, - s->h_edge_pos, v_edge_pos + 1 >> 1); - } + if (interlace) { + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, + srcY, + linesize << 1, + linesize << 1, + k, + v->field_mode ? k : (k << fieldmv) + 1 >> 1, + src_x - s->mspel, + src_y - (s->mspel << fieldmv) >> !v->field_mode, + s->h_edge_pos, + s->v_edge_pos >> 1); + if (!v->field_mode && !fieldmv) + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + linesize, + srcY + linesize, + linesize << 1, + linesize << 1, + k, + k >> 1, + src_x - s->mspel, + src_y - s->mspel + 1 >> 1, + s->h_edge_pos, + s->v_edge_pos >> 1); } else - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, srcY, - s->linesize, s->linesize, - k, k, - src_x - s->mspel, src_y - s->mspel, - s->h_edge_pos, v_edge_pos); + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, + srcY, + linesize, + linesize, + k, + v->field_mode ? (k << 1) - 1 : k << fieldmv, + src_x - s->mspel, + src_y - (s->mspel << fieldmv), + s->h_edge_pos, + s->v_edge_pos >> v->field_mode); srcY = s->sc.edge_emu_buffer; /* if we deal with range reduction we need to scale source blocks */ if (v->rangeredfrm) { @@ -630,6 +640,8 @@ void ff_vc1_mc_4mv_chroma(VC1Context *v, int dir) int v_edge_pos = s->v_edge_pos >> v->field_mode; uint8_t (*lutuv)[256]; int use_ic; + int interlace; + int uvlinesize; if (!v->field_mode && !v->s.last_picture.f->data[0]) return; @@ -654,6 +666,9 @@ void ff_vc1_mc_4mv_chroma(VC1Context *v, int dir) return; s->current_picture.motion_val[1][s->block_index[0] + v->blocks_off][0] = tx; s->current_picture.motion_val[1][s->block_index[0] + v->blocks_off][1] = ty; + + uvlinesize = s->current_picture_ptr->f->linesize[1]; + uvmx = (tx + ((tx & 3) == 3)) >> 1; uvmy = (ty + ((ty & 3) == 3)) >> 1; @@ -685,17 +700,20 @@ void ff_vc1_mc_4mv_chroma(VC1Context *v, int dir) srcV = s->current_picture.f->data[2]; lutuv = v->curr_lutuv; use_ic = *v->curr_use_ic; + interlace = 1; } else { srcU = s->last_picture.f->data[1]; srcV = s->last_picture.f->data[2]; lutuv = v->last_lutuv; use_ic = v->last_use_ic; + interlace = s->last_picture.f->interlaced_frame; } } else { srcU = s->next_picture.f->data[1]; srcV = s->next_picture.f->data[2]; lutuv = v->next_lutuv; use_ic = v->next_use_ic; + interlace = s->next_picture.f->interlaced_frame; } if (!srcU) { @@ -708,8 +726,8 @@ void ff_vc1_mc_4mv_chroma(VC1Context *v, int dir) if (v->field_mode) { if (chroma_ref_type) { - srcU += s->current_picture_ptr->f->linesize[1]; - srcV += s->current_picture_ptr->f->linesize[2]; + srcU += uvlinesize; + srcV += uvlinesize; } } @@ -717,14 +735,71 @@ void ff_vc1_mc_4mv_chroma(VC1Context *v, int dir) || s->h_edge_pos < 18 || v_edge_pos < 18 || (unsigned)uvsrc_x > (s->h_edge_pos >> 1) - 9 || (unsigned)uvsrc_y > (v_edge_pos >> 1) - 9) { - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, srcU, - s->uvlinesize, s->uvlinesize, - 8 + 1, 8 + 1, uvsrc_x, uvsrc_y, - s->h_edge_pos >> 1, v_edge_pos >> 1); - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + 16, srcV, - s->uvlinesize, s->uvlinesize, - 8 + 1, 8 + 1, uvsrc_x, uvsrc_y, - s->h_edge_pos >> 1, v_edge_pos >> 1); + if (interlace) { + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, + srcU, + uvlinesize << 1, + uvlinesize << 1, + 9, + v->field_mode ? 9 : 5, + uvsrc_x, + uvsrc_y >> !v->field_mode, + s->h_edge_pos >> 1, + s->v_edge_pos >> 2); + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + 16, + srcV, + uvlinesize << 1, + uvlinesize << 1, + 9, + v->field_mode ? 9 : 5, + uvsrc_x, + uvsrc_y >> !v->field_mode, + s->h_edge_pos >> 1, + s->v_edge_pos >> 2); + if (!v->field_mode) { + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + uvlinesize, + srcU + uvlinesize, + uvlinesize << 1, + uvlinesize << 1, + 9, + 4, + uvsrc_x, + uvsrc_y + 1 >> 1, + s->h_edge_pos >> 1, + s->v_edge_pos >> 2); + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + 16 + uvlinesize, + srcV + uvlinesize, + uvlinesize << 1, + uvlinesize << 1, + 9, + 4, + uvsrc_x, + uvsrc_y + 1 >> 1, + s->h_edge_pos >> 1, + s->v_edge_pos >> 2); + } + } else { + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, + srcU, + uvlinesize, + uvlinesize, + 9, + v->field_mode ? 17 : 9, + uvsrc_x, + uvsrc_y, + s->h_edge_pos >> 1, + s->v_edge_pos >> 1 + v->field_mode); + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + 16, + srcV, + uvlinesize, + uvlinesize, + 9, + v->field_mode ? 17 : 9, + uvsrc_x, + uvsrc_y, + s->h_edge_pos >> 1, + s->v_edge_pos >> 1 + v->field_mode); + } srcU = s->sc.edge_emu_buffer; srcV = s->sc.edge_emu_buffer + 16; @@ -772,11 +847,15 @@ void ff_vc1_mc_4mv_chroma4(VC1Context *v, int dir, int dir2, int avg) int v_dist = fieldmv ? 1 : 4; // vertical offset for lower sub-blocks int v_edge_pos = s->v_edge_pos >> 1; int use_ic; + int interlace; + int uvlinesize; uint8_t (*lutuv)[256]; if (CONFIG_GRAY && s->avctx->flags & AV_CODEC_FLAG_GRAY) return; + uvlinesize = s->current_picture_ptr->f->linesize[1]; + for (i = 0; i < 4; i++) { int d = i < 2 ? dir: dir2; tx = s->mv[d][i][0]; @@ -803,11 +882,13 @@ void ff_vc1_mc_4mv_chroma4(VC1Context *v, int dir, int dir2, int avg) srcV = s->next_picture.f->data[2]; lutuv = v->next_lutuv; use_ic = v->next_use_ic; + interlace = s->next_picture.f->interlaced_frame; } else { srcU = s->last_picture.f->data[1]; srcV = s->last_picture.f->data[2]; lutuv = v->last_lutuv; use_ic = v->last_use_ic; + interlace = s->last_picture.f->interlaced_frame; } if (!srcU) return; @@ -820,51 +901,70 @@ void ff_vc1_mc_4mv_chroma4(VC1Context *v, int dir, int dir2, int avg) || s->h_edge_pos < 10 || v_edge_pos < (5 << fieldmv) || (unsigned)uvsrc_x > (s->h_edge_pos >> 1) - 5 || (unsigned)uvsrc_y > v_edge_pos - (5 << fieldmv)) { - if (v->fcm == ILACE_FRAME) { - if (uvsrc_y & 1) { - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, srcU, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 5, (5 << fieldmv) + 1 >> 1, uvsrc_x, uvsrc_y >> 1, - s->h_edge_pos >> 1, v_edge_pos + 1 >> 1); - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + s->uvlinesize, srcU + s->uvlinesize, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 5, (5 << fieldmv) >> 1, uvsrc_x, uvsrc_y + 1 >> 1, - s->h_edge_pos >> 1, v_edge_pos >> 1); - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + 16, srcV, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 5, (5 << fieldmv) + 1 >> 1, uvsrc_x, uvsrc_y >> 1, - s->h_edge_pos >> 1, v_edge_pos + 1 >> 1); - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + 16 + s->uvlinesize, srcV + s->uvlinesize, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 5, (5 << fieldmv) >> 1, uvsrc_x, uvsrc_y + 1 >> 1, - s->h_edge_pos >> 1, v_edge_pos >> 1); - } else { - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, srcU, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 5, (5 << fieldmv) + 1 >> 1, uvsrc_x, uvsrc_y >> 1, - s->h_edge_pos >> 1, v_edge_pos >> 1); - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + s->uvlinesize, srcU + s->uvlinesize, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 5, (5 << fieldmv) >> 1, uvsrc_x, uvsrc_y + 1 >> 1, - s->h_edge_pos >> 1, v_edge_pos + 1 >> 1); - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + 16, srcV, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 5, (5 << fieldmv) + 1 >> 1, uvsrc_x, uvsrc_y >> 1, - s->h_edge_pos >> 1, v_edge_pos >> 1); - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + 16 + s->uvlinesize, srcV + s->uvlinesize, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 5, (5 << fieldmv) + 1, uvsrc_x, uvsrc_y + 1 >> 1, - s->h_edge_pos >> 1, v_edge_pos + 1 >> 1); + if (interlace) { + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, + srcU, + uvlinesize << 1, + uvlinesize << 1, + 5, + (5 << fieldmv) + 1 >> 1, + uvsrc_x, + uvsrc_y >> 1, + s->h_edge_pos >> 1, + s->v_edge_pos >> 2); + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + 16, + srcV, + uvlinesize << 1, + uvlinesize << 1, + 5, + (5 << fieldmv) + 1 >> 1, + uvsrc_x, + uvsrc_y >> 1, + s->h_edge_pos >> 1, + s->v_edge_pos >> 2); + if (!fieldmv) { + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + uvlinesize, + srcU + uvlinesize, + uvlinesize << 1, + uvlinesize << 1, + 5, + 2, + uvsrc_x, + uvsrc_y + 1 >> 1, + s->h_edge_pos >> 1, + s->v_edge_pos >> 2); + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + 16 + uvlinesize, + srcV + uvlinesize, + uvlinesize << 1, + uvlinesize << 1, + 5, + 2, + uvsrc_x, + uvsrc_y + 1 >> 1, + s->h_edge_pos >> 1, + s->v_edge_pos >> 2); } } else { - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, srcU, - s->uvlinesize, s->uvlinesize, - 5, 5, uvsrc_x, uvsrc_y, - s->h_edge_pos >> 1, v_edge_pos); - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + 16, srcV, - s->uvlinesize, s->uvlinesize, - 5, 5, uvsrc_x, uvsrc_y, - s->h_edge_pos >> 1, v_edge_pos); + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, + srcU, + uvlinesize, + uvlinesize, + 5, + 5 << fieldmv, + uvsrc_x, + uvsrc_y, + s->h_edge_pos >> 1, + s->v_edge_pos >> 1); + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + 16, + srcV, + uvlinesize, + uvlinesize, + 5, + 5 << fieldmv, + uvsrc_x, + uvsrc_y, + s->h_edge_pos >> 1, + s->v_edge_pos >> 1); } srcU = s->sc.edge_emu_buffer; srcV = s->sc.edge_emu_buffer + 16; @@ -907,10 +1007,15 @@ void ff_vc1_interp_mc(VC1Context *v) int dxy, mx, my, uvmx, uvmy, src_x, src_y, uvsrc_x, uvsrc_y; int v_edge_pos = s->v_edge_pos >> v->field_mode; int use_ic = v->next_use_ic; + int interlace; + int linesize, uvlinesize; if (!v->field_mode && !v->s.next_picture.f->data[0]) return; + linesize = s->current_picture_ptr->f->linesize[0]; + uvlinesize = s->current_picture_ptr->f->linesize[1]; + mx = s->mv[1][0][0]; my = s->mv[1][0][1]; uvmx = (mx + ((mx & 3) == 3)) >> 1; @@ -927,6 +1032,8 @@ void ff_vc1_interp_mc(VC1Context *v) srcU = s->next_picture.f->data[1]; srcV = s->next_picture.f->data[2]; + interlace = s->next_picture.f->interlaced_frame; + src_x = s->mb_x * 16 + (mx >> 2); src_y = s->mb_y * 16 + (my >> 2); uvsrc_x = s->mb_x * 8 + (uvmx >> 2); @@ -973,112 +1080,104 @@ void ff_vc1_interp_mc(VC1Context *v) const int k = 17 + s->mspel * 2; srcY -= s->mspel * (1 + s->linesize); - if (v->fcm == ILACE_FRAME) { - if (src_y - s->mspel & 1) { - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, - srcY, - 2 * s->linesize, - 2 * s->linesize, - k, - k + 1 >> 1, - src_x - s->mspel, - src_y - s->mspel >> 1, - s->h_edge_pos, - v_edge_pos + 1 >> 1); - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + s->linesize, - srcY + s->linesize, - 2 * s->linesize, - 2 * s->linesize, - k, - k >> 1, - src_x - s->mspel, - src_y - s->mspel + 1 >> 1, - s->h_edge_pos, - v_edge_pos >> 1); - } else { - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, - srcY, - 2 * s->linesize, - 2 * s->linesize, - k, - k + 1 >> 1, - src_x - s->mspel, - src_y - s->mspel >> 1, - s->h_edge_pos, - v_edge_pos >> 1); - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + s->linesize, - srcY + s->linesize, - 2 * s->linesize, - 2 * s->linesize, + if (interlace) { + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, + srcY, + linesize << 1, + linesize << 1, + k, + v->field_mode ? k : (k + 1 >> 1), + src_x - s->mspel, + src_y - s->mspel >> !v->field_mode, + s->h_edge_pos, + s->v_edge_pos >> 1); + if (!v->field_mode) + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer + linesize, + srcY + linesize, + linesize << 1, + linesize << 1, k, k >> 1, src_x - s->mspel, src_y - s->mspel + 1 >> 1, s->h_edge_pos, - v_edge_pos + 1 >> 1); - } + s->v_edge_pos >> 1); } else - s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, srcY, - s->linesize, s->linesize, - k, k, - src_x - s->mspel, src_y - s->mspel, - s->h_edge_pos, v_edge_pos); + s->vdsp.emulated_edge_mc(s->sc.edge_emu_buffer, + srcY, + linesize, + linesize, + k, + v->field_mode ? (k << 1) - 1 : k, + src_x - s->mspel, + src_y - s->mspel, + s->h_edge_pos, + s->v_edge_pos >> v->field_mode); srcY = s->sc.edge_emu_buffer; - if (v->fcm == ILACE_FRAME) { - if (uvsrc_y & 1) { - s->vdsp.emulated_edge_mc(ubuf, srcU, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 8 + 1, 5, - uvsrc_x, uvsrc_y >> 1, - s->h_edge_pos >> 1, (v_edge_pos >> 1) + 1 >> 1); - s->vdsp.emulated_edge_mc(ubuf + s->uvlinesize, srcU + s->uvlinesize, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 8 + 1, 4, - uvsrc_x, uvsrc_y + 1 >> 1, - s->h_edge_pos >> 1, (v_edge_pos >> 1) >> 1); - s->vdsp.emulated_edge_mc(vbuf, srcV, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 8 + 1, 5, - uvsrc_x, uvsrc_y >> 1, - s->h_edge_pos >> 1, (v_edge_pos >> 1) + 1 >> 1); - s->vdsp.emulated_edge_mc(vbuf + s->uvlinesize, srcV + s->uvlinesize, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 8 + 1, 4, - uvsrc_x, uvsrc_y + 1 >> 1, - s->h_edge_pos >> 1, (v_edge_pos >> 1) >> 1); - } else { - s->vdsp.emulated_edge_mc(ubuf, srcU, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 8 + 1, 5, - uvsrc_x, uvsrc_y >> 1, - s->h_edge_pos >> 1, (v_edge_pos >> 1) >> 1); - s->vdsp.emulated_edge_mc(ubuf + s->uvlinesize, srcU + s->uvlinesize, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 8 + 1, 4, - uvsrc_x, uvsrc_y + 1 >> 1, - s->h_edge_pos >> 1, (v_edge_pos >> 1) + 1 >> 1); - s->vdsp.emulated_edge_mc(vbuf, srcV, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 8 + 1, 5, - uvsrc_x, uvsrc_y >> 1, - s->h_edge_pos >> 1, (v_edge_pos >> 1) >> 1); - s->vdsp.emulated_edge_mc(vbuf + s->uvlinesize, srcV + s->uvlinesize, - 2 * s->uvlinesize, 2 * s->uvlinesize, - 8 + 1, 4, - uvsrc_x, uvsrc_y + 1 >> 1, - s->h_edge_pos >> 1, (v_edge_pos >> 1) + 1 >> 1); + if (interlace) { + s->vdsp.emulated_edge_mc(ubuf, + srcU, + uvlinesize << 1, + uvlinesize << 1, + 9, + v->field_mode ? 9 : 5, + uvsrc_x, + uvsrc_y >> !v->field_mode, + s->h_edge_pos >> 1, + s->v_edge_pos >> 2); + s->vdsp.emulated_edge_mc(vbuf, + srcV, + uvlinesize << 1, + uvlinesize << 1, + 9, + v->field_mode ? 9 : 5, + uvsrc_x, + uvsrc_y >> !v->field_mode, + s->h_edge_pos >> 1, + s->v_edge_pos >> 2); + if (!v->field_mode) { + s->vdsp.emulated_edge_mc(ubuf + uvlinesize, + srcU + uvlinesize, + uvlinesize << 1, + uvlinesize << 1, + 9, + 4, + uvsrc_x, + uvsrc_y + 1 >> 1, + s->h_edge_pos >> 1, + s->v_edge_pos >> 2); + s->vdsp.emulated_edge_mc(vbuf + uvlinesize, + srcV + uvlinesize, + uvlinesize << 1, + uvlinesize << 1, + 9, + 4, + uvsrc_x, + uvsrc_y + 1 >> 1, + s->h_edge_pos >> 1, + s->v_edge_pos >> 2); } } else { - s->vdsp.emulated_edge_mc(ubuf, srcU, - s->uvlinesize, s->uvlinesize, - 8 + 1, 8 + 1, - uvsrc_x, uvsrc_y, - s->h_edge_pos >> 1, v_edge_pos >> 1); - s->vdsp.emulated_edge_mc(vbuf, srcV, - s->uvlinesize, s->uvlinesize, - 8 + 1, 8 + 1, - uvsrc_x, uvsrc_y, - s->h_edge_pos >> 1, v_edge_pos >> 1); + s->vdsp.emulated_edge_mc(ubuf, + srcU, + uvlinesize, + uvlinesize, + 9, + v->field_mode ? 17 : 9, + uvsrc_x, + uvsrc_y, + s->h_edge_pos >> 1, + s->v_edge_pos >> 1 + v->field_mode); + s->vdsp.emulated_edge_mc(vbuf, + srcV, + uvlinesize, + uvlinesize, + 9, + v->field_mode ? 17 : 9, + uvsrc_x, + uvsrc_y, + s->h_edge_pos >> 1, + s->v_edge_pos >> 1 + v->field_mode); } srcU = ubuf; srcV = vbuf;