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

Re: [Xen-devel] [PATCH 2/3] xen: sched RTDS: use uint64_t for tracing time values



On Mon, 2016-03-14 at 03:07 -0600, Jan Beulich wrote:
> > > > On 12.03.16 at 12:34, <dario.faggioli@xxxxxxxxxx> wrote:
> > --- a/xen/common/sched_rt.c
> > +++ b/xen/common/sched_rt.c
> > @@ -361,17 +361,14 @@ rt_update_deadline(s_time_t now, struct
> > rt_vcpu *svc)
> >  
> >      /* TRACE */
> >      {
> > -        struct {
> > +        struct __packed {
> >              unsigned vcpu:16, dom:16;
> > -            unsigned cur_deadline_lo, cur_deadline_hi;
> > -            unsigned cur_budget_lo, cur_budget_hi;
> > +            uint64_t cur_deadline, cur_budget;
> >          } d;
> >          d.dom = svc->vcpu->domain->domain_id;
> >          d.vcpu = svc->vcpu->vcpu_id;
> > -        d.cur_deadline_lo = (unsigned) svc->cur_deadline;
> > -        d.cur_deadline_hi = (unsigned) (svc->cur_deadline >> 32);
> > -        d.cur_budget_lo = (unsigned) svc->cur_budget;
> > -        d.cur_budget_hi = (unsigned) (svc->cur_budget >> 32);
> > +        d.cur_deadline = (uint64_t) svc->cur_deadline;
> > +        d.cur_budget = (uint64_t) svc->cur_budget;
> I don't see the need for these cast (and similar ones further down),
> but you're the maintainer, so you know ...
> 
BTW, I saw you've committed this. Thanks for that, and sorry for not
replying. I wanted to, but I've had a couple of day full of Internet
connectivity issues.

I do think it's best to keep the casts, although strictly unnecessary.

About this series, I'm re-sending patch 1, by replying directly to that
same email of this thread.

So, for other maintainers (which I'm  Cc-ing to this message), or
should I say, for George :-), this is what should be considered for
review:
 - [PATCH 1/3 v2] xenalyze: handle DOM0 operations events
 - [PATCH 3/3] xenalyze: handle RTDS scheduler events

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®.