Re: [v6,2/8] dmaengine: Add DMA_MEMCPY_SG transaction op

Hi Vinod, On 31.08.2017 12:57, Robin Murphy wrote: On 30/08/17 19:25, Dave Jiang wrote: On 08/30/2017 11:18 AM, Robin Murphy wrote: On 25/08/17 21:59, Dave Jiang wrote: Adding a dmaengine transaction operation that allows copy to/from a scatterlist and a flat buffer. Apologies if I'm late

[RFC patch v2 0/7] ndctl: nvdimmd: notify/monitor the feathers of over threshold event

Hi, here is my second version of nvdimm daemon, It would be appreciated if you could check it. Change log since v1:  - Adding a config file(/etc/nvdimmd/nvdimmd.conf)  - Using struct log_ctx instead of syslog()    - Using log_syslog() to save the notify messages to syslog    - Using log_file()

Re: [PATCH] fs, dax: unify IOMAP_F_DIRTY read vs write handling policy in the dax core

On Mon, Nov 13, 2017 at 05:27:54PM -0800, Dan Williams wrote: > While reviewing whether MAP_SYNC should strengthen its current guarantee > of syncing writes from the initiating process to also include > third-party readers observing dirty metadata, Dave pointed out that the > check of IOMAP_WRITE

Re: [PATCH v2 1/2] ndctl: daxctl: fix mmap size

On 11/13/2017 04:40 PM, Dan Williams wrote: > On Mon, Nov 13, 2017 at 10:19 AM, Dave Jiang wrote: >> The size for mmap needs to be aligned to the region alignment. Fix the >> mapping size so that it satisfy the alignment requirement. >> >> Signed-off-by: Dave Jiang

Delivery reports about your e-mail

The original message was received at Tue, 14 Nov 2017 11:52:38 +0800 from lists.01.org [47.11.223.161] - The following addresses had permanent fatal errors - ___ Linux-nvdimm mailing list Linux-nvdimm@lists.01.org

Re: [PATCH v3 1/2] ndctl: daxctl: fix mmap size

On Mon, Nov 13, 2017 at 3:58 PM, Dave Jiang wrote: > The size for mmap needs to be aligned to the region alignment. Fix the > mapping size so that it satisfy the alignment requirement. > > Signed-off-by: Dave Jiang Thanks Dave, applied both.

Re: [PATCH v2 1/2] ndctl: daxctl: fix mmap size

On Mon, Nov 13, 2017 at 10:19 AM, Dave Jiang wrote: > The size for mmap needs to be aligned to the region alignment. Fix the > mapping size so that it satisfy the alignment requirement. > > Signed-off-by: Dave Jiang > --- > daxctl/io.c | 12

[PATCH v3 1/2] ndctl: daxctl: fix mmap size

The size for mmap needs to be aligned to the region alignment. Fix the mapping size so that it satisfy the alignment requirement. Signed-off-by: Dave Jiang --- daxctl/io.c | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/daxctl/io.c

[PATCH v3 2/2] ndctl: make mmap start at the offset

Instead of mmap at the beginning of the DAX region, map at the offset that is aligned down. This reduces the mmap size we need to create. Signed-off-by: Dave Jiang --- daxctl/io.c | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git

[PATCH] fs, dax: unify IOMAP_F_DIRTY read vs write handling policy in the dax core

While reviewing whether MAP_SYNC should strengthen its current guarantee of syncing writes from the initiating process to also include third-party readers observing dirty metadata, Dave pointed out that the check of IOMAP_WRITE is misplaced. The policy of what to with IOMAP_F_DIRTY should be

Re: [PATCH 17/18] xfs: Implement xfs_filemap_pfn_mkwrite() using __xfs_filemap_fault()

On Wed, Nov 01, 2017 at 04:36:46PM +0100, Jan Kara wrote: > xfs_filemap_pfn_mkwrite() duplicates a lot of __xfs_filemap_fault(). > It will also need to handle flushing for synchronous page faults. So > just make that function use __xfs_filemap_fault(). > > Signed-off-by: Jan Kara

Re: [PATCH] fs, dax: unify IOMAP_F_DIRTY read vs write handling policy in the dax core

> - if ((flags & IOMAP_WRITE) && xfs_ipincount(ip) && > - (ip->i_itemp->ili_fsync_fields & ~XFS_ILOG_TIMESTAMP)) > - iomap->flags |= IOMAP_F_DIRTY; > + if (xfs_ipincount(ip)) > + if (ip->i_itemp->ili_fsync_fields & ~XFS_ILOG_TIMESTAMP) > +

[RFC PATCH v2 4/7] ndctl: nvdimmd: add body file of nvdimm daemon

This patch adds the body file of nvdimm daemon and compiles nvdimmd with automake. Signed-off-by: QI Fuli --- Makefile.am | 2 +- configure.ac| 1 + nvdimmd/Makefile.am | 19 + nvdimmd/nvdimmd.c | 112

