Re: [PATCH 5/9] bio-integrity: fold bio_integrity_enabled to bio_integrity_prep

From: Hannes Reinecke
Date: Wed Apr 05 2017 - 02:37:11 EST


On 04/04/2017 08:56 PM, Dmitry Monakhov wrote:
> Currently all integrity prep hooks are open-coded, and if prepare fails
> we ignore it's code and fail bio with EIO. Let's return real error to
> upper layer, so later caller may react accordingly.
>
> In fact no one want to use bio_integrity_prep() w/o bio_integrity_enabled,
> so it is reasonable to fold it in to one function.
>
> Signed-off-by: Dmitry Monakhov <dmonakhov@xxxxxxxxxx>
> ---
> Documentation/block/data-integrity.txt | 3 --
> block/bio-integrity.c | 88 ++++++++++++++++++----------------
> block/blk-core.c | 5 +-
> block/blk-mq.c | 8 +---
> drivers/nvdimm/blk.c | 13 +----
> drivers/nvdimm/btt.c | 13 +----
> include/linux/bio.h | 6 ---
> 7 files changed, 55 insertions(+), 81 deletions(-)
>
Reviewed-by: Hannes Reinecke <hare@xxxxxxxx>

Cheers,

Hannes
--
Dr. Hannes Reinecke Teamlead Storage & Networking
hare@xxxxxxx +49 911 74053 688
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton
HRB 21284 (AG Nürnberg)