• Oded Gabbay's avatar
    drm/amdkfd: Remove sync_with_hw() from amdkfd · 939f4a20
    Oded Gabbay authored
    This patch completely removes the sync_with_hw() because it was broken and
    actually there is no point of using it.
    
    This function was used to:
    
    - Make sure that the submitted packet to the HIQ (which is a kernel queue) was
      read by the CP. However, it was discovered that the method this function used
      to do that (checking wptr == rptr) is not consistent with how the actual CP
      firmware works in all cases.
    
    - Make sure that the queue is empty before issuing the next packet. To achieve
      that, the function blocked amdkfd from continuing until the recently
      submitted packet was consumed. However, the acquire_packet_buffer() already
      checks if there is enough room for a new packet so calling sync_with_hw() is
      redundant.
    Signed-off-by: default avatarOded Gabbay <oded.gabbay@amd.com>
    Acked-by: default avatarAlex Deucher <alexander.deucher@amd.com>
    939f4a20
kfd_packet_manager.c 15.1 KB