Update GMM H/V alignment API

Change-Id: I2713b912cd93ae28de6c7ef6a8348107f0902368
Signed-off-by: Dunajski, Bartosz <bartosz.dunajski@intel.com>

Author: BartoszDunajski

runtime/gmm_helper/gmm.cpp

@@ -202,14 +202,6 @@ uint32_t Gmm::queryQPitch(GMM_RESOURCE_TYPE resType) {
     return gmmResourceInfo->getQPitch();
 }
 
-uint32_t Gmm::getRenderHAlignment() {
-    return GmmHelper::getRenderAlignment(gmmResourceInfo->getHAlign());
-}
-
-uint32_t Gmm::getRenderVAlignment() {
-    return GmmHelper::getRenderAlignment(gmmResourceInfo->getVAlign());
-}
-
 void Gmm::updateImgInfo(ImageInfo &imgInfo, cl_image_desc &imgDesc, cl_uint arrayIndex) {
     imgDesc.image_width = gmmResourceInfo->getBaseWidth();
     imgDesc.image_row_pitch = gmmResourceInfo->getRenderPitch();

runtime/gmm_helper/gmm.h

@@ -32,9 +32,6 @@ class Gmm {
 
     void queryImageParams(ImageInfo &inputOutputImgInfo);
 
-    uint32_t getRenderHAlignment();
-    uint32_t getRenderVAlignment();
-
     void applyAuxFlagsForBuffer(bool preferRenderCompression);
     void applyMemoryFlags(bool systemMemoryPool, StorageInfo &storageInfo);
 

runtime/gmm_helper/gmm_helper.cpp

@@ -106,18 +106,6 @@ uint64_t GmmHelper::decanonize(uint64_t address) {
     return (uint64_t)(address & 0xFFFFFFFFFFFF);
 }
 
-uint32_t GmmHelper::getRenderAlignment(uint32_t alignment) {
-    uint32_t returnAlign = 0;
-    if (alignment == 8) {
-        returnAlign = 2;
-    } else if (alignment == 16) {
-        returnAlign = 3;
-    } else {
-        returnAlign = 1;
-    }
-    return returnAlign;
-}
-
 uint32_t GmmHelper::getRenderMultisamplesCount(uint32_t numSamples) {
     if (numSamples == 2) {
         return 1;

runtime/gmm_helper/gmm_helper.h

@@ -47,7 +47,6 @@ class GmmHelper {
     static void queryImgFromBufferParams(ImageInfo &imgInfo, GraphicsAllocation *gfxAlloc);
     static GMM_CUBE_FACE_ENUM getCubeFaceIndex(uint32_t target);
     static bool allowTiling(const cl_image_desc &imageDesc);
-    static uint32_t getRenderAlignment(uint32_t alignment);
     static uint32_t getRenderMultisamplesCount(uint32_t numSamples);
     static GMM_YUV_PLANE convertPlane(OCLPlane oclPlane);
 

runtime/gmm_helper/resource_info.h

@@ -40,7 +40,9 @@ class GmmResourceInfo {
 
     MOCKABLE_VIRTUAL uint32_t getHAlign() { return resourceInfo->GetHAlign(); }
 
-    MOCKABLE_VIRTUAL uint32_t getVAlign() { return resourceInfo->GetVAlign(); }
+    MOCKABLE_VIRTUAL uint32_t getHAlignSurfaceState() { return resourceInfo->GetHAlignSurfaceState(); }
+
+    MOCKABLE_VIRTUAL uint32_t getVAlignSurfaceState() { return resourceInfo->GetVAlignSurfaceState(); }
 
     MOCKABLE_VIRTUAL uint32_t getMaxLod() { return resourceInfo->GetMaxLod(); }
 

runtime/mem_obj/image.inl

@@ -48,8 +48,8 @@ void ImageHw<GfxFamily>::setImageArg(void *memory, bool setAsMediaBlockImage, ui
     auto vAlign = RENDER_SURFACE_STATE::SURFACE_VERTICAL_ALIGNMENT_VALIGN_4;
 
     if (gmm) {
-        hAlign = static_cast<typename RENDER_SURFACE_STATE::SURFACE_HORIZONTAL_ALIGNMENT>(gmm->getRenderHAlignment());
-        vAlign = static_cast<typename RENDER_SURFACE_STATE::SURFACE_VERTICAL_ALIGNMENT>(gmm->getRenderVAlignment());
+        hAlign = static_cast<typename RENDER_SURFACE_STATE::SURFACE_HORIZONTAL_ALIGNMENT>(gmm->gmmResourceInfo->getHAlignSurfaceState());
+        vAlign = static_cast<typename RENDER_SURFACE_STATE::SURFACE_VERTICAL_ALIGNMENT>(gmm->gmmResourceInfo->getVAlignSurfaceState());
     }
 
     if (cubeFaceIndex != __GMM_NO_CUBE_MAP) {

unit_tests/gmm_helper/gmm_helper_tests.cpp

@@ -400,13 +400,6 @@ TEST_F(GmmTests, decanonize) {
     EXPECT_EQ(GmmHelper::decanonize(addr2), addrExpected2);
 }
 
-TEST_F(GmmTests, returnRenderTileMode) {
-    uint32_t alignments[5][2] = {{0, 1}, {4, 1}, {8, 2}, {16, 3}, {20, 1}}; // {given, expected}
-    for (uint32_t i = 0; i < 5; i++) {
-        EXPECT_EQ(GmmHelper::getRenderAlignment(alignments[i][0]), alignments[i][1]);
-    }
-}
-
 TEST_F(GmmTests, givenMipmapedInputWhenAskedForHalingThenNonDefaultValueIsReturned) {
     cl_image_desc imgDesc{};
     imgDesc.image_type = CL_MEM_OBJECT_IMAGE2D;

unit_tests/mocks/mock_gmm_resource_info.h

@@ -47,7 +47,9 @@ class MockGmmResourceInfo : public GmmResourceInfo {
 
     uint32_t getHAlign() override { return 4u; }
 
-    uint32_t getVAlign() override { return 4u; }
+    uint32_t getHAlignSurfaceState() override { return 1u; }
+
+    uint32_t getVAlignSurfaceState() override { return 1u; }
 
     uint32_t getMaxLod() override { return 7u; }