Re: [PATCH 00/10] Add support for OpenCAPI SCM devices

2019-10-25 Thread Geert Uytterhoeven
Hi Alastair, On Fri, Oct 25, 2019 at 6:48 AM Alastair D'Silva wrote: > From: Alastair D'Silva > > This series adds support for OpenCAPI SCM devices, exposing > them as nvdimms so that we can make use of the existing > infrastructure. Thanks for your series! The long CC list is a sign of

DHL PACKAGE DELIVERY

2019-10-25 Thread DHL DELIVERY SERVICE
=D H L= Dear Linux-nvdimm,Your business partner sent you a package sent to you via our courier service。Before we start the final delivery to your address, we need to confirm that you are the actual recipient.。Please click below to confirm your shipping address with us to ensure smooth and fast

[PATCH] uapi: Add the BSD-2-Clause license to ndctl.h

2019-10-25 Thread D Scott Phillips
Allow ndctl.h to be licensed with BSD-2-Clause so that other operating systems can provide the same user level interface. --- I've been working on nvdimm support in FreeBSD and would like to offer the same ndctl API there to ease porting of application code. Here I'm proposing to add the

RE: [PATCH] uapi: Add the BSD-2-Clause license to ndctl.h

2019-10-25 Thread Dexuan Cui
> From: D Scott Phillips > Sent: Friday, October 25, 2019 10:56 AM > To: Dan Williams ; David Howells > ; Dexuan Cui ; Jerry > Hoemann ; stuart hayes > ; Toshi Kani ; Vishal Verma > ; linux-nvdimm@lists.01.org > Cc: linux-ker...@vger.kernel.org > Subject: [PATCH] uapi: Add the BSD-2-Clause

Re: [PATCH] uapi: Add the BSD-2-Clause license to ndctl.h

2019-10-25 Thread Dan Williams
On Fri, Oct 25, 2019 at 10:55 AM D Scott Phillips wrote: > > Allow ndctl.h to be licensed with BSD-2-Clause so that other > operating systems can provide the same user level interface. > --- > > I've been working on nvdimm support in FreeBSD and would like to > offer the same ndctl API there to

Re: [ndctl patch 3/4] query_fw_finish_status: get rid of redundant variable

2019-10-25 Thread Verma, Vishal L
On Fri, 2019-10-25 at 15:21 -0700, Ira Weiny wrote: > How about this patch instead? Untested. > > Ira Not a big deal, but just a quick note - if you include a scissors line here, I can easily apply it via git am --scissors --8<-- Otherwise this looks good in principle. I've already got

[GIT PULL] dax fix for v5.4-rc5

2019-10-25 Thread Dan Williams
Hi Linus, please pull from: git://git.kernel.org/pub/scm/linux/kernel/git/nvdimm/nvdimm tags/dax-fix-5.4-rc5 ...to receive a regression fix for v5.4-rc5. It has appeared in a -next release with no reported issues, and picked up reviews from the regular dax contributors. --- The following

Re: [ndctl patch 3/4] query_fw_finish_status: get rid of redundant variable

2019-10-25 Thread Ira Weiny
On Wed, Oct 23, 2019 at 03:51:21PM -0700, 'Vishal Verma' wrote: > On Wed, 2019-10-23 at 22:28 +, Verma, Vishal L wrote: > > On Fri, 2019-10-18 at 17:06 -0400, Jeff Moyer wrote: > > > Ira Weiny writes: > > > > On Fri, Oct 18, 2019 at 04:23:01PM -0400, Jeff Moyer wrote: > > > > > The 'done'

[ndctl PATCH 2/2] test/dax.sh: Split into ext4 and xfs tests

2019-10-25 Thread Dan Williams
Given the test can fail based on the fs make it a passed in parameter and split the tests. Signed-off-by: Dan Williams --- test/Makefile.am |3 +- test/dax-ext4.sh |1 + test/dax-xfs.sh |1 + test/dax.sh | 101 +++--- 4 files

[ndctl PATCH 1/2] test/dax.sh: Make dax.sh more robust vs small namespaces

2019-10-25 Thread Dan Williams
If the namespace returned by test/dax-dev is too small ext4 may default to a 1K block-size. A 1K block-size precludes dax operation, so force a 4K block-size in all cases. Signed-off-by: Dan Williams --- test/dax.sh |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [ndctl PATCH 3/4] test/dax.sh: Validate huge page mappings

2019-10-25 Thread Verma, Vishal L
> On Wed, 2019-10-23 at 19:33 +, Verma, Vishal L wrote: > > > @@ -91,4 +111,4 @@ json=$($NDCTL create-namespace -m raw -f -e $dev) > > > eval $(json2var <<< "$json") > > > [ $mode != "fsdax" ] && echo "fail: $LINENO" && exit 1 > > > > same comment about quoting "$mode". If 'mode' happens

Re: [PATCH] uapi: Add the BSD-2-Clause license to ndctl.h

2019-10-25 Thread Verma, Vishal L
On Fri, 2019-10-25 at 15:45 -0700, Dan Williams wrote: > On Fri, Oct 25, 2019 at 10:55 AM D Scott Phillips > wrote: > > Allow ndctl.h to be licensed with BSD-2-Clause so that other > > operating systems can provide the same user level interface. > > --- > > > > I've been working on nvdimm

Re: [ndctl PATCH 3/4] test/dax.sh: Validate huge page mappings

2019-10-25 Thread Dan Williams
On Fri, Oct 25, 2019 at 3:44 PM Verma, Vishal L wrote: > > > On Wed, 2019-10-23 at 19:33 +, Verma, Vishal L wrote: > > > > @@ -91,4 +111,4 @@ json=$($NDCTL create-namespace -m raw -f -e $dev) > > > > eval $(json2var <<< "$json") > > > > [ $mode != "fsdax" ] && echo "fail: $LINENO" && exit