mbox series

[v12,0/6] support dirtyrate at the granualrity of vcpu

Message ID cover.1624982116.git.huangy81@chinatelecom.cn (mailing list archive)
Headers show
Series support dirtyrate at the granualrity of vcpu | expand

Message

Hyman Huang June 29, 2021, 4:01 p.m. UTC
From: Hyman Huang(黄勇) <huangy81@chinatelecom.cn>

v12
- adjust the order of calculating dirty rate
  let memory_global_dirty_log_sync before calculating as
  v11 version description.

v11
- rebase on master

- call memory_global_dirty_log_sync before stopping dirty log
  when we calculate dirtyrate. this can make dirtyrate result
  more accurate, since kvm-reaper may be sleeping when there are
  dirty pages but not very much, this may lead to miss some pages 
  when we are calculating. do the dirty log sync manually before
  we do the calculation, so that we can get the dirty page 
  produced as recently as possible.

v10
- rebase on master

- pre-check if dirty log has started when calling
  memory_global_dirty_log_stop in the ram_save_cleanup.
  since it will stop dirty log unconditionally, so add if statement
  to ensure that memory_global_dirty_log_start/stop used in pairs.

- modify the memory_global_dirty_log_start/log in xen vitualization
  and make the qemu works in the old way.

v9
- rebase on master

- make global_dirty_tracking a bitmask:
  pass the casted "flags" to the callback in the right way, and drop
  the malloc/free step.

- make bitmask comments more accurate

please review, thanks.

v8
- drop the "none" type of DirtyRateMeasureMode

- make global_dirty_tracking a bitmask:
  1. code clean: pass the casted "flags" as input parameter in
     memory_global_dirty_log_stop, and then drop the free step.
  2. squash commit "rename global_dirty_log to global_dirty_tracking"
     into commit "make global_dirty_tracking a bitmask"
  3. move "GLOBAL_DIRTY_MASK" macro to the bits's definations.

- drop the "cleanup_dirtyrate_stat" in commit
  "move init step of calculation to main thread" so that each commit
  keeps the old way working.

- add dirty rate unit "MB/s" in the output of hmp

please review, may be this is the last version of this patchset, :)
thanks for Peter's patient instructions and reviews.

Hyman Huang(黄勇)

v7
- fix the code style problem, sorry about that

v6:
- pick up commit "KVM: introduce dirty_pages and kvm_dirty_ring_enabled"
  which has been dropped in verison 5

v5:
- rename global_dirty_log to global_dirty_tracking on Peter's advice

- make global_dirty_tracking a bitmask:
  1. add assert statement to ensure starting dirty tracking repeatly
     not allowed.
  2. add assert statement to ensure dirty tracking cannot be stopped
     without having been started.

- protecting dirty rate stat info:
  1. drop the mutext for protecting dirty rate introduced in version 4
  2. change the code block in query_dirty_rate_info so that requirements
     of "safe racing" to the dirty rate stat can be meet

- make the helper function "record_dirtypages" inline and change
  the global var dirty_pages  to local var

- free DirtyRateVcpuList in case of memory leak

please review, thanks a lot.

v4:
- make global_dirty_log a bitmask:
  1. add comments about dirty log bitmask
  2. use assert statement to check validity of flags
  3. add trace to log bitmask changes

- introduce mode option to show what method calculation should be used,
  also, export mode option in the as last commmit

- split cleanup and init of dirty rate stat and move it in the main
  thread

- change the fields of DirtyPageRecord to uint64_t type so that we
  can calculation the increased dirty pages with the formula
  as Peter's advice: dirty pages = end_pages - start_pages

- introduce mutex to protect dirty rate stat info

- adjust order of registering thread

- drop the memory free callback

this version modify some code on Peter's advice, reference to:
https://lore.kernel.org/qemu-devel/YL5nNYXmrqMlXF3v@t490s/

thanks again.

v3:
- pick up "migration/dirtyrate: make sample page count configurable" to
  make patchset apply master correctly

v2:
- rebase to "migration/dirtyrate: make sample page count configurable"

- rename "vcpu" to "per_vcpu" to show the per-vcpu method

- squash patch 5/6 into a single one, squash patch 1/2 also

