On 5/22/20 8:07 AM, Thomas Huth wrote:
>> From: "Claudio Fontana" <cfont...@suse.de>
>> Sent: Thursday, May 21, 2020 8:54:05 PM
>>
>> this is a first step in the refactoring of cpus.c.
> 
> Could you maybe extend the commit message in the next version a little bit? 
> ... say something about *what* you are moving to a separate file (and maybe 
> why it is ok to move it), etc.?
> 
>  Thanks,
>   Thomas
> 
> 

Hello Thomas,

thanks for taking a look, I will add an explanatory message.

I was thinking something along the lines of:

"
move the vcpu throttling functionality into its own module.
It contains the controls to adjust and inspect vcpu throttling settings, start 
(set) and stop
vcpu throttling, and the throttling function itself that is run periodically on 
vcpus
to make them take a nap.
Execution of the throttling function on all vcpus is triggered by a timer,
registered at module initialization.

No functionality change.
"

Thanks,

Claudio






Reply via email to