Re: [PATCH v3 4/5] ext2: Add alignment check for DAX mount

2016-05-09 Thread Jan Kara
On Thu 05-05-16 18:29:55, Toshi Kani wrote: > When a partition is not aligned by 4KB, mount -o dax succeeds, > but any read/write access to the filesystem fails, except for > metadata update. > > Call bdev_supports_dax() to perform proper precondition checks > which includes this partition

Re: [PATCH v3 4/5] ext2: Add alignment check for DAX mount

2016-05-09 Thread Jan Kara
On Thu 05-05-16 18:29:55, Toshi Kani wrote: > When a partition is not aligned by 4KB, mount -o dax succeeds, > but any read/write access to the filesystem fails, except for > metadata update. > > Call bdev_supports_dax() to perform proper precondition checks > which includes this partition

Re: [PATCH v3 4/5] ext2: Add alignment check for DAX mount

2016-05-09 Thread Jan Kara
On Sun 08-05-16 01:59:37, Christoph Hellwig wrote: > Not really for the patch, but given that we have the right people > on CC: > > Do we really want to keep DAX support in ext2 in the long run? ext2 > is missing a lot of the useful features for a modern FS, shouldn't > we direct people to use

Re: [PATCH v3 4/5] ext2: Add alignment check for DAX mount

2016-05-09 Thread Jan Kara
On Sun 08-05-16 01:59:37, Christoph Hellwig wrote: > Not really for the patch, but given that we have the right people > on CC: > > Do we really want to keep DAX support in ext2 in the long run? ext2 > is missing a lot of the useful features for a modern FS, shouldn't > we direct people to use

Re: [PATCH v3 4/5] ext2: Add alignment check for DAX mount

2016-05-08 Thread Christoph Hellwig
Not really for the patch, but given that we have the right people on CC: Do we really want to keep DAX support in ext2 in the long run? ext2 is missing a lot of the useful features for a modern FS, shouldn't we direct people to use ext4 (in non-journal mode if needed) if they want to use DAX?

Re: [PATCH v3 4/5] ext2: Add alignment check for DAX mount

2016-05-08 Thread Christoph Hellwig
Not really for the patch, but given that we have the right people on CC: Do we really want to keep DAX support in ext2 in the long run? ext2 is missing a lot of the useful features for a modern FS, shouldn't we direct people to use ext4 (in non-journal mode if needed) if they want to use DAX?

[PATCH v3 4/5] ext2: Add alignment check for DAX mount

2016-05-05 Thread Toshi Kani
When a partition is not aligned by 4KB, mount -o dax succeeds, but any read/write access to the filesystem fails, except for metadata update. Call bdev_supports_dax() to perform proper precondition checks which includes this partition alignment check. Signed-off-by: Toshi Kani

[PATCH v3 4/5] ext2: Add alignment check for DAX mount

2016-05-05 Thread Toshi Kani
When a partition is not aligned by 4KB, mount -o dax succeeds, but any read/write access to the filesystem fails, except for metadata update. Call bdev_supports_dax() to perform proper precondition checks which includes this partition alignment check. Signed-off-by: Toshi Kani Cc: Jan Kara Cc: