Cleanup includes in sampler, image, builtin and event

Signed-off-by: Mateusz Jablonski <mateusz.jablonski@intel.com>

level_zero/api/extensions/public/ze_exp_ext.cpp

@@ -7,6 +7,8 @@
 
 #include "level_zero/api/extensions/public/ze_exp_ext.h"
 
+#include "level_zero/core/source/context/context.h"
+#include "level_zero/core/source/device/device.h"
 #include "level_zero/core/source/event/event.h"
 #include "level_zero/core/source/image/image.h"
 #include "level_zero/core/source/kernel/kernel.h"

level_zero/api/tools/zet_metric_api_entrypoints.h

@@ -8,6 +8,7 @@
 #pragma once
 
 #include "level_zero/core/source/cmdlist/cmdlist.h"
+#include "level_zero/core/source/context/context.h"
 #include "level_zero/core/source/device/device.h"
 #include "level_zero/tools/source/metrics/metric.h"
 #include <level_zero/zet_api.h>

level_zero/core/source/builtin/builtin_functions_lib_impl.cpp

@@ -9,7 +9,9 @@
 
 #include "shared/source/built_ins/built_ins.h"
 
+#include "level_zero/core/source/device/device.h"
 #include "level_zero/core/source/kernel/kernel.h"
+#include "level_zero/core/source/module/module.h"
 
 namespace NEO {
 const char *getAdditionalBuiltinAsString(EBuiltInOps::Type builtin) {

level_zero/core/source/builtin/builtin_functions_lib_impl.h

@@ -8,8 +8,6 @@
 #pragma once
 
 #include "level_zero/core/source/builtin/builtin_functions_lib.h"
-#include "level_zero/core/source/device/device.h"
-#include "level_zero/core/source/module/module.h"
 
 namespace NEO {
 namespace EBuiltInOps {
@@ -19,6 +17,10 @@ class BuiltIns;
 } // namespace NEO
 
 namespace L0 {
+struct Module;
+struct Kernel;
+struct Device;
+
 struct BuiltinFunctionsLibImpl : BuiltinFunctionsLib {
     struct BuiltinData;
     BuiltinFunctionsLibImpl(Device *device, NEO::BuiltIns *builtInsLib)

level_zero/core/source/cmdlist/cmdlist_hw.inl

@@ -43,6 +43,8 @@
 #include "level_zero/core/source/kernel/kernel.h"
 #include "level_zero/core/source/module/module.h"
 
+#include "CL/cl.h"
+
 #include <algorithm>
 
 namespace L0 {

level_zero/core/source/device/device.h

@@ -11,8 +11,6 @@
 #include <level_zero/ze_api.h>
 #include <level_zero/zet_api.h>
 
-#include "CL/cl.h"
-
 static_assert(NEO::HwInfoConfig::uuidSize == ZE_MAX_DEVICE_UUID_SIZE);
 
 struct _ze_device_handle_t {};

level_zero/core/source/event/event.h

@@ -9,10 +9,6 @@
 
 #include "shared/source/helpers/timestamp_packet.h"
 
-#include "level_zero/core/source/cmdlist/cmdlist.h"
-#include "level_zero/core/source/context/context_imp.h"
-#include "level_zero/core/source/device/device.h"
-#include "level_zero/core/source/driver/driver_handle.h"
 #include <level_zero/ze_api.h>
 
 #include <bitset>
@@ -27,6 +23,10 @@ namespace L0 {
 typedef uint64_t FlushStamp;
 struct EventPool;
 struct MetricStreamer;
+struct ContextImp;
+struct Context;
+struct DriverHandle;
+struct Device;
 
 namespace EventPacketsCount {
 constexpr uint32_t maxKernelSplit = 3;

level_zero/core/source/image/image.h

@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2020-2021 Intel Corporation
+ * Copyright (C) 2020-2022 Intel Corporation
  *
  * SPDX-License-Identifier: MIT
  *
@@ -7,19 +7,19 @@
 
 #pragma once
 
-#include "shared/source/command_container/cmdcontainer.h"
-
-#include "level_zero/core/source/device/device.h"
 #include <level_zero/ze_api.h>
 
 struct _ze_image_handle_t {};
 
 namespace NEO {
 struct ImageInfo;
+class GraphicsAllocation;
 struct ImageDescriptor;
 } // namespace NEO
 
 namespace L0 {
+struct Device;
+
 struct Image : _ze_image_handle_t {
     template <typename Type>
     struct Allocator {

level_zero/core/source/image/image_format_desc_helper.h

@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2020 Intel Corporation
+ * Copyright (C) 2020-2022 Intel Corporation
  *
  * SPDX-License-Identifier: MIT
  *
@@ -9,6 +9,8 @@
 
 #include "level_zero/core/source/image/image.h"
 
+#include "CL/cl.h"
+
 namespace L0 {
 
 struct swizzles {

level_zero/core/source/image/image_hw.inl

@@ -16,6 +16,7 @@
 #include "shared/source/memory_manager/allocation_properties.h"
 #include "shared/source/memory_manager/memory_manager.h"
 
+#include "level_zero/core/source/device/device.h"
 #include "level_zero/core/source/helpers/properties_parser.h"
 #include "level_zero/core/source/hw_helpers/l0_hw_helper.h"
 #include "level_zero/core/source/image/image_formats.h"

level_zero/core/source/image/image_imp.cpp

@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2020-2021 Intel Corporation
+ * Copyright (C) 2020-2022 Intel Corporation
  *
  * SPDX-License-Identifier: MIT
  *
@@ -9,6 +9,8 @@
 
 #include "shared/source/memory_manager/memory_manager.h"
 
+#include "level_zero/core/source/device/device.h"
+
 #include "igfxfmid.h"
 
 namespace L0 {

level_zero/core/source/sampler/sampler.h

@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2020-2021 Intel Corporation
+ * Copyright (C) 2020-2022 Intel Corporation
  *
  * SPDX-License-Identifier: MIT
  *
@@ -7,12 +7,12 @@
 
 #pragma once
 
-#include "level_zero/core/source/device/device.h"
 #include <level_zero/ze_api.h>
 
 struct _ze_sampler_handle_t {};
 
 namespace L0 {
+struct Device;
 
 struct Sampler : _ze_sampler_handle_t {
     template <typename Type>

level_zero/core/source/sampler/sampler_hw.inl

@@ -7,9 +7,11 @@
 
 #pragma once
 
+#include "shared/source/helpers/ptr_math.h"
 #include "shared/source/helpers/string.h"
 #include "shared/source/utilities/numeric.h"
 
+#include "level_zero/core/source/device/device.h"
 #include "level_zero/core/source/sampler/sampler_hw.h"
 
 namespace L0 {

level_zero/core/test/unit_tests/mocks/mock_builtin_functions_lib_impl_timestamps.h

@@ -8,6 +8,7 @@
 #include "shared/source/built_ins/built_ins.h"
 
 #include "level_zero/core/source/builtin/builtin_functions_lib_impl.h"
+#include "level_zero/core/source/module/module.h"
 
 namespace L0 {
 namespace ult {

level_zero/tools/source/metrics/linux/os_metric_oa_enumeration_imp_linux.cpp

@@ -9,6 +9,7 @@
 #include "shared/source/os_interface/linux/sys_calls.h"
 #include "shared/source/os_interface/os_interface.h"
 
+#include "level_zero/core/source/device/device.h"
 #include "level_zero/tools/source/metrics/metric_oa_enumeration_imp.h"
 #include "level_zero/tools/source/metrics/metric_oa_source.h"
 

level_zero/tools/source/metrics/metric_ip_sampling_streamer.cpp

@@ -7,6 +7,7 @@
 
 #include "level_zero/tools/source/metrics/metric_ip_sampling_streamer.h"
 
+#include "level_zero/core/source/device/device.h"
 #include "level_zero/tools/source/metrics/metric.h"
 #include "level_zero/tools/source/metrics/metric_ip_sampling_source.h"
 #include "level_zero/tools/source/metrics/os_metric_ip_sampling.h"

level_zero/tools/source/metrics/metric_oa_query_imp.cpp

@@ -19,6 +19,7 @@
 #include "level_zero/core/source/cmdlist/cmdlist_imp.h"
 #include "level_zero/core/source/device/device.h"
 #include "level_zero/core/source/device/device_imp.h"
+#include "level_zero/core/source/driver/driver_handle.h"
 #include "level_zero/tools/source/metrics/metric_oa_enumeration_imp.h"
 #include "level_zero/tools/source/metrics/metric_oa_source.h"
 

level_zero/tools/source/metrics/metric_oa_source.cpp

@@ -9,6 +9,7 @@
 
 #include "shared/source/os_interface/os_library.h"
 
+#include "level_zero/core/source/cmdlist/cmdlist.h"
 #include "level_zero/core/source/device/device_imp.h"
 #include "level_zero/tools/source/metrics/metric.h"
 #include "level_zero/tools/source/metrics/metric_oa_enumeration_imp.h"
@@ -147,4 +148,4 @@ OaMetricSourceImp &MetricDeviceContext::getMetricSource<OaMetricSourceImp>() con
     return static_cast<OaMetricSourceImp &>(*metricSources.at(MetricSource::SourceType::Oa));
 }
 
-} // namespace L0
+} // namespace L0

level_zero/tools/source/metrics/metric_oa_streamer_imp.cpp

@@ -9,6 +9,7 @@
 
 #include "shared/source/helpers/debug_helpers.h"
 
+#include "level_zero/core/source/cmdlist/cmdlist.h"
 #include "level_zero/core/source/device/device_imp.h"
 #include "level_zero/tools/source/metrics/metric_oa_enumeration_imp.h"
 #include "level_zero/tools/source/metrics/metric_oa_query_imp.h"

level_zero/tools/source/metrics/windows/os_metric_oa_enumeration_imp_windows.cpp

@@ -8,6 +8,7 @@
 #include "shared/source/os_interface/os_interface.h"
 #include "shared/source/os_interface/windows/wddm/wddm.h"
 
+#include "level_zero/core/source/device/device.h"
 #include "level_zero/tools/source/metrics/metric_oa_enumeration_imp.h"
 #include "level_zero/tools/source/metrics/metric_oa_source.h"
 

level_zero/tools/test/unit_tests/sources/metrics/test_metric_ip_sampling_enumeration.cpp

@@ -5,6 +5,7 @@
  *
  */
 
+#include "level_zero/core/source/cmdlist/cmdlist.h"
 #include "level_zero/core/test/unit_tests/fixtures/device_fixture.h"
 #include "level_zero/tools/source/metrics/metric_ip_sampling_source.h"
 #include "level_zero/tools/source/metrics/metric_oa_source.h"

level_zero/tools/test/unit_tests/sources/metrics/test_metric_ip_sampling_streamer.cpp

@@ -5,6 +5,7 @@
  *
  */
 
+#include "level_zero/core/source/cmdlist/cmdlist.h"
 #include "level_zero/core/test/unit_tests/fixtures/device_fixture.h"
 #include "level_zero/tools/source/metrics/metric_ip_sampling_source.h"
 #include "level_zero/tools/source/metrics/metric_oa_source.h"

level_zero/tools/test/unit_tests/sources/metrics/test_metric_oa_streamer_2.cpp

@@ -7,6 +7,7 @@
 
 #include "shared/test/common/test_macros/test.h"
 
+#include "level_zero/core/source/cmdlist/cmdlist.h"
 #include "level_zero/tools/source/metrics/metric_oa_source.h"
 #include "level_zero/tools/test/unit_tests/sources/metrics/mock_metric_oa.h"
 

level_zero/tools/test/unit_tests/sources/metrics/test_metric_oa_streamer_3.cpp

@@ -7,6 +7,7 @@
 
 #include "shared/test/common/test_macros/test.h"
 
+#include "level_zero/core/source/cmdlist/cmdlist.h"
 #include "level_zero/tools/test/unit_tests/sources/metrics/mock_metric_oa.h"
 
 using ::testing::_;