Skip to content

Commit

Permalink
avfilter/vf_yadif_cuda: Remove unnecessary stream synchronisation
Browse files Browse the repository at this point in the history
I put this call in by habit, rather than because there was any
actual need. The filter is simply processing frames one after
the other and has no need to synchronise.

malakudi on the devtalk forums noticed a slowdown when using nvenc
with temporal/spatial aq and that the slowdown went away if the
sync call was removed. I also verified that in the basic encoding
case there's an observable speedup.

I also verified that we aren't doing unnecessary sync calls in any
other filter.
  • Loading branch information
philipl committed Mar 30, 2019
1 parent 5282cba commit c0b6e4c
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions libavfilter/vf_yadif_cuda.c
Expand Up @@ -180,8 +180,6 @@ static void filter(AVFilterContext *ctx, AVFrame *dst,
parity, tff);
}

CHECK_CU(cu->cuStreamSynchronize(s->stream));

exit:
CHECK_CU(cu->cuCtxPopCurrent(&dummy));
return;
Expand Down

0 comments on commit c0b6e4c

Please sign in to comment.