Re: [Qemu-devel] [PATCH 40/54] ram: Remove compression_switch and inline its logic

2017-04-18 Thread Juan Quintela
Peter Xu wrote: > On Thu, Apr 06, 2017 at 03:08:59PM +0200, Juan Quintela wrote: >> We can calculate its value, so not create a varible for it. >> >> Signed-off-by: Juan Quintela >> >> -- >> >> After Peter and Dave review, I dropped the variable and just inlined >> the condition. >> >> Later,

Re: [Qemu-devel] [PATCH 40/54] ram: Remove compression_switch and inline its logic

2017-04-10 Thread Peter Xu
On Thu, Apr 06, 2017 at 03:08:59PM +0200, Juan Quintela wrote: > We can calculate its value, so not create a varible for it. > > Signed-off-by: Juan Quintela > > -- > > After Peter and Dave review, I dropped the variable and just inlined > the condition. > > Later, Juan. > > Signed-off-by: Ju

Re: [Qemu-devel] [PATCH 40/54] ram: Remove compression_switch and inline its logic

2017-04-07 Thread Dr. David Alan Gilbert
* Juan Quintela (quint...@redhat.com) wrote: > We can calculate its value, so not create a varible for it. > > Signed-off-by: Juan Quintela Reviewed-by: Dr. David Alan Gilbert > -- > > After Peter and Dave review, I dropped the variable and just inlined > the condition. > > Later, Juan. > >

[Qemu-devel] [PATCH 40/54] ram: Remove compression_switch and inline its logic

2017-04-06 Thread Juan Quintela
We can calculate its value, so not create a varible for it. Signed-off-by: Juan Quintela -- After Peter and Dave review, I dropped the variable and just inlined the condition. Later, Juan. Signed-off-by: Juan Quintela --- migration/ram.c | 12 1 file changed, 8 insertions(+), 4