From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752876AbdLFG2S (ORCPT ); Wed, 6 Dec 2017 01:28:18 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:48644 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752519AbdLFG2R (ORCPT ); Wed, 6 Dec 2017 01:28:17 -0500 Date: Wed, 6 Dec 2017 07:28:22 +0100 From: Greg KH To: Laura Abbott Cc: Benjamin Gaignard , Sumit Semwal , Arve =?iso-8859-1?B?SGr4bm5lduVn?= , Riley Andrews , Mark Brown , Dan Carpenter , driverdevel , Linux Kernel Mailing List , "dri-devel@lists.freedesktop.org" , linux-api@vger.kernel.org Subject: Re: [PATCH v6 2/2] staging: ion: create one device entry per heap Message-ID: <20171206062822.GB20995@kroah.com> References: <1509983985-20950-1-git-send-email-benjamin.gaignard@linaro.org> <1509983985-20950-3-git-send-email-benjamin.gaignard@linaro.org> <20171128133217.GA29995@kroah.com> <20171202155317.GB5311@kroah.com> <65ef83b2-db3e-eaf6-129f-40e156400624@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <65ef83b2-db3e-eaf6-129f-40e156400624@redhat.com> User-Agent: Mutt/1.9.1 (2017-09-22) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Dec 05, 2017 at 03:01:42PM -0800, Laura Abbott wrote: > On 12/02/2017 07:53 AM, Greg KH wrote: > > > This is one of the item in the TODO list before been able to unstage ION > > > which is my real need. > > Why does it matter where in the tree this code is? Don't go adding new > > things to it that are not needed. Who needs this? What userspace code > > wants this type of multiple ion devices? > > > > Requirements came in from several places to split /dev/ion -> /dev/ion0 > and /dev/ion1 so that security policy (i.e. selinux) could be used to > protect access to certain heaps. I wanted the ABI to be settled before > trying to move out of staging, hence the line in the TODO list about > doing the split. Ok, but we should have some way of testing it works, right? :)