- pick up "hmp: Add "calc_dirty_rate" and "info dirty_rate" cmds"

- make global_dirty_log a bitmask to make sure both migration and dirty
  could not intefer with each other

- add memory free callback to prevent memory leaking

the most different of v2 fron v1 is that we make the global_dirty_log a
bitmask. the reason is dirty rate measurement may start or stop dirty
logging during calculation. this conflict with migration because stop
dirty log make migration leave dirty pages out then that'll be a
problem.

make global_dirty_log a bitmask can let both migration and dirty
rate measurement work fine. introduce GLOBAL_DIRTY_MIGRATION and
GLOBAL_DIRTY_DIRTY_RATE to distinguish what current dirty log aims
for, migration or dirty rate.

all references to global_dirty_log should be untouched because any bit
set there should justify that global dirty logging is enabled.

Please review, thanks !

v1:

Since the Dirty Ring on QEMU part has been merged recently, how to use
this feature is under consideration.

In the scene of migration, it is valuable to provide a more accurante
interface to track dirty memory than existing one, so that the upper
layer application can make a wise decision, or whatever. More
importantly,
dirtyrate info at the granualrity of vcpu could provide a possibility to
make migration convergent by imposing restriction on vcpu. With Dirty
Ring, we can calculate dirtyrate efficiently and cheaply.

The old interface implemented by sampling pages, it consumes cpu
resource, and the larger guest memory size become, the more cpu resource
it consumes, namely, hard to scale. New interface has no such drawback.

Please review, thanks !

Best Regards !

Hyman Huang(黄勇) (6):
  KVM: introduce dirty_pages and kvm_dirty_ring_enabled
  memory: make global_dirty_tracking a bitmask
  migration/dirtyrate: introduce struct and adjust DirtyRateStat
  migration/dirtyrate: adjust order of registering thread
  migration/dirtyrate: move init step of calculation to main thread
  migration/dirtyrate: implement dirty-ring dirtyrate calculation

 accel/kvm/kvm-all.c     |   7 ++
 hmp-commands.hx         |   7 +-
 hw/i386/xen/xen-hvm.c   |   4 +-
 include/exec/memory.h   |  20 +++-
 include/exec/ram_addr.h |   4 +-
 include/hw/core/cpu.h   |   1 +
 include/sysemu/kvm.h    |   1 +
 migration/dirtyrate.c   | 266 +++++++++++++++++++++++++++++++++++++++++-------
 migration/dirtyrate.h   |  19 +++-
 migration/ram.c         |  15 ++-
 migration/trace-events  |   2 +
 qapi/migration.json     |  46 ++++++++-
 softmmu/memory.c        |  32 ++++--
 softmmu/trace-events    |   1 +
 14 files changed, 361 insertions(+), 64 deletions(-)

Comments

Hyman Huang Aug. 29, 2021, 11:11 p.m. UTC | #1
ping - hi, what would you think about this patchset ? :)

