On Mon, Sep 04, 2023 at 07:53:10PM +0300, Vladimir Sementsov-Ogievskiy wrote: > On 29.08.23 20:58, Eric Blake wrote: > > Upcoming additions to support NBD 64-bit effect lengths will add a new > > command flag NBD_CMD_FLAG_PAYLOAD_LEN that needs to be considered in > > our sanity checks of the client's messages (that is, more than just > > CMD_WRITE have the potential to carry a client payload when extended > > headers are in effect). But before we can start to support that, it > > is easier to first refactor the existing set of various if statements > > over open-coded combinations of request->type to instead be a single > > switch statement over all command types that sets witnesses, then > > straight-line processing based on the witnesses. No semantic change > > is intended. > > > > Signed-off-by: Eric Blake<ebl...@redhat.com> > > Reviewed-by: Vladimir Sementsov-Ogievskiy <vsement...@yandex-team.ru>
Patches 1-5 queued through my latest NBD pull request; will respin patches 6 and following on top of latest master and post a v7 for one more round of reviews. -- Eric Blake, Principal Software Engineer Red Hat, Inc. Virtualization: qemu.org | libguestfs.org
