diff mbox

opensm/osm_torus.c: Increase PORTGRP_MAX_PORTS define

Message ID 1418808490-27687-1-git-send-email-alexne@mellanox.com (mailing list archive)
State Not Applicable, archived
Headers show

Commit Message

Alex Netes Dec. 17, 2014, 9:28 a.m. UTC
Torus-2QoS fails to configure fabric with more than 16 HCAs.
This patch extends PORTGRP_MAX_PORTS to be 34. Maximal practical ports
per switch - 2 reserved ports (minimal of needed ports for switch to
switch connections in 2D Torus).

Signed-off-by: Alex Netes <alexne@mellanox.com>
---
 opensm/osm_torus.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Comments

Hal Rosenstock Dec. 17, 2014, 10:46 a.m. UTC | #1
On 12/17/2014 4:28 AM, Alex Netes wrote:
> Torus-2QoS fails to configure fabric with more than 16 HCAs.
> This patch extends PORTGRP_MAX_PORTS to be 34. Maximal practical ports
> per switch - 2 reserved ports (minimal of needed ports for switch to
> switch connections in 2D Torus).
> 
> Signed-off-by: Alex Netes <alexne@mellanox.com>
> ---
>  opensm/osm_torus.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/opensm/osm_torus.c b/opensm/osm_torus.c
> index 330f55d..b01f69f 100644
> --- a/opensm/osm_torus.c
> +++ b/opensm/osm_torus.c
> @@ -59,7 +59,7 @@
>  #include <opensm/osm_opensm.h>
>  
>  #define TORUS_MAX_DIM        3
> -#define PORTGRP_MAX_PORTS    16
> +#define PORTGRP_MAX_PORTS    34
>  #define SWITCH_MAX_PORTGRPS  (1 + 2 * TORUS_MAX_DIM)
>  #define DEFAULT_MAX_CHANGES  32
>  

There is portgroup_max_ports configuration parameter for torus to change this if needed.

man torus-2QoS.conf states:
       portgroup_max_ports max_ports
              This keyword specifies the maximum  number  of  parallel  inter-
              switch  links,  and  also  the  maximum number of host ports per
              switch, that torus-2QoS can accommodate.  The default  value  is
              16.   Torus-2QoS  will log an error message during topology dis-
              covery if this parameter needs to be increased.  If this keyword
              appears multiple times, the last instance prevails.

              Note  that  the  switch management port (switch port 0) gets put
              into the same port group with the host ports, so if you have  16
              host  ports  per switch, portgroup_max_ports would need to be at
              least 17.

Is there really a need to change the default for this ?

-- Hal
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Alex Netes Dec. 17, 2014, 10:59 a.m. UTC | #2
> > Torus-2QoS fails to configure fabric with more than 16 HCAs.
> > This patch extends PORTGRP_MAX_PORTS to be 34. Maximal practical
> ports
> > per switch - 2 reserved ports (minimal of needed ports for switch to
> > switch connections in 2D Torus).
> >
> > Signed-off-by: Alex Netes <alexne@mellanox.com>
> > ---
> >  opensm/osm_torus.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/opensm/osm_torus.c b/opensm/osm_torus.c index
> > 330f55d..b01f69f 100644
> > --- a/opensm/osm_torus.c
> > +++ b/opensm/osm_torus.c
> > @@ -59,7 +59,7 @@
> >  #include <opensm/osm_opensm.h>
> >
> >  #define TORUS_MAX_DIM        3
> > -#define PORTGRP_MAX_PORTS    16
> > +#define PORTGRP_MAX_PORTS    34
> >  #define SWITCH_MAX_PORTGRPS  (1 + 2 * TORUS_MAX_DIM)  #define
> > DEFAULT_MAX_CHANGES  32
> >
> 
> There is portgroup_max_ports configuration parameter for torus to change
> this if needed.
> 
> man torus-2QoS.conf states:
>        portgroup_max_ports max_ports
>               This keyword specifies the maximum  number  of  parallel  inter-
>               switch  links,  and  also  the  maximum number of host ports per
>               switch, that torus-2QoS can accommodate.  The default  value  is
>               16.   Torus-2QoS  will log an error message during topology dis-
>               covery if this parameter needs to be increased.  If this keyword
>               appears multiple times, the last instance prevails.
> 
>               Note  that  the  switch management port (switch port 0) gets put
>               into the same port group with the host ports, so if you have  16
>               host  ports  per switch, portgroup_max_ports would need to be at
>               least 17.
> 
> Is there really a need to change the default for this ?

No. Missed that. Please ignore.

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/opensm/osm_torus.c b/opensm/osm_torus.c
index 330f55d..b01f69f 100644
--- a/opensm/osm_torus.c
+++ b/opensm/osm_torus.c
@@ -59,7 +59,7 @@ 
 #include <opensm/osm_opensm.h>
 
 #define TORUS_MAX_DIM        3
-#define PORTGRP_MAX_PORTS    16
+#define PORTGRP_MAX_PORTS    34
 #define SWITCH_MAX_PORTGRPS  (1 + 2 * TORUS_MAX_DIM)
 #define DEFAULT_MAX_CHANGES  32