From patchwork Sun Aug 25 06:22:51 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Marc Carino X-Patchwork-Id: 269685 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id BB2292C00A8 for ; Sun, 25 Aug 2013 16:23:23 +1000 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756005Ab3HYGXV (ORCPT ); Sun, 25 Aug 2013 02:23:21 -0400 Received: from mail-ob0-f175.google.com ([209.85.214.175]:49423 "EHLO mail-ob0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756040Ab3HYGXV (ORCPT ); Sun, 25 Aug 2013 02:23:21 -0400 Received: by mail-ob0-f175.google.com with SMTP id xn12so2169688obc.20 for ; Sat, 24 Aug 2013 23:23:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=GmOWD9eQ6v0Z+eRKKfwDw3/V9gKNCLIAN0KMNflaGys=; b=HoETF3pG3DACDqwDC25NTmmmeVVViS9GKdOIUMfmfCnzTzFiZELlbe2rJJZwsSg11R ixX+sT+8XEGd9mAUNZksBMTU6jsmCi911Cconu2ISZ84jOlQbP5rBUg8anCvH1YQ4ZUC oZ6nedLCr79atqcsuQD+31wdIFxC5iXdIJDk/DGJIf8O3VerAaYPEFx8opyif7jXhO2j VwMJdm6u7KFtXHNy3MlSsgBW4w4eR1sLCliEvtn4VvGusry4yOgl01a7nVR0mKPEtKOK X44Q7nPeIaMnrIVX/J8WgFPqcFK970zSatSJGezDmaOnuu+dYrDDQkteYzqVI1AWjer3 hY/w== X-Received: by 10.60.161.108 with SMTP id xr12mr8014695oeb.32.1377411800790; Sat, 24 Aug 2013 23:23:20 -0700 (PDT) Received: from localhost.localdomain (ip68-231-212-75.oc.oc.cox.net. [68.231.212.75]) by mx.google.com with ESMTPSA id r3sm8593074oep.2.1969.12.31.16.00.00 (version=TLSv1.1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Sat, 24 Aug 2013 23:23:20 -0700 (PDT) From: Marc C To: tj@kernel.org, linux-ide@vger.kernel.org Cc: Marc Carino Subject: [PATCH v5 4/4] libata: Add support for queued DSM TRIM Date: Sat, 24 Aug 2013 23:22:51 -0700 Message-Id: <1377411771-3733-5-git-send-email-marc.ceeeee@gmail.com> X-Mailer: git-send-email 1.8.1.2 In-Reply-To: <1377411771-3733-1-git-send-email-marc.ceeeee@gmail.com> References: <1377411771-3733-1-git-send-email-marc.ceeeee@gmail.com> Sender: linux-ide-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-ide@vger.kernel.org From: Marc Carino Some new SSDs support the queued version of the DSM TRIM command. Let the driver use the new command if supported. Signed-off-by: Marc Carino --- drivers/ata/libata-scsi.c | 25 +++++++++++++++++++------ include/linux/libata.h | 7 +++++++ 2 files changed, 26 insertions(+), 6 deletions(-) diff --git a/drivers/ata/libata-scsi.c b/drivers/ata/libata-scsi.c index 83c0890..0c807a9 100644 --- a/drivers/ata/libata-scsi.c +++ b/drivers/ata/libata-scsi.c @@ -3098,12 +3098,25 @@ static unsigned int ata_scsi_write_same_xlat(struct ata_queued_cmd *qc) buf = page_address(sg_page(scsi_sglist(scmd))); size = ata_set_lba_range_entries(buf, 512, block, n_block); - tf->protocol = ATA_PROT_DMA; - tf->hob_feature = 0; - tf->feature = ATA_DSM_TRIM; - tf->hob_nsect = (size / 512) >> 8; - tf->nsect = size / 512; - tf->command = ATA_CMD_DSM; + if (ata_ncq_enabled(dev) && ata_fpdma_dsm_supported(dev)) { + /* Newer devices support queued TRIM commands */ + tf->protocol = ATA_PROT_NCQ; + tf->command = ATA_CMD_FPDMA_SEND; + tf->hob_nsect = ATA_SUBCMD_FPDMA_SEND_DSM & 0x1f; + tf->nsect = qc->tag << 3; + tf->hob_feature = (size / 512) >> 8; + tf->feature = size / 512; + + tf->auxiliary = 1; + } else { + tf->protocol = ATA_PROT_DMA; + tf->hob_feature = 0; + tf->feature = ATA_DSM_TRIM; + tf->hob_nsect = (size / 512) >> 8; + tf->nsect = size / 512; + tf->command = ATA_CMD_DSM; + } + tf->flags |= ATA_TFLAG_ISADDR | ATA_TFLAG_DEVICE | ATA_TFLAG_LBA48 | ATA_TFLAG_WRITE; diff --git a/include/linux/libata.h b/include/linux/libata.h index b925504..a6f53f0 100644 --- a/include/linux/libata.h +++ b/include/linux/libata.h @@ -1608,6 +1608,13 @@ static inline int ata_ncq_enabled(struct ata_device *dev) ATA_DFLAG_NCQ)) == ATA_DFLAG_NCQ; } +static inline bool ata_fpdma_dsm_supported(struct ata_device *dev) +{ + return (dev->flags & ATA_DFLAG_NCQ_SEND_RECV) && + (dev->ncq_send_recv_cmds[ATA_LOG_NCQ_SEND_RECV_DSM_OFFSET] & + ATA_LOG_NCQ_SEND_RECV_DSM_TRIM); +} + static inline void ata_qc_set_polling(struct ata_queued_cmd *qc) { qc->tf.ctl |= ATA_NIEN;