[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] vhost-user: Check vhost features for CONFIGURE_MEM_SLOTS
From: |
Dylan Reid |
Subject: |
Re: [PATCH] vhost-user: Check vhost features for CONFIGURE_MEM_SLOTS |
Date: |
Wed, 23 Dec 2020 20:13:11 +0000 |
>Agreed - VHOST_USER_PROTCOL_F_CONFIGURE_MEM_SLOTS is negotiated at the
>vhost-user, not virtio layer. The dev->protocol_features flags are taken
>from the VHOST_USER_GET_PROTOCOL_FEATURES message, which retrieves the
>supported vhost-user features from the backend. See libvhost-user for a
>simple reference implementation.
>
>The VHOST_USER_GET_FEATURES message retrieves the virtio protocol
>features, so checking features instead of dev->protocol_features would
>incorrectly check the supported virtio features instead of the vhost-user
>ones.
>
>Am I missing something here?
No, you're not missing anything, I am(so is the code I was working with).
I hade the message number and feature bit backwards.
Thanks for taking the time to read this, your explaination got me back on the
right track, I'll go fix the library I was using.