qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 2/2] e1000e: fix link state on resume


From: Laurent Vivier
Subject: Re: [PATCH v2 2/2] e1000e: fix link state on resume
Date: Tue, 5 Mar 2024 11:06:57 +0100
User-agent: Mozilla Thunderbird

On 2/1/24 06:45, Jason Wang wrote:
On Wed, Jan 24, 2024 at 6:40 PM Laurent Vivier <lvivier@redhat.com> wrote:

On resume e1000e_vm_state_change() always calls e1000e_autoneg_resume()
that sets link_down to false, and thus activates the link even
if we have disabled it.

The problem can be reproduced starting qemu in paused state (-S) and
then set the link to down. When we resume the machine the link appears
to be up.

Reproducer:

    # qemu-system-x86_64 ... -device e1000e,netdev=netdev0,id=net0 -S

    {"execute": "qmp_capabilities" }
    {"execute": "set_link", "arguments": {"name": "net0", "up": false}}
    {"execute": "cont" }

To fix the problem, merge the content of e1000e_vm_state_change()
into e1000e_core_post_load() as e1000 does.

Buglink: https://issues.redhat.com/browse/RHEL-21867
Fixes: 6f3fbe4ed06a ("net: Introduce e1000e device emulation")
Suggested-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
---


I've queued this.

Ping?

Thanks,
Laurent




reply via email to

[Prev in Thread] Current Thread [Next in Thread]