From: Nicholas Bellinger <nab@xxxxxxxxxxxxxxx> This patch adds UNMAP emulation support using Block layer DISCARD in iblock_emulate_scsi_cdb() -> transport_generic_unmap(). This includes the use of blk_queue_discard() in iblock_create_virtdevice() to determine when to set DEV_ATTRIB(dev)->emulate_tpe=1 to signal to TCM Core to perform the necessary control CDB emulation for TPE=1 / UNMAP. Signed-off-by: Nicholas A. Bellinger <nab@xxxxxxxxxxxxxxx> --- drivers/target/target_core_iblock.c | 19 ++++++++++++++++++- 1 files changed, 18 insertions(+), 1 deletions(-) diff --git a/drivers/target/target_core_iblock.c b/drivers/target/target_core_iblock.c index 1b148f5..089f353 100644 --- a/drivers/target/target_core_iblock.c +++ b/drivers/target/target_core_iblock.c @@ -221,6 +221,16 @@ static struct se_device *iblock_create_virtdevice( */ if (__iblock_do_sync_cache(dev) == 0) ib_dev->ibd_flags |= IBDF_BDEV_ISSUE_FLUSH; + /* + * Check if the underlying struct block_device request_queue supports + * the QUEUE_FLAG_DISCARD bit for UNMAP/WRITE_SAME in SCSI + TRIM + * in ATA and we need to set TPE=1 + */ + if (blk_queue_discard(bdev_get_queue(bd))) { + DEV_ATTRIB(dev)->emulate_tpe = 1; + printk(KERN_INFO "IBLOCK: Enabling BLOCK Discard" + " and TPE=1 emulation\n"); + } return dev; @@ -446,8 +456,10 @@ static int iblock_emulate_read_cap16(struct se_task *task) static int iblock_emulate_scsi_cdb(struct se_task *task) { - int ret; + struct iblock_dev *ibd = (struct iblock_dev *) task->se_dev->dev_ptr; + struct block_device *bd = ibd->ibd_bd; struct se_cmd *cmd = TASK_CMD(task); + int ret; switch (T_TASK(cmd)->t_task_cdb[0]) { case INQUIRY: @@ -490,6 +502,11 @@ static int iblock_emulate_scsi_cdb(struct se_task *task) if (ret < 0) return ret; break; + case UNMAP: + ret = transport_generic_unmap(cmd, bd); + if (ret < 0) + return ret; + break; case ALLOW_MEDIUM_REMOVAL: case ERASE: case REZERO_UNIT: -- 1.5.6.5 -- 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