Привет. Подскажите пожалуйста, использую pthread_mutex..., как лучше сделать: сначала записать

Question

Привет. Подскажите пожалуйста, использую pthread_mutex..., как лучше сделать: сначала записать

значения общего ресурса в локальные переменные и потом разлочить или можно оставить так, тоесть блокировать в начале и освобождать в конце?
/* Processing segment of array of URLs*/
for (long i = start; i < end; ++i) {
/* Getting element (url) from json array of urls */
pthread_mutex_lock(&lock);

cJSON* url = cJSON_GetArrayItem(common_fields.json_urls_array, i);
char* current_url = url->valuestring;

/* Making full path of file where it will be downloaded */
extract_absolute_path(download_absolute_path, current_url, common_fields.download_folder);
download_file(common_fields.download_folder, current_url, download_absolute_path);

pthread_mutex_unlock(&lock);

}

#c #embedded #programming #russian

0

09.10.2023

2 ответов

14 просмотров

Ilya Zviagin

Да, странный код. Явно надо pthread_mutex_unlock(&lock); перенести перед extract_absolute_path(download_absolute_path, current_url, common_fields.download_folder); А то получается, что ты делаешь download_file(common_fields.download_folder, current_url, download_absolute_path); строго в одном потоке одновременно (ещё говорят, что потоки полностью сериализуются) Это плохо. Также , есть сомнения, что тут cJSON* url = cJSON_GetArrayItem(common_fields.json_urls_array, i); char* current_url = url->valuestring; данные реально копируются из общих структур в локальные переменные. Локальные переменные получают только ссылки на общие структуры. Если они после этого поменяются, будешь без ног. (будут гонки в программе и скорее всего она крашнется) Так что надо реально копировать данные

0

10.10.2023

Ilya Zviagin · Accepted Answer

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

170 похожих чатов

Привет. Подскажите пожалуйста, использую pthread_mutex..., как лучше сделать: сначала записать

2 ответов

Похожие вопросы