From patchwork Fri Mar 29 18:40:16 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Zeng Tao X-Patchwork-Id: 161421 Delivered-To: patch@linaro.org Received: by 2002:a02:c6d8:0:0:0:0:0 with SMTP id r24csp1860269jan; Fri, 29 Mar 2019 03:40:04 -0700 (PDT) X-Google-Smtp-Source: APXvYqxP+CIZ9cnOOIAxxaLGhwZaMAcZR4QJzurRBtVUGB7M2DhHGb5gwIuTlQy04qqOK7T/9bsN X-Received: by 2002:aa7:8092:: with SMTP id v18mr30641880pff.35.1553856004097; Fri, 29 Mar 2019 03:40:04 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1553856004; cv=none; d=google.com; s=arc-20160816; b=Nc6NRLbLIUxtZROg7tKobk/7NZEIchHkmhK16L6c3sKEmqMwheP+nVR/7/GPSzGT8q EoYDy/LkyT4bq5DVp4yhql8tKQMz3GghGhKS+cpdt7MCLPYR3GIS5Mu1PC2cE26W5Lbg +lT99l40zKz2wXFFMelRVdSO2rfqCl7sP5EnrBI6I2ldBSePEXdo/iyYoZfG+7IU4rL1 dgBf9SSbfBOs4NpshJGDuquH5xwrREcr7Bn375izkrZb5d0g/RRuL7o47pvzaBJd8tAQ K4oOqm8JdxMgnNONQchYVFJ+QFud3sGZAaCISfpJ0PybedMPJCcIsSqEd/FVN4sqBbnE HWJg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:message-id:date:subject:cc :to:from; bh=0DtOUBEvYZCKxZrk9nPKHtyauo/AEdT9/qrLYDk5oL4=; b=SwEVwsghdkzAYyG/nGArL6m9DqEXiO36/VLu+nA82K/KSYgfzfUOUl1H73xu9OHHMf KTMG8+6hl2b1mcUqUlzwpBqSBGnP0kFQ+3XnvXdcjP1ujiTpXBHaFad2drFUM/mlKVuK ybIyjgDdsvH4OzK8+K8RCAuHxiZv8bjbsNuaGK13CkO22+GRLXVVRl8LTwhNMWwdP6zF TZejJUn+jl0vgbO95XK6WcsHyonoH94rJ1+Kgs14UkFdqjZYqbzCt6GyW9sOoGZTKkYm eCSxiQnXcCnu3HjmyKiXpX439B57PN3Q+9g+Q51Jh+SdFiuUenkAo/3Ti+bmqqHhsj8I H0lw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id x20si1463099pfm.282.2019.03.29.03.40.03; Fri, 29 Mar 2019 03:40:04 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729104AbfC2KkC (ORCPT + 31 others); Fri, 29 Mar 2019 06:40:02 -0400 Received: from szxga04-in.huawei.com ([45.249.212.190]:5205 "EHLO huawei.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1729050AbfC2KkB (ORCPT ); Fri, 29 Mar 2019 06:40:01 -0400 Received: from DGGEMS404-HUB.china.huawei.com (unknown [172.30.72.60]) by Forcepoint Email with ESMTP id 5A82454DC83587D6ECD0; Fri, 29 Mar 2019 18:39:58 +0800 (CST) Received: from dessert.huawei.com (10.69.192.158) by DGGEMS404-HUB.china.huawei.com (10.3.19.204) with Microsoft SMTP Server id 14.3.408.0; Fri, 29 Mar 2019 18:39:49 +0800 From: Zeng Tao To: , CC: , Greg Kroah-Hartman , =?utf-8?q?Arve_Hj=C3=B8nnev=C3=A5g?= , Todd Kjos , Martijn Coenen , Joel Fernandes , "Christian Brauner" , , , , Subject: [PATCH] staging: android: ion: refactory ion_alloc for kernel driver use Date: Sat, 30 Mar 2019 02:40:16 +0800 Message-ID: <1553884816-37850-1-git-send-email-prime.zeng@hisilicon.com> X-Mailer: git-send-email 2.7.4 MIME-Version: 1.0 X-Originating-IP: [10.69.192.158] X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org There are two reasons for this patch: 1. There are some potential requirements for ion_alloc in kernel space, some media drivers need to allocate media buffers from ion instead of buddy or dma framework, this is more convient and clean very for media drivers. And In that case, ion is the only media buffer provider, it's more easier to maintain. 2. Fd is only needed by user processes, not the kernel space, so dma_buf should be returned instead of fd for kernel space, and dma_buf_fd should be called only for userspace api. Signed-off-by: Zeng Tao --- drivers/staging/android/ion/ion.c | 32 +++++++++++++++++--------------- 1 file changed, 17 insertions(+), 15 deletions(-) -- 2.7.4 diff --git a/drivers/staging/android/ion/ion.c b/drivers/staging/android/ion/ion.c index 92c2914..e93fb49 100644 --- a/drivers/staging/android/ion/ion.c +++ b/drivers/staging/android/ion/ion.c @@ -387,13 +387,13 @@ static const struct dma_buf_ops dma_buf_ops = { .unmap = ion_dma_buf_kunmap, }; -static int ion_alloc(size_t len, unsigned int heap_id_mask, unsigned int flags) +struct dma_buf *ion_alloc(size_t len, unsigned int heap_id_mask, + unsigned int flags) { struct ion_device *dev = internal_dev; struct ion_buffer *buffer = NULL; struct ion_heap *heap; DEFINE_DMA_BUF_EXPORT_INFO(exp_info); - int fd; struct dma_buf *dmabuf; pr_debug("%s: len %zu heap_id_mask %u flags %x\n", __func__, @@ -407,7 +407,7 @@ static int ion_alloc(size_t len, unsigned int heap_id_mask, unsigned int flags) len = PAGE_ALIGN(len); if (!len) - return -EINVAL; + return ERR_PTR(-EINVAL); down_read(&dev->lock); plist_for_each_entry(heap, &dev->heaps, node) { @@ -421,10 +421,10 @@ static int ion_alloc(size_t len, unsigned int heap_id_mask, unsigned int flags) up_read(&dev->lock); if (!buffer) - return -ENODEV; + return ERR_PTR(-ENODEV); if (IS_ERR(buffer)) - return PTR_ERR(buffer); + return ERR_PTR(PTR_ERR(buffer)); exp_info.ops = &dma_buf_ops; exp_info.size = buffer->size; @@ -432,17 +432,12 @@ static int ion_alloc(size_t len, unsigned int heap_id_mask, unsigned int flags) exp_info.priv = buffer; dmabuf = dma_buf_export(&exp_info); - if (IS_ERR(dmabuf)) { + if (IS_ERR(dmabuf)) _ion_buffer_destroy(buffer); - return PTR_ERR(dmabuf); - } - fd = dma_buf_fd(dmabuf, O_CLOEXEC); - if (fd < 0) - dma_buf_put(dmabuf); - - return fd; + return dmabuf; } +EXPORT_SYMBOL(ion_alloc); static int ion_query_heaps(struct ion_heap_query *query) { @@ -539,12 +534,19 @@ static long ion_ioctl(struct file *filp, unsigned int cmd, unsigned long arg) case ION_IOC_ALLOC: { int fd; + struct dma_buf *dmabuf; - fd = ion_alloc(data.allocation.len, + dmabuf = ion_alloc(data.allocation.len, data.allocation.heap_id_mask, data.allocation.flags); - if (fd < 0) + if (IS_ERR(dmabuf)) + return PTR_ERR(dmabuf); + + fd = dma_buf_fd(dmabuf, O_CLOEXEC); + if (fd < 0) { + dma_buf_put(dmabuf); return fd; + } data.allocation.fd = fd;