From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dario Faggioli Subject: Re: [PATCH v2 00/16] Scheduling related tracing improvements Date: Fri, 4 Mar 2016 19:25:30 +0100 Message-ID: <1457115930.2959.584.camel@citrix.com> References: <20160216180550.27876.22680.stgit@Solace.station> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============7499947266098490169==" Return-path: Received: from mail6.bemta3.messagelabs.com ([195.245.230.39]) by lists.xen.org with esmtp (Exim 4.84) (envelope-from ) id 1abuQB-0006yW-Mw for xen-devel@lists.xenproject.org; Fri, 04 Mar 2016 18:25:51 +0000 In-Reply-To: <20160216180550.27876.22680.stgit@Solace.station> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: xen-devel@lists.xenproject.org, Konrad Rzeszutek Wilk , Jan Beulich , George Dunlap Cc: Ian Jackson , Tianyang Chen , Wei Liu , Meng Xu List-Id: xen-devel@lists.xenproject.org --===============7499947266098490169== Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="=-svmDR5zvf4ztaFPHnY3G" --=-svmDR5zvf4ztaFPHnY3G Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hello committers, George, This is basically a ping for this series, as I think most of it can actually go in, unless I've missed something. So, let me try to recap: On Tue, 2016-02-16 at 19:11 +0100, Dario Faggioli wrote: >=C2=A0 > Dario Faggioli (16): > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0xen: sched: __runq_tickle takes a use= less cpu parameter > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0xen: sched: move up the trace record = for vcpu_wake and > vcpu_sleep > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0xen: sched: improve domain creation t= racing > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0xen: credit2: pack trace data better = for xentrace_format > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0xen: RTDS: pack trace data better for= xentrace_format > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0xen: sched: tracing: enable TSC traci= ng for all events > Until here, it's in already. > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0xentrace: formats: update format of s= cheduling events > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0xentrace: formats: add events from Cr= edit scheduler > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0xentrace: formats: add events from Cr= edit2 scheduler > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0xentrace: formats: add events from RT= DS scheduler > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0xentrace: formats: add domain create = and destroy events. > About these, they've got Konrad's Reviewed-by, and George said: "I haven't reviewed the formats file changes, but all of them look reasonable; so for patches 7-11: Acked-by: George Dunlap " > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0xenalyze: handle scheduling events > Reviewed-by: Konrad Rzeszutek Wilk Acked-by: George Dunlap > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0xenalyze: handle Credit1 scheduler ev= ents > Reviewed-by: Konrad Rzeszutek Wilk Acked-by: George Dunlap > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0xenalyze: handle Credit2 scheduler ev= ents > Reviewed-by: Konrad Rzeszutek Wilk Acked-by: George Dunlap > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0xenalyze: handle RTDS scheduler event= s > This should be skipped. In fact, we agreed with George that the plan would be: "I think most of these patches can be checked in now.=C2=A0=C2=A0What about checking in the other patches, then sending a follow-up series with the struct changed in the scheduler, and then this patch with the resulting changes?" So, just ignore this patch. > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0xenalyze: handle DOM0 operaions event= s >=20 This one is actually missing Georges's Ack. I'd be fine with this being skipped as well, and I will resubmit as separate patch, or as part of the followup series mentioned above. Or George can Ack it, and it just can go in now. But this is not a big deal. What I would like, if possible, is for patches until 14 to be checked in, so I can submit the follow up (together with other patches that I have stacked on top of that). Thanks and regards, Dario --=20 <> (Raistlin Majere) ----------------------------------------------------------------- Dario Faggioli, Ph.D, http://about.me/dario.faggioli Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK) --=-svmDR5zvf4ztaFPHnY3G 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 iEYEABECAAYFAlbZ0xoACgkQk4XaBE3IOsTPcgCbBmy4hBh4fGGJcb5mRdhyUx/f MGgAnjRaueVxzZ7+8m6Ovvse1iya/0Q4 =RJJP -----END PGP SIGNATURE----- --=-svmDR5zvf4ztaFPHnY3G-- --===============7499947266098490169== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KWGVuLWRldmVs IG1haWxpbmcgbGlzdApYZW4tZGV2ZWxAbGlzdHMueGVuLm9yZwpodHRwOi8vbGlzdHMueGVuLm9y Zy94ZW4tZGV2ZWwK --===============7499947266098490169==--