Christoph Hellwig writes:
> Dan,
>
> can you make any sense of thos report?
>
> name='nfit_test'
> path='/lib/modules/5.11.0-rc5-3-g52f019d43c22/extra/test/nfit_test.ko'
>> check_set_config_data: dimm: 0 read2 data miscompare: 0
>> check_set_config_data: dimm: 0x1 read2 data miscompare: 0
>>
Michal Suchanek writes:
> There are cases not covered by the original fix and cases added by the
> latter patch.
>
> Also there is one case of usage added without returning from the
> function.
>
> Fixes: ff434d87ccbd ("dimm: fix potential fd leakage in dimm_action()")
> Fixes: 41a7e24af5db ("ndc
mode":"raw",
"size":0,
"uuid":"----",
"sector_size":512,
"state":"disabled"
}
This patch skips over namespaces with size=0 when zeroing out info
blocks.
Fixes: 46654c2d60b70
!test_bit(family, &nd_desc->bus_family_mask))
> return -EINVAL;
> + family = array_index_nospec(family,
> + NVDIMM_BUS_FAMILY_MAX + 1);
> dsm_mask = acpi_desc->family_dsm_mask[family];
>
ctl/namespace.c | 23 ++-
> test/libndctl.c| 16 +++-
> test/parent-uuid.c | 2 +-
> util/help.c| 8 +++-
> util/json.c| 3 +++
> 9 files changed, 59 insertions(+), 17 deletions(-)
Except
util_display_json_array(actx.f_out, actx.jdimms, flags);
> }
>
> - if (actx.f_out != stdout)
> - fclose(actx.f_out);
> -
> + out_close_fin_fout:
> if (actx.f_in != stdin)
> fclose(actx.f_in);
>
> + out_close_fout:
> + if
%s/%s/size", base, attrs);
> if (sysfs_read_attr(ctx, path, buf) == 0)
> @@ -314,6 +316,7 @@ static struct daxctl_region *add_dax_region(void *parent,
> int id,
> err_read:
> free(region->region_buf);
> free(region->
d_bus(void *parent, int id, const char
> *ctl_base)
> free(bus->wait_probe_path);
> free(bus->scrub_path);
> free(bus->provider);
> + free(bus->bus_path);
> free(bus->bus_buf);
> free(bus);
> err_bus:
Acked-by: Jeff Moyer
rt is pre-v1.2 namespace labels
Minor inconsistency in the last hunk. The empty line should come after
the return, no?
Other than that, LGTM.
Acked-by: Jeff Moyer
___
Linux-nvdimm mailing list -- linux-nvdimm@lists.01.org
To unsubscribe send an email to linux-nvdimm-le...@lists.01.org
Dan Williams writes:
>> What problems with 5.4.y and 5.6.y is this series fixing
>> that used to work before?
>
> The "used to work" bug fixed by this set is the fact that the kernel
> used to force a 128MB (memory hotplug section size) alignment padding
> on all persistent memory namespaces to e
Dan Williams writes:
>> But I agree with your concern that if we have older kernel/applications
>> that continue to use `dcbf` on future hardware we will end up
>> having issues w.r.t powerfail consistency. The plan is what you outlined
>> above as tighter ecosystem control. Considering we don't
Dan Williams writes:
> Given the current dearth of systems that supply an ACPI HMAT table, and
> the utility of being able to manually define device-dax "hmem" instances
> via the efi_fake_mem= option, relax the requirements for creating these
> devices. Specifically, add an option (numa=nohmat)
Dan Williams writes:
> On Wed, Feb 26, 2020 at 1:52 PM Jeff Moyer wrote:
>>
>> Dan Williams writes:
>>
>> >> Missing doctext. Specifically, there should be a big, fat warning
>> >> against changing the region alignment.
>> >
>&g
Dan Williams writes:
>> Missing doctext. Specifically, there should be a big, fat warning
>> against changing the region alignment.
>
> I don't mind adding one, but is this the right place to document an
> API warning? If the audience is future ndctl developers that should be
> warned to keep th
Dan Williams writes:
> On Mon, Feb 24, 2020 at 1:53 PM Jeff Moyer wrote:
>>
>> Dan Williams writes:
>>
>> >> Let's just focus on reporting errors when we know we have them.
>> >
>> > That's the problem in my eyes. If software nee
Dan Williams writes:
>> Let's just focus on reporting errors when we know we have them.
>
> That's the problem in my eyes. If software needs to contend with
> latent error reporting then it should always contend otherwise
> software has multiple error models to wrangle.
The only way for an appli
Dan Williams writes:
> On Sun, Feb 23, 2020 at 3:03 PM Dave Chinner wrote:
>>
>> On Fri, Feb 21, 2020 at 03:17:59PM -0500, Vivek Goyal wrote:
>> > On Fri, Feb 21, 2020 at 01:32:48PM -0500, Jeff Moyer wrote:
>> > > Vivek Goyal writes:
>> > >
&
Dan Williams writes:
> Oh you misunderstood my comment, the "move badblocks to filesystem"
> proposal is long term / down the road thing to consider. In the near
> term this unaligned block zeroing facility is an improvement.
I'm not sure I agree. I'm going to think about it and get back to you
Vivek Goyal writes:
> On Thu, Feb 20, 2020 at 04:35:17PM -0500, Jeff Moyer wrote:
>> Vivek Goyal writes:
>>
>> > Currently pmem_clear_poison() expects offset and len to be sector aligned.
>> > Atleast that seems to be the assumption with which code has been wri
Vivek Goyal writes:
> Currently pmem_clear_poison() expects offset and len to be sector aligned.
> Atleast that seems to be the assumption with which code has been written.
> It is called only from pmem_do_bvec() which is called only from pmem_rw_page()
> and pmem_make_request() which will only p
Dan Williams writes:
>> >> Will this break existing code that parses the javascript output?
>> >
>> > Always a potential for that. That said, I'd rather attempt to make it
>> > symmetric and replace it if someone screams, rather than let this
>> > quirk persist because it makes it impossible to i
Dan Williams writes:
> On Wed, Feb 19, 2020 at 9:56 AM Jeff Moyer wrote:
>>
>> Dan Williams writes:
>>
>> > The only expected difference between "ndctl list -R" and "ndctl list
>> > -Rv" is some additional output fields. Instead it cu
Dan Williams writes:
>
> +NDCTL_EXPORT unsigned long ndctl_region_get_align(struct ndctl_region
> *region)
> +{
> + return region->align;
> +}
> +
> +NDCTL_EXPORT int ndctl_region_set_align(struct ndctl_region *region,
> + unsigned long align)
> +{
> + struct ndctl_ctx *ctx
Dan Williams writes:
> The only expected difference between "ndctl list -R" and "ndctl list
> -Rv" is some additional output fields. Instead it currently results in
> the region array being contained in a named "regions" list object.
>
> # ndctl list -R -r 0
> [
> {
> "dev":"region0",
>
Dan Williams writes:
>> diff --git a/ndctl/namespace.c b/ndctl/namespace.c
>> index 7fb0007..b1f2158 100644
>> --- a/ndctl/namespace.c
>> +++ b/ndctl/namespace.c
>> @@ -1388,11 +1388,9 @@ static int do_xaction_namespace(const char *namespace,
>> (*processed
; +extern struct ndctl_dimm_ops * const msft_dimm_ops;
> +extern struct ndctl_dimm_ops * const hyperv_dimm_ops;
>
> static inline struct ndctl_bus *cmd_to_bus(struct ndctl_cmd *cmd)
> {
Acked-by: Jeff Moyer
___
Linux-nvdimm mailing list -- linux-nvdimm@lists.01.org
To unsubscribe send an email to linux-nvdimm-le...@lists.01.org
Yi Zhang writes:
> On some NVDIMM servers, scrub operation will take long time to be finished
> as it start on all nvdimm buses in the system, add the bus-id parameter to
> do the scrub on the NFIT_TEST_BUS0
>
> Signed-off-by: Yi Zhang
Reviewed-by: Jeff Moyer
> ---
>
Dan Williams writes:
> [ drop Ross, add Kirill, linux-mm, and lkml ]
>
> On Tue, Dec 24, 2019 at 9:42 PM Murphy Zhou wrote:
>>
>> By running xfstests with fsdax enabled, generic/437 always hits this
>> warning[1] since this commit:
>>
>> commit 83d116c53058d505ddef051e90ab27f57015b025
>> Author:
;
> }
> @@ -81,6 +82,7 @@ static int of_pmem_region_remove(struct platform_device
> *pdev)
> struct of_pmem_private *priv = platform_get_drvdata(pdev);
>
> nvdimm_bus_unregister(priv->bus);
> + kfree(priv->bus_desc.provider_name);
> kfree(priv);
>
&g
nlock;
>
> + if (cmd_rc < 0) {
> + rc = cmd_rc;
> + goto out_unlock;
> + }
> +
> if (!nvdimm && cmd == ND_CMD_CLEAR_ERROR && cmd_rc >= 0) {
> struct nd_cmd_clear_error *clear_err = buf;
Looks good
Dan Williams writes:
> Right now the kernel does not install a pte on faults that land on a
> page with known poison, but only because the error clearing path is so
> convoluted and could only claim that fallocate(PUNCH_HOLE) cleared
> errors because that was guaranteed to send 512-byte aligned z
Dan Williams writes:
> ---
>
> Explicit review requests, but any other feedback is of course
> appreciated:
>
> Patch1 needs an ack from ppc arch maintainers, and I'd like a tested-by
> from Aneesh that this still works to solve the ppc issue. Jeff, does
> this look good to you?
OK, I've reviewe
Dan Williams writes:
> On Thu, Feb 13, 2020 at 8:58 AM Jeff Moyer wrote:
>> I have just a couple of questions.
>>
>> First, can you please add a comment above the generic implementation of
>> memremap_compat_align describing its purpose, and why a platform m
ents to current align
> - namespace/size: trim free space to current align
>
> ...to keep the free space accounting conforming to the dynamic align
> setting.
>
> Reported-by: Aneesh Kumar K.V
> Reported-by: Jeff Moyer
> Signed-off-by: Dan Williams
> Reviewed-by: Aneesh Kum
Dan Williams writes:
> On Thu, Feb 13, 2020 at 1:55 PM Jeff Moyer wrote:
>>
>> Dan Williams writes:
>>
>> > The pmem driver on PowerPC crashes with the following signature when
>> > instantiating misaligned namespaces that map their capacity via
>>
> pmem_attach_disk+0x188/0x770
> nvdimm_bus_probe+0xd8/0x470
>
> With the assumption that only memremap_pages() has alignment
> constraints, enforce memremap_compat_align() for
> pmem_should_map_pages(), nd_pfn, or nd_dax cases.
>
> Reported-by: Aneesh Kumar K.V
> Cc
> pmem_attach_disk+0x188/0x770
> nvdimm_bus_probe+0xd8/0x470
>
> With the assumption that only memremap_pages() has alignment
> constraints, enforce memremap_compat_align() for
> pmem_should_map_pages(), nd_pfn, or nd_dax cases.
>
> Reported-by: Aneesh Kumar K.V
> Cc
it(NDD_ALIASING, &nvdimm->flags) ? "alias " : "",
> + test_bit(NDD_LABELING, &nvdimm->flags) ? "label" : "",
^
Missing a space.
The rest looks sane.
that. What's the relationship, there, and can
we please have a comment explaining it?
Thanks!
Jeff
>
> Link:
> http://lore.kernel.org/r/capcyv4gbgnp95apyabcsocea50tqj9b5h__83vgngjq3oug...@mail.gmail.com
> Reported-by: Aneesh Kumar K.V
> Reported-by: Jeff Moyer
> Cc: Benjamin Herre
Jan Kara writes:
> When a kernel is configured without CONFIG_DEV_DAX_PMEM_COMPAT, the
> compilation of tools/testing/nvdimm fails with:
>
> Building modules, stage 2.
> MODPOST 11 modules
> ERROR: "dax_pmem_compat_test" [tools/testing/nvdimm/test/nfit_test.ko]
> undefined!
>
> Fix the probl
Jan Kara writes:
> On Thu 06-02-20 09:33:39, Jeff Moyer wrote:
>> Jan Kara writes:
>>
>> > On Wed 05-02-20 14:15:58, Jeff Moyer wrote:
>> >> fstests generic/471 reports a failure when run with MOUNT_OPTIONS="-o
>> >> dax". The r
Jan Kara writes:
> On Wed 05-02-20 14:15:58, Jeff Moyer wrote:
>> fstests generic/471 reports a failure when run with MOUNT_OPTIONS="-o
>> dax". The reason is that the initial pwrite to an empty file with the
>> RWF_NOWAIT flag set does not return -EAGAIN. It
pplied, generic/471 passes for me.
Signed-off-by: Jeff Moyer
diff --git a/fs/dax.c b/fs/dax.c
index 1f1f0201cad1..0b0d8819cb1b 100644
--- a/fs/dax.c
+++ b/fs/dax.c
@@ -1207,6 +1207,9 @@ dax_iomap_rw(struct kiocb *iocb, struct iov_iter *iter,
lockdep_assert_held(
Hi, Dan,
Dan Williams writes:
> I'm going to take a look at how hard it would be to develop a kpartx
> fallback in udev. If that can live across the driver transition then
> maybe this can be a non-event for end users that already have that
> udev update deployed.
I just wanted to remind you th
"Aneesh Kumar K.V" writes:
>> /*
>> * Platform provides mechanisms to flush outstanding write data
>> * to pmem on system power loss.
>> */
>>
>
> Wanted to add more details. So with the above interpretation, if the
> persistence_domain is found to be 'cpu_cache', application can expect
> a
"Aneesh Kumar K.V" writes:
>> Would you also update of_pmem to indicate the persistence domain,
>> please?
>>
>
> sure.
Thanks!
>>> diff --git a/include/linux/libnvdimm.h b/include/linux/libnvdimm.h
>>> index f2a33f2e3ba8..9126737377e1 100644
>>> --- a/include/linux/libnvdimm.h
>>> +++ b/includ
"Aneesh Kumar K.V" writes:
> Currently, kernel shows the below values
> "persistence_domain":"cpu_cache"
> "persistence_domain":"memory_controller"
> "persistence_domain":"unknown"
>
> This patch updates the meaning of these values such that
>
> "cpu_cache" indicates no extra in
Hi, Aneesh,
After applying this patch series, several of my namespaces no longer
enumerate:
Before:
# ndctl list
[
{
"dev":"namespace0.2",
"mode":"sector",
"size":106541672960,
"uuid":"ea1122b2-c219-424c-b09c-38a6e94a1042",
"sector_size":512,
"blockdev":"pmem0.2s"
},
Dan Williams writes:
> On Fri, Nov 22, 2019 at 8:09 AM Jeff Moyer wrote:
>>
>> Dan Williams writes:
>>
>> > On Wed, Nov 20, 2019 at 9:26 AM Jeff Moyer wrote:
>> >>
>> >> Pankaj Gupta writes:
>> >>
>> >> > R
Dan Williams writes:
> On Wed, Nov 20, 2019 at 9:26 AM Jeff Moyer wrote:
>>
>> Pankaj Gupta writes:
>>
>> > Remove logic to create child bio in the async flush function which
>> > causes child bio to get executed after parent bio 'pmem_make_requ
tead we are performing flush from the parent bio to maintain the
> correct order. Also, returning from function 'pmem_make_request' if
> REQ_PREFLUSH returns an error.
>
> Reported-by: Jeff Moyer
> Signed-off-by: Pankaj Gupta
There's a slight change in behavior for
Masahiro Yamada writes:
> On Fri, Nov 1, 2019 at 1:51 AM Jeff Moyer wrote:
>>
>> Masahiro Yamada writes:
>>
>> > Now that there is no overwrap between symbols from ELF files and
>> > ones from Module.symvers.
>> >
>> > So, the 'e
Masahiro Yamada writes:
> Now that there is no overwrap between symbols from ELF files and
> ones from Module.symvers.
>
> So, the 'exported twice' warning should be reported irrespective
> of where the symbol in question came from. Only the exceptional case
> is when __crc_ symbol appears before
Ira Weiny writes:
> On Mon, Oct 28, 2019 at 03:37:48PM -0400, Jeff Moyer wrote:
>> Ira Weiny writes:
>>
>> >> (Watching the unit test run fall into an infinite loop..) Nope, the
>> >> break is in the switch scope, the while loop needs the 'goto out&
Ira Weiny writes:
> On Mon, Oct 28, 2019 at 03:37:48PM -0400, Jeff Moyer wrote:
>> Ira Weiny writes:
>>
>> >> (Watching the unit test run fall into an infinite loop..) Nope, the
>> >> break is in the switch scope, the while loop needs the 'goto out&
Ira Weiny writes:
>> (Watching the unit test run fall into an infinite loop..) Nope, the
>> break is in the switch scope, the while loop needs the 'goto out'.
>>
>> Yes this bit definitely needs to be refactored :)
>
> How about this patch instead? Untested.
I'm not a fan of the looping with g
Dan Williams writes:
> Check for NULL entries before checking the entry order, otherwise NULL
> is misinterpreted as a present pte conflict. The 'order' check needs to
> happen before the locked check as an unlocked entry at the wrong order
> must fallback to lookup the correct order.
Please inc
Ira Weiny writes:
> On Fri, Oct 18, 2019 at 04:23:01PM -0400, Jeff Moyer wrote:
>> The 'done' variable only adds confusion.
>>
>> Signed-off-by: Jeff Moyer
>> ---
>> ndctl/dimm.c | 7 +--
>> 1 file changed, 1 insertion(+), 6 deletions(-
The 'done' variable only adds confusion.
Signed-off-by: Jeff Moyer
---
ndctl/dimm.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/ndctl/dimm.c b/ndctl/dimm.c
index c8821d6..f28b9c1 100644
--- a/ndctl/dimm.c
+++ b/ndctl/dimm.c
@@ -682,7 +682,6 @@
Make currently fails with:
make[1]: *** No rule to make target 'libndctl.h', needed by 'tags-am'. Stop.
The path to libndctl.h is wrong in ndctl/lib/Makefile.am. Fix it.
Signed-off-by: Jeff Moyer
---
ndctl/lib/Makefile.am | 2 +-
1 file changed, 1 insertion(+), 1 delet
Static checkers complain about the unused assignment to pfx_len.
The code can obviously be simplified.
Signed-off-by: Jeff Moyer
---
util/abspath.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/util/abspath.c b/util/abspath.c
index 09bbd27..e44236f 100644
--- a/util
Signed-off-by: Jeff Moyer
---
ndctl/load-keys.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/ndctl/load-keys.c b/ndctl/load-keys.c
index 981f80f..f0b7a5a 100644
--- a/ndctl/load-keys.c
+++ b/ndctl/load-keys.c
@@ -185,7 +185,6 @@ static int load_keys(struct loadkeys *lk_ctx, const char
These patches are motivated by warnings from the clang static
analyzer. They don't fix bugs, and shouldn't result in any changes
in behavior. The one exception is the fix for building tags tables.
-Jeff
[ndctl patch 1/4] util/abspath: cleanup prefix_filename
[ndctl patch 2/4] fix building of ta
Joe Perches writes:
> Rather than have a local coding style, use the typical kernel style.
The coding style isn't that different from the core kernel, and it's
still quite readable. I'd rather avoid the churn and the risk of
introducing regressions. This will also make backports to stable more
ment: Maybe the solution is to keep the create-namespace
> semantics unchanged, and introduce a new command - 'create-namespaces'
> or 'create-names-ace-greedy' with the new behavior. I'm not sure if
> users will care deeply about either of the two points above, henc
Dan Williams writes:
> Changes since v1 [1]:
> - Cleanup patch1, simplify flags return in the overwrite case and
> consolidate frozen-state cases (Jeff)
> - Clarify the motivation for patch2 (Jeff)
> - Collect Dave's Reviewed-by
The series tests out fine for me.
Thanks, Dan!
-Jeff
>
> [1]:
x27;locked' state, but will need to revisit if there were cases where
> applications need 'frozen' to show up in the primary 'security'
> attribute. The expectation is that communicating 'frozen' is mostly a
> helper for debug and status monito
__security_store() rather than each of the helper routines to enable
> freeze to be run regardless of busy state.
>
> Reviewed-by: Dave Jiang
> Signed-off-by: Dan Williams
Reviewed-by: Jeff Moyer
> ---
> drivers/nvdimm/dimm_devs.c | 33 -
>
before the introduction
of the offending commit.
[1] https://lists.01.org/pipermail/linux-nvdimm/2019-July/022813.html
Fixes: commit a3619190d62e ("libnvdimm/pfn: stop padding pmem namespaces ...")
Reported-and-tested-by: Yi Zhang
Signed-off-by: Jeff Moyer
diff --git a/drivers/nvdimm/pfn_dev
Dan, we should probably fix this before 5.3 ships. Do you have any
concerns with the patch I attached? If not, I'll submit a proper one.
-Jeff
Jeff Moyer writes:
> Hi, Yi,
>
> Yi Zhang writes:
>
>> Hi Dan
>>
>> As subject, I found it failed from bellow co
Vishal Verma writes:
> When a dax-bus vs. dax-class expectation causes a failure, such as when
> reconfiguring device modes, print an error message directly pointing the
> user to the daxctl-migrate-device-model command.
>
> Reported-by: Dave Hansen
> Reported-by: Jeff Moyer
Jia He writes:
> commit c221c0b0308f ("device-dax: "Hotplug" persistent memory for use
> like normal RAM") helps to add persistent memory as normal RAM blocks.
> But this driver doesn't work if CONFIG_DEV_DAX_PMEM_COMPAT is enabled.
>
> Here is the debugging call trace when CONFIG_DEV_DAX_PMEM_CO
Dan Williams writes:
> On Fri, Aug 16, 2019 at 1:49 PM Jeff Moyer wrote:
>>
>> Dan Williams writes:
>>
>> > The blanket blocking of all security operations while the DIMM is in
>> > active use in a region is too restrictive. The only security operations
&
, just move the
> __security_store() entry point to live with the helpers.
>
> Cc: Dave Jiang
> Signed-off-by: Dan Williams
Fine by me.
Acked-by: Jeff Moyer
___
Linux-nvdimm mailing list
Linux-nvdimm@lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm
Dan Williams writes:
> The blanket blocking of all security operations while the DIMM is in
> active use in a region is too restrictive. The only security operations
> that need to be aware of the ->busy state are those that mutate the
> state of data, i.e. erase and overwrite.
>
> Refactor the -
x27;locked' state, but will need to revisit if there were cases where
> applications need 'frozen' to show up in the primary 'security'
> attribute. The expectation is that communicating 'frozen' is mostly a
> helper for debug and status monitoring.
&g
Hi, Yi,
Yi Zhang writes:
> Hi Dan
>
> As subject, I found it failed from bellow commit[1], steps list here
> [2] and I've attached the full dmesg, let me know if you need more
> info, thanks.
>
> [1]
> commit a3619190d62ed9d66416891be2416f6bea2b3ca4 (refs/bisect/bad)
> Author: Dan Williams
> Da
Vaibhav Jain writes:
> Thanks for reviewing this patch Jeff.
>
> Jeff Moyer writes:
>
>> Vaibhav Jain writes:
>>
>>> On PPC64 which uses a 64K page-size, ndtl-check command fails on a BTT
>>> namespace with following error:
>>>
Vaibhav Jain writes:
> On PPC64 which uses a 64K page-size, ndtl-check command fails on a BTT
> namespace with following error:
>
> $ sudo ndctl check-namespace namespace0.0 -vv
> namespace0.0: namespace_check: checking namespace0.0
> namespace0.0: btt_discover_arenas: found 1 BTT arena
> n
Dan Williams writes:
> On Mon, Aug 12, 2019 at 1:45 PM Jeff Moyer wrote:
>>
>> Dan Williams writes:
>>
>> > If the 'kmem' module is missing skip the test to support running the
>> > unit tests on older -stable kernels pre-v5.1.
>> >
&g
Dan Williams writes:
> On Mon, Aug 12, 2019 at 8:51 AM Jeff Moyer wrote:
>>
>> Dan Williams writes:
>>
>> > Currently, attempts to shutdown and re-enable a device-dax instance
>> > trigger:
>>
>> What does "shutdown and re-enable&qu
Dan Williams writes:
> If the 'kmem' module is missing skip the test to support running the
> unit tests on older -stable kernels pre-v5.1.
>
> Signed-off-by: Dan Williams
Note that the kernel module could also just not be configured.
> ---
> test/daxctl-devices.sh |7 +++
> 1 file ch
Dan Williams writes:
> Currently, attempts to shutdown and re-enable a device-dax instance
> trigger:
What does "shutdown and re-enable" translate to? If I disable and
re-enable a device-dax namespace, I don't see this behavior.
-Jeff
>
> Missing reference count teardown definition
>
em $mnt
Agreed. In the future, the options may not be mutually exclusive, but I
don't see any harm in always testing with reflink=0 for the existing
tests.
Acked-by: Jeff Moyer
>
> Cc: Jeff Moyer
> Signed-off-by: Dan Williams
> ---
> test/dax.sh |4 ++--
> test/mma
Dan Williams writes:
> ...but ABIs are forever, and ndctl has shipped:
>
> if (strcmp(buf, "disabled") == 0)
> return NDCTL_SECURITY_DISABLED;
> else if (strcmp(buf, "unlocked") == 0)
> return NDCTL_SECURITY_UNLOCKED;
> else if (strcmp(buf,
> Cc: Dan Williams
> Signed-off-by: Vishal Verma
> ---
>
> v2:
> - Remove the part about an address abstraction mechanism; It didn't add
> any value (Jeff)
> - Add an additional sentence about space reclamation semantics (Dan)
LGTM
Reviewed-by: Jeff Moyer
-Jeff
>
Vishal Verma writes:
> In the ndctl-create-namespace (and related) man pages, add a
> clarification note regarding some of the restrictions a user may see
> when operating on label-less namespaces.
>
> Link: https://github.com/pmem/ndctl/issues/52
> Reported-by: Jane Chu
> Cc: Dan Williams
> Si
Dan Williams writes:
> On Thu, Aug 1, 2019 at 2:55 PM Jeff Moyer wrote:
>>
>> If a dimm is frozen, it is currently reported as being "locked". While
>> that's not technically wrong, it is misleading as the dimm can't be
>> unlocked. Fix the confu
Dan Williams writes:
> On Mon, Aug 5, 2019 at 10:06 AM Jeff Moyer wrote:
>>
>> Dan Williams writes:
>>
>> > gcc 9.1.1 emits a slew of warnings for many of the command field
>> > accesses. I.e. warnings of the form:
>> >
>> > libndc
Dan Williams writes:
> gcc 9.1.1 emits a slew of warnings for many of the command field
> accesses. I.e. warnings of the form:
>
> libndctl.c:2586:21: warning: taking address of packed member of ‘struct
> nd_cmd_get_config_data_hdr’ may result in an unaligned pointer value
> [-Waddress-of-packe
If a dimm is frozen, it is currently reported as being "locked". While
that's not technically wrong, it is misleading as the dimm can't be
unlocked. Fix the confusion.
Thanks to Dan for pointing this out.
Signed-off-by: Jeff Moyer
diff --git a/drivers/acpi/nfit/intel.c
"Aneesh Kumar K.V" writes:
> On 6/14/19 10:06 PM, Dan Williams wrote:
>> On Fri, Jun 14, 2019 at 9:26 AM Aneesh Kumar K.V
>> wrote:
>
>>> Why not let the arch
>>> arch decide the SUBSECTION_SHIFT and default to one subsection per
>>> section if arch is not enabled to work with subsection.
>>
>>
Dan Williams writes:
> On Mon, Apr 22, 2019 at 8:59 AM Jeff Moyer wrote:
>>
>> Dan Williams writes:
>>
>> > On Thu, Apr 18, 2019 at 9:18 AM Christoph Hellwig
>> > wrote:
>> >>
>> >> On Thu, Apr 18, 2019 at 09:05:05AM -0700,
Dan Williams writes:
> On Thu, Apr 18, 2019 at 9:18 AM Christoph Hellwig wrote:
>>
>> On Thu, Apr 18, 2019 at 09:05:05AM -0700, Dan Williams wrote:
>> > > > I'd either add a comment about avoiding retpoline overhead here or just
>> > > > make ->flush == NULL mean generic_nvdimm_flush(). Just so
Dan Williams writes:
> On Fri, Apr 12, 2019 at 6:12 AM Jeff Moyer wrote:
>>
>> Jan Kara writes:
>>
>> > On Thu 11-04-19 07:51:48, Dan Williams wrote:
>> >> On Tue, Apr 9, 2019 at 9:09 PM Pankaj Gupta wrote:
>> >> > + } else
Dan Williams writes:
>> On Wed, Apr 17, 2019 at 3:59 PM Dan Williams
>> wrote:
>>
>> On Wed, Apr 17, 2019 at 3:04 PM Andrew Morton
>> wrote:
>> >
>> > On Wed, 17 Apr 2019 11:38:55 -0700 Dan Williams
>> > wrote:
>> >
>> > > The memory hotplug section is an arbitrary / convenient unit for mem
Dan Williams writes:
> On Mon, Apr 15, 2019 at 1:32 PM Jeff Moyer wrote:
>>
>> Hi,
>>
>> We ran into a situation where one dimm's label area was corrupt (it had
>> multiple entries for the same DPA), and so that nmem was disabled. The
>> region a
ng in this case. Check for that.
Signed-off-by: Jeff Moyer
Reported-by: Zhang Yi
diff --git a/drivers/nvdimm/label.c b/drivers/nvdimm/label.c
index f3d753d3169c..da1f28d8c9e6 100644
--- a/drivers/nvdimm/label.c
+++ b/drivers/nvdimm/label.c
@@ -1217,7 +1217,7 @@ static int del_labels(s
Jan Kara writes:
> On Thu 11-04-19 07:51:48, Dan Williams wrote:
>> On Tue, Apr 9, 2019 at 9:09 PM Pankaj Gupta wrote:
>> > + } else {
>> > + if (nd_region->flush(nd_region))
>> > + rc = -EIO;
>>
>> Given the common case wants to be fast and synchronous
page
> support THP)"
> Co-developed-by: Liang ZhiCheng
> Signed-off-by: Liang ZhiCheng
> Signed-off-by: Li RongQing
Reviewed-by: Jeff Moyer
> ---
> drivers/nvdimm/pmem.c | 8
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/dr
1 - 100 of 241 matches
Mail list logo
