[ndctl PATCH v2] ndctl, list: add alarm_enable_ to list

2018-08-07 Thread QI Fuli
This patch adds alarm_enable_ to list, so that users could check if the "ndctl inject-smart ---alarm=on/off" works well or not. Signed-off-by: QI Fuli --- v1 -> v2: remove the renaming for list items. ndctl/util/json-smart.c | 27 +++ 1 file changed, 27 insertions(+)

Re: [PATCH V2 2/4] mm: introduce memory type MEMORY_DEVICE_DEV_DAX

2018-08-07 Thread Zhang,Yi
On 2018年08月07日 17:11, Jan Kara wrote: > On Wed 11-07-18 01:01:59, Zhang Yi wrote: >> Currently, NVDIMM pages will be marked 'PageReserved'. However, unlike >> other reserved PFNs, pages on NVDIMM shall still behave like normal ones >> in many cases, i.e. when used as backend memory of KVM guest.

[PATCH v3] ndctl, test: add a new unit test for max_available_extent namespace

2018-08-07 Thread Masayoshi Mizuma
From: Masayoshi Mizuma Add a new unit test to test max_available_extent namespace. This feature is implemented by the following patches. kernel side: https://lists.01.org/pipermail/linux-nvdimm/2018-July/016731.html https://lists.01.org/pipermail/linux-nvdimm/2018-July/016732.html

RE: [ndctl PATCH 2/4] ndctl, monitor: set default log destination to syslog if "--daemon" is specified

2018-08-07 Thread Qi, Fuli
> -Original Message- > From: Masayoshi Mizuma [mailto:msys.miz...@gmail.com] > Sent: Wednesday, August 8, 2018 4:30 AM > To: Qi, Fuli/斉 福利 ; linux-nvdimm@lists.01.org > Subject: Re: [ndctl PATCH 2/4] ndctl, monitor: set default log destination to > syslog > if "--daemon" is specified > >

Re: [ndctl PATCH 3/4] ndctl, monitor: add timestamp and pid to log messages in log_file()

2018-08-07 Thread Masayoshi Mizuma
On 08/07/2018 08:54 PM, Qi, Fuli wrote: >> -Original Message- >> From: Masayoshi Mizuma [mailto:msys.miz...@gmail.com] >> Sent: Wednesday, August 8, 2018 3:40 AM >> To: Qi, Fuli/斉 福利 ; linux-nvdimm@lists.01.org >> Subject: Re: [ndctl PATCH 3/4] ndctl, monitor: add timestamp and pid to

RE: [ndctl PATCH 1/4] ndctl, monitor: fix the lack of detection of invalid path of log file

2018-08-07 Thread Qi, Fuli
> -Original Message- > From: Masayoshi Mizuma [mailto:msys.miz...@gmail.com] > Sent: Wednesday, August 8, 2018 4:06 AM > To: Qi, Fuli/斉 福利 ; linux-nvdimm@lists.01.org > Subject: Re: [ndctl PATCH 1/4] ndctl, monitor: fix the lack of detection of > invalid > path of log file > > Hi Qi, >

RE: [ndctl PATCH] ndctl, list: emit alarm_enable_ and clarify misc list items

2018-08-07 Thread Qi, Fuli
Hi Vishal, Thanks for your comments. > -Original Message- > From: Verma, Vishal L [mailto:vishal.l.ve...@intel.com] > Sent: Wednesday, August 8, 2018 5:15 AM > To: linux-nvdimm@lists.01.org; Qi, Fuli/斉 福利 > Subject: Re: [ndctl PATCH] ndctl, list: emit alarm_enable_ and clarify > misc >

RE: [ndctl PATCH 3/4] ndctl, monitor: add timestamp and pid to log messages in log_file()

2018-08-07 Thread Qi, Fuli
> -Original Message- > From: Masayoshi Mizuma [mailto:msys.miz...@gmail.com] > Sent: Wednesday, August 8, 2018 3:40 AM > To: Qi, Fuli/斉 福利 ; linux-nvdimm@lists.01.org > Subject: Re: [ndctl PATCH 3/4] ndctl, monitor: add timestamp and pid to log > messages > in log_file() > > Hi Qi, > >

Re: [RFC PATCH 1/1] device-dax: check for vma range while dax_mmap.

2018-08-07 Thread Dave Jiang
On 08/02/2018 02:32 AM, Zhang,Yi wrote: > > > On 2018年08月02日 03:40, Dave Jiang wrote: >> >> On 07/31/2018 04:46 AM, Zhang Yi wrote: >>> It should be prevent user map an illegal vma range which larger than >>> dax device phiscal resourse, as we don't have swap logic while page >>> faulting in

