On Tue, Aug 29, 2017 at 04:48:05PM +0200, Marc-André Lureau wrote:
> Commit e10e798c85c2331 switched to libvhost-user which lacked support
> for resuming the avail_idx based on used_idx.
>
> Fixes:
> https://bugzilla.redhat.com/show_bug.cgi?id=1485867
>
> Signed-off-by: Marc-André Lureau <[email protected]>
Can we add a test for reconnect to catch the regression?
> ---
> tests/vhost-user-bridge.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/tests/vhost-user-bridge.c b/tests/vhost-user-bridge.c
> index 1e5b5ca3da..dca48512ba 100644
> --- a/tests/vhost-user-bridge.c
> +++ b/tests/vhost-user-bridge.c
> @@ -466,11 +466,18 @@ vubr_panic(VuDev *dev, const char *msg)
> vubr->quit = 1;
> }
>
> +static bool
> +vubr_queue_resume_to_used(VuDev *dev, int qidx)
> +{
> + return true;
> +}
> +
> static const VuDevIface vuiface = {
> .get_features = vubr_get_features,
> .set_features = vubr_set_features,
> .process_msg = vubr_process_msg,
> .queue_set_started = vubr_queue_set_started,
> + .queue_resume_to_used = vubr_queue_resume_to_used,
> };
>
> static void
> --
> 2.14.1.146.gd35faa819
