Update pal from commit d198d6a7

Add missing isPhoenix2 in NdDevice.cpp
Fix autoBuffer zombie state on alloc failure
Optionally treat Execute Indirect as single draw
Sync cache allocation and entry insertion
Fix race condition in RemoveOldestFilesOfDirUntilSize()
[UseDccForImage] Fix DCC settings conflict when modifier is used
TraceSession allow multiple TraceSource instances
Remove CmdClearBufferView and CmdClearImageView
Fix PWS acquire point mapping table
Allow changing assert/print modes from a settings file
Fix disableCommandBufferPreemption
[MemoryReport] Fix env AMDPAL_NO_LEAK_REPORT not working as expected
Update address-lib submodule
Init m_settings.enableHardAssert by Util::IsAssertCategoryEnabled()
Cleanup dead setting
Fix the miniscule memory overhead in pm4CmdBuffer
PerfExperiment SPM Test Mode Support
Clean up fix copy dst metadata related codes
Unlimited chaining by default
Update rdf submodule
Fix IF version check
Make sure zwp_linux_dmabuf_v1 interface being used with compatible version
Add mutex to protect hash table in TrackingCacheLayer
Safe guards for invalid TS spill table address
Fix frame trace controller timing issue
Expose unpadded extents of image subresources to clients
Implement incrementing constant
Fix the header location of drm.h
Fix Timeout is not correctly set in no timeout waitIdle case
Correct usage of __builtin_assume_aligned
Fix capture/replay behavior for DescriptorTable memory
Optimize acquire release event barrier path
Fix float rounding on copy dst extent in scaled copy path
Fix color MSAA copy with stale FMask content
[Modifier] handle client passed DisplayDCC setting
Fix race conditions in AddEntryToCache
Handle MM12 formats during CmdCopyImage
With RDP in crash analysis mode observed that test samples does not show the overlay
Change ABI metadata .entry_point enum to .entry_point_symbol string
Update DevDriver submodule
Adjust RemoveOldestFilesOfDirUntilSize() logic
Restructure gpuProfilerQueue and add Mutex
Unify split barrier path codes
Fix CTS_1.4 32Bit full CTS test crash
StringTableSource changes, allow acquiring >1 table Ids at once, allow empty string table source
RGP trace of Vulkan compute example fails to load when using RenderOpTraceController
Fix time clock type using issue
Implement Dri3/X Explicit Sync support
Split amdgpu header modifications away from upstream
Remove SubresLayout::defaultGfxLayout
Clean up CmdResolveImage()
Clean up OptimizeStageMask and OptimizeAccessMask related codes
Handle UserMarkerOpHistory numOps == 0 properly

Author: qiaojbao

cmake/Modules/CMakeLists.txt

@@ -27,5 +27,4 @@ target_sources(pal PRIVATE
     CMakeLists.txt
     FindDRM.cmake
     FindWayland.cmake
-    FindXCB.cmake
 )

cmake/Modules/FindXCB.cmake

@@ -98,8 +98,13 @@ struct CbConstUsageMetadata
 /// Per-hardware stage metadata.
 struct HardwareStageMetadata
 {
-    /// The symbol pointing to this pipeline's stage entrypoint.
+    /// The symbol name pointing to this pipeline's stage entrypoint.
+    StringViewType          entryPointSymbol;
+#if PAL_CLIENT_INTERFACE_MAJOR_VERSION < 911
+    /// Deprecated.  The symbol pointing to this pipeline's stage entrypoint.
     Abi::PipelineSymbolType entryPoint;
+#endif
+
     /// Scratch memory size in bytes.
     uint32                  scratchMemorySize;
     /// size in bytes (per lane/thread) of the stack managed by the compiler backend.
@@ -112,6 +117,7 @@ struct HardwareStageMetadata
     uint32                  perfDataBufferSize;
     /// Number of VGPRs used.
     uint32                  vgprCount;
+
     /// Number of SGPRs used.
     uint32                  sgprCount;
     /// If non-zero, indicates the shader was compiled with a directive to instruct the compiler to limit the VGPR usage
@@ -193,17 +199,23 @@ struct HardwareStageMetadata
     {
         struct
         {
+            uint64 entryPointSymbol          : 1;
+#if PAL_CLIENT_INTERFACE_MAJOR_VERSION < 911
             uint64 entryPoint                : 1;
+#else
+            uint64 placeholder0              : 1;
+#endif
             uint64 scratchMemorySize         : 1;
             uint64 backendStackSize          : 1;
             uint64 frontendStackSize         : 1;
             uint64 ldsSize                   : 1;
             uint64 perfDataBufferSize        : 1;
             uint64 vgprCount                 : 1;
+            uint64 placeholder1              : 1;
             uint64 sgprCount                 : 1;
             uint64 vgprLimit                 : 1;
             uint64 sgprLimit                 : 1;
-            uint64 placeholder0              : 1;
+            uint64 placeholder2              : 1;
             uint64 threadgroupDimensions     : 1;
             uint64 origThreadgroupDimensions : 1;
             uint64 cbConstUsage              : 1;
@@ -232,8 +244,8 @@ struct HardwareStageMetadata
             uint64 writesDepth               : 1;
             uint64 usesAppendConsume         : 1;
             uint64 usesPrimId                : 1;
-            uint64 placeholder1              : 1;
-            uint64 reserved                  : 24;
+            uint64 placeholder3              : 1;
+            uint64 reserved                  : 22;
         };
         uint64 uAll;
     } hasEntry;
@@ -1310,8 +1322,7 @@ struct SpiBarycCntlMetadata
         {
             /// Whether to use the entire 32b value to determine front-facing.
             uint8 frontFaceAllBits : 1;
-            uint8 placeholder0     : 1;
-            uint8 reserved         : 6;
+            uint8 reserved         : 7;
         };
         uint8 uAll;
     } flags;
