On 29.04.22 21:01, Richard Weinberger via Xenomai wrote:
> On Fri, Apr 29, 2022 at 9:04 AM C Smith via Xenomai <[email protected]> 
> wrote:
>> int Lp_port_handler(rtdm_irq_t *irq_handle_p)
>> {
>>    static int err;
>>    unsigned long next;
>>    rtdm_irq_t *handle_p;
>>
>>    handle_p = rtdm_irq_get_arg(irq_handle_p, rtdm_irq_t);
>>
>>    next = rtdm_clock_read();
>>    // do some timing calculations with 'next' var here ...
>>    err = rtdm_irq_enable(handle_p);   //re-enable this for subsequent 
>> interrupts
> 
> You don't need this.
> Unconditionally enabling the interrupt line will confuse the IRQ subsystem.
> 

...and it was never supported in primary mode. If it worked, then by
chance and by disabling related checks. Enable/disable are no masking APIs.

Jan

>>   return 0;
> 
> Please use RTDM_IRQ_HANDLED here instead of raw values.
> 
>> }
> 
> Thanks,
> //richard
> 

-- 
Siemens AG, Technology
Competence Center Embedded Linux

Reply via email to