From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH] Osstest: stop testing SEDF, start testing RTDS Date: Fri, 22 May 2015 11:02:33 +0100 Message-ID: <1432288953.10746.162.camel@citrix.com> References: <1432288527.7907.108.camel@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta3.messagelabs.com ([195.245.230.39]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1YvjnE-0005z2-RZ for xen-devel@lists.xenproject.org; Fri, 22 May 2015 10:03:04 +0000 In-Reply-To: <1432288527.7907.108.camel@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Dario Faggioli Cc: "xen-devel@lists.xenproject.org" List-Id: xen-devel@lists.xenproject.org On Fri, 2015-05-22 at 11:55 +0200, Dario Faggioli wrote: > the SEDF scheduler is about to be deprecated and go > away (see [1]). OTOH, the RTDS scheduler is here to > stay. > > It therefore makes sense to stop smoke testing the > former in favour of the latter. Note that the > -sedf-pin jobs where only added in order to try to > debug a long standing issue with SEDF; it is not > necessary to have anything like that for RTDS. Does the RTDS scheduler work out of the box? I had a feeling that it required per domain config settings to be worked out? > > For now, as RTDS is still marked as experimental, > test failures are allowed, as it was for SEDF. > > [1] http://lists.xen.org/archives/html/xen-devel/2015-05/msg02874.html > > Signed-off-by: Dario Faggioli > --- > allow.all | 2 +- > make-flight | 31 +++++++++---------------------- > 2 files changed, 10 insertions(+), 23 deletions(-) > > diff --git a/allow.all b/allow.all > index 6715b2e..2897b2e 100644 > --- a/allow.all > +++ b/allow.all > @@ -1,4 +1,4 @@ > -test-@@-sedf@@ > +test-@@-rtds@@ > build-@@ logs-capture@@ > test-@@-pcipt@@ > test-@@-qemuu-@@ guest-localmigrate > diff --git a/make-flight b/make-flight > index 8a1fceb..837f372 100755 > --- a/make-flight > +++ b/make-flight > @@ -262,30 +262,16 @@ do_hvm_rhel6_tests () { > done > } > > -do_sedf_tests () { > +do_nondef_sched_tests () { > if [ $xenarch != $dom0arch ]; then > return > fi > > - for pin in '' -pin; do > - job_create_test test-$xenarch$kern-$dom0arch-xl-sedf$pin \ > - test-debian xl $xenarch $dom0arch \ > - guests_vcpus=4 \ > - xen_boot_append="sched=sedf loglvl=all ${pin:+dom0_vcpus_pin}" \ > - linux_boot_append='loglevel=9 debug' \ > - $debian_runvars all_hostflags=$most_hostflags > - done > -} > - > -do_credit2_tests () { > - if [ $xenarch != $dom0arch ]; then > - return > - fi > - > - job_create_test test-$xenarch$kern-$dom0arch-xl-credit2 \ > - test-debian xl $xenarch $dom0arch \ > - guests_vcpus=4 xen_boot_append='sched=credit2' \ > - $debian_runvars all_hostflags=$most_hostflags > + sched=$1 > + job_create_test test-$xenarch$kern-$dom0arch-xl-$sched \ > + test-debian xl $xenarch $dom0arch \ > + guests_vcpus=4 xen_boot_append="sched=$sched" \ > + $debian_runvars all_hostflags=$most_hostflags > } > > do_multivcpu_tests () { > @@ -350,8 +336,9 @@ test_matrix_do_one () { > do_pv_debian_tests > > do_multivcpu_tests > - do_sedf_tests > - do_credit2_tests > + for s in credit2 rtds; do > + do_nondef_sched_tests $s > + done > > # No further arm tests at the moment > if [ $dom0arch = armhf ]; then >