Re: [DISCUSS] tg3 reboot handling on Dell T440 (BCM5720)

["Noam D. Eliyahu" <noam.d.eliyahu@gmail.com>]

Hi Pavan,

Thanks for circling back on this!

On Fri Oct 23, 2026 at 6:40 AM IDT, Pavan Chebbi wrote:
> We haven't heard back from Lenny but I think this solution should work.

While waiting to hear back, I worked around this locally across my environments with a custom kernel. I also shared the solution in a few Reddit threads and received positive feedback from others hitting the exact same behavior. After getting things working locally, pushing it upstream slipped off my radar.

> Hello Noam, bumping up this old discussion... do you want to submit a
> patch for this?

I would be happy to submit the patch.
Locally, I had taken the quick path and stripped out the code in question, but for the upstream patch I will implement the proper approach we discussed: keeping the DMI table to guard the conditional tg3_power_down (restricted only to models where the original issue was observed, e.g., R650xs) and dropping the AER handling entirely.

I'll adapt my patch to this logic and test it across different firmware versions over the weekend to make sure everything behaves as expected due to the date gap between then and now.

As this is my first kernel submission, could you point me to any recommended guides or things to know before submitting the patch?

I know about:
- https://docs.kernel.org/process/submitting-patches.html
- https://docs.kernel.org/process/maintainer-netdev.html

Thanks again,
Noam