From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dario Faggioli Subject: Re: [PATCH] xen: add hypercall option to temporarily pin a vcpu Date: Fri, 26 Feb 2016 14:39:02 +0100 Message-ID: <1456493942.2959.187.camel@citrix.com> References: <1456419000-390-1-git-send-email-jgross@suse.com> <56D0395702000078000D69A6@suse.com> <56D033A8.7020009@suse.com> <1456485632.2959.154.camel@citrix.com> <56D0559C02000078000D6AFE@prv-mh.provo.novell.com> <1456492038.2959.170.camel@citrix.com> <56D0620A02000078000D6BB6@prv-mh.provo.novell.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============2332868027852814498==" Return-path: In-Reply-To: <56D0620A02000078000D6BB6@prv-mh.provo.novell.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: Jan Beulich Cc: Juergen Gross , george.dunlap@eu.citrix.com, andrew.cooper3@citrix.com, ian.jackson@eu.citrix.com, xen-devel@lists.xen.org, dgdegra@tycho.nsa.gov List-Id: xen-devel@lists.xenproject.org --===============2332868027852814498== Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="=-tKY1YDUrTa5UzyBQVDoj" --=-tKY1YDUrTa5UzyBQVDoj Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Fri, 2016-02-26 at 06:32 -0700, Jan Beulich wrote: > > > > On 26.02.16 at 14:07, wrote: > > We can argue about that being useful or not, and about it being > > (potentially) too noisy or not. I personally think it could be > > useful > > (it's XENLOG_DEBUG, after all), but I won't oppose getting rid of > > it... > > I am just not getting why you're saying "not currently running on > > pCPU0". > Oh, you're right, that was too strict - "not being allowed to run > on CPU0" would be the right description. And indeed that makes > it look not as noisy (but too much, since during suspend this is > what one has to expect would happen). > Yes, mostly on the ground that it's the intended behavior and (in that case really) temporary, I'm indeed ok silencing this on suspend. Since it's pretty independent, I'd prefer this to be done in a separate patch, together with the resume side, as (AFAIUI) Juergen is planning to do already. Dario --=20 <> (Raistlin Majere) ----------------------------------------------------------------- Dario Faggioli, Ph.D, http://about.me/dario.faggioli Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK) --=-tKY1YDUrTa5UzyBQVDoj 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 iEYEABECAAYFAlbQVXYACgkQk4XaBE3IOsRjXwCfZTtGATUk9R+InaVBefW7aQ+Q X30An33px0lwLbJyvg1XA9ui14tbCbxz =q+cT -----END PGP SIGNATURE----- --=-tKY1YDUrTa5UzyBQVDoj-- --===============2332868027852814498== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KWGVuLWRldmVs IG1haWxpbmcgbGlzdApYZW4tZGV2ZWxAbGlzdHMueGVuLm9yZwpodHRwOi8vbGlzdHMueGVuLm9y Zy94ZW4tZGV2ZWwK --===============2332868027852814498==--