From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dario Faggioli Subject: Re: [PATCH v1 2/4] libxc: add rt scheduler Date: Fri, 5 Sep 2014 12:34:24 +0200 Message-ID: <1409913264.2673.298.camel@Solace.lan> References: <1408921125-21470-1-git-send-email-mengxu@cis.upenn.edu> <1408921125-21470-3-git-send-email-mengxu@cis.upenn.edu> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============7895894185901700673==" Return-path: In-Reply-To: <1408921125-21470-3-git-send-email-mengxu@cis.upenn.edu> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Meng Xu Cc: ian.campbell@citrix.com, xisisu@gmail.com, stefano.stabellini@eu.citrix.com, george.dunlap@eu.citrix.com, ian.jackson@eu.citrix.com, xen-devel@lists.xen.org, xumengpanda@gmail.com, JBeulich@suse.com, chaowang@wustl.edu, lichong659@gmail.com, dgolomb@seas.upenn.edu List-Id: xen-devel@lists.xenproject.org --===============7895894185901700673== Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="=-ElgDoxllPxNKgvs0f+UE" --=-ElgDoxllPxNKgvs0f+UE Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On dom, 2014-08-24 at 18:58 -0400, Meng Xu wrote: > --- /dev/null > +++ b/tools/libxc/xc_rt.c > @@ -0,0 +1,90 @@ > +/***********************************************************************= ***** > + * > + * File: xc_rt.c > + * Author: Sisu Xi=20 > + * Meng Xu > + * > + * Description: XC Interface to the rt scheduler > + * > + * This library is free software; you can redistribute it and/or > + * modify it under the terms of the GNU Lesser General Public > + * License as published by the Free Software Foundation; > + * version 2.1 of the License. > + * > + * This library is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * Lesser General Public License for more details. > + * > + * You should have received a copy of the GNU Lesser General Public > + * License along with this library; if not, write to the Free Software > + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-= 1301 USA > + */ > + > +#include "xc_private.h" > + > +int xc_sched_rt_domain_set(xc_interface *xch, > + uint32_t domid, > + struct xen_domctl_sched_rt_params *sdom) > +{ > + int rc; > + DECLARE_DOMCTL; > + > + domctl.cmd =3D XEN_DOMCTL_scheduler_op; > + domctl.domain =3D (domid_t) domid; > + domctl.u.scheduler_op.sched_id =3D XEN_SCHEDULER_RT_DS; > Aha! So, I was not dreaming about the whole RT_DS thing! :-D Perhaps then had that discussion about the low level side of things then. Well, I think that, if the name of the scheduler is RT_DS, it should be that for libxl and xl as well. After all, the reason why we're calling RT_DS in Xen, is that we want to be able to add others RT_FOO, RT_BAR algorithm/schedulers, in future. If that will happen, we'll need a way to reference them from the higher layer of the toolstack as well. When we'll have RT_DS and RT_CBS in Xen, and RT in libxl, to which one the RT in libxl will refer? So, just push the RT_DS thing all the way up to libxl and xl. As we said for the Xen part, you can keep the source filenames _rt.c, but functions and defines needs to be specific. So, for instance, this file can continue being xc_rt.c, but this function needs to be called xc_sched_rtds_domain_set() (or .._rt_ds_domain_..). > + domctl.u.scheduler_op.cmd =3D XEN_DOMCTL_SCHEDOP_putinfo; > + domctl.u.scheduler_op.u.rt.vcpu_index =3D sdom->index; > + domctl.u.scheduler_op.u.rt.period =3D sdom->period; > + domctl.u.scheduler_op.u.rt.budget =3D sdom->budget; > + > + rc =3D do_domctl(xch, &domctl); > + > + return rc; > +} > + These functions are going to change quite a bit in next version, due to the interface changes we agreed upon. It'd then be quite pointless to put much more comments, but, overall, the hcall wrapping, the bouncing logic, and everything I can see here looks fine to me. Regards, Dario --=20 <> (Raistlin Majere) ----------------------------------------------------------------- Dario Faggioli, Ph.D, http://about.me/dario.faggioli Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK) --=-ElgDoxllPxNKgvs0f+UE Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part Content-Transfer-Encoding: 7bit -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iEYEABECAAYFAlQJkbAACgkQk4XaBE3IOsREFACfUKT1jxK+Kfubdoye+CFgIpKb V7oAn1XoZ8AuW8P9MR+K3/yqZVOM6lAQ =7Y0u -----END PGP SIGNATURE----- --=-ElgDoxllPxNKgvs0f+UE-- --===============7895894185901700673== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel --===============7895894185901700673==--