Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp4273431imu; Mon, 14 Jan 2019 19:10:24 -0800 (PST) X-Google-Smtp-Source: ALg8bN5k/wWvikpJkghzZRZ+d3s0jb/MgEQEcm228LnSX8Bk/bmHypOzL0Y8B/AEjzB3T9bxLw8d X-Received: by 2002:a17:902:1127:: with SMTP id d36mr1739776pla.299.1547521824499; Mon, 14 Jan 2019 19:10:24 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1547521824; cv=none; d=google.com; s=arc-20160816; b=RLBIEZhyAirg7SBlA2+2IiTL+vA+zDSgNMjmuAAoj2psTgafyXk9fF+1Ntxk7hEfQM 3sa/hdDQjLrj1yyeUk7sIwQoHNzJ12Cl3NrywZ85Rw4OmzG/RA1Uoyqsi+VrDf9vkoqA V2y3+zc1QaalzQE7GrJ+cuCEkIjjsQ4hNycWgl/GyvDlY6A2d44iq7qdrFE9dF0NpFAL h/T8N0aFCt8GBM6TJdH+EMRZr7NQrhZNerUfigZ/b6DcPor2hlDlXa6FrmbG2fELNDMm YKjdwPIVXeKoOARBmMxywXbc7ougCvKEw9WVqPN64Z2/gpzwH/MLoa/w23vGjJxyA8rH LCmw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject; bh=JPV5waa8vhG2OmIFrCe+tl2KDRgoPjfLoXs93T7qfME=; b=tdya/8m4oJ4FDkl2QElk4iWIq1svuaXVEYV63RUnzpCV6JcOfeV2Vmm+4Ig/pihKbQ sz/nOKY8pQPWpFMrxNix5hnzG2qcH0YPjyBWOpfYDZILzHFwlI6kE4A2PEZG0Zh8E1VL ywoe4CNbYaBFgsLqymzIt/BPQ0ISv1EZvJxlU97acHsOLVdbWFp3771zow8MxQssy2IH f01x1oYo8ZdiE67hzPaV6oIXqj3hZ9gFOVW6jVATJiQXj033eqrwpZO+N4LM6fG27QJE DWzU48eKz6WFQfLcIEoJ0yjCFmla66hkUEu1NGI/yGjWpjY0ISpahrS2nyW0bRDrt5G5 3CLA== 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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id r81si2315167pfr.164.2019.01.14.19.10.08; Mon, 14 Jan 2019 19:10:24 -0800 (PST) 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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727841AbfAOCjR (ORCPT + 99 others); Mon, 14 Jan 2019 21:39:17 -0500 Received: from mail-qt1-f193.google.com ([209.85.160.193]:39421 "EHLO mail-qt1-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727200AbfAOCjR (ORCPT ); Mon, 14 Jan 2019 21:39:17 -0500 Received: by mail-qt1-f193.google.com with SMTP id u47so1447263qtj.6 for ; Mon, 14 Jan 2019 18:39:17 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=JPV5waa8vhG2OmIFrCe+tl2KDRgoPjfLoXs93T7qfME=; b=ERJbFPZhhdT19IPsdRci5MCdYvGR1mMtDoEISDbq5rJVpo3IC33ghDppkUDIATqfSS +N9yeT6hla9rWhCA8qfYTYkGHDB1Qi5tvTNHm7qs79jDO0x2YMCR+BBqiauAPotuPH1n Tod3LA6MhbEtyLCa4XEmkl+Ia3eSGSzAQpMxgrZ2GfT2hcx38Dg/qw5Fu71VtJGwm40z +Q/AP/r/db1okt5Eklfl4kskGdXvG5CqcVybwXvgBhusYtF7GKdxWh+8jLNUlLOH92hT g9vXKtW2KEB2H8hRjFzrVWvZnLX9Vbr2ZlCy2o1HvcHBxYQ+i2p+m+n7LguJ2KCstLdf glHw== X-Gm-Message-State: AJcUukdYdov66TNGsJlAiAgHNxfF+OZmXxYY1LzUlmxc9BRK310BEmQN 8QnzNs3PSdYGEiIUYeLCUAgC+ioLlvo= X-Received: by 2002:a0c:b596:: with SMTP id g22mr1250591qve.88.1547519956614; Mon, 14 Jan 2019 18:39:16 -0800 (PST) Received: from ?IPv6:2601:602:9800:dae6::fb21? ([2601:602:9800:dae6::fb21]) by smtp.gmail.com with ESMTPSA id p72sm41864779qke.87.2019.01.14.18.39.14 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 14 Jan 2019 18:39:15 -0800 (PST) Subject: Re: [PATCH 00/14] Misc ION cleanups and adding unmapped heap To: "Andrew F. Davis" , Sumit Semwal , Greg Kroah-Hartman , =?UTF-8?Q?Arve_Hj=c3=b8nnev=c3=a5g?= Cc: devel@driverdev.osuosl.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org References: <20190111180523.27862-1-afd@ti.com> From: Laura Abbott Message-ID: <3b0d05e1-d437-801b-1c41-97d55f9ac10f@redhat.com> Date: Mon, 14 Jan 2019 18:39:14 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 MIME-Version: 1.0 In-Reply-To: <20190111180523.27862-1-afd@ti.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 1/11/19 10:05 AM, Andrew F. Davis wrote: > Hello all, > > This is a set of (hopefully) non-controversial cleanups for the ION > framework and current set of heaps. These were found as I start to > familiarize myself with the framework to help in whatever way I > can in getting all this up to the standards needed for de-staging. > > I would like to get some ideas of what is left to work on to get ION > out of staging. Has there been some kind of agreement on what ION should > eventually end up being? To me it looks like it is being whittled away at > to it's most core functions. To me that is looking like being a DMA-BUF > user-space front end, simply advertising available memory backings in a > system and providing allocations as DMA-BUF handles. If this is the case > then it looks close to being ready to me at least, but I would love to > hear any other opinions and concerns. > Yes, at this point the only functionality that people are really depending on is the ability to allocate a dma_buf easily from userspace. > Back to this patchset, the last patch may be a bit different than the > others, it adds an unmapped heaps type and creation helper. I wanted to > get this in to show off another heap type and maybe some issues we may > have with the current ION framework. The unmapped heap is used when the > backing memory should not (or cannot) be touched. Currently this kind > of heap is used for firewalled secure memory that can be allocated like > normal heap memory but only used by secure devices (OP-TEE, crypto HW, > etc). It is basically just copied from the "carveout" heap type with the > only difference being it is not mappable to userspace and we do not clear > the memory (as we should not map it either). So should this really be a > new heap type? Or maybe advertised as a carveout heap but with an > additional allocation flag? Perhaps we do away with "types" altogether > and just have flags, coherent/non-coherent, mapped/unmapped, etc. > > Maybe more thinking will be needed afterall.. > So the cleanup looks okay (I need to finish reviewing) but I'm not a fan of adding another heaptype without solving the problem of adding some sort of devicetree binding or other method of allocating and placing Ion heaps. That plus uncached buffers are one of the big open problems that need to be solved for destaging Ion. See https://lore.kernel.org/lkml/20181120164636.jcw7li2uaa3cmwc3@DESKTOP-E1NTVVP.localdomain/ for some background on that problem. Thanks, Laura > Thanks, > Andrew > > Andrew F. Davis (14): > staging: android: ion: Add proper header information > staging: android: ion: Remove empty ion_ioctl_dir() function > staging: android: ion: Merge ion-ioctl.c into ion.c > staging: android: ion: Remove leftover comment > staging: android: ion: Remove struct ion_platform_heap > staging: android: ion: Fixup some white-space issues > staging: android: ion: Sync comment docs with struct ion_buffer > staging: android: ion: Remove base from ion_carveout_heap > staging: android: ion: Remove base from ion_chunk_heap > staging: android: ion: Remove unused headers > staging: android: ion: Allow heap name to be null > staging: android: ion: Declare helpers for carveout and chunk heaps > staging: android: ion: Do not sync CPU cache on map/unmap > staging: android: ion: Add UNMAPPED heap type and helper > > drivers/staging/android/ion/Kconfig | 10 ++ > drivers/staging/android/ion/Makefile | 3 +- > drivers/staging/android/ion/ion-ioctl.c | 98 -------------- > drivers/staging/android/ion/ion.c | 93 +++++++++++-- > drivers/staging/android/ion/ion.h | 87 ++++++++----- > .../staging/android/ion/ion_carveout_heap.c | 19 +-- > drivers/staging/android/ion/ion_chunk_heap.c | 25 ++-- > drivers/staging/android/ion/ion_cma_heap.c | 6 +- > drivers/staging/android/ion/ion_heap.c | 8 +- > drivers/staging/android/ion/ion_page_pool.c | 2 +- > drivers/staging/android/ion/ion_system_heap.c | 8 +- > .../staging/android/ion/ion_unmapped_heap.c | 123 ++++++++++++++++++ > drivers/staging/android/uapi/ion.h | 3 + > 13 files changed, 307 insertions(+), 178 deletions(-) > delete mode 100644 drivers/staging/android/ion/ion-ioctl.c > create mode 100644 drivers/staging/android/ion/ion_unmapped_heap.c >