weizhouapache commented on PR #8520:
URL: https://github.com/apache/cloudstack/pull/8520#issuecomment-1910464449

   > > @GutoVeronezi you know some issues are difficult to reproduce. the issue 
could be related to the template (Windows version, virtio drivers, etc), and 
the host (Linux OS version, qemu/libvirt version), etc I did not face any issue 
with Linux VMs caused by it. so I made some changes to enable it on Linux VMs, 
but disable it for Windows VMs.
   > 
   > But then we do not have a solid reason to not change the value, just 
speculations. If at least we could reproduce the error you saying that exist, 
we could look for the real cause and solve the problem (or at least find an 
explanation).
   
   @GutoVeronezi 
   If you are running a production, you might know, when we face an issue 
(Windows VM got frozen after live migration in my case), the first thing we 
need to do, is to find the workaround which fix the issue and then avoid it (in 
my case, the workaround is disabling memory stats).
   root cause?  in my case, we need to dive into the source code of QEMU and 
virtio driver. the root cause is not important to me, as I can accept the 
problem that Windows VM has wrong memory usage (100%) which is a minor issue, 
espscially comparing with the issue that Windows VM got frozen.
   
   changing vm_memballoon_stats_period to 60, will add a new setting in the vm 
definition, which will probably cause some regression which is out of our 
control. We should disable it by default.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to