target: make iblock_emulate_sync_cache asynchronous
authorChristoph Hellwig <hch@infradead.org>
Fri, 14 Oct 2011 11:29:58 +0000 (07:29 -0400)
committerNicholas Bellinger <nab@linux-iscsi.org>
Mon, 24 Oct 2011 03:21:29 +0000 (03:21 +0000)
Do not block the submitting thread when handling a SYNCHRONIZE CACHE command,
but implement it asynchronously by sending the FLUSH command ourself and
calling transport_complete_sync_cache from the completion handler.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
drivers/target/target_core_iblock.c

index bf074c4b7a368374bcd7e71a2f84f82877e70f66..d9ad2a216eaa1968bd67509c03908e909a4bfd12 100644 (file)
@@ -313,37 +313,42 @@ static unsigned long long iblock_emulate_read_cap_with_block_size(
        return blocks_long;
 }
 
+static void iblock_end_io_flush(struct bio *bio, int err)
+{
+       struct se_cmd *cmd = bio->bi_private;
+
+       if (err)
+               pr_err("IBLOCK: cache flush failed: %d\n", err);
+
+       if (cmd)
+               transport_complete_sync_cache(cmd, err == 0);
+       bio_put(bio);
+}
+
 /*
- * Emulate SYCHRONIZE_CACHE_*
+ * Implement SYCHRONIZE CACHE.  Note that we can't handle lba ranges and must
+ * always flush the whole cache.
  */
 static void iblock_emulate_sync_cache(struct se_task *task)
 {
        struct se_cmd *cmd = task->task_se_cmd;
        struct iblock_dev *ib_dev = cmd->se_dev->dev_ptr;
        int immed = (cmd->t_task_cdb[1] & 0x2);
-       sector_t error_sector;
-       int ret;
+       struct bio *bio;
 
        /*
         * If the Immediate bit is set, queue up the GOOD response
-        * for this SYNCHRONIZE_CACHE op
+        * for this SYNCHRONIZE_CACHE op.
         */
        if (immed)
                transport_complete_sync_cache(cmd, 1);
 
-       /*
-        * blkdev_issue_flush() does not support a specifying a range, so
-        * we have to flush the entire cache.
-        */
-       ret = blkdev_issue_flush(ib_dev->ibd_bd, GFP_KERNEL, &error_sector);
-       if (ret != 0) {
-               pr_err("IBLOCK: block_issue_flush() failed: %d "
-                       " error_sector: %llu\n", ret,
-                       (unsigned long long)error_sector);
-       }
-
+       bio = bio_alloc(GFP_KERNEL, 0);
+       bio->bi_end_io = iblock_end_io_flush;
+       bio->bi_bdev = ib_dev->ibd_bd;
        if (!immed)
-               transport_complete_sync_cache(cmd, ret == 0);
+               bio->bi_private = cmd;
+       submit_bio(WRITE_FLUSH, bio);
 }
 
 /*