From dead206873c1840a21adf789b9e4b5d167012b19 Mon Sep 17 00:00:00 2001 From: Niklas Haas Date: Sun, 16 Jul 2017 17:17:39 +0200 Subject: vo_opengl: use glBufferSubData instead of glMapBufferRange Performance seems pretty much unchanged but I no longer get nasty spikes on NUMA systems, probably because glBufferSubData runs in the driver or something. As a simplification of the code, we also just size the PBO to always have the full size, even for cropped textures. This seems slower but not by relevant amounts, and only affects e.g. --vf=crop. It also slightly increases VRAM usage for textures with big strides. This new code path is especially nice because it no longer depends on GL_ARB_map_buffer_range, and no longer uses any functions that can possibly fail, thus simplifying control flow and seemingly deprecating the manpage's claim about possible image corruption. In theory we could also reduce NUM_PBO_BUFFERS since it doesn't seem like we're streaming uploads anyway, but leave it in there just in case some drivers disagree... --- DOCS/man/options.rst | 3 --- 1 file changed, 3 deletions(-) (limited to 'DOCS/man/options.rst') diff --git a/DOCS/man/options.rst b/DOCS/man/options.rst index 07f961af83..5c25794588 100644 --- a/DOCS/man/options.rst +++ b/DOCS/man/options.rst @@ -4136,9 +4136,6 @@ The following video options are currently all specific to ``--vo=opengl`` and source video size is huge (e.g. so called "4K" video). On other drivers it might be slower or cause latency issues. - In theory, this can sometimes lead to sporadic and temporary image - corruption (because reupload is not retried when it fails). - ``--dither-depth=`` Set dither target depth to N. Default: no. -- cgit v1.2.3