[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v3] schedule: move last_run_time to the credit scheduler privates

> On May 30, 2019, at 3:15 PM, Andrii Anisov <andrii.anisov@xxxxxxxxx> wrote:
> From: Andrii Anisov <andrii_anisov@xxxxxxxx>
> The structure member last_run_time is used by credit scheduler only.
> So move it from a generic vcpu structure to the credit scheduler private
> vcpu definition.

This seems like a useful change, and the commit message has a lot of good 
detail, thanks.  But I’m left wondering: Is the main idea here just to 
generally reduce code and data usage when not running the credit scheduler, or 
is there another reason?

If it’s the first, a quick note to that effect will help put archaeologist’s 
minds at ease. :-)  This could probably be added on commit.  (I’ll do a full 
review it in a day or two if Dario doesn’t beat me to it.)


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.