RE: Fix broken Raven system before Linux 4.18 release

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Yes, that patch won't change any code logic, and just a cleanup.

Thanks,
Ray

-----Original Message-----
From: Koenig, Christian 
Sent: Monday, July 30, 2018 9:31 PM
To: Paul Menzel <pmenzel+amd-gfx@xxxxxxxxxxxxx>; Huang, Ray <Ray.Huang@xxxxxxx>; Alex Deucher <alexdeucher@xxxxxxxxx>
Cc: amd-gfx@xxxxxxxxxxxxxxxxxxxxx; dri-devel@xxxxxxxxxxxxxxxxxxxxx
Subject: Re: Fix broken Raven system before Linux 4.18 release

Hi Paul,

well that is rather odd, cause the patch you pointed out is just a code cleanup without functional change.

Can you double check your bisect result again?

Thanks,
Christian.

Am 30.07.2018 um 14:35 schrieb Paul Menzel:
> Dear Linux folks,
>
>
> On 07/04/18 12:43, Christian König wrote:
>> Am 04.07.2018 um 12:16 schrieb Huang Rui:
>>> BO ptr already be initialized at definition, we needn't use the 
>>> complicated reference.
>>>
>>> Signed-off-by: Huang Rui <ray.huang@xxxxxxx>
>> There is a typo in the subject line, with that fixed the patch is
>> Reviewed-by: Christian König <christian.koenig@xxxxxxx>.
>>> ---
>>>    drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 7 +++----
>>>    1 file changed, 3 insertions(+), 4 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c 
>>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
>>> index 712af5c..b53562b 100644
>>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
>>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
>>> @@ -1645,18 +1645,17 @@ int amdgpu_vm_bo_update(struct amdgpu_device 
>>> *adev,
>>>        uint64_t flags;
>>>        int r;
>>>    -    if (clear || !bo_va->base.bo) {
>>> +    if (clear || !bo) {
>>>            mem = NULL;
>>>            nodes = NULL;
>>>            exclusive = NULL;
>>>        } else {
>>>            struct ttm_dma_tt *ttm;
>>>    -        mem = &bo_va->base.bo->tbo.mem;
>>> +        mem = &bo->tbo.mem;
>>>            nodes = mem->mm_node;
>>>            if (mem->mem_type == TTM_PL_TT) {
>>> -            ttm = container_of(bo_va->base.bo->tbo.ttm,
>>> -                       struct ttm_dma_tt, ttm);
>>> +            ttm = container_of(bo->tbo.ttm, struct ttm_dma_tt, 
>>> +ttm);
>>>                pages_addr = ttm->dma_address;
>>>            }
>>>            exclusive = reservation_object_get_excl(bo->tbo.resv);
> I didn’t see the v2 message in my INBOX, but this commit 7c4e581a
> (drm/amdgpu: simplify the bo reference on amdgpu_bo_update) should be 
> applied to current Linux 4.18-rc7 series, best before the Linux 4.18 
> release next Sunday. Otherwise the screen is not updated when 
> switching from EFI frame buffer to amdgpufb, and GDM doesn’t start on the MSI B350M MORTAR (MS-7A37).
> The problem is present in 4.18-rc1, and I bisected the branch
> *amd-staging-drm-next* (with the meaning of good and bad reversed).
>
> ```
> $ git bisect log
> git bisect start
> # bad: [27f0996a5a6a5052a60a76541f434cc7b2220489] drm/amdgpu: clean up 
> the superfluous space and align the comment text for amdgpu_ttm git 
> bisect bad 27f0996a5a6a5052a60a76541f434cc7b2220489
> # good: [ce397d215ccd07b8ae3f71db689aedb85d56ab40] Linux 4.18-rc1 git 
> bisect good ce397d215ccd07b8ae3f71db689aedb85d56ab40
> # good: [96157892c82a6274b07ad884aeeb9d7550540878] drm/amd/display: 
> move dcn1 dispclk programming to dccg git bisect good 
> 96157892c82a6274b07ad884aeeb9d7550540878
> # good: [617b0eb160a87ef4a61d6770f360c5e27e01c5ee] drm/amd/pp: fix 
> semicolon.cocci warnings git bisect good 
> 617b0eb160a87ef4a61d6770f360c5e27e01c5ee
> # bad: [68cf3a51430c7e6dd2bf7642aac6db1042f7955d] amdgpu: Do not evict 
> VRAM git bisect bad 68cf3a51430c7e6dd2bf7642aac6db1042f7955d
> # good: [de5d27d291047e1791a237f8a1c90215bb4db9ce] drm/amd/display: 
> properly turn autocal off git bisect good 
> de5d27d291047e1791a237f8a1c90215bb4db9ce
> # good: [c20182735c2fb85df0e039192ae3b46d9b6a2c68] drm/amdgpu: remove 
> internal/unused kernel module parameters git bisect good 
> c20182735c2fb85df0e039192ae3b46d9b6a2c68
> # bad: [ca5401ce7eb8ad6f3696a1afeed6dfd4a20d994f] drm/amdgpu: use 
> drm_fb helper for console_(un)lock git bisect bad 
> ca5401ce7eb8ad6f3696a1afeed6dfd4a20d994f
> # bad: [829e05edce80cf43a982eb4c1ce547a921e5afd2] drm/amdgpu: clean up 
> UVD instance handling v2 git bisect bad 
> 829e05edce80cf43a982eb4c1ce547a921e5afd2
> # bad: [97b89da992d5efd4239904365162081a6e2394f2] drm/amd/amdgpu: 
> creating two I2S instances for stoney/cz (v2) git bisect bad 
> 97b89da992d5efd4239904365162081a6e2394f2
> # bad: [3f9da96fa9d2aaaca452d7396dbd84eaa9ac6766] drm/amdgpu/pm: 
> Remove VLA usage git bisect bad 
> 3f9da96fa9d2aaaca452d7396dbd84eaa9ac6766
> ```
>
>
> Kind regards,
>
> Paul

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux