[GIT PULL] libnvdimm fix for 4.13-final
From: Williams, Dan J
Date: Wed Aug 30 2017 - 17:38:02 EST
Hi Linus, please pull from:
git://git.kernel.org/pub/scm/linux/kernel/git/nvdimm/nvdimm libnvdimm-fixes
...to receive a single patch removing some structure definitions from a
uapi header file. These payloads are never processed directly by the
kernel they are simply passed through an ioctl as opaque blobs to the
ACPI +AF8-DSM (Device Specific Method) interface.
Userspace should not be depending on the kernel to define these
payloads. We will instead provide these definitions via the existing
libndctl (https://github.com/pmem/ndctl) project that has NVDIMM
command helpers and other definitions.
---
The following changes since commit cc4a41fe5541a73019a864883297bd5043aa6d98:
Linux 4.13-rc7 (2017-08-27 17:20:40 -0700)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/nvdimm/nvdimm libnvdimm-fixes
for you to fetch changes up to 7a14724f54bf9889fcb1a9f1d4aa4e1d2e969d93:
libnvdimm: clean up command definitions (2017-08-28 08:33:20 -0700)
----------------------------------------------------------------
Dan Williams (1):
libnvdimm: clean up command definitions
include/uapi/linux/ndctl.h +AHw- 37 -------------------------------------
1 file changed, 37 deletions(-)
commit 7a14724f54bf9889fcb1a9f1d4aa4e1d2e969d93
Author: Dan Williams +ADw-dan.j.williams+AEA-intel.com+AD4-
Date: Mon Aug 28 08:33:20 2017 -0700
libnvdimm: clean up command definitions
Remove the command payloads that do not have an associated libnvdimm
ioctl. I.e. remove the payloads that would only ever be carried in the
ND+AF8-CMD+AF8-CALL envelope. This prevents userspace from growing unnecessary
dependencies on this kernel header when userspace already has everything
it needs to craft and send these commands.
Cc: Jerry Hoemann +ADw-jerry.hoemann+AEA-hpe.com+AD4-
Reported-by: Yasunori Goto +ADw-y-goto+AEA-jp.fujitsu.com+AD4-
Signed-off-by: Dan Williams +ADw-dan.j.williams+AEA-intel.com+AD4-
diff --git a/include/uapi/linux/ndctl.h b/include/uapi/linux/ndctl.h
index 6d3c54264d8e..3f03567631cb 100644
--- a/include/uapi/linux/ndctl.h
+-+-+- b/include/uapi/linux/ndctl.h
+AEAAQA- -145,43 +-145,6 +AEAAQA- struct nd+AF8-cmd+AF8-clear+AF8-error +AHs-
+AF8AXw-u64 cleared+ADs-
+AH0- +AF8AXw-packed+ADs-
-struct nd+AF8-cmd+AF8-trans+AF8-spa +AHs-
- +AF8AXw-u64 spa+ADs-
- +AF8AXw-u32 status+ADs-
- +AF8AXw-u8 flags+ADs-
- +AF8AXw-u8 +AF8-reserved+AFs-3+AF0AOw-
- +AF8AXw-u64 trans+AF8-length+ADs-
- +AF8AXw-u32 num+AF8-nvdimms+ADs-
- struct nd+AF8-nvdimm+AF8-device +AHs-
- +AF8AXw-u32 nfit+AF8-device+AF8-handle+ADs-
- +AF8AXw-u32 +AF8-reserved+ADs-
- +AF8AXw-u64 dpa+ADs-
- +AH0- +AF8AXw-packed devices+AFs-0+AF0AOw-
-
-+AH0- +AF8AXw-packed+ADs-
-
-struct nd+AF8-cmd+AF8-ars+AF8-err+AF8-inj +AHs-
- +AF8AXw-u64 err+AF8-inj+AF8-spa+AF8-range+AF8-base+ADs-
- +AF8AXw-u64 err+AF8-inj+AF8-spa+AF8-range+AF8-length+ADs-
- +AF8AXw-u8 err+AF8-inj+AF8-options+ADs-
- +AF8AXw-u32 status+ADs-
-+AH0- +AF8AXw-packed+ADs-
-
-struct nd+AF8-cmd+AF8-ars+AF8-err+AF8-inj+AF8-clr +AHs-
- +AF8AXw-u64 err+AF8-inj+AF8-clr+AF8-spa+AF8-range+AF8-base+ADs-
- +AF8AXw-u64 err+AF8-inj+AF8-clr+AF8-spa+AF8-range+AF8-length+ADs-
- +AF8AXw-u32 status+ADs-
-+AH0- +AF8AXw-packed+ADs-
-
-struct nd+AF8-cmd+AF8-ars+AF8-err+AF8-inj+AF8-stat +AHs-
- +AF8AXw-u32 status+ADs-
- +AF8AXw-u32 inj+AF8-err+AF8-rec+AF8-count+ADs-
- struct nd+AF8-error+AF8-stat+AF8-query+AF8-record +AHs-
- +AF8AXw-u64 err+AF8-inj+AF8-stat+AF8-spa+AF8-range+AF8-base+ADs-
- +AF8AXw-u64 err+AF8-inj+AF8-stat+AF8-spa+AF8-range+AF8-length+ADs-
- +AH0- +AF8AXw-packed record+AFs-0+AF0AOw-
-+AH0- +AF8AXw-packed+ADs-
-
enum +AHs-
ND+AF8-CMD+AF8-IMPLEMENTED +AD0- 0,