Skip to content

Commit caa4771

Browse files
Remove support for static gmmlib
Signed-off-by: Mateusz Jablonski <mateusz.jablonski@intel.com> #91 Change-Id: I90184d7aabf3dd0d8a74ff40115213241b4ca724
1 parent c75dc23 commit caa4771

File tree

8 files changed

+32
-87
lines changed

8 files changed

+32
-87
lines changed

CMakeLists.txt

Lines changed: 3 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -240,8 +240,8 @@ endif()
240240

241241
# GmmLib detection
242242
if(NOT GMMUMD_LIB_NAME)
243-
if(TARGET gmm_umd)
244-
set(GMMUMD_LIB_NAME "gmm_umd")
243+
if(TARGET igfx_gmmumd_dll)
244+
set(GMMUMD_LIB_NAME "igfx_gmmumd_dll")
245245
else()
246246
if(DEFINED GMM_SOURCE_DIR)
247247
get_filename_component(GMM_SOURCE_DIR "${GMM_SOURCE_DIR}" ABSOLUTE)
@@ -269,7 +269,6 @@ if(NOT GMMUMD_LIB_NAME)
269269
set(IGDRCL__GMM_LIBRARY_PATH "${IGDRCL__GMM_LIBDIR}")
270270
endif()
271271

272-
set(USE_STATIC_GMM FALSE)
273272
set(GMMUMD_LIB_NAME "igfx_gmmumd_dll")
274273

275274
set(IGDRCL__GMM_INCLUDE_DIR ${IGDRCL__GMM_INCLUDE_DIRS})
@@ -288,29 +287,19 @@ if(NOT GMMUMD_LIB_NAME)
288287
set(GMMUMD_LIB_NAME "igfx_gmmumd_dll")
289288
endif()
290289

291-
get_target_property(target_type ${GMMUMD_LIB_NAME} TYPE)
292-
293-
if(target_type STREQUAL "STATIC_LIBRARY")
294-
set(USE_STATIC_GMM TRUE)
295-
endif()
296290
set(UMKM_SHAREDDATA_INCLUDE_PATHS $<TARGET_PROPERTY:${GMMUMD_LIB_NAME},INTERFACE_INCLUDE_DIRECTORIES>)
297291
else()
298292
message(FATAL_ERROR "GmmLib not found!")
299293
endif()
300294
endif()
301295
else()
302-
get_target_property(target_type ${GMMUMD_LIB_NAME} TYPE)
303-
304-
if(target_type STREQUAL "STATIC_LIBRARY")
305-
set(USE_STATIC_GMM TRUE)
306-
endif()
307296
set(UMKM_SHAREDDATA_INCLUDE_PATHS $<TARGET_PROPERTY:${GMMUMD_LIB_NAME},INTERFACE_INCLUDE_DIRECTORIES>)
308297
endif()
309298

310299
set(GMM_LIBRARY_NAME "igdgmm${NEO_BITS}")
311300

312301
macro(copy_gmm_dll_for target)
313-
if(NOT USE_STATIC_GMM AND NOT IGDRCL__GMM_FOUND)
302+
if(NOT IGDRCL__GMM_FOUND)
314303
add_dependencies(${target} ${GMMUMD_LIB_NAME})
315304
add_custom_command(
316305
TARGET ${target}

runtime/CMakeLists.txt

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -82,15 +82,7 @@ else()
8282
)
8383
endif()
8484

85-
target_compile_definitions(${NEO_STATIC_LIB_NAME} PUBLIC DEFAULT_PLATFORM=${DEFAULT_SUPPORTED_PLATFORM})
86-
87-
link_directories(${GMM_LIB_PATHS})
88-
89-
if(USE_STATIC_GMM)
90-
target_link_libraries(${NEO_STATIC_LIB_NAME} ${GMMUMD_LIB_NAME})
91-
else()
92-
target_compile_definitions(${NEO_STATIC_LIB_NAME} PUBLIC GMM_LIB_DLL)
93-
endif()
85+
target_compile_definitions(${NEO_STATIC_LIB_NAME} PUBLIC GMM_LIB_DLL DEFAULT_PLATFORM=${DEFAULT_SUPPORTED_PLATFORM})
9486

9587
if(INSTRUMENTATION_LIB_NAME)
9688
add_dependencies(${NEO_STATIC_LIB_NAME} ${INSTRUMENTATION_LIB_NAME})

runtime/gmm_helper/CMakeLists.txt

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,6 @@
44
# SPDX-License-Identifier: MIT
55
#
66

