[Xenomai] potentially missing timer rating check in select_root_only_timer()
jan.kiszka at siemens.com
Wed May 2 07:34:03 CEST 2018
On 2018-05-02 03:13, Pham, Phong wrote:
> I am suspecting that in kernel/ipipe/timer.c:select_root_only_timer()
> When t->irq == per_cpu(ipipe_percpu.hrtimer_irq, icpu), a condition to test the timer rating against current timer rating before installing the timer is missing. That is t->rating > percpu_timer->rating (combined with same irq #) before the condition becomes true.
Conceptually, I-pipe timers should have highest rating. Can you explain
in more details why it should be there?
Also note that your own code looks suspicious: percpu_timer is a per-cpu
variable but is not accesses as such.
Siemens AG, Corporate Technology, CT RDA IOT SES-DE
Corporate Competence Center Embedded Linux
More information about the Xenomai