Re: [Xen-devel] [PATCH] xen:rtds: Update last_start whenever cur_budget is updated

[Meng Xu <xumengpanda@xxxxxxxxx>]

Hi Wei and Dario,


On Thu, Oct 27, 2016 at 6:03 AM, Wei Liu <wei.liu2@xxxxxxxxxx> wrote:
> On Thu, Oct 27, 2016 at 11:14:36AM +0200, Dario Faggioli wrote:
>> On Wed, 2016-10-26 at 15:06 -0400, Meng Xu wrote:
>> > We keep last_start updated whenever cur_budget is updated.
>> > This avoids subtle bugs in case burn_budget() will be called
>> > in other places in the future.
>> >
>> I'd change this into something like:
>>
>> "Make budget accounting code more consistent by making sure the values
>> used to compute how much budget has been consumed are updated together.
>>
>> This makes code resilient to calling burn_budget() from more than just
>> one place --in case we will need to do that-- without risking subtle
>> bugs."

I agree.


>>
>> Also, when that is the case, like here, you need to explicitly state
>> that no functional changes are intended.


>>
>> With all this done to the changelog:
>>
>> > Signed-off-by: Meng Xu <mengxu@xxxxxxxxxxxxx>
>> >
>> Acked-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
>>
>> This can go in 4.8, but can also well wait for 4.9.
>>
>
> Meng, let me know if you're OK with Dario's suggestion.
>
> Wei.
>
>> Regards,
>> Dario
>> --
>> <<This happens because I choose it to happen!>> (Raistlin Majere)
>> -----------------------------------------------------------------
>> Dario Faggioli, Ph.D, http://about.me/dario.faggioli
>> Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)
>
>



-- 
-----------
Meng Xu
PhD Student in Computer and Information Science
University of Pennsylvania
http://www.cis.upenn.edu/~mengxu/

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel