[PATCH 2/2] nfs: Add RPCd task affinity config option
Daniel Cederman
cederman at gaisler.com
Thu Nov 13 10:17:50 UTC 2014
On 2014-11-13 10:23, Sebastian Huber wrote:
> On 13/11/14 10:09, Daniel Cederman wrote:
>> @@ -407,6 +410,8 @@ static rtems_interval ticksPerSec; /*
>> cached system clock rate (WHO IS ASSUMED
>> */
>>
>> rtems_task_priority rpciodPriority = 0;
>> +cpu_set_t *rpciodCpuset = 0;
>> +size_t rpciodCpusetSize = 0;
>>
>> #if (DEBUG) & DEBUG_MALLOC
>> /* malloc wrappers for debugging */
>
> Missing #ifdef RTEMS_SMP ?
>
> I am not sure if we should really use #ifdef RTEMS_SMP for the #include
> <sys/cpuset.h>. We use also #if defined(__RTEMS_HAVE_SYS_CPUSET_H__) &&
> defined(RTEMS_SMP) sometimes. I would simply force users to update
> their Newlib version.
>
> Please fix the spacing and use the coding style of the file.
>
Thank you for you comments. I removed the #ifdef RTEMS_SMP around
<sys/cpuset.h> and have hopefully fixed the spacing issues. I will
submit new versions of the patches.
--
Daniel Cederman
Software Engineer
Aeroflex Gaisler AB
Aeroflex Microelectronic Solutions – HiRel
Kungsgatan 12
SE-411 19 Gothenburg, Sweden
Phone: +46 31 7758665
cederman at gaisler.com
www.Aeroflex.com/Gaisler
More information about the devel
mailing list