[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20220723042206.8104-1-vikas.gupta@broadcom.com>
Date: Sat, 23 Jul 2022 09:52:04 +0530
From: Vikas Gupta <vikas.gupta@...adcom.com>
To: jiri@...dia.com, kuba@...nel.org
Cc: netdev@...r.kernel.org, linux-kernel@...r.kernel.org,
davem@...emloft.net, dsahern@...nel.org,
stephen@...workplumber.org, edumazet@...gle.com, pabeni@...hat.com,
ast@...nel.org, leon@...nel.org, linux-doc@...r.kernel.org,
corbet@....net, michael.chan@...adcom.com,
andrew.gospodarek@...adcom.com,
Vikas Gupta <vikas.gupta@...adcom.com>
Subject: [PATCH net-next v6 0/2] add framework for selftests in devlink
Hi,
This patchset adds support for selftests in the devlink framework.
It adds a callback .selftests_check and .selftests_run in devlink_ops.
User can add test(s) suite which is subsequently passed to the driver
and driver can opt for running particular tests based on its capabilities.
Patchset adds a flash based test for the bnxt_en driver.
Suggested commands at user level would be as below:
changes from:
v5->v6:
Addressed change in .rst file only suggested by jiri@...dia.com in patch v5.
v4->v5:
Addressed the changes requested by jiri@...dia.com in patch v4.
v3->v4:
Addressed the changes requested by kuba@...nel.org in patch v3.
v2->v3:
1)
After discussions with jiri@...dia.com, passing a testmask from
user to kernel is removed and a flag based arguments are adopted.
This way we can have more than 32/64 selftests defined in the
kernel.
Below is the format from user to kernel and vice-versa.
Kernel to user for show command . Users can know what all tests are
supported by the driver. A return from kernel to user if driver
supports TEST1, TEST4, and TEST7.
______
|NEST |
|_____ |TEST1|TEST4|TEST7|...
User to kernel to execute test: If user wants to execute test4, test8,
test1...
______
|NEST |
|_____ |TEST4|TEST8|TEST1|...
After executing the tests kernel return to user.
|NEST |
|_____ | NEST| |NEST| |NEST|
TEST4,RES4 TEST8,RES8 TEST1, RES1
2) Added dumpit in devlink for list/show command.
v1->v2:
Addressed the changes requested by kuba@...nel.org in patch v1.
Fixed the style issues.
Thanks,
Vikas
Vikas Gupta (2):
devlink: introduce framework for selftests
bnxt_en: implement callbacks for devlink selftests
.../networking/devlink/devlink-selftests.rst | 38 +++
.../net/ethernet/broadcom/bnxt/bnxt_devlink.c | 61 +++++
.../net/ethernet/broadcom/bnxt/bnxt_ethtool.c | 24 +-
.../net/ethernet/broadcom/bnxt/bnxt_ethtool.h | 12 +
include/net/devlink.h | 21 ++
include/uapi/linux/devlink.h | 32 +++
net/core/devlink.c | 223 ++++++++++++++++++
7 files changed, 399 insertions(+), 12 deletions(-)
create mode 100644 Documentation/networking/devlink/devlink-selftests.rst
--
2.31.1
Download attachment "smime.p7s" of type "application/pkcs7-signature" (4206 bytes)
Powered by blists - more mailing lists