Re: [PATCH 1/6] dax: fix build breakage with ext4, dax and !iomap

2016-11-30 Thread Jan Kara
On Wed 30-11-16 12:04:31, Ross Zwisler wrote: > On Tue, Nov 29, 2016 at 09:53:03AM +0100, Jan Kara wrote: > > On Mon 28-11-16 12:15:04, Ross Zwisler wrote: > > > On Thu, Nov 24, 2016 at 10:02:39AM +0100, Jan Kara wrote: > > > > On Wed 23-11-16 11:44:17, Ross Zwisler wrote: > > > > > With the curren

Re: [PATCH 1/6] dax: fix build breakage with ext4, dax and !iomap

2016-11-30 Thread Ross Zwisler
On Tue, Nov 29, 2016 at 09:53:03AM +0100, Jan Kara wrote: > On Mon 28-11-16 12:15:04, Ross Zwisler wrote: > > On Thu, Nov 24, 2016 at 10:02:39AM +0100, Jan Kara wrote: > > > On Wed 23-11-16 11:44:17, Ross Zwisler wrote: > > > > With the current Kconfig setup it is possible to have the following: >

Re: [PATCH 1/6] dax: fix build breakage with ext4, dax and !iomap

2016-11-29 Thread Jan Kara
On Mon 28-11-16 12:15:04, Ross Zwisler wrote: > On Thu, Nov 24, 2016 at 10:02:39AM +0100, Jan Kara wrote: > > On Wed 23-11-16 11:44:17, Ross Zwisler wrote: > > > With the current Kconfig setup it is possible to have the following: > > > > > > CONFIG_EXT4_FS=y > > > CONFIG_FS_DAX=y > > > CONFIG_FS_

Re: [PATCH 1/6] dax: fix build breakage with ext4, dax and !iomap

2016-11-28 Thread Ross Zwisler
On Thu, Nov 24, 2016 at 10:02:39AM +0100, Jan Kara wrote: > On Wed 23-11-16 11:44:17, Ross Zwisler wrote: > > With the current Kconfig setup it is possible to have the following: > > > > CONFIG_EXT4_FS=y > > CONFIG_FS_DAX=y > > CONFIG_FS_IOMAP=n # this is in fs/Kconfig & isn't user accessible >

Re: [PATCH 1/6] dax: fix build breakage with ext4, dax and !iomap

2016-11-24 Thread Jan Kara
On Wed 23-11-16 11:44:17, Ross Zwisler wrote: > With the current Kconfig setup it is possible to have the following: > > CONFIG_EXT4_FS=y > CONFIG_FS_DAX=y > CONFIG_FS_IOMAP=n # this is in fs/Kconfig & isn't user accessible > > With this config we get build failures in ext4_dax_fault() becaus

[PATCH 1/6] dax: fix build breakage with ext4, dax and !iomap

2016-11-23 Thread Ross Zwisler
With the current Kconfig setup it is possible to have the following: CONFIG_EXT4_FS=y CONFIG_FS_DAX=y CONFIG_FS_IOMAP=n # this is in fs/Kconfig & isn't user accessible With this config we get build failures in ext4_dax_fault() because the iomap functions in fs/dax.c are missing: fs/built-i