Messages in this thread | ![/](/images/icornerl.gif) | | Date | Wed, 10 Apr 2024 13:46:09 +0200 | From | Peter Zijlstra <> | Subject | Re: [RFC PATCH 0/2] Disable RT-throttling for idle-inject threads |
| |
On Wed, Apr 10, 2024 at 04:59:33PM +0530, Atul Kumar Pant wrote:
> Hi Peter, > We are trying to add support for true 100% idle-injection ratio from > idle-injection framework.
Yeah, I got that. I'm saying that is broken. Both from a requirement POV and an implementation POV.
If your hardware needs 100% idle injection that means CPU availability is unreliable and everything that relies on CPU-masks will be broken.
Furthermore, since idle-injection is build on top of FIFO, anything with a higher priority (DL, stop) will simply preempt it anyway.
And, as already mentioned, hogging the system with FIFO will break things.
So I would *very* strongly urge you to push back on whoever thought this is 'needed'. This is *bad* hardware.
> It might happen that we want to run idle cycles for > slightly more time than permitted by RT-bandwidth control.
The thing is configurable for a reason.
> We understand the > concern about it hogging the cpu. Will it be better if we make it a choice for > the user who uses idle-inject framework, whether to have true 100% > idle-injection support or not?
None of the above mentioned issues will magically go away. Run a 100% FIFO task for an indeterminate amount of time and you get to keep the pieces.
Also, we'll be replacing the throttling code with DL servers 'soonish' at which point all this will stop working anyway, since DL will preempt anything FIFO, including your idle injection crud.
| ![\](/images/icornerr.gif) |