[RFC PATCH v2 7/7] ndctl: nvdimmd: add a temporary test for nvdimm daemon

This patch is used to test nvdimmd service temporarily. I will make it an ndctl command or an option of ndctl inject-error in next step. Currently, You can test nvdimm daemon with the following command. nvdimmd_test [nmemX] [all] Signed-off-by: QI Fuli ---

[RFC PATCH v2 0/7] ndctl: nvdimmd: notify/monitor the feathers of over threshold event

Hi, here is my second version of nvdimm daemon, It would be appreciated if you could check it. Change log since v1: - Adding a config file(/etc/nvdimmd/nvdimmd.conf) - Using struct log_ctx instead of syslog() - Using log_syslog() to save the notify messages to syslog - Using log_file() to

[RFC PATCH v2 1/7] ndctl: nvdimmd: add LOG_NOTICE level into log_priority

This patch adds LOG_NOTICE level into log_priority for the notifications. Because the LOG_INFO level is too low for the notifications of nvdimm dimm over threshold event, and the event notifications are not up to LOG_ERROR level. Signed-off-by: QI Fuli --- util/log.c | 2

[RFC PATCH v2 3/7] ndctl: nvdimmd: add nvdimmd necessary functions

Libnvdimmd is used to provide nvdimmd necessary functions. The get_nvdimmd_conf() will read the nvdimmd.conf and set the parameters' value as global variables when nvdimmd service starts. Signed-off-by: QI Fuli --- nvdimmd/libnvdimmd.c | 315

[RFC PATCH v2 2/7] ndctl: nvdimmd: add nvdimmd necessary util functions

This patch is used to provide helper functions needed for nvdimm daemon. These util functions can be used by other features as well in the future. Signed-off-by: QI Fuli --- nvdimmd/util.c | 80 ++ nvdimmd/util.h |

[RFC PATCH v2 5/7] ndctl: nvdimmd: add nvdimmd config file

This patch adds nvdimmd.conf. Autotools to install the config file for nvdimmd service. The nvdimmd.conf includs the following parameters. - LOGFILE: Users can choose either to write the notifications into syslog (LOGFILE=syslog), or to send into a special file under /var/log/nvdimmd/

[RFC PATCH v2 6/7] ndctl: nvdimmd: add the unit file of systemd for nvdimmd service

This patch adds and autotools to install the unit file of systemd for nvdimmd service. Signed-off-by: QI Fuli --- nvdimmd/Makefile.am | 7 +++ nvdimmd/nvdimmd.service | 7 +++ 2 files changed, 14 insertions(+) create mode 100644 nvdimmd/nvdimmd.service diff

Re: [RFC patch v2 0/7] ndctl: nvdimmd: notify/monitor the feathers of over threshold event

I am sorry that Imadea mistake encoding this mail. I havesent a new one. Please ignore this mail. ___ Linux-nvdimm mailing list Linux-nvdimm@lists.01.org https://lists.01.org/mailman/listinfo/linux-nvdimm

Re: [PATCH 18/18] xfs: support for synchronous DAX faults

On Wed, Nov 01, 2017 at 04:36:47PM +0100, Jan Kara wrote: > From: Christoph Hellwig > > Return IOMAP_F_DIRTY from xfs_file_iomap_begin() when asked to prepare > blocks for writing and the inode is pinned, and has dirty fields other > than the timestamps. In __xfs_filemap_fault() we

Re: [PATCH] fs, dax: unify IOMAP_F_DIRTY read vs write handling policy in the dax core

On Mon, Nov 13, 2017 at 05:27:54PM -0800, Dan Williams wrote: > diff --git a/fs/xfs/xfs_iomap.c b/fs/xfs/xfs_iomap.c > index b43be199fbdf..888b60189983 100644 > --- a/fs/xfs/xfs_iomap.c > +++ b/fs/xfs/xfs_iomap.c > @@ -1087,9 +1087,9 @@ xfs_file_iomap_begin( >

[PATCH v2 1/2] ndctl: daxctl: fix mmap size

The size for mmap needs to be aligned to the region alignment. Fix the mapping size so that it satisfy the alignment requirement. Signed-off-by: Dave Jiang --- daxctl/io.c | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/daxctl/io.c

[PATCH v2 2/2] ndctl: make mmap start at the offset

Instead of mmap at the beginning of the DAX region, map at the offset that is aligned down. This reduces the mmap size we need to create. Signed-off-by: Dave Jiang --- daxctl/io.c | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git

[ndctl PATCH] libdaxctl: use the same uuid definition as libdaxctl

libdaxtl inadvertently included the uuid-devel version of the uuid.h header rather than the libuuid header. Fix this up to avoid type conflicts for applications that include both libdaxctl.h and libndctl.h. Reported-by: Lukasz Plewa Signed-off-by: Dan Williams