Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1723679

[GIT PULL] libnvdimm fix for 4.13-final

From "Williams, Dan J" <dan.j.williams@intel.com>
Newsgroups linux.kernel
Subject [GIT PULL] libnvdimm fix for 4.13-final
Date 2017-08-30 23:40 +0200
Message-ID <ukiSn-5d9-31@gated-at.bofh.it> (permalink)
Organization linux.* mail to news gateway

Show all headers | View raw


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,
 

Back to linux.kernel | Previous | Next | Find similar | Unroll thread


Thread

[GIT PULL] libnvdimm fix for 4.13-final "Williams, Dan J" <dan.j.williams@intel.com> - 2017-08-30 23:40 +0200

csiph-web