Project

General

Profile

Actions

Bug #13208

closed

Create aggr fails when underlying links have more than 128 Tx rings

Added by Paul Winder over 1 year ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
networking
Start date:
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:

Description

Using:

dladm create-aggr -l mlxcx0 -l mlxcx1 -l mlxcx2 aggr0

Currently panics the system (see #13207). Needless, we should be able to create an aggr with more than 2 mlxcx links. It is cause because the default Tx rings per group in mlxcx is 64, so 2 mlxcx hits the global per group limit of 128 - MAX_RINGS_PER_GROUP.

The default is set by this constant:

#define MLXCX_TX_NRINGS_PER_GROUP_DFLT  64

This seems far too high, I propose the default be set to 16 - the same as number of rings per Rx group.

It can be overridden is mlxcx.conf using tx_nrings_per_group driver property.


Related issues

Related to illumos gate - Bug #13207: Creating an aggr with more than 128 Tx or Rx rings panicsClosedPaul Winder

Actions
Actions

Also available in: Atom PDF