[PATCH 2/3] drm/amdkfd: Remove unused old debugger implementation

Felix Kuehling felix.kuehling at amd.com
Mon Feb 7 15:33:20 UTC 2022


Am 2022-02-04 um 18:45 schrieb Mukul Joshi:
> Cleanup the kfd code by removing the unused old debugger
> implementation.
> Only a small piece of resetting wavefronts is kept and
> is moved to kfd_device_queue_manager.c
>
> Signed-off-by: Mukul Joshi <mukul.joshi at amd.com>

We should update include/uapi/linux/kfd_ioctl.h as well to mark the 
ioctls as deprecated. Just rename the AMDKFD_IOC_DBG_* macros to 
AMDKFD_IOC_DBG_*_DEPRECATED. A corresponding update to the Thunk would 
be good as a follow up.

I think there is also some more code you can remove in 
amdgpu_amdkfd_gfx_v*.c and kgd_kfd_interface.h:

         int (*address_watch_disable)(struct amdgpu_device *adev);
         int (*address_watch_execute)(struct amdgpu_device *adev,
                                         unsigned int watch_point_id,
                                         uint32_t cntl_val,
                                         uint32_t addr_hi,
                                         uint32_t addr_lo);
         uint32_t (*address_watch_get_offset)(struct amdgpu_device *adev,
                                         unsigned int watch_point_id,
                                         unsigned int reg_offset);

It seems wave_control_execute is still used in dbgdev_wave_reset_wavefronts.

Regards,
   Felix


> ---
>   drivers/gpu/drm/amd/amdkfd/Makefile           |   2 -
>   drivers/gpu/drm/amd/amdkfd/kfd_chardev.c      | 282 +-----
>   drivers/gpu/drm/amd/amdkfd/kfd_dbgdev.c       | 845 ------------------
>   drivers/gpu/drm/amd/amdkfd/kfd_dbgdev.h       | 230 -----
>   drivers/gpu/drm/amd/amdkfd/kfd_dbgmgr.c       | 158 ----
>   drivers/gpu/drm/amd/amdkfd/kfd_dbgmgr.h       | 293 ------
>   drivers/gpu/drm/amd/amdkfd/kfd_device.c       |   2 -
>   .../drm/amd/amdkfd/kfd_device_queue_manager.c |  59 ++
>   .../drm/amd/amdkfd/kfd_device_queue_manager.h |  35 +
>   drivers/gpu/drm/amd/amdkfd/kfd_iommu.c        |  12 -
>   drivers/gpu/drm/amd/amdkfd/kfd_priv.h         |   5 -
>   drivers/gpu/drm/amd/amdkfd/kfd_process.c      |  19 -
>   12 files changed, 98 insertions(+), 1844 deletions(-)
>   delete mode 100644 drivers/gpu/drm/amd/amdkfd/kfd_dbgdev.c
>   delete mode 100644 drivers/gpu/drm/amd/amdkfd/kfd_dbgdev.h
>   delete mode 100644 drivers/gpu/drm/amd/amdkfd/kfd_dbgmgr.c
>   delete mode 100644 drivers/gpu/drm/amd/amdkfd/kfd_dbgmgr.h
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/Makefile b/drivers/gpu/drm/amd/amdkfd/Makefile
> index c4f3aff11072..19cfbf9577b4 100644
> --- a/drivers/gpu/drm/amd/amdkfd/Makefile
> +++ b/drivers/gpu/drm/amd/amdkfd/Makefile
> @@ -51,8 +51,6 @@ AMDKFD_FILES	:= $(AMDKFD_PATH)/kfd_module.o \
>   		$(AMDKFD_PATH)/kfd_events.o \
>   		$(AMDKFD_PATH)/cik_event_interrupt.o \
>   		$(AMDKFD_PATH)/kfd_int_process_v9.o \
> -		$(AMDKFD_PATH)/kfd_dbgdev.o \
> -		$(AMDKFD_PATH)/kfd_dbgmgr.o \
>   		$(AMDKFD_PATH)/kfd_smi_events.o \
>   		$(AMDKFD_PATH)/kfd_crat.o
>   
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_chardev.c b/drivers/gpu/drm/amd/amdkfd/kfd_chardev.c
> index 64e3b4e3a712..cfe12525165f 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_chardev.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_chardev.c
> @@ -39,7 +39,6 @@
>   #include <asm/processor.h>
>   #include "kfd_priv.h"
>   #include "kfd_device_queue_manager.h"
> -#include "kfd_dbgmgr.h"
>   #include "kfd_svm.h"
>   #include "amdgpu_amdkfd.h"
>   #include "kfd_smi_events.h"
> @@ -580,299 +579,26 @@ static int kfd_ioctl_set_trap_handler(struct file *filep,
>   static int kfd_ioctl_dbg_register(struct file *filep,
>   				struct kfd_process *p, void *data)
>   {
> -	struct kfd_ioctl_dbg_register_args *args = data;
> -	struct kfd_dev *dev;
> -	struct kfd_dbgmgr *dbgmgr_ptr;
> -	struct kfd_process_device *pdd;
> -	bool create_ok;
> -	long status = 0;
> -
> -	mutex_lock(&p->mutex);
> -	pdd = kfd_process_device_data_by_id(p, args->gpu_id);
> -	if (!pdd) {
> -		status = -EINVAL;
> -		goto err_pdd;
> -	}
> -	dev = pdd->dev;
> -
> -	if (dev->adev->asic_type == CHIP_CARRIZO) {
> -		pr_debug("kfd_ioctl_dbg_register not supported on CZ\n");
> -		status = -EINVAL;
> -		goto err_chip_unsupp;
> -	}
> -
> -	mutex_lock(kfd_get_dbgmgr_mutex());
> -
> -	/*
> -	 * make sure that we have pdd, if this the first queue created for
> -	 * this process
> -	 */
> -	pdd = kfd_bind_process_to_device(dev, p);
> -	if (IS_ERR(pdd)) {
> -		status = PTR_ERR(pdd);
> -		goto out;
> -	}
> -
> -	if (!dev->dbgmgr) {
> -		/* In case of a legal call, we have no dbgmgr yet */
> -		create_ok = kfd_dbgmgr_create(&dbgmgr_ptr, dev);
> -		if (create_ok) {
> -			status = kfd_dbgmgr_register(dbgmgr_ptr, p);
> -			if (status != 0)
> -				kfd_dbgmgr_destroy(dbgmgr_ptr);
> -			else
> -				dev->dbgmgr = dbgmgr_ptr;
> -		}
> -	} else {
> -		pr_debug("debugger already registered\n");
> -		status = -EINVAL;
> -	}
> -
> -out:
> -	mutex_unlock(kfd_get_dbgmgr_mutex());
> -err_pdd:
> -err_chip_unsupp:
> -	mutex_unlock(&p->mutex);
> -
> -	return status;
> +	return -EPERM;
>   }
>   
>   static int kfd_ioctl_dbg_unregister(struct file *filep,
>   				struct kfd_process *p, void *data)
>   {
> -	struct kfd_ioctl_dbg_unregister_args *args = data;
> -	struct kfd_process_device *pdd;
> -	struct kfd_dev *dev;
> -	long status;
> -
> -	mutex_lock(&p->mutex);
> -	pdd = kfd_process_device_data_by_id(p, args->gpu_id);
> -	mutex_unlock(&p->mutex);
> -	if (!pdd || !pdd->dev->dbgmgr)
> -		return -EINVAL;
> -
> -	dev = pdd->dev;
> -
> -	if (dev->adev->asic_type == CHIP_CARRIZO) {
> -		pr_debug("kfd_ioctl_dbg_unregister not supported on CZ\n");
> -		return -EINVAL;
> -	}
> -
> -	mutex_lock(kfd_get_dbgmgr_mutex());
> -
> -	status = kfd_dbgmgr_unregister(dev->dbgmgr, p);
> -	if (!status) {
> -		kfd_dbgmgr_destroy(dev->dbgmgr);
> -		dev->dbgmgr = NULL;
> -	}
> -
> -	mutex_unlock(kfd_get_dbgmgr_mutex());
> -
> -	return status;
> +	return -EPERM;
>   }
>   
> -/*
> - * Parse and generate variable size data structure for address watch.
> - * Total size of the buffer and # watch points is limited in order
> - * to prevent kernel abuse. (no bearing to the much smaller HW limitation
> - * which is enforced by dbgdev module)
> - * please also note that the watch address itself are not "copied from user",
> - * since it be set into the HW in user mode values.
> - *
> - */
>   static int kfd_ioctl_dbg_address_watch(struct file *filep,
>   					struct kfd_process *p, void *data)
>   {
> -	struct kfd_ioctl_dbg_address_watch_args *args = data;
> -	struct kfd_dev *dev;
> -	struct kfd_process_device *pdd;
> -	struct dbg_address_watch_info aw_info;
> -	unsigned char *args_buff;
> -	long status;
> -	void __user *cmd_from_user;
> -	uint64_t watch_mask_value = 0;
> -	unsigned int args_idx = 0;
> -
> -	memset((void *) &aw_info, 0, sizeof(struct dbg_address_watch_info));
> -
> -	mutex_lock(&p->mutex);
> -	pdd = kfd_process_device_data_by_id(p, args->gpu_id);
> -	mutex_unlock(&p->mutex);
> -	if (!pdd) {
> -		pr_debug("Could not find gpu id 0x%x\n", args->gpu_id);
> -		return -EINVAL;
> -	}
> -	dev = pdd->dev;
> -
> -	if (dev->adev->asic_type == CHIP_CARRIZO) {
> -		pr_debug("kfd_ioctl_dbg_wave_control not supported on CZ\n");
> -		return -EINVAL;
> -	}
> -	cmd_from_user = (void __user *) args->content_ptr;
> -
> -	/* Validate arguments */
> -
> -	if ((args->buf_size_in_bytes > MAX_ALLOWED_AW_BUFF_SIZE) ||
> -		(args->buf_size_in_bytes <= sizeof(*args) + sizeof(int) * 2) ||
> -		(cmd_from_user == NULL))
> -		return -EINVAL;
> -
> -	/* this is the actual buffer to work with */
> -	args_buff = memdup_user(cmd_from_user,
> -				args->buf_size_in_bytes - sizeof(*args));
> -	if (IS_ERR(args_buff))
> -		return PTR_ERR(args_buff);
> -
> -	aw_info.process = p;
> -
> -	aw_info.num_watch_points = *((uint32_t *)(&args_buff[args_idx]));
> -	args_idx += sizeof(aw_info.num_watch_points);
> -
> -	aw_info.watch_mode = (enum HSA_DBG_WATCH_MODE *) &args_buff[args_idx];
> -	args_idx += sizeof(enum HSA_DBG_WATCH_MODE) * aw_info.num_watch_points;
> -
> -	/*
> -	 * set watch address base pointer to point on the array base
> -	 * within args_buff
> -	 */
> -	aw_info.watch_address = (uint64_t *) &args_buff[args_idx];
> -
> -	/* skip over the addresses buffer */
> -	args_idx += sizeof(aw_info.watch_address) * aw_info.num_watch_points;
> -
> -	if (args_idx >= args->buf_size_in_bytes - sizeof(*args)) {
> -		status = -EINVAL;
> -		goto out;
> -	}
> -
> -	watch_mask_value = (uint64_t) args_buff[args_idx];
> -
> -	if (watch_mask_value > 0) {
> -		/*
> -		 * There is an array of masks.
> -		 * set watch mask base pointer to point on the array base
> -		 * within args_buff
> -		 */
> -		aw_info.watch_mask = (uint64_t *) &args_buff[args_idx];
> -
> -		/* skip over the masks buffer */
> -		args_idx += sizeof(aw_info.watch_mask) *
> -				aw_info.num_watch_points;
> -	} else {
> -		/* just the NULL mask, set to NULL and skip over it */
> -		aw_info.watch_mask = NULL;
> -		args_idx += sizeof(aw_info.watch_mask);
> -	}
> -
> -	if (args_idx >= args->buf_size_in_bytes - sizeof(args)) {
> -		status = -EINVAL;
> -		goto out;
> -	}
> -
> -	/* Currently HSA Event is not supported for DBG */
> -	aw_info.watch_event = NULL;
> -
> -	mutex_lock(kfd_get_dbgmgr_mutex());
> -
> -	status = kfd_dbgmgr_address_watch(dev->dbgmgr, &aw_info);
> -
> -	mutex_unlock(kfd_get_dbgmgr_mutex());
> -
> -out:
> -	kfree(args_buff);
> -
> -	return status;
> +	return -EPERM;
>   }
>   
>   /* Parse and generate fixed size data structure for wave control */
>   static int kfd_ioctl_dbg_wave_control(struct file *filep,
>   					struct kfd_process *p, void *data)
>   {
> -	struct kfd_ioctl_dbg_wave_control_args *args = data;
> -	struct kfd_dev *dev;
> -	struct kfd_process_device *pdd;
> -	struct dbg_wave_control_info wac_info;
> -	unsigned char *args_buff;
> -	uint32_t computed_buff_size;
> -	long status;
> -	void __user *cmd_from_user;
> -	unsigned int args_idx = 0;
> -
> -	memset((void *) &wac_info, 0, sizeof(struct dbg_wave_control_info));
> -
> -	/* we use compact form, independent of the packing attribute value */
> -	computed_buff_size = sizeof(*args) +
> -				sizeof(wac_info.mode) +
> -				sizeof(wac_info.operand) +
> -				sizeof(wac_info.dbgWave_msg.DbgWaveMsg) +
> -				sizeof(wac_info.dbgWave_msg.MemoryVA) +
> -				sizeof(wac_info.trapId);
> -
> -	mutex_lock(&p->mutex);
> -	pdd = kfd_process_device_data_by_id(p, args->gpu_id);
> -	mutex_unlock(&p->mutex);
> -	if (!pdd) {
> -		pr_debug("Could not find gpu id 0x%x\n", args->gpu_id);
> -		return -EINVAL;
> -	}
> -	dev = pdd->dev;
> -
> -	if (dev->adev->asic_type == CHIP_CARRIZO) {
> -		pr_debug("kfd_ioctl_dbg_wave_control not supported on CZ\n");
> -		return -EINVAL;
> -	}
> -
> -	/* input size must match the computed "compact" size */
> -	if (args->buf_size_in_bytes != computed_buff_size) {
> -		pr_debug("size mismatch, computed : actual %u : %u\n",
> -				args->buf_size_in_bytes, computed_buff_size);
> -		return -EINVAL;
> -	}
> -
> -	cmd_from_user = (void __user *) args->content_ptr;
> -
> -	if (cmd_from_user == NULL)
> -		return -EINVAL;
> -
> -	/* copy the entire buffer from user */
> -
> -	args_buff = memdup_user(cmd_from_user,
> -				args->buf_size_in_bytes - sizeof(*args));
> -	if (IS_ERR(args_buff))
> -		return PTR_ERR(args_buff);
> -
> -	/* move ptr to the start of the "pay-load" area */
> -	wac_info.process = p;
> -
> -	wac_info.operand = *((enum HSA_DBG_WAVEOP *)(&args_buff[args_idx]));
> -	args_idx += sizeof(wac_info.operand);
> -
> -	wac_info.mode = *((enum HSA_DBG_WAVEMODE *)(&args_buff[args_idx]));
> -	args_idx += sizeof(wac_info.mode);
> -
> -	wac_info.trapId = *((uint32_t *)(&args_buff[args_idx]));
> -	args_idx += sizeof(wac_info.trapId);
> -
> -	wac_info.dbgWave_msg.DbgWaveMsg.WaveMsgInfoGen2.Value =
> -					*((uint32_t *)(&args_buff[args_idx]));
> -	wac_info.dbgWave_msg.MemoryVA = NULL;
> -
> -	mutex_lock(kfd_get_dbgmgr_mutex());
> -
> -	pr_debug("Calling dbg manager process %p, operand %u, mode %u, trapId %u, message %u\n",
> -			wac_info.process, wac_info.operand,
> -			wac_info.mode, wac_info.trapId,
> -			wac_info.dbgWave_msg.DbgWaveMsg.WaveMsgInfoGen2.Value);
> -
> -	status = kfd_dbgmgr_wave_control(dev->dbgmgr, &wac_info);
> -
> -	pr_debug("Returned status of dbg manager is %ld\n", status);
> -
> -	mutex_unlock(kfd_get_dbgmgr_mutex());
> -
> -	kfree(args_buff);
> -
> -	return status;
> +	return -EPERM;
>   }
>   
>   static int kfd_ioctl_get_clock_counters(struct file *filep,
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_dbgdev.c b/drivers/gpu/drm/amd/amdkfd/kfd_dbgdev.c
> deleted file mode 100644
> index 8eca9ed3ab36..000000000000
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_dbgdev.c
> +++ /dev/null
> @@ -1,845 +0,0 @@
> -/*
> - * Copyright 2014 Advanced Micro Devices, Inc.
> - *
> - * Permission is hereby granted, free of charge, to any person obtaining a
> - * copy of this software and associated documentation files (the "Software"),
> - * to deal in the Software without restriction, including without limitation
> - * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> - * and/or sell copies of the Software, and to permit persons to whom the
> - * Software is furnished to do so, subject to the following conditions:
> - *
> - * The above copyright notice and this permission notice shall be included in
> - * all copies or substantial portions of the Software.
> - *
> - * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> - * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> - * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> - * THE COPYRIGHT HOLDER(S) OR AUTHOR(S) BE LIABLE FOR ANY CLAIM, DAMAGES OR
> - * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
> - * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
> - * OTHER DEALINGS IN THE SOFTWARE.
> - *
> - */
> -
> -#include <linux/types.h>
> -#include <linux/kernel.h>
> -#include <linux/log2.h>
> -#include <linux/sched.h>
> -#include <linux/slab.h>
> -#include <linux/mutex.h>
> -#include <linux/device.h>
> -
> -#include "kfd_pm4_headers.h"
> -#include "kfd_pm4_headers_diq.h"
> -#include "kfd_kernel_queue.h"
> -#include "kfd_priv.h"
> -#include "kfd_pm4_opcodes.h"
> -#include "cik_regs.h"
> -#include "kfd_dbgmgr.h"
> -#include "kfd_dbgdev.h"
> -#include "kfd_device_queue_manager.h"
> -
> -static void dbgdev_address_watch_disable_nodiq(struct kfd_dev *dev)
> -{
> -	dev->kfd2kgd->address_watch_disable(dev->adev);
> -}
> -
> -static int dbgdev_diq_submit_ib(struct kfd_dbgdev *dbgdev,
> -				u32 pasid, uint64_t vmid0_address,
> -				uint32_t *packet_buff, size_t size_in_bytes)
> -{
> -	struct pm4__release_mem *rm_packet;
> -	struct pm4__indirect_buffer_pasid *ib_packet;
> -	struct kfd_mem_obj *mem_obj;
> -	size_t pq_packets_size_in_bytes;
> -	union ULARGE_INTEGER *largep;
> -	union ULARGE_INTEGER addr;
> -	struct kernel_queue *kq;
> -	uint64_t *rm_state;
> -	unsigned int *ib_packet_buff;
> -	int status;
> -
> -	if (WARN_ON(!size_in_bytes))
> -		return -EINVAL;
> -
> -	kq = dbgdev->kq;
> -
> -	pq_packets_size_in_bytes = sizeof(struct pm4__release_mem) +
> -				sizeof(struct pm4__indirect_buffer_pasid);
> -
> -	/*
> -	 * We acquire a buffer from DIQ
> -	 * The receive packet buff will be sitting on the Indirect Buffer
> -	 * and in the PQ we put the IB packet + sync packet(s).
> -	 */
> -	status = kq_acquire_packet_buffer(kq,
> -				pq_packets_size_in_bytes / sizeof(uint32_t),
> -				&ib_packet_buff);
> -	if (status) {
> -		pr_err("kq_acquire_packet_buffer failed\n");
> -		return status;
> -	}
> -
> -	memset(ib_packet_buff, 0, pq_packets_size_in_bytes);
> -
> -	ib_packet = (struct pm4__indirect_buffer_pasid *) (ib_packet_buff);
> -
> -	ib_packet->header.count = 3;
> -	ib_packet->header.opcode = IT_INDIRECT_BUFFER_PASID;
> -	ib_packet->header.type = PM4_TYPE_3;
> -
> -	largep = (union ULARGE_INTEGER *) &vmid0_address;
> -
> -	ib_packet->bitfields2.ib_base_lo = largep->u.low_part >> 2;
> -	ib_packet->bitfields3.ib_base_hi = largep->u.high_part;
> -
> -	ib_packet->control = (1 << 23) | (1 << 31) |
> -			((size_in_bytes / 4) & 0xfffff);
> -
> -	ib_packet->bitfields5.pasid = pasid;
> -
> -	/*
> -	 * for now we use release mem for GPU-CPU synchronization
> -	 * Consider WaitRegMem + WriteData as a better alternative
> -	 * we get a GART allocations ( gpu/cpu mapping),
> -	 * for the sync variable, and wait until:
> -	 * (a) Sync with HW
> -	 * (b) Sync var is written by CP to mem.
> -	 */
> -	rm_packet = (struct pm4__release_mem *) (ib_packet_buff +
> -			(sizeof(struct pm4__indirect_buffer_pasid) /
> -					sizeof(unsigned int)));
> -
> -	status = kfd_gtt_sa_allocate(dbgdev->dev, sizeof(uint64_t),
> -					&mem_obj);
> -
> -	if (status) {
> -		pr_err("Failed to allocate GART memory\n");
> -		kq_rollback_packet(kq);
> -		return status;
> -	}
> -
> -	rm_state = (uint64_t *) mem_obj->cpu_ptr;
> -
> -	*rm_state = QUEUESTATE__ACTIVE_COMPLETION_PENDING;
> -
> -	rm_packet->header.opcode = IT_RELEASE_MEM;
> -	rm_packet->header.type = PM4_TYPE_3;
> -	rm_packet->header.count = sizeof(struct pm4__release_mem) / 4 - 2;
> -
> -	rm_packet->bitfields2.event_type = CACHE_FLUSH_AND_INV_TS_EVENT;
> -	rm_packet->bitfields2.event_index =
> -				event_index___release_mem__end_of_pipe;
> -
> -	rm_packet->bitfields2.cache_policy = cache_policy___release_mem__lru;
> -	rm_packet->bitfields2.atc = 0;
> -	rm_packet->bitfields2.tc_wb_action_ena = 1;
> -
> -	addr.quad_part = mem_obj->gpu_addr;
> -
> -	rm_packet->bitfields4.address_lo_32b = addr.u.low_part >> 2;
> -	rm_packet->address_hi = addr.u.high_part;
> -
> -	rm_packet->bitfields3.data_sel =
> -				data_sel___release_mem__send_64_bit_data;
> -
> -	rm_packet->bitfields3.int_sel =
> -			int_sel___release_mem__send_data_after_write_confirm;
> -
> -	rm_packet->bitfields3.dst_sel =
> -			dst_sel___release_mem__memory_controller;
> -
> -	rm_packet->data_lo = QUEUESTATE__ACTIVE;
> -
> -	kq_submit_packet(kq);
> -
> -	/* Wait till CP writes sync code: */
> -	status = amdkfd_fence_wait_timeout(
> -			rm_state,
> -			QUEUESTATE__ACTIVE, 1500);
> -
> -	kfd_gtt_sa_free(dbgdev->dev, mem_obj);
> -
> -	return status;
> -}
> -
> -static int dbgdev_register_nodiq(struct kfd_dbgdev *dbgdev)
> -{
> -	/*
> -	 * no action is needed in this case,
> -	 * just make sure diq will not be used
> -	 */
> -
> -	dbgdev->kq = NULL;
> -
> -	return 0;
> -}
> -
> -static int dbgdev_register_diq(struct kfd_dbgdev *dbgdev)
> -{
> -	struct queue_properties properties;
> -	unsigned int qid;
> -	struct kernel_queue *kq = NULL;
> -	int status;
> -
> -	properties.type = KFD_QUEUE_TYPE_DIQ;
> -
> -	status = pqm_create_queue(dbgdev->pqm, dbgdev->dev, NULL,
> -				&properties, &qid, NULL, NULL, NULL, NULL);
> -
> -	if (status) {
> -		pr_err("Failed to create DIQ\n");
> -		return status;
> -	}
> -
> -	pr_debug("DIQ Created with queue id: %d\n", qid);
> -
> -	kq = pqm_get_kernel_queue(dbgdev->pqm, qid);
> -
> -	if (!kq) {
> -		pr_err("Error getting DIQ\n");
> -		pqm_destroy_queue(dbgdev->pqm, qid);
> -		return -EFAULT;
> -	}
> -
> -	dbgdev->kq = kq;
> -
> -	return status;
> -}
> -
> -static int dbgdev_unregister_nodiq(struct kfd_dbgdev *dbgdev)
> -{
> -	/* disable watch address */
> -	dbgdev_address_watch_disable_nodiq(dbgdev->dev);
> -	return 0;
> -}
> -
> -static int dbgdev_unregister_diq(struct kfd_dbgdev *dbgdev)
> -{
> -	/* todo - disable address watch */
> -	int status;
> -
> -	status = pqm_destroy_queue(dbgdev->pqm,
> -			dbgdev->kq->queue->properties.queue_id);
> -	dbgdev->kq = NULL;
> -
> -	return status;
> -}
> -
> -static void dbgdev_address_watch_set_registers(
> -			const struct dbg_address_watch_info *adw_info,
> -			union TCP_WATCH_ADDR_H_BITS *addrHi,
> -			union TCP_WATCH_ADDR_L_BITS *addrLo,
> -			union TCP_WATCH_CNTL_BITS *cntl,
> -			unsigned int index, unsigned int vmid)
> -{
> -	union ULARGE_INTEGER addr;
> -
> -	addr.quad_part = 0;
> -	addrHi->u32All = 0;
> -	addrLo->u32All = 0;
> -	cntl->u32All = 0;
> -
> -	if (adw_info->watch_mask)
> -		cntl->bitfields.mask =
> -			(uint32_t) (adw_info->watch_mask[index] &
> -					ADDRESS_WATCH_REG_CNTL_DEFAULT_MASK);
> -	else
> -		cntl->bitfields.mask = ADDRESS_WATCH_REG_CNTL_DEFAULT_MASK;
> -
> -	addr.quad_part = (unsigned long long) adw_info->watch_address[index];
> -
> -	addrHi->bitfields.addr = addr.u.high_part &
> -					ADDRESS_WATCH_REG_ADDHIGH_MASK;
> -	addrLo->bitfields.addr =
> -			(addr.u.low_part >> ADDRESS_WATCH_REG_ADDLOW_SHIFT);
> -
> -	cntl->bitfields.mode = adw_info->watch_mode[index];
> -	cntl->bitfields.vmid = (uint32_t) vmid;
> -	/* for now assume it is an ATC address */
> -	cntl->u32All |= ADDRESS_WATCH_REG_CNTL_ATC_BIT;
> -
> -	pr_debug("\t\t%20s %08x\n", "set reg mask :", cntl->bitfields.mask);
> -	pr_debug("\t\t%20s %08x\n", "set reg add high :",
> -			addrHi->bitfields.addr);
> -	pr_debug("\t\t%20s %08x\n", "set reg add low :",
> -			addrLo->bitfields.addr);
> -}
> -
> -static int dbgdev_address_watch_nodiq(struct kfd_dbgdev *dbgdev,
> -				      struct dbg_address_watch_info *adw_info)
> -{
> -	union TCP_WATCH_ADDR_H_BITS addrHi;
> -	union TCP_WATCH_ADDR_L_BITS addrLo;
> -	union TCP_WATCH_CNTL_BITS cntl;
> -	struct kfd_process_device *pdd;
> -	unsigned int i;
> -
> -	/* taking the vmid for that process on the safe way using pdd */
> -	pdd = kfd_get_process_device_data(dbgdev->dev,
> -					adw_info->process);
> -	if (!pdd) {
> -		pr_err("Failed to get pdd for wave control no DIQ\n");
> -		return -EFAULT;
> -	}
> -
> -	addrHi.u32All = 0;
> -	addrLo.u32All = 0;
> -	cntl.u32All = 0;
> -
> -	if ((adw_info->num_watch_points > MAX_WATCH_ADDRESSES) ||
> -			(adw_info->num_watch_points == 0)) {
> -		pr_err("num_watch_points is invalid\n");
> -		return -EINVAL;
> -	}
> -
> -	if (!adw_info->watch_mode || !adw_info->watch_address) {
> -		pr_err("adw_info fields are not valid\n");
> -		return -EINVAL;
> -	}
> -
> -	for (i = 0; i < adw_info->num_watch_points; i++) {
> -		dbgdev_address_watch_set_registers(adw_info, &addrHi, &addrLo,
> -						&cntl, i, pdd->qpd.vmid);
> -
> -		pr_debug("\t\t%30s\n", "* * * * * * * * * * * * * * * * * *");
> -		pr_debug("\t\t%20s %08x\n", "register index :", i);
> -		pr_debug("\t\t%20s %08x\n", "vmid is :", pdd->qpd.vmid);
> -		pr_debug("\t\t%20s %08x\n", "Address Low is :",
> -				addrLo.bitfields.addr);
> -		pr_debug("\t\t%20s %08x\n", "Address high is :",
> -				addrHi.bitfields.addr);
> -		pr_debug("\t\t%20s %08x\n", "Address high is :",
> -				addrHi.bitfields.addr);
> -		pr_debug("\t\t%20s %08x\n", "Control Mask is :",
> -				cntl.bitfields.mask);
> -		pr_debug("\t\t%20s %08x\n", "Control Mode is :",
> -				cntl.bitfields.mode);
> -		pr_debug("\t\t%20s %08x\n", "Control Vmid is :",
> -				cntl.bitfields.vmid);
> -		pr_debug("\t\t%20s %08x\n", "Control atc  is :",
> -				cntl.bitfields.atc);
> -		pr_debug("\t\t%30s\n", "* * * * * * * * * * * * * * * * * *");
> -
> -		pdd->dev->kfd2kgd->address_watch_execute(
> -						dbgdev->dev->adev,
> -						i,
> -						cntl.u32All,
> -						addrHi.u32All,
> -						addrLo.u32All);
> -	}
> -
> -	return 0;
> -}
> -
> -static int dbgdev_address_watch_diq(struct kfd_dbgdev *dbgdev,
> -				    struct dbg_address_watch_info *adw_info)
> -{
> -	struct pm4__set_config_reg *packets_vec;
> -	union TCP_WATCH_ADDR_H_BITS addrHi;
> -	union TCP_WATCH_ADDR_L_BITS addrLo;
> -	union TCP_WATCH_CNTL_BITS cntl;
> -	struct kfd_mem_obj *mem_obj;
> -	unsigned int aw_reg_add_dword;
> -	uint32_t *packet_buff_uint;
> -	unsigned int i;
> -	int status;
> -	size_t ib_size = sizeof(struct pm4__set_config_reg) * 4;
> -	/* we do not control the vmid in DIQ mode, just a place holder */
> -	unsigned int vmid = 0;
> -
> -	addrHi.u32All = 0;
> -	addrLo.u32All = 0;
> -	cntl.u32All = 0;
> -
> -	if ((adw_info->num_watch_points > MAX_WATCH_ADDRESSES) ||
> -			(adw_info->num_watch_points == 0)) {
> -		pr_err("num_watch_points is invalid\n");
> -		return -EINVAL;
> -	}
> -
> -	if (!adw_info->watch_mode || !adw_info->watch_address) {
> -		pr_err("adw_info fields are not valid\n");
> -		return -EINVAL;
> -	}
> -
> -	status = kfd_gtt_sa_allocate(dbgdev->dev, ib_size, &mem_obj);
> -
> -	if (status) {
> -		pr_err("Failed to allocate GART memory\n");
> -		return status;
> -	}
> -
> -	packet_buff_uint = mem_obj->cpu_ptr;
> -
> -	memset(packet_buff_uint, 0, ib_size);
> -
> -	packets_vec = (struct pm4__set_config_reg *) (packet_buff_uint);
> -
> -	packets_vec[0].header.count = 1;
> -	packets_vec[0].header.opcode = IT_SET_CONFIG_REG;
> -	packets_vec[0].header.type = PM4_TYPE_3;
> -	packets_vec[0].bitfields2.vmid_shift = ADDRESS_WATCH_CNTL_OFFSET;
> -	packets_vec[0].bitfields2.insert_vmid = 1;
> -	packets_vec[1].ordinal1 = packets_vec[0].ordinal1;
> -	packets_vec[1].bitfields2.insert_vmid = 0;
> -	packets_vec[2].ordinal1 = packets_vec[0].ordinal1;
> -	packets_vec[2].bitfields2.insert_vmid = 0;
> -	packets_vec[3].ordinal1 = packets_vec[0].ordinal1;
> -	packets_vec[3].bitfields2.vmid_shift = ADDRESS_WATCH_CNTL_OFFSET;
> -	packets_vec[3].bitfields2.insert_vmid = 1;
> -
> -	for (i = 0; i < adw_info->num_watch_points; i++) {
> -		dbgdev_address_watch_set_registers(adw_info,
> -						&addrHi,
> -						&addrLo,
> -						&cntl,
> -						i,
> -						vmid);
> -
> -		pr_debug("\t\t%30s\n", "* * * * * * * * * * * * * * * * * *");
> -		pr_debug("\t\t%20s %08x\n", "register index :", i);
> -		pr_debug("\t\t%20s %08x\n", "vmid is :", vmid);
> -		pr_debug("\t\t%20s %p\n", "Add ptr is :",
> -				adw_info->watch_address);
> -		pr_debug("\t\t%20s %08llx\n", "Add     is :",
> -				adw_info->watch_address[i]);
> -		pr_debug("\t\t%20s %08x\n", "Address Low is :",
> -				addrLo.bitfields.addr);
> -		pr_debug("\t\t%20s %08x\n", "Address high is :",
> -				addrHi.bitfields.addr);
> -		pr_debug("\t\t%20s %08x\n", "Control Mask is :",
> -				cntl.bitfields.mask);
> -		pr_debug("\t\t%20s %08x\n", "Control Mode is :",
> -				cntl.bitfields.mode);
> -		pr_debug("\t\t%20s %08x\n", "Control Vmid is :",
> -				cntl.bitfields.vmid);
> -		pr_debug("\t\t%20s %08x\n", "Control atc  is :",
> -				cntl.bitfields.atc);
> -		pr_debug("\t\t%30s\n", "* * * * * * * * * * * * * * * * * *");
> -
> -		aw_reg_add_dword =
> -				dbgdev->dev->kfd2kgd->address_watch_get_offset(
> -					dbgdev->dev->adev,
> -					i,
> -					ADDRESS_WATCH_REG_CNTL);
> -
> -		packets_vec[0].bitfields2.reg_offset =
> -					aw_reg_add_dword - AMD_CONFIG_REG_BASE;
> -
> -		packets_vec[0].reg_data[0] = cntl.u32All;
> -
> -		aw_reg_add_dword =
> -				dbgdev->dev->kfd2kgd->address_watch_get_offset(
> -					dbgdev->dev->adev,
> -					i,
> -					ADDRESS_WATCH_REG_ADDR_HI);
> -
> -		packets_vec[1].bitfields2.reg_offset =
> -					aw_reg_add_dword - AMD_CONFIG_REG_BASE;
> -		packets_vec[1].reg_data[0] = addrHi.u32All;
> -
> -		aw_reg_add_dword =
> -				dbgdev->dev->kfd2kgd->address_watch_get_offset(
> -					dbgdev->dev->adev,
> -					i,
> -					ADDRESS_WATCH_REG_ADDR_LO);
> -
> -		packets_vec[2].bitfields2.reg_offset =
> -				aw_reg_add_dword - AMD_CONFIG_REG_BASE;
> -		packets_vec[2].reg_data[0] = addrLo.u32All;
> -
> -		/* enable watch flag if address is not zero*/
> -		if (adw_info->watch_address[i] > 0)
> -			cntl.bitfields.valid = 1;
> -		else
> -			cntl.bitfields.valid = 0;
> -
> -		aw_reg_add_dword =
> -				dbgdev->dev->kfd2kgd->address_watch_get_offset(
> -					dbgdev->dev->adev,
> -					i,
> -					ADDRESS_WATCH_REG_CNTL);
> -
> -		packets_vec[3].bitfields2.reg_offset =
> -					aw_reg_add_dword - AMD_CONFIG_REG_BASE;
> -		packets_vec[3].reg_data[0] = cntl.u32All;
> -
> -		status = dbgdev_diq_submit_ib(
> -					dbgdev,
> -					adw_info->process->pasid,
> -					mem_obj->gpu_addr,
> -					packet_buff_uint,
> -					ib_size);
> -
> -		if (status) {
> -			pr_err("Failed to submit IB to DIQ\n");
> -			break;
> -		}
> -	}
> -
> -	kfd_gtt_sa_free(dbgdev->dev, mem_obj);
> -	return status;
> -}
> -
> -static int dbgdev_wave_control_set_registers(
> -				struct dbg_wave_control_info *wac_info,
> -				union SQ_CMD_BITS *in_reg_sq_cmd,
> -				union GRBM_GFX_INDEX_BITS *in_reg_gfx_index)
> -{
> -	int status = 0;
> -	union SQ_CMD_BITS reg_sq_cmd;
> -	union GRBM_GFX_INDEX_BITS reg_gfx_index;
> -	struct HsaDbgWaveMsgAMDGen2 *pMsg;
> -
> -	reg_sq_cmd.u32All = 0;
> -	reg_gfx_index.u32All = 0;
> -	pMsg = &wac_info->dbgWave_msg.DbgWaveMsg.WaveMsgInfoGen2;
> -
> -	switch (wac_info->mode) {
> -	/* Send command to single wave */
> -	case HSA_DBG_WAVEMODE_SINGLE:
> -		/*
> -		 * Limit access to the process waves only,
> -		 * by setting vmid check
> -		 */
> -		reg_sq_cmd.bits.check_vmid = 1;
> -		reg_sq_cmd.bits.simd_id = pMsg->ui32.SIMD;
> -		reg_sq_cmd.bits.wave_id = pMsg->ui32.WaveId;
> -		reg_sq_cmd.bits.mode = SQ_IND_CMD_MODE_SINGLE;
> -
> -		reg_gfx_index.bits.sh_index = pMsg->ui32.ShaderArray;
> -		reg_gfx_index.bits.se_index = pMsg->ui32.ShaderEngine;
> -		reg_gfx_index.bits.instance_index = pMsg->ui32.HSACU;
> -
> -		break;
> -
> -	/* Send command to all waves with matching VMID */
> -	case HSA_DBG_WAVEMODE_BROADCAST_PROCESS:
> -
> -		reg_gfx_index.bits.sh_broadcast_writes = 1;
> -		reg_gfx_index.bits.se_broadcast_writes = 1;
> -		reg_gfx_index.bits.instance_broadcast_writes = 1;
> -
> -		reg_sq_cmd.bits.mode = SQ_IND_CMD_MODE_BROADCAST;
> -
> -		break;
> -
> -	/* Send command to all CU waves with matching VMID */
> -	case HSA_DBG_WAVEMODE_BROADCAST_PROCESS_CU:
> -
> -		reg_sq_cmd.bits.check_vmid = 1;
> -		reg_sq_cmd.bits.mode = SQ_IND_CMD_MODE_BROADCAST;
> -
> -		reg_gfx_index.bits.sh_index = pMsg->ui32.ShaderArray;
> -		reg_gfx_index.bits.se_index = pMsg->ui32.ShaderEngine;
> -		reg_gfx_index.bits.instance_index = pMsg->ui32.HSACU;
> -
> -		break;
> -
> -	default:
> -		return -EINVAL;
> -	}
> -
> -	switch (wac_info->operand) {
> -	case HSA_DBG_WAVEOP_HALT:
> -		reg_sq_cmd.bits.cmd = SQ_IND_CMD_CMD_HALT;
> -		break;
> -
> -	case HSA_DBG_WAVEOP_RESUME:
> -		reg_sq_cmd.bits.cmd = SQ_IND_CMD_CMD_RESUME;
> -		break;
> -
> -	case HSA_DBG_WAVEOP_KILL:
> -		reg_sq_cmd.bits.cmd = SQ_IND_CMD_CMD_KILL;
> -		break;
> -
> -	case HSA_DBG_WAVEOP_DEBUG:
> -		reg_sq_cmd.bits.cmd = SQ_IND_CMD_CMD_DEBUG;
> -		break;
> -
> -	case HSA_DBG_WAVEOP_TRAP:
> -		if (wac_info->trapId < MAX_TRAPID) {
> -			reg_sq_cmd.bits.cmd = SQ_IND_CMD_CMD_TRAP;
> -			reg_sq_cmd.bits.trap_id = wac_info->trapId;
> -		} else {
> -			status = -EINVAL;
> -		}
> -		break;
> -
> -	default:
> -		status = -EINVAL;
> -		break;
> -	}
> -
> -	if (status == 0) {
> -		*in_reg_sq_cmd = reg_sq_cmd;
> -		*in_reg_gfx_index = reg_gfx_index;
> -	}
> -
> -	return status;
> -}
> -
> -static int dbgdev_wave_control_diq(struct kfd_dbgdev *dbgdev,
> -					struct dbg_wave_control_info *wac_info)
> -{
> -
> -	int status;
> -	union SQ_CMD_BITS reg_sq_cmd;
> -	union GRBM_GFX_INDEX_BITS reg_gfx_index;
> -	struct kfd_mem_obj *mem_obj;
> -	uint32_t *packet_buff_uint;
> -	struct pm4__set_config_reg *packets_vec;
> -	size_t ib_size = sizeof(struct pm4__set_config_reg) * 3;
> -
> -	reg_sq_cmd.u32All = 0;
> -
> -	status = dbgdev_wave_control_set_registers(wac_info, &reg_sq_cmd,
> -							&reg_gfx_index);
> -	if (status) {
> -		pr_err("Failed to set wave control registers\n");
> -		return status;
> -	}
> -
> -	/* we do not control the VMID in DIQ, so reset it to a known value */
> -	reg_sq_cmd.bits.vm_id = 0;
> -
> -	pr_debug("\t\t %30s\n", "* * * * * * * * * * * * * * * * * *");
> -
> -	pr_debug("\t\t mode      is: %u\n", wac_info->mode);
> -	pr_debug("\t\t operand   is: %u\n", wac_info->operand);
> -	pr_debug("\t\t trap id   is: %u\n", wac_info->trapId);
> -	pr_debug("\t\t msg value is: %u\n",
> -			wac_info->dbgWave_msg.DbgWaveMsg.WaveMsgInfoGen2.Value);
> -	pr_debug("\t\t vmid      is: N/A\n");
> -
> -	pr_debug("\t\t chk_vmid  is : %u\n", reg_sq_cmd.bitfields.check_vmid);
> -	pr_debug("\t\t command   is : %u\n", reg_sq_cmd.bitfields.cmd);
> -	pr_debug("\t\t queue id  is : %u\n", reg_sq_cmd.bitfields.queue_id);
> -	pr_debug("\t\t simd id   is : %u\n", reg_sq_cmd.bitfields.simd_id);
> -	pr_debug("\t\t mode      is : %u\n", reg_sq_cmd.bitfields.mode);
> -	pr_debug("\t\t vm_id     is : %u\n", reg_sq_cmd.bitfields.vm_id);
> -	pr_debug("\t\t wave_id   is : %u\n", reg_sq_cmd.bitfields.wave_id);
> -
> -	pr_debug("\t\t ibw       is : %u\n",
> -			reg_gfx_index.bitfields.instance_broadcast_writes);
> -	pr_debug("\t\t ii        is : %u\n",
> -			reg_gfx_index.bitfields.instance_index);
> -	pr_debug("\t\t sebw      is : %u\n",
> -			reg_gfx_index.bitfields.se_broadcast_writes);
> -	pr_debug("\t\t se_ind    is : %u\n", reg_gfx_index.bitfields.se_index);
> -	pr_debug("\t\t sh_ind    is : %u\n", reg_gfx_index.bitfields.sh_index);
> -	pr_debug("\t\t sbw       is : %u\n",
> -			reg_gfx_index.bitfields.sh_broadcast_writes);
> -
> -	pr_debug("\t\t %30s\n", "* * * * * * * * * * * * * * * * * *");
> -
> -	status = kfd_gtt_sa_allocate(dbgdev->dev, ib_size, &mem_obj);
> -
> -	if (status != 0) {
> -		pr_err("Failed to allocate GART memory\n");
> -		return status;
> -	}
> -
> -	packet_buff_uint = mem_obj->cpu_ptr;
> -
> -	memset(packet_buff_uint, 0, ib_size);
> -
> -	packets_vec =  (struct pm4__set_config_reg *) packet_buff_uint;
> -	packets_vec[0].header.count = 1;
> -	packets_vec[0].header.opcode = IT_SET_UCONFIG_REG;
> -	packets_vec[0].header.type = PM4_TYPE_3;
> -	packets_vec[0].bitfields2.reg_offset =
> -			GRBM_GFX_INDEX / 4 - USERCONFIG_REG_BASE;
> -
> -	packets_vec[0].bitfields2.insert_vmid = 0;
> -	packets_vec[0].reg_data[0] = reg_gfx_index.u32All;
> -
> -	packets_vec[1].header.count = 1;
> -	packets_vec[1].header.opcode = IT_SET_CONFIG_REG;
> -	packets_vec[1].header.type = PM4_TYPE_3;
> -	packets_vec[1].bitfields2.reg_offset = SQ_CMD / 4 - AMD_CONFIG_REG_BASE;
> -
> -	packets_vec[1].bitfields2.vmid_shift = SQ_CMD_VMID_OFFSET;
> -	packets_vec[1].bitfields2.insert_vmid = 1;
> -	packets_vec[1].reg_data[0] = reg_sq_cmd.u32All;
> -
> -	/* Restore the GRBM_GFX_INDEX register */
> -
> -	reg_gfx_index.u32All = 0;
> -	reg_gfx_index.bits.sh_broadcast_writes = 1;
> -	reg_gfx_index.bits.instance_broadcast_writes = 1;
> -	reg_gfx_index.bits.se_broadcast_writes = 1;
> -
> -
> -	packets_vec[2].ordinal1 = packets_vec[0].ordinal1;
> -	packets_vec[2].bitfields2.reg_offset =
> -				GRBM_GFX_INDEX / 4 - USERCONFIG_REG_BASE;
> -
> -	packets_vec[2].bitfields2.insert_vmid = 0;
> -	packets_vec[2].reg_data[0] = reg_gfx_index.u32All;
> -
> -	status = dbgdev_diq_submit_ib(
> -			dbgdev,
> -			wac_info->process->pasid,
> -			mem_obj->gpu_addr,
> -			packet_buff_uint,
> -			ib_size);
> -
> -	if (status)
> -		pr_err("Failed to submit IB to DIQ\n");
> -
> -	kfd_gtt_sa_free(dbgdev->dev, mem_obj);
> -
> -	return status;
> -}
> -
> -static int dbgdev_wave_control_nodiq(struct kfd_dbgdev *dbgdev,
> -					struct dbg_wave_control_info *wac_info)
> -{
> -	int status;
> -	union SQ_CMD_BITS reg_sq_cmd;
> -	union GRBM_GFX_INDEX_BITS reg_gfx_index;
> -	struct kfd_process_device *pdd;
> -
> -	reg_sq_cmd.u32All = 0;
> -
> -	/* taking the VMID for that process on the safe way using PDD */
> -	pdd = kfd_get_process_device_data(dbgdev->dev, wac_info->process);
> -
> -	if (!pdd) {
> -		pr_err("Failed to get pdd for wave control no DIQ\n");
> -		return -EFAULT;
> -	}
> -	status = dbgdev_wave_control_set_registers(wac_info, &reg_sq_cmd,
> -							&reg_gfx_index);
> -	if (status) {
> -		pr_err("Failed to set wave control registers\n");
> -		return status;
> -	}
> -
> -	/* for non DIQ we need to patch the VMID: */
> -
> -	reg_sq_cmd.bits.vm_id = pdd->qpd.vmid;
> -
> -	pr_debug("\t\t %30s\n", "* * * * * * * * * * * * * * * * * *");
> -
> -	pr_debug("\t\t mode      is: %u\n", wac_info->mode);
> -	pr_debug("\t\t operand   is: %u\n", wac_info->operand);
> -	pr_debug("\t\t trap id   is: %u\n", wac_info->trapId);
> -	pr_debug("\t\t msg value is: %u\n",
> -			wac_info->dbgWave_msg.DbgWaveMsg.WaveMsgInfoGen2.Value);
> -	pr_debug("\t\t vmid      is: %u\n", pdd->qpd.vmid);
> -
> -	pr_debug("\t\t chk_vmid  is : %u\n", reg_sq_cmd.bitfields.check_vmid);
> -	pr_debug("\t\t command   is : %u\n", reg_sq_cmd.bitfields.cmd);
> -	pr_debug("\t\t queue id  is : %u\n", reg_sq_cmd.bitfields.queue_id);
> -	pr_debug("\t\t simd id   is : %u\n", reg_sq_cmd.bitfields.simd_id);
> -	pr_debug("\t\t mode      is : %u\n", reg_sq_cmd.bitfields.mode);
> -	pr_debug("\t\t vm_id     is : %u\n", reg_sq_cmd.bitfields.vm_id);
> -	pr_debug("\t\t wave_id   is : %u\n", reg_sq_cmd.bitfields.wave_id);
> -
> -	pr_debug("\t\t ibw       is : %u\n",
> -			reg_gfx_index.bitfields.instance_broadcast_writes);
> -	pr_debug("\t\t ii        is : %u\n",
> -			reg_gfx_index.bitfields.instance_index);
> -	pr_debug("\t\t sebw      is : %u\n",
> -			reg_gfx_index.bitfields.se_broadcast_writes);
> -	pr_debug("\t\t se_ind    is : %u\n", reg_gfx_index.bitfields.se_index);
> -	pr_debug("\t\t sh_ind    is : %u\n", reg_gfx_index.bitfields.sh_index);
> -	pr_debug("\t\t sbw       is : %u\n",
> -			reg_gfx_index.bitfields.sh_broadcast_writes);
> -
> -	pr_debug("\t\t %30s\n", "* * * * * * * * * * * * * * * * * *");
> -
> -	return dbgdev->dev->kfd2kgd->wave_control_execute(dbgdev->dev->adev,
> -							reg_gfx_index.u32All,
> -							reg_sq_cmd.u32All);
> -}
> -
> -int dbgdev_wave_reset_wavefronts(struct kfd_dev *dev, struct kfd_process *p)
> -{
> -	int status = 0;
> -	unsigned int vmid;
> -	uint16_t queried_pasid;
> -	union SQ_CMD_BITS reg_sq_cmd;
> -	union GRBM_GFX_INDEX_BITS reg_gfx_index;
> -	struct kfd_process_device *pdd;
> -	struct dbg_wave_control_info wac_info;
> -	int first_vmid_to_scan = dev->vm_info.first_vmid_kfd;
> -	int last_vmid_to_scan = dev->vm_info.last_vmid_kfd;
> -
> -	reg_sq_cmd.u32All = 0;
> -	status = 0;
> -
> -	wac_info.mode = HSA_DBG_WAVEMODE_BROADCAST_PROCESS;
> -	wac_info.operand = HSA_DBG_WAVEOP_KILL;
> -
> -	pr_debug("Killing all process wavefronts\n");
> -
> -	/* Scan all registers in the range ATC_VMID8_PASID_MAPPING ..
> -	 * ATC_VMID15_PASID_MAPPING
> -	 * to check which VMID the current process is mapped to.
> -	 */
> -
> -	for (vmid = first_vmid_to_scan; vmid <= last_vmid_to_scan; vmid++) {
> -		status = dev->kfd2kgd->get_atc_vmid_pasid_mapping_info
> -				(dev->adev, vmid, &queried_pasid);
> -
> -		if (status && queried_pasid == p->pasid) {
> -			pr_debug("Killing wave fronts of vmid %d and pasid 0x%x\n",
> -					vmid, p->pasid);
> -			break;
> -		}
> -	}
> -
> -	if (vmid > last_vmid_to_scan) {
> -		pr_err("Didn't find vmid for pasid 0x%x\n", p->pasid);
> -		return -EFAULT;
> -	}
> -
> -	/* taking the VMID for that process on the safe way using PDD */
> -	pdd = kfd_get_process_device_data(dev, p);
> -	if (!pdd)
> -		return -EFAULT;
> -
> -	status = dbgdev_wave_control_set_registers(&wac_info, &reg_sq_cmd,
> -			&reg_gfx_index);
> -	if (status != 0)
> -		return -EINVAL;
> -
> -	/* for non DIQ we need to patch the VMID: */
> -	reg_sq_cmd.bits.vm_id = vmid;
> -
> -	dev->kfd2kgd->wave_control_execute(dev->adev,
> -					reg_gfx_index.u32All,
> -					reg_sq_cmd.u32All);
> -
> -	return 0;
> -}
> -
> -void kfd_dbgdev_init(struct kfd_dbgdev *pdbgdev, struct kfd_dev *pdev,
> -			enum DBGDEV_TYPE type)
> -{
> -	pdbgdev->dev = pdev;
> -	pdbgdev->kq = NULL;
> -	pdbgdev->type = type;
> -	pdbgdev->pqm = NULL;
> -
> -	switch (type) {
> -	case DBGDEV_TYPE_NODIQ:
> -		pdbgdev->dbgdev_register = dbgdev_register_nodiq;
> -		pdbgdev->dbgdev_unregister = dbgdev_unregister_nodiq;
> -		pdbgdev->dbgdev_wave_control = dbgdev_wave_control_nodiq;
> -		pdbgdev->dbgdev_address_watch = dbgdev_address_watch_nodiq;
> -		break;
> -	case DBGDEV_TYPE_DIQ:
> -	default:
> -		pdbgdev->dbgdev_register = dbgdev_register_diq;
> -		pdbgdev->dbgdev_unregister = dbgdev_unregister_diq;
> -		pdbgdev->dbgdev_wave_control =  dbgdev_wave_control_diq;
> -		pdbgdev->dbgdev_address_watch = dbgdev_address_watch_diq;
> -		break;
> -	}
> -
> -}
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_dbgdev.h b/drivers/gpu/drm/amd/amdkfd/kfd_dbgdev.h
> deleted file mode 100644
> index 0619c777b47e..000000000000
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_dbgdev.h
> +++ /dev/null
> @@ -1,230 +0,0 @@
> -/*
> - * Copyright 2014 Advanced Micro Devices, Inc.
> - *
> - * Permission is hereby granted, free of charge, to any person obtaining a
> - * copy of this software and associated documentation files (the "Software"),
> - * to deal in the Software without restriction, including without limitation
> - * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> - * and/or sell copies of the Software, and to permit persons to whom the
> - * Software is furnished to do so, subject to the following conditions:
> - *
> - * The above copyright notice and this permission notice shall be included in
> - * all copies or substantial portions of the Software.
> - *
> - * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> - * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> - * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> - * THE COPYRIGHT HOLDER(S) OR AUTHOR(S) BE LIABLE FOR ANY CLAIM, DAMAGES OR
> - * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
> - * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
> - * OTHER DEALINGS IN THE SOFTWARE.
> - */
> -
> -#ifndef KFD_DBGDEV_H_
> -#define KFD_DBGDEV_H_
> -
> -enum {
> -	SQ_CMD_VMID_OFFSET = 28,
> -	ADDRESS_WATCH_CNTL_OFFSET = 24
> -};
> -
> -enum {
> -	PRIV_QUEUE_SYNC_TIME_MS = 200
> -};
> -
> -/* CONTEXT reg space definition */
> -enum {
> -	CONTEXT_REG_BASE = 0xA000,
> -	CONTEXT_REG_END = 0xA400,
> -	CONTEXT_REG_SIZE = CONTEXT_REG_END - CONTEXT_REG_BASE
> -};
> -
> -/* USER CONFIG reg space definition */
> -enum {
> -	USERCONFIG_REG_BASE = 0xC000,
> -	USERCONFIG_REG_END = 0x10000,
> -	USERCONFIG_REG_SIZE = USERCONFIG_REG_END - USERCONFIG_REG_BASE
> -};
> -
> -/* CONFIG reg space definition */
> -enum {
> -	AMD_CONFIG_REG_BASE = 0x2000,	/* in dwords */
> -	AMD_CONFIG_REG_END = 0x2B00,
> -	AMD_CONFIG_REG_SIZE = AMD_CONFIG_REG_END - AMD_CONFIG_REG_BASE
> -};
> -
> -/* SH reg space definition */
> -enum {
> -	SH_REG_BASE = 0x2C00,
> -	SH_REG_END = 0x3000,
> -	SH_REG_SIZE = SH_REG_END - SH_REG_BASE
> -};
> -
> -/* SQ_CMD definitions */
> -#define SQ_CMD						0x8DEC
> -
> -enum SQ_IND_CMD_CMD {
> -	SQ_IND_CMD_CMD_NULL = 0x00000000,
> -	SQ_IND_CMD_CMD_HALT = 0x00000001,
> -	SQ_IND_CMD_CMD_RESUME = 0x00000002,
> -	SQ_IND_CMD_CMD_KILL = 0x00000003,
> -	SQ_IND_CMD_CMD_DEBUG = 0x00000004,
> -	SQ_IND_CMD_CMD_TRAP = 0x00000005,
> -};
> -
> -enum SQ_IND_CMD_MODE {
> -	SQ_IND_CMD_MODE_SINGLE = 0x00000000,
> -	SQ_IND_CMD_MODE_BROADCAST = 0x00000001,
> -	SQ_IND_CMD_MODE_BROADCAST_QUEUE = 0x00000002,
> -	SQ_IND_CMD_MODE_BROADCAST_PIPE = 0x00000003,
> -	SQ_IND_CMD_MODE_BROADCAST_ME = 0x00000004,
> -};
> -
> -union SQ_IND_INDEX_BITS {
> -	struct {
> -		uint32_t wave_id:4;
> -		uint32_t simd_id:2;
> -		uint32_t thread_id:6;
> -		 uint32_t:1;
> -		uint32_t force_read:1;
> -		uint32_t read_timeout:1;
> -		uint32_t unindexed:1;
> -		uint32_t index:16;
> -
> -	} bitfields, bits;
> -	uint32_t u32All;
> -	signed int i32All;
> -	float f32All;
> -};
> -
> -union SQ_IND_CMD_BITS {
> -	struct {
> -		uint32_t data:32;
> -	} bitfields, bits;
> -	uint32_t u32All;
> -	signed int i32All;
> -	float f32All;
> -};
> -
> -union SQ_CMD_BITS {
> -	struct {
> -		uint32_t cmd:3;
> -		 uint32_t:1;
> -		uint32_t mode:3;
> -		uint32_t check_vmid:1;
> -		uint32_t trap_id:3;
> -		 uint32_t:5;
> -		uint32_t wave_id:4;
> -		uint32_t simd_id:2;
> -		 uint32_t:2;
> -		uint32_t queue_id:3;
> -		 uint32_t:1;
> -		uint32_t vm_id:4;
> -	} bitfields, bits;
> -	uint32_t u32All;
> -	signed int i32All;
> -	float f32All;
> -};
> -
> -union SQ_IND_DATA_BITS {
> -	struct {
> -		uint32_t data:32;
> -	} bitfields, bits;
> -	uint32_t u32All;
> -	signed int i32All;
> -	float f32All;
> -};
> -
> -union GRBM_GFX_INDEX_BITS {
> -	struct {
> -		uint32_t instance_index:8;
> -		uint32_t sh_index:8;
> -		uint32_t se_index:8;
> -		 uint32_t:5;
> -		uint32_t sh_broadcast_writes:1;
> -		uint32_t instance_broadcast_writes:1;
> -		uint32_t se_broadcast_writes:1;
> -	} bitfields, bits;
> -	uint32_t u32All;
> -	signed int i32All;
> -	float f32All;
> -};
> -
> -union TCP_WATCH_ADDR_H_BITS {
> -	struct {
> -		uint32_t addr:16;
> -		 uint32_t:16;
> -
> -	} bitfields, bits;
> -	uint32_t u32All;
> -	signed int i32All;
> -	float f32All;
> -};
> -
> -union TCP_WATCH_ADDR_L_BITS {
> -	struct {
> -		uint32_t:6;
> -		uint32_t addr:26;
> -	} bitfields, bits;
> -	uint32_t u32All;
> -	signed int i32All;
> -	float f32All;
> -};
> -
> -enum {
> -	QUEUESTATE__INVALID = 0, /* so by default we'll get invalid state */
> -	QUEUESTATE__ACTIVE_COMPLETION_PENDING,
> -	QUEUESTATE__ACTIVE
> -};
> -
> -union ULARGE_INTEGER {
> -	struct {
> -		uint32_t low_part;
> -		uint32_t high_part;
> -	} u;
> -	unsigned long long quad_part;
> -};
> -
> -
> -#define KFD_CIK_VMID_START_OFFSET (8)
> -#define KFD_CIK_VMID_END_OFFSET (KFD_CIK_VMID_START_OFFSET + (8))
> -
> -
> -void kfd_dbgdev_init(struct kfd_dbgdev *pdbgdev, struct kfd_dev *pdev,
> -			enum DBGDEV_TYPE type);
> -
> -union TCP_WATCH_CNTL_BITS {
> -	struct {
> -		uint32_t mask:24;
> -		uint32_t vmid:4;
> -		uint32_t atc:1;
> -		uint32_t mode:2;
> -		uint32_t valid:1;
> -	} bitfields, bits;
> -	uint32_t u32All;
> -	signed int i32All;
> -	float f32All;
> -};
> -
> -enum {
> -	ADDRESS_WATCH_REG_CNTL_ATC_BIT = 0x10000000UL,
> -	ADDRESS_WATCH_REG_CNTL_DEFAULT_MASK = 0x00FFFFFF,
> -	ADDRESS_WATCH_REG_ADDLOW_MASK_EXTENSION = 0x03000000,
> -	/* extend the mask to 26 bits in order to match the low address field */
> -	ADDRESS_WATCH_REG_ADDLOW_SHIFT = 6,
> -	ADDRESS_WATCH_REG_ADDHIGH_MASK = 0xFFFF
> -};
> -
> -enum {
> -	MAX_TRAPID = 8,		/* 3 bits in the bitfield. */
> -	MAX_WATCH_ADDRESSES = 4
> -};
> -
> -enum {
> -	ADDRESS_WATCH_REG_ADDR_HI = 0,
> -	ADDRESS_WATCH_REG_ADDR_LO,
> -	ADDRESS_WATCH_REG_CNTL,
> -	ADDRESS_WATCH_REG_MAX
> -};
> -
> -#endif	/* KFD_DBGDEV_H_ */
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_dbgmgr.c b/drivers/gpu/drm/amd/amdkfd/kfd_dbgmgr.c
> deleted file mode 100644
> index 9bfa50633654..000000000000
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_dbgmgr.c
> +++ /dev/null
> @@ -1,158 +0,0 @@
> -/*
> - * Copyright 2014 Advanced Micro Devices, Inc.
> - *
> - * Permission is hereby granted, free of charge, to any person obtaining a
> - * copy of this software and associated documentation files (the "Software"),
> - * to deal in the Software without restriction, including without limitation
> - * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> - * and/or sell copies of the Software, and to permit persons to whom the
> - * Software is furnished to do so, subject to the following conditions:
> - *
> - * The above copyright notice and this permission notice shall be included in
> - * all copies or substantial portions of the Software.
> - *
> - * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> - * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> - * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> - * THE COPYRIGHT HOLDER(S) OR AUTHOR(S) BE LIABLE FOR ANY CLAIM, DAMAGES OR
> - * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
> - * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
> - * OTHER DEALINGS IN THE SOFTWARE.
> - *
> - */
> -#include <linux/types.h>
> -#include <linux/kernel.h>
> -#include <linux/log2.h>
> -#include <linux/sched.h>
> -#include <linux/slab.h>
> -#include <linux/device.h>
> -
> -#include "kfd_priv.h"
> -#include "cik_regs.h"
> -#include "kfd_pm4_headers.h"
> -#include "kfd_pm4_headers_diq.h"
> -#include "kfd_dbgmgr.h"
> -#include "kfd_dbgdev.h"
> -#include "kfd_device_queue_manager.h"
> -
> -static DEFINE_MUTEX(kfd_dbgmgr_mutex);
> -
> -struct mutex *kfd_get_dbgmgr_mutex(void)
> -{
> -	return &kfd_dbgmgr_mutex;
> -}
> -
> -
> -static void kfd_dbgmgr_uninitialize(struct kfd_dbgmgr *pmgr)
> -{
> -	kfree(pmgr->dbgdev);
> -
> -	pmgr->dbgdev = NULL;
> -	pmgr->pasid = 0;
> -	pmgr->dev = NULL;
> -}
> -
> -void kfd_dbgmgr_destroy(struct kfd_dbgmgr *pmgr)
> -{
> -	if (pmgr) {
> -		kfd_dbgmgr_uninitialize(pmgr);
> -		kfree(pmgr);
> -	}
> -}
> -
> -bool kfd_dbgmgr_create(struct kfd_dbgmgr **ppmgr, struct kfd_dev *pdev)
> -{
> -	enum DBGDEV_TYPE type = DBGDEV_TYPE_DIQ;
> -	struct kfd_dbgmgr *new_buff;
> -
> -	if (WARN_ON(!pdev->init_complete))
> -		return false;
> -
> -	new_buff = kfd_alloc_struct(new_buff);
> -	if (!new_buff) {
> -		pr_err("Failed to allocate dbgmgr instance\n");
> -		return false;
> -	}
> -
> -	new_buff->pasid = 0;
> -	new_buff->dev = pdev;
> -	new_buff->dbgdev = kfd_alloc_struct(new_buff->dbgdev);
> -	if (!new_buff->dbgdev) {
> -		pr_err("Failed to allocate dbgdev instance\n");
> -		kfree(new_buff);
> -		return false;
> -	}
> -
> -	/* get actual type of DBGDevice cpsch or not */
> -	if (pdev->dqm->sched_policy == KFD_SCHED_POLICY_NO_HWS)
> -		type = DBGDEV_TYPE_NODIQ;
> -
> -	kfd_dbgdev_init(new_buff->dbgdev, pdev, type);
> -	*ppmgr = new_buff;
> -
> -	return true;
> -}
> -
> -long kfd_dbgmgr_register(struct kfd_dbgmgr *pmgr, struct kfd_process *p)
> -{
> -	if (pmgr->pasid != 0) {
> -		pr_debug("H/W debugger is already active using pasid 0x%x\n",
> -				pmgr->pasid);
> -		return -EBUSY;
> -	}
> -
> -	/* remember pasid */
> -	pmgr->pasid = p->pasid;
> -
> -	/* provide the pqm for diq generation */
> -	pmgr->dbgdev->pqm = &p->pqm;
> -
> -	/* activate the actual registering */
> -	pmgr->dbgdev->dbgdev_register(pmgr->dbgdev);
> -
> -	return 0;
> -}
> -
> -long kfd_dbgmgr_unregister(struct kfd_dbgmgr *pmgr, struct kfd_process *p)
> -{
> -	/* Is the requests coming from the already registered process? */
> -	if (pmgr->pasid != p->pasid) {
> -		pr_debug("H/W debugger is not registered by calling pasid 0x%x\n",
> -				p->pasid);
> -		return -EINVAL;
> -	}
> -
> -	pmgr->dbgdev->dbgdev_unregister(pmgr->dbgdev);
> -
> -	pmgr->pasid = 0;
> -
> -	return 0;
> -}
> -
> -long kfd_dbgmgr_wave_control(struct kfd_dbgmgr *pmgr,
> -				struct dbg_wave_control_info *wac_info)
> -{
> -	/* Is the requests coming from the already registered process? */
> -	if (pmgr->pasid != wac_info->process->pasid) {
> -		pr_debug("H/W debugger support was not registered for requester pasid 0x%x\n",
> -				wac_info->process->pasid);
> -		return -EINVAL;
> -	}
> -
> -	return (long) pmgr->dbgdev->dbgdev_wave_control(pmgr->dbgdev, wac_info);
> -}
> -
> -long kfd_dbgmgr_address_watch(struct kfd_dbgmgr *pmgr,
> -				struct dbg_address_watch_info *adw_info)
> -{
> -	/* Is the requests coming from the already registered process? */
> -	if (pmgr->pasid != adw_info->process->pasid) {
> -		pr_debug("H/W debugger support was not registered for requester pasid 0x%x\n",
> -				adw_info->process->pasid);
> -		return -EINVAL;
> -	}
> -
> -	return (long) pmgr->dbgdev->dbgdev_address_watch(pmgr->dbgdev,
> -							adw_info);
> -}
> -
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_dbgmgr.h b/drivers/gpu/drm/amd/amdkfd/kfd_dbgmgr.h
> deleted file mode 100644
> index f9c6df1fdc5c..000000000000
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_dbgmgr.h
> +++ /dev/null
> @@ -1,293 +0,0 @@
> -/*
> - * Copyright 2014 Advanced Micro Devices, Inc.
> - *
> - * Permission is hereby granted, free of charge, to any person obtaining a
> - * copy of this software and associated documentation files (the "Software"),
> - * to deal in the Software without restriction, including without limitation
> - * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> - * and/or sell copies of the Software, and to permit persons to whom the
> - * Software is furnished to do so, subject to the following conditions:
> - *
> - * The above copyright notice and this permission notice shall be included in
> - * all copies or substantial portions of the Software.
> - *
> - * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> - * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> - * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> - * THE COPYRIGHT HOLDER(S) OR AUTHOR(S) BE LIABLE FOR ANY CLAIM, DAMAGES OR
> - * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
> - * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
> - * OTHER DEALINGS IN THE SOFTWARE.
> - *
> - */
> -
> -#ifndef KFD_DBGMGR_H_
> -#define KFD_DBGMGR_H_
> -
> -#include "kfd_priv.h"
> -
> -/* must align with hsakmttypes definition */
> -#pragma pack(push, 4)
> -
> -enum HSA_DBG_WAVEOP {
> -	HSA_DBG_WAVEOP_HALT = 1,   /* Halts a wavefront */
> -	HSA_DBG_WAVEOP_RESUME = 2, /* Resumes a wavefront */
> -	HSA_DBG_WAVEOP_KILL = 3,   /* Kills a wavefront */
> -	HSA_DBG_WAVEOP_DEBUG = 4,  /* Causes wavefront to enter dbg mode */
> -	HSA_DBG_WAVEOP_TRAP = 5,   /* Causes wavefront to take a trap */
> -	HSA_DBG_NUM_WAVEOP = 5,
> -	HSA_DBG_MAX_WAVEOP = 0xFFFFFFFF
> -};
> -
> -enum HSA_DBG_WAVEMODE {
> -	/* send command to a single wave */
> -	HSA_DBG_WAVEMODE_SINGLE = 0,
> -	/*
> -	 * Broadcast to all wavefronts of all processes is not
> -	 * supported for HSA user mode
> -	 */
> -
> -	/* send to waves within current process */
> -	HSA_DBG_WAVEMODE_BROADCAST_PROCESS = 2,
> -	/* send to waves within current process on CU  */
> -	HSA_DBG_WAVEMODE_BROADCAST_PROCESS_CU = 3,
> -	HSA_DBG_NUM_WAVEMODE = 3,
> -	HSA_DBG_MAX_WAVEMODE = 0xFFFFFFFF
> -};
> -
> -enum HSA_DBG_WAVEMSG_TYPE {
> -	HSA_DBG_WAVEMSG_AUTO = 0,
> -	HSA_DBG_WAVEMSG_USER = 1,
> -	HSA_DBG_WAVEMSG_ERROR = 2,
> -	HSA_DBG_NUM_WAVEMSG,
> -	HSA_DBG_MAX_WAVEMSG = 0xFFFFFFFF
> -};
> -
> -enum HSA_DBG_WATCH_MODE {
> -	HSA_DBG_WATCH_READ = 0,		/* Read operations only */
> -	HSA_DBG_WATCH_NONREAD = 1,	/* Write or Atomic operations only */
> -	HSA_DBG_WATCH_ATOMIC = 2,	/* Atomic Operations only */
> -	HSA_DBG_WATCH_ALL = 3,		/* Read, Write or Atomic operations */
> -	HSA_DBG_WATCH_NUM,
> -	HSA_DBG_WATCH_SIZE = 0xFFFFFFFF
> -};
> -
> -/* This structure is hardware specific and may change in the future */
> -struct HsaDbgWaveMsgAMDGen2 {
> -	union {
> -		struct ui32 {
> -			uint32_t UserData:8;	/* user data */
> -			uint32_t ShaderArray:1;	/* Shader array */
> -			uint32_t Priv:1;	/* Privileged */
> -			uint32_t Reserved0:4;	/* Reserved, should be 0 */
> -			uint32_t WaveId:4;	/* wave id */
> -			uint32_t SIMD:2;	/* SIMD id */
> -			uint32_t HSACU:4;	/* Compute unit */
> -			uint32_t ShaderEngine:2;/* Shader engine */
> -			uint32_t MessageType:2;	/* see HSA_DBG_WAVEMSG_TYPE */
> -			uint32_t Reserved1:4;	/* Reserved, should be 0 */
> -		} ui32;
> -		uint32_t Value;
> -	};
> -	uint32_t Reserved2;
> -};
> -
> -union HsaDbgWaveMessageAMD {
> -	struct HsaDbgWaveMsgAMDGen2 WaveMsgInfoGen2;
> -	/* for future HsaDbgWaveMsgAMDGen3; */
> -};
> -
> -struct HsaDbgWaveMessage {
> -	void *MemoryVA;		/* ptr to associated host-accessible data */
> -	union HsaDbgWaveMessageAMD DbgWaveMsg;
> -};
> -
> -/*
> - * TODO: This definitions to be MOVED to kfd_event, once it is implemented.
> - *
> - * HSA sync primitive, Event and HW Exception notification API definitions.
> - * The API functions allow the runtime to define a so-called sync-primitive,
> - * a SW object combining a user-mode provided "syncvar" and a scheduler event
> - * that can be signaled through a defined GPU interrupt. A syncvar is
> - * a process virtual memory location of a certain size that can be accessed
> - * by CPU and GPU shader code within the process to set and query the content
> - * within that memory. The definition of the content is determined by the HSA
> - * runtime and potentially GPU shader code interfacing with the HSA runtime.
> - * The syncvar values may be commonly written through an PM4 WRITE_DATA packet
> - * in the user mode instruction stream. The OS scheduler event is typically
> - * associated and signaled by an interrupt issued by the GPU, but other HSA
> - * system interrupt conditions from other HW (e.g. IOMMUv2) may be surfaced
> - * by the KFD by this mechanism, too.
> - */
> -
> -/* these are the new definitions for events */
> -enum HSA_EVENTTYPE {
> -	HSA_EVENTTYPE_SIGNAL = 0,	/* user-mode generated GPU signal */
> -	HSA_EVENTTYPE_NODECHANGE = 1,	/* HSA node change (attach/detach) */
> -	HSA_EVENTTYPE_DEVICESTATECHANGE = 2,	/* HSA device state change
> -						 * (start/stop)
> -						 */
> -	HSA_EVENTTYPE_HW_EXCEPTION = 3,	/* GPU shader exception event */
> -	HSA_EVENTTYPE_SYSTEM_EVENT = 4,	/* GPU SYSCALL with parameter info */
> -	HSA_EVENTTYPE_DEBUG_EVENT = 5,	/* GPU signal for debugging */
> -	HSA_EVENTTYPE_PROFILE_EVENT = 6,/* GPU signal for profiling */
> -	HSA_EVENTTYPE_QUEUE_EVENT = 7,	/* GPU signal queue idle state
> -					 * (EOP pm4)
> -					 */
> -	/* ...  */
> -	HSA_EVENTTYPE_MAXID,
> -	HSA_EVENTTYPE_TYPE_SIZE = 0xFFFFFFFF
> -};
> -
> -/* Sub-definitions for various event types: Syncvar */
> -struct HsaSyncVar {
> -	union SyncVar {
> -		void *UserData;	/* pointer to user mode data */
> -		uint64_t UserDataPtrValue; /* 64bit compatibility of value */
> -	} SyncVar;
> -	uint64_t SyncVarSize;
> -};
> -
> -/* Sub-definitions for various event types: NodeChange */
> -
> -enum HSA_EVENTTYPE_NODECHANGE_FLAGS {
> -	HSA_EVENTTYPE_NODECHANGE_ADD = 0,
> -	HSA_EVENTTYPE_NODECHANGE_REMOVE = 1,
> -	HSA_EVENTTYPE_NODECHANGE_SIZE = 0xFFFFFFFF
> -};
> -
> -struct HsaNodeChange {
> -	/* HSA node added/removed on the platform */
> -	enum HSA_EVENTTYPE_NODECHANGE_FLAGS Flags;
> -};
> -
> -/* Sub-definitions for various event types: DeviceStateChange */
> -enum HSA_EVENTTYPE_DEVICESTATECHANGE_FLAGS {
> -	/* device started (and available) */
> -	HSA_EVENTTYPE_DEVICESTATUSCHANGE_START = 0,
> -	/* device stopped (i.e. unavailable) */
> -	HSA_EVENTTYPE_DEVICESTATUSCHANGE_STOP = 1,
> -	HSA_EVENTTYPE_DEVICESTATUSCHANGE_SIZE = 0xFFFFFFFF
> -};
> -
> -enum HSA_DEVICE {
> -	HSA_DEVICE_CPU = 0,
> -	HSA_DEVICE_GPU = 1,
> -	MAX_HSA_DEVICE = 2
> -};
> -
> -struct HsaDeviceStateChange {
> -	uint32_t NodeId;	/* F-NUMA node that contains the device */
> -	enum HSA_DEVICE Device;	/* device type: GPU or CPU */
> -	enum HSA_EVENTTYPE_DEVICESTATECHANGE_FLAGS Flags; /* event flags */
> -};
> -
> -struct HsaEventData {
> -	enum HSA_EVENTTYPE EventType; /* event type */
> -	union EventData {
> -		/*
> -		 * return data associated with HSA_EVENTTYPE_SIGNAL
> -		 * and other events
> -		 */
> -		struct HsaSyncVar SyncVar;
> -
> -		/* data associated with HSA_EVENTTYPE_NODE_CHANGE */
> -		struct HsaNodeChange NodeChangeState;
> -
> -		/* data associated with HSA_EVENTTYPE_DEVICE_STATE_CHANGE */
> -		struct HsaDeviceStateChange DeviceState;
> -	} EventData;
> -
> -	/* the following data entries are internal to the KFD & thunk itself */
> -
> -	/* internal thunk store for Event data (OsEventHandle) */
> -	uint64_t HWData1;
> -	/* internal thunk store for Event data (HWAddress) */
> -	uint64_t HWData2;
> -	/* internal thunk store for Event data (HWData) */
> -	uint32_t HWData3;
> -};
> -
> -struct HsaEventDescriptor {
> -	/* event type to allocate */
> -	enum HSA_EVENTTYPE EventType;
> -	/* H-NUMA node containing GPU device that is event source */
> -	uint32_t NodeId;
> -	/* pointer to user mode syncvar data, syncvar->UserDataPtrValue
> -	 * may be NULL
> -	 */
> -	struct HsaSyncVar SyncVar;
> -};
> -
> -struct HsaEvent {
> -	uint32_t EventId;
> -	struct HsaEventData EventData;
> -};
> -
> -#pragma pack(pop)
> -
> -enum DBGDEV_TYPE {
> -	DBGDEV_TYPE_ILLEGAL = 0,
> -	DBGDEV_TYPE_NODIQ = 1,
> -	DBGDEV_TYPE_DIQ = 2,
> -	DBGDEV_TYPE_TEST = 3
> -};
> -
> -struct dbg_address_watch_info {
> -	struct kfd_process *process;
> -	enum HSA_DBG_WATCH_MODE *watch_mode;
> -	uint64_t *watch_address;
> -	uint64_t *watch_mask;
> -	struct HsaEvent *watch_event;
> -	uint32_t num_watch_points;
> -};
> -
> -struct dbg_wave_control_info {
> -	struct kfd_process *process;
> -	uint32_t trapId;
> -	enum HSA_DBG_WAVEOP operand;
> -	enum HSA_DBG_WAVEMODE mode;
> -	struct HsaDbgWaveMessage dbgWave_msg;
> -};
> -
> -struct kfd_dbgdev {
> -
> -	/* The device that owns this data. */
> -	struct kfd_dev *dev;
> -
> -	/* kernel queue for DIQ */
> -	struct kernel_queue *kq;
> -
> -	/* a pointer to the pqm of the calling process */
> -	struct process_queue_manager *pqm;
> -
> -	/* type of debug device ( DIQ, non DIQ, etc. ) */
> -	enum DBGDEV_TYPE type;
> -
> -	/* virtualized function pointers to device dbg */
> -	int (*dbgdev_register)(struct kfd_dbgdev *dbgdev);
> -	int (*dbgdev_unregister)(struct kfd_dbgdev *dbgdev);
> -	int (*dbgdev_address_watch)(struct kfd_dbgdev *dbgdev,
> -				struct dbg_address_watch_info *adw_info);
> -	int (*dbgdev_wave_control)(struct kfd_dbgdev *dbgdev,
> -				struct dbg_wave_control_info *wac_info);
> -
> -};
> -
> -struct kfd_dbgmgr {
> -	u32 pasid;
> -	struct kfd_dev *dev;
> -	struct kfd_dbgdev *dbgdev;
> -};
> -
> -/* prototypes for debug manager functions */
> -struct mutex *kfd_get_dbgmgr_mutex(void);
> -void kfd_dbgmgr_destroy(struct kfd_dbgmgr *pmgr);
> -bool kfd_dbgmgr_create(struct kfd_dbgmgr **ppmgr, struct kfd_dev *pdev);
> -long kfd_dbgmgr_register(struct kfd_dbgmgr *pmgr, struct kfd_process *p);
> -long kfd_dbgmgr_unregister(struct kfd_dbgmgr *pmgr, struct kfd_process *p);
> -long kfd_dbgmgr_wave_control(struct kfd_dbgmgr *pmgr,
> -				struct dbg_wave_control_info *wac_info);
> -long kfd_dbgmgr_address_watch(struct kfd_dbgmgr *pmgr,
> -			struct dbg_address_watch_info *adw_info);
> -#endif /* KFD_DBGMGR_H_ */
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device.c b/drivers/gpu/drm/amd/amdkfd/kfd_device.c
> index 5a47f437b455..dbb877fba724 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_device.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_device.c
> @@ -575,8 +575,6 @@ bool kgd2kfd_device_init(struct kfd_dev *kfd,
>   	if (kfd_resume(kfd))
>   		goto kfd_resume_error;
>   
> -	kfd->dbgmgr = NULL;
> -
>   	if (kfd_topology_add_device(kfd)) {
>   		dev_err(kfd_device, "Error adding device to topology\n");
>   		goto kfd_topology_add_device_error;
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> index 63b3c7af681b..f5d82f257dbb 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> @@ -480,6 +480,65 @@ static inline void deallocate_hqd(struct device_queue_manager *dqm,
>   	dqm->allocated_queues[q->pipe] |= (1 << q->queue);
>   }
>   
> +#define SQ_IND_CMD_CMD_KILL		0x00000003
> +#define SQ_IND_CMD_MODE_BROADCAST	0x00000001
> +
> +static int dbgdev_wave_reset_wavefronts(struct kfd_dev *dev, struct kfd_process *p)
> +{
> +	int status = 0;
> +	unsigned int vmid;
> +	uint16_t queried_pasid;
> +	union SQ_CMD_BITS reg_sq_cmd;
> +	union GRBM_GFX_INDEX_BITS reg_gfx_index;
> +	struct kfd_process_device *pdd;
> +	int first_vmid_to_scan = dev->vm_info.first_vmid_kfd;
> +	int last_vmid_to_scan = dev->vm_info.last_vmid_kfd;
> +
> +	reg_sq_cmd.u32All = 0;
> +	reg_gfx_index.u32All = 0;
> +
> +	pr_debug("Killing all process wavefronts\n");
> +
> +	/* Scan all registers in the range ATC_VMID8_PASID_MAPPING ..
> +	 * ATC_VMID15_PASID_MAPPING
> +	 * to check which VMID the current process is mapped to.
> +	 */
> +
> +	for (vmid = first_vmid_to_scan; vmid <= last_vmid_to_scan; vmid++) {
> +		status = dev->kfd2kgd->get_atc_vmid_pasid_mapping_info
> +				(dev->adev, vmid, &queried_pasid);
> +
> +		if (status && queried_pasid == p->pasid) {
> +			pr_debug("Killing wave fronts of vmid %d and pasid 0x%x\n",
> +					vmid, p->pasid);
> +			break;
> +		}
> +	}
> +
> +	if (vmid > last_vmid_to_scan) {
> +		pr_err("Didn't find vmid for pasid 0x%x\n", p->pasid);
> +		return -EFAULT;
> +	}
> +
> +	/* taking the VMID for that process on the safe way using PDD */
> +	pdd = kfd_get_process_device_data(dev, p);
> +	if (!pdd)
> +		return -EFAULT;
> +
> +	reg_gfx_index.bits.sh_broadcast_writes = 1;
> +	reg_gfx_index.bits.se_broadcast_writes = 1;
> +	reg_gfx_index.bits.instance_broadcast_writes = 1;
> +	reg_sq_cmd.bits.mode = SQ_IND_CMD_MODE_BROADCAST;
> +	reg_sq_cmd.bits.cmd = SQ_IND_CMD_CMD_KILL;
> +	reg_sq_cmd.bits.vm_id = vmid;
> +
> +	dev->kfd2kgd->wave_control_execute(dev->adev,
> +					reg_gfx_index.u32All,
> +					reg_sq_cmd.u32All);
> +
> +	return 0;
> +}
> +
>   /* Access to DQM has to be locked before calling destroy_queue_nocpsch_locked
>    * to avoid asynchronized access
>    */
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h
> index a7d2e3323977..05a9c17daa3e 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.h
> @@ -39,6 +39,41 @@ struct device_process_node {
>   	struct list_head list;
>   };
>   
> +union SQ_CMD_BITS {
> +	struct {
> +		uint32_t cmd:3;
> +		uint32_t:1;
> +		uint32_t mode:3;
> +		uint32_t check_vmid:1;
> +		uint32_t trap_id:3;
> +		uint32_t:5;
> +		uint32_t wave_id:4;
> +		uint32_t simd_id:2;
> +		uint32_t:2;
> +		uint32_t queue_id:3;
> +		uint32_t:1;
> +		uint32_t vm_id:4;
> +	} bitfields, bits;
> +	uint32_t u32All;
> +	signed int i32All;
> +	float f32All;
> +};
> +
> +union GRBM_GFX_INDEX_BITS {
> +	struct {
> +		uint32_t instance_index:8;
> +		uint32_t sh_index:8;
> +		uint32_t se_index:8;
> +		uint32_t:5;
> +		uint32_t sh_broadcast_writes:1;
> +		uint32_t instance_broadcast_writes:1;
> +		uint32_t se_broadcast_writes:1;
> +	} bitfields, bits;
> +	uint32_t u32All;
> +	signed int i32All;
> +	float f32All;
> +};
> +
>   /**
>    * struct device_queue_manager_ops
>    *
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_iommu.c b/drivers/gpu/drm/amd/amdkfd/kfd_iommu.c
> index 66ad8d0b8f7f..fe62407dacb2 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_iommu.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_iommu.c
> @@ -30,7 +30,6 @@
>   #include <linux/pci.h>
>   #include <linux/amd-iommu.h>
>   #include "kfd_priv.h"
> -#include "kfd_dbgmgr.h"
>   #include "kfd_topology.h"
>   #include "kfd_iommu.h"
>   
> @@ -163,17 +162,6 @@ static void iommu_pasid_shutdown_callback(struct pci_dev *pdev, u32 pasid)
>   
>   	pr_debug("Unbinding process 0x%x from IOMMU\n", pasid);
>   
> -	mutex_lock(kfd_get_dbgmgr_mutex());
> -
> -	if (dev->dbgmgr && dev->dbgmgr->pasid == p->pasid) {
> -		if (!kfd_dbgmgr_unregister(dev->dbgmgr, p)) {
> -			kfd_dbgmgr_destroy(dev->dbgmgr);
> -			dev->dbgmgr = NULL;
> -		}
> -	}
> -
> -	mutex_unlock(kfd_get_dbgmgr_mutex());
> -
>   	mutex_lock(&p->mutex);
>   
>   	pdd = kfd_get_process_device_data(dev, p);
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
> index 74ff4132a163..465aae8e5697 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
> @@ -300,9 +300,6 @@ struct kfd_dev {
>   	 */
>   	bool interrupts_active;
>   
> -	/* Debug manager */
> -	struct kfd_dbgmgr *dbgmgr;
> -
>   	/* Firmware versions */
>   	uint16_t mec_fw_version;
>   	uint16_t mec2_fw_version;
> @@ -1331,8 +1328,6 @@ void kfd_signal_poison_consumed_event(struct kfd_dev *dev, u32 pasid);
>   
>   void kfd_flush_tlb(struct kfd_process_device *pdd, enum TLB_FLUSH_TYPE type);
>   
> -int dbgdev_wave_reset_wavefronts(struct kfd_dev *dev, struct kfd_process *p);
> -
>   bool kfd_is_locked(void);
>   
>   /* Compute profile */
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_process.c b/drivers/gpu/drm/amd/amdkfd/kfd_process.c
> index 8e2780d2f735..8c6a48add76e 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_process.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_process.c
> @@ -40,7 +40,6 @@ struct mm_struct;
>   
>   #include "kfd_priv.h"
>   #include "kfd_device_queue_manager.h"
> -#include "kfd_dbgmgr.h"
>   #include "kfd_iommu.h"
>   #include "kfd_svm.h"
>   
> @@ -1158,7 +1157,6 @@ static void kfd_process_notifier_release(struct mmu_notifier *mn,
>   					struct mm_struct *mm)
>   {
>   	struct kfd_process *p;
> -	int i;
>   
>   	/*
>   	 * The kfd_process structure can not be free because the
> @@ -1178,23 +1176,6 @@ static void kfd_process_notifier_release(struct mmu_notifier *mn,
>   
>   	mutex_lock(&p->mutex);
>   
> -	/* Iterate over all process device data structures and if the
> -	 * pdd is in debug mode, we should first force unregistration,
> -	 * then we will be able to destroy the queues
> -	 */
> -	for (i = 0; i < p->n_pdds; i++) {
> -		struct kfd_dev *dev = p->pdds[i]->dev;
> -
> -		mutex_lock(kfd_get_dbgmgr_mutex());
> -		if (dev && dev->dbgmgr && dev->dbgmgr->pasid == p->pasid) {
> -			if (!kfd_dbgmgr_unregister(dev->dbgmgr, p)) {
> -				kfd_dbgmgr_destroy(dev->dbgmgr);
> -				dev->dbgmgr = NULL;
> -			}
> -		}
> -		mutex_unlock(kfd_get_dbgmgr_mutex());
> -	}
> -
>   	kfd_process_dequeue_from_all_devices(p);
>   	pqm_uninit(&p->pqm);
>   


More information about the amd-gfx mailing list