> From: Thomas Hellström <[email protected]> > > drmm_cgroup_register_region() is called before INIT_LIST_HEAD() and > gpu_buddy_init() in amdgpu_vram_mgr_init(). If it fails, the function > returns early and bypasses those initializations. > > Since adev->mman.initialized is set to true before amdgpu_vram_mgr_init() > is called, a failure triggers amdgpu_ttm_fini(), which calls > amdgpu_vram_mgr_fini(), which then: > > - Calls list_for_each_entry_safe() on reservations_pending and > reserved_pages, whose list_head::next pointers are zero-initialized > (NULL). The loop does not recognize them as empty and dereferences NULL. > > - Calls gpu_buddy_fini(), which iterates free_trees[] unconditionally > [ ... ] > > https://virtuozzo.atlassian.net/browse/VSTOR-153702 > Feature: fix ms/drm > > Signed-off-by: Mirian Shilakadze <[email protected]>
Sashiko has reviewed this patch and found no issues. It looks great! -- Sashiko AI review · https://sashiko.vzint.dev/#/patchset/[email protected]?part=1 _______________________________________________ Devel mailing list [email protected] https://lists.openvz.org/mailman/listinfo/devel
