[PATCH 00/34] Add HMM-based SVM memory manager to KFD v4

Felix Kuehling felix.kuehling at amd.com
Thu Apr 8 15:10:56 UTC 2021


Am 2021-04-08 um 11:02 a.m. schrieb Jason Gunthorpe:
> On Mon, Apr 05, 2021 at 09:45:55PM -0400, Felix Kuehling wrote:
>> Rebased on upstream. Dropped already upstream patch
>> "drm/amdgpu: reserve fence slot to update page table".
>>
>> Added more fixes:
>> - Fixed kernel test robot warnings about static functions
>> - Fixed a kernel test robot warning about an unused variable
>> - Fixed a kernel test robot warning about select DEVICE_PRIVATE.
>>   Using "depends on" now. (see patch 34)
>> - Proportionally longer timeout for hmm_range_fault on large address ranges
>>   (see patch 6)
>> - Fixed PTE flags for XGMI mappings on Arcturus and Aldebaran (see patch 17)
>> - Fixed range-list cleanup on process termination to avoid BUGs from dangling
>>   interval notifiers (see patch 16)
>> - Fixed SVM range locking and interval notifier sequence update
>>   (see patch 8 and related tweaks in patches 10, 11, 21)
>>
>> Added my Reviewed-by to all patches primarily authored by Philip and Alex.
>> I believe this patch series is nearly ready to go.
>>
>> This series and the corresponding ROCm Thunk and KFDTest changes are also
>> available on gitub and patchwork.
>>
>> Link: https://github.com/RadeonOpenCompute/ROCK-Kernel-Driver/tree/fxkamd/hmm-wip
>> Link: https://github.com/RadeonOpenCompute/ROCT-Thunk-Interface/tree/fxkamd/hmm-wip
>> Link: https://patchwork.freedesktop.org/series/85563/
>> CC: Jérôme Glisse <jglisse at redhat.com>
>> CC: Jason Gunthorpe <jgg at ziepe.ca>
> This series is huge, but it looks like it still doesn't fix the
> FIXME's around the AMD driver use of hmm.
>
> Can you fix them before piling on more stuff?

It does avoid making the same mistakes in the new code. I'll take
another look at the pre-existing FIXMEs with the experience gained
working on this series.

Regards,
  Felix


>
> Jaason


More information about the amd-gfx mailing list