[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <de5758c6-4379-1b70-19ff-d6dd2b3ea269@helixd.com>
Date: Fri, 6 Aug 2021 09:03:15 -0700
From: Dario Alcocer <dalcocer@...ixd.com>
To: Andrew Lunn <andrew@...n.ch>
Cc: netdev@...r.kernel.org
Subject: Re: Marvell switch port shows LOWERLAYERDOWN, ping fails
I got mv88e6xxx_dump working by doing the following:
* copying nexthop.h and rtnetlink.h from /usr/include/uapi/linux to
/usr/include/linux
* commenting out the undefined symbol references (see patch below)
diff --git a/desc-devlink.c b/desc-devlink.c
index 820121e..1876b65 100644
--- a/desc-devlink.c
+++ b/desc-devlink.c
@@ -182,24 +182,24 @@ static const struct pretty_nla_desc __attr_desc[] = {
NLATTR_DESC_STRING(DEVLINK_ATTR_FLASH_UPDATE_FILE_NAME),
NLATTR_DESC_STRING(DEVLINK_ATTR_FLASH_UPDATE_COMPONENT),
- NLATTR_DESC_STRING(DEVLINK_ATTR_FLASH_UPDATE_STATUS_MSG),
- NLATTR_DESC_U64(DEVLINK_ATTR_FLASH_UPDATE_STATUS_DONE),
- NLATTR_DESC_U64(DEVLINK_ATTR_FLASH_UPDATE_STATUS_TOTAL),
+ //NLATTR_DESC_STRING(DEVLINK_ATTR_FLASH_UPDATE_STATUS_MSG),
+ //NLATTR_DESC_U64(DEVLINK_ATTR_FLASH_UPDATE_STATUS_DONE),
+ //NLATTR_DESC_U64(DEVLINK_ATTR_FLASH_UPDATE_STATUS_TOTAL),
- NLATTR_DESC_U16(DEVLINK_ATTR_PORT_PCI_PF_NUMBER),
- NLATTR_DESC_U16(DEVLINK_ATTR_PORT_PCI_VF_NUMBER),
+ //NLATTR_DESC_U16(DEVLINK_ATTR_PORT_PCI_PF_NUMBER),
+ //NLATTR_DESC_U16(DEVLINK_ATTR_PORT_PCI_VF_NUMBER),
//DEVLINK_ATTR_STATS, /* nested */
- NLATTR_DESC_STRING(DEVLINK_ATTR_TRAP_NAME),
- NLATTR_DESC_U8(DEVLINK_ATTR_TRAP_ACTION),
- NLATTR_DESC_U8(DEVLINK_ATTR_TRAP_TYPE),
- NLATTR_DESC_FLAG(DEVLINK_ATTR_TRAP_GENERIC),
+ //NLATTR_DESC_STRING(DEVLINK_ATTR_TRAP_NAME),
+ //NLATTR_DESC_U8(DEVLINK_ATTR_TRAP_ACTION),
+ //NLATTR_DESC_U8(DEVLINK_ATTR_TRAP_TYPE),
+ //NLATTR_DESC_FLAG(DEVLINK_ATTR_TRAP_GENERIC),
//DEVLINK_ATTR_TRAP_METADATA, /* nested */
- NLATTR_DESC_STRING(DEVLINK_ATTR_TRAP_GROUP_NAME),
+ //NLATTR_DESC_STRING(DEVLINK_ATTR_TRAP_GROUP_NAME),
- NLATTR_DESC_U8(DEVLINK_ATTR_RELOAD_FAILED),
- NLATTR_DESC_U64(DEVLINK_ATTR_HEALTH_REPORTER_DUMP_TS_NS),
+ //NLATTR_DESC_U8(DEVLINK_ATTR_RELOAD_FAILED),
+ //NLATTR_DESC_U64(DEVLINK_ATTR_HEALTH_REPORTER_DUMP_TS_NS),
};
const struct pretty_nlmsg_desc devlink_msg_desc[] = {
@@ -264,16 +264,16 @@ const struct pretty_nlmsg_desc devlink_msg_desc[] = {
NLMSG_DESC(DEVLINK_CMD_HEALTH_REPORTER_DUMP_GET, attr),
NLMSG_DESC(DEVLINK_CMD_HEALTH_REPORTER_DUMP_CLEAR, attr),
NLMSG_DESC(DEVLINK_CMD_FLASH_UPDATE, attr),
- NLMSG_DESC(DEVLINK_CMD_FLASH_UPDATE_END, attr),
- NLMSG_DESC(DEVLINK_CMD_FLASH_UPDATE_STATUS, attr),
- NLMSG_DESC(DEVLINK_CMD_TRAP_GET, attr),
- NLMSG_DESC(DEVLINK_CMD_TRAP_SET, attr),
- NLMSG_DESC(DEVLINK_CMD_TRAP_NEW, attr),
- NLMSG_DESC(DEVLINK_CMD_TRAP_DEL, attr),
- NLMSG_DESC(DEVLINK_CMD_TRAP_GROUP_GET, attr),
- NLMSG_DESC(DEVLINK_CMD_TRAP_GROUP_SET, attr),
- NLMSG_DESC(DEVLINK_CMD_TRAP_GROUP_NEW, attr),
- NLMSG_DESC(DEVLINK_CMD_TRAP_GROUP_DEL, attr),
+ //NLMSG_DESC(DEVLINK_CMD_FLASH_UPDATE_END, attr),
+ //NLMSG_DESC(DEVLINK_CMD_FLASH_UPDATE_STATUS, attr),
+ //NLMSG_DESC(DEVLINK_CMD_TRAP_GET, attr),
+ //NLMSG_DESC(DEVLINK_CMD_TRAP_SET, attr),
+ //NLMSG_DESC(DEVLINK_CMD_TRAP_NEW, attr),
+ //NLMSG_DESC(DEVLINK_CMD_TRAP_DEL, attr),
+ //NLMSG_DESC(DEVLINK_CMD_TRAP_GROUP_GET, attr),
+ //NLMSG_DESC(DEVLINK_CMD_TRAP_GROUP_SET, attr),
+ //NLMSG_DESC(DEVLINK_CMD_TRAP_GROUP_NEW, attr),
+ //NLMSG_DESC(DEVLINK_CMD_TRAP_GROUP_DEL, attr),
};
const unsigned int devlink_msg_n_desc = ARRAY_SIZE(devlink_msg_desc);
diff --git a/mv88e6xxx_dump.c b/mv88e6xxx_dump.c
index 09a67c1..a744a74 100644
--- a/mv88e6xxx_dump.c
+++ b/mv88e6xxx_dump.c
@@ -220,17 +220,17 @@ static const enum mnl_attr_data_type
devlink_policy[DEVLINK_ATTR_MAX + 1] = {
[DEVLINK_ATTR_HEALTH_REPORTER_DUMP_TS] = MNL_TYPE_U64,
[DEVLINK_ATTR_HEALTH_REPORTER_GRACEFUL_PERIOD] = MNL_TYPE_U64,
[DEVLINK_ATTR_FLASH_UPDATE_COMPONENT] = MNL_TYPE_STRING,
- [DEVLINK_ATTR_FLASH_UPDATE_STATUS_MSG] = MNL_TYPE_STRING,
- [DEVLINK_ATTR_FLASH_UPDATE_STATUS_DONE] = MNL_TYPE_U64,
- [DEVLINK_ATTR_FLASH_UPDATE_STATUS_TOTAL] = MNL_TYPE_U64,
- [DEVLINK_ATTR_STATS] = MNL_TYPE_NESTED,
- [DEVLINK_ATTR_TRAP_NAME] = MNL_TYPE_STRING,
- [DEVLINK_ATTR_TRAP_ACTION] = MNL_TYPE_U8,
- [DEVLINK_ATTR_TRAP_TYPE] = MNL_TYPE_U8,
- [DEVLINK_ATTR_TRAP_GENERIC] = MNL_TYPE_FLAG,
- [DEVLINK_ATTR_TRAP_METADATA] = MNL_TYPE_NESTED,
- [DEVLINK_ATTR_TRAP_GROUP_NAME] = MNL_TYPE_STRING,
- [DEVLINK_ATTR_RELOAD_FAILED] = MNL_TYPE_U8,
+ //[DEVLINK_ATTR_FLASH_UPDATE_STATUS_MSG] = MNL_TYPE_STRING,
+ //[DEVLINK_ATTR_FLASH_UPDATE_STATUS_DONE] = MNL_TYPE_U64,
+ //[DEVLINK_ATTR_FLASH_UPDATE_STATUS_TOTAL] = MNL_TYPE_U64,
+ //[DEVLINK_ATTR_STATS] = MNL_TYPE_NESTED,
+ //[DEVLINK_ATTR_TRAP_NAME] = MNL_TYPE_STRING,
+ //[DEVLINK_ATTR_TRAP_ACTION] = MNL_TYPE_U8,
+ //[DEVLINK_ATTR_TRAP_TYPE] = MNL_TYPE_U8,
+ //[DEVLINK_ATTR_TRAP_GENERIC] = MNL_TYPE_FLAG,
+ //[DEVLINK_ATTR_TRAP_METADATA] = MNL_TYPE_NESTED,
+ //[DEVLINK_ATTR_TRAP_GROUP_NAME] = MNL_TYPE_STRING,
+ //[DEVLINK_ATTR_RELOAD_FAILED] = MNL_TYPE_U8,
};
static int attr_cb(const struct nlattr *attr, void *data)
Powered by blists - more mailing lists