Merge "Fix AIBinder_setMinSchedulerPolicy failed to set sched policy" into main

Hongguang Chen · Gerrit Code Review · commit f44d068f5c24 · 2024-01-31T07:41:42.000Z
diff --git a/services/surfaceflinger/DisplayHardware/AidlComposerHal.cpp b/services/surfaceflinger/DisplayHardware/AidlComposerHal.cpp
@@ -325,7 +325,9 @@ void AidlComposer::registerCallback(HWC2::ComposerCallback& callback) {
     }
 
     mAidlComposerCallback = ndk::SharedRefBase::make<AidlIComposerCallbackWrapper>(callback);
-    AIBinder_setMinSchedulerPolicy(mAidlComposerCallback->asBinder().get(), SCHED_FIFO, 2);
+
+    ndk::SpAIBinder binder = mAidlComposerCallback->asBinder();
+    AIBinder_setMinSchedulerPolicy(binder.get(), SCHED_FIFO, 2);
 
     const auto status = mAidlComposerClient->registerCallback(mAidlComposerCallback);
     if (!status.isOk()) {

Original file line number	Diff line number	Diff line change
`@@ -325,7 +325,9 @@ void AidlComposer::registerCallback(HWC2::ComposerCallback& callback) {`
`325`	`325`	`}`
`326`	`326`
`327`	`327`	`mAidlComposerCallback = ndk::SharedRefBase::make<AidlIComposerCallbackWrapper>(callback);`
`328`		`- AIBinder_setMinSchedulerPolicy(mAidlComposerCallback->asBinder().get(), SCHED_FIFO, 2);`
	`328`	`+`
	`329`	`+ ndk::SpAIBinder binder = mAidlComposerCallback->asBinder();`
	`330`	`+ AIBinder_setMinSchedulerPolicy(binder.get(), SCHED_FIFO, 2);`
`329`	`331`
`330`	`332`	`const auto status = mAidlComposerClient->registerCallback(mAidlComposerCallback);`
`331`	`333`	`if (!status.isOk()) {`