7-
if(UNIX AND USE_STATIC_GMM)
8-
set(GMM_INTERFACE_FILE ${CMAKE_CURRENT_SOURCE_DIR}/gmm_interface_static.cpp)
9-
else()
10-
set(GMM_INTERFACE_FILE ${CMAKE_CURRENT_SOURCE_DIR}/gmm_interface_dynamic.cpp)
11-
endif()
12-
137
set(RUNTIME_SRCS_GMM_HELPER_BASE
148
${CMAKE_CURRENT_SOURCE_DIR}/CMakeLists.txt
159
${CMAKE_CURRENT_SOURCE_DIR}/gmm.cpp
@@ -18,9 +12,9 @@ set(RUNTIME_SRCS_GMM_HELPER_BASE
1812
${CMAKE_CURRENT_SOURCE_DIR}/client_context/gmm_client_context_base.h
1913
${CMAKE_CURRENT_SOURCE_DIR}/client_context${BRANCH_DIR_SUFFIX}/gmm_client_context.cpp
2014
${CMAKE_CURRENT_SOURCE_DIR}/client_context${BRANCH_DIR_SUFFIX}/gmm_client_context.h
21-
${GMM_INTERFACE_FILE}
2215
${CMAKE_CURRENT_SOURCE_DIR}/gmm_helper.cpp
2316
${CMAKE_CURRENT_SOURCE_DIR}/gmm_helper.h
17+
${CMAKE_CURRENT_SOURCE_DIR}/gmm_interface.cpp
2418
${CMAKE_CURRENT_SOURCE_DIR}/gmm_lib.h
2519
${CMAKE_CURRENT_SOURCE_DIR}/resource_info.h
2620
${CMAKE_CURRENT_SOURCE_DIR}/resource_info_impl.cpp
File renamed without changes.

runtime/gmm_helper/gmm_interface_static.cpp

Lines changed: 0 additions & 21 deletions
This file was deleted.

unit_tests/gmm_helper/CMakeLists.txt

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,7 @@
77
set(IGDRCL_SRCS_tests_gmm_helper
88
${CMAKE_CURRENT_SOURCE_DIR}/CMakeLists.txt
99
${CMAKE_CURRENT_SOURCE_DIR}/gmm_helper_tests.cpp
10+
${CMAKE_CURRENT_SOURCE_DIR}/gmm_interface_tests.cpp
1011
)
11-
if(NOT USE_STATIC_GMM)
12-
list(APPEND IGDRCL_SRCS_tests_gmm_helper ${CMAKE_CURRENT_SOURCE_DIR}/gmm_interface_tests.cpp)
13-
endif()
1412
target_sources(igdrcl_tests PRIVATE ${IGDRCL_SRCS_tests_gmm_helper})
1513
add_subdirectories()

unit_tests/mock_gmm/CMakeLists.txt

Lines changed: 25 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -4,35 +4,32 @@
44
# SPDX-License-Identifier: MIT
55
#
66

7-
if(NOT USE_STATIC_GMM)
8-
set(target_name mock_gmm)
9-
project(${target_name})
7+
set(target_name mock_gmm)
8+
project(${target_name})
109

11-
# Setting up our local list of test files
12-
set(IGDRCL_SRCS_tests_mock_gmm
13-
${CMAKE_CURRENT_SOURCE_DIR}/CMakeLists.txt
14-
${CMAKE_CURRENT_SOURCE_DIR}/mock_gmm.cpp
15-
${CMAKE_CURRENT_SOURCE_DIR}/mock_gmm.def
16-
)
17-
add_library(${target_name} EXCLUDE_FROM_ALL SHARED ${IGDRCL_SRCS_tests_mock_gmm})
10+
# Setting up our local list of test files
11+
set(IGDRCL_SRCS_tests_mock_gmm
12+
${CMAKE_CURRENT_SOURCE_DIR}/CMakeLists.txt
13+
${CMAKE_CURRENT_SOURCE_DIR}/mock_gmm.cpp
14+
${CMAKE_CURRENT_SOURCE_DIR}/mock_gmm.def
15+
)
16+
add_library(${target_name} EXCLUDE_FROM_ALL SHARED ${IGDRCL_SRCS_tests_mock_gmm})
1817

19-
target_include_directories(${target_name} PRIVATE
20-
${WDK_INCLUDE_PATHS}
21-
${UMKM_SHAREDDATA_INCLUDE_PATHS}
22-
)
18+
target_include_directories(${target_name} PRIVATE
19+
${WDK_INCLUDE_PATHS}
20+
${UMKM_SHAREDDATA_INCLUDE_PATHS}
21+
)
2322

24-
set_target_properties(
25-
${target_name}
26-
PROPERTIES DEBUG_OUTPUT_NAME ${target_name}
27-
RELEASE_OUTPUT_NAME ${target_name}
28-
RELEASEINTERNAL_OUTPUT_NAME ${target_name}
29-
OUTPUT_NAME ${target_name}
30-
)
23+
set_target_properties(
24+
${target_name}
25+
PROPERTIES DEBUG_OUTPUT_NAME ${target_name}
26+
RELEASE_OUTPUT_NAME ${target_name}
27+
RELEASEINTERNAL_OUTPUT_NAME ${target_name}
28+
OUTPUT_NAME ${target_name}
29+
)
30+
31+
create_project_source_tree(${target_name})
32+
set_target_properties(${target_name} PROPERTIES FOLDER "test mocks")
33+
target_compile_definitions(${target_name} PUBLIC)
34+
add_dependencies(unit_tests ${target_name})
3135

32-
create_project_source_tree(${target_name})
33-
set_target_properties(${target_name} PROPERTIES FOLDER "test mocks")
34-
target_compile_definitions(${target_name} PUBLIC)
35-
if(NOT USE_STATIC_GMM)
36-
add_dependencies(unit_tests ${target_name})
37-
endif()
38-
endif()

unit_tests/mt_tests/CMakeLists.txt

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ endif()
4646

4747
add_custom_target(run_mt_unit_tests)
4848

49-
add_dependencies(igdrcl_mt_tests test_dynamic_lib)
49+
add_dependencies(igdrcl_mt_tests test_dynamic_lib mock_gmm)
5050

5151
create_project_source_tree(igdrcl_mt_tests ${IGDRCL_SOURCE_DIR}/runtime ${IGDRCL_SOURCE_DIR}/unit_tests)
5252

@@ -58,10 +58,6 @@ endif()
5858

5959
set_target_properties(run_mt_unit_tests PROPERTIES FOLDER ${TEST_PROJECTS_FOLDER})
6060

61-
if(NOT USE_STATIC_GMM)
62-
add_dependencies(igdrcl_mt_tests mock_gmm)
63-
endif()
64-
6561
function(run_mt_tests target slices subslices eu_per_ss)
6662
add_custom_target(run_${target}_mt_unit_tests DEPENDS igdrcl_mt_tests)
6763
if(NOT WIN32)

0 commit comments

Comments
 (0)