[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] xen: sched: get rid of static private schedulers' structures



On Tue, 2016-02-09 at 04:27 -0700, Jan Beulich wrote:
> > > > On 08.02.16 at 14:19, <dario.faggioli@xxxxxxxxxx> wrote:
> > @@ -1995,13 +1996,11 @@ static void csched_tick_resume(const struct
> > scheduler *ops, unsigned int cpu)
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂ- now % MICROSECS(prv->tick_period_us) );
> > Â}
> > Â
> > -static struct csched_private _csched_priv;
> > -
> > Âstatic const struct scheduler sched_credit_def = {
> > ÂÂÂÂÂ.nameÂÂÂÂÂÂÂÂÂÂÂ= "SMP Credit Scheduler",
> > ÂÂÂÂÂ.opt_nameÂÂÂÂÂÂÂ= "credit",
> > ÂÂÂÂÂ.sched_idÂÂÂÂÂÂÂ= XEN_SCHEDULER_CREDIT,
> > -ÂÂÂÂ.sched_dataÂÂÂÂÂ= &_csched_priv,
> > +ÂÂÂÂ.sched_dataÂÂÂÂÂ= NULL,
> 
> You're maintainer of this code, so you know whether you really
> want these NULL initializers, but the look pretty pointless to me.
> 
Yes, I've also been unsure about what to do here.

The reason why I put them there is that, when looking at the code, it
is important to be able to quickly get the idea of where sched_data
points, at any given time. Therefore, being a little bit more verbose
than necessary would be, in this case, actually helpful.

Anyway, this is not a super strong opinion, so I can be talked into
getting rid of them, if others think we better don't have them too.

Thanks and Regards,
Dario
-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.