Re: [PATCH 05/13] block: Inline bdrv_drain_invoke()

2022-11-14 Thread Hanna Reitz
On 08.11.22 13:37, Kevin Wolf wrote: bdrv_drain_invoke() has now two entirely separate cases that share no code any more and are selected depending on a bool parameter. Each case has only one caller. Just inline the function. Signed-off-by: Kevin Wolf --- block/io.c | 23 ++---

Re: [PATCH 05/13] block: Inline bdrv_drain_invoke()

2022-11-11 Thread Emanuele Giuseppe Esposito
Am 08/11/2022 um 13:37 schrieb Kevin Wolf: > bdrv_drain_invoke() has now two entirely separate cases that share no > code any more and are selected depending on a bool parameter. Each case > has only one caller. Just inline the function. > > Signed-off-by: Kevin Wolf > Reviewed-by: Emanuele G

Re: [PATCH 05/13] block: Inline bdrv_drain_invoke()

2022-11-10 Thread Stefan Hajnoczi
On Tue, Nov 08, 2022 at 01:37:30PM +0100, Kevin Wolf wrote: > bdrv_drain_invoke() has now two entirely separate cases that share no > code any more and are selected depending on a bool parameter. Each case > has only one caller. Just inline the function. > > Signed-off-by: Kevin Wolf > --- > blo

Re: [PATCH 05/13] block: Inline bdrv_drain_invoke()

2022-11-09 Thread Vladimir Sementsov-Ogievskiy
On 11/8/22 15:37, Kevin Wolf wrote: bdrv_drain_invoke() has now two entirely separate cases that share no code any more and are selected depending on a bool parameter. Each case has only one caller. Just inline the function. Signed-off-by: Kevin Wolf Reviewed-by: Vladimir Sementsov-Ogievskiy

[PATCH 05/13] block: Inline bdrv_drain_invoke()

2022-11-08 Thread Kevin Wolf
bdrv_drain_invoke() has now two entirely separate cases that share no code any more and are selected depending on a bool parameter. Each case has only one caller. Just inline the function. Signed-off-by: Kevin Wolf --- block/io.c | 23 ++- 1 file changed, 6 insertions(+), 17