Hi, I'm trying to submit the patch but without much success... mvarlese@linux-yk3w:~/repos> git clone https://git.linaro.org/lng/odp.git
mvarlese@linux-yk3w:~/repos/odp> git config --global user.name "Marco Varlese" mvarlese@linux-yk3w:~/repos/odp> git config --global user.email "marco.varlese@s use.com" mvarlese@linux-yk3w:~/repos/odp> git checkout -b fix_cpuset_size_assert mvarlese@linux-yk3w:~/repos/odp> git add platform/linux- generic/include/odp/api/plat/cpumask_types.h mvarlese@linux-yk3w:~/repos/odp> git commit -s [fix_cpuset_size_assert 2728409] As per mailing-list and meeting conversation: this patch addresses the build failure on operating systems where CPU_SETSIZE is bigger than 1024 entries. 1 file changed, 1 insertion(+), 1 deletion(-) mvarlese@linux-yk3w:~/repos/odp> mvarlese@linux-yk3w:~/repos/odp> git push --set-upstream origin fix_cpuset_size_assert fatal: unable to access 'https://git.linaro.org/lng/odp.git/': The requested URL returned error: 403 Are you positive I don't need any special step to be done (e.g. register account, SSH key uploaded, etc.)? Thanks, Marco On Tue, 2017-02-28 at 06:38 -0600, Bill Fischofer wrote: > > > > I've added this topic to the agenda for today's ODP public call. Marco, > > > > if you're able to join us to provide your input to the discussion please do so. The call is at 15:00 UTC. To join just go to http://meetings.opendataplane.org . We use the BlueJeans conferencing system. Thanks. > > > On Tue, Feb 28, 2017 at 3:02 AM, Maxim Uvarov <maxim.uva...@linaro.org> > > wrote: > > the problem is that if you compile binary on let's say ubuntu it might not > > work on suse with more then 1024 cpus. > > > > > > > > > > On 28 February 2017 at 11:52, Marco Varlese <marco.varl...@suse.com> > > > > wrote: > > > > > > > > > On Tue, 2017-02-28 at 08:12 +0000, Savolainen, Petri (Nokia - FI/Espoo) > > > > > wrote: > > > > > > > > > > [nip] > > > > > > > > > > > /** > > > > > > * @def ODP_CPUMASK_SIZE > > > > > > * Maximum cpumask size, this definition limits the number of individual > > > > > CPUs > > > > > > * that can be accessed in this system. > > > > > > */ > > > > > > > > > > > > It cannot be removed, it's part of API. Just need to find a way to set > > > > > it to > > > > > > CPU_SETSIZE (on Linux systems). ODP API spec is OS agnostic. The same > > > > > > application code should run also on non-Linux implementations (e.g. ODP > > > > > API > > > > > > spec cannot refer to CPU_SETSIZE). The assert is needed as long as > > > > > > ODP_CPUMASK_SIZE value is not set from CPU_SETSIZE. > > > > > > > > > > > > -Petri > > > > > > > > > > > I thought it could not be removed as well. > > > > > What about something like this... > > > > > > > > > > ==== > > > > > #ifdef __unix__ > > > > > > > > > > #define _GNU_SOURCE > > > > > #include <sched.h> > > > > > #define ODP_CPUMASK_SIZE CPU_SETSIZE > > > > > > > > > > #elif > > > > > > > > > > #define ODP_CPUMASK_SIZE 1024 > > > > > > > > > > #endif > > > > > > > > > > #define ODP_CPUMASK_STR_SIZE ((ODP_CPUMASK_SIZE + 3) / 4 + 3) > > > > > ==== > > > > > > > > > > However, something is going on with the ODP build system since it cannot > > > > > find > > > > > CPU_SETSIZE (/usr/include/sched.h)... > > > > > > > > > > > > > > > Cheers, > > > > > Marco > > > > > > > > > > > > > > > > > > > >
