Re: [2/2] drm/ttm: make TT creation purely optional v3

2020-08-06 Thread Christian König
Am 06.08.20 um 11:05 schrieb Michel Dänzer: On 2020-08-06 10:43 a.m., Christian König wrote: Am 06.08.20 um 10:01 schrieb Michel Dänzer: On 2020-08-06 9:13 a.m., Christian König wrote: Am 05.08.20 um 18:16 schrieb Michel Dänzer: On 2020-06-29 5:19 p.m., Christian König wrote: We only need th

Re: [2/2] drm/ttm: make TT creation purely optional v3

2020-08-06 Thread Michel Dänzer
On 2020-08-06 10:43 a.m., Christian König wrote: > Am 06.08.20 um 10:01 schrieb Michel Dänzer: >> On 2020-08-06 9:13 a.m., Christian König wrote: >>> Am 05.08.20 um 18:16 schrieb Michel Dänzer: On 2020-06-29 5:19 p.m., Christian König wrote: > We only need the page array when the BO is abo

Re: [2/2] drm/ttm: make TT creation purely optional v3

2020-08-06 Thread Christian König
Am 06.08.20 um 10:01 schrieb Michel Dänzer: On 2020-08-06 9:13 a.m., Christian König wrote: Am 05.08.20 um 18:16 schrieb Michel Dänzer: On 2020-06-29 5:19 p.m., Christian König wrote: We only need the page array when the BO is about to be accessed. So not only populate, but also create it on

Re: [2/2] drm/ttm: make TT creation purely optional v3

2020-08-06 Thread Michel Dänzer
On 2020-08-06 9:13 a.m., Christian König wrote: > Am 05.08.20 um 18:16 schrieb Michel Dänzer: >> On 2020-06-29 5:19 p.m., Christian König wrote: >>> We only need the page array when the BO is about to be accessed. >>> >>> So not only populate, but also create it on demand. >>> >>> v2: move NULL che

Re: [2/2] drm/ttm: make TT creation purely optional v3

2020-08-06 Thread Christian König
Am 05.08.20 um 18:16 schrieb Michel Dänzer: On 2020-06-29 5:19 p.m., Christian König wrote: We only need the page array when the BO is about to be accessed. So not only populate, but also create it on demand. v2: move NULL check into ttm_tt_create() v3: fix the occurrence in ttm_bo_kmap_ttm as

Re: [2/2] drm/ttm: make TT creation purely optional v3

2020-08-05 Thread Michel Dänzer
On 2020-06-29 5:19 p.m., Christian König wrote: > We only need the page array when the BO is about to be accessed. > > So not only populate, but also create it on demand. > > v2: move NULL check into ttm_tt_create() > v3: fix the occurrence in ttm_bo_kmap_ttm as well This broke amdgpu userptr fu

[PATCH 2/2] drm/ttm: make TT creation purely optional v3

2020-06-29 Thread Christian König
We only need the page array when the BO is about to be accessed. So not only populate, but also create it on demand. v2: move NULL check into ttm_tt_create() v3: fix the occurrence in ttm_bo_kmap_ttm as well Signed-off-by: Christian König Reviewed-by: Michael J. Ruhl --- drivers/gpu/drm/ttm/t