在 2021/6/30 0:01, huangy81@chinatelecom.cn 写道:
> From: Hyman Huang(黄勇) <huangy81@chinatelecom.cn>
> 
> v12
> - adjust the order of calculating dirty rate
>    let memory_global_dirty_log_sync before calculating as
>    v11 version description.
> 
> v11
> - rebase on master
> 
> - call memory_global_dirty_log_sync before stopping dirty log
>    when we calculate dirtyrate. this can make dirtyrate result
>    more accurate, since kvm-reaper may be sleeping when there are
>    dirty pages but not very much, this may lead to miss some pages
>    when we are calculating. do the dirty log sync manually before
>    we do the calculation, so that we can get the dirty page
>    produced as recently as possible.
> 
> v10
> - rebase on master
> 
> - pre-check if dirty log has started when calling
>    memory_global_dirty_log_stop in the ram_save_cleanup.
>    since it will stop dirty log unconditionally, so add if statement
>    to ensure that memory_global_dirty_log_start/stop used in pairs.
> 
> - modify the memory_global_dirty_log_start/log in xen vitualization
>    and make the qemu works in the old way.
> 
> v9
> - rebase on master
> 
> - make global_dirty_tracking a bitmask:
>    pass the casted "flags" to the callback in the right way, and drop
>    the malloc/free step.
> 
> - make bitmask comments more accurate
> 
> please review, thanks.
> 
> v8
> - drop the "none" type of DirtyRateMeasureMode
> 
> - make global_dirty_tracking a bitmask:
>    1. code clean: pass the casted "flags" as input parameter in
>       memory_global_dirty_log_stop, and then drop the free step.
>    2. squash commit "rename global_dirty_log to global_dirty_tracking"
>       into commit "make global_dirty_tracking a bitmask"
>    3. move "GLOBAL_DIRTY_MASK" macro to the bits's definations.
> 
> - drop the "cleanup_dirtyrate_stat" in commit
>    "move init step of calculation to main thread" so that each commit
>    keeps the old way working.
> 
> - add dirty rate unit "MB/s" in the output of hmp
> 
> please review, may be this is the last version of this patchset, :)
> thanks for Peter's patient instructions and reviews.
> 
> Hyman Huang(黄勇)
> 
> v7
> - fix the code style problem, sorry about that
> 
> v6:
> - pick up commit "KVM: introduce dirty_pages and kvm_dirty_ring_enabled"
>    which has been dropped in verison 5
> 
> v5:
> - rename global_dirty_log to global_dirty_tracking on Peter's advice
> 
> - make global_dirty_tracking a bitmask:
>    1. add assert statement to ensure starting dirty tracking repeatly
>       not allowed.
>    2. add assert statement to ensure dirty tracking cannot be stopped
>       without having been started.
> 
> - protecting dirty rate stat info:
>    1. drop the mutext for protecting dirty rate introduced in version 4
>    2. change the code block in query_dirty_rate_info so that requirements
>       of "safe racing" to the dirty rate stat can be meet
> 
> - make the helper function "record_dirtypages" inline and change
>    the global var dirty_pages  to local var
> 
> - free DirtyRateVcpuList in case of memory leak
> 
> please review, thanks a lot.
> 
> v4:
> - make global_dirty_log a bitmask:
>    1. add comments about dirty log bitmask
>    2. use assert statement to check validity of flags
>    3. add trace to log bitmask changes
> 
> - introduce mode option to show what method calculation should be used,
>    also, export mode option in the as last commmit
> 
> - split cleanup and init of dirty rate stat and move it in the main
>    thread
> 
> - change the fields of DirtyPageRecord to uint64_t type so that we
>    can calculation the increased dirty pages with the formula
>    as Peter's advice: dirty pages = end_pages - start_pages
> 
> - introduce mutex to protect dirty rate stat info
> 
> - adjust order of registering thread
> 
> - drop the memory free callback
> 
> this version modify some code on Peter's advice, reference to:
> https://lore.kernel.org/qemu-devel/YL5nNYXmrqMlXF3v@t490s/
> 
> thanks again.
> 
> v3:
> - pick up "migration/dirtyrate: make sample page count configurable" to
>    make patchset apply master correctly
> 
> v2:
> - rebase to "migration/dirtyrate: make sample page count configurable"
> 
> - rename "vcpu" to "per_vcpu" to show the per-vcpu method
> 
> - squash patch 5/6 into a single one, squash patch 1/2 also
> 
> - pick up "hmp: Add "calc_dirty_rate" and "info dirty_rate" cmds"
> 
> - make global_dirty_log a bitmask to make sure both migration and dirty
>    could not intefer with each other
> 
> - add memory free callback to prevent memory leaking
> 
> the most different of v2 fron v1 is that we make the global_dirty_log a
> bitmask. the reason is dirty rate measurement may start or stop dirty
> logging during calculation. this conflict with migration because stop
> dirty log make migration leave dirty pages out then that'll be a
> problem.
> 
> make global_dirty_log a bitmask can let both migration and dirty
> rate measurement work fine. introduce GLOBAL_DIRTY_MIGRATION and
> GLOBAL_DIRTY_DIRTY_RATE to distinguish what current dirty log aims
> for, migration or dirty rate.
> 
> all references to global_dirty_log should be untouched because any bit
> set there should justify that global dirty logging is enabled.
> 
> Please review, thanks !
> 
> v1:
> 
> Since the Dirty Ring on QEMU part has been merged recently, how to use
> this feature is under consideration.
> 
> In the scene of migration, it is valuable to provide a more accurante
> interface to track dirty memory than existing one, so that the upper
> layer application can make a wise decision, or whatever. More
> importantly,
> dirtyrate info at the granualrity of vcpu could provide a possibility to
> make migration convergent by imposing restriction on vcpu. With Dirty
> Ring, we can calculate dirtyrate efficiently and cheaply.
> 
> The old interface implemented by sampling pages, it consumes cpu
> resource, and the larger guest memory size become, the more cpu resource
> it consumes, namely, hard to scale. New interface has no such drawback.
> 
> Please review, thanks !
> 
> Best Regards !
> 
> Hyman Huang(黄勇) (6):
>    KVM: introduce dirty_pages and kvm_dirty_ring_enabled
>    memory: make global_dirty_tracking a bitmask
>    migration/dirtyrate: introduce struct and adjust DirtyRateStat
>    migration/dirtyrate: adjust order of registering thread
>    migration/dirtyrate: move init step of calculation to main thread
>    migration/dirtyrate: implement dirty-ring dirtyrate calculation
> 
>   accel/kvm/kvm-all.c     |   7 ++
>   hmp-commands.hx         |   7 +-
>   hw/i386/xen/xen-hvm.c   |   4 +-
>   include/exec/memory.h   |  20 +++-
>   include/exec/ram_addr.h |   4 +-
>   include/hw/core/cpu.h   |   1 +
>   include/sysemu/kvm.h    |   1 +
>   migration/dirtyrate.c   | 266 +++++++++++++++++++++++++++++++++++++++++-------
>   migration/dirtyrate.h   |  19 +++-
>   migration/ram.c         |  15 ++-
>   migration/trace-events  |   2 +
>   qapi/migration.json     |  46 ++++++++-
>   softmmu/memory.c        |  32 ++++--
>   softmmu/trace-events    |   1 +
>   14 files changed, 361 insertions(+), 64 deletions(-)
>
Hyman Huang Sept. 20, 2021, 3:45 p.m. UTC | #2
在 2021/8/30 7:11, Hyman 写道:
> ping - hi, what would you think about this patchset ? :)
ping again, just make sure this patchset not be ignored,
any advice or comment are welcome. :)
> 
> 在 2021/6/30 0:01, huangy81@chinatelecom.cn 写道:
>> From: Hyman Huang(黄勇) <huangy81@chinatelecom.cn>
>>
>> v12
>> - adjust the order of calculating dirty rate
>>    let memory_global_dirty_log_sync before calculating as
>>    v11 version description.
>>
>> v11
>> - rebase on master
>>
>> - call memory_global_dirty_log_sync before stopping dirty log
>>    when we calculate dirtyrate. this can make dirtyrate result
>>    more accurate, since kvm-reaper may be sleeping when there are
>>    dirty pages but not very much, this may lead to miss some pages
>>    when we are calculating. do the dirty log sync manually before
>>    we do the calculation, so that we can get the dirty page
>>    produced as recently as possible.
>>
>> v10
>> - rebase on master
>>
>> - pre-check if dirty log has started when calling
>>    memory_global_dirty_log_stop in the ram_save_cleanup.
>>    since it will stop dirty log unconditionally, so add if statement
>>    to ensure that memory_global_dirty_log_start/stop used in pairs.
>>
>> - modify the memory_global_dirty_log_start/log in xen vitualization
>>    and make the qemu works in the old way.
>>
>> v9
>> - rebase on master
>>
>> - make global_dirty_tracking a bitmask:
>>    pass the casted "flags" to the callback in the right way, and drop
>>    the malloc/free step.
>>
>> - make bitmask comments more accurate
>>
>> please review, thanks.
>>
>> v8
>> - drop the "none" type of DirtyRateMeasureMode
>>
>> - make global_dirty_tracking a bitmask:
>>    1. code clean: pass the casted "flags" as input parameter in
>>       memory_global_dirty_log_stop, and then drop the free step.
>>    2. squash commit "rename global_dirty_log to global_dirty_tracking"
>>       into commit "make global_dirty_tracking a bitmask"
>>    3. move "GLOBAL_DIRTY_MASK" macro to the bits's definations.
>>
>> - drop the "cleanup_dirtyrate_stat" in commit
>>    "move init step of calculation to main thread" so that each commit
>>    keeps the old way working.
>>
>> - add dirty rate unit "MB/s" in the output of hmp
>>
>> please review, may be this is the last version of this patchset, :)
>> thanks for Peter's patient instructions and reviews.
>>
>> Hyman Huang(黄勇)
>>
>> v7
>> - fix the code style problem, sorry about that
>>
>> v6:
>> - pick up commit "KVM: introduce dirty_pages and kvm_dirty_ring_enabled"
>>    which has been dropped in verison 5
>>
>> v5:
>> - rename global_dirty_log to global_dirty_tracking on Peter's advice
>>
>> - make global_dirty_tracking a bitmask:
>>    1. add assert statement to ensure starting dirty tracking repeatly
>>       not allowed.
>>    2. add assert statement to ensure dirty tracking cannot be stopped
>>       without having been started.
>>
>> - protecting dirty rate stat info:
>>    1. drop the mutext for protecting dirty rate introduced in version 4
>>    2. change the code block in query_dirty_rate_info so that requirements
>>       of "safe racing" to the dirty rate stat can be meet
>>
>> - make the helper function "record_dirtypages" inline and change
>>    the global var dirty_pages  to local var
>>
>> - free DirtyRateVcpuList in case of memory leak
>>
>> please review, thanks a lot.
>>
>> v4:
>> - make global_dirty_log a bitmask:
>>    1. add comments about dirty log bitmask
>>    2. use assert statement to check validity of flags
>>    3. add trace to log bitmask changes
>>
>> - introduce mode option to show what method calculation should be used,
>>    also, export mode option in the as last commmit
>>
>> - split cleanup and init of dirty rate stat and move it in the main
>>    thread
>>
>> - change the fields of DirtyPageRecord to uint64_t type so that we
>>    can calculation the increased dirty pages with the formula
>>    as Peter's advice: dirty pages = end_pages - start_pages
>>
>> - introduce mutex to protect dirty rate stat info
>>
>> - adjust order of registering thread
>>
>> - drop the memory free callback
>>
>> this version modify some code on Peter's advice, reference to:
>> https://lore.kernel.org/qemu-devel/YL5nNYXmrqMlXF3v@t490s/
>>
>> thanks again.
>>
>> v3:
>> - pick up "migration/dirtyrate: make sample page count configurable" to
>>    make patchset apply master correctly
>>
>> v2:
>> - rebase to "migration/dirtyrate: make sample page count configurable"
>>
>> - rename "vcpu" to "per_vcpu" to show the per-vcpu method
>>
>> - squash patch 5/6 into a single one, squash patch 1/2 also
>>
>> - pick up "hmp: Add "calc_dirty_rate" and "info dirty_rate" cmds"
>>
>> - make global_dirty_log a bitmask to make sure both migration and dirty
>>    could not intefer with each other
>>
>> - add memory free callback to prevent memory leaking
>>
>> the most different of v2 fron v1 is that we make the global_dirty_log a
>> bitmask. the reason is dirty rate measurement may start or stop dirty
>> logging during calculation. this conflict with migration because stop
>> dirty log make migration leave dirty pages out then that'll be a
>> problem.
>>
>> make global_dirty_log a bitmask can let both migration and dirty
>> rate measurement work fine. introduce GLOBAL_DIRTY_MIGRATION and
>> GLOBAL_DIRTY_DIRTY_RATE to distinguish what current dirty log aims
>> for, migration or dirty rate.
>>
>> all references to global_dirty_log should be untouched because any bit
>> set there should justify that global dirty logging is enabled.
>>
>> Please review, thanks !
>>
>> v1:
>>
>> Since the Dirty Ring on QEMU part has been merged recently, how to use
>> this feature is under consideration.
>>
>> In the scene of migration, it is valuable to provide a more accurante
>> interface to track dirty memory than existing one, so that the upper
>> layer application can make a wise decision, or whatever. More
>> importantly,
>> dirtyrate info at the granualrity of vcpu could provide a possibility to
>> make migration convergent by imposing restriction on vcpu. With Dirty
>> Ring, we can calculate dirtyrate efficiently and cheaply.
>>
>> The old interface implemented by sampling pages, it consumes cpu
>> resource, and the larger guest memory size become, the more cpu resource
>> it consumes, namely, hard to scale. New interface has no such drawback.
>>
>> Please review, thanks !
>>
>> Best Regards !
>>
>> Hyman Huang(黄勇) (6):
>>    KVM: introduce dirty_pages and kvm_dirty_ring_enabled
>>    memory: make global_dirty_tracking a bitmask
>>    migration/dirtyrate: introduce struct and adjust DirtyRateStat
>>    migration/dirtyrate: adjust order of registering thread
>>    migration/dirtyrate: move init step of calculation to main thread
>>    migration/dirtyrate: implement dirty-ring dirtyrate calculation
>>
>>   accel/kvm/kvm-all.c     |   7 ++
>>   hmp-commands.hx         |   7 +-
>>   hw/i386/xen/xen-hvm.c   |   4 +-
>>   include/exec/memory.h   |  20 +++-
>>   include/exec/ram_addr.h |   4 +-
>>   include/hw/core/cpu.h   |   1 +
>>   include/sysemu/kvm.h    |   1 +
>>   migration/dirtyrate.c   | 266 
>> +++++++++++++++++++++++++++++++++++++++++-------
>>   migration/dirtyrate.h   |  19 +++-
>>   migration/ram.c         |  15 ++-
>>   migration/trace-events  |   2 +
>>   qapi/migration.json     |  46 ++++++++-
>>   softmmu/memory.c        |  32 ++++--
>>   softmmu/trace-events    |   1 +
>>   14 files changed, 361 insertions(+), 64 deletions(-)
>>
Peter Xu Sept. 20, 2021, 4 p.m. UTC | #3
On Mon, Sep 20, 2021 at 11:45:21PM +0800, Hyman wrote:
> 
> 在 2021/8/30 7:11, Hyman 写道:
> > ping - hi, what would you think about this patchset ? :)
> ping again, just make sure this patchset not be ignored,
> any advice or comment are welcome. :)

