On Tue, Sep 15, 2026 at 06:44:01AM -0400, Michael S. Tsirkin wrote:
vhost-user.rst says the legacy SET_MEM_TABLE contains 8 regions, but this does not make sense - we have a num_regions field for this. And it is not true in practice: if QEMU has less, it sends less.The incorrect wording was introduced, without explanation, as part of an editorial cleanup. Probably an oversight. It's really "up to 8". Fixes: 1958bb23b9 ("docs: fix vhost-user protocol doc")
I can't find this SHA, I guess the right one is b87ea798eb The change LGTM, so with that fixed: Reviewed-by: Stefano Garzarella <[email protected]>
Cc: luzhixing12345 <[email protected]> Signed-off-by: Michael S. Tsirkin <[email protected]> --- docs/interop/vhost-user.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/interop/vhost-user.rst b/docs/interop/vhost-user.rst index c1e02ff7e9..488214a306 100644 --- a/docs/interop/vhost-user.rst +++ b/docs/interop/vhost-user.rst @@ -248,7 +248,7 @@ Multiple Memory regions description :padding: 32-bit -:regions: regions field contains 8 regions of type :ref:`Memory region description <memory_region_description>`. +:regions: regions field contains up to 8 regions of type :ref:`Memory region description <memory_region_description>`. Log description ^^^^^^^^^^^^^^^ -- MST
