Commit c4a67a21 authored by Jakub Kicinski's avatar Jakub Kicinski

Revert "Merge branch 'mlxsw-line-card-model'"

This reverts commit 5e927a9f, reversing
changes made to cfc1d91a.

The discussion is still ongoing so let's remove the uAPI
until the discussion settles.

Link: https://lore.kernel.org/all/20220425090021.32e9a98f@kernel.org/Reviewed-by: default avatarIdo Schimmel <idosch@nvidia.com>
Link: https://lore.kernel.org/r/20220504154037.539442-1-kuba@kernel.orgSigned-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent c8227d56
...@@ -14,7 +14,6 @@ system. Following operations are provided: ...@@ -14,7 +14,6 @@ system. Following operations are provided:
* Get a list of supported line card types. * Get a list of supported line card types.
* Provision of a slot with specific line card type. * Provision of a slot with specific line card type.
* Get and monitor of line card state and its change. * Get and monitor of line card state and its change.
* Get information about line card versions and devices.
Line card according to the type may contain one or more gearboxes Line card according to the type may contain one or more gearboxes
to mux the lanes with certain speed to multiple ports with lanes to mux the lanes with certain speed to multiple ports with lanes
...@@ -121,6 +120,3 @@ Example usage ...@@ -121,6 +120,3 @@ Example usage
# Set slot 8 to be unprovisioned: # Set slot 8 to be unprovisioned:
$ devlink lc set pci/0000:01:00.0 lc 8 notype $ devlink lc set pci/0000:01:00.0 lc 8 notype
# Set info for slot 8:
$ devlink lc info pci/0000:01:00.0 lc 8
...@@ -58,39 +58,6 @@ The ``mlxsw`` driver reports the following versions ...@@ -58,39 +58,6 @@ The ``mlxsw`` driver reports the following versions
- running - running
- Three digit firmware version - Three digit firmware version
Line card info versions
=======================
The ``mlxsw`` driver reports the following versions for line cards
.. list-table:: devlink line card info versions implemented
:widths: 5 5 90
* - Name
- Type
- Description
* - ``hw.revision``
- fixed
- The hardware revision for this line card
* - ``ini.version``
- running
- Version of line card INI loaded
Line card device info versions
==============================
The ``mlxsw`` driver reports the following versions for line card devices
.. list-table:: devlink line card device info versions implemented
:widths: 5 5 90
* - Name
- Type
- Description
* - ``fw.version``
- running
- Three digit firmware version
Driver-specific Traps Driver-specific Traps
===================== =====================
......
...@@ -581,7 +581,6 @@ struct mlxsw_linecard { ...@@ -581,7 +581,6 @@ struct mlxsw_linecard {
active:1; active:1;
u16 hw_revision; u16 hw_revision;
u16 ini_version; u16 ini_version;
struct list_head device_list;
}; };
struct mlxsw_linecard_types_info; struct mlxsw_linecard_types_info;
......
...@@ -87,191 +87,11 @@ static const char *mlxsw_linecard_type_name(struct mlxsw_linecard *linecard) ...@@ -87,191 +87,11 @@ static const char *mlxsw_linecard_type_name(struct mlxsw_linecard *linecard)
return linecard->name; return linecard->name;
} }
struct mlxsw_linecard_device_info {
u16 fw_major;
u16 fw_minor;
u16 fw_sub_minor;
};
struct mlxsw_linecard_device {
struct list_head list;
u8 index;
struct mlxsw_linecard *linecard;
struct devlink_linecard_device *devlink_device;
struct mlxsw_linecard_device_info info;
};
static struct mlxsw_linecard_device *
mlxsw_linecard_device_lookup(struct mlxsw_linecard *linecard, u8 index)
{
struct mlxsw_linecard_device *device;
list_for_each_entry(device, &linecard->device_list, list)
if (device->index == index)
return device;
return NULL;
}
static int mlxsw_linecard_device_attach(struct mlxsw_core *mlxsw_core,
struct mlxsw_linecard *linecard,
u8 device_index, bool flash_owner)
{
struct mlxsw_linecard_device *device;
int err;
device = kzalloc(sizeof(*device), GFP_KERNEL);
if (!device)
return -ENOMEM;
device->index = device_index;
device->linecard = linecard;
device->devlink_device = devlink_linecard_device_create(linecard->devlink_linecard,
device_index, device);
if (IS_ERR(device->devlink_device)) {
err = PTR_ERR(device->devlink_device);
goto err_devlink_linecard_device_attach;
}
list_add_tail(&device->list, &linecard->device_list);
return 0;
err_devlink_linecard_device_attach:
kfree(device);
return err;
}
static void mlxsw_linecard_device_detach(struct mlxsw_core *mlxsw_core,
struct mlxsw_linecard *linecard,
struct mlxsw_linecard_device *device)
{
list_del(&device->list);
devlink_linecard_device_destroy(linecard->devlink_linecard,
device->devlink_device);
kfree(device);
}
static void mlxsw_linecard_devices_detach(struct mlxsw_linecard *linecard)
{
struct mlxsw_core *mlxsw_core = linecard->linecards->mlxsw_core;
struct mlxsw_linecard_device *device, *tmp;
list_for_each_entry_safe(device, tmp, &linecard->device_list, list)
mlxsw_linecard_device_detach(mlxsw_core, linecard, device);
}
static int mlxsw_linecard_devices_attach(struct mlxsw_linecard *linecard)
{
struct mlxsw_core *mlxsw_core = linecard->linecards->mlxsw_core;
u8 msg_seq = 0;
int err;
do {
char mddq_pl[MLXSW_REG_MDDQ_LEN];
bool flash_owner;
bool data_valid;
u8 device_index;
mlxsw_reg_mddq_device_info_pack(mddq_pl, linecard->slot_index,
msg_seq);
err = mlxsw_reg_query(mlxsw_core, MLXSW_REG(mddq), mddq_pl);
if (err)
return err;
mlxsw_reg_mddq_device_info_unpack(mddq_pl, &msg_seq,
&data_valid, &flash_owner,
&device_index, NULL,
NULL, NULL);
if (!data_valid)
break;
err = mlxsw_linecard_device_attach(mlxsw_core, linecard,
device_index, flash_owner);
if (err)
goto rollback;
} while (msg_seq);
return 0;
rollback:
mlxsw_linecard_devices_detach(linecard);
return err;
}
static void mlxsw_linecard_device_update(struct mlxsw_linecard *linecard,
u8 device_index,
struct mlxsw_linecard_device_info *info)
{
struct mlxsw_linecard_device *device;
device = mlxsw_linecard_device_lookup(linecard, device_index);
if (!device)
return;
device->info = *info;
}
static int mlxsw_linecard_devices_update(struct mlxsw_linecard *linecard)
{
struct mlxsw_core *mlxsw_core = linecard->linecards->mlxsw_core;
u8 msg_seq = 0;
do {
struct mlxsw_linecard_device_info info;
char mddq_pl[MLXSW_REG_MDDQ_LEN];
bool data_valid;
u8 device_index;
int err;
mlxsw_reg_mddq_device_info_pack(mddq_pl, linecard->slot_index,
msg_seq);
err = mlxsw_reg_query(mlxsw_core, MLXSW_REG(mddq), mddq_pl);
if (err)
return err;
mlxsw_reg_mddq_device_info_unpack(mddq_pl, &msg_seq,
&data_valid, NULL,
&device_index,
&info.fw_major,
&info.fw_minor,
&info.fw_sub_minor);
if (!data_valid)
break;
mlxsw_linecard_device_update(linecard, device_index, &info);
} while (msg_seq);
return 0;
}
static int
mlxsw_linecard_device_info_get(struct devlink_linecard_device *devlink_linecard_device,
void *priv, struct devlink_info_req *req,
struct netlink_ext_ack *extack)
{
struct mlxsw_linecard_device *device = priv;
struct mlxsw_linecard_device_info *info;
struct mlxsw_linecard *linecard;
char buf[32];
linecard = device->linecard;
mutex_lock(&linecard->lock);
if (!linecard->active) {
mutex_unlock(&linecard->lock);
return 0;
}
info = &device->info;
sprintf(buf, "%u.%u.%u", info->fw_major, info->fw_minor,
info->fw_sub_minor);
mutex_unlock(&linecard->lock);
return devlink_info_version_running_put(req,
DEVLINK_INFO_VERSION_GENERIC_FW,
buf);
}
static void mlxsw_linecard_provision_fail(struct mlxsw_linecard *linecard) static void mlxsw_linecard_provision_fail(struct mlxsw_linecard *linecard)
{ {
linecard->provisioned = false; linecard->provisioned = false;
linecard->ready = false; linecard->ready = false;
linecard->active = false; linecard->active = false;
mlxsw_linecard_devices_detach(linecard);
devlink_linecard_provision_fail(linecard->devlink_linecard); devlink_linecard_provision_fail(linecard->devlink_linecard);
} }
...@@ -412,7 +232,6 @@ mlxsw_linecard_provision_set(struct mlxsw_linecard *linecard, u8 card_type, ...@@ -412,7 +232,6 @@ mlxsw_linecard_provision_set(struct mlxsw_linecard *linecard, u8 card_type,
{ {
struct mlxsw_linecards *linecards = linecard->linecards; struct mlxsw_linecards *linecards = linecard->linecards;
const char *type; const char *type;
int err;
type = mlxsw_linecard_types_lookup(linecards, card_type); type = mlxsw_linecard_types_lookup(linecards, card_type);
mlxsw_linecard_status_event_done(linecard, mlxsw_linecard_status_event_done(linecard,
...@@ -430,11 +249,6 @@ mlxsw_linecard_provision_set(struct mlxsw_linecard *linecard, u8 card_type, ...@@ -430,11 +249,6 @@ mlxsw_linecard_provision_set(struct mlxsw_linecard *linecard, u8 card_type,
return PTR_ERR(type); return PTR_ERR(type);
} }
} }
err = mlxsw_linecard_devices_attach(linecard);
if (err) {
mlxsw_linecard_provision_fail(linecard);
return err;
}
linecard->provisioned = true; linecard->provisioned = true;
linecard->hw_revision = hw_revision; linecard->hw_revision = hw_revision;
linecard->ini_version = ini_version; linecard->ini_version = ini_version;
...@@ -447,7 +261,6 @@ static void mlxsw_linecard_provision_clear(struct mlxsw_linecard *linecard) ...@@ -447,7 +261,6 @@ static void mlxsw_linecard_provision_clear(struct mlxsw_linecard *linecard)
mlxsw_linecard_status_event_done(linecard, mlxsw_linecard_status_event_done(linecard,
MLXSW_LINECARD_STATUS_EVENT_TYPE_UNPROVISION); MLXSW_LINECARD_STATUS_EVENT_TYPE_UNPROVISION);
linecard->provisioned = false; linecard->provisioned = false;
mlxsw_linecard_devices_detach(linecard);
devlink_linecard_provision_clear(linecard->devlink_linecard); devlink_linecard_provision_clear(linecard->devlink_linecard);
} }
...@@ -479,18 +292,11 @@ static int mlxsw_linecard_ready_clear(struct mlxsw_linecard *linecard) ...@@ -479,18 +292,11 @@ static int mlxsw_linecard_ready_clear(struct mlxsw_linecard *linecard)
return 0; return 0;
} }
static int mlxsw_linecard_active_set(struct mlxsw_linecard *linecard) static void mlxsw_linecard_active_set(struct mlxsw_linecard *linecard)
{ {
int err;
err = mlxsw_linecard_devices_update(linecard);
if (err)
return err;
mlxsw_linecard_active_ops_call(linecard); mlxsw_linecard_active_ops_call(linecard);
linecard->active = true; linecard->active = true;
devlink_linecard_activate(linecard->devlink_linecard); devlink_linecard_activate(linecard->devlink_linecard);
return 0;
} }
static void mlxsw_linecard_active_clear(struct mlxsw_linecard *linecard) static void mlxsw_linecard_active_clear(struct mlxsw_linecard *linecard)
...@@ -539,11 +345,8 @@ static int mlxsw_linecard_status_process(struct mlxsw_linecards *linecards, ...@@ -539,11 +345,8 @@ static int mlxsw_linecard_status_process(struct mlxsw_linecards *linecards,
goto out; goto out;
} }
if (active && linecard->active != active) { if (active && linecard->active != active)
err = mlxsw_linecard_active_set(linecard); mlxsw_linecard_active_set(linecard);
if (err)
goto out;
}
if (!active && linecard->active != active) if (!active && linecard->active != active)
mlxsw_linecard_active_clear(linecard); mlxsw_linecard_active_clear(linecard);
...@@ -934,44 +737,12 @@ static void mlxsw_linecard_types_get(struct devlink_linecard *devlink_linecard, ...@@ -934,44 +737,12 @@ static void mlxsw_linecard_types_get(struct devlink_linecard *devlink_linecard,
*type_priv = ini_file; *type_priv = ini_file;
} }
static int
mlxsw_linecard_info_get(struct devlink_linecard *devlink_linecard, void *priv,
struct devlink_info_req *req,
struct netlink_ext_ack *extack)
{
struct mlxsw_linecard *linecard = priv;
char buf[32];
int err;
mutex_lock(&linecard->lock);
if (!linecard->provisioned) {
err = 0;
goto unlock;
}
sprintf(buf, "%d", linecard->hw_revision);
err = devlink_info_version_fixed_put(req, "hw.revision", buf);
if (err)
goto unlock;
sprintf(buf, "%d", linecard->ini_version);
err = devlink_info_version_running_put(req, "ini.version", buf);
if (err)
goto unlock;
unlock:
mutex_unlock(&linecard->lock);
return err;
}
static const struct devlink_linecard_ops mlxsw_linecard_ops = { static const struct devlink_linecard_ops mlxsw_linecard_ops = {
.provision = mlxsw_linecard_provision, .provision = mlxsw_linecard_provision,
.unprovision = mlxsw_linecard_unprovision, .unprovision = mlxsw_linecard_unprovision,
.same_provision = mlxsw_linecard_same_provision, .same_provision = mlxsw_linecard_same_provision,
.types_count = mlxsw_linecard_types_count, .types_count = mlxsw_linecard_types_count,
.types_get = mlxsw_linecard_types_get, .types_get = mlxsw_linecard_types_get,
.info_get = mlxsw_linecard_info_get,
.device_info_get = mlxsw_linecard_device_info_get,
}; };
struct mlxsw_linecard_status_event { struct mlxsw_linecard_status_event {
...@@ -1069,7 +840,6 @@ static int mlxsw_linecard_init(struct mlxsw_core *mlxsw_core, ...@@ -1069,7 +840,6 @@ static int mlxsw_linecard_init(struct mlxsw_core *mlxsw_core,
linecard->slot_index = slot_index; linecard->slot_index = slot_index;
linecard->linecards = linecards; linecard->linecards = linecards;
mutex_init(&linecard->lock); mutex_init(&linecard->lock);
INIT_LIST_HEAD(&linecard->device_list);
devlink_linecard = devlink_linecard_create(priv_to_devlink(mlxsw_core), devlink_linecard = devlink_linecard_create(priv_to_devlink(mlxsw_core),
slot_index, &mlxsw_linecard_ops, slot_index, &mlxsw_linecard_ops,
...@@ -1115,7 +885,6 @@ static void mlxsw_linecard_fini(struct mlxsw_core *mlxsw_core, ...@@ -1115,7 +885,6 @@ static void mlxsw_linecard_fini(struct mlxsw_core *mlxsw_core,
mlxsw_core_flush_owq(); mlxsw_core_flush_owq();
if (linecard->active) if (linecard->active)
mlxsw_linecard_active_clear(linecard); mlxsw_linecard_active_clear(linecard);
mlxsw_linecard_devices_detach(linecard);
devlink_linecard_destroy(linecard->devlink_linecard); devlink_linecard_destroy(linecard->devlink_linecard);
mutex_destroy(&linecard->lock); mutex_destroy(&linecard->lock);
} }
......
...@@ -11643,11 +11643,7 @@ MLXSW_ITEM32(reg, mddq, sie, 0x00, 31, 1); ...@@ -11643,11 +11643,7 @@ MLXSW_ITEM32(reg, mddq, sie, 0x00, 31, 1);
enum mlxsw_reg_mddq_query_type { enum mlxsw_reg_mddq_query_type {
MLXSW_REG_MDDQ_QUERY_TYPE_SLOT_INFO = 1, MLXSW_REG_MDDQ_QUERY_TYPE_SLOT_INFO = 1,
MLXSW_REG_MDDQ_QUERY_TYPE_DEVICE_INFO, /* If there are no devices MLXSW_REG_MDDQ_QUERY_TYPE_SLOT_NAME = 3,
* on the slot, data_valid
* will be '0'.
*/
MLXSW_REG_MDDQ_QUERY_TYPE_SLOT_NAME,
}; };
/* reg_mddq_query_type /* reg_mddq_query_type
...@@ -11661,28 +11657,6 @@ MLXSW_ITEM32(reg, mddq, query_type, 0x00, 16, 8); ...@@ -11661,28 +11657,6 @@ MLXSW_ITEM32(reg, mddq, query_type, 0x00, 16, 8);
*/ */
MLXSW_ITEM32(reg, mddq, slot_index, 0x00, 0, 4); MLXSW_ITEM32(reg, mddq, slot_index, 0x00, 0, 4);
/* reg_mddq_response_msg_seq
* Response message sequential number. For a specific request, the response
* message sequential number is the following one. In addition, the last
* message should be 0.
* Access: RO
*/
MLXSW_ITEM32(reg, mddq, response_msg_seq, 0x04, 16, 8);
/* reg_mddq_request_msg_seq
* Request message sequential number.
* The first message number should be 0.
* Access: Index
*/
MLXSW_ITEM32(reg, mddq, request_msg_seq, 0x04, 0, 8);
/* reg_mddq_data_valid
* If set, the data in the data field is valid and contain the information
* for the queried index.
* Access: RO
*/
MLXSW_ITEM32(reg, mddq, data_valid, 0x08, 31, 1);
/* reg_mddq_slot_info_provisioned /* reg_mddq_slot_info_provisioned
* If set, the INI file is applied and the card is provisioned. * If set, the INI file is applied and the card is provisioned.
* Access: RO * Access: RO
...@@ -11769,65 +11743,6 @@ mlxsw_reg_mddq_slot_info_unpack(const char *payload, u8 *p_slot_index, ...@@ -11769,65 +11743,6 @@ mlxsw_reg_mddq_slot_info_unpack(const char *payload, u8 *p_slot_index,
*p_card_type = mlxsw_reg_mddq_slot_info_card_type_get(payload); *p_card_type = mlxsw_reg_mddq_slot_info_card_type_get(payload);
} }
/* reg_mddq_device_info_flash_owner
* If set, the device is the flash owner. Otherwise, a shared flash
* is used by this device (another device is the flash owner).
* Access: RO
*/
MLXSW_ITEM32(reg, mddq, device_info_flash_owner, 0x10, 30, 1);
/* reg_mddq_device_info_device_index
* Device index. The first device should number 0.
* Access: RO
*/
MLXSW_ITEM32(reg, mddq, device_info_device_index, 0x10, 0, 8);
/* reg_mddq_device_info_fw_major
* Major FW version number.
* Access: RO
*/
MLXSW_ITEM32(reg, mddq, device_info_fw_major, 0x14, 16, 16);
/* reg_mddq_device_info_fw_minor
* Minor FW version number.
* Access: RO
*/
MLXSW_ITEM32(reg, mddq, device_info_fw_minor, 0x18, 16, 16);
/* reg_mddq_device_info_fw_sub_minor
* Sub-minor FW version number.
* Access: RO
*/
MLXSW_ITEM32(reg, mddq, device_info_fw_sub_minor, 0x18, 0, 16);
static inline void
mlxsw_reg_mddq_device_info_pack(char *payload, u8 slot_index,
u8 request_msg_seq)
{
__mlxsw_reg_mddq_pack(payload, slot_index,
MLXSW_REG_MDDQ_QUERY_TYPE_DEVICE_INFO);
mlxsw_reg_mddq_request_msg_seq_set(payload, request_msg_seq);
}
static inline void
mlxsw_reg_mddq_device_info_unpack(const char *payload, u8 *p_response_msg_seq,
bool *p_data_valid, bool *p_flash_owner,
u8 *p_device_index, u16 *p_fw_major,
u16 *p_fw_minor, u16 *p_fw_sub_minor)
{
*p_response_msg_seq = mlxsw_reg_mddq_response_msg_seq_get(payload);
*p_data_valid = mlxsw_reg_mddq_data_valid_get(payload);
if (p_flash_owner)
*p_flash_owner = mlxsw_reg_mddq_device_info_flash_owner_get(payload);
*p_device_index = mlxsw_reg_mddq_device_info_device_index_get(payload);
if (p_fw_major)
*p_fw_major = mlxsw_reg_mddq_device_info_fw_major_get(payload);
if (p_fw_minor)
*p_fw_minor = mlxsw_reg_mddq_device_info_fw_minor_get(payload);
if (p_fw_sub_minor)
*p_fw_sub_minor = mlxsw_reg_mddq_device_info_fw_sub_minor_get(payload);
}
#define MLXSW_REG_MDDQ_SLOT_ASCII_NAME_LEN 20 #define MLXSW_REG_MDDQ_SLOT_ASCII_NAME_LEN 20
/* reg_mddq_slot_ascii_name /* reg_mddq_slot_ascii_name
......
...@@ -150,9 +150,6 @@ struct devlink_port_new_attrs { ...@@ -150,9 +150,6 @@ struct devlink_port_new_attrs {
sfnum_valid:1; sfnum_valid:1;
}; };
struct devlink_info_req;
struct devlink_linecard_device;
/** /**
* struct devlink_linecard_ops - Linecard operations * struct devlink_linecard_ops - Linecard operations
* @provision: callback to provision the linecard slot with certain * @provision: callback to provision the linecard slot with certain
...@@ -171,8 +168,6 @@ struct devlink_linecard_device; ...@@ -171,8 +168,6 @@ struct devlink_linecard_device;
* provisioned. * provisioned.
* @types_count: callback to get number of supported types * @types_count: callback to get number of supported types
* @types_get: callback to get next type in list * @types_get: callback to get next type in list
* @info_get: callback to get linecard info
* @device_info_get: callback to get linecard device info
*/ */
struct devlink_linecard_ops { struct devlink_linecard_ops {
int (*provision)(struct devlink_linecard *linecard, void *priv, int (*provision)(struct devlink_linecard *linecard, void *priv,
...@@ -187,12 +182,6 @@ struct devlink_linecard_ops { ...@@ -187,12 +182,6 @@ struct devlink_linecard_ops {
void (*types_get)(struct devlink_linecard *linecard, void (*types_get)(struct devlink_linecard *linecard,
void *priv, unsigned int index, const char **type, void *priv, unsigned int index, const char **type,
const void **type_priv); const void **type_priv);
int (*info_get)(struct devlink_linecard *linecard, void *priv,
struct devlink_info_req *req,
struct netlink_ext_ack *extack);
int (*device_info_get)(struct devlink_linecard_device *device,
void *priv, struct devlink_info_req *req,
struct netlink_ext_ack *extack);
}; };
struct devlink_sb_pool_info { struct devlink_sb_pool_info {
...@@ -639,6 +628,7 @@ struct devlink_flash_update_params { ...@@ -639,6 +628,7 @@ struct devlink_flash_update_params {
#define DEVLINK_SUPPORT_FLASH_UPDATE_OVERWRITE_MASK BIT(1) #define DEVLINK_SUPPORT_FLASH_UPDATE_OVERWRITE_MASK BIT(1)
struct devlink_region; struct devlink_region;
struct devlink_info_req;
/** /**
* struct devlink_region_ops - Region operations * struct devlink_region_ops - Region operations
...@@ -1588,12 +1578,6 @@ struct devlink_linecard * ...@@ -1588,12 +1578,6 @@ struct devlink_linecard *
devlink_linecard_create(struct devlink *devlink, unsigned int linecard_index, devlink_linecard_create(struct devlink *devlink, unsigned int linecard_index,
const struct devlink_linecard_ops *ops, void *priv); const struct devlink_linecard_ops *ops, void *priv);
void devlink_linecard_destroy(struct devlink_linecard *linecard); void devlink_linecard_destroy(struct devlink_linecard *linecard);
struct devlink_linecard_device *
devlink_linecard_device_create(struct devlink_linecard *linecard,
unsigned int device_index, void *priv);
void
devlink_linecard_device_destroy(struct devlink_linecard *linecard,
struct devlink_linecard_device *linecard_device);
void devlink_linecard_provision_set(struct devlink_linecard *linecard, void devlink_linecard_provision_set(struct devlink_linecard *linecard,
const char *type); const char *type);
void devlink_linecard_provision_clear(struct devlink_linecard *linecard); void devlink_linecard_provision_clear(struct devlink_linecard *linecard);
......
...@@ -136,8 +136,6 @@ enum devlink_command { ...@@ -136,8 +136,6 @@ enum devlink_command {
DEVLINK_CMD_LINECARD_NEW, DEVLINK_CMD_LINECARD_NEW,
DEVLINK_CMD_LINECARD_DEL, DEVLINK_CMD_LINECARD_DEL,
DEVLINK_CMD_LINECARD_INFO_GET, /* can dump */
/* add new commands above here */ /* add new commands above here */
__DEVLINK_CMD_MAX, __DEVLINK_CMD_MAX,
DEVLINK_CMD_MAX = __DEVLINK_CMD_MAX - 1 DEVLINK_CMD_MAX = __DEVLINK_CMD_MAX - 1
...@@ -577,9 +575,6 @@ enum devlink_attr { ...@@ -577,9 +575,6 @@ enum devlink_attr {
DEVLINK_ATTR_LINECARD_STATE, /* u8 */ DEVLINK_ATTR_LINECARD_STATE, /* u8 */
DEVLINK_ATTR_LINECARD_TYPE, /* string */ DEVLINK_ATTR_LINECARD_TYPE, /* string */
DEVLINK_ATTR_LINECARD_SUPPORTED_TYPES, /* nested */ DEVLINK_ATTR_LINECARD_SUPPORTED_TYPES, /* nested */
DEVLINK_ATTR_LINECARD_DEVICE_LIST, /* nested */
DEVLINK_ATTR_LINECARD_DEVICE, /* nested */
DEVLINK_ATTR_LINECARD_DEVICE_INDEX, /* u32 */
/* add new attributes above here, update the policy in devlink.c */ /* add new attributes above here, update the policy in devlink.c */
......
This diff is collapsed.
...@@ -152,7 +152,6 @@ unprovision_test() ...@@ -152,7 +152,6 @@ unprovision_test()
LC_16X100G_TYPE="16x100G" LC_16X100G_TYPE="16x100G"
LC_16X100G_PORT_COUNT=16 LC_16X100G_PORT_COUNT=16
LC_16X100G_DEVICE_COUNT=4
supported_types_check() supported_types_check()
{ {
...@@ -178,42 +177,6 @@ supported_types_check() ...@@ -178,42 +177,6 @@ supported_types_check()
check_err $? "16X100G not found between supported types of linecard $lc" check_err $? "16X100G not found between supported types of linecard $lc"
} }
lc_info_check()
{
local lc=$1
local fixed_hw_revision
local running_ini_version
fixed_hw_revision=$(devlink lc -v info $DEVLINK_DEV lc $lc -j | \
jq -e -r '.[][][].versions.fixed."hw.revision"')
check_err $? "Failed to get linecard $lc fixed.hw.revision"
log_info "Linecard $lc fixed.hw.revision: \"$fixed_hw_revision\""
running_ini_version=$(devlink lc -v info $DEVLINK_DEV lc $lc -j | \
jq -e -r '.[][][].versions.running."ini.version"')
check_err $? "Failed to get linecard $lc running.ini.version"
log_info "Linecard $lc running.ini.version: \"$running_ini_version\""
}
lc_devices_check()
{
local lc=$1
local expected_device_count=$2
local device_count
local device
device_count=$(devlink lc show $DEVLINK_DEV lc $lc -j | \
jq -e -r ".[][][].devices |length")
check_err $? "Failed to get linecard $lc device count"
[ $device_count != 0 ]
check_err $? "No device found on linecard $lc"
[ $device_count == $expected_device_count ]
check_err $? "Unexpected device count on linecard $lc (got $expected_device_count, expected $device_count)"
for (( device=0; device<device_count; device++ ))
do
log_info "Linecard $lc device $device"
done
}
ports_check() ports_check()
{ {
local lc=$1 local lc=$1
...@@ -243,8 +206,6 @@ provision_test() ...@@ -243,8 +206,6 @@ provision_test()
unprovision_one $lc unprovision_one $lc
fi fi
provision_one $lc $LC_16X100G_TYPE provision_one $lc $LC_16X100G_TYPE
lc_devices_check $lc $LC_16X100G_DEVICE_COUNT
lc_info_check $lc
ports_check $lc $LC_16X100G_PORT_COUNT ports_check $lc $LC_16X100G_PORT_COUNT
log_test "Provision" log_test "Provision"
} }
...@@ -259,26 +220,6 @@ interface_check() ...@@ -259,26 +220,6 @@ interface_check()
setup_wait setup_wait
} }
lc_devices_info_check()
{
local lc=$1
local expected_device_count=$2
local device_count
local device
local running_device_fw
device_count=$(devlink lc info $DEVLINK_DEV lc $lc -j | \
jq -e -r ".[][][].devices |length")
check_err $? "Failed to get linecard $lc device count"
for (( device=0; device<device_count; device++ ))
do
running_device_fw=$(devlink lc -v info $DEVLINK_DEV lc $lc -j | \
jq -e -r ".[][][].devices[$device].versions.running.fw")
check_err $? "Failed to get linecard $lc device $device running fw version"
log_info "Linecard $lc device $device running.fw: \"$running_device_fw\""
done
}
activation_16x100G_test() activation_16x100G_test()
{ {
RET=0 RET=0
...@@ -295,8 +236,6 @@ activation_16x100G_test() ...@@ -295,8 +236,6 @@ activation_16x100G_test()
$ACTIVATION_TIMEOUT) $ACTIVATION_TIMEOUT)
check_err $? "Failed to get linecard $lc activated (timeout)" check_err $? "Failed to get linecard $lc activated (timeout)"
lc_devices_info_check $lc $LC_16X100G_DEVICE_COUNT
interface_check interface_check
log_test "Activation 16x100G" log_test "Activation 16x100G"
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment