mbox series

[ndctl,0/3] Add missing firmware_status checks

Message ID 20190111225339.6939-1-vishal.l.verma@intel.com (mailing list archive)
Headers show
Series Add missing firmware_status checks | expand

Message

Verma, Vishal L Jan. 11, 2019, 10:53 p.m. UTC
There were several places in ndctl where we neglected to check the
firmware_status from a command submission.

Add an ndctl_dimm_op target to perform error translation for a DSM
family, and provide a new helper -ndctl_cmd_submit_xlat - that will
submit the command and call an error translation routing if one has been
registered.

Finally, switch the call sites in ndctl/inject-smart.c and
ndctl/monitor.c over to the new command submission helper.

Vishal Verma (3):
  libndctl, intel: Add infrastructure for firmware_status translation
  ndctl, inject-smart: switch to ndctl_cmd_submit_xlat
  ndctl, monitor: switch to ndctl_cmd_submit_xlat

 ndctl/inject-smart.c   |  8 +++----
 ndctl/lib/intel.c      | 49 ++++++++++++++++++++++++++++++++++++++++++
 ndctl/lib/intel.h      |  1 +
 ndctl/lib/libndctl.c   | 26 ++++++++++++++++++++++
 ndctl/lib/libndctl.sym |  6 ++++++
 ndctl/lib/private.h    |  1 +
 ndctl/libndctl.h       |  3 +++
 ndctl/monitor.c        |  4 ++--
 8 files changed, 92 insertions(+), 6 deletions(-)