[Libguestfs] [PATCH v3 04/14] nbd: Prepare for 64-bit request effect lengths

Vladimir Sementsov-Ogievskiy vsementsov at yandex-team.ru
Tue May 30 13:05:14 UTC 2023


On 15.05.23 22:53, Eric Blake wrote:
> Widen the length field of NBDRequest to 64-bits, although we can
> assert that all current uses are still under 32 bits.  Move the
> request magic number to nbd.h, to live alongside the reply magic
> number.  Convert 'bool structured_reply' into a tri-state enum that
> will eventually track whether the client successfully negotiated
> extended headers with the server, allowing the nbd driver to pass
> larger requests along where possible; although in this patch the enum
> never surpasses structured replies, for no semantic change yet.
> 
> Signed-off-by: Eric Blake<eblake at redhat.com>

Seems too much for one patch, could it could be split into
- Convert 'bool structured_reply'
- introduce third parameter for nbd_send_request()
- rework len to 64bit

otherwise, looks good to me

-- 
Best regards,
Vladimir



More information about the Libguestfs mailing list