On Thu, 2014-02-20 at 22:44 +0900, Akinobu Mita wrote: > Thanks for the patch. I have just one nitpicking. > > > diff --git a/fs/bio-integrity.c b/fs/bio-integrity.c > > index 0bad24d..47ed160 100644 > > --- a/fs/bio-integrity.c > > +++ b/fs/bio-integrity.c > > @@ -476,7 +476,6 @@ static int bio_integrity_verify(struct bio *bio) > > sector += sectors; > > prot_buf += sectors * bi->tuple_size; > > total += sectors * bi->tuple_size; > > - BUG_ON(total > bio->bi_integrity->bip_iter.bi_size); > > The variable 'total' is only used for the BUG_ON. So it can be removed. Duh, yes of course.. MKP + Jens, any concerns wrt dropping this BUG_ON for v3.14-rc code..? --nab -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
