On Tue, 2023-12-12 at 18:53 -0500, Benjamin Marzinski wrote: > Make cli_del_maps() return -EBUSY like cli_del_map() if it fails > because > a device is in use and it doesn't run into any other type of > failures. > > Signed-off-by: Benjamin Marzinski <bmarzins@xxxxxxxxxx> It's kind of weird that we use both 1 and negative error codes in our cli handlers to indicate failure. Maybe we should get rid of the catch- all error code 1. Anyway, that's not the fault of your patch, so Reviewed-by: Martin Wilck <mwilck@xxxxxxxx>