tree b3b1bc58b31e784f1f5876501d11d54e086aaef3
parent a8fe3bf687691488c856a6f4414c9144709335f5
author NeilBrown <neilb@suse.com> 1491790380 +1000
committer Alexander Martinz <alex@amartinz.at> 1664458453 +0200
gpgsig -----BEGIN PGP SIGNATURE-----
 
 iQIzBAABCAAdFiEEpJShu+9pnB0vSPYRH4MFVmchqx4FAmM1ntUACgkQH4MFVmch
 qx7mIQ//XX3h32XuckYVXUYPTqG/LzlUnvxRs/pRUsC9P9xXRMjVMYXja24mWVQ/
 54i//YVkAKW8iYNRYeUMTH3eURjU+vuhWMOnrzjIPIX1IMl3OyLB+JC6vz0Dk4rX
 XzmSBHpO/SDOtfldvsODTWJADVe5+y3xkKjzWzmUmE8+ysT/I/JOBZJ/mN4RsGXK
 pGIolgmTEO9bt1FPFAQ13E/fr6MP0nqfHPQ/eQ6m8r6B+Sok4TWC87d5AlQGHedM
 59Bt+qFMXguUlDVrqGceRqhF1PBwTS1IX4a56BnH/jnrS11sH0vthwe4bldJprtD
 uvnK2toy9o2k5RxHEQHGybbthOe8cio+DnHHzAGqwY9lCbIwGNKRrDGMdQ6XZkck
 2ScIo99V8+03Jq58uKErhyh8MqSvsVrrMKq7L340KBO7+HvBgyfqRDvSF6zPSjYz
 pieeK2XRcroBEJ7lg6SL5YzWp+GXsVNkh8v312G+pOjSAT6NF4cOJXZ326r3r2Qe
 HhRrXvSZT9EJ8YifXYCEr3KftuSMoxz6u8WQJBAE/uILAS2F5nySqwSVRXswepkO
 iiLTDiELVMDlQBl/Ql/rRA1JsWCkW6+QDLlAd7/3c9GMvx6sMkog8dQrOeXTESD+
 q+BH+WhN9NvfQ0RgUMcbDlVzVp4I/+ihBl8R6HzSv5RH6mBsqlM=
 =k0/0
 -----END PGP SIGNATURE-----

dm verity fec: fix GFP flags used with mempool_alloc()

mempool_alloc() cannot fail for GFP_NOIO allocation, so there is no
point testing for failure.

One place the code tested for failure was passing "0" as the GFP
flags.  This is most unusual and is probably meant to be GFP_NOIO,
so that is changed.

Also, allocation from ->extra_pool and ->prealloc_pool are repeated
before releasing the previous allocation.  This can deadlock if the code
is servicing a write under high memory pressure.  To avoid deadlocks,
change these to use GFP_NOWAIT and leave the error handling in place.

Change-Id: Ic4374d9c2ca9bb55e32f2f65a45b01a26f90943c
Signed-off-by: NeilBrown <neilb@suse.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>