block: Clean up the code used to generate and verify integrity metadata
[linux-2.6-block.git] / block / bio-integrity.c
index e84f7fb8694b674f4435cc42fc7f73617c658fd5..fe4de033b34c0479c84c8f2c5633c884ce45ee27 100644 (file)
@@ -185,95 +185,65 @@ bool bio_integrity_enabled(struct bio *bio)
 EXPORT_SYMBOL(bio_integrity_enabled);
 
 /**
- * bio_integrity_hw_sectors - Convert 512b sectors to hardware ditto
+ * bio_integrity_intervals - Return number of integrity intervals for a bio
  * @bi:                blk_integrity profile for device
- * @sectors:   Number of 512 sectors to convert
+ * @sectors:   Size of the bio in 512-byte sectors
  *
  * Description: The block layer calculates everything in 512 byte
- * sectors but integrity metadata is done in terms of the hardware
- * sector size of the storage device.  Convert the block layer sectors
- * to physical sectors.
+ * sectors but integrity metadata is done in terms of the data integrity
+ * interval size of the storage device.  Convert the block layer sectors
+ * to the appropriate number of integrity intervals.
  */
-static inline unsigned int bio_integrity_hw_sectors(struct blk_integrity *bi,
-                                                   unsigned int sectors)
+static inline unsigned int bio_integrity_intervals(struct blk_integrity *bi,
+                                                  unsigned int sectors)
 {
-       /* At this point there are only 512b or 4096b DIF/EPP devices */
-       if (bi->sector_size == 4096)
-               return sectors >>= 3;
-
-       return sectors;
+       return sectors >> (ilog2(bi->interval) - 9);
 }
 
 static inline unsigned int bio_integrity_bytes(struct blk_integrity *bi,
                                               unsigned int sectors)
 {
-       return bio_integrity_hw_sectors(bi, sectors) * bi->tuple_size;
+       return bio_integrity_intervals(bi, sectors) * bi->tuple_size;
 }
 
 /**
- * bio_integrity_generate_verify - Generate/verify integrity metadata for a bio
+ * bio_integrity_process - Process integrity metadata for a bio
  * @bio:       bio to generate/verify integrity metadata for
- * @operate:   operate number, 1 for generate, 0 for verify
+ * @proc_fn:   Pointer to the relevant processing function
  */
-static int bio_integrity_generate_verify(struct bio *bio, int operate)
+static int bio_integrity_process(struct bio *bio,
+                                integrity_processing_fn *proc_fn)
 {
        struct blk_integrity *bi = bdev_get_integrity(bio->bi_bdev);
-       struct blk_integrity_exchg bix;
+       struct blk_integrity_iter iter;
        struct bio_vec *bv;
        struct bio_integrity_payload *bip = bio_integrity(bio);
-       sector_t sector;
-       unsigned int sectors, ret = 0, i;
+       unsigned int i, ret = 0;
        void *prot_buf = page_address(bip->bip_vec->bv_page) +
                bip->bip_vec->bv_offset;
 
-       if (operate)
-               sector = bio->bi_iter.bi_sector;
-       else
-               sector = bip->bip_iter.bi_sector;
-
-       bix.disk_name = bio->bi_bdev->bd_disk->disk_name;
-       bix.sector_size = bi->sector_size;
+       iter.disk_name = bio->bi_bdev->bd_disk->disk_name;
+       iter.interval = bi->interval;
+       iter.seed = bip_get_seed(bip);
+       iter.prot_buf = prot_buf;
 
        bio_for_each_segment_all(bv, bio, i) {
                void *kaddr = kmap_atomic(bv->bv_page);
-               bix.data_buf = kaddr + bv->bv_offset;
-               bix.data_size = bv->bv_len;
-               bix.prot_buf = prot_buf;
-               bix.sector = sector;
-
-               if (operate)
-                       bi->generate_fn(&bix);
-               else {
-                       ret = bi->verify_fn(&bix);
-                       if (ret) {
-                               kunmap_atomic(kaddr);
-                               return ret;
-                       }
-               }
 
-               sectors = bv->bv_len / bi->sector_size;
-               sector += sectors;
-               prot_buf += sectors * bi->tuple_size;
+               iter.data_buf = kaddr + bv->bv_offset;
+               iter.data_size = bv->bv_len;
+
+               ret = proc_fn(&iter);
+               if (ret) {
+                       kunmap_atomic(kaddr);
+                       return ret;
+               }
 
                kunmap_atomic(kaddr);
        }
        return ret;
 }
 
