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

Re: [Xen-devel] [PATCH v4 for Xen 4.6 0/4] Enable per-VCPU parameter settings for RTDS scheduler



On Sat, 2015-07-11 at 15:33 +0100, Wei Liu wrote:
> Hi Chong
> 
> This series is marked as "for 4.6", but we just hit feature freeze
> yesterday.
> 
Yeah, I wanted to reply myself about this, but Wei beat me... Good job
as release manager, I would say. :-)

> Given the status of this series (missing many acks), I am sorry to say
> this series will have to wait until next release.
> 
Indeed. The series is starting to look good, and, Chong, you're doing a
great work, especially by replying promptly to reviews, and reposting
new versions very quickly.

However, this series arrived a bit late in the dev cycle, and suffered
from some delay in reviewing (from me as well, sorry for that), but
(both) this things happen in (Open Source) software development, and we
can't do much about it.

Also, the original goal was to pull RTDS out of experimental, but, even
with this series in, we wouldn't get to there as:
 - not enough testing: it entered OSSTest not so long ago, which, e.g.,
   showed up it's failing on ARM!
 - not enough benchmarks/performance figures: I'd like to have the
   latency numbers, e.g., from cyclictest, we've spoke many times with
   Meng, give our official blessing at using it
 - the work Dagaen's doing is a rather fundamental restructuring, and it
   makes sense to do all the above (testing and performance evaluation)
   on top of the result of that for a bit, before declaring things
   stable and supported (or we risk disrupting that because of it, and
   since it's already ongoing, I'll really let him finish)

So, for the following reasons (coming from the above reasoning):
 - the series is good, but certainly still not ready;
 - having the series in, would not change much wrt RTDS in 4.6

I, as the maintainer of this feature, agree with Wei that we should work
toward merging this series really soon... at the beginning of 4.7
development cycle! :-D

> We will review this series in timely manner provided there are no other
> urgent matters for the release. Please keep up with your good work.
> 
Indeed. Thanks a log again to you, Meng, Dagaen, and everyone.

I'll review the series ASAP.

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)

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
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®.