From mboxrd@z Thu Jan  1 00:00:00 1970
From: Dario Faggioli <dario.faggioli@citrix.com>
Subject: Re: [Patch 2/2]xen/sched_credit2.c : Runqueue per core
Date: Mon, 9 Mar 2015 13:03:42 +0000
Message-ID: <1425906220.2729.40.camel@citrix.com>
References: <20150309085501.GA4280@gmail.com>
	<CAFLBxZZgiMQOzc0ROmOTe6qGOEaGk0YcLwtWwph3Fq_S=b5-nQ@mail.gmail.com>
Mime-Version: 1.0
Content-Type: multipart/mixed; boundary="===============0814884449070899029=="
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <CAFLBxZZgiMQOzc0ROmOTe6qGOEaGk0YcLwtWwph3Fq_S=b5-nQ@mail.gmail.com>
Content-Language: en-US
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: "dunlapg@umich.edu" <dunlapg@umich.edu>
Cc: "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>, George Dunlap <George.Dunlap@citrix.com>, "uma.sharma523@gmail.com" <uma.sharma523@gmail.com>
List-Id: xen-devel@lists.xenproject.org

--===============0814884449070899029==
Content-Language: en-US
Content-Type: multipart/signed; micalg=pgp-sha1;
	protocol="application/pgp-signature"; boundary="=-Hq0sD/izWpuK6CtAIiAx"

--=-Hq0sD/izWpuK6CtAIiAx
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Mon, 2015-03-09 at 12:18 +0000, George Dunlap wrote:
> On Mon, Mar 9, 2015 at 8:55 AM, Uma Sharma <uma.sharma523@gmail.com> wrot=
e:

> > --- a/xen/common/sched_credit2.c
> > +++ b/xen/common/sched_credit2.c

> > @@ -1935,15 +1938,36 @@ static void init_pcpu(const struct scheduler *o=
ps, int cpu)
> >          return;
> >      }
> >
> > +    /*Figure out which type of runqueue are to be created */
> > +    if (!strcmp(opt_credit2_runquque, "socket")) {
> > +        rq =3D 's';
> > +    } else if (!strcmp(opt_credit2_runquque, "core")) {
> > +        rq =3D 'c';
> > +    } else {
> > +        rq =3D 's';
> > +    }
>=20

> It would be more typical, rather than have this be a char resolving to
> 's' and 'c', to have it be an int, and have the values be #defines;
> for example, "CREDIT2_OPT_RUNQUEUE_CORE" and
> "CREDIT2_OPT_RUNQUEUE_SOCKET".
>=20
I was about to suggest the same.

> Also, given that your experiments show 'core' to work quite a bit
> better than 'socket', I'd suggest making it default to core rather
> than socket. :-)
>=20
+1.

Of course, as I said already, you should explain and provide the numbers
about this performance improvement in the cover letter of the series
and, IMO, reference that in the changelog of this patch too (not putting
the full results, but a quick summary of them would be good).

Regards,
Dario

--=-Hq0sD/izWpuK6CtAIiAx
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

iEYEABECAAYFAlT9miwACgkQk4XaBE3IOsQQJgCeIE56moUZfGq+FSpgOaTUwVDs
RV8AoJEKtBMJ+YE9it/P+/tAv0W+G/8T
=INxy
-----END PGP SIGNATURE-----

--=-Hq0sD/izWpuK6CtAIiAx--


--===============0814884449070899029==
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

--===============0814884449070899029==--