[Xenomai-core] Re: [patches] 1-bundle-email-spelling-help 2-rfc-add-newer-bench-tests

[Jan Kiszka <jan.kiszka@domain.hid>]

[-- Attachment #1: Type: text/plain, Size: 2589 bytes --]

Jim Cromie wrote:
> Jan Kiszka wrote:
>> Jim Cromie wrote:
>>  
>>> im trying to inline this patch,
>>> pls let me know if its still ws fouled
>>> (in thunderbird, cp from svn diff |less, preformat b4 paste)
>>>
>>>
>>>
>>>
>>> Index: CREDITS
>>> ===================================================================
>>> --- CREDITS     (revision 1412)
>>> +++ CREDITS     (working copy)
>>> @@ -43,7 +43,7 @@
>>> D: the map.
>>>
>>> N: Jim Cromie
>>> -E: jcromie@domain.hid
>>> +E: jim.cromie@domain.hid
>>> D: Comprehensive statistics collection for the testsuite.
>>> D: Validation test script. Various script fixes and sanitization.
>>>     
>>
>> Obviously damaged, at least here on the list (leading single whitespace
>> missing in unmodified lines). This also applies to your second patch.
>>
>> The required steps with Thunderbird are: 1. start a new mail or reply as
>> HTML (bah!), 2. set the text style of everything to preformat, 3. switch
>> back to "Plain Text Only" under Options/Format. Hope somebody will once
>> hack a plugin for disabling line wrapping on demand without this dance...
>>
>>   
> <marge-simpson-voice>  HRMM
> maybe copy-src is the factor - Ive had luck my way (which yours sounds
> like)
> when cpying the diff from an emacs window.
> 
> No Matter  ( machts nichts, or after americanization,  MoxNix  )
> attachments are easier anyway, and RPM seems not to care either way.
> 
> So, 2 patches:
> 
> 1- bundle: email, Kconfig spelling, switchbench print in us, not ns.
> issues :
> - switchbench segfaults(iirc) for me - both before and after patch,
> so its not tested - and printf format purposely not tweaked, so as
> to give someone else the itch ;-)
> 

Would you mind to attach a debugger and trace this a bit down? I do not
see this here. Or are there any special switches to apply?

> 
> 2 -  add newer benchmark tests to xeno-test  RFC
> - irqbench (all 4 ways)

See my other mail. This requires a second box and some explanation for
the poor newbie who's just looking for a self-contained test run.

> - switchbench - segfaults
> - switchtest
> by the way, could we get renames ?    switch{,bench,test}/switch.c
>    is just unnecessarily confusing !
> 
> issues :
> - are tests ready to add ?  perhaps, if optional ?
> - config dependence ignored
>    we have no guarantee of .config availability (or do we?)
> - XENOT_*  for tool(*) specific option setting from env
>        warrants better prefix, but XENOTEST_TOPTS_* seemed too much
> 
> thanks
> -jimc
> 

Jan


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 250 bytes --]