drm/xe: Backport fix for crash in xe_migrate_fini during driver unload #244
Conversation
CI Bug Log - changes from xkb-6.14-198-0db7ec117573b2944f4ca001e591a52931c92271_BAT -> xkb-pr-6.14-244-426-37ac3b586116bb5284b36717b7d797df7c28f840_BATSummarySUCCESS No regressions found. Participating hosts (5 -> 5)No changes in participating hosts Known issuesHere are the changes found in xkb-pr-6.14-244-426-37ac3b586116bb5284b36717b7d797df7c28f840_BAT that come from known issues: IGT changesIssues hit
Warnings
Build changes
igt-xkb-92-165f8a8a8: 165f8a8a8 |
| backport/patches/features/sriov/0001-drm-xe-Protect-against-unset-LRC-when-pausing-submis.patch | ||
| backport/patches/features/sriov/0001-drm-xe-vf-Start-re-emission-from-first-unsignaled-jo.patch | ||
| backport/patches/features/sriov/0001-drm-xe-vf-Stop-waiting-for-ring-space-on-VF-post-mig.patch | ||
| backport/patches/features/sriov/0001-drm-xe-migrate-Switch-from-drm-to-dev-managed-action.patch |
There was a problem hiding this comment.
why this patch was added to the 'sriov' section?
There was a problem hiding this comment.
All dependent patches are located in the sriov folder. These dependencies were
originally backported as part of the SR-IOV work, hence their placement in the
sriov directory. Given these dependencies, this patch is now included as part
of the SR-IOV patch series.
There was a problem hiding this comment.
@pravalikagurram1 / @mwajdecz the changes are added as dependencies as part of sriov feature backport and they were only required for SRIOV at that time. I agree on the term that it should be part of base. But 6.14 currently is reaching EoL by EoM. So we will proceed further with the current approach. However, we will movements in 6.17 branch if and as needed.
There was a problem hiding this comment.
Base Section:
Used for patches that need to be included in both HWE (Hardware Enablement) and -intel (PPA) kernels. These are foundational patches required across different kernel distributions.
Feature Sections:
Used to enable specific features that are not part of the base kernel. Each feature section is self-contained and includes all necessary dependencies within the same section.
Important Note:
Feature sections (e.g., feature/xyz) may contain additional dependencies as needed to support the feature implementation. This modular approach ensures features can be enabled independently with their complete dependency chain.
Backport to fix use-after-free crash during driver unload Backport upstream changes to fix a use-after-free crash in xe_migrate_fini during driver unload. The issue occurs when xe_migrate_fini is called during drm_managed_release and attempts to access GGTT resources that have already been freed.i Fix: Switch from drmm_add_action_or_reset to devm_add_action_or_reset to ensure xe_migrate_fini runs before underlying device resources are torn down. Also remove unused drm_device parameter. This backport is needed to prevent system crashes during driver unload/device removal scenarios. Signed-off-by: Pravalika Gurram <pravalika.gurram@intel.com>
4 participants
[ Backport to fix use-after-free crash during driver unload ]
Backport upstream changes to fix a use-after-free crash in
xe_migrate_fini during driver unload. The issue occurs when
xe_migrate_fini is called during drm_managed_release and attempts
to access GGTT resources that have already been freed.i
Fix: Switch from drmm_add_action_or_reset to devm_add_action_or_reset
to ensure xe_migrate_fini runs before underlying device resources are
torn down. Also remove unused drm_device parameter.
This backport is needed to prevent system crashes during driver
unload/device removal scenarios.
Signed-off-by: Pravalika Gurram pravalika.gurram@intel.com