Thanks for following this up, Yong.  Yes, I keep thinking this series (and
actually the other series to enable dirty log dirty measurement) is a nice
thing to have, and worth merging.
Peter Xu Oct. 15, 2021, 1:32 a.m. UTC | #4
On Wed, Jun 30, 2021 at 12:01:17AM +0800, huangy81@chinatelecom.cn wrote:
> From: Hyman Huang(黄勇) <huangy81@chinatelecom.cn>
> 
> v12
> - adjust the order of calculating dirty rate
>   let memory_global_dirty_log_sync before calculating as
>   v11 version description.

Ping for Yong.

Dave/Juan, any plan to review/merge this series (along with the other series of
dirty logging)?

I found it useful when I wanted to modify the program I used to generate
constant dirty workload - this series can help me to verify the change.

I still keep thinking this series is something good to have.  Thanks,
Hyman Huang Oct. 15, 2021, 2:07 a.m. UTC | #5
在 2021/10/15 9:32, Peter Xu 写道:
> On Wed, Jun 30, 2021 at 12:01:17AM +0800, huangy81@chinatelecom.cn wrote:
>> From: Hyman Huang(黄勇) <huangy81@chinatelecom.cn>
>>
>> v12
>> - adjust the order of calculating dirty rate
>>    let memory_global_dirty_log_sync before calculating as
>>    v11 version description.
> 
> Ping for Yong. >
> Dave/Juan, any plan to review/merge this series (along with the other series of
> dirty logging)?
> 
> I found it useful when I wanted to modify the program I used to generate
> constant dirty workload - this series can help me to verify the change.
> 
> I still keep thinking this series is something good to have.  Thanks,
the dirtyrate calculation has already been used to estimate time of live 
migration in "e cloud" production of chinatelecom, it also predict the 
migration success ratio, which provide valuable information for the 
cloud management plane when selecting which vm should be migrated.
>
Zheng Chuan Oct. 27, 2021, 6:31 a.m. UTC | #6
Hi.
I have no objection for the implement code itself.
But we should know or let the user know the performance penalty and conflicted with migration compared to the hash method, especially for the performance of vm with hugepages.

