wsl/hsakmt: Standardize warning logging with pr_warn_once

Signed-off-by: Horatio Zhang <Hongkun.Zhang@amd.com> Reviewed-by: Flora Cui <flora.cui@amd.com> Reviewed-by: Shane Xiao <shane.xiao@amd.com> Part-of: <http://10.67.69.192/wsl/libhsakmt/-/merge_requests/40>
2024-10-09 16:31:52 +08:00
@@ -32,11 +32,13 @@ static uint32_t runtime_capabilities_mask = 0;

 HSAKMT_STATUS HSAKMTAPI hsaKmtDbgRegister(HSAuint32 NodeId) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

 HSAKMT_STATUS HSAKMTAPI hsaKmtDbgUnregister(HSAuint32 NodeId) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

@@ -44,6 +46,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtDbgWavefrontControl(
    HSAuint32 NodeId, HSA_DBG_WAVEOP Operand, HSA_DBG_WAVEMODE Mode,
    HSAuint32 TrapId, HsaDbgWaveMessage *DbgWaveMsgRing) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

@@ -51,11 +54,13 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtDbgAddressWatch(
    HSAuint32 NodeId, HSAuint32 NumWatchPoints, HSA_DBG_WATCH_MODE WatchMode[],
    void *WatchAddress[], HSAuint64 WatchMask[], HsaEvent *WatchEvent[]) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

 HSAKMT_STATUS HSAKMTAPI hsaKmtCheckRuntimeDebugSupport(void) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

@@ -88,11 +93,12 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtGetRuntimeCapabilities(HSAuint32 *caps_mask) {
 HSAKMT_STATUS HSAKMTAPI hsaKmtDbgEnable(void **runtime_info,
                                        HSAuint32 *data_size) {
  CHECK_DXG_OPEN();
-
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }
 HSAKMT_STATUS HSAKMTAPI hsaKmtDbgDisable(void) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

@@ -100,6 +106,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtDbgGetDeviceData(void **data,
                                               HSAuint32 *n_entries,
                                               HSAuint32 *entry_size) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

@@ -107,6 +114,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtDbgGetQueueData(void **data, HSAuint32 *n_entries,
                                              HSAuint32 *entry_size,
                                              bool suspend_queues) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

@@ -114,5 +122,6 @@ HSAKMT_STATUS HSAKMTAPI
 hsaKmtDebugTrapIoctl(struct kfd_ioctl_dbg_trap_args *args, HSA_QUEUEID *Queues,
                     HSAuint64 *DebugReturn) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }
@@ -33,14 +33,14 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtCreateEvent(HsaEventDescriptor *EventDesc,
                                          bool ManualReset, bool IsSignaled,
                                          HsaEvent **Event) {
  CHECK_DXG_OPEN();
-
+  pr_warn_once("not supported\n");
  assert(false);
  return HSAKMT_STATUS_SUCCESS;
 }

 HSAKMT_STATUS HSAKMTAPI hsaKmtDestroyEvent(HsaEvent *Event) {
  CHECK_DXG_OPEN();
-
+  pr_warn_once("not supported\n");
  if (!Event)
    return HSAKMT_STATUS_INVALID_HANDLE;

@@ -50,7 +50,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtDestroyEvent(HsaEvent *Event) {

 HSAKMT_STATUS HSAKMTAPI hsaKmtSetEvent(HsaEvent *Event) {
  CHECK_DXG_OPEN();
-
+  pr_warn_once("not supported\n");
  if (!Event)
    return HSAKMT_STATUS_INVALID_HANDLE;

@@ -60,7 +60,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtSetEvent(HsaEvent *Event) {

 HSAKMT_STATUS HSAKMTAPI hsaKmtResetEvent(HsaEvent *Event) {
  CHECK_DXG_OPEN();
-
+  pr_warn_once("not supported\n");
  if (!Event)
    return HSAKMT_STATUS_INVALID_HANDLE;

@@ -70,7 +70,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtResetEvent(HsaEvent *Event) {

 HSAKMT_STATUS HSAKMTAPI hsaKmtQueryEventState(HsaEvent *Event) {
  CHECK_DXG_OPEN();
-
+  pr_warn_once("not supported\n");
  if (!Event)
    return HSAKMT_STATUS_INVALID_HANDLE;

@@ -122,7 +122,6 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtWaitOnMultipleEvents_Ext(HsaEvent *Events[],

 HSAKMT_STATUS HSAKMTAPI hsaKmtOpenSMI(HSAuint32 NodeId, int *fd) {
  CHECK_DXG_OPEN();
-
  pr_debug("node id %d\n", NodeId);
  assert(false);
  return HSAKMT_STATUS_SUCCESS;
@@ -74,7 +74,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtSetMemoryPolicy(HSAuint32 Node,
                                              void *MemoryAddressAlternate,
                                              HSAuint64 MemorySizeInBytes) {
  CHECK_DXG_OPEN();
-
+  pr_warn_once("not implemented\n");
  assert(false);
  return HSAKMT_STATUS_SUCCESS;
 }
@@ -236,7 +236,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtAvailableMemory(HSAuint32 Node,
 HSAKMT_STATUS HSAKMTAPI hsaKmtRegisterMemory(void *MemoryAddress,
                                             HSAuint64 MemorySizeInBytes) {
  CHECK_DXG_OPEN();
-
+  pr_warn_once("not implemented\n");
  assert(false);
  return HSAKMT_STATUS_SUCCESS;
 }
@@ -359,7 +359,7 @@ HSAKMT_STATUS HSAKMTAPI
 hsaKmtShareMemory(void *MemoryAddress, HSAuint64 SizeInBytes,
                  HsaSharedMemoryHandle *SharedMemoryHandle) {
  CHECK_DXG_OPEN();
-
+  pr_warn_once("not implemented\n");
  assert(false);
  return HSAKMT_STATUS_SUCCESS;
 }
@@ -368,7 +368,7 @@ HSAKMT_STATUS HSAKMTAPI
 hsaKmtRegisterSharedHandle(const HsaSharedMemoryHandle *SharedMemoryHandle,
                           void **MemoryAddress, HSAuint64 *SizeInBytes) {
  CHECK_DXG_OPEN();
-
+  pr_warn_once("not implemented\n");
  assert(false);
  return HSAKMT_STATUS_SUCCESS;
 }
@@ -377,7 +377,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtRegisterSharedHandleToNodes(
    const HsaSharedMemoryHandle *SharedMemoryHandle, void **MemoryAddress,
    HSAuint64 *SizeInBytes, HSAuint64 NumberOfNodes, HSAuint32 *NodeArray) {
  CHECK_DXG_OPEN();
-
+  pr_warn_once("not implemented\n");
  assert(false);
  return HSAKMT_STATUS_SUCCESS;
 }
@@ -388,8 +388,8 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtProcessVMRead(HSAuint32 Pid,
                                            HsaMemoryRange *RemoteMemoryArray,
                                            HSAuint64 RemoteMemoryArrayCount,
                                            HSAuint64 *SizeCopied) {
-  pr_err("[%s] Deprecated\n", __func__);
-
+  CHECK_DXG_OPEN();
+  pr_warn_once("has been deprecated\n");
  assert(false);
  return HSAKMT_STATUS_NOT_IMPLEMENTED;
 }
@@ -400,8 +400,8 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtProcessVMWrite(HSAuint32 Pid,
                                             HsaMemoryRange *RemoteMemoryArray,
                                             HSAuint64 RemoteMemoryArrayCount,
                                             HSAuint64 *SizeCopied) {
-  pr_err("[%s] Deprecated\n", __func__);
-
+  CHECK_DXG_OPEN();
+  pr_warn_once("has been deprecated\n");
  assert(false);
  return HSAKMT_STATUS_NOT_IMPLEMENTED;
 }
@@ -542,6 +542,8 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtMapGraphicHandle(HSAuint32 NodeId,
                                               HSAuint64 GraphicResourceOffset,
                                               HSAuint64 GraphicResourceSize,
                                               HSAuint64 *FlatMemoryAddress) {
+  CHECK_DXG_OPEN();
+  pr_warn_once("not implemented\n");
  /* This API was only ever implemented in KFD for Kaveri and
   * was never upstreamed. There are no open-source users of
   * this interface. It has been superseded by
@@ -554,7 +556,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtUnmapGraphicHandle(HSAuint32 NodeId,
                                                 HSAuint64 FlatMemoryAddress,
                                                 HSAuint64 SizeInBytes) {
  CHECK_DXG_OPEN();
-
+  pr_warn_once("not implemented\n");
  assert(false);
  return HSAKMT_STATUS_SUCCESS;
 }
@@ -562,7 +564,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtUnmapGraphicHandle(HSAuint32 NodeId,
 HSAKMT_STATUS HSAKMTAPI hsaKmtGetTileConfig(HSAuint32 NodeId,
                                            HsaGpuTileConfig *config) {
  CHECK_DXG_OPEN();
-
+  pr_warn_once("not implemented\n");
  assert(false);
  return HSAKMT_STATUS_SUCCESS;
 }
@@ -609,12 +611,14 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtQueryPointerInfo(const void *Pointer,
 HSAKMT_STATUS HSAKMTAPI hsaKmtSetMemoryUserData(const void *Pointer,
                                                void *UserData) {
  CHECK_DXG_OPEN();
-
+  pr_warn_once("not implemented\n");
  assert(false);
  return HSAKMT_STATUS_SUCCESS;
 }

 HSAKMT_STATUS HSAKMTAPI hsaKmtReplaceAsanHeaderPage(void *addr) {
+  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  assert(false);
 #ifdef SANITIZER_AMDGPU
  pr_debug("address %p\n", addr);
@@ -627,6 +631,8 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtReplaceAsanHeaderPage(void *addr) {
 }

 HSAKMT_STATUS HSAKMTAPI hsaKmtReturnAsanHeaderPage(void *addr) {
+  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  assert(false);
 #ifdef SANITIZER_AMDGPU
  pr_debug("address %p\n", addr);
@@ -32,6 +32,7 @@
 HSAKMT_STATUS HSAKMTAPI hsaKmtPcSamplingSupport(void) {
  CHECK_DXG_OPEN();
  // Used for profiling tools
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

@@ -40,6 +41,7 @@ hsaKmtPcSamplingQueryCapabilities(HSAuint32 NodeId, void *sample_info,
                                  HSAuint32 sample_info_sz, HSAuint32 *size) {
  CHECK_DXG_OPEN();
  // Used for profiling tools
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

@@ -48,6 +50,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtPcSamplingCreate(HSAuint32 NodeId,
                                               HsaPcSamplingTraceId *traceId) {
  CHECK_DXG_OPEN();
  // Used for profiling tools
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

@@ -55,6 +58,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtPcSamplingDestroy(HSAuint32 NodeId,
                                                HsaPcSamplingTraceId traceId) {
  CHECK_DXG_OPEN();
  // Used for profiling tools
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

@@ -62,6 +66,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtPcSamplingStart(HSAuint32 NodeId,
                                              HsaPcSamplingTraceId traceId) {
  CHECK_DXG_OPEN();
  // Used for profiling tools
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

@@ -69,5 +74,6 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtPcSamplingStop(HSAuint32 NodeId,
                                             HsaPcSamplingTraceId traceId) {
  CHECK_DXG_OPEN();
  // Used for profiling tools
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }
@@ -26,6 +26,8 @@

 HSAKMT_STATUS HSAKMTAPI hsaKmtPmcGetCounterProperties(
    HSAuint32 NodeId, HsaCounterProperties **CounterProperties) {
+  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

@@ -35,6 +37,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtPmcRegisterTrace(HSAuint32 NodeId,
                                               HsaCounter *Counters,
                                               HsaPmcTraceRoot *TraceRoot) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

@@ -43,18 +46,21 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtPmcRegisterTrace(HSAuint32 NodeId,
 HSAKMT_STATUS HSAKMTAPI hsaKmtPmcUnregisterTrace(HSAuint32 NodeId,
                                                 HSATraceId TraceId) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

 HSAKMT_STATUS HSAKMTAPI hsaKmtPmcAcquireTraceAccess(HSAuint32 NodeId,
                                                    HSATraceId TraceId) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

 HSAKMT_STATUS HSAKMTAPI hsaKmtPmcReleaseTraceAccess(HSAuint32 NodeId,
                                                    HSATraceId TraceId) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

@@ -64,6 +70,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtPmcStartTrace(HSATraceId TraceId,
                                            void *TraceBuffer,
                                            HSAuint64 TraceBufferSizeBytes) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

@@ -71,6 +78,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtPmcStartTrace(HSATraceId TraceId,
 * has registered */
 HSAKMT_STATUS HSAKMTAPI hsaKmtPmcQueryTrace(HSATraceId TraceId) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

@@ -78,5 +86,6 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtPmcQueryTrace(HSATraceId TraceId) {
 * counters */
 HSAKMT_STATUS HSAKMTAPI hsaKmtPmcStopTrace(HSATraceId TraceId) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }
@@ -158,7 +158,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtSetQueueCUMask(HSA_QUEUEID QueueId,
  if (CUMaskCount == 0 || !QueueCUMask || ((CUMaskCount % 32) != 0))
    return HSAKMT_STATUS_INVALID_PARAMETER;

-  pr_debug("%s not implemented\n", __func__);
+  pr_warn_once("not implemented\n");

  return HSAKMT_STATUS_SUCCESS;
 }
@@ -181,8 +181,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtSetTrapHandler(HSAuint32 Node,
                                             void *TrapBufferBaseAddress,
                                             HSAuint64 TrapBufferSizeInBytes) {
  CHECK_DXG_OPEN();
-
-  pr_debug("%s not implemented\n", __func__);
+  pr_warn_once("not implemented\n");
  return HSAKMT_STATUS_SUCCESS;
 }

@@ -30,6 +30,7 @@
 HSAKMT_STATUS HSAKMTAPI hsaKmtSPMAcquire(HSAuint32 PreferredNode) {
  CHECK_DXG_OPEN();
  // Used for profiling tools
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

@@ -38,11 +39,13 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtSPMSetDestBuffer(
    HSAuint32 *SizeCopied, void *DestMemoryAddress, bool *isSPMDataLoss) {
  CHECK_DXG_OPEN();
  // Used for profiling tools
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

 HSAKMT_STATUS HSAKMTAPI hsaKmtSPMRelease(HSAuint32 PreferredNode) {
  CHECK_DXG_OPEN();
  // Used for profiling tools
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }
@@ -30,6 +30,7 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtSVMSetAttr(void *start_addr, HSAuint64 size,
                                         unsigned int nattr,
                                         HSA_SVM_ATTRIBUTE *attrs) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

@@ -37,16 +38,19 @@ HSAKMT_STATUS HSAKMTAPI hsaKmtSVMGetAttr(void *start_addr, HSAuint64 size,
                                         unsigned int nattr,
                                         HSA_SVM_ATTRIBUTE *attrs) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

 HSAKMT_STATUS HSAKMTAPI hsaKmtSetXNACKMode(HSAint32 enable) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  return HSAKMT_STATUS_NOT_SUPPORTED;
 }

 HSAKMT_STATUS HSAKMTAPI hsaKmtGetXNACKMode(HSAint32 *enable) {
  CHECK_DXG_OPEN();
+  pr_warn_once("not supported\n");
  *enable = false;
  return HSAKMT_STATUS_SUCCESS;
 }