Re: [Xen-devel] [PATCH v4 for Xen 4.6 0/4] Enable per-VCPU parameter settings for RTDS scheduler
On Mon, Jul 27, 2015 at 10:14 AM, Dario Faggioli wrote: > On Fri, 2015-07-10 at 23:52 -0500, Chong Li wrote: > > > Or by "just" adding a note before the actual output: > > # xl sched-rtds -d vm1 > Showing per-vm(s) default scheduling parameters, > use `-v' for seeing the actual parameters of each vcpu > NameIDPeriodBudget > vm1 1 1 4000 > > The latter is more verbose, but I think is what I see as more useful > (and, probably, the easier to implement). I agree. I'll add it in the next version. > > Regards, > Dario > -- > <> (Raistlin Majere) > - > Dario Faggioli, Ph.D, http://about.me/dario.faggioli > Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK) -- Chong Li Department of Computer Science and Engineering Washington University in St.louis ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH v4 for Xen 4.6 0/4] Enable per-VCPU parameter settings for RTDS scheduler
On Fri, 2015-07-10 at 23:52 -0500, Chong Li wrote: > Using "xl sched-rtds" will output the default scheduling parameters > for each domain. An example would be like: > > # xl sched-rtds > Cpupool Pool-0: sched=RTDS > NameIDPeriodBudget > Domain-0 0 1 4000 > vm1 1 1 4000 > vm2 2 1 4000 > Using command, e.g., "xl sched-rtds -d vm1" will output the default > scheduling parameters of vm1. An example would be like: > > # xl sched-rtds -d vm1 > NameIDPeriodBudget > vm1 1 1 4000 > Thanks for making these two works. I think it would be great if we could, in this case, output somehow and somewhere that we are actually showing some defaults, instead than anything that is really in use. I don't know whether with something like this: # xl sched-rtds Cpupool Pool-0: sched=RTDS NameIDPeriodBudget Domain-0 (defaults) 0 1 4000 vm1 (defaults) 1 1 4000 vm2 (defaults) 2 1 4000 # xl sched-rtds -d vm1 NameIDPeriodBudget vm1 (defaults) 1 1 4000 Or by "just" adding a note before the actual output: # xl sched-rtds -d vm1 Showing per-vm(s) default scheduling parameters, use `-v' for seeing the actual parameters of each vcpu NameIDPeriodBudget vm1 1 1 4000 The latter is more verbose, but I think is what I see as more useful (and, probably, the easier to implement). Regards, Dario -- <> (Raistlin Majere) - Dario Faggioli, Ph.D, http://about.me/dario.faggioli Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK) signature.asc Description: This is a digitally signed message part ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH v4 for Xen 4.6 0/4] Enable per-VCPU parameter settings for RTDS scheduler
On Mon, 2015-07-13 at 22:45 -0700, Meng Xu wrote: > Hi Dario, > Hi, > - 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 > > > Considering t > he improved version Dagaen is working on should be able to improve the > efficiency of the scheduler "a lot" > , > I will evaluate the performance of the RTDS scheduler with cyclictest > once Dagaen have the improved version done. > Sure thing! It actually was part of my own reasoning about why we want to wait. :-) Regards, Dario -- <> (Raistlin Majere) - Dario Faggioli, Ph.D, http://about.me/dario.faggioli Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK) signature.asc Description: This is a digitally signed message part ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH v4 for Xen 4.6 0/4] Enable per-VCPU parameter settings for RTDS scheduler
Hi Dario, - 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 Considering t he improved version Dagaen is working on should be able to improve the efficiency of the scheduler "a lot" , I will evaluate the performance of the RTDS scheduler with cyclictest once Dagaen have the improved version done. Thanks, 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@lists.xen.org http://lists.xen.org/xen-devel
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 -- <> (Raistlin Majere) - Dario Faggioli, Ph.D, http://about.me/dario.faggioli Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK) signature.asc Description: This is a digitally signed message part ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH v4 for Xen 4.6 0/4] Enable per-VCPU parameter settings for RTDS scheduler
Hi Chong This series is marked as "for 4.6", but we just hit feature freeze yesterday. Given the status of this series (missing many acks), I am sorry to say this series will have to wait until next release. 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. Wei. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel