<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 13, 2017 at 8:32 PM, Andy Furniss <span dir="ltr"><<a href="mailto:adf.lists@gmail.com" target="_blank">adf.lists@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>Nayan Deshmukh wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Andy,<br>
<br>
Please test this patch for regressions.<br>
</blockquote>
<br></span>
Do you have a testcase to show the fix?<br>
<br>
TBH I've not tested gstreamer with mpeg2 before as vaapi mpeg2<br>
h/w dec never worked properly anyway.<br>
<br>
<a href="https://bugs.freedesktop.org/show_bug.cgi?id=93760" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/s<wbr>how_bug.cgi?id=93760</a><br>
<br>
With mpv --hwdec=vaapi it doesn't seem to regress anything.<br>
<br></blockquote><div>I was talking about --hwdec=vaapi. Before this patch I was not able to play</div><div>any mpeg videos with vaapi as mpv --hwdec=vaapi --vo=vaapi always </div><div>segfaulted. With this patch I can see videos properly. Just wanted to </div><div>make sure it did not cause any regression when using hardware decoder. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
With gstreamer - I can display junk and segfault with or without<br>
the patch.<br>
<br>
This is the first time trying though, and I just don't know whether it's<br>
just me messing up demuxing mpeg container to feed vaapi or whether<br>
it's using vaapi sink (normally I only test x264/mkv/mp4/raw encode/transcode).<br>
<br>
If you have a working gstreamer commandline to demux mpegps decode and display<br>
it would be handy :-)<br>
<br>
<br>
More generally - it's really good you are working on vaapi - I don't<br>
know what you've discusses with anyone but did you see the old threads<br>
around VAAPI_DISABLE_INTERLACE?<br>
<br></blockquote><div>I haven't discussed it with anyone but I will try reading the old threads and the </div><div>bug reports.   </div><div><br></div><div>Regards,</div><div>Nayan</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I was meaning to bring this up via bug/finding replying to old mails.<br>
It was only meant to be a short term fix and as time goes on it's<br>
getting more problematic.<br>
<br>
Soon ffmpeg will enable de-interlacer, which depending on env, may lock<br>
peoples GPUs if they paste some example from the wiki.<br>
<br>
mpv --vo=vaapi is also borked = vmfaults probably related to interlaces<br>
vs progressive buffers (though may be more complicated than that as<br>
hwdec + vo does seem OK)<br>
</blockquote></div><br></div></div>