A minor function interface change and some memcpy bugfixing.

Hooray!! it sort of works with a fixed AGP area as faked VRAM.
main
Thomas Hellstrom 2007-02-08 16:21:38 +01:00
parent 1257907fa9
commit e4b2da4406
4 changed files with 31 additions and 29 deletions

View File

@ -697,9 +697,8 @@ typedef struct drm_bo_driver{
int (*init_mem_type)(struct drm_device *dev, uint32_t type, int (*init_mem_type)(struct drm_device *dev, uint32_t type,
drm_mem_type_manager_t *man); drm_mem_type_manager_t *man);
uint32_t (*evict_flags) (struct drm_device *dev, uint32_t type); uint32_t (*evict_flags) (struct drm_device *dev, uint32_t type);
int (*move)(struct drm_device *dev, int (*move)(struct drm_buffer_object *bo,
struct drm_ttm *ttm, int evict, int no_wait, int evict, int no_wait,
struct drm_bo_mem_reg *old_mem,
struct drm_bo_mem_reg *new_mem); struct drm_bo_mem_reg *new_mem);
} drm_bo_driver_t; } drm_bo_driver_t;
@ -1517,15 +1516,13 @@ extern int drm_fence_buffer_objects(drm_file_t * priv,
* drm_bo_move.c * drm_bo_move.c
*/ */
extern int drm_bo_move_ttm(drm_device_t *dev, extern int drm_bo_move_ttm(drm_buffer_object_t *bo,
drm_ttm_t *ttm, int evict, int evict,
int no_wait, int no_wait,
drm_bo_mem_reg_t *old_mem,
drm_bo_mem_reg_t *new_mem); drm_bo_mem_reg_t *new_mem);
extern int drm_bo_move_memcpy(drm_device_t *dev, extern int drm_bo_move_memcpy(drm_buffer_object_t *bo,
drm_ttm_t *ttm, int evict, int evict,
int no_wait, int no_wait,
drm_bo_mem_reg_t *old_mem,
drm_bo_mem_reg_t *new_mem); drm_bo_mem_reg_t *new_mem);
extern void drm_core_ioremap(struct drm_map *map, struct drm_device *dev); extern void drm_core_ioremap(struct drm_map *map, struct drm_device *dev);
extern void drm_core_ioremapfree(struct drm_map *map, struct drm_device *dev); extern void drm_core_ioremapfree(struct drm_map *map, struct drm_device *dev);

View File

@ -124,7 +124,6 @@ static int drm_bo_add_ttm(drm_buffer_object_t * bo)
{ {
drm_device_t *dev = bo->dev; drm_device_t *dev = bo->dev;
int ret = 0; int ret = 0;
bo->ttm = NULL; bo->ttm = NULL;
switch (bo->type) { switch (bo->type) {
@ -174,14 +173,11 @@ static int drm_bo_handle_move_mem(drm_buffer_object_t *bo,
if (!(old_man->flags & _DRM_FLAG_MEMTYPE_FIXED) && if (!(old_man->flags & _DRM_FLAG_MEMTYPE_FIXED) &&
!(new_man->flags & _DRM_FLAG_MEMTYPE_FIXED)) { !(new_man->flags & _DRM_FLAG_MEMTYPE_FIXED)) {
ret = drm_bo_move_ttm(dev, bo->ttm, evict, no_wait, ret = drm_bo_move_ttm(bo, evict, no_wait, mem);
&bo->mem, mem);
} else if (dev->driver->bo_driver->move) { } else if (dev->driver->bo_driver->move) {
ret = dev->driver->bo_driver->move(dev, bo->ttm, evict, ret = dev->driver->bo_driver->move(bo, evict, no_wait, mem);
no_wait, &bo->mem, mem);
} else { } else {
ret = drm_bo_move_memcpy(dev, bo->ttm, evict, no_wait, ret = drm_bo_move_memcpy(bo, evict, no_wait, mem);
&bo->mem, mem);
} }
if (old_is_pci || new_is_pci) if (old_is_pci || new_is_pci)

View File

@ -30,13 +30,14 @@
#include "drmP.h" #include "drmP.h"
int drm_bo_move_ttm(drm_device_t *dev, int drm_bo_move_ttm(drm_buffer_object_t *bo,
drm_ttm_t *ttm,
int evict, int evict,
int no_wait, int no_wait,
drm_bo_mem_reg_t *old_mem,
drm_bo_mem_reg_t *new_mem) drm_bo_mem_reg_t *new_mem)
{ {
drm_device_t *dev = bo->dev;
drm_ttm_t *ttm = bo->ttm;
drm_bo_mem_reg_t *old_mem = &bo->mem;
uint32_t save_flags = old_mem->flags; uint32_t save_flags = old_mem->flags;
uint32_t save_mask = old_mem->mask; uint32_t save_mask = old_mem->mask;
int ret; int ret;
@ -135,8 +136,9 @@ void drm_mem_reg_iounmap(drm_device_t *dev, drm_bo_mem_reg_t *mem,
bm = &dev->bm; bm = &dev->bm;
man = &bm->man[mem->mem_type]; man = &bm->man[mem->mem_type];
if (virtual && (man->flags & _DRM_FLAG_NEEDS_IOREMAP)) if (virtual && (man->flags & _DRM_FLAG_NEEDS_IOREMAP)) {
iounmap(virtual); iounmap(virtual);
}
} }
@ -188,13 +190,16 @@ static int drm_copy_ttm_io_page(drm_ttm_t *ttm, void *dst, unsigned long page)
} }
int drm_bo_move_memcpy(drm_device_t *dev, int drm_bo_move_memcpy(drm_buffer_object_t *bo,
drm_ttm_t *ttm,
int evict, int evict,
int no_wait, int no_wait,
drm_bo_mem_reg_t *old_mem,
drm_bo_mem_reg_t *new_mem) drm_bo_mem_reg_t *new_mem)
{ {
drm_device_t *dev = bo->dev;
drm_mem_type_manager_t *man = &dev->bm.man[new_mem->mem_type];
drm_ttm_t *ttm = bo->ttm;
drm_bo_mem_reg_t *old_mem = &bo->mem;
drm_bo_mem_reg_t old_copy = *old_mem;
void *old_iomap; void *old_iomap;
void *new_iomap; void *new_iomap;
int ret; int ret;
@ -205,7 +210,6 @@ int drm_bo_move_memcpy(drm_device_t *dev,
unsigned long add = 0; unsigned long add = 0;
int dir; int dir;
ret = drm_mem_reg_ioremap(dev, old_mem, &old_iomap); ret = drm_mem_reg_ioremap(dev, old_mem, &old_iomap);
if (ret) if (ret)
return ret; return ret;
@ -237,15 +241,22 @@ int drm_bo_move_memcpy(drm_device_t *dev,
if (ret) if (ret)
goto out1; goto out1;
} }
mb();
out2: out2:
*old_mem = *new_mem; *old_mem = *new_mem;
new_mem->mm_node = NULL; new_mem->mm_node = NULL;
old_mem->mask = save_mask; old_mem->mask = save_mask;
DRM_FLAG_MASKED(save_flags, new_mem->flags, DRM_BO_MASK_MEMTYPE); DRM_FLAG_MASKED(save_flags, new_mem->flags, DRM_BO_MASK_MEMTYPE);
if ((man->flags & _DRM_FLAG_MEMTYPE_FIXED) && (ttm != NULL)) {
drm_ttm_unbind(ttm);
drm_destroy_ttm(ttm);
bo->ttm = NULL;
}
out1: out1:
drm_mem_reg_iounmap(dev, new_mem, &new_iomap); drm_mem_reg_iounmap(dev, new_mem, new_iomap);
out: out:
drm_mem_reg_iounmap(dev, old_mem, old_iomap); drm_mem_reg_iounmap(dev, &old_copy, old_iomap);
return ret; return ret;
} }

View File

@ -250,7 +250,6 @@ struct page *drm_bo_vm_nopage(struct vm_area_struct *vma,
} }
get_page(page); get_page(page);
out_unlock: out_unlock:
mutex_unlock(&bo->mutex); mutex_unlock(&bo->mutex);
return page; return page;
@ -274,7 +273,6 @@ int drm_bo_map_bound(struct vm_area_struct *vma)
if (bus_size) { if (bus_size) {
unsigned long pfn = (bus_base + bus_offset) >> PAGE_SHIFT; unsigned long pfn = (bus_base + bus_offset) >> PAGE_SHIFT;
pgprot_t pgprot = drm_io_prot(_DRM_AGP, vma); pgprot_t pgprot = drm_io_prot(_DRM_AGP, vma);
ret = io_remap_pfn_range(vma, vma->vm_start, pfn, ret = io_remap_pfn_range(vma, vma->vm_start, pfn,
vma->vm_end - vma->vm_start, vma->vm_end - vma->vm_start,
pgprot); pgprot);