dax: Make huge page handling depend of CONFIG_BROKEN
authorJan Kara <jack@suse.cz>
Thu, 12 May 2016 16:29:15 +0000 (18:29 +0200)
committerRoss Zwisler <ross.zwisler@linux.intel.com>
Thu, 19 May 2016 21:13:17 +0000 (15:13 -0600)
commit348e967ab07c96a9e7a6a194812254a8df2045c0
tree2f593288ff249a8b496a5b04c0bdfcc61ab71adc
parentb9953536c95fd0013695542bfa4694c7468673aa
dax: Make huge page handling depend of CONFIG_BROKEN

Currently the handling of huge pages for DAX is racy. For example the
following can happen:

CPU0 (THP write fault) CPU1 (normal read fault)

__dax_pmd_fault() __dax_fault()
  get_block(inode, block, &bh, 0) -> not mapped
get_block(inode, block, &bh, 0)
  -> not mapped
  if (!buffer_mapped(&bh) && write)
    get_block(inode, block, &bh, 1) -> allocates blocks
  truncate_pagecache_range(inode, lstart, lend);
dax_load_hole();

This results in data corruption since process on CPU1 won't see changes
into the file done by CPU0.

The race can happen even if two normal faults race however with THP the
situation is even worse because the two faults don't operate on the same
entries in the radix tree and we want to use these entries for
serialization. So make THP support in DAX code depend on CONFIG_BROKEN
for now.

Signed-off-by: Jan Kara <jack@suse.cz>
Signed-off-by: Ross Zwisler <ross.zwisler@linux.intel.com>
fs/Kconfig
fs/dax.c
include/linux/dax.h