From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id 130AB4065E for ; Thu, 27 Jan 2022 08:38:34 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 88DF768B1FC; Thu, 27 Jan 2022 10:38:32 +0200 (EET) Received: from out162-62-57-49.mail.qq.com (out162-62-57-49.mail.qq.com [162.62.57.49]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 217B368AFD1 for ; Thu, 27 Jan 2022 10:38:24 +0200 (EET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=foxmail.com; s=s201512; t=1643272699; bh=v0xKQs9uOysV72UuTY2/l9oErIJs5VPsbqBgB2J4Dho=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=zywJ1g4hFdrqDuy7bq3njgwrge5mkOEKWOS+JTJuHCPYYT2mNTuooyRlRIV31JBhO ycJVpROVfCYjhOf/SdvA19yYdOrCg/9iF+wbpNMvTEuNjWE4WhaWOrBaS1y4gEEBLq GnQeSOLpb3PH/P1p32PeQDW0Do4gKTlkyADQqMhk= Received: from ZHILIZHAO-MB2.tencent.com ([113.108.77.72]) by newxmesmtplogicsvrszc7.qq.com (NewEsmtp) with SMTP id 991B38FA; Thu, 27 Jan 2022 16:38:17 +0800 X-QQ-mid: xmsmtpt1643272697tl05bbkxx Message-ID: X-QQ-XMAILINFO: MIXpHopat2IawWSLgqSD1uQ5TG8D79Ik+BA7cKXmA1GyS6w2Iw1EZ1d55e2vUA t63NY/y2ICAZsZAlgni2N+vYqZpPIUAUH5HH6PMoAUKJSCkD2eFTJBvDKnjCJDqVjRZ0Kpm3AH2O hcXTAmHyhtPao7ocAD0wEM5on4W3AFTO6pZ7TChngUanPDhMz8fnWdSa+tkzjcCM5YYgMteoCNtG hcUKvNqWCiCyEUMXhl2VzsbE9WTK+gMxs+AZxFSVrB5w5AvUi3FCUJlCMXSywmDP+PeBuMg3uYrx cQnH/YwNgd6K3FTcRWWKIleFiEQ9Y0k/mWDP9htJ0cRBK7a80Nj2dkA47DPAiXZuLMWPcie8fCFz vHDtCOqYa8d0JE2osnAvOou5Ybj22d7t3nh4OI28Yo7+M4oEFNfK06alKxNXXSaHRp6lbw9z675p HseaO6AHtEYDohEvOctz3WFHtsR8KQ1QiC4U5aRzg8Kxx77AbFnN8HMWL2cMYCnKc9MNemJbXQDf g1S7anW9ditVjgqjVaISkcxSIHaSFjb2+tbjw3h3gapnIkB2Q3IDDp9Ysn2HiTbre7yOuTl/3FqK dwvKDdMeh4rU+Shsgpx4wW3hq9hv/MVRmu7HjbIoV2FBwBmC0TeaF/wyblEZVtrlw4XIfsRiGFLw YTqUv1tbDOkkW47LoVbM1K5tt28qBr898mchyezFtxz31+LAB/xzuBSSghX6JcwJcfIE5WORdIpf cyDlG0P2OvQc4PBUjF9d66Rj/+SfPrGONLYD+bIZUTsGe5qzDxUnI7nOqB5utEnOORziALRjgQdv FExJa9o3tZCa9mDVEVSLhBSXradA82AIb7HqOM5KJTIxy6bq0q9oSSMUTYderLjIiLhkIhX7rl2L dQeetVe5fZ/GLAJF1i6G/7Iqvkz2FSA0GlNASOJlaVXF4lY3xiyKNpkZ49K0+ibQ== From: Zhao Zhili To: ffmpeg-devel@ffmpeg.org Date: Thu, 27 Jan 2022 16:38:15 +0800 X-OQ-MSGID: <20220127083815.82194-1-quinkblack@foxmail.com> X-Mailer: git-send-email 2.31.1 In-Reply-To: References: MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH v2] hwcontext_vulkan: workaround MoltenVK's bug which leads to segmentation fault X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Cc: Zhao Zhili Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: MoltenVK doesn't reset instance when CreateInstance() failed, then DestroyInstance() leads to segmentation fault. MoltenVK's bug has been fixed by [1], which doesn't available on homebrew yet. Regardless MoltenVK's bug, we shouldn't call DestroyInstance() in the case of CreateInstance() failed, so reset instance making sense. [1] https://github.com/KhronosGroup/MoltenVK/commit/86a1fbdb8 --- libavutil/hwcontext_vulkan.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavutil/hwcontext_vulkan.c b/libavutil/hwcontext_vulkan.c index 2e219511c9..d3bd37a6a7 100644 --- a/libavutil/hwcontext_vulkan.c +++ b/libavutil/hwcontext_vulkan.c @@ -719,6 +719,8 @@ static int create_instance(AVHWDeviceContext *ctx, AVDictionary *opts) if (ret != VK_SUCCESS) { av_log(ctx, AV_LOG_ERROR, "Instance creation failure: %s\n", vk_ret2str(ret)); + /* Workaround MoltenVK's bug which doesn't reset instance. */ + hwctx->inst = VK_NULL_HANDLE; err = AVERROR_EXTERNAL; goto fail; } -- 2.31.1 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".