On Wed, Jul 12, 2023 at 10:16:57AM +0000, André Pribil wrote:
> Hi,
>
> I think there is a flaw in the current preempt-rt 4.14 patch (probably since v4.14.302-rt142).
>
> The "0515-timers-Prepare-support-for-PREEMPT_RT.patch" uses at two location the
> following configure option:
> #ifdef CONFIG_PREEMPT_RT
>
> I guess that "CONFIG_PREEMPT_RT" is not defined for the 4.14-rt kernel and should be
> "CONFIG_PREEMPT_RT_FULL" instead, or?
Thank you for spotting this!
I will soon release a RC with the specific change you mentioned and the
backport of this commit from v4.19-rt that has been in my list for a while:
67821680a2b8 rt: PREEMPT_RT safety net for backported patches
Luis
> The problem that I was facing was that a tcdrain() call from userspace was spinning forever.
> I traced this down to an endless loop in del_timer_sync().
> After replacing "CONFIG_PREEMPT_RT" with "CONFIG_PREEMPT_RT_FULL", the problem
> seems to be fixed.
>
> Regards,
> Andre
>
---end quoted text---