Re: [PATCH 1/2] Create Intel pmem shutdown latch rule

2018-08-07 Thread Verma, Vishal L
On Tue, 2018-08-07 at 16:26 -0600, Keith Busch wrote: > This patch provides a udev rule and the program it runs for Intel > nvdimms. The program the rule executes runs two commands to allow > applications to manage unsafe shutdowns. > > The first command enables the shutdown latch. Without

Re: [PATCH 2/2] ndctl, intel: Fallback to smart cached shutdown_count

2018-08-07 Thread Verma, Vishal L
On Tue, 2018-08-07 at 16:26 -0600, Keith Busch wrote: > A user space rule saves the unsafe shutdown count to a fixed location > for Intel dimms so that non-root users can read it. If the smart command > submission fails, this patch uses that saved value in a newly added > ndctl_cmd

Re: [PATCH 1/2] Create Intel pmem shutdown latch rule

2018-08-07 Thread Verma, Vishal L
On Tue, 2018-08-07 at 16:26 -0600, Keith Busch wrote: > This patch provides a udev rule and the program it runs for Intel Like we talked about, I think this should work for all NVDIMMs (at least any that use the generic DIMM ops APIs that libndctl provides), so we can remove any Intel

Re: [PATCH] ACPI: nfit: adjust annotation for why return 0 if fail to find NFIT at startup

2018-08-07 Thread Dave Jiang
On 08/07/2018 12:25 PM, Verma, Vishal L wrote: > > On Tue, 2018-08-07 at 02:15 -0400, Ocean He wrote: >> From: Ocean He > > Just some minor nits in commit message wording: > >> >> Add detail explanation why it's ok to return 0 if fail to find NFIT at > detailed ^for

Re: [PATCH] libnvdimm, bus: check id immediately following ida_simple_get

2018-08-07 Thread Dave Jiang
On 08/03/2018 05:08 AM, Ocean He wrote: > From: Ocean He > > The id check was not executed immediately following ida_simple_get. Just > change the codes position, without function change. > > Signed-off-by: Ocean He Applied. > --- > drivers/nvdimm/bus.c | 4 ++-- > 1 file changed, 2

[PATCH 2/2] ndctl, intel: Fallback to smart cached shutdown_count

2018-08-07 Thread Keith Busch
A user space rule saves the unsafe shutdown count to a fixed location for Intel dimms so that non-root users can read it. If the smart command submission fails, this patch uses that saved value in a newly added ndctl_cmd "handle_error" callback. If the cached value is available, the error handler

[PATCH 1/2] Create Intel pmem shutdown latch rule

2018-08-07 Thread Keith Busch
This patch provides a udev rule and the program it runs for Intel nvdimms. The program the rule executes runs two commands to allow applications to manage unsafe shutdowns. The first command enables the shutdown latch. Without this, the last shutdown status will remain fixed to the status set

Re: [PATCH 1/2] Create Intel pmem shutdown latch rule

2018-08-07 Thread Keith Busch
Oops, please ignore this set. Had the send-email pointing to the wrong version of this patch. Will send the correct one in a moment. ___ Linux-nvdimm mailing list Linux-nvdimm@lists.01.org https://lists.01.org/mailman/listinfo/linux-nvdimm

[PATCH 2/2] ndctl, intel: Pre-initialize smart shutdown_count

2018-08-07 Thread Keith Busch
A user space rule saves the unsafe shutdown count to a fixed location for Intel dimms so that non-root users can read it. This patch uses that saved value if it is available in case the command can't be executed due to user permissions, allowing an application to detect if an unsafe shutdown

[PATCH 1/2] Create Intel pmem shutdown latch rule

2018-08-07 Thread Keith Busch
This patch provides a udev rule and the program it runs for Intel nvdimms. The program the rule executes runs two commands to allow applications to manage unsafe shutdowns. The first command enables the shutdown latch. Without this, the last shutdown status will remain fixed to the status set

[PATCH 1/2] ext4: Close race between direct IO and ext4_break_layouts()

2018-08-07 Thread Dave Jiang
From: Ross Zwisler If the refcount of a page is lowered between the time that it is returned by dax_busy_page() and when the refcount is again checked in ext4_break_layouts() => ___wait_var_event(), the waiting function ext4_wait_dax_page() will never be called. This means that

[PATCH 2/2] [PATCH] xfs: Close race between direct IO and xfs_break_layouts()

