Re: [Libguestfs] [libnbd PATCH 0/3] Simplify nbd_shutdown vs. opt mode

On Tue, Aug 29, 2023 at 05:20:40PM -0500, Eric Blake wrote: > While working on a larger set of patches to make nbdinfo favor > NBD_OPT_INFO over NBD_OPT_GO where possible (which requires use of > nbd_set_opt_mode(,true) in more cases), I noticed that it got unwieldy > to have to pick the correct shutdown function in all code paths. So I > propose making the API smarter, by adding an opt-in flag that does the > right thing on my behalf. > > If you have an idea for a better name for the flag, or think this > functionality should be enabled by default, let me know. Part of the > reason for choosing a new flag is that it becomes a compile-time > witness of whether nbd_shutdown has the desired capability (if we > allow it to auto-opt_abort without a flag, it's harder to tell whether > we are running against an older libnbd where it errors out instead). My feeling is this should be enabled by default, as that does the right thing by default.

Whether or not we need to have a flag to disable it (ie the opposite sense to the proposed flag) is up to you.

For the series: Reviewed-by: Richard W.M. Jones <rjones(a)redhat.com&gt;