Skip to content

vk: Forgotten improvements #17041

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 13 commits into from
Apr 18, 2025
Merged

vk: Forgotten improvements #17041

merged 13 commits into from
Apr 18, 2025

Conversation

kd-11
Copy link
Contributor

@kd-11 kd-11 commented Apr 14, 2025

This is a collection of minor optimizations and reworks that were done over the past few months. Should result in a minor performance boost in most RSX-limited scenarios when using Vulkan.

@Megamouse Megamouse added the Optimization Optimizes existing code label Apr 14, 2025
@digant73
Copy link
Contributor

it does not compile on windows

@kd-11
Copy link
Contributor Author

kd-11 commented Apr 14, 2025

it does not compile on windows

Indeed. I haven't tested it on windows yet, this code was lost on my linux dev machine.

@kd-11 kd-11 marked this pull request as draft April 14, 2025 12:23
@AniLeo
Copy link
Member

AniLeo commented Apr 14, 2025

19a435e causes a ~10 FPS performance drop in dice
image

Undoing the commit brings back performance
image

@digant73
Copy link
Contributor

does the PR cover #16843 (so possibly it could fix some DEVICE-LOST errors)?
Anyway, I will start to test the PR this night

@digant73
Copy link
Contributor

some testing on FOM on the same place where I get a systematic DEVICE-LOST.
Something is wrong when managing the fault info. Below the error on log file

·F 0:01:57.908452 {RSX [0x1dd2308]} SIG: Thread terminated due to fatal error: Verification failed (object: 0x0)
(in file G:\console\git\rpcs3_kd-11\rpcs3\Emu\RSX\VK\vkutils\shared.cpp:20[:3], in function 'std::basic_string<char,std::char_traits<char>,std::allocator<char> > vk::retrieve_device_fault_info()') (error=0x578)

Attached the log file

RPCS3.log

@digant73
Copy link
Contributor

digant73 commented Apr 17, 2025

Killzone 3 crashed when loading the selected chapter. Same crash for other games (KZ2, R3 etc.)

RPCS3.log

@AniLeo
Copy link
Member

AniLeo commented Apr 18, 2025

Looks good to me

Screenshot_20250418_081350

@kd-11
Copy link
Contributor Author

kd-11 commented Apr 18, 2025

@digant73 Your killzone issue is fixed now?

@digant73
Copy link
Contributor

@digant73 Your killzone issue is fixed now?

yes and also R3 etc. But I will test in deep in few hours

@digant73
Copy link
Contributor

made some testing and I didn't see regressions. DEVICE-LOST issue is still there but it seems that issue was not the target of the PR

@kd-11
Copy link
Contributor Author

kd-11 commented Apr 18, 2025

Thanks for checking. I'll clean this up for merge.

@kd-11 kd-11 marked this pull request as ready for review April 18, 2025 21:14
@kd-11 kd-11 merged commit 564c903 into RPCS3:master Apr 18, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants