Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752090Ab3HTU0p (ORCPT ); Tue, 20 Aug 2013 16:26:45 -0400 Received: from mail-oa0-f46.google.com ([209.85.219.46]:57872 "EHLO mail-oa0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751641Ab3HTUYA (ORCPT ); Tue, 20 Aug 2013 16:24:00 -0400 From: "Nicholas A. Bellinger" To: target-devel Cc: lkml , linux-scsi , Christoph Hellwig , Hannes Reinecke , Martin Petersen , Chris Mason , James Bottomley , Nicholas Bellinger , Nicholas Bellinger Subject: [PATCH 3/9] target: Add memory allocation for bidirectional commands Date: Tue, 20 Aug 2013 20:07:54 +0000 Message-Id: <1377029280-19144-4-git-send-email-nab@daterainc.com> X-Mailer: git-send-email 1.7.2.5 In-Reply-To: <1377029280-19144-1-git-send-email-nab@daterainc.com> References: <1377029280-19144-1-git-send-email-nab@daterainc.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2915 Lines: 98 From: Nicholas Bellinger This adds transport_generic_get_mem_bidi() to perform scatterlist allocation for bidirectional commands. Also, update transport_generic_new_cmd() to call this new function when SCF_PASSTHROUGH_SG_TO_MEM_NOALLOC has not been set. Cc: Christoph Hellwig Cc: Hannes Reinecke Cc: Martin Petersen Cc: Chris Mason Cc: James Bottomley Cc: Nicholas Bellinger Signed-off-by: Nicholas Bellinger --- drivers/target/target_core_transport.c | 53 ++++++++++++++++++++++++++++++++ 1 file changed, 53 insertions(+) diff --git a/drivers/target/target_core_transport.c b/drivers/target/target_core_transport.c index 53d1d75..5746d85 100644 --- a/drivers/target/target_core_transport.c +++ b/drivers/target/target_core_transport.c @@ -2092,6 +2092,53 @@ void transport_kunmap_data_sg(struct se_cmd *cmd) EXPORT_SYMBOL(transport_kunmap_data_sg); static int +transport_generic_get_mem_bidi(struct se_cmd *cmd) +{ + struct se_device *dev = cmd->se_dev; + struct page *page; + gfp_t zero_flag; + u32 length; + unsigned int nents; + int i = 0; + + if (cmd->t_task_cdb[0] == COMPARE_AND_WRITE) + length = cmd->t_task_nolb * dev->dev_attrib.block_size; + else + length = cmd->data_length; + + nents = DIV_ROUND_UP(length, PAGE_SIZE); + cmd->t_bidi_data_sg = kmalloc(sizeof(struct scatterlist) * nents, GFP_KERNEL); + if (!cmd->t_bidi_data_sg) + return -ENOMEM; + + cmd->t_bidi_data_nents = nents; + sg_init_table(cmd->t_bidi_data_sg, nents); + + zero_flag = cmd->se_cmd_flags & SCF_SCSI_DATA_CDB ? 0 : __GFP_ZERO; + + while (length) { + u32 page_len = min_t(u32, length, PAGE_SIZE); + page = alloc_page(GFP_KERNEL | zero_flag); + if (!page) + goto out; + + sg_set_page(&cmd->t_bidi_data_sg[i], page, page_len, 0); + length -= page_len; + i++; + } + return 0; + +out: + while (i > 0) { + i--; + __free_page(sg_page(&cmd->t_bidi_data_sg[i])); + } + kfree(cmd->t_bidi_data_sg); + cmd->t_bidi_data_sg = NULL; + return -ENOMEM; +} + +static int transport_generic_get_mem(struct se_cmd *cmd) { u32 length = cmd->data_length; @@ -2149,6 +2196,12 @@ transport_generic_new_cmd(struct se_cmd *cmd) */ if (!(cmd->se_cmd_flags & SCF_PASSTHROUGH_SG_TO_MEM_NOALLOC) && cmd->data_length) { + if (cmd->se_cmd_flags & SCF_BIDI) { + ret = transport_generic_get_mem_bidi(cmd); + if (ret < 0) + return TCM_LOGICAL_UNIT_COMMUNICATION_FAILURE; + } + ret = transport_generic_get_mem(cmd); if (ret < 0) return TCM_LOGICAL_UNIT_COMMUNICATION_FAILURE; -- 1.7.10.4 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/