From: Greg KH <gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org>
To: Laura Abbott <labbott-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
Cc: "Benjamin Gaignard"
<benjamin.gaignard-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
"Sumit Semwal"
<sumit.semwal-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
"Arve Hjønnevåg" <arve-z5hGa2qSFaRBDgjK7y7TUQ@public.gmane.org>,
"Riley Andrews"
<riandrews-z5hGa2qSFaRBDgjK7y7TUQ@public.gmane.org>,
"Mark Brown" <broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
"Dan Carpenter"
<dan.carpenter-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>,
driverdevel
<devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b@public.gmane.org>,
"Linux Kernel Mailing List"
<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org"
<dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>,
linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH v6 2/2] staging: ion: create one device entry per heap
Date: Wed, 6 Dec 2017 07:28:22 +0100 [thread overview]
Message-ID: <20171206062822.GB20995@kroah.com> (raw)
In-Reply-To: <65ef83b2-db3e-eaf6-129f-40e156400624-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.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? :)
WARNING: multiple messages have this Message-ID (diff)
From: Greg KH <gregkh@linuxfoundation.org>
To: Laura Abbott <labbott@redhat.com>
Cc: "Benjamin Gaignard" <benjamin.gaignard@linaro.org>,
"Sumit Semwal" <sumit.semwal@linaro.org>,
"Arve Hjønnevåg" <arve@android.com>,
"Riley Andrews" <riandrews@android.com>,
"Mark Brown" <broonie@kernel.org>,
"Dan Carpenter" <dan.carpenter@oracle.com>,
driverdevel <devel@driverdev.osuosl.org>,
"Linux Kernel Mailing List" <linux-kernel@vger.kernel.org>,
"dri-devel@lists.freedesktop.org"
<dri-devel@lists.freedesktop.org>,
linux-api@vger.kernel.org
Subject: Re: [PATCH v6 2/2] staging: ion: create one device entry per heap
Date: Wed, 6 Dec 2017 07:28:22 +0100 [thread overview]
Message-ID: <20171206062822.GB20995@kroah.com> (raw)
In-Reply-To: <65ef83b2-db3e-eaf6-129f-40e156400624@redhat.com>
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? :)
next prev parent reply other threads:[~2017-12-06 6:28 UTC|newest]
Thread overview: 60+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-06 15:59 [PATCH v6 0/2] staging: ion: get one device per heap Benjamin Gaignard
2017-11-06 15:59 ` Benjamin Gaignard
2017-11-06 15:59 ` [PATCH v6 1/2] staging: ion: reorder include Benjamin Gaignard
2017-11-06 15:59 ` Benjamin Gaignard
2017-11-09 21:02 ` Laura Abbott
2017-11-09 21:02 ` Laura Abbott
2017-11-28 13:20 ` Greg KH
2017-11-28 13:20 ` Greg KH
2017-11-28 13:34 ` Benjamin Gaignard
2017-11-28 13:34 ` Benjamin Gaignard
2017-11-28 13:40 ` Greg KH
2017-11-28 13:40 ` Greg KH
2017-11-06 15:59 ` [PATCH v6 2/2] staging: ion: create one device entry per heap Benjamin Gaignard
2017-11-09 21:17 ` Laura Abbott
2017-11-09 21:17 ` Laura Abbott
2017-11-27 10:46 ` Benjamin Gaignard
2017-11-27 10:46 ` Benjamin Gaignard
[not found] ` <CA+M3ks6vP+W77QBzbczxzcV7YZCoBusC=wgjZPUNLrCYFFDARA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-11-27 11:43 ` Greg Kroah-Hartman
2017-11-27 11:43 ` Greg Kroah-Hartman
2017-11-27 16:12 ` Daniel Vetter
2017-11-27 16:12 ` Daniel Vetter
2017-11-27 16:30 ` Mark Brown
2017-11-28 13:32 ` Greg KH
2017-11-28 13:32 ` Greg KH
[not found] ` <20171128133217.GA29995-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2017-11-28 16:26 ` Mark Brown
2017-11-28 16:26 ` Mark Brown
2017-11-28 17:08 ` Greg KH
2017-11-28 17:08 ` Greg KH
2017-11-28 17:12 ` Mark Brown
2017-11-28 17:12 ` Mark Brown
2017-11-28 17:28 ` Greg KH
2017-11-28 17:28 ` Greg KH
2017-11-28 17:37 ` Mark Brown
2017-11-28 17:37 ` Mark Brown
2017-11-28 18:00 ` Greg KH
2017-11-28 18:00 ` Greg KH
2017-11-29 14:00 ` Benjamin Gaignard
2017-11-29 14:00 ` Benjamin Gaignard
2017-12-02 15:53 ` Greg KH
2017-12-02 15:53 ` Greg KH
[not found] ` <20171202155317.GB5311-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2017-12-05 23:01 ` Laura Abbott
2017-12-05 23:01 ` Laura Abbott
[not found] ` <65ef83b2-db3e-eaf6-129f-40e156400624-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2017-12-06 6:28 ` Greg KH [this message]
2017-12-06 6:28 ` Greg KH
-- strict thread matches above, loose matches on Subject: below --
2017-10-23 15:55 [PATCH v6 0/2] staging: ion: get one device " Benjamin Gaignard
2017-10-23 15:55 ` [PATCH v6 2/2] staging: ion: create one device entry " Benjamin Gaignard
2017-10-23 15:55 ` Benjamin Gaignard
2017-10-24 16:14 ` Jordan Crouse
2017-10-31 19:03 ` Laura Abbott
2017-10-31 19:03 ` Laura Abbott
[not found] ` <3e3af8e3-76c5-754c-caaa-3f8c9b54723f-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2017-10-31 19:11 ` Mark Brown
2017-10-31 19:11 ` Mark Brown
2017-10-31 19:45 ` Laura Abbott
2017-10-31 19:45 ` Laura Abbott
2017-11-02 10:44 ` Greg KH
2017-11-02 11:10 ` Mark Brown
2017-11-02 11:10 ` Mark Brown
2017-11-06 14:42 ` Benjamin Gaignard
2017-11-06 14:42 ` Benjamin Gaignard
[not found] ` <CA+M3ks5pV=j_DeQ9ohhtv62WL=pzWH0Q5jdKtQby5vs=NUZT-Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-11-06 14:46 ` Greg KH
2017-11-06 14:46 ` Greg KH
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171206062822.GB20995@kroah.com \
--to=gregkh-hqyy1w1ycw8ekmwlsbkhg0b+6bgklq7r@public.gmane.org \
--cc=arve-z5hGa2qSFaRBDgjK7y7TUQ@public.gmane.org \
--cc=benjamin.gaignard-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
--cc=broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=dan.carpenter-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org \
--cc=devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b@public.gmane.org \
--cc=dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
--cc=labbott-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
--cc=linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=riandrews-z5hGa2qSFaRBDgjK7y7TUQ@public.gmane.org \
--cc=sumit.semwal-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.