Always enable L3 cache for non zero copy buffers.

Change-Id: Ie83a2b6b040bd0bab2cfd5dbb0e788af881f966f

Author: MichalMrozek

runtime/mem_obj/buffer.inl

@@ -65,7 +65,7 @@ void BufferHw<GfxFamily>::setArgStateful(void *memory, bool forceNonAuxMode) {
     surfaceState->setVerticalLineStride(0);
     surfaceState->setVerticalLineStrideOffset(0);
     if ((isAligned<MemoryConstants::cacheLineSize>(bufferAddress) && isAligned<MemoryConstants::cacheLineSize>(bufferSize)) ||
-        ((getFlags() & CL_MEM_READ_ONLY)) != 0) {
+        ((getFlags() & CL_MEM_READ_ONLY)) != 0 || !this->isMemObjZeroCopy()) {
         surfaceState->setMemoryObjectControlState(gmmHelper->getMOCS(GMM_RESOURCE_USAGE_OCL_BUFFER));
     } else {
         surfaceState->setMemoryObjectControlState(gmmHelper->getMOCS(GMM_RESOURCE_USAGE_OCL_BUFFER_CACHELINE_MISALIGNED));

unit_tests/gen9/kernel_tests_gen9.cpp

@@ -1,10 +1,12 @@
 /*
- * Copyright (C) 2018 Intel Corporation
+ * Copyright (C) 2018-2019 Intel Corporation
  *
  * SPDX-License-Identifier: MIT
  *
  */
 
+#include "runtime/gmm_helper/gmm_helper.h"
+#include "runtime/mem_obj/buffer.h"
 #include "runtime/helpers/kernel_commands.h"
 #include "unit_tests/fixtures/device_fixture.h"
 #include "unit_tests/mocks/mock_kernel.h"
@@ -22,3 +24,29 @@ using Gen9KernelCommandsTest = testing::Test;
 GEN9TEST_F(Gen9KernelCommandsTest, givenGen9PlatformWhenDoBindingTablePrefetchIsCalledThenReturnsTrue) {
     EXPECT_TRUE(KernelCommandsHelper<FamilyType>::doBindingTablePrefetch());
 }
+
+GEN9TEST_F(Gen9KernelCommandsTest, givenBufferThatIsNotZeroCopyWhenSurfaceStateisSetThenL3IsTurnedOn) {
+    MockContext context;
+
+    auto retVal = CL_SUCCESS;
+    char ptr[16u] = {};
+
+    std::unique_ptr<Buffer> buffer(Buffer::create(
+        &context,
+        CL_MEM_USE_HOST_PTR,
+        16u,
+        ptr,
+        retVal));
+
+    EXPECT_FALSE(buffer->isMemObjZeroCopy());
+
+    using RENDER_SURFACE_STATE = typename SKLFamily::RENDER_SURFACE_STATE;
+    RENDER_SURFACE_STATE surfaceState = {};
+
+    auto gmmHelper = context.getDevice(0)->getExecutionEnvironment()->getGmmHelper();
+    gmmHelper->setSimplifiedMocsTableUsage(true);
+
+    buffer->setArgStateful(&surfaceState, false);
+    //make sure proper mocs is selected
+    EXPECT_EQ(2u, surfaceState.getMemoryObjectControlStateIndexToMocsTables());
+}