Thread (44 messages) 44 messages, 4 authors, 2017-07-29

Re: [resend PATCH v2 11/33] dm: add dax_device and dax_operations support

From: Dan Williams <hidden>
Date: 2017-07-29 19:57:19
Also in: dm-devel, linux-block, lkml, nvdimm

On Fri, Jul 28, 2017 at 9:17 AM, Bart Van Assche [off-list ref] wrote:
On Mon, 2017-04-17 at 12:09 -0700, Dan Williams wrote:
quoted
diff --git a/drivers/md/Kconfig b/drivers/md/Kconfig
index b7767da50c26..1de8372d9459 100644
--- a/drivers/md/Kconfig
+++ b/drivers/md/Kconfig
@@ -200,6 +200,7 @@ config BLK_DEV_DM_BUILTIN
 config BLK_DEV_DM
      tristate "Device mapper support"
      select BLK_DEV_DM_BUILTIN
+     select DAX
      ---help---
        Device-mapper is a low level volume manager.  It works by allowing
        people to specify mappings for ranges of logical sectors.  Various
(replying to an e-mail of three months ago)

Hello Dan,

While building a v4.12 kernel I noticed that enabling device mapper support
now unconditionally enables DAX. I think there are plenty of systems that use
dm but do not need DAX. Have you considered to rework this such that instead
of dm selecting DAX that DAX support is only enabled in dm if CONFIG_DAX is
enabled?
I'd rather flip this around and add a CONFIG_DM_DAX that gates whether
DM enables / links to the DAX core. I'll take a look at a patch.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help