Bug 111522

Related to bug 110765 ?

Could you try if this fixes your issue?
https://gitlab.freedesktop.org/mesa/mesa/merge_requests/1819

(In reply to Eric Engestrom from comment #2)
> Could you try if this fixes your issue?
> https://gitlab.freedesktop.org/mesa/mesa/merge_requests/1819

Not alone, as that MR enables the workaround only for gfxbench (testfw_app binary name).

What binary name Supraland uses?

The binary is "Supraland-Linux-Shipping"
It may also affect others UE4 games. Haven't tested yet

Alright, you'll need to add this to your ~/.drirc on top of MR !1819:

<driconf>
    <device>
        <application name="Supraland" executable="Supraland-Linux-Shipping">
            <option name="override_min_image_count" value="2" />
        </application>
    </device>
</driconf>

If you could test other UE4 games too that would be great :)

Applying 1819 patch + /.drirc file doesn't have any effects.

Just to confirm, is this the title causing problems? : https://store.steampowered.com/app/813630/Supraland/

Eric, can you make a similar fix to what you did for minimagecount dri config?

Yes Lionel

I've put up another MR : https://gitlab.freedesktop.org/mesa/mesa/merge_requests/1883

Here is the ~/.drirc I was using :

<driconf>
   <device>
      <application name="Supraland" executable="Supraland-Linux-Shipping">
         <option name="vk_x11_override_min_image_count" value="2" />
         <option name="vk_x11_strict_image_count" value="true" />
      </application>
   </device>
</driconf>

If you could test this that would be great.


Note that for me with this fix, the game crashes at start with the following backtrace :

Program terminated with signal SIGSEGV, Segmentation fault.
#0  0x00007f051ec464e5 in ralloc_parent (ptr=0x7f04fd83d760) at ../src/util/ralloc.c:356
356        return info->parent ? PTR_FROM_HEADER(info->parent) : NULL;
[Current thread is 1 (Thread 0x7f0505afb700 (LWP 15827))]
(gdb) bt
#0  0x00007f051ec464e5 in ralloc_parent (ptr=0x7f04fd83d760) at ../src/util/ralloc.c:356
#1  0x00007f051ec464b8 in ralloc_parent (ptr=0x7f04fc891a00) at ../src/util/ralloc.c:353
#2  0x00007f051ec464b8 in ralloc_parent (ptr=0x7f04fce65fe0) at ../src/util/ralloc.c:353
#3  0x00007f051ec463d9 in ralloc_adopt (new_ctx=0x7f051ec463d9 <ralloc_adopt+48>, old_ctx=0x7f0505aebde0) at ../src/util/ralloc.c:326
#4  0x00007f051e8ff6b3 in anv_pipeline_compile_graphics (pipeline=0x7f04fd443410, cache=0x8d20800, info=0x7f0505af9c68) at ../src/intel/vulkan/anv_pipeline.c:1448
#5  0x00007f051e900b5f in anv_pipeline_init (pipeline=0x7f04fd443410, device=0x8d208f0, cache=0x8d20800, pCreateInfo=0x7f0505af9c68, alloc=0x8d208f8) at ../src/intel/vulkan/anv_pipeline.c:1930
#6  0x00007f051e9f4793 in gen9_graphics_pipeline_create (_device=0x8d208f0, cache=0x8d20800, pCreateInfo=0x7f0505af9c68, pAllocator=0x0, pPipeline=0x7f04a4b01c10)
    at ../src/intel/vulkan/genX_pipeline.c:2135
#7  0x00007f051e9f5200 in VALGRIND_PRINTF (format=0x7f051e9e82aa <_anv_combine_address+105> "H\215\065\357\t?") at /usr/include/valgrind/valgrind.h:6248
#8  0x00007f051e27d20c in vkCreateGraphicsPipelines (device=0x8d208f0, pipelineCache=0x8d20800, createInfoCount=1, pCreateInfos=0x7f0505af9c68, pAllocator=0x0, pPipelines=0x7f04a4b01c10)
    at layersvt/api_dump.cpp:8318
#9  0x00007f051dcd7078 in ?? () from /home/djdeath/.steam/ubuntu12_64/libVkLayer_steam_fossilize.so
#10 0x00007f0528057c94 in vkCreateGraphicsPipelines (device=0x8d208f0, pipelineCache=0x8d20800, createInfoCount=1, pCreateInfos=0x7f0505af9c68, pAllocator=0x0, pPipelines=0x7f04a4b01c10)
    at ../loader/trampoline.c:1275
#11 0x000000000467195d in FVulkanPipelineStateCacheManager::CreateGfxPipelineFromEntry(FVulkanPipelineStateCacheManager::FGfxPipelineEntry*, FVulkanShader**, FVulkanGfxPipeline*) ()
#12 0x0000000004670ee1 in FVulkanPipelineStateCacheManager::CreateAndAdd(FGraphicsPipelineStateInitializer const&, FGfxPSIKey, TSharedPtr<FVulkanPipelineStateCacheManager::FGfxPipelineEntry, (ESPMode)1>, FGfxEntryKey) ()
#13 0x0000000004674dcb in FVulkanDynamicRHI::RHICreateGraphicsPipelineState(FGraphicsPipelineStateInitializer const&) ()
#14 0x000000000472a1b6 in PipelineStateCache::GetAndOrCreateGraphicsPipelineState(FRHICommandList&, FGraphicsPipelineStateInitializer const&, EApplyRendertargetOption) ()
#15 0x0000000004729e3f in SetGraphicsPipelineState(FRHICommandList&, FGraphicsPipelineStateInitializer const&, EApplyRendertargetOption) ()
#16 0x00000000042c3f86 in FRCPassPostProcessCombineLUTs::Process(FRenderingCompositePassContext&) ()
#17 0x00000000043820d8 in FRenderingCompositionGraph::RecursivelyProcess(FRenderingCompositeOutputRef const&, FRenderingCompositePassContext&) const ()
#18 0x0000000004381d94 in FRenderingCompositePassContext::Process(TArray<FRenderingCompositePass*, FDefaultAllocator> const&, char16_t const*) ()
#19 0x00000000042d43f1 in FPostProcessing::Process(FRHICommandListImmediate&, FViewInfo const&, TRefCountPtr<IPooledRenderTarget>&) ()
#20 0x000000000415007b in FDeferredShadingSceneRenderer::Render(FRHICommandListImmediate&) ()
#21 0x000000000446c7c6 in ?? ()
#22 0x00000000044772ba in ?? ()
#23 0x000000000391c18f in FNamedTaskThread::ProcessTasksNamedThread(int, bool) ()
#24 0x000000000391bdf3 in FNamedTaskThread::ProcessTasksUntilQuit(int) ()
#25 0x000000000475e0b2 in FRenderingThread::Run() ()
#26 0x0000000003954b03 in FRunnableThreadPThread::Run() ()
#27 0x0000000003946aad in FRunnableThreadPThread::_ThreadProc(void*) ()
#28 0x00007f054e77d182 in start_thread (arg=<optimised out>) at pthread_create.c:486
#29 0x00007f054dd88b1f in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:95


Though running the game under valgrind shows incorrect free() from the application so I believe the above backtrace is the result of a previous memory corruption.

Thanks!

I can confirm that the last MR + .drirc solves this issue, at least on radv. Anv may have additional issues.

May be unrelated but I have a lot of
"SPIR-V WARNING:
    In file ../src/compiler/spirv/spirv_to_nir.c:826
    Decoration not allowed on struct members: SpvDecorationInvariant
" 
in the console

I just compiled the Scifi Hallway demo with Unreal Engine 4.22. I have the same issue so it probably affect all UE4 applications using vulkan.

This time, I have some info from the log:
"Assertion failed: Images.Num() == NUM_BUFFERS [File:D:\Build\++UE4\Sync\Engine\Source\Runtime\VulkanRHI\Private\VulkanViewport.cpp] [Line: 610] 
Actual Num: 5"

I'll attach the full log.

