Commit 09855acb authored by Jerome Glisse's avatar Jerome Glisse Committed by Dave Airlie

drm/ttm: Convert ttm_buffer_object_init to use ttm_placement

Convert ttm_buffer_object_init to use struct ttm_placement and
rename to ttm_bo_init for consistency with function naming. This
allow to give more complex placement at buffer creation. For
instance you ask to allocate bo into vram first but if there is
not enough vram you can give system as a second possible
placement. It also allow to create buffer in a specific range.

Also rename ttm_buffer_object_validate to ttm_bo_validate.
Signed-off-by: default avatarJerome Glisse <jglisse@redhat.com>
Signed-off-by: default avatarDave Airlie <airlied@redhat.com>
parent 4361e52a
...@@ -1002,7 +1002,7 @@ static int ttm_bo_mem_compat(struct ttm_placement *placement, ...@@ -1002,7 +1002,7 @@ static int ttm_bo_mem_compat(struct ttm_placement *placement,
return -1; return -1;
} }
int ttm_buffer_object_validate(struct ttm_buffer_object *bo, int ttm_bo_validate(struct ttm_buffer_object *bo,
struct ttm_placement *placement, struct ttm_placement *placement,
bool interruptible, bool no_wait) bool interruptible, bool no_wait)
{ {
...@@ -1040,44 +1040,48 @@ int ttm_buffer_object_validate(struct ttm_buffer_object *bo, ...@@ -1040,44 +1040,48 @@ int ttm_buffer_object_validate(struct ttm_buffer_object *bo,
} }
return 0; return 0;
} }
EXPORT_SYMBOL(ttm_buffer_object_validate); EXPORT_SYMBOL(ttm_bo_validate);
int int ttm_bo_check_placement(struct ttm_buffer_object *bo,
ttm_bo_check_placement(struct ttm_buffer_object *bo, struct ttm_placement *placement)
uint32_t set_flags, uint32_t clr_flags)
{ {
uint32_t new_mask = set_flags | clr_flags; int i;
if ((bo->type == ttm_bo_type_user) && if (placement->fpfn || placement->lpfn) {
(clr_flags & TTM_PL_FLAG_CACHED)) { if (bo->mem.num_pages > (placement->lpfn - placement->fpfn)) {
printk(KERN_ERR TTM_PFX printk(KERN_ERR TTM_PFX "Page number range to small "
"User buffers require cache-coherent memory.\n"); "Need %lu pages, range is [%u, %u]\n",
bo->mem.num_pages, placement->fpfn,
placement->lpfn);
return -EINVAL; return -EINVAL;
} }
}
for (i = 0; i < placement->num_placement; i++) {
if (!capable(CAP_SYS_ADMIN)) { if (!capable(CAP_SYS_ADMIN)) {
if (new_mask & TTM_PL_FLAG_NO_EVICT) { if (placement->placement[i] & TTM_PL_FLAG_NO_EVICT) {
printk(KERN_ERR TTM_PFX "Need to be root to modify" printk(KERN_ERR TTM_PFX "Need to be root to "
" NO_EVICT status.\n"); "modify NO_EVICT status.\n");
return -EINVAL; return -EINVAL;
} }
}
if ((clr_flags & bo->mem.placement & TTM_PL_MASK_MEMTYPE) && }
(bo->mem.placement & TTM_PL_FLAG_NO_EVICT)) { for (i = 0; i < placement->num_busy_placement; i++) {
printk(KERN_ERR TTM_PFX if (!capable(CAP_SYS_ADMIN)) {
"Incompatible memory specification" if (placement->busy_placement[i] & TTM_PL_FLAG_NO_EVICT) {
" for NO_EVICT buffer.\n"); printk(KERN_ERR TTM_PFX "Need to be root to "
"modify NO_EVICT status.\n");
return -EINVAL; return -EINVAL;
} }
} }
}
return 0; return 0;
} }
int ttm_buffer_object_init(struct ttm_bo_device *bdev, int ttm_bo_init(struct ttm_bo_device *bdev,
struct ttm_buffer_object *bo, struct ttm_buffer_object *bo,
unsigned long size, unsigned long size,
enum ttm_bo_type type, enum ttm_bo_type type,
uint32_t flags, struct ttm_placement *placement,
uint32_t page_alignment, uint32_t page_alignment,
unsigned long buffer_start, unsigned long buffer_start,
bool interruptible, bool interruptible,
...@@ -1085,10 +1089,8 @@ int ttm_buffer_object_init(struct ttm_bo_device *bdev, ...@@ -1085,10 +1089,8 @@ int ttm_buffer_object_init(struct ttm_bo_device *bdev,
size_t acc_size, size_t acc_size,
void (*destroy) (struct ttm_buffer_object *)) void (*destroy) (struct ttm_buffer_object *))
{ {
int i, c, ret = 0; int ret = 0;
unsigned long num_pages; unsigned long num_pages;
uint32_t placements[8];
struct ttm_placement placement;
size += buffer_start & ~PAGE_MASK; size += buffer_start & ~PAGE_MASK;
num_pages = (size + PAGE_SIZE - 1) >> PAGE_SHIFT; num_pages = (size + PAGE_SIZE - 1) >> PAGE_SHIFT;
...@@ -1123,38 +1125,21 @@ int ttm_buffer_object_init(struct ttm_bo_device *bdev, ...@@ -1123,38 +1125,21 @@ int ttm_buffer_object_init(struct ttm_bo_device *bdev,
bo->acc_size = acc_size; bo->acc_size = acc_size;
atomic_inc(&bo->glob->bo_count); atomic_inc(&bo->glob->bo_count);
ret = ttm_bo_check_placement(bo, flags, 0ULL); ret = ttm_bo_check_placement(bo, placement);
if (unlikely(ret != 0)) if (unlikely(ret != 0))
goto out_err; goto out_err;
/*
* If no caching attributes are set, accept any form of caching.
*/
if ((flags & TTM_PL_MASK_CACHING) == 0)
flags |= TTM_PL_MASK_CACHING;
/* /*
* For ttm_bo_type_device buffers, allocate * For ttm_bo_type_device buffers, allocate
* address space from the device. * address space from the device.
*/ */
if (bo->type == ttm_bo_type_device) { if (bo->type == ttm_bo_type_device) {
ret = ttm_bo_setup_vm(bo); ret = ttm_bo_setup_vm(bo);
if (ret) if (ret)
goto out_err; goto out_err;
} }
placement.fpfn = 0; ret = ttm_bo_validate(bo, placement, interruptible, false);
placement.lpfn = 0;
for (i = 0, c = 0; i <= TTM_PL_PRIV5; i++)
if (flags & (1 << i))
placements[c++] = (flags & ~TTM_PL_MASK_MEM) | (1 << i);
placement.placement = placements;
placement.num_placement = c;
placement.busy_placement = placements;
placement.num_busy_placement = c;
ret = ttm_buffer_object_validate(bo, &placement, interruptible, false);
if (ret) if (ret)
goto out_err; goto out_err;
...@@ -1167,7 +1152,7 @@ out_err: ...@@ -1167,7 +1152,7 @@ out_err:
return ret; return ret;
} }
EXPORT_SYMBOL(ttm_buffer_object_init); EXPORT_SYMBOL(ttm_bo_init);
static inline size_t ttm_bo_size(struct ttm_bo_global *glob, static inline size_t ttm_bo_size(struct ttm_bo_global *glob,
unsigned long num_pages) unsigned long num_pages)
...@@ -1178,10 +1163,10 @@ static inline size_t ttm_bo_size(struct ttm_bo_global *glob, ...@@ -1178,10 +1163,10 @@ static inline size_t ttm_bo_size(struct ttm_bo_global *glob,
return glob->ttm_bo_size + 2 * page_array_size; return glob->ttm_bo_size + 2 * page_array_size;
} }
int ttm_buffer_object_create(struct ttm_bo_device *bdev, int ttm_bo_create(struct ttm_bo_device *bdev,
unsigned long size, unsigned long size,
enum ttm_bo_type type, enum ttm_bo_type type,
uint32_t flags, struct ttm_placement *placement,
uint32_t page_alignment, uint32_t page_alignment,
unsigned long buffer_start, unsigned long buffer_start,
bool interruptible, bool interruptible,
...@@ -1205,9 +1190,8 @@ int ttm_buffer_object_create(struct ttm_bo_device *bdev, ...@@ -1205,9 +1190,8 @@ int ttm_buffer_object_create(struct ttm_bo_device *bdev,
return -ENOMEM; return -ENOMEM;
} }
ret = ttm_buffer_object_init(bdev, bo, size, type, flags, ret = ttm_bo_init(bdev, bo, size, type, placement, page_alignment,
page_alignment, buffer_start, buffer_start, interruptible,
interruptible,
persistant_swap_storage, acc_size, NULL); persistant_swap_storage, acc_size, NULL);
if (likely(ret == 0)) if (likely(ret == 0))
*p_bo = bo; *p_bo = bo;
......
...@@ -308,7 +308,7 @@ ttm_bo_reference(struct ttm_buffer_object *bo) ...@@ -308,7 +308,7 @@ ttm_bo_reference(struct ttm_buffer_object *bo)
extern int ttm_bo_wait(struct ttm_buffer_object *bo, bool lazy, extern int ttm_bo_wait(struct ttm_buffer_object *bo, bool lazy,
bool interruptible, bool no_wait); bool interruptible, bool no_wait);
/** /**
* ttm_buffer_object_validate * ttm_bo_validate
* *
* @bo: The buffer object. * @bo: The buffer object.
* @placement: Proposed placement for the buffer object. * @placement: Proposed placement for the buffer object.
...@@ -323,7 +323,7 @@ extern int ttm_bo_wait(struct ttm_buffer_object *bo, bool lazy, ...@@ -323,7 +323,7 @@ extern int ttm_bo_wait(struct ttm_buffer_object *bo, bool lazy,
* -EBUSY if no_wait is true and buffer busy. * -EBUSY if no_wait is true and buffer busy.
* -ERESTARTSYS if interrupted by a signal. * -ERESTARTSYS if interrupted by a signal.
*/ */
extern int ttm_buffer_object_validate(struct ttm_buffer_object *bo, extern int ttm_bo_validate(struct ttm_buffer_object *bo,
struct ttm_placement *placement, struct ttm_placement *placement,
bool interruptible, bool no_wait); bool interruptible, bool no_wait);
...@@ -362,7 +362,7 @@ ttm_bo_synccpu_write_grab(struct ttm_buffer_object *bo, bool no_wait); ...@@ -362,7 +362,7 @@ ttm_bo_synccpu_write_grab(struct ttm_buffer_object *bo, bool no_wait);
extern void ttm_bo_synccpu_write_release(struct ttm_buffer_object *bo); extern void ttm_bo_synccpu_write_release(struct ttm_buffer_object *bo);
/** /**
* ttm_buffer_object_init * ttm_bo_init
* *
* @bdev: Pointer to a ttm_bo_device struct. * @bdev: Pointer to a ttm_bo_device struct.
* @bo: Pointer to a ttm_buffer_object to be initialized. * @bo: Pointer to a ttm_buffer_object to be initialized.
...@@ -393,11 +393,11 @@ extern void ttm_bo_synccpu_write_release(struct ttm_buffer_object *bo); ...@@ -393,11 +393,11 @@ extern void ttm_bo_synccpu_write_release(struct ttm_buffer_object *bo);
* -ERESTARTSYS: Interrupted by signal while sleeping waiting for resources. * -ERESTARTSYS: Interrupted by signal while sleeping waiting for resources.
*/ */
extern int ttm_buffer_object_init(struct ttm_bo_device *bdev, extern int ttm_bo_init(struct ttm_bo_device *bdev,
struct ttm_buffer_object *bo, struct ttm_buffer_object *bo,
unsigned long size, unsigned long size,
enum ttm_bo_type type, enum ttm_bo_type type,
uint32_t flags, struct ttm_placement *placement,
uint32_t page_alignment, uint32_t page_alignment,
unsigned long buffer_start, unsigned long buffer_start,
bool interrubtible, bool interrubtible,
...@@ -424,19 +424,18 @@ extern int ttm_buffer_object_init(struct ttm_bo_device *bdev, ...@@ -424,19 +424,18 @@ extern int ttm_buffer_object_init(struct ttm_bo_device *bdev,
* GEM user interface. * GEM user interface.
* @p_bo: On successful completion *p_bo points to the created object. * @p_bo: On successful completion *p_bo points to the created object.
* *
* This function allocates a ttm_buffer_object, and then calls * This function allocates a ttm_buffer_object, and then calls ttm_bo_init
* ttm_buffer_object_init on that object. * on that object. The destroy function is set to kfree().
* The destroy function is set to kfree().
* Returns * Returns
* -ENOMEM: Out of memory. * -ENOMEM: Out of memory.
* -EINVAL: Invalid placement flags. * -EINVAL: Invalid placement flags.
* -ERESTARTSYS: Interrupted by signal while waiting for resources. * -ERESTARTSYS: Interrupted by signal while waiting for resources.
*/ */
extern int ttm_buffer_object_create(struct ttm_bo_device *bdev, extern int ttm_bo_create(struct ttm_bo_device *bdev,
unsigned long size, unsigned long size,
enum ttm_bo_type type, enum ttm_bo_type type,
uint32_t flags, struct ttm_placement *placement,
uint32_t page_alignment, uint32_t page_alignment,
unsigned long buffer_start, unsigned long buffer_start,
bool interruptible, bool interruptible,
...@@ -447,17 +446,15 @@ extern int ttm_buffer_object_create(struct ttm_bo_device *bdev, ...@@ -447,17 +446,15 @@ extern int ttm_buffer_object_create(struct ttm_bo_device *bdev,
* ttm_bo_check_placement * ttm_bo_check_placement
* *
* @bo: the buffer object. * @bo: the buffer object.
* @set_flags: placement flags to set. * @placement: placements
* @clr_flags: placement flags to clear.
* *
* Performs minimal validity checking on an intended change of * Performs minimal validity checking on an intended change of
* placement flags. * placement flags.
* Returns * Returns
* -EINVAL: Intended change is invalid or not allowed. * -EINVAL: Intended change is invalid or not allowed.
*/ */
extern int ttm_bo_check_placement(struct ttm_buffer_object *bo, extern int ttm_bo_check_placement(struct ttm_buffer_object *bo,
uint32_t set_flags, uint32_t clr_flags); struct ttm_placement *placement);
/** /**
* ttm_bo_init_mm * ttm_bo_init_mm
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment