On Fri, Mar 15, 2024 at 1:27 PM Andrey M. Borodin <x4...@yandex-team.ru> wrote:
>
> > On 15 Mar 2024, at 15:44, Alexander Korotkov <aekorot...@gmail.com> wrote:
> >
> >  We loop in the interrupt checking, given that the injection point handler 
> > checks for interrupts internally.  I propose to unset the timeout flag 
> > before the injection point (see the attached patch).
>
> Oh, cool.
> As far as I understand, this is only necessary for the test with injection 
> point.
> So, maybe unset it only when injection points are enabled? Something like 
> this is already used in GIN.
>
> #ifdef USE_INJECTION_POINTS
> ....
> #endif


Maybe, but do you see any negative side effects of the unconditionally
unset of flags?  If not, I would prefer to keep the code simple.

------
Regards,
Alexander Korotkov


Reply via email to