With UE4.22+, using "-vulkanpresentmode=0" also solves the crash. (Does not work with Supraland, as it's a UE4.21 game)

Created attachment 145284 [details]
Full log from UE4.22 Scifi Hallway demo

I've got an e-mail thread going with some people at Epic.  They're going to be looking into fixing the issue in UE4.  Until then, driver workarounds will be needed. :-(

Can these workarounds be applied to all applications using this engine with information in VkApplicationInfo->pEngineName?

(In reply to MWATTT from comment #15)
> Can these workarounds be applied to all applications using this engine with
> information in VkApplicationInfo->pEngineName?

Yes, will look into that.

(In reply to Lionel Landwerlin from comment #16)
> (In reply to MWATTT from comment #15)
> > Can these workarounds be applied to all applications using this engine with
> > information in VkApplicationInfo->pEngineName?
> 
> Yes, will look into that.

Done, updated the MR.

Supraland does start with the updated MR. (I suppose .drirc is only updated if using ninja install so I manually put content from 00-mesa-defaults.conf to .drirc).

Note that will only affect UE4.21 applications. For instance, my previous UE4.22 sample have the same issue, unless I modify .drirc with "UnrealEngine4.22"

A workaround is to set
<engine engine_name="UnrealEngine4" engine_version="0:23">
on .drirc and use strstr instead of strcmp on xmlconfig.c at line 793, if it's acceptable

(In reply to MWATTT from comment #18)
> Supraland does start with the updated MR. (I suppose .drirc is only updated
> if using ninja install so I manually put content from 00-mesa-defaults.conf
> to .drirc).
> 
> Note that will only affect UE4.21 applications. For instance, my previous
> UE4.22 sample have the same issue, unless I modify .drirc with
> "UnrealEngine4.22"
> 
> A workaround is to set
> <engine engine_name="UnrealEngine4" engine_version="0:23">
> on .drirc and use strstr instead of strcmp on xmlconfig.c at line 793, if
> it's acceptable

I've been wondering about regexps but that's another thing to add to mesa.
It's also unfortunate that Unreal puts its engien version in the engine name.

(In reply to Lionel Landwerlin from comment #19)
> (In reply to MWATTT from comment #18)
> > Supraland does start with the updated MR. (I suppose .drirc is only updated
> > if using ninja install so I manually put content from 00-mesa-defaults.conf
> > to .drirc).
> > 
> > Note that will only affect UE4.21 applications. For instance, my previous
> > UE4.22 sample have the same issue, unless I modify .drirc with
> > "UnrealEngine4.22"
> > 
> > A workaround is to set
> > <engine engine_name="UnrealEngine4" engine_version="0:23">
> > on .drirc and use strstr instead of strcmp on xmlconfig.c at line 793, if
> > it's acceptable
> 
> I've been wondering about regexps but that's another thing to add to mesa.
> It's also unfortunate that Unreal puts its engien version in the engine name.

The filename matching code is already full of platform specific paths so adding regexps here would be fairly simple as we can just use the posix functions.

(In reply to Timothy Arceri from comment #20)
> (In reply to Lionel Landwerlin from comment #19)
> > I've been wondering about regexps but that's another thing to add to mesa.
> > It's also unfortunate that Unreal puts its engien version in the engine name.
> 
> The filename matching code is already full of platform specific paths so
> adding regexps here would be fairly simple as we can just use the posix
> functions.

Wouldn't globbing (fnmatch()) be sufficient and easier to end users to understand than regexps?

Your last MR works fine for me. 
Note that the last Unreal Engine version is 4.23 and is still affected by this bug, so I'll be better with "engine_version="0:23" in .drirc

Should fixed on master with :

commit 0616b7ac90cf4f86bb409d34101e3a3cceac8cbe
Author: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Date:   Thu Sep 5 23:54:53 2019 +0300

    vulkan: add vk_x11_strict_image_count option

(In reply to MWATTT from comment #22)
> Note that the last Unreal Engine version is 4.23 and is still affected by
> this bug, so I'll be better with "engine_version="0:23" in .drirc

Lionel, you put the workaround up to version 4.21 in 0616b7ac90cf4f86bb409d34101e3a3cceac8cbe; did you mean to put 4.23?

If so, you can push this with my r-b:
---8<---
--- a/src/util/00-mesa-defaults.conf
+++ b/src/util/00-mesa-defaults.conf
@@ -475,7 +475,7 @@ TODO: document the other workarounds.
         <!-- Works around the game not starting (does not deal with
              the implementation returning more images than the minimum
              specified by the application. -->
-        <engine engine_name_match="UnrealEngine4.*" engine_versions="0:21">
+        <engine engine_name_match="UnrealEngine4.*" engine_versions="0:23">
             <option name="vk_x11_strict_image_count" value="true" />
         </engine>
     </device>
--->8---

(In reply to Eric Engestrom from comment #24)
> (In reply to MWATTT from comment #22)
> > Note that the last Unreal Engine version is 4.23 and is still affected by
> > this bug, so I'll be better with "engine_version="0:23" in .drirc
> 
> Lionel, you put the workaround up to version 4.21 in
> 0616b7ac90cf4f86bb409d34101e3a3cceac8cbe; did you mean to put 4.23?

Yes, because of the comment in https://bugs.freedesktop.org/show_bug.cgi?id=111522#c22

> 
> If so, you can push this with my r-b:
> ---8<---
> --- a/src/util/00-mesa-defaults.conf
> +++ b/src/util/00-mesa-defaults.conf
> @@ -475,7 +475,7 @@ TODO: document the other workarounds.
>          <!-- Works around the game not starting (does not deal with
>               the implementation returning more images than the minimum
>               specified by the application. -->
> -        <engine engine_name_match="UnrealEngine4.*" engine_versions="0:21">
> +        <engine engine_name_match="UnrealEngine4.*" engine_versions="0:23">
>              <option name="vk_x11_strict_image_count" value="true" />
>          </engine>
>      </device>
> --->8---