2018-08-07 Thread Dave Jiang
This patch is the duplicate of ross's fix for ext4 for xfs. If the refcount of a page is lowered between the time that it is returned by dax_busy_page() and when the refcount is again checked in xfs_break_layouts() => ___wait_var_event(), the waiting function xfs_wait_dax_page() will never be

Re: [ndctl PATCH] ndctl, list: emit alarm_enable_ and clarify misc list items

2018-08-07 Thread Verma, Vishal L
On Wed, 2018-08-08 at 00:35 +0900, QI Fuli wrote: > This patch adds alarm_enable_ to list. Therefore, users can know Hi Qi, Thanks, I was meaning to do this work but you beat me to it :) I just have a few nits, see below. But otherwise this looks good. > if the "ndclt inject-smart

Hello!

2018-08-07 Thread androulla
Dear Job Seeker, Position Name: Project Manager (Supply Chain); Company: DSC Logistics Functional area: Logistics Services; Area of Interest: Logistics Services; Salary: $98,000/year; Job Offer Available: 15th August 2018; Job Type: Permanent Position; Location: USA/15 States (can be

Re: [ndctl PATCH 2/4] ndctl, monitor: set default log destination to syslog if "--daemon" is specified

2018-08-07 Thread Masayoshi Mizuma
Hi Qi, On 08/07/2018 09:17 AM, QI Fuli wrote: > When running monitor as a daemon, if the log destination is "standard" or > a relative path for log file, the messages will not be able to be logged. > Sometimes, users may not notice that the default log destination is "standard" > when they start

Re: [PATCH] ndctl, documentation: Clarify the dimm id for ndctl list d option

2018-08-07 Thread Verma, Vishal L
On Mon, 2018-08-06 at 17:21 -0400, Masayoshi Mizuma wrote: > From: Masayoshi Mizuma > > User may be confused the dimm id description of ndctl list -d > option because ndctl list says id, but the id is not useful to > d option. > > ]# ndctl list -d nmem7 | jq -r .id > cdab-0a-07e0-fefffeff >

Re: [PATCH] ACPI: nfit: adjust annotation for why return 0 if fail to find NFIT at startup

2018-08-07 Thread Verma, Vishal L
On Tue, 2018-08-07 at 02:15 -0400, Ocean He wrote: > From: Ocean He Just some minor nits in commit message wording: > > Add detail explanation why it's ok to return 0 if fail to find NFIT at detailed ^for ^we ^an > startup, refer to chapter

Re: [ndctl PATCH 1/4] ndctl, monitor: fix the lack of detection of invalid path of log file

2018-08-07 Thread Masayoshi Mizuma
Hi Qi, On 08/07/2018 09:17 AM, QI Fuli wrote: > Currently the monitor can be started even with an invalid path of log file. > This patch adds a detection of invalid path of log file when starting monitor. > If the path of log file is invalid, the monitor will refuse to be started. > >

Re: [ndctl PATCH 4/4] ndctl, monitor: add [Install] Section to systemd unit file of ndctl-monitor

2018-08-07 Thread Masayoshi Mizuma
Hi Qi, Looks good to me. Please free to add: Reviewed-by: Masayoshi Mizuma Thanks, Masa On 08/07/2018 09:17 AM, QI Fuli wrote: > This patch is used to add "[Install]" section to systemd unit file of > ndctl-monitor. Therefore, users can set ndctl-monitor unit enable. > > Signed-off-by:

Re: [ndctl PATCH 3/4] ndctl, monitor: add timestamp and pid to log messages in log_file()

2018-08-07 Thread Masayoshi Mizuma
Hi Qi, On 08/07/2018 09:17 AM, QI Fuli wrote: > This patch is used to add timestamp and process id to log messages when > logging > messages to a file. > > Signed-off-by: QI Fuli > --- > ndctl/monitor.c | 11 ++- > 1 file changed, 10 insertions(+), 1 deletion(-) > > diff --git

转发:<合伙人制度> 8月11日·苏州站

2018-08-07 Thread 徐主任
转发邮件信息 发件人:nkk...@gxw.net 发送日期:2018-8-8 0:35:13 收件人:linux-nvdimm@lists.01.org 全国畅销书<合伙人制度> 企业家,创业者,成功者创业路上必看! 作者:郑指梁老师来昆山现场指导了!! 邀请对象:企业董事长、总经理、集团总裁、资源大咖、投资大咖、创业者、有项目需要合伙者等。 企业是否面临以下难题: 1、在企业发展过程中优秀员工与高管不断流失,外部人才难进入,如何通过合伙人制度解决? 2、在企业初创期如何实现新员工梦想、调动老员工动力、解决企业元老发挥余热?

