RFC: Trivial version check change in signal.send.part3 in signal.stp

[Phil Muldoon]

Frank Ch. Eigler wrote:
> Phil Muldoon <pmuldoon@redhat.com> writes:
>
>   
>> Corresponding bug: http://sourceware.org/bugzilla/show_bug.cgi?id=6752
>> [...]
>>     
>
>   
>>  probe _signal.send.part3 = kernel.function("send_sigqueue")
>>  {
>>      name = "send_sigqueue"
>>     
>
> (This could just be probefunc().)
>
>   
>> -%( kernel_v > "2.6.25" %?
>> +%( kernel_v > "2.6.25.10" %?
>>      task = $t
>>      sig = $q->info->si_signo
>>  %:
>>     
>
> The parameter (task_struct *p -> *t) rename occurred in commit
> ac6c215.  It was after the 2.6.25 release, and included in 2.6.26.
> Fedora's new naming convention is confusing the kernel_v test.
> The test could be changed to  ... >=  "2.6.26" ..., or indeed
> the translator's "kernel_v" expansion could be truncated to A.B.C.
>
> I'd prefer the former change for now, although if there are many other
> problematic cases with ">", and if non-fedora people also number their
> kernels A.B.C.D, then the latter may make more sense.

Cool, all good. I thought this was a Fedora specific issue, but I do 
indeed take your point well. As it happen. It turns out that 
proc_snoop.stp still does not work after this change (my own script does 
which prompted the change). Similar issues as before, the latest around 
$sig variable. Anyway, so this points to other problems or missing 
conditionals in signal.stp. I'll investigate these. Would it preferable 
to assimilate all of these in one big patch/pr report? Or treat them 
individually?

Regards

Phil