[PATCH 1/2] drm/radeon: stop re-init the TTM page pool

[Christian König]

Am 08.01.21 um 15:31 schrieb Daniel Vetter:
> On Thu, Jan 07, 2021 at 09:08:29PM +0100, Christian König wrote:
>> Am 07.01.21 um 19:07 schrieb Daniel Vetter:
>>> On Tue, Jan 05, 2021 at 07:23:08PM +0100, Christian König wrote:
>>>> Drivers are not supposed to init the page pool directly any more.
>>>>
>>>> Signed-off-by: Christian König <christian.koenig at amd.com>
>>> Please include reported-by credits and link to the bug reports on
>>> lore.kernel.org when merging this. Also I guess this should have a Fixes:
>>> line?
>> I'm not aware of a bug report, but the reported-by/Fixes lines are indeed
>> missing.
> This one here:
>
> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flore.kernel.org%2Fdri-devel%2F20201231104020.GA4504%40zn.tnic%2F&data=04%7C01%7Cchristian.koenig%40amd.com%7C3aede203348b4f32ea3108d8b3e224ec%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637457131179258488%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=bHVw4lj1f8g4ohVTeuKNkb0LAbLwY6N4KWortr3VtAo%3D&reserved=0
>
> Or did I get confused, and the above is yet another bug?

Yeah, but that was just reported by mail. The bug tracker I've saw was 
opened after the patch was already pushed.

>
>> BTW: Any idea why dim add-link doesn't work?
> Hm we occasionally have fun with email parsing (it's hard) and especially
> python changes in how encodings are handled differently between python2
> and python3. If you have a specific example I can try and take a look why
> it doesn't work.

It just looks up and doesn't seem to do anything. I'm not familiar with 
python so I can just describe the symptoms.

Christian.

> -Daniel
>
>>> And maybe some words on how/why stuff blows up.
>> Just a typo. I've forgot to remove two lines in radeon while rebasing and
>> still had the symbols exported so never noticed this.
>>
>> Christian.
>>
>>> -Daniel
>>>
>>>> ---
>>>>    drivers/gpu/drm/radeon/radeon_ttm.c | 3 ---
>>>>    1 file changed, 3 deletions(-)
>>>>
>>>> diff --git a/drivers/gpu/drm/radeon/radeon_ttm.c b/drivers/gpu/drm/radeon/radeon_ttm.c
>>>> index d4328ff57757..35b715f82ed8 100644
>>>> --- a/drivers/gpu/drm/radeon/radeon_ttm.c
>>>> +++ b/drivers/gpu/drm/radeon/radeon_ttm.c
>>>> @@ -729,9 +729,6 @@ int radeon_ttm_init(struct radeon_device *rdev)
>>>>    	}
>>>>    	rdev->mman.initialized = true;
>>>> -	ttm_pool_init(&rdev->mman.bdev.pool, rdev->dev, rdev->need_swiotlb,
>>>> -		      dma_addressing_limited(&rdev->pdev->dev));
>>>> -
>>>>    	r = radeon_ttm_init_vram(rdev);
>>>>    	if (r) {
>>>>    		DRM_ERROR("Failed initializing VRAM heap.\n");
>>>> -- 
>>>> 2.25.1
>>>>
>>>> _______________________________________________
>>>> dri-devel mailing list
>>>> dri-devel at lists.freedesktop.org
>>>> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Fdri-devel&data=04%7C01%7Cchristian.koenig%40amd.com%7C3aede203348b4f32ea3108d8b3e224ec%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637457131179258488%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=pHzuAsJcXf5HlAfJ7wYYc%2BsizZhSLBGWXBhCyVNRCfo%3D&reserved=0