[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC 1/3] cpu-throttle: new module, extracted from cpus.c
From: |
Claudio Fontana |
Subject: |
Re: [RFC 1/3] cpu-throttle: new module, extracted from cpus.c |
Date: |
Fri, 22 May 2020 12:54:27 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.4.1 |
On 5/22/20 12:26 PM, Alex Bennée wrote:
>
> Claudio Fontana <address@hidden> writes:
>
>> On 5/22/20 8:07 AM, Thomas Huth wrote:
>>>> From: "Claudio Fontana" <address@hidden>
>>>> 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.
>> "
>
> Is vcpu throttling a TCG only feature?
>
No, are you suggesting we only refactor code out of cpus.c based on whether
it's tcg or not?
Ciao,
Claudio
- [RFC 0/3] QEMU cpus.c refactoring, Claudio Fontana, 2020/05/21
- [RFC 1/3] cpu-throttle: new module, extracted from cpus.c, Claudio Fontana, 2020/05/21
- Re: [RFC 1/3] cpu-throttle: new module, extracted from cpus.c, Thomas Huth, 2020/05/22
- Re: [RFC 1/3] cpu-throttle: new module, extracted from cpus.c, Claudio Fontana, 2020/05/22
- Re: [RFC 1/3] cpu-throttle: new module, extracted from cpus.c, Alex Bennée, 2020/05/22
- Re: [RFC 1/3] cpu-throttle: new module, extracted from cpus.c,
Claudio Fontana <=
- Re: [RFC 1/3] cpu-throttle: new module, extracted from cpus.c, Alex Bennée, 2020/05/22
- Re: [RFC 1/3] cpu-throttle: new module, extracted from cpus.c, Claudio Fontana, 2020/05/22
[RFC 3/3] cpus: implement cpus interfaces for per-accel threads, Claudio Fontana, 2020/05/21
[RFC 2/3] cpu-timers: new module extracted from cpus.c, Claudio Fontana, 2020/05/21