Re: [PATCH] Avoid AdapterDisable bugcheck on a disabled Adapter

["Tu Dinh" <ngoc-tu.dinh@xxxxxxxxxx>]

On 08/08/2025 16:14, Owen Smith wrote:
> MiniportShutdownEx maybe called when the miniport is in any state, including
> the Paused state after MiniportPause. Both MiniportPause and 
> MiniportShutdownEx
> call AdapterDisable. The first call will unregister the suspend callback, and
> the second call will bugcheck, as the SuspendCallbackLate handle is now NULL.
> 
> Signed-off-by: Owen Smith <owen.smith@xxxxxxxxx>
> ---
>   src/xennet/adapter.c | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/src/xennet/adapter.c b/src/xennet/adapter.c
> index 67ced3e..840e7a8 100644
> --- a/src/xennet/adapter.c
> +++ b/src/xennet/adapter.c
> @@ -1965,6 +1965,8 @@ AdapterDisable(
>       )
>   {
>       ASSERT(Adapter->Enabled);
> +    if (!Adapter->Enabled)
> +        return;
>       Adapter->Enabled = FALSE;
>   
>       ReceiverDisable(Adapter->Receiver);

ASSERT(Adapter->Enabled); should be removed.


Ngoc Tu Dinh | Vates XCP-ng Developer

XCP-ng & Xen Orchestra - Vates solutions

web: https://vates.tech