* [PATCH net 1/2] docs: net: tweak devlink health documentation
@ 2021-03-13 0:30 Jakub Kicinski
2021-03-13 0:30 ` [PATCH net 2/2] docs: net: add missing devlink health cmd - trigger Jakub Kicinski
2021-03-14 2:20 ` [PATCH net 1/2] docs: net: tweak devlink health documentation patchwork-bot+netdevbpf
0 siblings, 2 replies; 3+ messages in thread
From: Jakub Kicinski @ 2021-03-13 0:30 UTC (permalink / raw)
To: davem
Cc: netdev, ayal, eranbe, jiri, linux-doc, jacob.e.keller, Jakub Kicinski
Minor tweaks and improvement of wording about the diagnose callback.
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
---
Documentation/networking/devlink/devlink-health.rst | 13 +++++++------
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/Documentation/networking/devlink/devlink-health.rst b/Documentation/networking/devlink/devlink-health.rst
index 0c99b11f05f9..d52c9a605188 100644
--- a/Documentation/networking/devlink/devlink-health.rst
+++ b/Documentation/networking/devlink/devlink-health.rst
@@ -24,7 +24,7 @@ attributes of the health reporting and recovery procedures.
The ``devlink`` health reporter:
Device driver creates a "health reporter" per each error/health type.
-Error/Health type can be a known/generic (eg pci error, fw error, rx/tx error)
+Error/Health type can be a known/generic (e.g. PCI error, fw error, rx/tx error)
or unknown (driver specific).
For each registered health reporter a driver can issue error/health reports
asynchronously. All health reports handling is done by ``devlink``.
@@ -48,6 +48,7 @@ Actions
* Object dump is being taken and saved at the reporter instance (as long as
there is no other dump which is already stored)
* Auto recovery attempt is being done. Depends on:
+
- Auto-recovery configuration
- Grace period vs. time passed since last recover
@@ -72,14 +73,14 @@ User can access/change each reporter's parameters and driver specific callbacks
* - ``DEVLINK_CMD_HEALTH_REPORTER_SET``
- Allows reporter-related configuration setting.
* - ``DEVLINK_CMD_HEALTH_REPORTER_RECOVER``
- - Triggers a reporter's recovery procedure.
+ - Triggers reporter's recovery procedure.
* - ``DEVLINK_CMD_HEALTH_REPORTER_DIAGNOSE``
- - Retrieves diagnostics data from a reporter on a device.
+ - Retrieves current device state related to the reporter.
* - ``DEVLINK_CMD_HEALTH_REPORTER_DUMP_GET``
- Retrieves the last stored dump. Devlink health
- saves a single dump. If an dump is not already stored by the devlink
+ saves a single dump. If an dump is not already stored by devlink
for this reporter, devlink generates a new dump.
- dump output is defined by the reporter.
+ Dump output is defined by the reporter.
* - ``DEVLINK_CMD_HEALTH_REPORTER_DUMP_CLEAR``
- Clears the last saved dump file for the specified reporter.
@@ -93,7 +94,7 @@ User can access/change each reporter's parameters and driver specific callbacks
+--------------------------+
|request for ops
|(diagnose,
- mlx5_core devlink |recover,
+ driver devlink |recover,
|dump)
+--------+ +--------------------------+
| | | reporter| |
--
2.30.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH net 2/2] docs: net: add missing devlink health cmd - trigger
2021-03-13 0:30 [PATCH net 1/2] docs: net: tweak devlink health documentation Jakub Kicinski
@ 2021-03-13 0:30 ` Jakub Kicinski
2021-03-14 2:20 ` [PATCH net 1/2] docs: net: tweak devlink health documentation patchwork-bot+netdevbpf
1 sibling, 0 replies; 3+ messages in thread
From: Jakub Kicinski @ 2021-03-13 0:30 UTC (permalink / raw)
To: davem
Cc: netdev, ayal, eranbe, jiri, linux-doc, jacob.e.keller, Jakub Kicinski
Documentation is missing and it's not very clear what
this callback is for - presumably testing the recovery?
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
---
Documentation/networking/devlink/devlink-health.rst | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/Documentation/networking/devlink/devlink-health.rst b/Documentation/networking/devlink/devlink-health.rst
index d52c9a605188..e37f77734b5b 100644
--- a/Documentation/networking/devlink/devlink-health.rst
+++ b/Documentation/networking/devlink/devlink-health.rst
@@ -74,6 +74,10 @@ User can access/change each reporter's parameters and driver specific callbacks
- Allows reporter-related configuration setting.
* - ``DEVLINK_CMD_HEALTH_REPORTER_RECOVER``
- Triggers reporter's recovery procedure.
+ * - ``DEVLINK_CMD_HEALTH_REPORTER_TEST``
+ - Triggers a fake health event on the reporter. The effects of the test
+ event in terms of recovery flow should follow closely that of a real
+ event.
* - ``DEVLINK_CMD_HEALTH_REPORTER_DIAGNOSE``
- Retrieves current device state related to the reporter.
* - ``DEVLINK_CMD_HEALTH_REPORTER_DUMP_GET``
--
2.30.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH net 1/2] docs: net: tweak devlink health documentation
2021-03-13 0:30 [PATCH net 1/2] docs: net: tweak devlink health documentation Jakub Kicinski
2021-03-13 0:30 ` [PATCH net 2/2] docs: net: add missing devlink health cmd - trigger Jakub Kicinski
@ 2021-03-14 2:20 ` patchwork-bot+netdevbpf
1 sibling, 0 replies; 3+ messages in thread
From: patchwork-bot+netdevbpf @ 2021-03-14 2:20 UTC (permalink / raw)
To: Jakub Kicinski
Cc: davem, netdev, ayal, eranbe, jiri, linux-doc, jacob.e.keller
Hello:
This series was applied to netdev/net-next.git (refs/heads/master):
On Fri, 12 Mar 2021 16:30:25 -0800 you wrote:
> Minor tweaks and improvement of wording about the diagnose callback.
>
> Signed-off-by: Jakub Kicinski <kuba@kernel.org>
> ---
> Documentation/networking/devlink/devlink-health.rst | 13 +++++++------
> 1 file changed, 7 insertions(+), 6 deletions(-)
Here is the summary with links:
- [net,1/2] docs: net: tweak devlink health documentation
https://git.kernel.org/netdev/net-next/c/3cc9b29ac0e1
- [net,2/2] docs: net: add missing devlink health cmd - trigger
https://git.kernel.org/netdev/net-next/c/6f1629093399
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-03-14 2:21 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-13 0:30 [PATCH net 1/2] docs: net: tweak devlink health documentation Jakub Kicinski
2021-03-13 0:30 ` [PATCH net 2/2] docs: net: add missing devlink health cmd - trigger Jakub Kicinski
2021-03-14 2:20 ` [PATCH net 1/2] docs: net: tweak devlink health documentation patchwork-bot+netdevbpf
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.