From patchwork Mon Aug 29 17:30:51 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: John Stultz X-Patchwork-Id: 74920 Delivered-To: patches@linaro.org Received: by 10.140.29.52 with SMTP id a49csp1742178qga; Mon, 29 Aug 2016 10:31:16 -0700 (PDT) X-Received: by 10.66.144.200 with SMTP id so8mr34456021pab.94.1472491869744; Mon, 29 Aug 2016 10:31:09 -0700 (PDT) Return-Path: Received: from mail-pf0-x22d.google.com (mail-pf0-x22d.google.com. [2607:f8b0:400e:c00::22d]) by mx.google.com with ESMTPS id ur7si22803607pab.3.2016.08.29.10.31.09 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 29 Aug 2016 10:31:09 -0700 (PDT) Received-SPF: pass (google.com: domain of john.stultz@linaro.org designates 2607:f8b0:400e:c00::22d as permitted sender) client-ip=2607:f8b0:400e:c00::22d; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org; spf=pass (google.com: domain of john.stultz@linaro.org designates 2607:f8b0:400e:c00::22d as permitted sender) smtp.mailfrom=john.stultz@linaro.org; dmarc=pass (p=NONE dis=NONE) header.from=linaro.org Received: by mail-pf0-x22d.google.com with SMTP id p64so54182165pfb.1 for ; Mon, 29 Aug 2016 10:31:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=jGbwspKLNyHNbWmNmY7LTV+j8rS6M1hQD1298eaOjMY=; b=V8R91fgJjcrKm++u/7fjcBIOXkjpaIlCc1teJmA972kJ1vah0KlhEffCzbPojAg6kS 3NJUfyFajBU+P0xLB/jkkSeUjOcIlTcHkRDHEGhuCgOUooAkMgl9FccVJwze1Fm3caun luLJyDvQJYpLvyIGcQdUUfhZZZEeTafj8k59U= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=jGbwspKLNyHNbWmNmY7LTV+j8rS6M1hQD1298eaOjMY=; b=FLlJFk7OzXn0sNI8Rof909j7joARhZBUojFCoj4nkk3CXCBnUCzyv5y1zgE6libtWj 1pbYKY+W8CBa823ZEq/hQu+VqhzeAOl7huxzWTkZCCgMZSKKg3IlIWRrT4HWIPIi57cn x/JpQW2kH6GNz98BewkjiIK5oC7HT4RBtlua5vR72SiLnuO6vp8aPduTQdAadukgxkFB Ix3BI9qRyXNGUCKGL99YaWNG0Uu7MgkzBk0O14ecXlNg6f/USn8ttwUMW6uVcipXalq+ ZmScmWBu1ugeZ+8nwNKj8/98I4gcKS90uVLt9T+J46c7JW4Z/uYPZ/mXgu/GmT1hJ17R EzJA== X-Gm-Message-State: AE9vXwOnZ3x2NiKNteo6NVSdUldKwGLwKnxYtNTu/2LfWuPiLaLJw/K/CFdwGS+EjekAAdb5mx8= X-Received: by 10.98.27.11 with SMTP id b11mr34829805pfb.111.1472491869471; Mon, 29 Aug 2016 10:31:09 -0700 (PDT) Return-Path: Received: from localhost.localdomain (c-73-67-244-238.hsd1.or.comcast.net. [73.67.244.238]) by smtp.gmail.com with ESMTPSA id j2sm50794464paa.46.2016.08.29.10.31.08 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Mon, 29 Aug 2016 10:31:08 -0700 (PDT) From: John Stultz To: lkml Cc: John Stultz , Zhangfei Gao , Krzysztof Kozlowski , Maxime Ripard , Vinod Koul , Dan Williams , Mark Brown , Andy Green Subject: [PATCH 5/7 v5] k3dma: Fix memory handling in preparation for cyclic mode Date: Mon, 29 Aug 2016 10:30:51 -0700 Message-Id: <1472491853-15644-6-git-send-email-john.stultz@linaro.org> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1472491853-15644-1-git-send-email-john.stultz@linaro.org> References: <1472491853-15644-1-git-send-email-john.stultz@linaro.org> With cyclic mode, the shared virt-dma logic doesn't actually manage the descriptor state, nor the calling of the descriptor free callback. This results in leaking a desc structure every time we start an audio transfer. Thus we must manage it ourselves. The k3dma driver already keeps track of the active and finished descriptors via ds_run and ds_done pointers, so cleanup how we handle those two values, so when we tear down everything in terminate_all, call free_desc on the ds_run and ds_done pointers if they are not null. NOTE: HiKey doesn't use the non-cyclic dma modes, so I'm not been able to test those modes. But with this patch we no longer leak the desc structures. Cc: Zhangfei Gao Cc: Krzysztof Kozlowski Cc: Maxime Ripard Cc: Vinod Koul Cc: Dan Williams Cc: Mark Brown Cc: Andy Green Acked-by: Zhangfei Gao Signed-off-by: John Stultz --- drivers/dma/k3dma.c | 38 ++++++++++++++++++++++++-------------- 1 file changed, 24 insertions(+), 14 deletions(-) -- 1.9.1 diff --git a/drivers/dma/k3dma.c b/drivers/dma/k3dma.c index 9d96c95..8108fa1 100644 --- a/drivers/dma/k3dma.c +++ b/drivers/dma/k3dma.c @@ -212,7 +212,9 @@ static irqreturn_t k3_dma_int_handler(int irq, void *dev_id) spin_lock_irqsave(&c->vc.lock, flags); vchan_cookie_complete(&p->ds_run->vd); + WARN_ON_ONCE(p->ds_done); p->ds_done = p->ds_run; + p->ds_run = NULL; spin_unlock_irqrestore(&c->vc.lock, flags); } irq_chan |= BIT(i); @@ -253,14 +255,14 @@ static int k3_dma_start_txd(struct k3_dma_chan *c) * so vc->desc_issued only contains desc pending */ list_del(&ds->vd.node); + + WARN_ON_ONCE(c->phy->ds_run); + WARN_ON_ONCE(c->phy->ds_done); c->phy->ds_run = ds; - c->phy->ds_done = NULL; /* start dma */ k3_dma_set_desc(c->phy, &ds->desc_hw[0]); return 0; } - c->phy->ds_done = NULL; - c->phy->ds_run = NULL; return -EAGAIN; } @@ -594,6 +596,16 @@ static int k3_dma_config(struct dma_chan *chan, return 0; } +static void k3_dma_free_desc(struct virt_dma_desc *vd) +{ + struct k3_dma_desc_sw *ds = + container_of(vd, struct k3_dma_desc_sw, vd); + struct k3_dma_dev *d = to_k3_dma(vd->tx.chan->device); + + dma_pool_free(d->pool, ds->desc_hw, ds->desc_hw_lli); + kfree(ds); +} + static int k3_dma_terminate_all(struct dma_chan *chan) { struct k3_dma_chan *c = to_k3_chan(chan); @@ -617,7 +629,15 @@ static int k3_dma_terminate_all(struct dma_chan *chan) k3_dma_terminate_chan(p, d); c->phy = NULL; p->vchan = NULL; - p->ds_run = p->ds_done = NULL; + if (p->ds_run) { + k3_dma_free_desc(&p->ds_run->vd); + p->ds_run = NULL; + } + if (p->ds_done) { + k3_dma_free_desc(&p->ds_done->vd); + p->ds_done = NULL; + } + } spin_unlock_irqrestore(&c->vc.lock, flags); vchan_dma_desc_free_list(&c->vc, &head); @@ -670,16 +690,6 @@ static int k3_dma_transfer_resume(struct dma_chan *chan) return 0; } -static void k3_dma_free_desc(struct virt_dma_desc *vd) -{ - struct k3_dma_desc_sw *ds = - container_of(vd, struct k3_dma_desc_sw, vd); - struct k3_dma_dev *d = to_k3_dma(vd->tx.chan->device); - - dma_pool_free(d->pool, ds->desc_hw, ds->desc_hw_lli); - kfree(ds); -} - static const struct of_device_id k3_pdma_dt_ids[] = { { .compatible = "hisilicon,k3-dma-1.0", }, {}