Re: [Xenomai] Failed testsuite/latency

[Philippe Gerum <rpm@xenomai.org>]

On 03/03/2015 04:29 PM, Philippe Gerum wrote:
> On 03/02/2015 03:56 PM, Philippe Gerum wrote:
>> On 02/25/2015 09:33 AM, Bart Vissers wrote:
>>> Hi Philippe,
>>>
>>> Before making any changes, I ran xeno-config --info:
>>> Xenomai version: Xenomai/cobalt v3.0-rc3 --
>>> Linux rtpc 3.16.7-xenomai-3.0-rc3 #1 SMP Mon Feb 23 12:30:46 CET 2015
>>> i686 GNU/Linux
>>> Kernel parameters: BOOT_IMAGE=/boot/vmlinuz-3.16.7-xenomai-3.0-rc3
>>> root=UUID=0925426e-e058-4093-963c-8b7de072e229 ro quiet
>>> I-pipe release #2 detected
>>> Cobalt core 3.0-rc3 detected
>>> Build args: --prefix=/usr --includedir=/usr/include/xenomai
>>> --mandir=/usr/share/man --with-testdir=/usr/lib/xenomai/testsuite
>>> --build i486-linux-gnu build_alias=i486-linux-gnu
>>>
>>> namelist did not work, as latency does not contain any symbols. How
>>> can I include these when building debian packages?
>>>
>>> Unfortunately, I got compilation errors after applying the patch to
>>> xenomai-3.0-rc3: error: ‘XENO_WARNING’ undeclared (first use in this
>>> function)
>>> So I decided to repeat my steps with the git next branch, 52fb91e...
>>> So, same linux kernel, i-pipe patch and kernel config. Also included
>>> the printk patch. Output:
>>>
>>> == Sampling period: 100 us
>>> == Test mode: periodic user-mode task
>>> == All results in microseconds
>>> warming up...
>>> latency: failed to set periodic, code 9
>>> ^C---|-----------|-----------|-----------|--------|------|-------------------------
>>> RTS|  10000.000|      0.000| -10000.000|       0|     0|    00:03:26/00:03:26
>>>
>>> Notice the ^C. I killed it because it was running indefinitely. This
>>> was also the case in my previous attempts.
>>>
>>>
>>
>> Can any of you send me the generated debian package privately? I would
>> also need to know the gcc version used.
>>
> 
> Never mind, I could reproduce this bug eventually. This issue is due to
> some register trashing at work in the pipeline code, not to the Xenomai
> core. More later.
>

This patch will fix this issue.

diff --git a/arch/x86/kernel/entry_32.S b/arch/x86/kernel/entry_32.S
index 6e6b88a..e834437 100644
--- a/arch/x86/kernel/entry_32.S
+++ b/arch/x86/kernel/entry_32.S
@@ -517,6 +517,7 @@ sysenter_exit_fast:
 	jmp sysenter_exit
 sysenter_exit_root:
 	call __ipipe_root_sync
+	movl PT_EAX(%esp),%eax
 	jmp sysenter_exit_check
 #endif

-- 
Philippe.