Re: Kernel and ADM hardware roulette ( was AMD graphics performance regression in 4.15 and later )

[Christian KÃnig]

Am 06.06.2018 um 16:12 schrieb Michel DÃnzer:
> On 2018-06-06 03:33 PM, Gabriel C wrote:
> > 2018-06-06 14:19 GMT+02:00 Christian KÃnig <christian.koenig@xxxxxxx>:
> > > Am 06.06.2018 um 14:08 schrieb Gabriel C:
> > > > 2018-06-06 13:33 GMT+02:00 Christian KÃnig <christian.koenig@xxxxxxx>:
> > > > > Am 06.06.2018 um 13:28 schrieb Gabriel C:
> > > > > > 2018-04-11 7:02 GMT+02:00 Gabriel C <nix.or.die@xxxxxxxxx>:
> > > > > > > [ÂÂÂ 6.337838] [drm] PCIE GART of 2048M enabled (table at
> > > > > > > 0x00000000001D6000).
> > > > > > > [ÂÂÂ 6.338210] radeon 0000:21:00.0: (-12) create WB bo failed
> > > > > > > [ÂÂÂ 6.338214] radeon 0000:21:00.0: disabling GPU acceleration
> > > > > > >
> > > > > > > ...
> > > > > > I have the same Issue now on final 4.17.
>
> Please file a bug report, and ideally bisect which commit(s) 
> introduced the issue(s).
>
>
> > > > http://ftp.frugalware.org/pub/other/people/crazy/radeon/dmesg-iommu-sr-iov-off.txt 
> > > >
> > > >
> > > > http://ftp.frugalware.org/pub/other/people/crazy/radeon/dmesg-iommu-sr-iov-on.txt 
> > > >
> > > >
> > > > Also nothing else changed in that setup just testing kernel 4.17.
> > >
> > >
> > > That has nothing TODO with the driver nor the original bug you 
> > > reported. The
> > > problem is that SME is active and that is currently not supported at 
> > > all
> > > with a that hardware.
> >
> > Ok .. so are we playing now kernel an AMD Hardware roulette on each 
> > release ?
> >
> > SME was like this in kernel 4.16.x here and all worked.
>
> If that is true, again please bisect which commit broke it.
>
> All the reports I've seen before this indicated that at least amdgpu 
> has never worked with SME (which BTW doesn't mean it's never going to 
> work or that we don't want to support it, just that as far as we know 
> it's currently not working).

At least in theory it should work when we use the coherent DMA allocator.

When that really worked before, so the most likely commit which broke 
this is:

commit fd5fd480dd8fe4910546e7b080b3ae345e57fe9f
Author: Chunming Zhou <david1.zhou@xxxxxxx>
Date:ÂÂ Fri Feb 9 10:44:09 2018 +0800

ÂÂÂ drm/amdgpu: only enable swiotlb alloc when need v2

ÂÂÂ get the max io mapping address of system memory to see if it is over
ÂÂÂ our card accessing range.
ÂÂÂ v2: move checking later

ÂÂÂ Signed-off-by: Chunming Zhou <david1.zhou@xxxxxxx>
ÂÂÂ Reviewed-by: Monk Liu <monk.liu@xxxxxxx>
ÂÂÂ Reviewed-by: Christian KÃnig <christian.koenig@xxxxxxx>
ÂÂÂ Signed-off-by: Alex Deucher <alexander.deucher@xxxxxxx>

Currently looking into how we could somehow improve this detection.

Regards,
Christian.