ANDROID: sched/fair: fix place_entity() vendor hook
place_entity() vendor hook is meant to tweak vruntime by vendor
modules as needed, but with current form of the hook that is not
possible as vruntime is passed by it's value. Fix it by switching
to pass by reference.
Bug: 175448877
Change-Id: Ibb51592f94da31019fa98a6767d080ec61daafe6
Signed-off-by: Satya Durga Srinivasu Prabhala <satyap@codeaurora.org>
diff --git a/include/trace/hooks/sched.h b/include/trace/hooks/sched.h
index 30ad44c..a383eec 100644
--- a/include/trace/hooks/sched.h
+++ b/include/trace/hooks/sched.h
@@ -174,7 +174,7 @@ DECLARE_RESTRICTED_HOOK(android_rvh_account_irq,
struct sched_entity;
DECLARE_RESTRICTED_HOOK(android_rvh_place_entity,
- TP_PROTO(struct cfs_rq *cfs_rq, struct sched_entity *se, int initial, u64 vruntime),
+ TP_PROTO(struct cfs_rq *cfs_rq, struct sched_entity *se, int initial, u64 *vruntime),
TP_ARGS(cfs_rq, se, initial, vruntime), 1);
DECLARE_RESTRICTED_HOOK(android_rvh_build_perf_domains,
diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
index ca5c372..17bf4caa 100644
--- a/kernel/sched/fair.c
+++ b/kernel/sched/fair.c
@@ -4194,7 +4194,7 @@ place_entity(struct cfs_rq *cfs_rq, struct sched_entity *se, int initial)
/* ensure we never gain time by being placed backwards. */
se->vruntime = max_vruntime(se->vruntime, vruntime);
- trace_android_rvh_place_entity(cfs_rq, se, initial, vruntime);
+ trace_android_rvh_place_entity(cfs_rq, se, initial, &vruntime);
}
static void check_enqueue_throttle(struct cfs_rq *cfs_rq);
