All of lore.kernel.org
 help / color / mirror / Atom feed
From: xielinfei <xielinfei@allwinnertech.com>
To: Philippe Gerum <rpm@xenomai.org>, xenomai@xenomai.org
Subject: Re: [Xenomai] [可能是垃圾邮件]  Re:  [help] porting I-pipe patch on linux-3.4 ARM(cortex-a7) soc
Date: Wed, 24 Aug 2016 18:33:57 +0800	[thread overview]
Message-ID: <57BD7815.4090007@allwinnertech.com> (raw)
In-Reply-To: <3abb8d06-cf67-d90f-de3f-6e3d935d93d3@xenomai.org>

Hi Philippe,

       Thank for your suggestion!
       I downgraded ipipe-3.10.32 to linux-3.4.39 and it works,
       However, I get some negative value, is this normal?
--------------------------------------------------------------------------------------------
/usr/xenomai/bin # ./k /usr/xenomai/bin # ./klatency
== Sampling period: 100 us
== Test mode: in-kernel periodic task
== All results in microseconds
warming up...
RTT|  00:00:00  (in-kernel periodic task, 100 us period, priority 99)
RTH|-----lat min|-----lat avg|-----lat max|-overrun|----lat best|---lat
worst
RTD|      -8.459|      -8.290|      -2.918|       0| -8.459|      -2.918
RTD|      -8.459|      -8.297|      -4.835|       0| -8.459|      -2.918
RTD|      -8.418|      -8.247|      -0.335|       0| -8.459|      -0.335
RTD|      -8.460|      -8.294|      -4.460|       0| -8.460|      -0.335
RTD|      -8.460|      -8.198|      -2.002|       0| -8.460|      -0.335
-----------------------------------------------------------------------------------------------
/usr/xenomai/bin # ./l /usr/xenomai/bin # ./latency ^[[J
== Sampling period: 1000 us
== Test mode: periodic user-mode task
== All results in microseconds
warming up...
RTT|  00:00:01  (periodic user-mode task, 1000 us period, priority 99)
RTH|----lat min|----lat avg|----lat max|-overrun|---msw|---lat
best|--lat worst
RTD|     -3.417|     -3.042|      3.041|       0|     0| -3.417|      3.041
RTD|     -3.917|     -3.292|     19.624|       0|     0| -3.917|     19.624
RTD|     -3.917|     -3.292|     18.541|       0|     0| -3.917|     19.624
RTD|     -3.917|     -3.292|     17.541|       0|     0| -3.917|     19.624
RTD|     -3.917|     -3.334|     12.124|       0|     0| -3.917|     19.624
--------------------------------------------------------------------------------------------------

On 2016年08月22日 02:27, Philippe Gerum wrote:
> On 08/19/2016 02:23 PM, xielinfei wrote:
>> Hi Philippe,
>>
>>
>> test-3.4.6-patch: it is the official I-pipe patch
>> ipipe-core-3.4.6-arm-4.patch
>> <http://xenomai.org/downloads/ipipe/v3.x/arm/older/ipipe-core-3.4.6-arm-4.patch>
>> test-3.4.v3s-patch: It is the vendor sdk with the modified I-pipe patch
>> (mainly based on ipipe-core-3.4.6-arm-4.patch
>> <http://xenomai.org/downloads/ipipe/v3.x/arm/older/ipipe-core-3.4.6-arm-4.patch>)
>>
>> The mainly differ as follow:
>> --------------------------------------------------------------------------------------------------------------------------------------
>> diff -r test-3.4.6-patch/arch/arm/kernel/entry-armv.S
>> test-3.4.v3s-patch/arch/arm/kernel/entry-armv.S
>> 269a270,282
>>> __und_fault:
>>>      @ Correct the PC such that it is pointing at the instruction
>>>      @ which caused the fault.  If the faulting instruction was ARM
>>>      @ the PC will be pointing at the next instruction, and have to
>>>      @ subtract 4.  Otherwise, it is Thumb, and the PC will be
>>>      @ pointing at the second half of the Thumb instruction.  We
>>>      @ have to subtract 2.
>>>      ldr    r2, [r0, #S_PC]
>>>      sub    r2, r2, r1
>>>      str    r2, [r0, #S_PC]
>>>      b    do_undefinstr
>>> ENDPROC(__und_fault)
>>>
>> 295c308
>> < #ifndef    CONFIG_THUMB2_KERNEL
>
> Please always send unified diffs (diff -u) to express changes in source
> code, this is easier to analyze than the antiquated original diff format.
>
> I don't see any hunks fixing up the architected timer to make it
> pipeline-aware. Those changes should take place in
> drivers/clocksource/arm_arch_timer.c. Some of them also allow a userland
> code to read the MMIO registers taping into the clock source, this will
> be required for running Xenomai apps.
>
> On a general note, I would not try to upgrade ipipe-3.4.6 for supporting
> the architected timer, I would rather downgrade ipipe-3.18 to match your
> 3.4.6 kernel. This way, the port would include all the post-3.4 fixes,
> and the support for the arch timer as well.
>
> If you chose to do so, you only need to port the changes affecting the
> devices provided by your SoC, ignoring any hunk which belong to other
> ARM SoC/variants (e.g. timers, clock sources, irq chip controllers, gpio
> modules etc).
>
> You could run splitdiff -a on the ipipe-3.18 patch, cherry-picking all
> the needed changes.
>

--

Best regards!
—————————————————————
谢林菲
珠海全志科技股份有限公司     BU1-PSW
地址:广东省珠海市高新区唐家湾镇科技2路9号
邮编:519000
TEL:15018321890
公司网址: http://www.allwinnertech.com

NOTICE: This e-mail and any included attachments are intended only for the sole use of named and intended recipient (s) only. If you are the named and intended recipient, please note that the information contained in this email and its embedded files are confidential and privileged. If you are neither the intended nor named recipient, you are hereby notified that any unauthorized review, use, disclosure, dissemination, distribution, or copying of this communication, or any of its contents, is strictly prohibited. Please reply to the sender and destroy the original message and all your records of this message (whether electronic or otherwise). Furthermore, you should not disclose to any other person, use, copy or disseminate the contents of this e-mail and/or the documents accompanying it.


  reply	other threads:[~2016-08-24 10:33 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-17  8:13 [Xenomai] [help] porting I-pipe patch on linux-3.4 ARM(cortex-a7) soc xielinfei
2016-08-17  9:13 ` xielinfei
2016-08-17 17:08   ` Lennart Sorensen
2016-08-18  1:11     ` xielinfei
2016-08-18 19:50 ` Philippe Gerum
2016-08-18 20:01   ` Philippe Gerum
2016-08-19  3:31     ` xielinfei
2016-08-19 12:23       ` xielinfei
2016-08-21 18:27         ` Philippe Gerum
2016-08-24 10:33           ` xielinfei [this message]
2016-08-24 10:50             ` [Xenomai] [可能是垃圾邮件] " Philippe Gerum

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=57BD7815.4090007@allwinnertech.com \
    --to=xielinfei@allwinnertech.com \
    --cc=rpm@xenomai.org \
    --cc=xenomai@xenomai.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.