So then I guess we will need to init rv to 0.

So if the interrupt doesn’t go through the true branches, we certainly know 
that it’s not for us.

> On 25 Mar 2025, at 1:53 PM, Manuel Bouyer <bou...@antioche.eu.org> wrote:
> 
> On Tue, Mar 25, 2025 at 01:31:46PM +0000, Emmanuel Nyarko wrote:
>> 
>> https://github.com/Emmankoko/altq_refactoring_gsoc/commit/918c9c8056ccc2ed7da214bbf3c8c3cd4071fa8c?
>> returning garbage value · Emmankoko/altq_refactoring_gsoc@918c9c8
>> github.com
>> 
>> I don?t really understand the need of the return value in this case. But 
>> would appreciate some global reviews here.
>> 
>> Is a return 0 or 1 relevant to the logic here. It looks like some interrupt 
>> handling.
> 
> it is interrupt handling; return 0 if the iterrupt was not for us, 1 if it 
> was.
> 
>> 
>> Is it supposed to always return 1?
> 
> No, if we know the interrupt was not for us return 0.
> 
> -- 
> Manuel Bouyer <bou...@antioche.eu.org>
>     NetBSD: 26 ans d'experience feront toujours la difference
> --


Emmanuel





Reply via email to