Skip to content

Commit 43dd16e

Browse files
kgibalaCompute-Runtime-Automation
authored andcommitted
Add unit tests to check value shift
Related-To: NEO-3962 Change-Id: Ia3ba560201dd495a22742ef9fef767d6aa4c35f9 Signed-off-by: Gibala <krzysztof.gibala@intel.com>
1 parent b1fbced commit 43dd16e

File tree

3 files changed

+39
-0
lines changed

3 files changed

+39
-0
lines changed

unit_tests/gen11/image_tests_gen11.cpp

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -88,3 +88,16 @@ GEN11TEST_F(gen11ImageTests, givenImageForGen11WhenClearColorParametersAreSetThe
8888

8989
EXPECT_EQ(0, memcmp(&surfaceStateBefore, &surfaceStateAfter, sizeof(RENDER_SURFACE_STATE)));
9090
}
91+
92+
using Gen11RenderSurfaceStateDataTests = ::testing::Test;
93+
94+
GEN11TEST_F(Gen11RenderSurfaceStateDataTests, WhenMemoryObjectControlStateIndexToMocsTablesIsSetThenValueIsShift) {
95+
using RENDER_SURFACE_STATE = typename FamilyType::RENDER_SURFACE_STATE;
96+
auto surfaceState = FamilyType::cmdInitRenderSurfaceState;
97+
98+
uint32_t value = 4;
99+
surfaceState.setMemoryObjectControlStateIndexToMocsTables(value);
100+
101+
EXPECT_EQ(surfaceState.TheStructure.Common.MemoryObjectControlState_IndexToMocsTables, value >> 1);
102+
EXPECT_EQ(surfaceState.getMemoryObjectControlStateIndexToMocsTables(), value);
103+
}

unit_tests/gen12lp/image_tests_gen12lp.inl

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -159,3 +159,16 @@ GEN12LPTEST_F(gen12LpImageTests, givenCompressionEnabledWhenAppendingSurfaceStat
159159
image->appendSurfaceStateParams(&rss);
160160
EXPECT_EQ(expectedCompressionFormat, rss.getCompressionFormat());
161161
}
162+
163+
using Gen12lpRenderSurfaceStateDataTests = ::testing::Test;
164+
165+
GEN12LPTEST_F(Gen12lpRenderSurfaceStateDataTests, WhenMemoryObjectControlStateIndexToMocsTablesIsSetThenValueIsShift) {
166+
using RENDER_SURFACE_STATE = typename FamilyType::RENDER_SURFACE_STATE;
167+
auto surfaceState = FamilyType::cmdInitRenderSurfaceState;
168+
169+
uint32_t value = 4;
170+
surfaceState.setMemoryObjectControlStateIndexToMocsTables(value);
171+
172+
EXPECT_EQ(surfaceState.TheStructure.Common.MemoryObjectControlStateIndexToMocsTables, value >> 1);
173+
EXPECT_EQ(surfaceState.getMemoryObjectControlStateIndexToMocsTables(), value);
174+
}

unit_tests/gen9/image_tests_gen9.cpp

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,3 +27,16 @@ GEN9TEST_F(gen9ImageTests, appendSurfaceStateParamsDoesNothing) {
2727

2828
EXPECT_EQ(0, memcmp(&surfaceStateBefore, &surfaceStateAfter, sizeof(RENDER_SURFACE_STATE)));
2929
}
30+
31+
using Gen9RenderSurfaceStateDataTests = ::testing::Test;
32+
33+
GEN9TEST_F(Gen9RenderSurfaceStateDataTests, WhenMemoryObjectControlStateIndexToMocsTablesIsSetThenValueIsShift) {
34+
using RENDER_SURFACE_STATE = typename FamilyType::RENDER_SURFACE_STATE;
35+
auto surfaceState = FamilyType::cmdInitRenderSurfaceState;
36+
37+
uint32_t value = 4;
38+
surfaceState.setMemoryObjectControlStateIndexToMocsTables(value);
39+
40+
EXPECT_EQ(surfaceState.TheStructure.Common.MemoryObjectControlState_IndexToMocsTables, value >> 1);
41+
EXPECT_EQ(surfaceState.getMemoryObjectControlStateIndexToMocsTables(), value);
42+
}

0 commit comments

Comments
 (0)