Re: [PATCH-for-5.2 v2 16/25] block/nvme: Simplify nvme_cmd_sync()

2020-10-30 Thread Stefan Hajnoczi
On Thu, Oct 29, 2020 at 10:32:57AM +0100, Philippe Mathieu-Daudé wrote: > As all commands use the ADMIN queue, it is pointless to pass > it as argument each time. Remove the argument, and rename the > function as nvme_admin_cmd_sync() to make this new behavior > clearer. > > Reviewed-by: Eric

[PATCH-for-5.2 v2 16/25] block/nvme: Simplify nvme_cmd_sync()

2020-10-29 Thread Philippe Mathieu-Daudé
As all commands use the ADMIN queue, it is pointless to pass it as argument each time. Remove the argument, and rename the function as nvme_admin_cmd_sync() to make this new behavior clearer. Reviewed-by: Eric Auger Tested-by: Eric Auger Signed-off-by: Philippe Mathieu-Daudé --- block/nvme.c