tree 58cf3933db1cb2083d6a758828fc448a8b955646
parent 5b73cd8d8446080a37494d82527646820b467c4f
author NeilBrown <neilb@suse.com> 1491790380 +1000
committer Alexander Martinz <amartinz@shiftphones.com> 1671559002 +0100
gpgsig -----BEGIN PGP SIGNATURE-----
 
 iQIzBAABCAAdFiEEioN3RtFSeUXyJOKbzxW92aMhqbIFAmOh91oACgkQzxW92aMh
 qbLoPBAAhZqrkLLCsiyOOKU2r2KvR+znUtD00ovDYF6KbOTJE2HW4ZlEvTHXwQFl
 nzMBB1bZaegLqFNLZWG2BUZ63uI7G3OOZrro/6+gOyIWcYTKFHoJYEDKEizaZBYS
 2U+hmtzQ8xLsY6qFAzSdUpdwPPpBgSbuAYEXZYdBUt6OqnK4JMJRWG/oLZHpH1gZ
 vm6u1Cni1SXzJ0TuWgsKMiyuwdmpkg4t1YAEU/EHfQ4CwaotbJvDO5PgoLPdEiob
 4lYx23r/eoZDZuof7/2qZjljtCWXTRuZj2hJ+nFbNdqFFj15dn+spFw1bmCkGlvB
 cc77+j5nlj9a4F2wVjZAjFOcfYoTbYUiFwci/Ya86iSSDKFWSx6JLkATwfA5uGxI
 n7ntApsRdHgydsZCJNtgZLD2QGGfMYSzRpnC5Rhd2JyGRhzK+eFh6KoT3yNMH4De
 Y26BJdbrCUnvatgH9i1ce/IWYVqhc61uWMxHDXxQepdaAFb8Av+LV/KGRDFbDDCb
 BviEh92SRDx390bVaOdqK7cwS/clvMNwXj+r7AKTbGRvwxETIx5CsokYLJZDpy5r
 9izrgfT6MOjzihwvM0D+XAIxOA1A8BuMJXUogf23lsBFelxmNr//5i+Ucah7Qmkl
 qdRhBFdWnZMW6bYSLOoIDblfDqljOl5WD+poHtXNYFeEBUS4Ek4=
 =gCeM
 -----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>