mbox series

[0/3] migration/postcopy: unsentmap is not necessary

Message ID 20190819061843.28642-1-richardw.yang@linux.intel.com (mailing list archive)
Headers show
Series migration/postcopy: unsentmap is not necessary | expand

Message

Wei Yang Aug. 19, 2019, 6:18 a.m. UTC
Three patches to cleanup postcopy:

[1]: split canonicalize bitmap and discard page
[2]: remove unsentmap since it is not necessary
[3]: cleanup the get_queued_page_not_dirty

Wei Yang (3):
  migration/postcopy: not necessary to do discard when canonicalizing
    bitmap
  migration/postcopy: unsentmap is not necessary for postcopy
  migration: remove sent parameter in get_queued_page_not_dirty

 include/exec/ram_addr.h |  6 ---
 migration/ram.c         | 94 +++++++----------------------------------
 migration/trace-events  |  2 +-
 3 files changed, 16 insertions(+), 86 deletions(-)

Comments

Wei Yang Sept. 16, 2019, 6:06 a.m. UTC | #1
Hi, Dave and Juan

Would you mind taking a look at this one?

On Mon, Aug 19, 2019 at 02:18:40PM +0800, Wei Yang wrote:
>Three patches to cleanup postcopy:
>
>[1]: split canonicalize bitmap and discard page
>[2]: remove unsentmap since it is not necessary
>[3]: cleanup the get_queued_page_not_dirty
>
>Wei Yang (3):
>  migration/postcopy: not necessary to do discard when canonicalizing
>    bitmap
>  migration/postcopy: unsentmap is not necessary for postcopy
>  migration: remove sent parameter in get_queued_page_not_dirty
>
> include/exec/ram_addr.h |  6 ---
> migration/ram.c         | 94 +++++++----------------------------------
> migration/trace-events  |  2 +-
> 3 files changed, 16 insertions(+), 86 deletions(-)
>
>-- 
>2.17.1
Dr. David Alan Gilbert Sept. 20, 2019, 6:53 p.m. UTC | #2
* Wei Yang (richardw.yang@linux.intel.com) wrote:
> Hi, Dave and Juan
> 
> Would you mind taking a look at this one?

Yes, I'll have a look, I'm just fighting through a bunch of other stuff.

Dave

> On Mon, Aug 19, 2019 at 02:18:40PM +0800, Wei Yang wrote:
> >Three patches to cleanup postcopy:
> >
> >[1]: split canonicalize bitmap and discard page
> >[2]: remove unsentmap since it is not necessary
> >[3]: cleanup the get_queued_page_not_dirty
> >
> >Wei Yang (3):
> >  migration/postcopy: not necessary to do discard when canonicalizing
> >    bitmap
> >  migration/postcopy: unsentmap is not necessary for postcopy
> >  migration: remove sent parameter in get_queued_page_not_dirty
> >
> > include/exec/ram_addr.h |  6 ---
> > migration/ram.c         | 94 +++++++----------------------------------
> > migration/trace-events  |  2 +-
> > 3 files changed, 16 insertions(+), 86 deletions(-)
> >
> >-- 
> >2.17.1
> 
> -- 
> Wei Yang
> Help you, Help me
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
Wei Yang Sept. 23, 2019, 5:56 a.m. UTC | #3
On Fri, Sep 20, 2019 at 07:53:51PM +0100, Dr. David Alan Gilbert wrote:
>* Wei Yang (richardw.yang@linux.intel.com) wrote:
>> Hi, Dave and Juan
>> 
>> Would you mind taking a look at this one?
>
>Yes, I'll have a look, I'm just fighting through a bunch of other stuff.
>

Yep, thanks ~ Me too :-)

Have a good day

>Dave
>
>> On Mon, Aug 19, 2019 at 02:18:40PM +0800, Wei Yang wrote:
>> >Three patches to cleanup postcopy:
>> >
>> >[1]: split canonicalize bitmap and discard page
>> >[2]: remove unsentmap since it is not necessary
>> >[3]: cleanup the get_queued_page_not_dirty
>> >
>> >Wei Yang (3):
>> >  migration/postcopy: not necessary to do discard when canonicalizing
>> >    bitmap
>> >  migration/postcopy: unsentmap is not necessary for postcopy
>> >  migration: remove sent parameter in get_queued_page_not_dirty
>> >
>> > include/exec/ram_addr.h |  6 ---
>> > migration/ram.c         | 94 +++++++----------------------------------
>> > migration/trace-events  |  2 +-
>> > 3 files changed, 16 insertions(+), 86 deletions(-)
>> >
>> >-- 
>> >2.17.1
>> 
>> -- 
>> Wei Yang
>> Help you, Help me
>--
>Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
Dr. David Alan Gilbert Sept. 25, 2019, 9:52 a.m. UTC | #4
* Wei Yang (richardw.yang@linux.intel.com) wrote:
> Three patches to cleanup postcopy:
> 
> [1]: split canonicalize bitmap and discard page
> [2]: remove unsentmap since it is not necessary
> [3]: cleanup the get_queued_page_not_dirty

Queued

> 
> Wei Yang (3):
>   migration/postcopy: not necessary to do discard when canonicalizing
>     bitmap
>   migration/postcopy: unsentmap is not necessary for postcopy
>   migration: remove sent parameter in get_queued_page_not_dirty
> 
>  include/exec/ram_addr.h |  6 ---
>  migration/ram.c         | 94 +++++++----------------------------------
>  migration/trace-events  |  2 +-
>  3 files changed, 16 insertions(+), 86 deletions(-)
> 
> -- 
> 2.17.1
> 
> 
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK