Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-13 Thread Minchan Kim
On 05/12/2012 01:48 PM, InKi Dae wrote: 2012/5/12 KOSAKI Motohiro kosaki.motoh...@gmail.com: On Fri, May 11, 2012 at 7:29 PM, Jerome Glisse j.gli...@gmail.com wrote: On Fri, May 11, 2012 at 6:59 PM, KOSAKI Motohiro kosaki.motoh...@gmail.com wrote: My point is this ioctl will be restricted to

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-12 Thread KOSAKI Motohiro
(5/10/12 11:01 PM), Jerome Glisse wrote: On Thu, May 10, 2012 at 10:51 PM, KOSAKI Motohiro kosaki.motoh...@gmail.com wrote: (5/10/12 8:50 PM), Minchan Kim wrote: Hi KOSAKI, On 05/11/2012 02:53 AM, KOSAKI Motohiro wrote: let's assume that one application want to allocate user space memory

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-12 Thread KOSAKI Motohiro
My point is this ioctl will be restricted to one user (Xserver if i understand) and only this user, there is no fork in it so no need to worry about fork, just setting the vma as locked will be enough. But i don't want people reading this driver suddenly think that what it's doing is ok, it's

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-12 Thread KOSAKI Motohiro
On Fri, May 11, 2012 at 7:29 PM, Jerome Glisse j.gli...@gmail.com wrote: On Fri, May 11, 2012 at 6:59 PM, KOSAKI Motohiro kosaki.motoh...@gmail.com wrote: My point is this ioctl will be restricted to one user (Xserver if i understand) and only this user, there is no fork in it so no need to

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-11 Thread KOSAKI Motohiro
let's assume that one application want to allocate user space memory region using malloc() and then write something on the region. as you may know, user space buffer doen't have real physical pages once malloc() call so if user tries to access the region then page fault handler would be triggered

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-11 Thread Minchan Kim
Hi KOSAKI, On 05/11/2012 02:53 AM, KOSAKI Motohiro wrote: let's assume that one application want to allocate user space memory region using malloc() and then write something on the region. as you may know, user space buffer doen't have real physical pages once malloc() call so if user tries

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-11 Thread Minchan Kim
Subject: Re: [PATCH 2/2 v3] drm/exynos: added userptr feature. On Thu, May 10, 2012 at 11:31 AM, Daniel Vetter dan...@ffwll.ch wrote: On Thu, May 10, 2012 at 11:05:07AM -0400, Jerome Glisse wrote: On Wed, May 9, 2012 at 10:44 PM, Inki Dae inki@samsung.com wrote: Hi Jerome, Thank you again

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-11 Thread Jerome Glisse
On Fri, May 11, 2012 at 5:20 PM, KOSAKI Motohiro kosaki.motoh...@gmail.com wrote: (5/10/12 11:01 PM), Jerome Glisse wrote: On Thu, May 10, 2012 at 10:51 PM, KOSAKI Motohiro kosaki.motoh...@gmail.com  wrote: (5/10/12 8:50 PM), Minchan Kim wrote: Hi KOSAKI, On 05/11/2012 02:53 AM, KOSAKI

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-11 Thread Jerome Glisse
On Fri, May 11, 2012 at 6:59 PM, KOSAKI Motohiro kosaki.motoh...@gmail.com wrote: My point is this ioctl will be restricted to one user (Xserver if i understand) and only this user, there is no fork in it so no need to worry about fork, just setting the vma as locked will be enough. But i

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-11 Thread InKi Dae
2012/5/12 KOSAKI Motohiro kosaki.motoh...@gmail.com: On Fri, May 11, 2012 at 7:29 PM, Jerome Glisse j.gli...@gmail.com wrote: On Fri, May 11, 2012 at 6:59 PM, KOSAKI Motohiro kosaki.motoh...@gmail.com wrote: My point is this ioctl will be restricted to one user (Xserver if i understand) and

