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

Re: [Xen-devel] [PATCH 1/1] Improved RTDS scheduler



On Mon, Jan 4, 2016 at 7:42 PM, Ian Campbell <ian.campbell@xxxxxxxxxx> wrote:
> On Thu, 2015-12-31 at 04:45 -0500, Tianyang Chen wrote:
>> Budget replenishment is now handled by a dedicated timer which is
>> triggered at the most imminent release time of all runnable vcpus.
>>
>> Signed-off-by: Tianyang Chen <tiche@xxxxxxxxxxxxxx>
>> Signed-off-by: Meng Xu <mengxu@xxxxxxxxxxxxx>
>> Signed-off-by: Dagaen Golomb <dgolomb@xxxxxxxxxxxxxx>
>> ---
>>  0000-cover-letter.patch            |   16 +++
>>  0001-Improved-RTDS-scheduler.patch |  280 
>> ++++++++++++++++++++++++++++++++++++
>>  bak                                |   62 ++++++++
>
> I assume you didn't actually intend to commit those ;-)

No. This version was sent by mistake. Please ignore this version. :-(

Tianyang sent another version (v2) with the subject "[PATCH V2 1/1]
Improved RTDS scheduler".


>
> (In general single patches do not require a separate cover letter unless
> the required context contains a large amount of information which is not
> appropriate for the commit message of the actual change, I'll leave it to
> you and the scheduler maintainers to decide how much of your cover letter
> it would be appropriate to move to the commit message).

The cover letter is supposed to explain the design idea of the
improved RTDS scheduler so that reviewers could (potentially) save
time in reviewing the patch, we think. :-)
Probably, the commit message should be refined and self-contained?

Thank you very much for your comments and suggestions!

Best,

Meng
-- 


-----------
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
http://lists.xen.org/xen-devel


 


Rackspace

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