Am 09.01.25 um 14:27 schrieb Tvrtko Ursulin:
[SNIP]
@@ -259,7 +258,7 @@ struct drm_sched_rq {
spinlock_t lock;
/* Following members are protected by the @lock: */
- struct drm_sched_entity *current_entity;
+ ktime_t rr_deadline;
struct list_head entities;
If I'm not completely mistaken you can now also nuke this entities
list if you haven't already removed all users.
I had a version which did that too. But then I dropped it in favour of
only keeping entities with queued jobs in the tree. (Before both the
list and the tree contained entities which submitted at least one job
in the past.)
I kind of like keeping the tree trimmed (would it lower the rb tree
re-balancing costs?) in which case the full list is needed for that
karma processing business.
Well, is anybody still using this karma stuff (maybe amdgpu, but we
could drop that)? That as well turned out to be a quite broken approach.
Basically the idea behind karma was that on a crash you re-submit the
same job over and over again until it either doesn't crash any more or
your karma became to bad.
And when you now think of what Einstein once said about insanity then
yeah that was also my first thought when I learned about that :)
Cheers,
Christian.
Regards,
Tvrtko
Regards,
Christian.
struct rb_root_cached rb_tree_root;
};
