On Fri, Jul 12, 2024 at 07:30:47PM +0800, Jun Wang wrote:
> The code to update link status is not safe in secondary process.
> If called from secondary it will crash, example from dumpcap:
> eth_em_link_update
>
> Signed-off-by: Jun Wang <[email protected]>
> ---
> drivers/net/e1000/em_ethdev.c | 3 +++
> 1 file changed, 3 insertions(+)
>
Given this is fixing an issue experienced in the real-world I think we
should take this patch. As Stephen says, a better solution would be to have
the whole function work properly in secondary, but I'd rather avoid crashes
as a priority.
Fixes: 805803445a02 ("e1000: support EM devices (also known as e1000/e1000e)")
Cc: [email protected]
Acked-by: Bruce Richardson <[email protected]>
