lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<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

Powered by Openwall GNU/*/Linux Powered by OpenVZ