Turn on EnableDebugBreak by default and infrastructure update

Change-Id: Ie7e12a7e1509f0a7a5962e97042dca8fa714d132

manifests/manifest.yml

@@ -31,7 +31,7 @@ components:
     branch: infra
     clean_on_sync: true
     dest_dir: infra
-    revision: 0ae2c97b2fbb2ed20a0af1f662441cd9b57539d9
+    revision: 108792cd99cedee5e5028311fb76a3553826ef1d
     type: git
   internal:
     branch: master

runtime/os_interface/DebugVariables.inl

@@ -20,7 +20,7 @@
  * OTHER DEALINGS IN THE SOFTWARE.
  */
 /*DEBUG FLAGS*/
-DECLARE_DEBUG_VARIABLE(bool, EnableDebugBreak, false, "Enable DEBUG_BREAKs")
+DECLARE_DEBUG_VARIABLE(bool, EnableDebugBreak, true, "Enable DEBUG_BREAKs")
 DECLARE_DEBUG_VARIABLE(bool, FlushAllCaches, false, "pipe controls between enqueues flush all possible caches")
 DECLARE_DEBUG_VARIABLE(bool, MakeEachEnqueueBlocking, false, "equivalent of finish after each enqueue")
 DECLARE_DEBUG_VARIABLE(bool, DoCpuCopyOnReadBuffer, false, "triggers CPU copy path for Read Buffer calls, only supported for some basic use cases ( no events, not blocked calls )")

unit_tests/helpers/debug_helpers_tests.cpp

@@ -21,8 +21,15 @@
  */
 
 #include "runtime/helpers/debug_helpers.h"
+#include "runtime/os_interface/debug_settings_manager.h"
 #include "gtest/gtest.h"
 
-TEST(debug_break, whenDebugBreakCalledThenAssertIsThrown) {
+using namespace OCLRT;
+
+TEST(debugBreak, whenDebugBreakCalledInTestThenNothingIsThrown) {
     DEBUG_BREAK_IF(!false);
 }
+
+TEST(debugBreak, givenEnableDebugBreakWhenFlagIsEnabledThenReturnTrue) {
+    EXPECT_EQ(true, DebugManager.flags.EnableDebugBreak.get());
+}

unit_tests/test_files/igdrcl.config

@@ -44,7 +44,7 @@ OverrideKmdNotifyDelayMs = -1
 Enable64kbpages = -1
 NodeOrdinal = 0
 ProductFamilyOverride = unk
-EnableDebugBreak = false
+EnableDebugBreak = true
 EnableComputeWorkSizeND = true
 EventsDebugEnable = false
 UseMaxSimdSizeToDeduceMaxWorkgroupSize = false