mirror of
git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2025-09-04 20:19:47 +08:00

Split the vmbind case into a separate helper function
submit_lock_objects_vmbind() to fix objtool warning:
drivers/gpu/drm/msm/msm.o: warning: objtool: submit_lock_objects+0x451:
sibling call from callable instruction with modified stack frame
The drm_exec_until_all_locked() macro uses computed gotos internally
for its retry loop. Having return statements inside this macro, or
immediately after it in certain code paths, confuses objtool's static
analysis of stack frames, causing it to incorrectly flag tail call
optimizations.
Fixes: 92395af63a
("drm/msm: Add VM_BIND submitqueue")
Signed-off-by: Sasha Levin <sashal@kernel.org>
Patchwork: https://patchwork.freedesktop.org/patch/667539/
Signed-off-by: Rob Clark <robin.clark@oss.qualcomm.com>
854 lines
21 KiB
C
854 lines
21 KiB
C
// SPDX-License-Identifier: GPL-2.0-only
|
|
/*
|
|
* Copyright (C) 2013 Red Hat
|
|
* Author: Rob Clark <robdclark@gmail.com>
|
|
*/
|
|
|
|
#include <linux/dma-fence-unwrap.h>
|
|
#include <linux/file.h>
|
|
#include <linux/sync_file.h>
|
|
#include <linux/uaccess.h>
|
|
|
|
#include <drm/drm_drv.h>
|
|
#include <drm/drm_file.h>
|
|
#include <drm/drm_syncobj.h>
|
|
|
|
#include "msm_drv.h"
|
|
#include "msm_gpu.h"
|
|
#include "msm_gem.h"
|
|
#include "msm_gpu_trace.h"
|
|
#include "msm_syncobj.h"
|
|
|
|
/* For userspace errors, use DRM_UT_DRIVER.. so that userspace can enable
|
|
* error msgs for debugging, but we don't spam dmesg by default
|
|
*/
|
|
#define SUBMIT_ERROR(err, submit, fmt, ...) \
|
|
UERR(err, (submit)->dev, fmt, ##__VA_ARGS__)
|
|
|
|
/*
|
|
* Cmdstream submission:
|
|
*/
|
|
|
|
static struct msm_gem_submit *submit_create(struct drm_device *dev,
|
|
struct msm_gpu *gpu,
|
|
struct msm_gpu_submitqueue *queue, uint32_t nr_bos,
|
|
uint32_t nr_cmds, u64 drm_client_id)
|
|
{
|
|
static atomic_t ident = ATOMIC_INIT(0);
|
|
struct msm_gem_submit *submit;
|
|
uint64_t sz;
|
|
int ret;
|
|
|
|
sz = struct_size(submit, bos, nr_bos) +
|
|
((u64)nr_cmds * sizeof(submit->cmd[0]));
|
|
|
|
if (sz > SIZE_MAX)
|
|
return ERR_PTR(-ENOMEM);
|
|
|
|
submit = kzalloc(sz, GFP_KERNEL | __GFP_NOWARN);
|
|
if (!submit)
|
|
return ERR_PTR(-ENOMEM);
|
|
|
|
submit->hw_fence = msm_fence_alloc();
|
|
if (IS_ERR(submit->hw_fence)) {
|
|
ret = PTR_ERR(submit->hw_fence);
|
|
kfree(submit);
|
|
return ERR_PTR(ret);
|
|
}
|
|
|
|
ret = drm_sched_job_init(&submit->base, queue->entity, 1, queue,
|
|
drm_client_id);
|
|
if (ret) {
|
|
kfree(submit->hw_fence);
|
|
kfree(submit);
|
|
return ERR_PTR(ret);
|
|
}
|
|
|
|
kref_init(&submit->ref);
|
|
submit->dev = dev;
|
|
submit->vm = msm_context_vm(dev, queue->ctx);
|
|
submit->gpu = gpu;
|
|
submit->cmd = (void *)&submit->bos[nr_bos];
|
|
submit->queue = queue;
|
|
submit->pid = get_pid(task_pid(current));
|
|
submit->ring = gpu->rb[queue->ring_nr];
|
|
submit->fault_dumped = false;
|
|
|
|
/* Get a unique identifier for the submission for logging purposes */
|
|
submit->ident = atomic_inc_return(&ident) - 1;
|
|
|
|
INIT_LIST_HEAD(&submit->node);
|
|
|
|
return submit;
|
|
}
|
|
|
|
void __msm_gem_submit_destroy(struct kref *kref)
|
|
{
|
|
struct msm_gem_submit *submit =
|
|
container_of(kref, struct msm_gem_submit, ref);
|
|
unsigned i;
|
|
|
|
/*
|
|
* In error paths, we could unref the submit without calling
|
|
* drm_sched_entity_push_job(), so msm_job_free() will never
|
|
* get called. Since drm_sched_job_cleanup() will NULL out
|
|
* s_fence, we can use that to detect this case.
|
|
*/
|
|
if (submit->base.s_fence)
|
|
drm_sched_job_cleanup(&submit->base);
|
|
|
|
if (submit->fence_id) {
|
|
spin_lock(&submit->queue->idr_lock);
|
|
idr_remove(&submit->queue->fence_idr, submit->fence_id);
|
|
spin_unlock(&submit->queue->idr_lock);
|
|
}
|
|
|
|
dma_fence_put(submit->user_fence);
|
|
|
|
/*
|
|
* If the submit is freed before msm_job_run(), then hw_fence is
|
|
* just some pre-allocated memory, not a reference counted fence.
|
|
* Once the job runs and the hw_fence is initialized, it will
|
|
* have a refcount of at least one, since the submit holds a ref
|
|
* to the hw_fence.
|
|
*/
|
|
if (kref_read(&submit->hw_fence->refcount) == 0) {
|
|
kfree(submit->hw_fence);
|
|
} else {
|
|
dma_fence_put(submit->hw_fence);
|
|
}
|
|
|
|
put_pid(submit->pid);
|
|
msm_submitqueue_put(submit->queue);
|
|
|
|
for (i = 0; i < submit->nr_cmds; i++)
|
|
kfree(submit->cmd[i].relocs);
|
|
|
|
kfree(submit);
|
|
}
|
|
|
|
static int submit_lookup_objects(struct msm_gem_submit *submit,
|
|
struct drm_msm_gem_submit *args, struct drm_file *file)
|
|
{
|
|
unsigned i;
|
|
int ret = 0;
|
|
|
|
for (i = 0; i < args->nr_bos; i++) {
|
|
struct drm_msm_gem_submit_bo submit_bo;
|
|
void __user *userptr =
|
|
u64_to_user_ptr(args->bos + (i * sizeof(submit_bo)));
|
|
|
|
/* make sure we don't have garbage flags, in case we hit
|
|
* error path before flags is initialized:
|
|
*/
|
|
submit->bos[i].flags = 0;
|
|
|
|
if (copy_from_user(&submit_bo, userptr, sizeof(submit_bo))) {
|
|
ret = -EFAULT;
|
|
i = 0;
|
|
goto out;
|
|
}
|
|
|
|
/* at least one of READ and/or WRITE flags should be set: */
|
|
#define MANDATORY_FLAGS (MSM_SUBMIT_BO_READ | MSM_SUBMIT_BO_WRITE)
|
|
|
|
if ((submit_bo.flags & ~MSM_SUBMIT_BO_FLAGS) ||
|
|
!(submit_bo.flags & MANDATORY_FLAGS)) {
|
|
ret = SUBMIT_ERROR(EINVAL, submit, "invalid flags: %x\n", submit_bo.flags);
|
|
i = 0;
|
|
goto out;
|
|
}
|
|
|
|
submit->bos[i].handle = submit_bo.handle;
|
|
submit->bos[i].flags = submit_bo.flags;
|
|
}
|
|
|
|
spin_lock(&file->table_lock);
|
|
|
|
for (i = 0; i < args->nr_bos; i++) {
|
|
struct drm_gem_object *obj;
|
|
|
|
/* normally use drm_gem_object_lookup(), but for bulk lookup
|
|
* all under single table_lock just hit object_idr directly:
|
|
*/
|
|
obj = idr_find(&file->object_idr, submit->bos[i].handle);
|
|
if (!obj) {
|
|
ret = SUBMIT_ERROR(EINVAL, submit, "invalid handle %u at index %u\n", submit->bos[i].handle, i);
|
|
goto out_unlock;
|
|
}
|
|
|
|
drm_gem_object_get(obj);
|
|
|
|
submit->bos[i].obj = obj;
|
|
}
|
|
|
|
out_unlock:
|
|
spin_unlock(&file->table_lock);
|
|
|
|
out:
|
|
submit->nr_bos = i;
|
|
|
|
return ret;
|
|
}
|
|
|
|
static int submit_lookup_cmds(struct msm_gem_submit *submit,
|
|
struct drm_msm_gem_submit *args, struct drm_file *file)
|
|
{
|
|
struct msm_context *ctx = file->driver_priv;
|
|
unsigned i;
|
|
size_t sz;
|
|
int ret = 0;
|
|
|
|
for (i = 0; i < args->nr_cmds; i++) {
|
|
struct drm_msm_gem_submit_cmd submit_cmd;
|
|
void __user *userptr =
|
|
u64_to_user_ptr(args->cmds + (i * sizeof(submit_cmd)));
|
|
|
|
ret = copy_from_user(&submit_cmd, userptr, sizeof(submit_cmd));
|
|
if (ret) {
|
|
ret = -EFAULT;
|
|
goto out;
|
|
}
|
|
|
|
/* validate input from userspace: */
|
|
switch (submit_cmd.type) {
|
|
case MSM_SUBMIT_CMD_BUF:
|
|
case MSM_SUBMIT_CMD_IB_TARGET_BUF:
|
|
case MSM_SUBMIT_CMD_CTX_RESTORE_BUF:
|
|
break;
|
|
default:
|
|
return SUBMIT_ERROR(EINVAL, submit, "invalid type: %08x\n", submit_cmd.type);
|
|
}
|
|
|
|
if (submit_cmd.size % 4) {
|
|
ret = SUBMIT_ERROR(EINVAL, submit, "non-aligned cmdstream buffer size: %u\n",
|
|
submit_cmd.size);
|
|
goto out;
|
|
}
|
|
|
|
if (msm_context_is_vmbind(ctx)) {
|
|
if (submit_cmd.nr_relocs) {
|
|
ret = SUBMIT_ERROR(EINVAL, submit, "nr_relocs must be zero");
|
|
goto out;
|
|
}
|
|
|
|
if (submit_cmd.submit_idx || submit_cmd.submit_offset) {
|
|
ret = SUBMIT_ERROR(EINVAL, submit, "submit_idx/offset must be zero");
|
|
goto out;
|
|
}
|
|
|
|
submit->cmd[i].iova = submit_cmd.iova;
|
|
}
|
|
|
|
submit->cmd[i].type = submit_cmd.type;
|
|
submit->cmd[i].size = submit_cmd.size / 4;
|
|
submit->cmd[i].offset = submit_cmd.submit_offset / 4;
|
|
submit->cmd[i].idx = submit_cmd.submit_idx;
|
|
submit->cmd[i].nr_relocs = submit_cmd.nr_relocs;
|
|
|
|
userptr = u64_to_user_ptr(submit_cmd.relocs);
|
|
|
|
sz = array_size(submit_cmd.nr_relocs,
|
|
sizeof(struct drm_msm_gem_submit_reloc));
|
|
/* check for overflow: */
|
|
if (sz == SIZE_MAX) {
|
|
ret = -ENOMEM;
|
|
goto out;
|
|
}
|
|
submit->cmd[i].relocs = kmalloc(sz, GFP_KERNEL | __GFP_NOWARN);
|
|
if (!submit->cmd[i].relocs) {
|
|
ret = -ENOMEM;
|
|
goto out;
|
|
}
|
|
ret = copy_from_user(submit->cmd[i].relocs, userptr, sz);
|
|
if (ret) {
|
|
ret = -EFAULT;
|
|
goto out;
|
|
}
|
|
}
|
|
|
|
out:
|
|
return ret;
|
|
}
|
|
|
|
static int submit_lock_objects_vmbind(struct msm_gem_submit *submit)
|
|
{
|
|
unsigned flags = DRM_EXEC_INTERRUPTIBLE_WAIT | DRM_EXEC_IGNORE_DUPLICATES;
|
|
struct drm_exec *exec = &submit->exec;
|
|
int ret = 0;
|
|
|
|
drm_exec_init(&submit->exec, flags, submit->nr_bos);
|
|
|
|
drm_exec_until_all_locked (&submit->exec) {
|
|
ret = drm_gpuvm_prepare_vm(submit->vm, exec, 1);
|
|
drm_exec_retry_on_contention(exec);
|
|
if (ret)
|
|
break;
|
|
|
|
ret = drm_gpuvm_prepare_objects(submit->vm, exec, 1);
|
|
drm_exec_retry_on_contention(exec);
|
|
if (ret)
|
|
break;
|
|
}
|
|
|
|
return ret;
|
|
}
|
|
|
|
/* This is where we make sure all the bo's are reserved and pin'd: */
|
|
static int submit_lock_objects(struct msm_gem_submit *submit)
|
|
{
|
|
unsigned flags = DRM_EXEC_INTERRUPTIBLE_WAIT;
|
|
int ret = 0;
|
|
|
|
if (msm_context_is_vmbind(submit->queue->ctx))
|
|
return submit_lock_objects_vmbind(submit);
|
|
|
|
drm_exec_init(&submit->exec, flags, submit->nr_bos);
|
|
|
|
drm_exec_until_all_locked (&submit->exec) {
|
|
ret = drm_exec_lock_obj(&submit->exec,
|
|
drm_gpuvm_resv_obj(submit->vm));
|
|
drm_exec_retry_on_contention(&submit->exec);
|
|
if (ret)
|
|
break;
|
|
for (unsigned i = 0; i < submit->nr_bos; i++) {
|
|
struct drm_gem_object *obj = submit->bos[i].obj;
|
|
ret = drm_exec_prepare_obj(&submit->exec, obj, 1);
|
|
drm_exec_retry_on_contention(&submit->exec);
|
|
if (ret)
|
|
break;
|
|
}
|
|
}
|
|
|
|
return ret;
|
|
}
|
|
|
|
static int submit_fence_sync(struct msm_gem_submit *submit)
|
|
{
|
|
int i, ret = 0;
|
|
|
|
for (i = 0; i < submit->nr_bos; i++) {
|
|
struct drm_gem_object *obj = submit->bos[i].obj;
|
|
bool write = submit->bos[i].flags & MSM_SUBMIT_BO_WRITE;
|
|
|
|
/* Otherwise userspace can ask for implicit sync to be
|
|
* disabled on specific buffers. This is useful for internal
|
|
* usermode driver managed buffers, suballocation, etc.
|
|
*/
|
|
if (submit->bos[i].flags & MSM_SUBMIT_BO_NO_IMPLICIT)
|
|
continue;
|
|
|
|
ret = drm_sched_job_add_implicit_dependencies(&submit->base,
|
|
obj,
|
|
write);
|
|
if (ret)
|
|
break;
|
|
}
|
|
|
|
return ret;
|
|
}
|
|
|
|
static int submit_pin_objects(struct msm_gem_submit *submit)
|
|
{
|
|
struct msm_drm_private *priv = submit->dev->dev_private;
|
|
int i, ret = 0;
|
|
|
|
for (i = 0; i < submit->nr_bos; i++) {
|
|
struct drm_gem_object *obj = submit->bos[i].obj;
|
|
struct drm_gpuva *vma;
|
|
|
|
/* if locking succeeded, pin bo: */
|
|
vma = msm_gem_get_vma_locked(obj, submit->vm);
|
|
if (IS_ERR(vma)) {
|
|
ret = PTR_ERR(vma);
|
|
break;
|
|
}
|
|
|
|
ret = msm_gem_pin_vma_locked(obj, vma);
|
|
if (ret)
|
|
break;
|
|
|
|
submit->bos[i].vm_bo = drm_gpuvm_bo_get(vma->vm_bo);
|
|
submit->bos[i].iova = vma->va.addr;
|
|
}
|
|
|
|
/*
|
|
* A second loop while holding the LRU lock (a) avoids acquiring/dropping
|
|
* the LRU lock for each individual bo, while (b) avoiding holding the
|
|
* LRU lock while calling msm_gem_pin_vma_locked() (which could trigger
|
|
* get_pages() which could trigger reclaim.. and if we held the LRU lock
|
|
* could trigger deadlock with the shrinker).
|
|
*/
|
|
mutex_lock(&priv->lru.lock);
|
|
for (i = 0; i < submit->nr_bos; i++) {
|
|
msm_gem_pin_obj_locked(submit->bos[i].obj);
|
|
}
|
|
mutex_unlock(&priv->lru.lock);
|
|
|
|
submit->bos_pinned = true;
|
|
|
|
return ret;
|
|
}
|
|
|
|
static void submit_unpin_objects(struct msm_gem_submit *submit)
|
|
{
|
|
if (!submit->bos_pinned)
|
|
return;
|
|
|
|
for (int i = 0; i < submit->nr_bos; i++) {
|
|
struct drm_gem_object *obj = submit->bos[i].obj;
|
|
|
|
msm_gem_unpin_locked(obj);
|
|
}
|
|
|
|
submit->bos_pinned = false;
|
|
}
|
|
|
|
static void submit_attach_object_fences(struct msm_gem_submit *submit)
|
|
{
|
|
struct msm_gem_vm *vm = to_msm_vm(submit->vm);
|
|
struct dma_fence *last_fence;
|
|
|
|
if (msm_context_is_vmbind(submit->queue->ctx)) {
|
|
drm_gpuvm_resv_add_fence(submit->vm, &submit->exec,
|
|
submit->user_fence,
|
|
DMA_RESV_USAGE_BOOKKEEP,
|
|
DMA_RESV_USAGE_BOOKKEEP);
|
|
return;
|
|
}
|
|
|
|
for (unsigned i = 0; i < submit->nr_bos; i++) {
|
|
struct drm_gem_object *obj = submit->bos[i].obj;
|
|
|
|
if (submit->bos[i].flags & MSM_SUBMIT_BO_WRITE)
|
|
dma_resv_add_fence(obj->resv, submit->user_fence,
|
|
DMA_RESV_USAGE_WRITE);
|
|
else if (submit->bos[i].flags & MSM_SUBMIT_BO_READ)
|
|
dma_resv_add_fence(obj->resv, submit->user_fence,
|
|
DMA_RESV_USAGE_READ);
|
|
}
|
|
|
|
last_fence = vm->last_fence;
|
|
vm->last_fence = dma_fence_unwrap_merge(submit->user_fence, last_fence);
|
|
dma_fence_put(last_fence);
|
|
}
|
|
|
|
static int submit_bo(struct msm_gem_submit *submit, uint32_t idx,
|
|
struct drm_gem_object **obj, uint64_t *iova)
|
|
{
|
|
if (idx >= submit->nr_bos) {
|
|
return SUBMIT_ERROR(EINVAL, submit, "invalid buffer index: %u (out of %u)\n",
|
|
idx, submit->nr_bos);
|
|
}
|
|
|
|
if (obj)
|
|
*obj = submit->bos[idx].obj;
|
|
if (iova)
|
|
*iova = submit->bos[idx].iova;
|
|
|
|
return 0;
|
|
}
|
|
|
|
/* process the reloc's and patch up the cmdstream as needed: */
|
|
static int submit_reloc(struct msm_gem_submit *submit, struct drm_gem_object *obj,
|
|
uint32_t offset, uint32_t nr_relocs, struct drm_msm_gem_submit_reloc *relocs)
|
|
{
|
|
uint32_t i, last_offset = 0;
|
|
uint32_t *ptr;
|
|
int ret = 0;
|
|
|
|
if (offset % 4)
|
|
return SUBMIT_ERROR(EINVAL, submit, "non-aligned cmdstream buffer: %u\n", offset);
|
|
|
|
/* For now, just map the entire thing. Eventually we probably
|
|
* to do it page-by-page, w/ kmap() if not vmap()d..
|
|
*/
|
|
ptr = msm_gem_get_vaddr_locked(obj);
|
|
|
|
if (IS_ERR(ptr)) {
|
|
ret = PTR_ERR(ptr);
|
|
DBG("failed to map: %d", ret);
|
|
return ret;
|
|
}
|
|
|
|
for (i = 0; i < nr_relocs; i++) {
|
|
struct drm_msm_gem_submit_reloc submit_reloc = relocs[i];
|
|
uint32_t off;
|
|
uint64_t iova;
|
|
|
|
if (submit_reloc.submit_offset % 4) {
|
|
ret = SUBMIT_ERROR(EINVAL, submit, "non-aligned reloc offset: %u\n",
|
|
submit_reloc.submit_offset);
|
|
goto out;
|
|
}
|
|
|
|
/* offset in dwords: */
|
|
off = submit_reloc.submit_offset / 4;
|
|
|
|
if ((off >= (obj->size / 4)) ||
|
|
(off < last_offset)) {
|
|
ret = SUBMIT_ERROR(EINVAL, submit, "invalid offset %u at reloc %u\n", off, i);
|
|
goto out;
|
|
}
|
|
|
|
ret = submit_bo(submit, submit_reloc.reloc_idx, NULL, &iova);
|
|
if (ret)
|
|
goto out;
|
|
|
|
iova += submit_reloc.reloc_offset;
|
|
|
|
if (submit_reloc.shift < 0)
|
|
iova >>= -submit_reloc.shift;
|
|
else
|
|
iova <<= submit_reloc.shift;
|
|
|
|
ptr[off] = iova | submit_reloc.or;
|
|
|
|
last_offset = off;
|
|
}
|
|
|
|
out:
|
|
msm_gem_put_vaddr_locked(obj);
|
|
|
|
return ret;
|
|
}
|
|
|
|
/* Cleanup submit at end of ioctl. In the error case, this also drops
|
|
* references, unpins, and drops active refcnt. In the non-error case,
|
|
* this is done when the submit is retired.
|
|
*/
|
|
static void submit_cleanup(struct msm_gem_submit *submit, bool error)
|
|
{
|
|
if (error)
|
|
submit_unpin_objects(submit);
|
|
|
|
if (submit->exec.objects)
|
|
drm_exec_fini(&submit->exec);
|
|
|
|
/* if job wasn't enqueued to scheduler, early retirement: */
|
|
if (error)
|
|
msm_submit_retire(submit);
|
|
}
|
|
|
|
void msm_submit_retire(struct msm_gem_submit *submit)
|
|
{
|
|
int i;
|
|
|
|
for (i = 0; i < submit->nr_bos; i++) {
|
|
struct drm_gem_object *obj = submit->bos[i].obj;
|
|
struct drm_gpuvm_bo *vm_bo = submit->bos[i].vm_bo;
|
|
|
|
msm_gem_lock(obj);
|
|
drm_gpuvm_bo_put(vm_bo);
|
|
msm_gem_unlock(obj);
|
|
drm_gem_object_put(obj);
|
|
}
|
|
}
|
|
|
|
int msm_ioctl_gem_submit(struct drm_device *dev, void *data,
|
|
struct drm_file *file)
|
|
{
|
|
struct msm_drm_private *priv = dev->dev_private;
|
|
struct drm_msm_gem_submit *args = data;
|
|
struct msm_context *ctx = file->driver_priv;
|
|
struct msm_gem_submit *submit = NULL;
|
|
struct msm_gpu *gpu = priv->gpu;
|
|
struct msm_gpu_submitqueue *queue;
|
|
struct msm_ringbuffer *ring;
|
|
struct msm_syncobj_post_dep *post_deps = NULL;
|
|
struct drm_syncobj **syncobjs_to_reset = NULL;
|
|
struct sync_file *sync_file = NULL;
|
|
unsigned cmds_to_parse;
|
|
int out_fence_fd = -1;
|
|
unsigned i;
|
|
int ret;
|
|
|
|
if (!gpu)
|
|
return -ENXIO;
|
|
|
|
if (args->pad)
|
|
return -EINVAL;
|
|
|
|
if (to_msm_vm(ctx->vm)->unusable)
|
|
return UERR(EPIPE, dev, "context is unusable");
|
|
|
|
/* for now, we just have 3d pipe.. eventually this would need to
|
|
* be more clever to dispatch to appropriate gpu module:
|
|
*/
|
|
if (MSM_PIPE_ID(args->flags) != MSM_PIPE_3D0)
|
|
return UERR(EINVAL, dev, "invalid pipe");
|
|
|
|
if (MSM_PIPE_FLAGS(args->flags) & ~MSM_SUBMIT_FLAGS)
|
|
return UERR(EINVAL, dev, "invalid flags");
|
|
|
|
if (args->flags & MSM_SUBMIT_SUDO) {
|
|
if (!IS_ENABLED(CONFIG_DRM_MSM_GPU_SUDO) ||
|
|
!capable(CAP_SYS_RAWIO))
|
|
return -EINVAL;
|
|
}
|
|
|
|
queue = msm_submitqueue_get(ctx, args->queueid);
|
|
if (!queue)
|
|
return -ENOENT;
|
|
|
|
if (queue->flags & MSM_SUBMITQUEUE_VM_BIND) {
|
|
ret = UERR(EINVAL, dev, "Invalid queue type");
|
|
goto out_post_unlock;
|
|
}
|
|
|
|
ring = gpu->rb[queue->ring_nr];
|
|
|
|
if (args->flags & MSM_SUBMIT_FENCE_FD_OUT) {
|
|
out_fence_fd = get_unused_fd_flags(O_CLOEXEC);
|
|
if (out_fence_fd < 0) {
|
|
ret = out_fence_fd;
|
|
goto out_post_unlock;
|
|
}
|
|
}
|
|
|
|
submit = submit_create(dev, gpu, queue, args->nr_bos, args->nr_cmds,
|
|
file->client_id);
|
|
if (IS_ERR(submit)) {
|
|
ret = PTR_ERR(submit);
|
|
goto out_post_unlock;
|
|
}
|
|
|
|
trace_msm_gpu_submit(pid_nr(submit->pid), ring->id, submit->ident,
|
|
args->nr_bos, args->nr_cmds);
|
|
|
|
ret = mutex_lock_interruptible(&queue->lock);
|
|
if (ret)
|
|
goto out_post_unlock;
|
|
|
|
if (args->flags & MSM_SUBMIT_SUDO)
|
|
submit->in_rb = true;
|
|
|
|
if (args->flags & MSM_SUBMIT_FENCE_FD_IN) {
|
|
struct dma_fence *in_fence;
|
|
|
|
in_fence = sync_file_get_fence(args->fence_fd);
|
|
|
|
if (!in_fence) {
|
|
ret = UERR(EINVAL, dev, "invalid in-fence");
|
|
goto out_unlock;
|
|
}
|
|
|
|
ret = drm_sched_job_add_dependency(&submit->base, in_fence);
|
|
if (ret)
|
|
goto out_unlock;
|
|
}
|
|
|
|
if (args->flags & MSM_SUBMIT_SYNCOBJ_IN) {
|
|
syncobjs_to_reset = msm_syncobj_parse_deps(dev, &submit->base,
|
|
file, args->in_syncobjs,
|
|
args->nr_in_syncobjs,
|
|
args->syncobj_stride);
|
|
if (IS_ERR(syncobjs_to_reset)) {
|
|
ret = PTR_ERR(syncobjs_to_reset);
|
|
goto out_unlock;
|
|
}
|
|
}
|
|
|
|
if (args->flags & MSM_SUBMIT_SYNCOBJ_OUT) {
|
|
post_deps = msm_syncobj_parse_post_deps(dev, file,
|
|
args->out_syncobjs,
|
|
args->nr_out_syncobjs,
|
|
args->syncobj_stride);
|
|
if (IS_ERR(post_deps)) {
|
|
ret = PTR_ERR(post_deps);
|
|
goto out_unlock;
|
|
}
|
|
}
|
|
|
|
ret = submit_lookup_objects(submit, args, file);
|
|
if (ret)
|
|
goto out;
|
|
|
|
ret = submit_lookup_cmds(submit, args, file);
|
|
if (ret)
|
|
goto out;
|
|
|
|
/* copy_*_user while holding a ww ticket upsets lockdep */
|
|
ret = submit_lock_objects(submit);
|
|
if (ret)
|
|
goto out;
|
|
|
|
if (!(args->flags & MSM_SUBMIT_NO_IMPLICIT)) {
|
|
ret = submit_fence_sync(submit);
|
|
if (ret)
|
|
goto out;
|
|
}
|
|
|
|
ret = submit_pin_objects(submit);
|
|
if (ret)
|
|
goto out;
|
|
|
|
cmds_to_parse = msm_context_is_vmbind(ctx) ? 0 : args->nr_cmds;
|
|
|
|
for (i = 0; i < cmds_to_parse; i++) {
|
|
struct drm_gem_object *obj;
|
|
uint64_t iova;
|
|
|
|
ret = submit_bo(submit, submit->cmd[i].idx, &obj, &iova);
|
|
if (ret)
|
|
goto out;
|
|
|
|
if (!submit->cmd[i].size ||
|
|
(size_add(submit->cmd[i].size, submit->cmd[i].offset) > obj->size / 4)) {
|
|
ret = UERR(EINVAL, dev, "invalid cmdstream size: %u\n",
|
|
submit->cmd[i].size * 4);
|
|
goto out;
|
|
}
|
|
|
|
submit->cmd[i].iova = iova + (submit->cmd[i].offset * 4);
|
|
|
|
if (likely(!submit->cmd[i].nr_relocs))
|
|
continue;
|
|
|
|
if (!gpu->allow_relocs) {
|
|
ret = UERR(EINVAL, dev, "relocs not allowed\n");
|
|
goto out;
|
|
}
|
|
|
|
ret = submit_reloc(submit, obj, submit->cmd[i].offset * 4,
|
|
submit->cmd[i].nr_relocs, submit->cmd[i].relocs);
|
|
if (ret)
|
|
goto out;
|
|
}
|
|
|
|
submit->nr_cmds = args->nr_cmds;
|
|
|
|
idr_preload(GFP_KERNEL);
|
|
|
|
spin_lock(&queue->idr_lock);
|
|
|
|
/*
|
|
* If using userspace provided seqno fence, validate that the id
|
|
* is available before arming sched job. Since access to fence_idr
|
|
* is serialized on the queue lock, the slot should be still avail
|
|
* after the job is armed
|
|
*/
|
|
if ((args->flags & MSM_SUBMIT_FENCE_SN_IN) &&
|
|
(!args->fence || idr_find(&queue->fence_idr, args->fence))) {
|
|
spin_unlock(&queue->idr_lock);
|
|
idr_preload_end();
|
|
ret = UERR(EINVAL, dev, "invalid in-fence-sn");
|
|
goto out;
|
|
}
|
|
|
|
drm_sched_job_arm(&submit->base);
|
|
|
|
submit->user_fence = dma_fence_get(&submit->base.s_fence->finished);
|
|
|
|
if (args->flags & MSM_SUBMIT_FENCE_SN_IN) {
|
|
/*
|
|
* Userspace has assigned the seqno fence that it wants
|
|
* us to use. It is an error to pick a fence sequence
|
|
* number that is not available.
|
|
*/
|
|
submit->fence_id = args->fence;
|
|
ret = idr_alloc_u32(&queue->fence_idr, submit->user_fence,
|
|
&submit->fence_id, submit->fence_id,
|
|
GFP_NOWAIT);
|
|
/*
|
|
* We've already validated that the fence_id slot is valid,
|
|
* so if idr_alloc_u32 failed, it is a kernel bug
|
|
*/
|
|
WARN_ON(ret);
|
|
} else {
|
|
/*
|
|
* Allocate an id which can be used by WAIT_FENCE ioctl to map
|
|
* back to the underlying fence.
|
|
*/
|
|
submit->fence_id = idr_alloc_cyclic(&queue->fence_idr,
|
|
submit->user_fence, 1,
|
|
INT_MAX, GFP_NOWAIT);
|
|
}
|
|
|
|
spin_unlock(&queue->idr_lock);
|
|
idr_preload_end();
|
|
|
|
if (submit->fence_id < 0) {
|
|
ret = submit->fence_id;
|
|
submit->fence_id = 0;
|
|
}
|
|
|
|
if (ret == 0 && args->flags & MSM_SUBMIT_FENCE_FD_OUT) {
|
|
sync_file = sync_file_create(submit->user_fence);
|
|
if (!sync_file)
|
|
ret = -ENOMEM;
|
|
}
|
|
|
|
if (ret)
|
|
goto out;
|
|
|
|
submit_attach_object_fences(submit);
|
|
|
|
if (msm_context_is_vmbind(ctx)) {
|
|
/*
|
|
* If we are not using VM_BIND, submit_pin_vmas() will validate
|
|
* just the BOs attached to the submit. In that case we don't
|
|
* need to validate the _entire_ vm, because userspace tracked
|
|
* what BOs are associated with the submit.
|
|
*/
|
|
ret = drm_gpuvm_validate(submit->vm, &submit->exec);
|
|
if (ret)
|
|
goto out;
|
|
}
|
|
|
|
/* The scheduler owns a ref now: */
|
|
msm_gem_submit_get(submit);
|
|
|
|
msm_rd_dump_submit(priv->rd, submit, NULL);
|
|
|
|
drm_sched_entity_push_job(&submit->base);
|
|
|
|
args->fence = submit->fence_id;
|
|
queue->last_fence = submit->fence_id;
|
|
|
|
msm_syncobj_reset(syncobjs_to_reset, args->nr_in_syncobjs);
|
|
msm_syncobj_process_post_deps(post_deps, args->nr_out_syncobjs, submit->user_fence);
|
|
|
|
out:
|
|
submit_cleanup(submit, !!ret);
|
|
out_unlock:
|
|
mutex_unlock(&queue->lock);
|
|
out_post_unlock:
|
|
if (ret) {
|
|
if (out_fence_fd >= 0)
|
|
put_unused_fd(out_fence_fd);
|
|
if (sync_file)
|
|
fput(sync_file->file);
|
|
} else if (sync_file) {
|
|
fd_install(out_fence_fd, sync_file->file);
|
|
args->fence_fd = out_fence_fd;
|
|
}
|
|
|
|
if (!IS_ERR_OR_NULL(submit)) {
|
|
msm_gem_submit_put(submit);
|
|
} else {
|
|
/*
|
|
* If the submit hasn't yet taken ownership of the queue
|
|
* then we need to drop the reference ourself:
|
|
*/
|
|
msm_submitqueue_put(queue);
|
|
}
|
|
if (!IS_ERR_OR_NULL(post_deps)) {
|
|
for (i = 0; i < args->nr_out_syncobjs; ++i) {
|
|
kfree(post_deps[i].chain);
|
|
drm_syncobj_put(post_deps[i].syncobj);
|
|
}
|
|
kfree(post_deps);
|
|
}
|
|
|
|
if (!IS_ERR_OR_NULL(syncobjs_to_reset)) {
|
|
for (i = 0; i < args->nr_in_syncobjs; ++i) {
|
|
if (syncobjs_to_reset[i])
|
|
drm_syncobj_put(syncobjs_to_reset[i]);
|
|
}
|
|
kfree(syncobjs_to_reset);
|
|
}
|
|
|
|
return ret;
|
|
}
|