Increase maximum number of fanout CPUs from 128 to 256
There is a define which limits the maximum number of fanout CPUs, it is:
#define MRP_NCPUS 128
I think we should change it to
#define MRP_NCPUS NCPU
Updated by Paul Winder almost 3 years ago
Using NCPU won't work - header is included in user land.
We have never hit the limit of 128 fanout CPUs yet anyway, I'll increase to 256 and add an appropriate guard to prevent overflowing any arrays.
It is related to the efforts in #13218
Updated by Electric Monk almost 3 years ago
- Status changed from In Progress to Closed
- % Done changed from 0 to 100
commit 3714f7be8e09c39a0ea7ce7ef44cb495ce250913 Author: Paul Winder <firstname.lastname@example.org> Date: 2020-12-16T14:23:43.000Z 13218 "Stack smashing detected" panic when creating vnic over aggr with 4 mlxcx links 13222 Increase maximum number of fanout CPUs from 128 to 256 Reviewed by: Robert Mustacchi <email@example.com> Reviewed by: Garrett D'Amore <firstname.lastname@example.org> Approved by: Dan McDonald <email@example.com>