@@ -1322,10 +1333,7 @@ struct SpiBarycCntlMetadata
         {
             uint8 posFloatLocation : 1;
             uint8 frontFaceAllBits : 1;
-            uint8 placeholder0     : 1;
-            uint8 placeholder1     : 1;
-            uint8 placeholder2     : 1;
-            uint8 reserved         : 3;
+            uint8 reserved         : 6;
         };
         uint8 uAll;
     } hasEntry;
@@ -2161,7 +2169,6 @@ namespace SpiBarycCntlMetadataKey
 {
     static constexpr char PosFloatLocation[] = ".pos_float_location";
     static constexpr char FrontFaceAllBits[] = ".front_face_all_bits";
-
 };
 
 namespace PaScShaderControlMetadataKey
@@ -2439,13 +2446,18 @@ namespace PsInputSemanticMetadataKey
 
 namespace HardwareStageMetadataKey
 {
+    static constexpr char EntryPointSymbol[]          = ".entry_point_symbol";
+#if PAL_CLIENT_INTERFACE_MAJOR_VERSION < 911
     static constexpr char EntryPoint[]                = ".entry_point";
+#endif
+
     static constexpr char ScratchMemorySize[]         = ".scratch_memory_size";
     static constexpr char BackendStackSize[]          = ".backend_stack_size";
     static constexpr char FrontendStackSize[]         = ".frontend_stack_size";
     static constexpr char LdsSize[]                   = ".lds_size";
     static constexpr char PerfDataBufferSize[]        = ".perf_data_buffer_size";
     static constexpr char VgprCount[]                 = ".vgpr_count";
+
     static constexpr char SgprCount[]                 = ".sgpr_count";
     static constexpr char VgprLimit[]                 = ".vgpr_limit";
     static constexpr char SgprLimit[]                 = ".sgpr_limit";

inc/core/g_palPipelineAbiMetadata.h

@@ -1092,12 +1092,21 @@ inline Result DeserializeHardwareStageMetadata(
         {
             switch (HashString(key))
             {
+            case HashLiteralString(HardwareStageMetadataKey::EntryPointSymbol):
+                PAL_ASSERT(pMetadata->hasEntry.entryPointSymbol == 0);
+                result = pReader->UnpackNext(&pMetadata->entryPointSymbol);
+                pMetadata->hasEntry.entryPointSymbol = (result == Result::Success);;
+                break;
+
+#if PAL_CLIENT_INTERFACE_MAJOR_VERSION < 911
             case HashLiteralString(HardwareStageMetadataKey::EntryPoint):
                 PAL_ASSERT(pMetadata->hasEntry.entryPoint == 0);
                 result = DeserializeEnum(pReader, &pMetadata->entryPoint);
                 pMetadata->hasEntry.entryPoint = (result == Result::Success);;
                 break;
 
+#endif
+
             case HashLiteralString(HardwareStageMetadataKey::ScratchMemorySize):
                 PAL_ASSERT(pMetadata->hasEntry.scratchMemorySize == 0);
                 result = pReader->UnpackNext(&pMetadata->scratchMemorySize);

inc/core/g_palPipelineAbiMetadataImpl.h

@@ -3938,6 +3938,7 @@ class ICmdBuffer : public IDestroyable
         const Rect*        pRects,
         uint32             flags) = 0;
 
+#if PAL_CLIENT_INTERFACE_MAJOR_VERSION < 910
     /// Clears a range of GPU memory to the specified clear color using the specified buffer view SRD.
     ///
     /// The maximum clear range is determined by the view; if any Ranges are specified they must fit within the view's
@@ -3988,6 +3989,7 @@ class ICmdBuffer : public IDestroyable
         const void*       pImageViewSrd,
         uint32            rectCount = 0,
         const Rect*       pRects    = nullptr) = 0;
+#endif
 
     /// Resolves multiple regions of a multisampled image to a single-sampled image.
     ///

inc/core/palCmdBuffer.h

@@ -1247,8 +1247,10 @@ struct DeviceProperties
 
                 /// Set if images created on this device support being created with corner sampling.
                 uint32 supportsCornerSampling       :  1;
-                /// Placeholder, do not use.
+
+                /// Load Bearing placeholder, do not touch.
                 uint32 placeholder0                 :  1;
+
                 /// Reserved for future use.
                 uint32 reserved                     : 29;
             };
@@ -1434,7 +1436,9 @@ struct DeviceProperties
                 uint64 supportBFloat16                    :  1; ///< HW supports bf16 instructions.
                 uint64 supportFloat8                      :  1; ///< HW supports float 8-bit instructions.
                 uint64 supportInt4                        :  1; ///< HW supports integer 4-bit instructions.
-                uint64 reserved                           :  62; ///< Reserved for future use.
+                uint64 placeholder13                      :  1;
+                uint64 placeholder14                      :  1;
+                uint64 reserved                           :  60; ///< Reserved for future use.
             };
             uint64 u64All[2];           ///< Flags packed as 32-bit uint.
         } flags;                     ///< Device IP property flags.

