Re: [PATCH net v5 1/4] net: ethernet: oa_tc6: Interrupt is active low, level triggered.

[Parthiban.Veerasooran]

Hi Selvamani,

On 25/06/26 9:01 pm, Selvamani Rajagopal wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> Parthiban,
> 
> Let me know if you prefer updating the patchset. I certainly prefer adding a NULL check
> In oa_tc6_update_rx_skb function.
Sorry for the delayed response. I see you already shared the patches for 
the fixes. Today I will test the below patch series and share the 
feedback ASAP.

https://patchwork.kernel.org/project/netdevbpf/patch/20260626-fix-race-condition-and-crash-v2-1-b6c5c10e604f@xxxxxxxxxx/

Best regards,
Parthiban V
> 
> 
>>
>> Root cause seems to be same. When oa_tc6_update_rx_skb function is called, tc6-
>>> rx_skb
>> seems to be NULL, which may mean, controller seems to be not getting start
>>
>> I have a theory. Look at line #933. We have the following comment. I am sure this could
>> be true
>> for the call to oa_tc6_prcs_rx_frame_end at line #926 or oa_tc6_prcs_ongoing_rx_frame
>> at line #950.
>>                 /* After rx buffer overflow error received, there might be a
>>                   * possibility of getting an end valid of a previously
>>                   * incomplete rx frame along with the new rx frame start valid.
>>                   */
>>
>