Generic fixes for Linear tiling selection (#65)

johnmach · web-flow · commit 1c975c526341 · 2021-09-07T15:29:33.000+05:30
diff --git a/Source/GmmLib/Resource/GmmResourceInfoCommonEx.cpp b/Source/GmmLib/Resource/GmmResourceInfoCommonEx.cpp
@@ -75,8 +75,9 @@ bool GmmLib::GmmResourceInfoCommon::CopyClientParams(GMM_RESCREATE_PARAMS &Creat
                     CreateParams.Flags.Info.TiledX = true;
                 }
             }
-            else
-            {
+            //Auto-tiling selection if not Linear already
+            else if(CreateParams.Flags.Info.Linear == 0)
+             {
                 // Xe_HP onwards.
                 if((CreateParams.Flags.Info.TiledYs +
                     CreateParams.Flags.Info.TiledYf +
@@ -119,8 +120,9 @@ bool GmmLib::GmmResourceInfoCommon::CopyClientParams(GMM_RESCREATE_PARAMS &Creat
                 }
             }
         }
-        else
-        {
+        //Convert non linear & non-tiledX tiling selection by client to proper tiling.
+        else if(CreateParams.Flags.Info.Linear + CreateParams.Flags.Info.TiledX == 0)
+         {
             if(!pGmmGlobalContext->GetSkuTable().FtrTileY)
             {
                 __GMM_ASSERT(!(CreateParams.Flags.Info.TiledYs ||

Original file line number	Diff line number	Diff line change
`@@ -75,8 +75,9 @@ bool GmmLib::GmmResourceInfoCommon::CopyClientParams(GMM_RESCREATE_PARAMS &Creat`
`75`	`75`	`CreateParams.Flags.Info.TiledX = true;`
`76`	`76`	`}`
`77`	`77`	`}`
`78`		`- else`
`79`		`- {`
	`78`	`+ //Auto-tiling selection if not Linear already`
	`79`	`+ else if(CreateParams.Flags.Info.Linear == 0)`
	`80`	`+ {`
`80`	`81`	`// Xe_HP onwards.`
`81`	`82`	`if((CreateParams.Flags.Info.TiledYs +`
`82`	`83`	`CreateParams.Flags.Info.TiledYf +`
`@@ -119,8 +120,9 @@ bool GmmLib::GmmResourceInfoCommon::CopyClientParams(GMM_RESCREATE_PARAMS &Creat`
`119`	`120`	`}`
`120`	`121`	`}`
`121`	`122`	`}`
`122`		`- else`
`123`		`- {`
	`123`	`+ //Convert non linear & non-tiledX tiling selection by client to proper tiling.`
	`124`	`+ else if(CreateParams.Flags.Info.Linear + CreateParams.Flags.Info.TiledX == 0)`
	`125`	`+ {`
`124`	`126`	`if(!pGmmGlobalContext->GetSkuTable().FtrTileY)`
`125`	`127`	`{`
`126`	`128`	`__GMM_ASSERT(!(CreateParams.Flags.Info.TiledYs \|\|`