From 443d3a91d335188be15c3bd2616d2cc9c4728566 Mon Sep 17 00:00:00 2001 From: wm4 Date: Sat, 28 Jan 2017 18:27:30 +0100 Subject: vaapi: remove central lock around vaapi API calls The lock was disabled recently. This commit gets rid of the dummied out calls. The main reason for removing it is that there is no apparent need for it anymore, and the new FFmpeg vaapi code does not use or provide such a lock (there are some places which we cannot control and which do vaapi API calls, like frame destructors). --- video/out/vo_vaapi.c | 10 ---------- 1 file changed, 10 deletions(-) (limited to 'video/out/vo_vaapi.c') diff --git a/video/out/vo_vaapi.c b/video/out/vo_vaapi.c index 4aa1ef37df..85b8159f68 100644 --- a/video/out/vo_vaapi.c +++ b/video/out/vo_vaapi.c @@ -205,8 +205,6 @@ static bool render_to_screen(struct priv *p, struct mp_image *mpi) if (surface == VA_INVALID_ID) return false; - va_lock(p->mpvaapi); - for (int n = 0; n < MAX_OSD_PARTS; n++) { struct vaapi_osd_part *part = &p->osd_parts[n]; if (part->active) { @@ -252,8 +250,6 @@ static bool render_to_screen(struct priv *p, struct mp_image *mpi) } } - va_unlock(p->mpvaapi); - return true; } @@ -424,8 +420,6 @@ static void draw_osd(struct vo *vo) if (!p->osd_format.fourcc) return; - va_lock(p->mpvaapi); - struct mp_osd_res vid_res = osd_res_from_image_params(vo->params); struct mp_osd_res *res; @@ -438,8 +432,6 @@ static void draw_osd(struct vo *vo) for (int n = 0; n < MAX_OSD_PARTS; n++) p->osd_parts[n].active = false; osd_draw(vo->osd, *res, pts, 0, osd_formats, draw_osd_cb, p); - - va_unlock(p->mpvaapi); } static int get_displayattribtype(const char *name) @@ -520,9 +512,7 @@ static int set_equalizer(struct priv *p, const char *name, int value) MP_VERBOSE(p, "Changing '%s' (range [%d, %d]) to %d\n", name, attr->max_value, attr->min_value, attr->value); - va_lock(p->mpvaapi); status = vaSetDisplayAttributes(p->display, attr, 1); - va_unlock(p->mpvaapi); if (!CHECK_VA_STATUS(p, "vaSetDisplayAttributes()")) return VO_FALSE; return VO_TRUE; -- cgit v1.2.3