Re: [PATCH 5/6] drm/ttm: revert "implement LRU add callbacks v2"
Minor typo On Fri, Jan 13, 2017 at 10:51:10AM +0100, Christian König wrote: > From: Christian König> > The additional housekeeping had to much CPU overhead, ^ > let's use the BO priorities instead. > > Signed-off-by: Christian König > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 2 -- > drivers/gpu/drm/ast/ast_ttm.c | 2 -- > drivers/gpu/drm/bochs/bochs_mm.c| 2 -- > drivers/gpu/drm/cirrus/cirrus_ttm.c | 2 -- > drivers/gpu/drm/mgag200/mgag200_ttm.c | 2 -- > drivers/gpu/drm/nouveau/nouveau_bo.c| 2 -- > drivers/gpu/drm/qxl/qxl_ttm.c | 2 -- > drivers/gpu/drm/radeon/radeon_ttm.c | 2 -- > drivers/gpu/drm/ttm/ttm_bo.c| 19 +-- > drivers/gpu/drm/virtio/virtgpu_ttm.c| 2 -- > drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c | 2 -- > include/drm/ttm/ttm_bo_driver.h | 9 - > 12 files changed, 5 insertions(+), 43 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c > b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c > index dd086d8..0402a11 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c > @@ -1086,8 +1086,6 @@ static struct ttm_bo_driver amdgpu_bo_driver = { > .fault_reserve_notify = _bo_fault_reserve_notify, > .io_mem_reserve = _ttm_io_mem_reserve, > .io_mem_free = _ttm_io_mem_free, > - .lru_tail = _bo_default_lru_tail, > - .swap_lru_tail = _bo_default_swap_lru_tail, > }; > > int amdgpu_ttm_init(struct amdgpu_device *adev) > diff --git a/drivers/gpu/drm/ast/ast_ttm.c b/drivers/gpu/drm/ast/ast_ttm.c > index 2a1368f..50c910e 100644 > --- a/drivers/gpu/drm/ast/ast_ttm.c > +++ b/drivers/gpu/drm/ast/ast_ttm.c > @@ -236,8 +236,6 @@ struct ttm_bo_driver ast_bo_driver = { > .verify_access = ast_bo_verify_access, > .io_mem_reserve = _ttm_io_mem_reserve, > .io_mem_free = _ttm_io_mem_free, > - .lru_tail = _bo_default_lru_tail, > - .swap_lru_tail = _bo_default_swap_lru_tail, > }; > > int ast_mm_init(struct ast_private *ast) > diff --git a/drivers/gpu/drm/bochs/bochs_mm.c > b/drivers/gpu/drm/bochs/bochs_mm.c > index 099a3c6..e4c1125 100644 > --- a/drivers/gpu/drm/bochs/bochs_mm.c > +++ b/drivers/gpu/drm/bochs/bochs_mm.c > @@ -205,8 +205,6 @@ struct ttm_bo_driver bochs_bo_driver = { > .verify_access = bochs_bo_verify_access, > .io_mem_reserve = _ttm_io_mem_reserve, > .io_mem_free = _ttm_io_mem_free, > - .lru_tail = _bo_default_lru_tail, > - .swap_lru_tail = _bo_default_swap_lru_tail, > }; > > int bochs_mm_init(struct bochs_device *bochs) > diff --git a/drivers/gpu/drm/cirrus/cirrus_ttm.c > b/drivers/gpu/drm/cirrus/cirrus_ttm.c > index d6da848..f53aa8f 100644 > --- a/drivers/gpu/drm/cirrus/cirrus_ttm.c > +++ b/drivers/gpu/drm/cirrus/cirrus_ttm.c > @@ -236,8 +236,6 @@ struct ttm_bo_driver cirrus_bo_driver = { > .verify_access = cirrus_bo_verify_access, > .io_mem_reserve = _ttm_io_mem_reserve, > .io_mem_free = _ttm_io_mem_free, > - .lru_tail = _bo_default_lru_tail, > - .swap_lru_tail = _bo_default_swap_lru_tail, > }; > > int cirrus_mm_init(struct cirrus_device *cirrus) > diff --git a/drivers/gpu/drm/mgag200/mgag200_ttm.c > b/drivers/gpu/drm/mgag200/mgag200_ttm.c > index 5e20220..657598b 100644 > --- a/drivers/gpu/drm/mgag200/mgag200_ttm.c > +++ b/drivers/gpu/drm/mgag200/mgag200_ttm.c > @@ -236,8 +236,6 @@ struct ttm_bo_driver mgag200_bo_driver = { > .verify_access = mgag200_bo_verify_access, > .io_mem_reserve = _ttm_io_mem_reserve, > .io_mem_free = _ttm_io_mem_free, > - .lru_tail = _bo_default_lru_tail, > - .swap_lru_tail = _bo_default_swap_lru_tail, > }; > > int mgag200_mm_init(struct mga_device *mdev) > diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c > b/drivers/gpu/drm/nouveau/nouveau_bo.c > index 5b0a28b..3949a74 100644 > --- a/drivers/gpu/drm/nouveau/nouveau_bo.c > +++ b/drivers/gpu/drm/nouveau/nouveau_bo.c > @@ -1570,8 +1570,6 @@ struct ttm_bo_driver nouveau_bo_driver = { > .fault_reserve_notify = _ttm_fault_reserve_notify, > .io_mem_reserve = _ttm_io_mem_reserve, > .io_mem_free = _ttm_io_mem_free, > - .lru_tail = _bo_default_lru_tail, > - .swap_lru_tail = _bo_default_swap_lru_tail, > }; > > struct nvkm_vma * > diff --git a/drivers/gpu/drm/qxl/qxl_ttm.c b/drivers/gpu/drm/qxl/qxl_ttm.c > index f3939a9..2955f91 100644 > --- a/drivers/gpu/drm/qxl/qxl_ttm.c > +++ b/drivers/gpu/drm/qxl/qxl_ttm.c > @@ -395,8 +395,6 @@ static struct ttm_bo_driver qxl_bo_driver = { > .io_mem_reserve = _ttm_io_mem_reserve, > .io_mem_free = _ttm_io_mem_free, > .move_notify = _bo_move_notify, > - .lru_tail = _bo_default_lru_tail, > - .swap_lru_tail = _bo_default_swap_lru_tail, > }; > > int qxl_ttm_init(struct qxl_device *qdev) > diff --git a/drivers/gpu/drm/radeon/radeon_ttm.c >
RE: [PATCH 5/6] drm/ttm: revert "implement LRU add callbacks v2"
Series patches: Reviewed-by: Roger.He <hongbo...@amd.com> and Test-by Roger.He -Original Message- From: Christian König [mailto:deathsim...@vodafone.de] Sent: Friday, January 13, 2017 5:51 PM To: He, Hongbo <hongbo...@amd.com> Cc: amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: [PATCH 5/6] drm/ttm: revert "implement LRU add callbacks v2" From: Christian König <christian.koe...@amd.com> The additional housekeeping had to much CPU overhead, let's use the BO priorities instead. Signed-off-by: Christian König <christian.koe...@amd.com> --- drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 2 -- drivers/gpu/drm/ast/ast_ttm.c | 2 -- drivers/gpu/drm/bochs/bochs_mm.c| 2 -- drivers/gpu/drm/cirrus/cirrus_ttm.c | 2 -- drivers/gpu/drm/mgag200/mgag200_ttm.c | 2 -- drivers/gpu/drm/nouveau/nouveau_bo.c| 2 -- drivers/gpu/drm/qxl/qxl_ttm.c | 2 -- drivers/gpu/drm/radeon/radeon_ttm.c | 2 -- drivers/gpu/drm/ttm/ttm_bo.c| 19 +-- drivers/gpu/drm/virtio/virtgpu_ttm.c| 2 -- drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c | 2 -- include/drm/ttm/ttm_bo_driver.h | 9 - 12 files changed, 5 insertions(+), 43 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c index dd086d8..0402a11 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c @@ -1086,8 +1086,6 @@ static struct ttm_bo_driver amdgpu_bo_driver = { .fault_reserve_notify = _bo_fault_reserve_notify, .io_mem_reserve = _ttm_io_mem_reserve, .io_mem_free = _ttm_io_mem_free, - .lru_tail = _bo_default_lru_tail, - .swap_lru_tail = _bo_default_swap_lru_tail, }; int amdgpu_ttm_init(struct amdgpu_device *adev) diff --git a/drivers/gpu/drm/ast/ast_ttm.c b/drivers/gpu/drm/ast/ast_ttm.c index 2a1368f..50c910e 100644 --- a/drivers/gpu/drm/ast/ast_ttm.c +++ b/drivers/gpu/drm/ast/ast_ttm.c @@ -236,8 +236,6 @@ struct ttm_bo_driver ast_bo_driver = { .verify_access = ast_bo_verify_access, .io_mem_reserve = _ttm_io_mem_reserve, .io_mem_free = _ttm_io_mem_free, - .lru_tail = _bo_default_lru_tail, - .swap_lru_tail = _bo_default_swap_lru_tail, }; int ast_mm_init(struct ast_private *ast) diff --git a/drivers/gpu/drm/bochs/bochs_mm.c b/drivers/gpu/drm/bochs/bochs_mm.c index 099a3c6..e4c1125 100644 --- a/drivers/gpu/drm/bochs/bochs_mm.c +++ b/drivers/gpu/drm/bochs/bochs_mm.c @@ -205,8 +205,6 @@ struct ttm_bo_driver bochs_bo_driver = { .verify_access = bochs_bo_verify_access, .io_mem_reserve = _ttm_io_mem_reserve, .io_mem_free = _ttm_io_mem_free, - .lru_tail = _bo_default_lru_tail, - .swap_lru_tail = _bo_default_swap_lru_tail, }; int bochs_mm_init(struct bochs_device *bochs) diff --git a/drivers/gpu/drm/cirrus/cirrus_ttm.c b/drivers/gpu/drm/cirrus/cirrus_ttm.c index d6da848..f53aa8f 100644 --- a/drivers/gpu/drm/cirrus/cirrus_ttm.c +++ b/drivers/gpu/drm/cirrus/cirrus_ttm.c @@ -236,8 +236,6 @@ struct ttm_bo_driver cirrus_bo_driver = { .verify_access = cirrus_bo_verify_access, .io_mem_reserve = _ttm_io_mem_reserve, .io_mem_free = _ttm_io_mem_free, - .lru_tail = _bo_default_lru_tail, - .swap_lru_tail = _bo_default_swap_lru_tail, }; int cirrus_mm_init(struct cirrus_device *cirrus) diff --git a/drivers/gpu/drm/mgag200/mgag200_ttm.c b/drivers/gpu/drm/mgag200/mgag200_ttm.c index 5e20220..657598b 100644 --- a/drivers/gpu/drm/mgag200/mgag200_ttm.c +++ b/drivers/gpu/drm/mgag200/mgag200_ttm.c @@ -236,8 +236,6 @@ struct ttm_bo_driver mgag200_bo_driver = { .verify_access = mgag200_bo_verify_access, .io_mem_reserve = _ttm_io_mem_reserve, .io_mem_free = _ttm_io_mem_free, - .lru_tail = _bo_default_lru_tail, - .swap_lru_tail = _bo_default_swap_lru_tail, }; int mgag200_mm_init(struct mga_device *mdev) diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c b/drivers/gpu/drm/nouveau/nouveau_bo.c index 5b0a28b..3949a74 100644 --- a/drivers/gpu/drm/nouveau/nouveau_bo.c +++ b/drivers/gpu/drm/nouveau/nouveau_bo.c @@ -1570,8 +1570,6 @@ struct ttm_bo_driver nouveau_bo_driver = { .fault_reserve_notify = _ttm_fault_reserve_notify, .io_mem_reserve = _ttm_io_mem_reserve, .io_mem_free = _ttm_io_mem_free, - .lru_tail = _bo_default_lru_tail, - .swap_lru_tail = _bo_default_swap_lru_tail, }; struct nvkm_vma * diff --git a/drivers/gpu/drm/qxl/qxl_ttm.c b/drivers/gpu/drm/qxl/qxl_ttm.c index f3939a9..2955f91 100644 --- a/drivers/gpu/drm/qxl/qxl_ttm.c +++ b/drivers/gpu/drm/qxl/qxl_ttm.c @@ -395,8 +395,6 @@ static struct ttm_bo_driver qxl_bo_driver = { .io_mem_reserve = _ttm_io_mem_reserve, .io_mem_free = _ttm_io_mem_free, .move_notify = _bo_move_notify, - .lru_tail = _bo_default_lru_tail, - .swap_lr
[PATCH 5/6] drm/ttm: revert "implement LRU add callbacks v2"
From: Christian KönigThe additional housekeeping had to much CPU overhead, let's use the BO priorities instead. Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 2 -- drivers/gpu/drm/ast/ast_ttm.c | 2 -- drivers/gpu/drm/bochs/bochs_mm.c| 2 -- drivers/gpu/drm/cirrus/cirrus_ttm.c | 2 -- drivers/gpu/drm/mgag200/mgag200_ttm.c | 2 -- drivers/gpu/drm/nouveau/nouveau_bo.c| 2 -- drivers/gpu/drm/qxl/qxl_ttm.c | 2 -- drivers/gpu/drm/radeon/radeon_ttm.c | 2 -- drivers/gpu/drm/ttm/ttm_bo.c| 19 +-- drivers/gpu/drm/virtio/virtgpu_ttm.c| 2 -- drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c | 2 -- include/drm/ttm/ttm_bo_driver.h | 9 - 12 files changed, 5 insertions(+), 43 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c index dd086d8..0402a11 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c @@ -1086,8 +1086,6 @@ static struct ttm_bo_driver amdgpu_bo_driver = { .fault_reserve_notify = _bo_fault_reserve_notify, .io_mem_reserve = _ttm_io_mem_reserve, .io_mem_free = _ttm_io_mem_free, - .lru_tail = _bo_default_lru_tail, - .swap_lru_tail = _bo_default_swap_lru_tail, }; int amdgpu_ttm_init(struct amdgpu_device *adev) diff --git a/drivers/gpu/drm/ast/ast_ttm.c b/drivers/gpu/drm/ast/ast_ttm.c index 2a1368f..50c910e 100644 --- a/drivers/gpu/drm/ast/ast_ttm.c +++ b/drivers/gpu/drm/ast/ast_ttm.c @@ -236,8 +236,6 @@ struct ttm_bo_driver ast_bo_driver = { .verify_access = ast_bo_verify_access, .io_mem_reserve = _ttm_io_mem_reserve, .io_mem_free = _ttm_io_mem_free, - .lru_tail = _bo_default_lru_tail, - .swap_lru_tail = _bo_default_swap_lru_tail, }; int ast_mm_init(struct ast_private *ast) diff --git a/drivers/gpu/drm/bochs/bochs_mm.c b/drivers/gpu/drm/bochs/bochs_mm.c index 099a3c6..e4c1125 100644 --- a/drivers/gpu/drm/bochs/bochs_mm.c +++ b/drivers/gpu/drm/bochs/bochs_mm.c @@ -205,8 +205,6 @@ struct ttm_bo_driver bochs_bo_driver = { .verify_access = bochs_bo_verify_access, .io_mem_reserve = _ttm_io_mem_reserve, .io_mem_free = _ttm_io_mem_free, - .lru_tail = _bo_default_lru_tail, - .swap_lru_tail = _bo_default_swap_lru_tail, }; int bochs_mm_init(struct bochs_device *bochs) diff --git a/drivers/gpu/drm/cirrus/cirrus_ttm.c b/drivers/gpu/drm/cirrus/cirrus_ttm.c index d6da848..f53aa8f 100644 --- a/drivers/gpu/drm/cirrus/cirrus_ttm.c +++ b/drivers/gpu/drm/cirrus/cirrus_ttm.c @@ -236,8 +236,6 @@ struct ttm_bo_driver cirrus_bo_driver = { .verify_access = cirrus_bo_verify_access, .io_mem_reserve = _ttm_io_mem_reserve, .io_mem_free = _ttm_io_mem_free, - .lru_tail = _bo_default_lru_tail, - .swap_lru_tail = _bo_default_swap_lru_tail, }; int cirrus_mm_init(struct cirrus_device *cirrus) diff --git a/drivers/gpu/drm/mgag200/mgag200_ttm.c b/drivers/gpu/drm/mgag200/mgag200_ttm.c index 5e20220..657598b 100644 --- a/drivers/gpu/drm/mgag200/mgag200_ttm.c +++ b/drivers/gpu/drm/mgag200/mgag200_ttm.c @@ -236,8 +236,6 @@ struct ttm_bo_driver mgag200_bo_driver = { .verify_access = mgag200_bo_verify_access, .io_mem_reserve = _ttm_io_mem_reserve, .io_mem_free = _ttm_io_mem_free, - .lru_tail = _bo_default_lru_tail, - .swap_lru_tail = _bo_default_swap_lru_tail, }; int mgag200_mm_init(struct mga_device *mdev) diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c b/drivers/gpu/drm/nouveau/nouveau_bo.c index 5b0a28b..3949a74 100644 --- a/drivers/gpu/drm/nouveau/nouveau_bo.c +++ b/drivers/gpu/drm/nouveau/nouveau_bo.c @@ -1570,8 +1570,6 @@ struct ttm_bo_driver nouveau_bo_driver = { .fault_reserve_notify = _ttm_fault_reserve_notify, .io_mem_reserve = _ttm_io_mem_reserve, .io_mem_free = _ttm_io_mem_free, - .lru_tail = _bo_default_lru_tail, - .swap_lru_tail = _bo_default_swap_lru_tail, }; struct nvkm_vma * diff --git a/drivers/gpu/drm/qxl/qxl_ttm.c b/drivers/gpu/drm/qxl/qxl_ttm.c index f3939a9..2955f91 100644 --- a/drivers/gpu/drm/qxl/qxl_ttm.c +++ b/drivers/gpu/drm/qxl/qxl_ttm.c @@ -395,8 +395,6 @@ static struct ttm_bo_driver qxl_bo_driver = { .io_mem_reserve = _ttm_io_mem_reserve, .io_mem_free = _ttm_io_mem_free, .move_notify = _bo_move_notify, - .lru_tail = _bo_default_lru_tail, - .swap_lru_tail = _bo_default_swap_lru_tail, }; int qxl_ttm_init(struct qxl_device *qdev) diff --git a/drivers/gpu/drm/radeon/radeon_ttm.c b/drivers/gpu/drm/radeon/radeon_ttm.c index 0cf03cc..d610481 100644 --- a/drivers/gpu/drm/radeon/radeon_ttm.c +++ b/drivers/gpu/drm/radeon/radeon_ttm.c @@ -871,8 +871,6 @@ static struct ttm_bo_driver radeon_bo_driver = { .fault_reserve_notify = _bo_fault_reserve_notify,