msdk: don't destroy an image twice

This gets rid of annoying message in the log, e.g. run the pipeline
below:

gst-launch-1.0 videotestsrc num-buffers=100 ! \
video/x-raw,format=NV12,width=352,height=288 ! msdkh264enc ! filesink \
location=test.h264

[LIBVA]:CRITICAL - DdiMedia_DestroyImage:4357: Invalid image
This commit is contained in:
Haihao Xiang 2019-01-18 15:14:59 +08:00 committed by Víctor Manuel Jáquez Leal
parent 348a4b570d
commit fffb3f628d

View file

@ -129,8 +129,12 @@ gst_msdk_frame_alloc (mfxHDL pthis, mfxFrameAllocRequest * req,
if (MFX_ERR_NONE != status) { if (MFX_ERR_NONE != status) {
GST_ERROR ("failed to get dmabuf handle"); GST_ERROR ("failed to get dmabuf handle");
vaDestroyImage (gst_msdk_context_get_handle (context), va_status = vaDestroyImage (gst_msdk_context_get_handle (context),
msdk_mids[i].image.image_id); msdk_mids[i].image.image_id);
if (va_status == VA_STATUS_SUCCESS) {
msdk_mids[i].image.image_id = VA_INVALID_ID;
msdk_mids[i].image.buf = VA_INVALID_ID;
}
} }
} else { } else {
/* useful to check the image mapping state later */ /* useful to check the image mapping state later */
@ -209,7 +213,11 @@ gst_msdk_frame_free (mfxHDL pthis, mfxFrameAllocResponse * resp)
if (mem->info.mem_type == VA_SURFACE_ATTRIB_MEM_TYPE_DRM_PRIME) if (mem->info.mem_type == VA_SURFACE_ATTRIB_MEM_TYPE_DRM_PRIME)
vaReleaseBufferHandle (dpy, mem->image.buf); vaReleaseBufferHandle (dpy, mem->image.buf);
vaDestroyImage (dpy, mem->image.image_id); if (mem->image.image_id != VA_INVALID_ID &&
vaDestroyImage (dpy, mem->image.image_id) == VA_STATUS_SUCCESS) {
mem_id->image.image_id = VA_INVALID_ID;
mem_id->image.buf = VA_INVALID_ID;
}
} }
va_status = va_status =
@ -265,6 +273,10 @@ gst_msdk_frame_lock (mfxHDL pthis, mfxMemId mid, mfxFrameData * data)
if (status != MFX_ERR_NONE) { if (status != MFX_ERR_NONE) {
GST_WARNING ("failed to map"); GST_WARNING ("failed to map");
if (vaDestroyImage (dpy, mem_id->image.image_id) == VA_STATUS_SUCCESS) {
mem_id->image.image_id = VA_INVALID_ID;
mem_id->image.buf = VA_INVALID_ID;
}
return status; return status;
} }
@ -331,6 +343,11 @@ gst_msdk_frame_unlock (mfxHDL pthis, mfxMemId mid, mfxFrameData * ptr)
if (mem_id->fourcc != MFX_FOURCC_P8) { if (mem_id->fourcc != MFX_FOURCC_P8) {
vaUnmapBuffer (dpy, mem_id->image.buf); vaUnmapBuffer (dpy, mem_id->image.buf);
va_status = vaDestroyImage (dpy, mem_id->image.image_id); va_status = vaDestroyImage (dpy, mem_id->image.image_id);
if (va_status == VA_STATUS_SUCCESS) {
mem_id->image.image_id = VA_INVALID_ID;
mem_id->image.buf = VA_INVALID_ID;
}
} else { } else {
va_status = vaUnmapBuffer (dpy, *(mem_id->surface)); va_status = vaUnmapBuffer (dpy, *(mem_id->surface));
} }