FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Redhat > Crash Utility

 
 
LinkBack Thread Tools
 
Old 05-23-2012, 10:14 AM
"WeiJiangang"
 
Default add -t option for runq command

Hi Dave,

I agree with your opinion,the first suggestion (1) seems better.I have already accepted
your specification and modified the patch. Specific modification as follows:

(1) The option “-t” is accepted, while the option "-c" was abandoned.
(2) Turn key word '*_clock' into '*_timestamp'.
(3) Add information about active task on each cpu,like this:
CPU 0: 67393251944103
[67393251944103] PID: 14776 TASK: ea54aab0 COMMAND: "crash"
CPU 1: 66789719628339
[67393251858710] PID: 14769 TASK: f4bdc030 COMMAND: "crash"

Do you have any other suggestion or requirement?
--
Crash-utility mailing list
Crash-utility@redhat.com
https://www.redhat.com/mailman/listinfo/crash-utility
 
Old 05-23-2012, 10:21 AM
"WeiJiangang"
 
Default add -t option for runq command

Hi Dave,

I agree with your opinion,the first suggestion (1) seems better.I have already accepted
your specification and modified the patch. Specific modification as follows:

(1) The option “-t” is accepted, while the option "-c" was abandoned.
(2) Turn key word '*_clock' into '*_timestamp'.
(3) Add information about active task on each cpu,like this:
CPU 0: 67393251944103
[67393251944103] PID: 14776 TASK: ea54aab0 COMMAND: "crash"
CPU 1: 66789719628339
[67393251858710] PID: 14769 TASK: f4bdc030 COMMAND: "crash"

Do you have any other suggestion or requirement?

Regards,
Wei Jiangang
--
Crash-utility mailing list
Crash-utility@redhat.com
https://www.redhat.com/mailman/listinfo/crash-utility
 
Old 05-23-2012, 07:56 PM
Dave Anderson
 
Default add -t option for runq command

----- Original Message -----
> Hi Dave,
>
> I agree with your opinion,the first suggestion (1) seems better.I
> have already accepted
> your specification and modified the patch. Specific modification as
> follows:
>
> (1) The option “-t” is accepted, while the option "-c" was
> abandoned.
> (2) Turn key word '*_clock' into '*_timestamp'.
> (3) Add information about active task on each cpu,like this:
> CPU 0: 67393251944103
> [67393251944103] PID: 14776 TASK: ea54aab0 COMMAND: "crash"
> CPU 1: 66789719628339
> [67393251858710] PID: 14769 TASK: f4bdc030 COMMAND: "crash"
>
> Do you have any other suggestion or requirement?

No, that's fine.

I did make a slight change to the output display so that the task's
timestamp value is directly under the cpu's value and without the
brackets, making a comparison much easier. I also re-worded the help
page description, and added the display of the new rq_timestamp offset
value in dump_offset_table().

The patch is queued for crash-6.0.7.

Thanks,
Dave

--
Crash-utility mailing list
Crash-utility@redhat.com
https://www.redhat.com/mailman/listinfo/crash-utility
 

Thread Tools




All times are GMT. The time now is 12:09 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org