Re: [PATCH] tg3: add new module param to force device power down on reboot

From: Michael Chan
Date: Wed Jan 10 2024 - 12:02:00 EST


On Tue, Jan 9, 2024 at 11:34 PM Heiner Kallweit <hkallweit1@xxxxxxxxx> wrote:
>
> On 10.01.2024 08:17, Michael Chan wrote:
> > We already call dev_close() which will call tg3_close() -> tg3_stop()
> > a few lines above.
>
> tg3_stop() calls tg3_disable_ints(), so I wonder how a MSI interrupt can
> occur after that. Does tg3_disable_ints() disable interrupts synchronously?
> Or maybe some kind of commit is needed?
>

Yes, it is synchronous. The tg3_full_lock() call before
tg3_disable_ints() makes it synchronous.

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature