Hi! > From: Rouven Czerwinski <[email protected]> > > commit dcb3b06d9c34f33a249f65c08805461fb0c4325b upstream. > > might_sleep() is a debugging aid and triggers rescheduling only for > certain kernel configurations. Replace with an explicit check and > reschedule to work for all kernel configurations. Fixes the following > trace:
The need_resched() part is unneccessary/wrong AFAICT. And it seems
there's already a follow up patch that fixes this one.
(I'm not sure it causes any problem).
Best regards,
Pavel
> @@ -148,7 +149,8 @@ u32 optee_do_call_with_arg(struct tee_co
> */
> optee_cq_wait_for_completion(&optee->call_queue, &w);
> } else if (OPTEE_SMC_RETURN_IS_RPC(res.a0)) {
> - might_sleep();
> + if (need_resched())
> + cond_resched();
> param.a0 = res.a0;
> param.a1 = res.a1;
> param.a2 = res.a2;
>
--
http://www.livejournal.com/~pavelmachek
signature.asc
Description: PGP signature
