ivampiresp/Spdk
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

memory: lock global mem_map only when necessary

Browse Source 
Minor cleanup.

Change-Id: I9554163ae22836b50b954ec27ed27bcb848cb193
Signed-off-by: Darek Stojaczyk <dariusz.stojaczyk@intel.com>
Reviewed-on: https://review.gerrithub.io/428883
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>
Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
This commit is contained in:
Darek Stojaczyk 2018-10-11 09:49:41 +02:00 committed by Jim Harris
parent 438bae7966
commit a9e484a660
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lib/env_dpdk/memory.c
View File

@ -258,9 +258,8 @@ spdk_mem_map_alloc(uint64_t default_translation, const struct spdk_mem_map_ops *
map->ops = *ops; map->ops = *ops;
} }
pthread_mutex_lock(&g_spdk_mem_map_mutex);
if (ops && ops->notify_cb) { if (ops && ops->notify_cb) {
pthread_mutex_lock(&g_spdk_mem_map_mutex);
rc = spdk_mem_map_notify_walk(map, SPDK_MEM_MAP_NOTIFY_REGISTER); rc = spdk_mem_map_notify_walk(map, SPDK_MEM_MAP_NOTIFY_REGISTER);
if (rc != 0) { if (rc != 0) {
pthread_mutex_unlock(&g_spdk_mem_map_mutex); pthread_mutex_unlock(&g_spdk_mem_map_mutex);
@ -270,9 +269,8 @@ spdk_mem_map_alloc(uint64_t default_translation, const struct spdk_mem_map_ops *
return NULL; return NULL;
} }
TAILQ_INSERT_TAIL(&g_spdk_mem_maps, map, tailq); TAILQ_INSERT_TAIL(&g_spdk_mem_maps, map, tailq);
}
pthread_mutex_unlock(&g_spdk_mem_map_mutex); pthread_mutex_unlock(&g_spdk_mem_map_mutex);
}
return map; return map;
} }