lkml.org 
[lkml]   [2002]   [Feb]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: Improved ksymoops output
Hi,

[reduced cc list, since I don't think the rest of the guys are
interested in non-kernel related things]

> As already pointed out, format is horrendous, but prettier format
> requires much more serious hacking in ksymoops sources instead of my
> quick and very dirty tricks.

Yep, I see. After rethinking about this I have to agree with Russell.
And I mean you still can't educate people to read FAQ's and manuals.
Even with your output people that have no knowledge about this output
will the things you mentioned as rationale. If one sees your inital oops
where the hell does he get the information about calling your scripts
without reading some document you must be providing. Well, and if he
needs to read your docu he can as well read oops-tracing.txt. This is my
little world, YMMV.

>>o run faster (5%) ;)
>>o should never have problems when one day there will be a lot of *.c
>> files. In your approach LIST could someday not hold all entries
>> anymore.
>>
>
>>o simplifies the bash 'regexp' to snip away the '.c' and print the rest
>>
>
> Hmm... is it faster than original?

I would say so, "b=${a%.*}" is always faster and legible than
"l=$((${#a}-2)); b=${a:0:$l}"
in the way it is used in your script.
Maybe not easily measurable but it seems obvious to me and is easier to
read and overall time reported less time used for the whole script to
run. ;)

And the second tiny cleanup regarding regexp was to use "${a:2}" instead
of "${a:2:9999}". It should be an improvement too, if I remember the
bash source correctly. But this is senseless nitpicking. The real speed
improvement you get by avoiding the "LIST=`find -name '*.c' | xargs`".
>>I'm propably going to rewrite the python script in bash too, since I
>>don't run python on my distro (and I do not intend to use 2.5.x anytime
>>soon).
>>
>
> Care to show the result to me?

Well, if there is a real interest from other parties, I will definitely
spend some time to do it. OTOH, bash's regexp handling isn't as strong
as python's AFAICS so it might be a pain in the ass to do it but this is
getting off-topic.

Best regards,
Roberto Nibali, ratz

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:24    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site