linux-kernel - [PATCH 0/2] Fix expected set_vq_state behavior on vdpa

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-Id: <20230118164359.1523760-1-eperezma@redhat.com>
Date:   Wed, 18 Jan 2023 17:43:57 +0100
From:   Eugenio Pérez <eperezma@...hat.com>
To:     mst@...hat.com
Cc:     leiyang@...hat.com, Laurent Vivier <lvivier@...hat.com>,
        sgarzare@...hat.com, jasowang@...hat.com,
        Zhu Lingshan <lingshan.zhu@...el.com>,
        virtualization@...ts.linux-foundation.org, si-wei.liu@...cle.com,
        linux-kernel@...r.kernel.org, lulu@...hat.com,
        Gautam Dawar <gdawar@...inx.com>, alvaro.karsz@...id-run.com
Subject: [PATCH 0/2] Fix expected set_vq_state behavior on vdpa_sim

The use of set_vq_state is to indicate vdpa device the state of a virtqueue.
In the case of split, it means the avail_idx.  This is mandatory for use
cases like live migration.

However, vdpa_sim reset the vq state at vdpasim_queue_ready since it calls
vringh_init_iotlb.

Also, to starting from an used_idx different than 0 is needed in use cases like
virtual machine migration.  Not doing so and letting the caller set an avail
idx different than 0 causes destination device to try to use old buffers that
source driver already recover and are not available anymore.

This series fixes both problems allowing to migrate to a vdpa_sim_net device.

Eugenio Pérez (2):
  vdpa_sim: not reset state in vdpasim_queue_ready
  vringh: fetch used_idx from vring at vringh_init_iotlb

 drivers/vdpa/vdpa_sim/vdpa_sim.c |  2 ++
 drivers/vhost/vringh.c           | 25 +++++++++++++++++++++++--
 2 files changed, 25 insertions(+), 2 deletions(-)

-- 
2.31.1