On 2021/10/15 10:07, Hyman Huang wrote:
> 
> 
> 在 2021/10/15 9:32, Peter Xu 写道:
>> On Wed, Jun 30, 2021 at 12:01:17AM +0800, huangy81@chinatelecom.cn wrote:
>>> From: Hyman Huang(黄勇) <huangy81@chinatelecom.cn>
>>>
>>> v12
>>> - adjust the order of calculating dirty rate
>>>    let memory_global_dirty_log_sync before calculating as
>>>    v11 version description.
>>
>> Ping for Yong. >
>> Dave/Juan, any plan to review/merge this series (along with the other series of
>> dirty logging)?
>>
>> I found it useful when I wanted to modify the program I used to generate
>> constant dirty workload - this series can help me to verify the change.
>>
>> I still keep thinking this series is something good to have.  Thanks,
> the dirtyrate calculation has already been used to estimate time of live migration in "e cloud" production of chinatelecom, it also predict the migration success ratio, which provide valuable information for the cloud management plane when selecting which vm should be migrated.
>>
>
Hyman Huang Oct. 28, 2021, 4:26 a.m. UTC | #7
在 2021/10/27 14:31, Zheng Chuan 写道:
> Hi.
> I have no objection for the implement code itself.
> But we should know or let the user know the performance penalty and conflicted with migration compared to the hash method, especially for the performance of vm with hugepages.

