| Message ID | 20180730193522.30458-1-vishal.l.verma@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [ndctl] ndctl, test/monitor: fix inject-smart field in test_filter_dimmevent | expand |
> -----Original Message----- > From: Vishal Verma [mailto:vishal.l.verma@intel.com] > Sent: Tuesday, July 31, 2018 4:35 AM > To: linux-nvdimm@lists.01.org > Cc: Vishal Verma <vishal.l.verma@intel.com>; Qi, Fuli/斉 福利 > <qi.fuli@jp.fujitsu.com> > Subject: [ndctl PATCH] ndctl, test/monitor: fix inject-smart field in > test_filter_dimmevent > > Change the inject-smart field from -s to -m when testing the dimm-media-temperature > event. The -s seems to have been a copy-paste error. > > Cc: QI Fuli <qi.fuli@jp.fujitsu.com> > Signed-off-by: Vishal Verma <vishal.l.verma@intel.com> > --- > test/monitor.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/test/monitor.sh b/test/monitor.sh index 1bbfd14..29d4ea1 100755 > --- a/test/monitor.sh > +++ b/test/monitor.sh > @@ -152,7 +152,7 @@ test_filter_dimmevent() > inject_value=$($NDCTL list -H -d $monitor_dimms | jq > -r .[]."health"."temperature_threshold") > inject_value=$((inject_value + 1)) > start_monitor "-d $monitor_dimms -D dimm-media-temperature" > - inject_smart "-s $inject_value" > + inject_smart "-m $inject_value" > check_result "$monitor_dimms" > stop_monitor > } Hi Vishal, Yes, this is a copy-paste error and I am sorry for this. Thank you very much. QI > -- > 2.14.4 > >
diff --git a/test/monitor.sh b/test/monitor.sh index 1bbfd14..29d4ea1 100755 --- a/test/monitor.sh +++ b/test/monitor.sh @@ -152,7 +152,7 @@ test_filter_dimmevent() inject_value=$($NDCTL list -H -d $monitor_dimms | jq -r .[]."health"."temperature_threshold") inject_value=$((inject_value + 1)) start_monitor "-d $monitor_dimms -D dimm-media-temperature" - inject_smart "-s $inject_value" + inject_smart "-m $inject_value" check_result "$monitor_dimms" stop_monitor }
Change the inject-smart field from -s to -m when testing the dimm-media-temperature event. The -s seems to have been a copy-paste error. Cc: QI Fuli <qi.fuli@jp.fujitsu.com> Signed-off-by: Vishal Verma <vishal.l.verma@intel.com> --- test/monitor.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)