]> granicus.if.org Git - zfs/commit
Remove the bio_empty_barrier() check.
authorEtienne Dechamps <etienne@edechamps.fr>
Sun, 24 Feb 2013 11:22:07 +0000 (11:22 +0000)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Sun, 24 Feb 2013 18:22:34 +0000 (10:22 -0800)
commitd9b0ebbe824469d178a05b0fb9004e4afce86009
tree9c7a5c87282f39a353e37cb51e8ba57bada60ada
parentd75af3c0ebb4e31ece087aeecfeff282cb6f1361
Remove the bio_empty_barrier() check.

To determine whether the kernel is capable of handling empty barrier
BIOs, we check for the presence of the bio_empty_barrier() macro,
which was introduced in 2.6.24. If this macro is defined, then we can
flush disk vdevs; if it isn't, then flushing is disabled.

Unfortunately, the bio_empty_barrier() macro was removed in 2.6.37,
even though the kernel is still capable of handling empty barrier BIOs.

As a result, flushing is effectively disabled on kernels >= 2.6.37,
meaning that starting from this kernel version, zfs doesn't use
barriers to guarantee on-disk data consistency. This is quite bad and
can lead to potential data corruption on power failures.

This patch fixes the issue by removing the configure check for
bio_empty_barrier(), as we don't support kernels <= 2.6.24 anymore.

Thanks to Richard Kojedzinszky for catching this nasty bug.

Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #1318
config/kernel-bio-empty-barrier.m4 [deleted file]
config/kernel.m4
module/zfs/vdev_disk.c