i dirty guest memory with 1G and do the measurement with two method.

the copy rate is almost 1,665 MB/s in vm

the following output is guest memory performance when do measurement 
with dirty ring method:
/init (00001): INFO: 1635392998977ms copied 1 GB in 00616ms
/init (00001): INFO: 1635392999593ms copied 1 GB in 00615ms
/init (00001): INFO: 1635393000211ms copied 1 GB in 00616ms
----------------- start measurement -----------------------
/init (00001): INFO: 1635393000884ms copied 1 GB in 00672ms		
/init (00001): INFO: 1635393001849ms copied 1 GB in 00963ms	
/init (00001): INFO: 1635393002578ms copied 1 GB in 00727ms
----------------- end measurement -----------------------
/init (00001): INFO: 1635393003195ms copied 1 GB in 00615ms
/init (00001): INFO: 1635393003811ms copied 1 GB in 00614ms
/init (00001): INFO: 1635393004427ms copied 1 GB in 00615ms

guest memory performance do not trigger any changes almostly with hash 
method:

the following is test results (measurment interval=1s):

method		measurement result  	copy rate during measurement
hash		44 MB/s			1,665MB/s	
dirty ring   	1167 MB/s		1,523MB/s、1,063MB/s、1,408MB/s

the max penalty is 36% during test interval(1s), the average penalty is 
20%。

if we trade off accurance, the dirty ring method may be a availiabe 
method for user. users can select a appropriate method as they need.

> 
> On 2021/10/15 10:07, Hyman Huang wrote:
>>
>>
>> 在 2021/10/15 9:32, Peter Xu 写道:
>>> On Wed, Jun 30, 2021 at 12:01:17AM +0800, huangy81@chinatelecom.cn wrote:
>>>> From: Hyman Huang(黄勇) <huangy81@chinatelecom.cn>
>>>>
>>>> v12
>>>> - adjust the order of calculating dirty rate
>>>>     let memory_global_dirty_log_sync before calculating as
>>>>     v11 version description.
>>>
>>> Ping for Yong. >
>>> Dave/Juan, any plan to review/merge this series (along with the other series of
>>> dirty logging)?
>>>
>>> I found it useful when I wanted to modify the program I used to generate
>>> constant dirty workload - this series can help me to verify the change.
>>>
>>> I still keep thinking this series is something good to have.  Thanks,
>> the dirtyrate calculation has already been used to estimate time of live migration in "e cloud" production of chinatelecom, it also predict the migration success ratio, which provide valuable information for the cloud management plane when selecting which vm should be migrated.
>>>
>>
>