On Tue, Oct 17, 2023 at 12:10 PM Akihiko Odaki <akihiko.od...@daynix.com> wrote:
>
> The member is not cleared during reset so may have a stale value.
Nit: I guess it should be "always unset populate_hash"?
Thanks
>
> Signed-off-by: Akihiko Odaki <akihiko.od...@daynix.com>
> ---
> hw/net/virtio-net.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
> index 20feb20bb1..6eb206f297 100644
> --- a/hw/net/virtio-net.c
> +++ b/hw/net/virtio-net.c
> @@ -643,6 +643,7 @@ static void virtio_net_set_mrg_rx_bufs(VirtIONet *n, int
> mergeable_rx_bufs,
> n->guest_hdr_len = n->mergeable_rx_bufs ?
> sizeof(struct virtio_net_hdr_mrg_rxbuf) :
> sizeof(struct virtio_net_hdr);
> + n->rss_data.populate_hash = false;
> }
>
> for (i = 0; i < n->max_queue_pairs; i++) {
> --
> 2.42.0
>