inc/core/palDevice.h

@@ -593,16 +593,19 @@ struct SubresLayout
     Extent3d blockSize;     ///< Size of a tile block in texels - micro tile for 1D tiling and macro tile for 2D tiling.
     Offset3d mipTailCoord;  ///< coords of the subresource within the mip tail
 
-    /// Extent of the subresource in texels, including all internal padding for this subresource.
-    Extent3d paddedExtent;
+    Extent3d extentTexels;   ///< Unpadded extent of the subresource in texels.
+    Extent3d extentElements; ///< Unpadded extent of the subresource in elements.
+    Extent3d paddedExtent;   ///< Extent of the subresource in elements, including all internal padding for this subresource.
 
+#if PAL_CLIENT_INTERFACE_MAJOR_VERSION < 912
     /// Reports supported engines and usages for this subresource while it can remain in its optimal compression state.
     /// Clients using CmdRelease()/CmdAcquire() without complete knowledge of the application's next usage during
     /// CmdRelease() or its previous usage at CmdAcquire() can treat this layout as a performant target for an
     /// intermediate state that will avoid unnecessary decompressions.
     ///
     /// This value is only valid if supportSplitReleaseAcquire is set in @ref DeviceProperties.
     ImageLayout defaultGfxLayout;
+#endif
 
     SwizzledFormat planeFormat; ///< Swizzled format for plane. Planar resource like D32-S8
                                 /// will have different swizzled format per plane.

inc/core/palImage.h

@@ -114,6 +114,12 @@ struct IndirectParam
         {
             uint32  bufferId;       ///< Vertex bufer slot ID to set.
         } vertexData;               ///< Additional information about a 'BindVertexData' indirect command parameter.
+
+        struct
+        {
+            bool constantDrawIndex; ///< If set, does not increment the Draw Index per draw.
+        } drawData;                 ///< Additional information about 'Draw', 'DrawIndexed' and 'DispatchMesh'
+                                    ///  indirect command parameters.
     };
 };
 

inc/core/palIndirectCmdGenerator.h

@@ -47,7 +47,7 @@
 #endif
 ///
 /// @ingroup LibInit
-#define PAL_INTERFACE_MAJOR_VERSION 909
+#define PAL_INTERFACE_MAJOR_VERSION 912
 
 #if PAL_CLIENT_INTERFACE_MAJOR_VERSION < 831
 /// Minor interface version.  Note that the interface version is distinct from the PAL version itself, which is returned
@@ -247,12 +247,7 @@ struct GpuInfo
 enum class ClientApi : uint32
 {
     Pal     = 0,
-    Dx9     = 1,
-    Dx12    = 3,
     Vulkan  = 4,
-    OpenCl  = 7,
-    Hip     = 8,
-    Amf     = 9,
 };
 
 /// Specifies properties for @ref IPlatform creation. Input structure to Pal::CreatePlatform().

inc/core/palLib.h

@@ -812,7 +812,7 @@ class IPipeline : public IDestroyable
     virtual void SetStackSizeInBytes(
         uint32 stackSizeInBytes) = 0;
 
-    /// Retrieve the stack sizes managed by compiler, including the frontend stack and the frontend stack.
+    /// Retrieve the stack sizes managed by compiler, including the frontend stack and the backend stack.
     ///
     /// @param [out] pSizes  To be filled with both the frontend stack size and the backend stack size, in bytes.
     ///