[Regression] Please revert a91a2785b20

From: Thomas Gleixner
Date: Mon Mar 28 2011 - 18:35:33 EST


Out of the blue all my perfectly fine working test machines which use
RAID stopped working with the very helpful error message:

md/raid1:md1: active with 2 out of 2 mirrors
md: pers->run() failed ...

Reverting a91a2785b20 fixes the problem.

Neither the subject line:

block: Require subsystems to explicitly allocate bio_set integrity mempool

nor the changelog have any hint why that wreckage is in any way
sensible.

The wreckage happens due to:

- md_integrity_register(mddev);
- return 0;
+ return md_integrity_register(mddev);

But the changelog does not give the courtesy of explaining these
changes. Also there is no fcking reason why the kernel cannot deal
with the missing integrity capabilities of a drive just by emitting a
warning msg and dealing gracefully with the outcome.

All my RAID setups have been working perfectly fine until now, so
what's the rationale to break this?

Did anyone test this shite on a non enterprise class hardware with a
distro default config ? Obviously _NOT_.

FYI, the config files of those machines are based off a fedora default
config, so this would hit all raid users based on popular distro
configs sooner than later.

Thanks for stealing my time,

tglx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/