[ndctl PATCH] ndctl, list: emit alarm_enable_ and clarify misc list items

2018-08-07 Thread QI Fuli
This patch adds alarm_enable_ to list. Therefore, users can know if the "ndclt inject-smart ---alarm=on/off" works or not. Users may confuse "alarm_enable" with "alarm_flag" and "media_temperature" with "ctrl_temperature", this patch also used to clarify these items. Signed-off-by: QI Fuli ---

[ndctl PATCH 4/4] ndctl, monitor: add [Install] Section to systemd unit file of ndctl-monitor

2018-08-07 Thread QI Fuli
This patch is used to add "[Install]" section to systemd unit file of ndctl-monitor. Therefore, users can set ndctl-monitor unit enable. Signed-off-by: QI Fuli --- ndctl/ndctl-monitor.service | 3 +++ 1 file changed, 3 insertions(+) diff --git a/ndctl/ndctl-monitor.service

[ndctl PATCH 2/4] ndctl, monitor: set default log destination to syslog if "--daemon" is specified

2018-08-07 Thread QI Fuli
When running monitor as a daemon, if the log destination is "standard" or a relative path for log file, the messages will not be able to be logged. Sometimes, users may not notice that the default log destination is "standard" when they start monitor daemon by systemctl, so they will lose

[ndctl PATCH 3/4] ndctl, monitor: add timestamp and pid to log messages in log_file()

2018-08-07 Thread QI Fuli
This patch is used to add timestamp and process id to log messages when logging messages to a file. Signed-off-by: QI Fuli --- ndctl/monitor.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/ndctl/monitor.c b/ndctl/monitor.c index 2f3d751..d29e378 100644 ---

[ndctl PATCH 1/4] ndctl, monitor: fix the lack of detection of invalid path of log file

2018-08-07 Thread QI Fuli
Currently the monitor can be started even with an invalid path of log file. This patch adds a detection of invalid path of log file when starting monitor. If the path of log file is invalid, the monitor will refuse to be started. Signed-off-by: QI Fuli --- ndctl/monitor.c | 11 ++- 1

[ndctl PATCH 0/4] Some fixups and improvements ndctl monitor

2018-08-07 Thread QI Fuli
Fix a couple of issues about log collection, and add [Install] section to systemd unit file of ndclt-monitor. Signed-off-by: QI Fuli --- QI Fuli (4): ndctl, monitor: fix the lack of detection of invalid path of log file ndctl, monitor: set default log destination to syslog if "--daemon" is

Re: [PATCH V2 3/4] mm: add a function to differentiate the pages is from DAX device memory

2018-08-07 Thread Jan Kara
On Wed 11-07-18 01:03:51, Zhang Yi wrote: > DAX driver hotplug the device memory and move it to memory zone, these > pages will be marked reserved flag, however, some other kernel componet > will misconceive these pages are reserved mmio (ex: we map these dev_dax > or fs_dax pages to kvm for

Re: [PATCH V2 2/4] mm: introduce memory type MEMORY_DEVICE_DEV_DAX

2018-08-07 Thread Jan Kara
On Wed 11-07-18 01:01:59, Zhang Yi wrote: > Currently, NVDIMM pages will be marked 'PageReserved'. However, unlike > other reserved PFNs, pages on NVDIMM shall still behave like normal ones > in many cases, i.e. when used as backend memory of KVM guest. This patch > introduces a new memory type,

Re: [PATCH v4 0/2] ext4: fix DAX dma vs truncate/hole-punch

2018-08-07 Thread Jan Kara
On Fri 27-07-18 10:28:51, Ross Zwisler wrote: > + fsdevel and the xfs list. > > On Wed, Jul 25, 2018 at 4:28 PM Ross Zwisler > wrote: > > On Wed, Jul 11, 2018 at 10:17:41AM +0200, Jan Kara wrote: > > > On Tue 10-07-18 13:10:29, Ross Zwisler wrote: > > > > Changes since v3: > > > > * Added an

[PATCH] ACPI: nfit: adjust annotation for why return 0 if fail to find NFIT at startup

2018-08-07 Thread Ocean He
From: Ocean He Add detail explanation why it's ok to return 0 if fail to find NFIT at startup, refer to chapter 9.20.2 NVDIMM Root Device in ACPI 6.2 spec. Signed-off-by: Ocean He --- v1: https://lists.01.org/pipermail/linux-nvdimm/2018-August/017311.html v2: Per Vishal's comments of v1, it