linux-block.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/1] null_blk: describe the usage of fault injection param
@ 2020-03-04 19:16 Dongli Zhang
  2020-03-12 13:57 ` Jens Axboe
  0 siblings, 1 reply; 3+ messages in thread
From: Dongli Zhang @ 2020-03-04 19:16 UTC (permalink / raw)
  To: linux-block; +Cc: axboe, linux-kernel

As null_blk is a very good start point to test block layer, this patch adds
description and comments to 'timeout' and 'requeue' to explain how to use
fault injection with null_blk.

The nvme has similar with nvme_core.fail_request in the form of comment.

Signed-off-by: Dongli Zhang <dongli.zhang@oracle.com>
---
 drivers/block/null_blk_main.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/drivers/block/null_blk_main.c b/drivers/block/null_blk_main.c
index 133060431dbd..1ee5aaacdb0f 100644
--- a/drivers/block/null_blk_main.c
+++ b/drivers/block/null_blk_main.c
@@ -96,11 +96,17 @@ module_param_named(home_node, g_home_node, int, 0444);
 MODULE_PARM_DESC(home_node, "Home node for the device");
 
 #ifdef CONFIG_BLK_DEV_NULL_BLK_FAULT_INJECTION
+/*
+ * For more details about fault injection, please refer to
+ * Documentation/fault-injection/fault-injection.rst.
+ */
 static char g_timeout_str[80];
 module_param_string(timeout, g_timeout_str, sizeof(g_timeout_str), 0444);
+MODULE_PARM_DESC(timeout, "Fault injection. timeout=<interval>,<probability>,<space>,<times>");
 
 static char g_requeue_str[80];
 module_param_string(requeue, g_requeue_str, sizeof(g_requeue_str), 0444);
+MODULE_PARM_DESC(requeue, "Fault injection. requeue=<interval>,<probability>,<space>,<times>");
 #endif
 
 static int g_queue_mode = NULL_Q_MQ;
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/1] null_blk: describe the usage of fault injection param
  2020-03-04 19:16 [PATCH 1/1] null_blk: describe the usage of fault injection param Dongli Zhang
@ 2020-03-12 13:57 ` Jens Axboe
  2020-03-12 20:42   ` dongli.zhang
  0 siblings, 1 reply; 3+ messages in thread
From: Jens Axboe @ 2020-03-12 13:57 UTC (permalink / raw)
  To: Dongli Zhang, linux-block; +Cc: linux-kernel

On 3/4/20 12:16 PM, Dongli Zhang wrote:
> As null_blk is a very good start point to test block layer, this patch adds
> description and comments to 'timeout' and 'requeue' to explain how to use
> fault injection with null_blk.
> 
> The nvme has similar with nvme_core.fail_request in the form of comment.

This doesn't apply to for-5.7/drivers, care to resend?

-- 
Jens Axboe


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/1] null_blk: describe the usage of fault injection param
  2020-03-12 13:57 ` Jens Axboe
@ 2020-03-12 20:42   ` dongli.zhang
  0 siblings, 0 replies; 3+ messages in thread
From: dongli.zhang @ 2020-03-12 20:42 UTC (permalink / raw)
  To: Jens Axboe, linux-block; +Cc: linux-kernel



On 3/12/20 6:57 AM, Jens Axboe wrote:
> On 3/4/20 12:16 PM, Dongli Zhang wrote:
>> As null_blk is a very good start point to test block layer, this patch adds
>> description and comments to 'timeout' and 'requeue' to explain how to use
>> fault injection with null_blk.
>>
>> The nvme has similar with nvme_core.fail_request in the form of comment.
> 
> This doesn't apply to for-5.7/drivers, care to resend?
> 

I would resend based on for-5.7/drivers.

Thank you very much!

Dongli Zhang

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-03-12 20:42 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-04 19:16 [PATCH 1/1] null_blk: describe the usage of fault injection param Dongli Zhang
2020-03-12 13:57 ` Jens Axboe
2020-03-12 20:42   ` dongli.zhang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).