[dpdk-dev] [PATCH v2] vhost: translate addrs in set_vring_addr if ring already enabled

Yuanhan Liu yliu at fridaylinux.org
Tue Oct 17 15:13:54 CEST 2017


On Mon, Oct 16, 2017 at 04:56:27PM +0200, Maxime Coquelin wrote:
> Commit 3ea7052f4b1b ("vhost: postpone rings addresses translation")
> moves rings addresses translation at either vring kick or enable
> time, depending on whether protocol features are enabled or not.
> This is done not interpret ring information as long as the vring
> is not fully initialized.
> 
> The problem is that with old QEMU versions, like v2.5, the ring
> is enabled before addresses are sent, so addresses are never
> translated.
> 
> This patch fixes the issue by doing the translation in
> VHOST_USER_SET_VRING_ADDR handling if ring is already enabled.
> 
> Fixes: 3ea7052f4b1b ("vhost: postpone rings addresses translation")
> 
> Reported-by: Lei Yao <lei.a.yao at intel.com>
> Signed-off-by: Maxime Coquelin <maxime.coquelin at redhat.com>
> ---
> v2: Fixed commit informations (Fixes:, Reported-by:)

Applied to dpdk-next-virtio.

Thanks.

	--yliu


More information about the dev mailing list