RE: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-10 Thread Inki Dae
; kyungmin.p...@samsung.com; sw0312@samsung.com; linux...@kvack.org Subject: Re: [PATCH 2/2 v3] drm/exynos: added userptr feature. On Wed, May 9, 2012 at 2:17 AM, Inki Dae inki@samsung.com wrote: this feature is used to import user space region allocated by malloc() or mmaped into a gem

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-10 Thread InKi Dae
...@samsung.com; sw0312@samsung.com; linux...@kvack.org Subject: Re: [PATCH 2/2 v3] drm/exynos: added userptr feature. On 05/10/2012 10:39 AM, Inki Dae wrote: Hi Jerome, -Original Message- From: Jerome Glisse [mailto:j.gli...@gmail.com] Sent: Wednesday, May 09, 2012 11:46 PM

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-10 Thread Minchan Kim
, -Original Message- From: Jerome Glisse [mailto:j.gli...@gmail.com] Sent: Wednesday, May 09, 2012 11:46 PM To: Inki Dae Cc: airl...@linux.ie; dri-devel@lists.freedesktop.org; kyungmin.p...@samsung.com; sw0312@samsung.com; linux...@kvack.org Subject: Re: [PATCH 2/2 v3] drm/exynos: added

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-10 Thread KOSAKI Motohiro
...@samsung.com; sw0312@samsung.com; linux...@kvack.org Subject: Re: [PATCH 2/2 v3] drm/exynos: added userptr feature. On Wed, May 9, 2012 at 2:17 AM, Inki Daeinki@samsung.com wrote: this feature is used to import user space region allocated by malloc() or mmaped into a gem. and to guarantee

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-10 Thread Minchan Kim
; linux...@kvack.org Subject: Re: [PATCH 2/2 v3] drm/exynos: added userptr feature. On 05/10/2012 10:39 AM, Inki Dae wrote: Hi Jerome, -Original Message- From: Jerome Glisse [mailto:j.gli...@gmail.com] Sent: Wednesday, May 09, 2012 11:46 PM To: Inki Dae Cc: airl...@linux.ie; dri

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-10 Thread Minchan Kim
; dri-devel@lists.freedesktop.org; kyungmin.p...@samsung.com; sw0312@samsung.com; linux...@kvack.org Subject: Re: [PATCH 2/2 v3] drm/exynos: added userptr feature. On Wed, May 9, 2012 at 2:17 AM, Inki Daeinki@samsung.com wrote: this feature is used to import user space region allocated

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-10 Thread Minchan Kim
[mailto:j.gli...@gmail.com] Sent: Wednesday, May 09, 2012 11:46 PM To: Inki Dae Cc: airl...@linux.ie; dri-devel@lists.freedesktop.org; kyungmin.p...@samsung.com; sw0312@samsung.com; linux...@kvack.org Subject: Re: [PATCH 2/2 v3] drm/exynos: added userptr feature. On Wed, May 9, 2012 at 2:17 AM

RE: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-10 Thread Inki Dae
PM To: Inki Dae Cc: 'Jerome Glisse'; airl...@linux.ie; dri- de...@lists.freedesktop.org; kyungmin.p...@samsung.com; sw0312@samsung.com; linux...@kvack.org Subject: Re: [PATCH 2/2 v3] drm/exynos: added userptr feature. On 05/10/2012 10:39 AM, Inki Dae wrote: Hi Jerome

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-10 Thread Minchan Kim
; dri-devel@lists.freedesktop.org; kyungmin.p...@samsung.com; sw0312@samsung.com; linux...@kvack.org Subject: Re: [PATCH 2/2 v3] drm/exynos: added userptr feature. On 05/10/2012 10:39 AM, Inki Dae wrote: Hi Jerome, -Original Message- From: Jerome Glisse [mailto:j.gli...@gmail.com

RE: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-10 Thread Inki Dae
- From: Jerome Glisse [mailto:j.gli...@gmail.com] Sent: Thursday, May 10, 2012 3:33 AM To: Inki Dae Cc: airl...@linux.ie; dri-devel@lists.freedesktop.org; kyungmin.p...@samsung.com; sw0312@samsung.com; linux- m...@kvack.org Subject: Re: [PATCH 2/2 v3] drm/exynos: added userptr

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-10 Thread Jerome Glisse
On Thu, May 10, 2012 at 10:51 PM, KOSAKI Motohiro kosaki.motoh...@gmail.com wrote: (5/10/12 8:50 PM), Minchan Kim wrote: Hi KOSAKI, On 05/11/2012 02:53 AM, KOSAKI Motohiro wrote: let's assume that one application want to allocate user space memory region using malloc() and then write

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-09 Thread Jerome Glisse
On Wed, May 9, 2012 at 10:45 AM, Jerome Glisse j.gli...@gmail.com wrote: On Wed, May 9, 2012 at 2:17 AM, Inki Dae inki@samsung.com wrote: this feature is used to import user space region allocated by malloc() or mmaped into a gem. and to guarantee the pages to user space not to be swapped

RE: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-09 Thread Inki Dae
] drm/exynos: added userptr feature. On Wed, May 9, 2012 at 2:17 AM, Inki Dae inki@samsung.com wrote: this feature is used to import user space region allocated by malloc() or mmaped into a gem. and to guarantee the pages to user space not to be swapped out, the VMAs within the user

RE: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-09 Thread Inki Dae
: [PATCH 2/2 v3] drm/exynos: added userptr feature. On Wed, May 9, 2012 at 10:45 AM, Jerome Glisse j.gli...@gmail.com wrote: On Wed, May 9, 2012 at 2:17 AM, Inki Dae inki@samsung.com wrote: this feature is used to import user space region allocated by malloc() or mmaped into a gem

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

2012-05-09 Thread Minchan Kim
; linux...@kvack.org Subject: Re: [PATCH 2/2 v3] drm/exynos: added userptr feature. On Wed, May 9, 2012 at 2:17 AM, Inki Dae inki@samsung.com wrote: this feature is used to import user space region allocated by malloc() or mmaped into a gem. and to guarantee the pages to user space