Re: [Crash-utility] patch for slight modification to runq -g command
----- Original Message -----
Hi Dave,
The rb_root pointer is pretty useless in our debugging. I thought about
removing it but decided not to do it thinking other people may need it.
I can get you another patch that removes the rb_root display from the
"group" line. Please let me know what you think.
OK good, that's what I figured. And since the cfs_rq address is being
displayed now, if it's really necessary to see the the rb_root, it can
easily be accessed from there.
No need for another patch -- I've been tinkering with the results of
your changes.
Thanks again,
Dave
Thanks,
Anthony
> -----Original Message-----
> From: crash-utility-bounces(a)redhat.com [mailto:crash-utility-
> bounces(a)redhat.com] On Behalf Of Dave Anderson
> Sent: Wednesday, November 13, 2013 1:51 PM
> To: Discussion list for crash utility usage, maintenance and
> development
> Subject: Re: [Crash-utility] patch for slight modification to runq -g
> command
>
>
>
> ----- Original Message -----
> >
> >
> > ----- Original Message -----
> > > My mistake, I didn't read your email carefully enough. Here's
the
> > > new
> > > patch.
> > >
> > > Thanks,
> > > Anthony
> >
> > Thanks Anthony -- your changes are queued for crash-7.0.4.
> >
> > Dave
>
> Anthony,
>
> Given your interest in this command, I have a question re: the display.
>
> Do you ever have any interest in the rb_root addresses in each group?
> I'm thinking that with your patch adding the task_group and cfs_rq
> addresses, that the rb_root address is fairly useless. It was
> pretty much carried forward from the base "runq" command display,
> where it makes some sense.
>
> Dave
>
>
> --
> Crash-utility mailing list
> Crash-utility(a)redhat.com
> https://www.redhat.com/mailman/listinfo/crash-utility