Re: [PATCH] migration: Don't return for postcopy_send_discard_bm_ram()

2022-01-26 Thread Juan Quintela
Philippe Mathieu-Daudé wrote: > postcopy_send_discard_bm_ram() always return zero. Since it can't > fail, simplify and do not return anything. > > Signed-off-by: Philippe Mathieu-Daudé > --- > Based-on: <20211224065000.97572-1-pet...@redhat.com> And here, I learn to read Based-on: Reviewed-by:

Re: [PATCH] migration: Don't return for postcopy_send_discard_bm_ram()

2022-01-04 Thread Peter Xu
On Tue, Jan 04, 2022 at 10:15:16AM +0100, Juan Quintela wrote: > Philippe Mathieu-Daudé wrote: > > postcopy_send_discard_bm_ram() always return zero. Since it can't > > fail, simplify and do not return anything. > > > > Signed-off-by: Philippe Mathieu-Daudé > > --- > > Based-on: <20211224065000.9

Re: [PATCH] migration: Don't return for postcopy_send_discard_bm_ram()

2022-01-04 Thread Juan Quintela
Philippe Mathieu-Daudé wrote: > postcopy_send_discard_bm_ram() always return zero. Since it can't > fail, simplify and do not return anything. > > Signed-off-by: Philippe Mathieu-Daudé > --- > Based-on: <20211224065000.97572-1-pet...@redhat.com> > --- > migration/ram.c | 6 +- > 1 file chang

Re: [PATCH] migration: Don't return for postcopy_send_discard_bm_ram()

2022-01-04 Thread David Edmondson
On Thursday, 2021-12-30 at 17:05:25 +01, Philippe Mathieu-Daudé wrote: > postcopy_send_discard_bm_ram() always return zero. Since it can't > fail, simplify and do not return anything. > > Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: David Edmondson > --- > Based-on: <20211224065000.97572

[PATCH] migration: Don't return for postcopy_send_discard_bm_ram()

2021-12-30 Thread Philippe Mathieu-Daudé
postcopy_send_discard_bm_ram() always return zero. Since it can't fail, simplify and do not return anything. Signed-off-by: Philippe Mathieu-Daudé --- Based-on: <20211224065000.97572-1-pet...@redhat.com> --- migration/ram.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a