From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dario Faggioli Subject: Re: [PATCH v6 for Xen 4.7 1/4] xen: enable per-VCPU parameter settings for RTDS scheduler Date: Wed, 16 Mar 2016 15:53:59 +0100 Message-ID: <1458140039.3102.925.camel@citrix.com> References: <1457286958-5427-1-git-send-email-lichong659@gmail.com> <1457286958-5427-2-git-send-email-lichong659@gmail.com> <20160308190950.GT31271@citrix.com> <1457539804.3102.425.camel@citrix.com> <56E05F8F02000078000DAF15@prv-mh.provo.novell.com> <56E6866202000078000DBECD@prv-mh.provo.novell.com> <1457946623.3102.636.camel@citrix.com> <56E68F5202000078000DBF46@prv-mh.provo.novell.com> <1457949930.3102.641.camel@citrix.com> <1458060092.3102.721.camel@citrix.com> <1458116597.3102.773.camel@citrix.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============6788636238987995058==" Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: Meng Xu Cc: Chong Li , Wei Liu , Sisu Xi , GeorgeDunlap , xen-devel , Jan Beulich , Chong Li , Dagaen Golomb List-Id: xen-devel@lists.xenproject.org --===============6788636238987995058== Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="=-6DSaTxtjs76bwhHYUtjP" --=-6DSaTxtjs76bwhHYUtjP Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Wed, 2016-03-16 at 10:37 -0400, Meng Xu wrote: > On Wed, Mar 16, 2016 at 4:23 AM, Dario Faggioli > wrote: > >=C2=A0 > > I continue to think that it could be useful to have this logged, > > but > > I'm leaning toward just killing it for now (and maybe finding > > another > > way to check and warn about the same thing or one of the effects it > > produces, later). > >=20 > > Meng, what do you think? > I'm thinking about if it may not be worthwhile *for now only* to > provide such information with so much effort and the danger of > introducing more serious issues. >=20 Yeah, exactly what I was also saying. > Right, race condition occurs on the global variable and I believe we > don't want to encounter this race condition. > So let's just not use the global variable. >=20 Well, it would just affect the logging itself. But then, why clobber the (clarity of the) code for introducing proper per-domain logging, and ending up with something that may not actually be per-domain.. that's how I was thinking at it. > We should definitely put a large warning in the wiki for the RTDS > scheduler about the parameter settings. Incorrect setting should > never > crash system but may lead to poor real-time performance users want. >=20 For example, one way of dealing with this would be to allow the user/sysadmin to set what the minimum budget and period they want to be able to use would be, by means of SYSCTLs. We kind of like have a static and a dynamic limit. The former, hardcoded in Xen, to something that we know is unreasonable on any hardware platform available at that time. The latter, we can set to a rather conservative high value by default, but let users that wants to lower it, do that... and there is where we can print all the warnings we want! I'm just tossing this out there, though, and this is certainly something that can come as future work. Regards, Dario --=20 <> (Raistlin Majere) ----------------------------------------------------------------- Dario Faggioli, Ph.D, http://about.me/dario.faggioli Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK) --=-6DSaTxtjs76bwhHYUtjP 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 iEYEABECAAYFAlbpc4cACgkQk4XaBE3IOsRnjgCgiWDrmDIAsglZv0T3PiCPP72H Cf0An2K4PMLCO05u1d6iRkFrZXGcedni =xDii -----END PGP SIGNATURE----- --=-6DSaTxtjs76bwhHYUtjP-- --===============6788636238987995058== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KWGVuLWRldmVs IG1haWxpbmcgbGlzdApYZW4tZGV2ZWxAbGlzdHMueGVuLm9yZwpodHRwOi8vbGlzdHMueGVuLm9y Zy94ZW4tZGV2ZWwK --===============6788636238987995058==--