From 4462b7be5cbe4f15f861ff9d322154d786f901e0 Mon Sep 17 00:00:00 2001
From: Eladash <elad3356p@gmail.com>
Date: Wed, 23 Aug 2023 08:45:13 +0300
Subject: [PATCH] cellGame: Make cellGameDataCheck slower

* Turns out cellGameBootCheck is actually quite fast.
* cellGameDataCheck is incredibly slow, slower for DISC type.
* Set 0 sizeKB for when RET_NONE is about to be returned.
---
 rpcs3/Emu/Cell/Modules/cellGame.cpp | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/rpcs3/Emu/Cell/Modules/cellGame.cpp b/rpcs3/Emu/Cell/Modules/cellGame.cpp
index 27766431d3..d34539c2f0 100644
--- a/rpcs3/Emu/Cell/Modules/cellGame.cpp
+++ b/rpcs3/Emu/Cell/Modules/cellGame.cpp
@@ -703,7 +703,7 @@ error_code cellGameBootCheck(vm::ptr<u32> type, vm::ptr<u32> attributes, vm::ptr
 
 	auto& perm = g_fxo->get<content_permission>();
 
-	lv2_sleep(5000);
+	lv2_sleep(500);
 
 	const auto init = perm.init.init();
 
@@ -831,8 +831,6 @@ error_code cellGameDataCheck(u32 type, vm::cptr<char> dirName, vm::ptr<CellGameC
 
 	const std::string dir = type == CELL_GAME_GAMETYPE_DISC ? "/dev_bdvd/PS3_GAME"s : "/dev_hdd0/game/" + name;
 
-	lv2_sleep(5000);
-
 	// TODO: not sure what should be checked there
 
 	auto& perm = g_fxo->get<content_permission>();
@@ -841,9 +839,14 @@ error_code cellGameDataCheck(u32 type, vm::cptr<char> dirName, vm::ptr<CellGameC
 
 	if (!init)
 	{
+		lv2_sleep(300);
 		return CELL_GAME_ERROR_BUSY;
 	}
 
+	// This function is incredibly slow, slower for DISC type and even if the game/disc data does not exist
+	// Null size does not change it
+	lv2_sleep(type == CELL_GAME_GAMETYPE_DISC ? 300000 : 120000);
+
 	auto [sfo, psf_error] = psf::load(vfs::get(dir + "/PARAM.SFO"));
 
 	if (const std::string_view cat = psf::get_string(sfo, "CATEGORY"); [&]()
@@ -870,7 +873,7 @@ error_code cellGameDataCheck(u32 type, vm::cptr<char> dirName, vm::ptr<CellGameC
 		size->hddFreeSizeKB = 40 * 1024 * 1024 - 1; // Read explanation in cellHddGameCheck
 
 		// TODO: Calculate data size for game data, if necessary.
-		size->sizeKB = CELL_GAME_SIZEKB_NOTCALC;
+		size->sizeKB = sfo.empty() ? 0 : CELL_GAME_SIZEKB_NOTCALC;
 		size->sysSizeKB = 0; // TODO
 	}