forked from mirrors/linux
		
	drm/lima: recover task by enlarging heap buffer
Increase heap buffer backup memory when GP receive PLBU out of memory interrupt, then resume the task. Reviewed-by: Vasily Khoruzhick <anarsoul@gmail.com> Tested-by: Andreas Baierl <ichgeh@imkreisrum.de> Signed-off-by: Qiang Yu <yuq825@gmail.com> Link: https://patchwork.freedesktop.org/patch/msgid/20200116131157.13346-5-yuq825@gmail.com
This commit is contained in:
		
							parent
							
								
									6aebc51d7a
								
							
						
					
					
						commit
						2081e8dcf1
					
				
					 5 changed files with 97 additions and 6 deletions
				
			
		| 
						 | 
					@ -11,6 +11,8 @@
 | 
				
			||||||
#include "lima_device.h"
 | 
					#include "lima_device.h"
 | 
				
			||||||
#include "lima_gp.h"
 | 
					#include "lima_gp.h"
 | 
				
			||||||
#include "lima_regs.h"
 | 
					#include "lima_regs.h"
 | 
				
			||||||
 | 
					#include "lima_gem.h"
 | 
				
			||||||
 | 
					#include "lima_vm.h"
 | 
				
			||||||
 | 
					
 | 
				
			||||||
#define gp_write(reg, data) writel(data, ip->iomem + reg)
 | 
					#define gp_write(reg, data) writel(data, ip->iomem + reg)
 | 
				
			||||||
#define gp_read(reg) readl(ip->iomem + reg)
 | 
					#define gp_read(reg) readl(ip->iomem + reg)
 | 
				
			||||||
| 
						 | 
					@ -20,6 +22,7 @@ static irqreturn_t lima_gp_irq_handler(int irq, void *data)
 | 
				
			||||||
	struct lima_ip *ip = data;
 | 
						struct lima_ip *ip = data;
 | 
				
			||||||
	struct lima_device *dev = ip->dev;
 | 
						struct lima_device *dev = ip->dev;
 | 
				
			||||||
	struct lima_sched_pipe *pipe = dev->pipe + lima_pipe_gp;
 | 
						struct lima_sched_pipe *pipe = dev->pipe + lima_pipe_gp;
 | 
				
			||||||
 | 
						struct lima_sched_task *task = pipe->current_task;
 | 
				
			||||||
	u32 state = gp_read(LIMA_GP_INT_STAT);
 | 
						u32 state = gp_read(LIMA_GP_INT_STAT);
 | 
				
			||||||
	u32 status = gp_read(LIMA_GP_STATUS);
 | 
						u32 status = gp_read(LIMA_GP_STATUS);
 | 
				
			||||||
	bool done = false;
 | 
						bool done = false;
 | 
				
			||||||
