From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dario Faggioli Subject: [PATCHv2 0 of 1] Rework locking for sched_adjust. Date: Wed, 07 Dec 2011 15:49:11 +0100 Message-ID: <1323269351.22009.15.camel@Solace> References: <1322060131.30168.15.camel@Abyss> <1322060876.30168.18.camel@Abyss> <1322065473.1005.151.camel@zakaz.uk.xensource.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============9030526257597187537==" Return-path: In-Reply-To: <1322065473.1005.151.camel@zakaz.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: "xen-devel@lists.xensource.com" Cc: George Dunlap , Juergen Gross , "Keir (Xen.org)" , Ian Campbell List-Id: xen-devel@lists.xenproject.org --===============9030526257597187537== Content-Type: multipart/signed; micalg="pgp-sha1"; protocol="application/pgp-signature"; boundary="=-vp+evIGRiKfSyFLKUcqq" --=-vp+evIGRiKfSyFLKUcqq Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hi everyone, Here it is v2 of the lock reworking around and within sched-adjust. With respect to the first posting [1]: - I _did_not_ move the per-pluggable scheduler lock toward schedule.c, as agreed with George during review; - I fixed the bug in sedf spotted by Juergen the way he suggested; - I've finally been able to test it under all the three schedulers,=20 and it is doing its job, at least here; Notice the series "collapsed" in one signle patch, as it was being hard to find a breakdown of it that does not introduce regressions and/or transient deadlock situations worse than the ones it's trying to cure... I hope it's still readable and comfortable to review. :-) Thanks to everyone who provided his feedback on the first version of this. Regards, Dario [1] http://xen.1045712.n5.nabble.com/RFC-RFT-PATCH-0-of-3-rework-locking-in= -sched-adjust-td5016899.html -- xen/common/sched_credit.c | 10 ++++++-- xen/common/sched_credit2.c | 21 ++++++++++--------- xen/common/sched_sedf.c | 156 ++++++++++++++++++++++++++++++++++++++++= +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++= ++++----------------------------------- xen/common/schedule.c | 34 +------------------------------- 4 files changed, 140 insertions(+), 81 deletions(-) --=20 <> (Raistlin Majere) ------------------------------------------------------------------- Dario Faggioli, http://retis.sssup.it/people/faggioli Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK) PhD Candidate, ReTiS Lab, Scuola Superiore Sant'Anna, Pisa (Italy) --=-vp+evIGRiKfSyFLKUcqq 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 v1.4.11 (GNU/Linux) iEYEABECAAYFAk7ffOcACgkQk4XaBE3IOsSVdgCdGBkXi865t9vTvqWkKx+dZ7VZ ldMAoIeeab4df5Ltlp1rFiZXwXvliEnX =XK9u -----END PGP SIGNATURE----- --=-vp+evIGRiKfSyFLKUcqq-- --===============9030526257597187537== 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.xensource.com http://lists.xensource.com/xen-devel --===============9030526257597187537==--