Public bug reported: [ Impact ]
* Optimisation of the drm/xe scheduler. [ Fix ] Cherrypick: cd8e995655 drm/xe: Take job list lock in xe_sched_first_pending_job [ Test Plan ] * Compiled and boot tested [ Where problems could occur ] * Performance regression with the drm/xe driver. [ Other Info ] * From https://patchwork.freedesktop.org/patch/msgid/20241105160327.2970277-1-nirmoy....@intel.com : Is this showing up in any bug reports? The only user of this function is guc_exec_queue_stop which as stopped the scheduler worker so I think accessing to the pending_list is in fact safe without this lock. This is however a micro-optimization in a non-hot path which I think is frowned upon. ** Affects: linux (Ubuntu) Importance: Undecided Status: New ** Affects: linux (Ubuntu Oracular) Importance: Wishlist Assignee: Thibault Ferrante (thibf) Status: New ** Also affects: linux (Ubuntu Oracular) Importance: Undecided Status: New ** Changed in: linux (Ubuntu Oracular) Importance: Undecided => Wishlist ** Changed in: linux (Ubuntu Oracular) Assignee: (unassigned) => Thibault Ferrante (thibf) -- You received this bug notification because you are a member of Kernel Packages, which is subscribed to linux in Ubuntu. https://bugs.launchpad.net/bugs/2106651 Title: drm/xe: Take job list lock in xe_sched_first_pending_job Status in linux package in Ubuntu: New Status in linux source package in Oracular: New Bug description: [ Impact ] * Optimisation of the drm/xe scheduler. [ Fix ] Cherrypick: cd8e995655 drm/xe: Take job list lock in xe_sched_first_pending_job [ Test Plan ] * Compiled and boot tested [ Where problems could occur ] * Performance regression with the drm/xe driver. [ Other Info ] * From https://patchwork.freedesktop.org/patch/msgid/20241105160327.2970277-1-nirmoy....@intel.com : Is this showing up in any bug reports? The only user of this function is guc_exec_queue_stop which as stopped the scheduler worker so I think accessing to the pending_list is in fact safe without this lock. This is however a micro-optimization in a non-hot path which I think is frowned upon. To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/2106651/+subscriptions -- Mailing list: https://launchpad.net/~kernel-packages Post to : kernel-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~kernel-packages More help : https://help.launchpad.net/ListHelp
