Am 07.02.19 um 09:59 schrieb Thomas Zimmermann:> Almost all TTM-based drivers use the same values for the mmap-able
> range of BO addresses. Each driver therefore duplicates the
> DRM_FILE_PAGE_OFFSET constant. OTOH, the mmap range's size is not
> configurable by drivers.
>
> This patch set replaces driver-specific configuration with a single
> setup. All code is located within TTM. TTM and GEM share the same
> range for mmap-able BOs.
>
> Thomas Zimmermann (5):
> staging/vboxvideo: Use same BO mmap offset as other drivers
> drm/ttm: Define a single DRM_FILE_PAGE_OFFSET constant
> drm/ttm: Remove file_page_offset parameter from ttm_bo_device_init()
> drm/ttm: Quick-test mmap offset in ttm_bo_mmap()
> drm: Use the same mmap-range offset and size for GEM and TTM
Reviewed-by: Christian K?nig <christian.koenig at amd.com> for the whole
series.
Nice cleanup! Thanks,
Christian.
>
> drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 12 ++----------
> drivers/gpu/drm/ast/ast_drv.h | 2 --
> drivers/gpu/drm/ast/ast_ttm.c | 10 ++--------
> drivers/gpu/drm/bochs/bochs.h | 2 --
> drivers/gpu/drm/bochs/bochs_mm.c | 10 ++--------
> drivers/gpu/drm/cirrus/cirrus_drv.h | 1 -
> drivers/gpu/drm/cirrus/cirrus_ttm.c | 10 ++--------
> drivers/gpu/drm/drm_gem.c | 17 -----------------
> drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c | 12 ++----------
> drivers/gpu/drm/mgag200/mgag200_drv.h | 1 -
> drivers/gpu/drm/mgag200/mgag200_ttm.c | 10 ++--------
> drivers/gpu/drm/nouveau/nouveau_drv.h | 2 --
> drivers/gpu/drm/nouveau/nouveau_ttm.c | 4 ----
> drivers/gpu/drm/qxl/qxl_drv.h | 3 ---
> drivers/gpu/drm/qxl/qxl_ttm.c | 11 +++--------
> drivers/gpu/drm/radeon/radeon_ttm.c | 13 ++-----------
> drivers/gpu/drm/ttm/ttm_bo.c | 6 +++---
> drivers/gpu/drm/ttm/ttm_bo_vm.c | 3 +++
> drivers/gpu/drm/virtio/virtgpu_ttm.c | 4 +---
> drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 1 -
> drivers/gpu/drm/vmwgfx/vmwgfx_drv.h | 1 -
> drivers/gpu/drm/vmwgfx/vmwgfx_ttm_glue.c | 11 ++---------
> drivers/staging/vboxvideo/vbox_drv.h | 2 --
> drivers/staging/vboxvideo/vbox_ttm.c | 12 +++---------
> include/drm/drm_vma_manager.h | 12 ++++++++++++
> include/drm/ttm/ttm_bo_driver.h | 2 +-
> 26 files changed, 42 insertions(+), 132 deletions(-)
>