Clean up noarch ipipe queue

Jan Kiszka jan.kiszka at siemens.com
Thu Feb 28 09:50:44 CET 2019


On 27.02.19 09:13, Philippe Gerum wrote:
> On 2/27/19 7:21 AM, Jan Kiszka wrote:
>> Hi Philippe,
>>
>> I started to revert the patches in the x86 tree that are touching
>> arch/arm. One fragment may be reverted for everyone:
>>> --- a/arch/arm/include/asm/arch_timer.h
>> +++ b/arch/arm/include/asm/arch_timer.h
>> @@ -106,10 +106,6 @@ static inline u32 arch_timer_get_cntkctl(void)
>>
>>   static inline void arch_timer_set_cntkctl(u32 cntkctl)
>>   {
>> -#ifdef CONFIG_IPIPE
>> -    /* Enable access to user-space (may not be needed) */
>> -    cntkctl |= ARCH_TIMER_USR_PCT_ACCESS_EN;
>> -#endif
>>       asm volatile("mcr p15, 0, %0, c14, c1, 0" : : "r" (cntkctl));
>>   }
>>
>>
>> Or are we exploiting user space access?
> 
> We are, for reading the timer counter directly via a memory mapping on
> arm and arm64. This is a prerequisite for the ipipe_tsc abstraction on
> these archs.

OK, then I will sent a revert of this hunk for noarch and a patch re-adding this 
to the ARM tree with this explanation.

Jan

-- 
Siemens AG, Corporate Technology, CT RDA IOT SES-DE
Corporate Competence Center Embedded Linux



More information about the Xenomai mailing list