Thread (23 messages) flat view 23 messages, 5 authors, 2017-12-06

Re: [PATCH v6 2/2] staging: ion: create one device entry per heap

From: Daniel Vetter <hidden>
Date: 2017-11-27 16:12:29
Also in: dri-devel, lkml

On Mon, Nov 27, 2017 at 12:43:57PM +0100, Greg Kroah-Hartman wrote:
On Mon, Nov 27, 2017 at 11:46:18AM +0100, Benjamin Gaignard wrote:
quoted
2017-11-09 22:17 GMT+01:00 Laura Abbott [off-list ref]:
quoted
On 11/06/2017 07:59 AM, Benjamin Gaignard wrote:
quoted
Instead a getting only one common device "/dev/ion" for
all the heaps this patch allow to create one device
entry ("/dev/ionX") per heap.
Getting an entry per heap could allow to set security rules
per heap and global ones for all heaps.

Allocation requests will be only allowed if the mask_id
match with device minor.
Query request could be done on any of the devices.
With this patch, sysfs looks like:

$ ls /sys/devices/
breakpoint ion platform software system virtual

From an Ion perspective, you can have

Acked-by: Laura Abbott <redacted>

Another Ack for the device model stuff would be good but I'll
assume deafening silence means nobody hates it.
Greg, can we get your point of view of this ?
It's 1 day after the merge window has closed, and my todo patch queue
looks like this:
	$ mdfrm -c ~/mail/todo/
	1523 messages in /home/gregkh/mail/todo/

Please give me a chance to catch up...
commit model ftw, we have 400+ patches for 4.16 already merged and tested
and all ready, right when -rc1 gets tagged. Makes the merge window the
most relaxed time of all, because all the other maintainers are drowning
and wont pester you.

Just saying, this is an entirely fixable problem :-P

Cheers, Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help