Theard - Cpu usage :)

[coNsept]

Доброго времени суток уважаемые форумчане, у меня возникла маленькая проблемка, а проблемка заключается в следующем, при использование потока, нагрузка на процессор увеличивается с 20% до 80%, что не есть очень хорошо, вот собственно код.
Собственно как эту нагрузку уменьшить, как этот поток остановить после того как я проверю данные из файла? Пробовал и Sleep'ом и как только не извращался, но почему-то не помогло

#include "..."

DWORD WINAPI Timer(LPVOID lpParam)
{	
    Sleep(3000); 
    QuestSystem.d_LOAD("..\\Settings\\QuestSystem.ini");
    return 1;	
} 

extern "C" _declspec(dllexport) void Initialize()
{
    DWORD OldProtect;
	
    if(VirtualProtect(LPVOID(0x401000), 3543191, PAGE_EXECUTE_READWRITE, &OldProtect))
    {
        QuestSystem.Initialize();
                
	CloseHandle(CreateThread(NULL, 0, Timer, NULL, 0, NULL));

	_asm
	{
	    Mov DWord Ptr Ds:[00600060H], Offset ProtocolServer
	    Mov DWord Ptr Ds:[00600064H], Offset ProtocolClient
	}

	HookThis((DWORD)&ProtocolCore, 0x00403314);
    }
    else
    {
	MessageBoxA(NULL, "Could not begin the dll", "MyDLL.dll", MB_OK);
    }
}

[coNsept]

Проблему исправил, топик можно удалить, либо закрыть. (Может кому и пригодится)

#include "..."

void UpdateQuestListThread()
{
    while ( TRUE )
    {
	Sleep(60000);
	QuestSystem.d_LOAD("..\\Settings\\QuestSystem.ini"); 
    }
}

extern "C" _declspec(dllexport) void Initialize()
{
    DWORD OldProtect;
	
    if(VirtualProtect(LPVOID(0x401000), 3543191, PAGE_EXECUTE_READWRITE, &OldProtect))
    {
        QuestSystem.Initialize();
            
	CreateThread(NULL, NULL, LPTHREAD_START_ROUTINE(UpdateQuestListThread), NULL, 0, 0);

	_asm
	{
	    Mov DWord Ptr Ds:[00600060H], Offset ProtocolServer
	    Mov DWord Ptr Ds:[00600064H], Offset ProtocolClient
	}

	HookThis((DWORD)&ProtocolCore, 0x00403314);
    }
    else
    {
	MessageBoxA(NULL, "Could not begin the dll", "MyDLL.dll", MB_OK);
    }
}

[netrino]

Кстати, если уж на то пошло, то передаваемая в CreateThread функция должна иметь тип, как в первом примере. Ибо могут быть ошибки во время выполнения на 64-х битных виндах.

DWORD WINAPI UpdateQuestListThread(LPVOID)

[coNsept]

Ага, так уже и сделал, и поток послал спать на веке веков :D (пасябкин)
А мой второй пример, кстати тоже не правильный, так как поток ожидал всего 1 минуту и далее уходил в процедуру, опять же нагружая процесор... Новый вариант, все работает как часы

void SleepThread(void * lpParam)
{
	while(true)
	{
		Sleep(60000);
	}
	_endthread();
}

DWORD WINAPI Timer(LPVOID lpParam)
{	
	Sleep(4100);
	//ItemDropRate(0, 0, "..\\Settings\\ItemDropRate.txt");
	QuestSystem.Load("..\\Settings\\QuestSystem.ini");
	_beginthread(SleepThread, 0, NULL );
	return 1;
}

extern "C" _declspec(dllexport) void Initialize()
{
	DWORD OldProtect;

	if(VirtualProtect(LPVOID(0x401000), 3543191, PAGE_EXECUTE_READWRITE, &OldProtect))
	{
	
		QuestSystem.Initialize();
		//ItemDropRate(0, 0, "..\\Settings\\ItemDropRate.txt");

		CloseHandle(CreateThread(NULL, NULL, LPTHREAD_START_ROUTINE(Timer), NULL, 0, 0));

		_asm
		{
			Mov Dword Ptr Ds:[00600060H],Offset ProtocolServer
			Mov Dword Ptr Ds:[00600064H],Offset ProtocolClient
		}

		HookThis((DWORD)&ProtocolCore, 0x00403314);
		HookThis((DWORD)&ProtocolCoreSend,0x0040533F);
		HookThis((DWORD)&MonsterDropCall, 0x00404E4E);
	}
	else
	{ MessageBoxA(NULL, "Could not begin the dll", "MyDLL.dll", MB_OK); }
}

[netrino]

Ну, чтобы усыпить поток, не обязательно цикл делать, можно просто Sleep (INFINITE);
А вообще зачем это вам? Вы запускаете отдельный поток только для того чтобы он спал, в этом есть резон?

[Teksa]

если честно не понял зачем "спящий красавец" рожается.

как этот поток остановить после того как я проверю данные из файла

TerminateThread()

[coNsept]

Данные функций в функции Timer обновляются по изменению значений установленных в определенном файле, поэтому чтобы не перезапускать программу, что не очень желательно, данные должны обновляться, как бы в режиме реального времени, после чего я отправляю этот поток спать дабы не грузить процессор. Вот и весь компот

Ну, чтобы усыпить поток, не обязательно цикл делать, можно просто Sleep (INFINITE);

Спасябки

[netrino]

Сообщение от coNsept

Данные функций в функции Timer обновляются по изменению значений установленных в определенном файле, поэтому чтобы не перезапускать программу, что не очень желательно, данные должны обновляться, как бы в режиме реального времени, после чего я отправляю этот поток спать дабы не грузить процессор. Вот и весь компот


Спасябки

Но зачем отправлять его спать, если по выполнению всех действия и выходу из callback-функции поток завершается?..
И как оно обновляет в реальном времени, если поток спит? Или вы каждый раз запускаете поток из основного? В таком случае тем более нельзя отправлять поток спать не давая ему завершится, иначе, со временем, программа может превысить лимит потоков.

[Teksa]

после Sleep(дофига); поток уже ничего не будет делать...

void SleepThread(void * lpParam)
{

	while(true) //этот самый слип(дофига) отсюда поток уже не выйдет и ничего полезного не сделает..
	{
		Sleep(60000);
	}
	_endthread();
}

так и не понял зачем СлипФриад он стартует в процедуре таймер, входит в бесконечный цикл в котором отдает процессор на интервал от 60000 и так while (true).

в результате в функции Timer также запускающейся в отдельном потоке мы ждем 4100 мс, грузим файлик и начинаем новый бесконечный поток... при этом поток Timer умирает с криком "1"...

если цель - точное измерение времени то мне тут хорошую вещь подсказали: timeSetEvent(); позволяет устанавливать точность.

зы, а всетаки какие данные изменяет спящий поток, он ж даже при инициализации ничего не делает, только изредка захватывает проц?

[coNsept]

Или вы каждый раз запускаете поток из основного?

Да, именно так, а на счет лимита я небыл в курсе...

Вот что получилось

// ----

void SleepThread(void * lpParam)
{
	TerminateThread(SleepThread, 1); 
        // ----
        Sleep(INFINITE);
}

DWORD WINAPI Timer(LPVOID lpParam)
{	
	Sleep(4100);
	//ItemDropRate(0, 0, "..\\Settings\\ItemDropRate.txt");
	g_Quest.LoadFile(QUESTPATHTXT);
	_beginthread(SleepThread, 0, NULL );
	return 1;
}

// ----

Вот что я еще вычитал

TerminateProcess/TerminateThread - это самый крайний случай. Т.к. здесь есть шанс не только угробить открытые файлы, вызвать утечку памаяти и пр., но даже повесить систему.

п.с. - А так ребята, всем большое спасибо за помощь