diff options
author | wm4 <wm4@nowhere> | 2016-05-09 19:42:03 +0200 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2016-05-09 20:03:22 +0200 |
commit | 46fff8d31af0b79fe3de4aaee93bb66c248118a0 (patch) | |
tree | efe5b04a4e9daf89195a4c758fe93ace9bee385f /video/out/opengl/hwdec.h | |
parent | ee4c00698f7e4b04579494f262e668840c2668b3 (diff) | |
download | mpv-46fff8d31af0b79fe3de4aaee93bb66c248118a0.tar.bz2 mpv-46fff8d31af0b79fe3de4aaee93bb66c248118a0.tar.xz |
video: refactor how VO exports hwdec device handles
The main change is with video/hwdec.h. mp_hwdec_info is made opaque (and
renamed to mp_hwdec_devices). Its accessors are mainly thread-safe (or
documented where not), which makes the whole thing saner and cleaner. In
particular, thread-safety rules become less subtle and more obvious.
The new internal API makes it easier to support multiple OpenGL interop
backends. (Although this is not done yet, and it's not clear whether it
ever will.)
This also removes all the API-specific fields from mp_hwdec_ctx and
replaces them with a "ctx" field. For d3d in particular, we drop the
mp_d3d_ctx struct completely, and pass the interfaces directly.
Remove the emulation checks from vaapi.c and vdpau.c; they are
pointless, and the checks that matter are done on the VO layer.
The d3d hardware decoders might slightly change behavior: dxva2-copy
will not use the VO device anymore if the VO supports proper interop.
This pretty much assumes that any in such cases the VO will not use any
form of exclusive mode, which makes using the VO device in copy mode
unnecessary.
This is a big refactor. Some things may be untested and could be broken.
Diffstat (limited to 'video/out/opengl/hwdec.h')
-rw-r--r-- | video/out/opengl/hwdec.h | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/video/out/opengl/hwdec.h b/video/out/opengl/hwdec.h index a9d524a1d7..fcc6d3c11e 100644 --- a/video/out/opengl/hwdec.h +++ b/video/out/opengl/hwdec.h @@ -4,14 +4,12 @@ #include "common.h" #include "video/hwdec.h" -struct mp_hwdec_info; - struct gl_hwdec { const struct gl_hwdec_driver *driver; struct mp_log *log; struct mpv_global *global; GL *gl; - struct mp_hwdec_ctx *hwctx; + struct mp_hwdec_devices *devs; // For free use by hwdec driver void *priv; // For working around the vdpau vs. vaapi mess. @@ -33,7 +31,7 @@ struct gl_hwdec_driver { enum hwdec_type api; // The hardware surface IMGFMT_ that must be passed to map_image later. int imgfmt; - // Create the hwdec device. It must fill in hw->info, if applicable. + // Create the hwdec device. It must add it to hw->devs, if applicable. // This also must set hw->converted_imgfmt. int (*create)(struct gl_hwdec *hw); // Prepare for rendering video. (E.g. create textures.) @@ -49,9 +47,9 @@ struct gl_hwdec_driver { }; struct gl_hwdec *gl_hwdec_load_api(struct mp_log *log, GL *gl, - struct mpv_global *g, const char *api_name); -struct gl_hwdec *gl_hwdec_load_api_id(struct mp_log *log, GL *gl, - struct mpv_global *g, int id); + struct mpv_global *g, + struct mp_hwdec_devices *devs, + enum hwdec_type api); void gl_hwdec_uninit(struct gl_hwdec *hwdec); |