On 7/28/23 19:20, Alex Deucher wrote: > On Fri, Jul 28, 2023 at 1:19 PM Michel Dänzer <michel@xxxxxxxxxxx> wrote: >> On 7/28/23 18:43, Alex Deucher wrote: >>> On Fri, Jul 28, 2023 at 10:25 AM Michel Dänzer <michel@xxxxxxxxxxx> wrote: >>>> On 7/28/23 11:30, Michel Dänzer wrote: >>>>> On 7/28/23 03:38, Zhang, Jesse(Jie) wrote: >>>>>> >>>>>> Could you try the patch again ? That work for me. >>>>>> >>>>>> https://patchwork.freedesktop.org/patch/549605/ <https://patchwork.freedesktop.org/patch/549605/> >>>>> >>>>> This patch fixes the symptoms described in https://gitlab.freedesktop.org/drm/amd/-/issues/2659 for me as well. >>>>> >>>>> However, it does not fix the IOMMU page faults[0] or the IB test failures on the compute rings. Should I try amdgpu.ignore_crat=1 for these symptoms as well? >>>> >>>> I tried ignore_crat=1, it avoids the remaining symptoms as well. >>> >>> The first 3 patches of this set may also fix it: >>> https://patchwork.freedesktop.org/series/121538/ >> >> They do fix all symptoms I'm seeing. >> >> Is this a feasible solution for the final 6.5 release? > > I think the first3 are. Even with Felix's feedback on patch 2? If so, will you merge these for 6.5 final? Or should we revert 84b4dd3f84de ("drm/amdkfd: Refactor migrate init to support partition switch") for that? -- Earthling Michel Dänzer | https://redhat.com Libre software enthusiast | Mesa and Xwayland developer
