-----Original Message-----
From: Sebastian Huber <sebastian.hu...@embedded-brains.de> 
Sent: Friday, December 11, 2020 13:00
To: Kinsey Moore <kinsey.mo...@oarcorp.com>; devel@rtems.org
Subject: Re: [PATCH 1/2] tm27: Use generic cpu index accessor

> On 11/12/2020 16:09, Kinsey Moore wrote:
>
>> Unfortunately, that's the only API currently available to get that 
>> information easily on both architectures. The base function that is wrapped 
>> (_CPU_SMP_Get_current_processor) could be provided outside the RTEMS_SMP 
>> ifdef block and that could be used instead since this is specific to 
>> ARM/AArch64. Would that be acceptable?
>
> If _CPU_SMP_Get_current_processor() returns != 0 if 
> CONFIGURE_MAXIMUM_PROCESSORS == 1, then this is undefined behaviour, 
> since this value is used to index _Per_CPU_Information.
>
> I think for now, the patch is fine.

I'll commit this patch as is and the other with the alternate cast.

Kinsey
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

Reply via email to