Hi,
> On Fri, Feb 9, 2018 at 12:02 AM, QI Fuli wrote:
> > This patch is used to add $ndctl create-monitor command, by which users can
> > create a new monitor. Users can select the DIMMS to be monitored by using
> > [--dimm] [--bus] [--region] [--namespace] options. The notifications can
> > be o
> Hi,
>
> > On Fri, Feb 9, 2018 at 12:02 AM, QI Fuli wrote:
> > > This patch is used to add $ndctl create-monitor command, by which users
> > > can
> > > create a new monitor. Users can select the DIMMS to be monitored by using
> > > [--dimm] [--bus] [--region] [--namespace] options. The notific
Dan Williams writes:
> On Mon, Feb 12, 2018 at 2:53 PM, Jeff Moyer wrote:
>> Dave Jiang writes:
>>
>>> Re-enable deep flush so that users always have a way to be sure that a write
>>> does make it all the way out to the NVDIMM. The PMEM driver writes always
>>> make it "all the way to the NVDIM
Hi, Dave,
Dave Jiang writes:
> Adding firmware output of firmware information when ndctl list -D -F is used.
> Components displayed are current firmware version, updated firmware version,
> and if a coldboot is required (firmware updated).
>
> Signed-off-by: Dave Jiang
> ---
> Documentation/nd
Looks good:
Reviewed-by: Christoph Hellwig
Can you wire up your test cases for blktests?
___
Linux-nvdimm mailing list
Linux-nvdimm@lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm
On Tue, Feb 13, 2018 at 5:17 AM, Jeff Moyer wrote:
> Dan Williams writes:
>
>> On Mon, Feb 12, 2018 at 2:53 PM, Jeff Moyer wrote:
>>> Dave Jiang writes:
>>>
Re-enable deep flush so that users always have a way to be sure that a
write
does make it all the way out to the NVDIMM. T
Adding firmware output of firmware information when ndctl list -D -F is used.
Components displayed are current firmware version, updated firmware version,
and if a coldboot is required (firmware updated).
Signed-off-by: Dave Jiang
---
v2:
- Added copyright
- Added support for human readable opti
Dave Jiang writes:
> Adding firmware output of firmware information when ndctl list -D -F is used.
> Components displayed are current firmware version, updated firmware version,
> and if a coldboot is required (firmware updated).
So now I don't get any output if I run:
ndctl list -DF
on an NVD
On Tue, Feb 13, 2018 at 11:03 AM, Jeff Moyer wrote:
> Dave Jiang writes:
>
>> Adding firmware output of firmware information when ndctl list -D -F is used.
>> Components displayed are current firmware version, updated firmware version,
>> and if a coldboot is required (firmware updated).
>
> So n
On Tue, Feb 13, 2018 at 03:54:04PM +0100, Christoph Hellwig wrote:
> Looks good:
>
> Reviewed-by: Christoph Hellwig
>
> Can you wire up your test cases for blktests?
Is blktests really the right place for this test? This failure is highly
dependent on the configuration of the filesystem that i
On 02/13/2018 12:03 PM, Jeff Moyer wrote:
> Dave Jiang writes:
>
>> Adding firmware output of firmware information when ndctl list -D -F is used.
>> Components displayed are current firmware version, updated firmware version,
>> and if a coldboot is required (firmware updated).
>
> So now I do
Adding firmware output of firmware information when ndctl list -D -F is used.
Components displayed are current firmware version, updated firmware version,
and if a coldboot is required (firmware updated).
Signed-off-by: Dave Jiang
---
v3:
- Fixed issue where it skips displaying rest of the detail
Dave Jiang writes:
> Adding firmware output of firmware information when ndctl list -D -F is used.
> Components displayed are current firmware version, updated firmware version,
> and if a coldboot is required (firmware updated).
>
> Signed-off-by: Dave Jiang
Looks good to me! Thanks!
Tested-
On Thu, Feb 08, 2018 at 05:38:19PM -0700, Dave Jiang wrote:
> The function return values are confusing with the way the function is
> named. We expect a true or false return value but it actually returns
> 0/-errno. This makes the code very confusing. Changing the return values
> to return a bool
On 02/13/2018 01:31 PM, Ross Zwisler wrote:
> On Thu, Feb 08, 2018 at 05:38:19PM -0700, Dave Jiang wrote:
>> The function return values are confusing with the way the function is
>> named. We expect a true or false return value but it actually returns
>> 0/-errno. This makes the code very confus
On Tue, Feb 13, 2018 at 01:35:19PM -0700, Dave Jiang wrote:
>
>
> On 02/13/2018 01:31 PM, Ross Zwisler wrote:
> > On Thu, Feb 08, 2018 at 05:38:19PM -0700, Dave Jiang wrote:
> >> The function return values are confusing with the way the function is
> >> named. We expect a true or false return val
On Tue, Feb 13, 2018 at 11:22 AM, Ross Zwisler
wrote:
> On Tue, Feb 13, 2018 at 03:54:04PM +0100, Christoph Hellwig wrote:
>> Looks good:
>>
>> Reviewed-by: Christoph Hellwig
>>
>> Can you wire up your test cases for blktests?
>
> Is blktests really the right place for this test? This failure is
On Tue, Feb 13, 2018 at 11:41 AM, Dave Jiang wrote:
> Adding firmware output of firmware information when ndctl list -D -F is used.
> Components displayed are current firmware version, updated firmware version,
> and if a coldboot is required (firmware updated).
>
> Signed-off-by: Dave Jiang
> --
Adding firmware output of firmware information when ndctl list -D -F is used.
Components displayed are current firmware version, updated firmware version,
and if a coldboot is required (firmware updated).
Signed-off-by: Dave Jiang
Tested-by: Jeff Moyer
---
v4:
- Remove output when updated_versio
19 matches
Mail list logo