| 
						 | 
					@ -29,8 +32,16 @@ static irqreturn_t lima_gp_irq_handler(int irq, void *data)
 | 
				
			||||||
		return IRQ_NONE;
 | 
							return IRQ_NONE;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	if (state & LIMA_GP_IRQ_MASK_ERROR) {
 | 
						if (state & LIMA_GP_IRQ_MASK_ERROR) {
 | 
				
			||||||
 | 
							if ((state & LIMA_GP_IRQ_MASK_ERROR) ==
 | 
				
			||||||
 | 
							    LIMA_GP_IRQ_PLBU_OUT_OF_MEM) {
 | 
				
			||||||
 | 
								dev_dbg(dev->dev, "gp out of heap irq status=%x\n",
 | 
				
			||||||
 | 
									status);
 | 
				
			||||||
 | 
							} else {
 | 
				
			||||||
			dev_err(dev->dev, "gp error irq state=%x status=%x\n",
 | 
								dev_err(dev->dev, "gp error irq state=%x status=%x\n",
 | 
				
			||||||
				state, status);
 | 
									state, status);
 | 
				
			||||||
 | 
								if (task)
 | 
				
			||||||
 | 
									task->recoverable = false;
 | 
				
			||||||
 | 
							}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
		/* mask all interrupts before hard reset */
 | 
							/* mask all interrupts before hard reset */
 | 
				
			||||||
		gp_write(LIMA_GP_INT_MASK, 0);
 | 
							gp_write(LIMA_GP_INT_MASK, 0);
 | 
				
			||||||
| 
						 | 
					@ -43,6 +54,7 @@ static irqreturn_t lima_gp_irq_handler(int irq, void *data)
 | 
				
			||||||
		bool active = status & (LIMA_GP_STATUS_VS_ACTIVE |
 | 
							bool active = status & (LIMA_GP_STATUS_VS_ACTIVE |
 | 
				
			||||||
					LIMA_GP_STATUS_PLBU_ACTIVE);
 | 
										LIMA_GP_STATUS_PLBU_ACTIVE);
 | 
				
			||||||
		done = valid && !active;
 | 
							done = valid && !active;
 | 
				
			||||||
 | 
							pipe->error = false;
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	gp_write(LIMA_GP_INT_CLEAR, state);
 | 
						gp_write(LIMA_GP_INT_CLEAR, state);
 | 
				
			||||||
| 
						 | 
					@ -121,6 +133,22 @@ static void lima_gp_task_run(struct lima_sched_pipe *pipe,
 | 
				
			||||||
	u32 cmd = 0;
 | 
						u32 cmd = 0;
 | 
				
			||||||
	int i;
 | 
						int i;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						/* update real heap buffer size for GP */
 | 
				
			||||||
 | 
						for (i = 0; i < task->num_bos; i++) {
 | 
				
			||||||
 | 
							struct lima_bo *bo = task->bos[i];
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
							if (bo->heap_size &&
 | 
				
			||||||
 | 
							    lima_vm_get_va(task->vm, bo) ==
 | 
				
			||||||
 | 
							    f[LIMA_GP_PLBU_ALLOC_START_ADDR >> 2]) {
 | 
				
			||||||
 | 
								f[LIMA_GP_PLBU_ALLOC_END_ADDR >> 2] =
 | 
				
			||||||
 | 
									f[LIMA_GP_PLBU_ALLOC_START_ADDR >> 2] +
 | 
				
			||||||
 | 
									bo->heap_size;
 | 
				
			||||||
 | 
								task->recoverable = true;
 | 
				
			||||||
 | 
								task->heap = bo;
 | 
				
			||||||
 | 
								break;
 | 
				
			||||||
 | 
							}
 | 
				
			||||||
 | 
						}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	if (f[LIMA_GP_VSCL_START_ADDR >> 2] !=
 | 
						if (f[LIMA_GP_VSCL_START_ADDR >> 2] !=
 | 
				
			||||||
	    f[LIMA_GP_VSCL_END_ADDR >> 2])
 | 
						    f[LIMA_GP_VSCL_END_ADDR >> 2])
 | 
				
			||||||
		cmd |= LIMA_GP_CMD_START_VS;
 | 
							cmd |= LIMA_GP_CMD_START_VS;
 | 
				
			||||||
| 
						 | 
					@ -184,6 +212,31 @@ static void lima_gp_task_mmu_error(struct lima_sched_pipe *pipe)
 | 
				
			||||||
	lima_sched_pipe_task_done(pipe);
 | 
						lima_sched_pipe_task_done(pipe);
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					static int lima_gp_task_recover(struct lima_sched_pipe *pipe)
 | 
				
			||||||
 | 
					{
 | 
				
			||||||
 | 
						struct lima_ip *ip = pipe->processor[0];
 | 
				
			||||||
 | 
						struct lima_sched_task *task = pipe->current_task;
 | 
				
			||||||
 | 
						struct drm_lima_gp_frame *frame = task->frame;
 | 
				
			||||||
 | 
						u32 *f = frame->frame;
 | 
				
			||||||
 | 
						size_t fail_size =
 | 
				
			||||||
 | 
							f[LIMA_GP_PLBU_ALLOC_END_ADDR >> 2] -
 | 
				
			||||||
 | 
							f[LIMA_GP_PLBU_ALLOC_START_ADDR >> 2];
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						if (fail_size == task->heap->heap_size) {
 | 
				
			||||||
 | 
							int ret;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
							ret = lima_heap_alloc(task->heap, task->vm);
 | 
				
			||||||
 | 
							if (ret < 0)
 | 
				
			||||||
 | 
								return ret;
 | 
				
			||||||
 | 
						}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						gp_write(LIMA_GP_INT_MASK, LIMA_GP_IRQ_MASK_USED);
 | 
				
			||||||
 | 
						gp_write(LIMA_GP_PLBU_ALLOC_END_ADDR,
 | 
				
			||||||
 | 
							 f[LIMA_GP_PLBU_ALLOC_START_ADDR >> 2] + task->heap->heap_size);
 | 
				
			||||||
 | 
						gp_write(LIMA_GP_CMD, LIMA_GP_CMD_UPDATE_PLBU_ALLOC);
 | 
				
			||||||
 | 
						return 0;
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
static void lima_gp_print_version(struct lima_ip *ip)
 | 
					static void lima_gp_print_version(struct lima_ip *ip)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
	u32 version, major, minor;
 | 
						u32 version, major, minor;
 | 
				
			||||||
| 
						 | 
					@ -270,6 +323,7 @@ int lima_gp_pipe_init(struct lima_device *dev)
 | 
				
			||||||
	pipe->task_fini = lima_gp_task_fini;
 | 
						pipe->task_fini = lima_gp_task_fini;
 | 
				
			||||||
	pipe->task_error = lima_gp_task_error;
 | 
						pipe->task_error = lima_gp_task_error;
 | 
				
			||||||
	pipe->task_mmu_error = lima_gp_task_mmu_error;
 | 
						pipe->task_mmu_error = lima_gp_task_mmu_error;
 | 
				
			||||||
 | 
						pipe->task_recover = lima_gp_task_recover;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	return 0;
 | 
						return 0;
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -99,6 +99,11 @@ void lima_mmu_fini(struct lima_ip *ip)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					void lima_mmu_flush_tlb(struct lima_ip *ip)
 | 
				
			||||||
 | 
					{
 | 
				
			||||||
 | 
						mmu_write(LIMA_MMU_COMMAND, LIMA_MMU_COMMAND_ZAP_CACHE);
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
void lima_mmu_switch_vm(struct lima_ip *ip, struct lima_vm *vm)
 | 
					void lima_mmu_switch_vm(struct lima_ip *ip, struct lima_vm *vm)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
	struct lima_device *dev = ip->dev;
 | 
						struct lima_device *dev = ip->dev;
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -10,6 +10,7 @@ struct lima_vm;
 | 
				
			||||||
int lima_mmu_init(struct lima_ip *ip);
 | 
					int lima_mmu_init(struct lima_ip *ip);
 | 
				
			||||||
void lima_mmu_fini(struct lima_ip *ip);
 | 
					void lima_mmu_fini(struct lima_ip *ip);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					void lima_mmu_flush_tlb(struct lima_ip *ip);
 | 
				
			||||||
void lima_mmu_switch_vm(struct lima_ip *ip, struct lima_vm *vm);
 | 
					void lima_mmu_switch_vm(struct lima_ip *ip, struct lima_vm *vm);
 | 
				
			||||||
void lima_mmu_page_fault_resume(struct lima_ip *ip);
 | 
					void lima_mmu_page_fault_resume(struct lima_ip *ip);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -312,6 +312,26 @@ static const struct drm_sched_backend_ops lima_sched_ops = {
 | 
				
			||||||
	.free_job = lima_sched_free_job,
 | 
						.free_job = lima_sched_free_job,
 | 
				
			||||||
};
 | 
					};
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					static void lima_sched_recover_work(struct work_struct *work)
 | 
				
			||||||
 | 
					{
 | 
				
			||||||
 | 
						struct lima_sched_pipe *pipe =
 | 
				
			||||||
 | 
							container_of(work, struct lima_sched_pipe, recover_work);
 | 
				
			||||||
 | 
						int i;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						for (i = 0; i < pipe->num_l2_cache; i++)
 | 
				
			||||||
 | 
							lima_l2_cache_flush(pipe->l2_cache[i]);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						if (pipe->bcast_mmu) {
 | 
				
			||||||
 | 
							lima_mmu_flush_tlb(pipe->bcast_mmu);
 | 
				
			||||||
 | 
						} else {
 | 
				
			||||||
 | 
							for (i = 0; i < pipe->num_mmu; i++)
 | 
				
			||||||
 | 
								lima_mmu_flush_tlb(pipe->mmu[i]);
 | 
				
			||||||
 | 
						}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						if (pipe->task_recover(pipe))
 | 
				
			||||||
 | 
							drm_sched_fault(&pipe->base);
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
int lima_sched_pipe_init(struct lima_sched_pipe *pipe, const char *name)
 | 
					int lima_sched_pipe_init(struct lima_sched_pipe *pipe, const char *name)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
	unsigned int timeout = lima_sched_timeout_ms > 0 ?
 | 
						unsigned int timeout = lima_sched_timeout_ms > 0 ?
 | 
				
			||||||
| 
						 | 
					@ -320,6 +340,8 @@ int lima_sched_pipe_init(struct lima_sched_pipe *pipe, const char *name)
 | 
				
			||||||
	pipe->fence_context = dma_fence_context_alloc(1);
 | 
						pipe->fence_context = dma_fence_context_alloc(1);
 | 
				
			||||||
	spin_lock_init(&pipe->fence_lock);
 | 
						spin_lock_init(&pipe->fence_lock);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						INIT_WORK(&pipe->recover_work, lima_sched_recover_work);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	return drm_sched_init(&pipe->base, &lima_sched_ops, 1, 0,
 | 
						return drm_sched_init(&pipe->base, &lima_sched_ops, 1, 0,
 | 
				
			||||||
			      msecs_to_jiffies(timeout), name);
 | 
								      msecs_to_jiffies(timeout), name);
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
| 
						 | 
					@ -331,11 +353,14 @@ void lima_sched_pipe_fini(struct lima_sched_pipe *pipe)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
void lima_sched_pipe_task_done(struct lima_sched_pipe *pipe)
 | 
					void lima_sched_pipe_task_done(struct lima_sched_pipe *pipe)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
	if (pipe->error)
 | 
					 | 
				
			||||||
		drm_sched_fault(&pipe->base);
 | 
					 | 
				
			||||||
	else {
 | 
					 | 
				
			||||||
	struct lima_sched_task *task = pipe->current_task;
 | 
						struct lima_sched_task *task = pipe->current_task;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						if (pipe->error) {
 | 
				
			||||||
 | 
							if (task && task->recoverable)
 | 
				
			||||||
 | 
								schedule_work(&pipe->recover_work);
 | 
				
			||||||
 | 
							else
 | 
				
			||||||
 | 
								drm_sched_fault(&pipe->base);
 | 
				
			||||||
 | 
						} else {
 | 
				
			||||||
		pipe->task_fini(pipe);
 | 
							pipe->task_fini(pipe);
 | 
				
			||||||
		dma_fence_signal(task->fence);
 | 
							dma_fence_signal(task->fence);
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -20,6 +20,9 @@ struct lima_sched_task {
 | 
				
			||||||
	struct lima_bo **bos;
 | 
						struct lima_bo **bos;
 | 
				
			||||||
	int num_bos;
 | 
						int num_bos;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						bool recoverable;
 | 
				
			||||||
 | 
						struct lima_bo *heap;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	/* pipe fence */
 | 
						/* pipe fence */
 | 
				
			||||||
	struct dma_fence *fence;
 | 
						struct dma_fence *fence;
 | 
				
			||||||
};
 | 
					};
 | 
				
			||||||
| 
						 | 
					@ -68,6 +71,9 @@ struct lima_sched_pipe {
 | 
				
			||||||
	void (*task_fini)(struct lima_sched_pipe *pipe);
 | 
						void (*task_fini)(struct lima_sched_pipe *pipe);
 | 
				
			||||||
	void (*task_error)(struct lima_sched_pipe *pipe);
 | 
						void (*task_error)(struct lima_sched_pipe *pipe);
 | 
				
			||||||
	void (*task_mmu_error)(struct lima_sched_pipe *pipe);
 | 
						void (*task_mmu_error)(struct lima_sched_pipe *pipe);
 | 
				
			||||||
 | 
						int (*task_recover)(struct lima_sched_pipe *pipe);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						struct work_struct recover_work;
 | 
				
			||||||
};
 | 
					};
 | 
				
			||||||
 | 
					
 | 
				
			||||||
int lima_sched_task_init(struct lima_sched_task *task,
 | 
					int lima_sched_task_init(struct lima_sched_task *task,
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in a new issue