Re: [PATCH v3 2/8] ttm: turn ttm_bo_device.vma_manager into a pointer

2019-08-08 Thread Gerd Hoffmann
Hi, > > > That would also make the verify_access callback completely superfluous > > > and looks like a good step into the right direction of de-midlayering. > > > > Hmm, right, noticed that too while working on another patch series. > > Guess I'll try to merge those two and see where I end up

Re: [PATCH v3 2/8] ttm: turn ttm_bo_device.vma_manager into a pointer

2019-08-08 Thread Koenig, Christian
Am 08.08.19 um 14:43 schrieb Thomas Hellström (VMware): > On 8/8/19 2:02 PM, Daniel Vetter wrote: >> On Thu, Aug 08, 2019 at 12:35:21PM +0200, Gerd Hoffmann wrote: >>> On Thu, Aug 08, 2019 at 09:48:49AM +, Koenig, Christian wrote: Am 08.08.19 um 11:36 schrieb Gerd Hoffmann: > Rename th

Re: [PATCH v3 2/8] ttm: turn ttm_bo_device.vma_manager into a pointer

2019-08-08 Thread VMware
On 8/8/19 2:02 PM, Daniel Vetter wrote: On Thu, Aug 08, 2019 at 12:35:21PM +0200, Gerd Hoffmann wrote: On Thu, Aug 08, 2019 at 09:48:49AM +, Koenig, Christian wrote: Am 08.08.19 um 11:36 schrieb Gerd Hoffmann: Rename the embedded struct vma_offset_manager, it is named _vma_manager now. tt

Re: [PATCH v3 2/8] ttm: turn ttm_bo_device.vma_manager into a pointer

2019-08-08 Thread Daniel Vetter
On Thu, Aug 08, 2019 at 12:35:21PM +0200, Gerd Hoffmann wrote: > On Thu, Aug 08, 2019 at 09:48:49AM +, Koenig, Christian wrote: > > Am 08.08.19 um 11:36 schrieb Gerd Hoffmann: > > > Rename the embedded struct vma_offset_manager, it is named _vma_manager > > > now. ttm_bo_device.vma_manager is

Re: [PATCH v3 2/8] ttm: turn ttm_bo_device.vma_manager into a pointer

2019-08-08 Thread Gerd Hoffmann
On Thu, Aug 08, 2019 at 09:48:49AM +, Koenig, Christian wrote: > Am 08.08.19 um 11:36 schrieb Gerd Hoffmann: > > Rename the embedded struct vma_offset_manager, it is named _vma_manager > > now. ttm_bo_device.vma_manager is a pointer now, pointing to the > > embedded ttm_bo_device._vma_manager

Re: [PATCH v3 2/8] ttm: turn ttm_bo_device.vma_manager into a pointer

2019-08-08 Thread Koenig, Christian
Am 08.08.19 um 11:36 schrieb Gerd Hoffmann: > Rename the embedded struct vma_offset_manager, it is named _vma_manager > now. ttm_bo_device.vma_manager is a pointer now, pointing to the > embedded ttm_bo_device._vma_manager by default. > > Add ttm_bo_device_init_with_vma_manager() function which al

[PATCH v3 2/8] ttm: turn ttm_bo_device.vma_manager into a pointer

2019-08-08 Thread Gerd Hoffmann
Rename the embedded struct vma_offset_manager, it is named _vma_manager now. ttm_bo_device.vma_manager is a pointer now, pointing to the embedded ttm_bo_device._vma_manager by default. Add ttm_bo_device_init_with_vma_manager() function which allows to initialize ttm with a different vma manager.