-/**
- * bio_integrity_generate - Generate integrity metadata for a bio
- * @bio:       bio to generate integrity metadata for
- *
- * Description: Generates integrity metadata for a bio by calling the
- * block device's generation callback function.  The bio must have a
- * bip attached with enough room to accommodate the generated
- * integrity metadata.
- */
-static void bio_integrity_generate(struct bio *bio)
-{
-       bio_integrity_generate_verify(bio, 1);
-}
-
 /**
  * bio_integrity_prep - Prepare bio for integrity I/O
  * @bio:       bio to prepare
@@ -294,17 +264,17 @@ int bio_integrity_prep(struct bio *bio)
        unsigned long start, end;
        unsigned int len, nr_pages;
        unsigned int bytes, offset, i;
-       unsigned int sectors;
+       unsigned int intervals;
 
        bi = bdev_get_integrity(bio->bi_bdev);
        q = bdev_get_queue(bio->bi_bdev);
        BUG_ON(bi == NULL);
        BUG_ON(bio_integrity(bio));
 
-       sectors = bio_integrity_hw_sectors(bi, bio_sectors(bio));
+       intervals = bio_integrity_intervals(bi, bio_sectors(bio));
 
        /* Allocate kernel buffer for protection data */
-       len = sectors * bi->tuple_size;
+       len = intervals * bi->tuple_size;
        buf = kmalloc(len, GFP_NOIO | q->bounce_gfp);
        if (unlikely(buf == NULL)) {
                printk(KERN_ERR "could not allocate integrity buffer\n");
@@ -325,7 +295,7 @@ int bio_integrity_prep(struct bio *bio)
 
        bip->bip_owns_buf = 1;
        bip->bip_iter.bi_size = len;
-       bip->bip_iter.bi_sector = bio->bi_iter.bi_sector;
+       bip_set_seed(bip, bio->bi_iter.bi_sector);
 
        /* Map it */
        offset = offset_in_page(buf);
@@ -361,25 +331,12 @@ int bio_integrity_prep(struct bio *bio)
 
        /* Auto-generate integrity metadata if this is a write */
        if (bio_data_dir(bio) == WRITE)
-               bio_integrity_generate(bio);
+               bio_integrity_process(bio, bi->generate_fn);
 
        return 0;
 }
 EXPORT_SYMBOL(bio_integrity_prep);
 
-/**
- * bio_integrity_verify - Verify integrity metadata for a bio
- * @bio:       bio to verify
- *
- * Description: This function is called to verify the integrity of a
- * bio.         The data in the bio io_vec is compared to the integrity
- * metadata returned by the HBA.
- */
-static int bio_integrity_verify(struct bio *bio)
-{
-       return bio_integrity_generate_verify(bio, 0);
-}
-
 /**
  * bio_integrity_verify_fn - Integrity I/O completion worker
  * @work:      Work struct stored in bio to be verified
@@ -393,9 +350,10 @@ static void bio_integrity_verify_fn(struct work_struct *work)
        struct bio_integrity_payload *bip =
                container_of(work, struct bio_integrity_payload, bip_work);
        struct bio *bio = bip->bip_bio;
+       struct blk_integrity *bi = bdev_get_integrity(bio->bi_bdev);
        int error;
 
-       error = bio_integrity_verify(bio);
+       error = bio_integrity_process(bio, bi->verify_fn);
 
        /* Restore original bio completion handler */
        bio->bi_end_io = bip->bip_end_io;