[FFmpeg-devel] [PATCH] libavcodec/qsvenc.c: More correct selection of alignment of a frame height depending whether an encoded sequence progressive or not.

Ivan Uskov ivan.uskov at nablet.com
Sat Jul 4 16:00:39 CEST 2015

Hello Hendrik,

The harm of hard-coded alignment 32 that typical 1280x720 progressive
video should be encoded as 1280x736. I.e. absolutely superfluous
stripe of macroblocks should be encoded.
I can see that CODEC_FLAG_INTERLACED_DCT currently checks into
Why it can not be used by similar way in qsvenc.c?

Saturday, July 4, 2015, 4:34:32 PM, you wrote:

HL> On Sat, Jul 4, 2015 at 2:38 PM, Ivan Uskov <ivan.uskov at nablet.com> wrote:
>> Hello All,
>> There is patch which fixes too strong frame height rounding for case
>> when progressive frame sequence encodes by QSV.

HL> Is there any harm in always using 32 alignment for the height?
HL> Because the flag you are checking there is not correct. In fact, there
HL> is no reliable global interlaced flag.

Best regards,
 Ivan                            mailto:ivan.uskov at nablet.com

More information about the ffmpeg-devel mailing list