* [PATCH 1/2] drm/rockchip: create the fake mmap offset during gem creation
@ 2015-11-10 6:17 ` Mark yao
0 siblings, 0 replies; 18+ messages in thread
From: Mark yao @ 2015-11-10 6:17 UTC (permalink / raw)
To: linux-arm-kernel
Hi Heiko
I don't think this patch is needed for rockchip drm, since
rockchip drm only use kms.
I saw the discussion about ("drm/exynos: create a fake mmap offset with
gem creation"),
Inki said that: "This patch makes drm_gem_create_mmap_offset to be
called even in case of
not using dumb* interfaces. I.e., exynos_drm_gem_create_ioctl ->
exynos_drm_gem_mmap"
So I think this patch doesn't make sense for rockchip drm.
On 2015?11?10? 03:50, Heiko Stuebner wrote:
> Similarly to the equivalent exynos-fix in commit 48cf53f4343a
> ("drm/exynos: create a fake mmap offset with gem creation") the rockchip
> drm driver also calls the drm_gem_create_mmap_offset fake-offset creation
> on every invocation of rockchip_gem_dumb_map_offset.
>
> Move that to the creation of the gem to only create that offset once.
>
> Signed-off-by: Heiko Stuebner <heiko@sntech.de>
> ---
> drivers/gpu/drm/rockchip/rockchip_drm_gem.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> index 4ab2f20..e50d113 100644
> --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> @@ -126,8 +126,14 @@ struct rockchip_gem_object *
> if (ret)
> goto err_free_rk_obj;
>
> + ret = drm_gem_create_mmap_offset(obj);
> + if (ret)
> + goto err_free_buf;
> +
> return rk_obj;
>
> +err_free_buf:
> + rockchip_gem_free_buf(rk_obj);
> err_free_rk_obj:
> kfree(rk_obj);
> return ERR_PTR(ret);
> @@ -196,7 +202,6 @@ int rockchip_gem_dumb_map_offset(struct drm_file *file_priv,
> uint64_t *offset)
> {
> struct drm_gem_object *obj;
> - int ret;
>
> obj = drm_gem_object_lookup(dev, file_priv, handle);
> if (!obj) {
> @@ -204,14 +209,9 @@ int rockchip_gem_dumb_map_offset(struct drm_file *file_priv,
> return -EINVAL;
> }
>
> - ret = drm_gem_create_mmap_offset(obj);
> - if (ret)
> - goto out;
> -
> *offset = drm_vma_node_offset_addr(&obj->vma_node);
> DRM_DEBUG_KMS("offset = 0x%llx\n", *offset);
>
> -out:
> drm_gem_object_unreference_unlocked(obj);
>
> return 0;
--
?ark Yao
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [PATCH 1/2] drm/rockchip: create the fake mmap offset during gem creation
2015-11-10 6:17 ` Mark yao
@ 2015-11-10 6:46 ` Heiko Stuebner
-1 siblings, 0 replies; 18+ messages in thread
From: Heiko Stuebner @ 2015-11-10 6:46 UTC (permalink / raw)
To: Mark yao; +Cc: linux-rockchip, linux-arm-kernel, dri-devel
Hi Mark,
Am Dienstag, 10. November 2015, 14:17:43 schrieb Mark yao:
> Hi Heiko
> I don't think this patch is needed for rockchip drm, since
> rockchip drm only use kms.
> I saw the discussion about ("drm/exynos: create a fake mmap offset with
> gem creation"),
> Inki said that: "This patch makes drm_gem_create_mmap_offset to be
> called even in case of
> not using dumb* interfaces. I.e., exynos_drm_gem_create_ioctl ->
> exynos_drm_gem_mmap"
>
> So I think this patch doesn't make sense for rockchip drm.
ok :-) . I'm not that knowledgable when it comes to drm and just found that
similar change in exynos while searching for the dma issue.
So we can drop this patch and you could just merge patch2 as well as
[RESEND PATCH] drm/rockchip: vop: Correct enabled clocks during setup
from Sjoerd and send these two to Dave as fixes for 4.4?
Thanks
Heiko
>
>
> On 2015年11月10日 03:50, Heiko Stuebner wrote:
> > Similarly to the equivalent exynos-fix in commit 48cf53f4343a
> > ("drm/exynos: create a fake mmap offset with gem creation") the rockchip
> > drm driver also calls the drm_gem_create_mmap_offset fake-offset creation
> > on every invocation of rockchip_gem_dumb_map_offset.
> >
> > Move that to the creation of the gem to only create that offset once.
> >
> > Signed-off-by: Heiko Stuebner <heiko@sntech.de>
> > ---
> > drivers/gpu/drm/rockchip/rockchip_drm_gem.c | 12 ++++++------
> > 1 file changed, 6 insertions(+), 6 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > index 4ab2f20..e50d113 100644
> > --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > @@ -126,8 +126,14 @@ struct rockchip_gem_object *
> > if (ret)
> > goto err_free_rk_obj;
> >
> > + ret = drm_gem_create_mmap_offset(obj);
> > + if (ret)
> > + goto err_free_buf;
> > +
> > return rk_obj;
> >
> > +err_free_buf:
> > + rockchip_gem_free_buf(rk_obj);
> > err_free_rk_obj:
> > kfree(rk_obj);
> > return ERR_PTR(ret);
> > @@ -196,7 +202,6 @@ int rockchip_gem_dumb_map_offset(struct drm_file
*file_priv,
> > uint64_t *offset)
> > {
> > struct drm_gem_object *obj;
> > - int ret;
> >
> > obj = drm_gem_object_lookup(dev, file_priv, handle);
> > if (!obj) {
> > @@ -204,14 +209,9 @@ int rockchip_gem_dumb_map_offset(struct drm_file
*file_priv,
> > return -EINVAL;
> > }
> >
> > - ret = drm_gem_create_mmap_offset(obj);
> > - if (ret)
> > - goto out;
> > -
> > *offset = drm_vma_node_offset_addr(&obj->vma_node);
> > DRM_DEBUG_KMS("offset = 0x%llx\n", *offset);
> >
> > -out:
> > drm_gem_object_unreference_unlocked(obj);
> >
> > return 0;
>
>
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 18+ messages in thread* [PATCH 1/2] drm/rockchip: create the fake mmap offset during gem creation
@ 2015-11-10 6:46 ` Heiko Stuebner
0 siblings, 0 replies; 18+ messages in thread
From: Heiko Stuebner @ 2015-11-10 6:46 UTC (permalink / raw)
To: linux-arm-kernel
Hi Mark,
Am Dienstag, 10. November 2015, 14:17:43 schrieb Mark yao:
> Hi Heiko
> I don't think this patch is needed for rockchip drm, since
> rockchip drm only use kms.
> I saw the discussion about ("drm/exynos: create a fake mmap offset with
> gem creation"),
> Inki said that: "This patch makes drm_gem_create_mmap_offset to be
> called even in case of
> not using dumb* interfaces. I.e., exynos_drm_gem_create_ioctl ->
> exynos_drm_gem_mmap"
>
> So I think this patch doesn't make sense for rockchip drm.
ok :-) . I'm not that knowledgable when it comes to drm and just found that
similar change in exynos while searching for the dma issue.
So we can drop this patch and you could just merge patch2 as well as
[RESEND PATCH] drm/rockchip: vop: Correct enabled clocks during setup
from Sjoerd and send these two to Dave as fixes for 4.4?
Thanks
Heiko
>
>
> On 2015?11?10? 03:50, Heiko Stuebner wrote:
> > Similarly to the equivalent exynos-fix in commit 48cf53f4343a
> > ("drm/exynos: create a fake mmap offset with gem creation") the rockchip
> > drm driver also calls the drm_gem_create_mmap_offset fake-offset creation
> > on every invocation of rockchip_gem_dumb_map_offset.
> >
> > Move that to the creation of the gem to only create that offset once.
> >
> > Signed-off-by: Heiko Stuebner <heiko@sntech.de>
> > ---
> > drivers/gpu/drm/rockchip/rockchip_drm_gem.c | 12 ++++++------
> > 1 file changed, 6 insertions(+), 6 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > index 4ab2f20..e50d113 100644
> > --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > @@ -126,8 +126,14 @@ struct rockchip_gem_object *
> > if (ret)
> > goto err_free_rk_obj;
> >
> > + ret = drm_gem_create_mmap_offset(obj);
> > + if (ret)
> > + goto err_free_buf;
> > +
> > return rk_obj;
> >
> > +err_free_buf:
> > + rockchip_gem_free_buf(rk_obj);
> > err_free_rk_obj:
> > kfree(rk_obj);
> > return ERR_PTR(ret);
> > @@ -196,7 +202,6 @@ int rockchip_gem_dumb_map_offset(struct drm_file
*file_priv,
> > uint64_t *offset)
> > {
> > struct drm_gem_object *obj;
> > - int ret;
> >
> > obj = drm_gem_object_lookup(dev, file_priv, handle);
> > if (!obj) {
> > @@ -204,14 +209,9 @@ int rockchip_gem_dumb_map_offset(struct drm_file
*file_priv,
> > return -EINVAL;
> > }
> >
> > - ret = drm_gem_create_mmap_offset(obj);
> > - if (ret)
> > - goto out;
> > -
> > *offset = drm_vma_node_offset_addr(&obj->vma_node);
> > DRM_DEBUG_KMS("offset = 0x%llx\n", *offset);
> >
> > -out:
> > drm_gem_object_unreference_unlocked(obj);
> >
> > return 0;
>
>
>
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [PATCH 1/2] drm/rockchip: create the fake mmap offset during gem creation
2015-11-10 6:46 ` Heiko Stuebner
@ 2015-11-10 9:26 ` Mark yao
-1 siblings, 0 replies; 18+ messages in thread
From: Mark yao @ 2015-11-10 9:26 UTC (permalink / raw)
To: Heiko Stuebner; +Cc: linux-rockchip, linux-arm-kernel, dri-devel
On 2015年11月10日 14:46, Heiko Stuebner wrote:
> Hi Mark,
>
> Am Dienstag, 10. November 2015, 14:17:43 schrieb Mark yao:
>> Hi Heiko
>> I don't think this patch is needed for rockchip drm, since
>> rockchip drm only use kms.
>> I saw the discussion about ("drm/exynos: create a fake mmap offset with
>> gem creation"),
>> Inki said that: "This patch makes drm_gem_create_mmap_offset to be
>> called even in case of
>> not using dumb* interfaces. I.e., exynos_drm_gem_create_ioctl ->
>> exynos_drm_gem_mmap"
>>
>> So I think this patch doesn't make sense for rockchip drm.
> ok :-) . I'm not that knowledgable when it comes to drm and just found that
> similar change in exynos while searching for the dma issue.
>
> So we can drop this patch and you could just merge patch2 as well as
>
> [RESEND PATCH] drm/rockchip: vop: Correct enabled clocks during setup
>
> from Sjoerd and send these two to Dave as fixes for 4.4?
>
Right, that patch is needed, and also I want to land two patch:
[RESEND PATCH] drm/rockchip: import dma_buf to gem
[PATCH] Revert "drm/rockchip: Convert the probe function to the
generic drm_of_component_probe()"
> Thanks
> Heiko
>
>>
>> On 2015年11月10日 03:50, Heiko Stuebner wrote:
>>> Similarly to the equivalent exynos-fix in commit 48cf53f4343a
>>> ("drm/exynos: create a fake mmap offset with gem creation") the rockchip
>>> drm driver also calls the drm_gem_create_mmap_offset fake-offset creation
>>> on every invocation of rockchip_gem_dumb_map_offset.
>>>
>>> Move that to the creation of the gem to only create that offset once.
>>>
>>> Signed-off-by: Heiko Stuebner <heiko@sntech.de>
>>> ---
>>> drivers/gpu/drm/rockchip/rockchip_drm_gem.c | 12 ++++++------
>>> 1 file changed, 6 insertions(+), 6 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>> index 4ab2f20..e50d113 100644
>>> --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>> @@ -126,8 +126,14 @@ struct rockchip_gem_object *
>>> if (ret)
>>> goto err_free_rk_obj;
>>>
>>> + ret = drm_gem_create_mmap_offset(obj);
>>> + if (ret)
>>> + goto err_free_buf;
>>> +
>>> return rk_obj;
>>>
>>> +err_free_buf:
>>> + rockchip_gem_free_buf(rk_obj);
>>> err_free_rk_obj:
>>> kfree(rk_obj);
>>> return ERR_PTR(ret);
>>> @@ -196,7 +202,6 @@ int rockchip_gem_dumb_map_offset(struct drm_file
> *file_priv,
>>> uint64_t *offset)
>>> {
>>> struct drm_gem_object *obj;
>>> - int ret;
>>>
>>> obj = drm_gem_object_lookup(dev, file_priv, handle);
>>> if (!obj) {
>>> @@ -204,14 +209,9 @@ int rockchip_gem_dumb_map_offset(struct drm_file
> *file_priv,
>>> return -EINVAL;
>>> }
>>>
>>> - ret = drm_gem_create_mmap_offset(obj);
>>> - if (ret)
>>> - goto out;
>>> -
>>> *offset = drm_vma_node_offset_addr(&obj->vma_node);
>>> DRM_DEBUG_KMS("offset = 0x%llx\n", *offset);
>>>
>>> -out:
>>> drm_gem_object_unreference_unlocked(obj);
>>>
>>> return 0;
>>
>>
>
>
>
--
Ｍark Yao
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 18+ messages in thread* [PATCH 1/2] drm/rockchip: create the fake mmap offset during gem creation
@ 2015-11-10 9:26 ` Mark yao
0 siblings, 0 replies; 18+ messages in thread
From: Mark yao @ 2015-11-10 9:26 UTC (permalink / raw)
To: linux-arm-kernel
On 2015?11?10? 14:46, Heiko Stuebner wrote:
> Hi Mark,
>
> Am Dienstag, 10. November 2015, 14:17:43 schrieb Mark yao:
>> Hi Heiko
>> I don't think this patch is needed for rockchip drm, since
>> rockchip drm only use kms.
>> I saw the discussion about ("drm/exynos: create a fake mmap offset with
>> gem creation"),
>> Inki said that: "This patch makes drm_gem_create_mmap_offset to be
>> called even in case of
>> not using dumb* interfaces. I.e., exynos_drm_gem_create_ioctl ->
>> exynos_drm_gem_mmap"
>>
>> So I think this patch doesn't make sense for rockchip drm.
> ok :-) . I'm not that knowledgable when it comes to drm and just found that
> similar change in exynos while searching for the dma issue.
>
> So we can drop this patch and you could just merge patch2 as well as
>
> [RESEND PATCH] drm/rockchip: vop: Correct enabled clocks during setup
>
> from Sjoerd and send these two to Dave as fixes for 4.4?
>
Right, that patch is needed, and also I want to land two patch:
[RESEND PATCH] drm/rockchip: import dma_buf to gem
[PATCH] Revert "drm/rockchip: Convert the probe function to the
generic drm_of_component_probe()"
> Thanks
> Heiko
>
>>
>> On 2015?11?10? 03:50, Heiko Stuebner wrote:
>>> Similarly to the equivalent exynos-fix in commit 48cf53f4343a
>>> ("drm/exynos: create a fake mmap offset with gem creation") the rockchip
>>> drm driver also calls the drm_gem_create_mmap_offset fake-offset creation
>>> on every invocation of rockchip_gem_dumb_map_offset.
>>>
>>> Move that to the creation of the gem to only create that offset once.
>>>
>>> Signed-off-by: Heiko Stuebner <heiko@sntech.de>
>>> ---
>>> drivers/gpu/drm/rockchip/rockchip_drm_gem.c | 12 ++++++------
>>> 1 file changed, 6 insertions(+), 6 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>> index 4ab2f20..e50d113 100644
>>> --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>> @@ -126,8 +126,14 @@ struct rockchip_gem_object *
>>> if (ret)
>>> goto err_free_rk_obj;
>>>
>>> + ret = drm_gem_create_mmap_offset(obj);
>>> + if (ret)
>>> + goto err_free_buf;
>>> +
>>> return rk_obj;
>>>
>>> +err_free_buf:
>>> + rockchip_gem_free_buf(rk_obj);
>>> err_free_rk_obj:
>>> kfree(rk_obj);
>>> return ERR_PTR(ret);
>>> @@ -196,7 +202,6 @@ int rockchip_gem_dumb_map_offset(struct drm_file
> *file_priv,
>>> uint64_t *offset)
>>> {
>>> struct drm_gem_object *obj;
>>> - int ret;
>>>
>>> obj = drm_gem_object_lookup(dev, file_priv, handle);
>>> if (!obj) {
>>> @@ -204,14 +209,9 @@ int rockchip_gem_dumb_map_offset(struct drm_file
> *file_priv,
>>> return -EINVAL;
>>> }
>>>
>>> - ret = drm_gem_create_mmap_offset(obj);
>>> - if (ret)
>>> - goto out;
>>> -
>>> *offset = drm_vma_node_offset_addr(&obj->vma_node);
>>> DRM_DEBUG_KMS("offset = 0x%llx\n", *offset);
>>>
>>> -out:
>>> drm_gem_object_unreference_unlocked(obj);
>>>
>>> return 0;
>>
>>
>
>
>
--
?ark Yao
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [PATCH 1/2] drm/rockchip: create the fake mmap offset during gem creation
2015-11-10 9:26 ` Mark yao
@ 2015-11-10 9:28 ` Heiko Stuebner
-1 siblings, 0 replies; 18+ messages in thread
From: Heiko Stuebner @ 2015-11-10 9:28 UTC (permalink / raw)
To: Mark yao; +Cc: linux-rockchip, linux-arm-kernel, dri-devel
Am Dienstag, 10. November 2015, 17:26:10 schrieb Mark yao:
> On 2015年11月10日 14:46, Heiko Stuebner wrote:
> > Hi Mark,
> >
> > Am Dienstag, 10. November 2015, 14:17:43 schrieb Mark yao:
> >> Hi Heiko
> >> I don't think this patch is needed for rockchip drm, since
> >> rockchip drm only use kms.
> >> I saw the discussion about ("drm/exynos: create a fake mmap offset with
> >> gem creation"),
> >> Inki said that: "This patch makes drm_gem_create_mmap_offset to be
> >> called even in case of
> >> not using dumb* interfaces. I.e., exynos_drm_gem_create_ioctl ->
> >> exynos_drm_gem_mmap"
> >>
> >> So I think this patch doesn't make sense for rockchip drm.
> > ok :-) . I'm not that knowledgable when it comes to drm and just found that
> > similar change in exynos while searching for the dma issue.
> >
> > So we can drop this patch and you could just merge patch2 as well as
> >
> > [RESEND PATCH] drm/rockchip: vop: Correct enabled clocks during setup
> >
> > from Sjoerd and send these two to Dave as fixes for 4.4?
> >
> Right, that patch is needed, and also I want to land two patch:
> [RESEND PATCH] drm/rockchip: import dma_buf to gem
the import should go into 4.5 though (as it is a new feature), while
the other three are fixes that should go into the current 4.4
> [PATCH] Revert "drm/rockchip: Convert the probe function to the
> generic drm_of_component_probe()"
>
> > Thanks
> > Heiko
> >
> >>
> >> On 2015年11月10日 03:50, Heiko Stuebner wrote:
> >>> Similarly to the equivalent exynos-fix in commit 48cf53f4343a
> >>> ("drm/exynos: create a fake mmap offset with gem creation") the rockchip
> >>> drm driver also calls the drm_gem_create_mmap_offset fake-offset creation
> >>> on every invocation of rockchip_gem_dumb_map_offset.
> >>>
> >>> Move that to the creation of the gem to only create that offset once.
> >>>
> >>> Signed-off-by: Heiko Stuebner <heiko@sntech.de>
> >>> ---
> >>> drivers/gpu/drm/rockchip/rockchip_drm_gem.c | 12 ++++++------
> >>> 1 file changed, 6 insertions(+), 6 deletions(-)
> >>>
> >>> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> >>> index 4ab2f20..e50d113 100644
> >>> --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> >>> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> >>> @@ -126,8 +126,14 @@ struct rockchip_gem_object *
> >>> if (ret)
> >>> goto err_free_rk_obj;
> >>>
> >>> + ret = drm_gem_create_mmap_offset(obj);
> >>> + if (ret)
> >>> + goto err_free_buf;
> >>> +
> >>> return rk_obj;
> >>>
> >>> +err_free_buf:
> >>> + rockchip_gem_free_buf(rk_obj);
> >>> err_free_rk_obj:
> >>> kfree(rk_obj);
> >>> return ERR_PTR(ret);
> >>> @@ -196,7 +202,6 @@ int rockchip_gem_dumb_map_offset(struct drm_file
> > *file_priv,
> >>> uint64_t *offset)
> >>> {
> >>> struct drm_gem_object *obj;
> >>> - int ret;
> >>>
> >>> obj = drm_gem_object_lookup(dev, file_priv, handle);
> >>> if (!obj) {
> >>> @@ -204,14 +209,9 @@ int rockchip_gem_dumb_map_offset(struct drm_file
> > *file_priv,
> >>> return -EINVAL;
> >>> }
> >>>
> >>> - ret = drm_gem_create_mmap_offset(obj);
> >>> - if (ret)
> >>> - goto out;
> >>> -
> >>> *offset = drm_vma_node_offset_addr(&obj->vma_node);
> >>> DRM_DEBUG_KMS("offset = 0x%llx\n", *offset);
> >>>
> >>> -out:
> >>> drm_gem_object_unreference_unlocked(obj);
> >>>
> >>> return 0;
> >>
> >>
> >
> >
> >
>
>
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 18+ messages in thread* [PATCH 1/2] drm/rockchip: create the fake mmap offset during gem creation
@ 2015-11-10 9:28 ` Heiko Stuebner
0 siblings, 0 replies; 18+ messages in thread
From: Heiko Stuebner @ 2015-11-10 9:28 UTC (permalink / raw)
To: linux-arm-kernel
Am Dienstag, 10. November 2015, 17:26:10 schrieb Mark yao:
> On 2015?11?10? 14:46, Heiko Stuebner wrote:
> > Hi Mark,
> >
> > Am Dienstag, 10. November 2015, 14:17:43 schrieb Mark yao:
> >> Hi Heiko
> >> I don't think this patch is needed for rockchip drm, since
> >> rockchip drm only use kms.
> >> I saw the discussion about ("drm/exynos: create a fake mmap offset with
> >> gem creation"),
> >> Inki said that: "This patch makes drm_gem_create_mmap_offset to be
> >> called even in case of
> >> not using dumb* interfaces. I.e., exynos_drm_gem_create_ioctl ->
> >> exynos_drm_gem_mmap"
> >>
> >> So I think this patch doesn't make sense for rockchip drm.
> > ok :-) . I'm not that knowledgable when it comes to drm and just found that
> > similar change in exynos while searching for the dma issue.
> >
> > So we can drop this patch and you could just merge patch2 as well as
> >
> > [RESEND PATCH] drm/rockchip: vop: Correct enabled clocks during setup
> >
> > from Sjoerd and send these two to Dave as fixes for 4.4?
> >
> Right, that patch is needed, and also I want to land two patch:
> [RESEND PATCH] drm/rockchip: import dma_buf to gem
the import should go into 4.5 though (as it is a new feature), while
the other three are fixes that should go into the current 4.4
> [PATCH] Revert "drm/rockchip: Convert the probe function to the
> generic drm_of_component_probe()"
>
> > Thanks
> > Heiko
> >
> >>
> >> On 2015?11?10? 03:50, Heiko Stuebner wrote:
> >>> Similarly to the equivalent exynos-fix in commit 48cf53f4343a
> >>> ("drm/exynos: create a fake mmap offset with gem creation") the rockchip
> >>> drm driver also calls the drm_gem_create_mmap_offset fake-offset creation
> >>> on every invocation of rockchip_gem_dumb_map_offset.
> >>>
> >>> Move that to the creation of the gem to only create that offset once.
> >>>
> >>> Signed-off-by: Heiko Stuebner <heiko@sntech.de>
> >>> ---
> >>> drivers/gpu/drm/rockchip/rockchip_drm_gem.c | 12 ++++++------
> >>> 1 file changed, 6 insertions(+), 6 deletions(-)
> >>>
> >>> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> > b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> >>> index 4ab2f20..e50d113 100644
> >>> --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> >>> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> >>> @@ -126,8 +126,14 @@ struct rockchip_gem_object *
> >>> if (ret)
> >>> goto err_free_rk_obj;
> >>>
> >>> + ret = drm_gem_create_mmap_offset(obj);
> >>> + if (ret)
> >>> + goto err_free_buf;
> >>> +
> >>> return rk_obj;
> >>>
> >>> +err_free_buf:
> >>> + rockchip_gem_free_buf(rk_obj);
> >>> err_free_rk_obj:
> >>> kfree(rk_obj);
> >>> return ERR_PTR(ret);
> >>> @@ -196,7 +202,6 @@ int rockchip_gem_dumb_map_offset(struct drm_file
> > *file_priv,
> >>> uint64_t *offset)
> >>> {
> >>> struct drm_gem_object *obj;
> >>> - int ret;
> >>>
> >>> obj = drm_gem_object_lookup(dev, file_priv, handle);
> >>> if (!obj) {
> >>> @@ -204,14 +209,9 @@ int rockchip_gem_dumb_map_offset(struct drm_file
> > *file_priv,
> >>> return -EINVAL;
> >>> }
> >>>
> >>> - ret = drm_gem_create_mmap_offset(obj);
> >>> - if (ret)
> >>> - goto out;
> >>> -
> >>> *offset = drm_vma_node_offset_addr(&obj->vma_node);
> >>> DRM_DEBUG_KMS("offset = 0x%llx\n", *offset);
> >>>
> >>> -out:
> >>> drm_gem_object_unreference_unlocked(obj);
> >>>
> >>> return 0;
> >>
> >>
> >
> >
> >
>
>
>
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [PATCH 1/2] drm/rockchip: create the fake mmap offset during gem creation
2015-11-10 9:28 ` Heiko Stuebner
@ 2015-11-10 9:38 ` Mark yao
-1 siblings, 0 replies; 18+ messages in thread
From: Mark yao @ 2015-11-10 9:38 UTC (permalink / raw)
To: Heiko Stuebner; +Cc: linux-rockchip, linux-arm-kernel, dri-devel
On 2015年11月10日 17:28, Heiko Stuebner wrote:
> Am Dienstag, 10. November 2015, 17:26:10 schrieb Mark yao:
>> On 2015年11月10日 14:46, Heiko Stuebner wrote:
>>> Hi Mark,
>>>
>>> Am Dienstag, 10. November 2015, 14:17:43 schrieb Mark yao:
>>>> Hi Heiko
>>>> I don't think this patch is needed for rockchip drm, since
>>>> rockchip drm only use kms.
>>>> I saw the discussion about ("drm/exynos: create a fake mmap offset with
>>>> gem creation"),
>>>> Inki said that: "This patch makes drm_gem_create_mmap_offset to be
>>>> called even in case of
>>>> not using dumb* interfaces. I.e., exynos_drm_gem_create_ioctl ->
>>>> exynos_drm_gem_mmap"
>>>>
>>>> So I think this patch doesn't make sense for rockchip drm.
>>> ok :-) . I'm not that knowledgable when it comes to drm and just found that
>>> similar change in exynos while searching for the dma issue.
>>>
>>> So we can drop this patch and you could just merge patch2 as well as
>>>
>>> [RESEND PATCH] drm/rockchip: vop: Correct enabled clocks during setup
>>>
>>> from Sjoerd and send these two to Dave as fixes for 4.4?
>>>
>> Right, that patch is needed, and also I want to land two patch:
>> [RESEND PATCH] drm/rockchip: import dma_buf to gem
> the import should go into 4.5 though (as it is a new feature), while
> the other three are fixes that should go into the current 4.4
>
OK, Got it
>> [PATCH] Revert "drm/rockchip: Convert the probe function to the
>> generic drm_of_component_probe()"
>>
>>> Thanks
>>> Heiko
>>>
>>>> On 2015年11月10日 03:50, Heiko Stuebner wrote:
>>>>> Similarly to the equivalent exynos-fix in commit 48cf53f4343a
>>>>> ("drm/exynos: create a fake mmap offset with gem creation") the rockchip
>>>>> drm driver also calls the drm_gem_create_mmap_offset fake-offset creation
>>>>> on every invocation of rockchip_gem_dumb_map_offset.
>>>>>
>>>>> Move that to the creation of the gem to only create that offset once.
>>>>>
>>>>> Signed-off-by: Heiko Stuebner <heiko@sntech.de>
>>>>> ---
>>>>> drivers/gpu/drm/rockchip/rockchip_drm_gem.c | 12 ++++++------
>>>>> 1 file changed, 6 insertions(+), 6 deletions(-)
>>>>>
>>>>> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>> b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>>>> index 4ab2f20..e50d113 100644
>>>>> --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>>>> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>>>> @@ -126,8 +126,14 @@ struct rockchip_gem_object *
>>>>> if (ret)
>>>>> goto err_free_rk_obj;
>>>>>
>>>>> + ret = drm_gem_create_mmap_offset(obj);
>>>>> + if (ret)
>>>>> + goto err_free_buf;
>>>>> +
>>>>> return rk_obj;
>>>>>
>>>>> +err_free_buf:
>>>>> + rockchip_gem_free_buf(rk_obj);
>>>>> err_free_rk_obj:
>>>>> kfree(rk_obj);
>>>>> return ERR_PTR(ret);
>>>>> @@ -196,7 +202,6 @@ int rockchip_gem_dumb_map_offset(struct drm_file
>>> *file_priv,
>>>>> uint64_t *offset)
>>>>> {
>>>>> struct drm_gem_object *obj;
>>>>> - int ret;
>>>>>
>>>>> obj = drm_gem_object_lookup(dev, file_priv, handle);
>>>>> if (!obj) {
>>>>> @@ -204,14 +209,9 @@ int rockchip_gem_dumb_map_offset(struct drm_file
>>> *file_priv,
>>>>> return -EINVAL;
>>>>> }
>>>>>
>>>>> - ret = drm_gem_create_mmap_offset(obj);
>>>>> - if (ret)
>>>>> - goto out;
>>>>> -
>>>>> *offset = drm_vma_node_offset_addr(&obj->vma_node);
>>>>> DRM_DEBUG_KMS("offset = 0x%llx\n", *offset);
>>>>>
>>>>> -out:
>>>>> drm_gem_object_unreference_unlocked(obj);
>>>>>
>>>>> return 0;
>>>>
>>>
>>>
>>
>>
>
>
>
--
Ｍark Yao
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 18+ messages in thread* [PATCH 1/2] drm/rockchip: create the fake mmap offset during gem creation
@ 2015-11-10 9:38 ` Mark yao
0 siblings, 0 replies; 18+ messages in thread
From: Mark yao @ 2015-11-10 9:38 UTC (permalink / raw)
To: linux-arm-kernel
On 2015?11?10? 17:28, Heiko Stuebner wrote:
> Am Dienstag, 10. November 2015, 17:26:10 schrieb Mark yao:
>> On 2015?11?10? 14:46, Heiko Stuebner wrote:
>>> Hi Mark,
>>>
>>> Am Dienstag, 10. November 2015, 14:17:43 schrieb Mark yao:
>>>> Hi Heiko
>>>> I don't think this patch is needed for rockchip drm, since
>>>> rockchip drm only use kms.
>>>> I saw the discussion about ("drm/exynos: create a fake mmap offset with
>>>> gem creation"),
>>>> Inki said that: "This patch makes drm_gem_create_mmap_offset to be
>>>> called even in case of
>>>> not using dumb* interfaces. I.e., exynos_drm_gem_create_ioctl ->
>>>> exynos_drm_gem_mmap"
>>>>
>>>> So I think this patch doesn't make sense for rockchip drm.
>>> ok :-) . I'm not that knowledgable when it comes to drm and just found that
>>> similar change in exynos while searching for the dma issue.
>>>
>>> So we can drop this patch and you could just merge patch2 as well as
>>>
>>> [RESEND PATCH] drm/rockchip: vop: Correct enabled clocks during setup
>>>
>>> from Sjoerd and send these two to Dave as fixes for 4.4?
>>>
>> Right, that patch is needed, and also I want to land two patch:
>> [RESEND PATCH] drm/rockchip: import dma_buf to gem
> the import should go into 4.5 though (as it is a new feature), while
> the other three are fixes that should go into the current 4.4
>
OK, Got it
>> [PATCH] Revert "drm/rockchip: Convert the probe function to the
>> generic drm_of_component_probe()"
>>
>>> Thanks
>>> Heiko
>>>
>>>> On 2015?11?10? 03:50, Heiko Stuebner wrote:
>>>>> Similarly to the equivalent exynos-fix in commit 48cf53f4343a
>>>>> ("drm/exynos: create a fake mmap offset with gem creation") the rockchip
>>>>> drm driver also calls the drm_gem_create_mmap_offset fake-offset creation
>>>>> on every invocation of rockchip_gem_dumb_map_offset.
>>>>>
>>>>> Move that to the creation of the gem to only create that offset once.
>>>>>
>>>>> Signed-off-by: Heiko Stuebner <heiko@sntech.de>
>>>>> ---
>>>>> drivers/gpu/drm/rockchip/rockchip_drm_gem.c | 12 ++++++------
>>>>> 1 file changed, 6 insertions(+), 6 deletions(-)
>>>>>
>>>>> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>> b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>>>> index 4ab2f20..e50d113 100644
>>>>> --- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>>>> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
>>>>> @@ -126,8 +126,14 @@ struct rockchip_gem_object *
>>>>> if (ret)
>>>>> goto err_free_rk_obj;
>>>>>
>>>>> + ret = drm_gem_create_mmap_offset(obj);
>>>>> + if (ret)
>>>>> + goto err_free_buf;
>>>>> +
>>>>> return rk_obj;
>>>>>
>>>>> +err_free_buf:
>>>>> + rockchip_gem_free_buf(rk_obj);
>>>>> err_free_rk_obj:
>>>>> kfree(rk_obj);
>>>>> return ERR_PTR(ret);
>>>>> @@ -196,7 +202,6 @@ int rockchip_gem_dumb_map_offset(struct drm_file
>>> *file_priv,
>>>>> uint64_t *offset)
>>>>> {
>>>>> struct drm_gem_object *obj;
>>>>> - int ret;
>>>>>
>>>>> obj = drm_gem_object_lookup(dev, file_priv, handle);
>>>>> if (!obj) {
>>>>> @@ -204,14 +209,9 @@ int rockchip_gem_dumb_map_offset(struct drm_file
>>> *file_priv,
>>>>> return -EINVAL;
>>>>> }
>>>>>
>>>>> - ret = drm_gem_create_mmap_offset(obj);
>>>>> - if (ret)
>>>>> - goto out;
>>>>> -
>>>>> *offset = drm_vma_node_offset_addr(&obj->vma_node);
>>>>> DRM_DEBUG_KMS("offset = 0x%llx\n", *offset);
>>>>>
>>>>> -out:
>>>>> drm_gem_object_unreference_unlocked(obj);
>>>>>
>>>>> return 0;
>>>>
>>>
>>>
>>
>>
>
>
>
--
?ark Yao
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 1/2] drm/rockchip: create the fake mmap offset during gem creation
2015-11-10 6:17 ` Mark yao
@ 2015-11-16 16:21 ` Daniel Vetter
-1 siblings, 0 replies; 18+ messages in thread
From: Daniel Vetter @ 2015-11-16 16:21 UTC (permalink / raw)
To: Mark yao; +Cc: linux-rockchip, dri-devel, linux-arm-kernel
On Tue, Nov 10, 2015 at 02:17:43PM +0800, Mark yao wrote:
> Hi Heiko
> I don't think this patch is needed for rockchip drm, since rockchip drm
> only use kms.
> I saw the discussion about ("drm/exynos: create a fake mmap offset with gem
> creation"),
> Inki said that: "This patch makes drm_gem_create_mmap_offset to be called
> even in case of
> not using dumb* interfaces. I.e., exynos_drm_gem_create_ioctl ->
> exynos_drm_gem_mmap"
>
> So I think this patch doesn't make sense for rockchip drm.
>
>
> On 2015年11月10日 03:50, Heiko Stuebner wrote:
> >Similarly to the equivalent exynos-fix in commit 48cf53f4343a
This exynos patch and your rockchip patch look really strange - the point
_is_ to only create fake mmap offset when someone actually tries to mmap a
bo. drm_gem_create_mmap_offset internally (and in a race-free manner)
checks whether it's been called already, so it's perfectly safe to call
mutliple times.
Imo drop this (and probably revert 48cf53f4343a as unjustified).
-Daniel
> >("drm/exynos: create a fake mmap offset with gem creation") the rockchip
> >drm driver also calls the drm_gem_create_mmap_offset fake-offset creation
> >on every invocation of rockchip_gem_dumb_map_offset.
> >
> >Move that to the creation of the gem to only create that offset once.
> >
> >Signed-off-by: Heiko Stuebner <heiko@sntech.de>
> >---
> > drivers/gpu/drm/rockchip/rockchip_drm_gem.c | 12 ++++++------
> > 1 file changed, 6 insertions(+), 6 deletions(-)
> >
> >diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> >index 4ab2f20..e50d113 100644
> >--- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> >+++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> >@@ -126,8 +126,14 @@ struct rockchip_gem_object *
> > if (ret)
> > goto err_free_rk_obj;
> >+ ret = drm_gem_create_mmap_offset(obj);
> >+ if (ret)
> >+ goto err_free_buf;
> >+
> > return rk_obj;
> >+err_free_buf:
> >+ rockchip_gem_free_buf(rk_obj);
> > err_free_rk_obj:
> > kfree(rk_obj);
> > return ERR_PTR(ret);
> >@@ -196,7 +202,6 @@ int rockchip_gem_dumb_map_offset(struct drm_file *file_priv,
> > uint64_t *offset)
> > {
> > struct drm_gem_object *obj;
> >- int ret;
> > obj = drm_gem_object_lookup(dev, file_priv, handle);
> > if (!obj) {
> >@@ -204,14 +209,9 @@ int rockchip_gem_dumb_map_offset(struct drm_file *file_priv,
> > return -EINVAL;
> > }
> >- ret = drm_gem_create_mmap_offset(obj);
> >- if (ret)
> >- goto out;
> >-
> > *offset = drm_vma_node_offset_addr(&obj->vma_node);
> > DRM_DEBUG_KMS("offset = 0x%llx\n", *offset);
> >-out:
> > drm_gem_object_unreference_unlocked(obj);
> > return 0;
>
>
> --
> Ｍark Yao
>
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 18+ messages in thread* [PATCH 1/2] drm/rockchip: create the fake mmap offset during gem creation
@ 2015-11-16 16:21 ` Daniel Vetter
0 siblings, 0 replies; 18+ messages in thread
From: Daniel Vetter @ 2015-11-16 16:21 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Nov 10, 2015 at 02:17:43PM +0800, Mark yao wrote:
> Hi Heiko
> I don't think this patch is needed for rockchip drm, since rockchip drm
> only use kms.
> I saw the discussion about ("drm/exynos: create a fake mmap offset with gem
> creation"),
> Inki said that: "This patch makes drm_gem_create_mmap_offset to be called
> even in case of
> not using dumb* interfaces. I.e., exynos_drm_gem_create_ioctl ->
> exynos_drm_gem_mmap"
>
> So I think this patch doesn't make sense for rockchip drm.
>
>
> On 2015?11?10? 03:50, Heiko Stuebner wrote:
> >Similarly to the equivalent exynos-fix in commit 48cf53f4343a
This exynos patch and your rockchip patch look really strange - the point
_is_ to only create fake mmap offset when someone actually tries to mmap a
bo. drm_gem_create_mmap_offset internally (and in a race-free manner)
checks whether it's been called already, so it's perfectly safe to call
mutliple times.
Imo drop this (and probably revert 48cf53f4343a as unjustified).
-Daniel
> >("drm/exynos: create a fake mmap offset with gem creation") the rockchip
> >drm driver also calls the drm_gem_create_mmap_offset fake-offset creation
> >on every invocation of rockchip_gem_dumb_map_offset.
> >
> >Move that to the creation of the gem to only create that offset once.
> >
> >Signed-off-by: Heiko Stuebner <heiko@sntech.de>
> >---
> > drivers/gpu/drm/rockchip/rockchip_drm_gem.c | 12 ++++++------
> > 1 file changed, 6 insertions(+), 6 deletions(-)
> >
> >diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> >index 4ab2f20..e50d113 100644
> >--- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> >+++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c
> >@@ -126,8 +126,14 @@ struct rockchip_gem_object *
> > if (ret)
> > goto err_free_rk_obj;
> >+ ret = drm_gem_create_mmap_offset(obj);
> >+ if (ret)
> >+ goto err_free_buf;
> >+
> > return rk_obj;
> >+err_free_buf:
> >+ rockchip_gem_free_buf(rk_obj);
> > err_free_rk_obj:
> > kfree(rk_obj);
> > return ERR_PTR(ret);
> >@@ -196,7 +202,6 @@ int rockchip_gem_dumb_map_offset(struct drm_file *file_priv,
> > uint64_t *offset)
> > {
> > struct drm_gem_object *obj;
> >- int ret;
> > obj = drm_gem_object_lookup(dev, file_priv, handle);
> > if (!obj) {
> >@@ -204,14 +209,9 @@ int rockchip_gem_dumb_map_offset(struct drm_file *file_priv,
> > return -EINVAL;
> > }
> >- ret = drm_gem_create_mmap_offset(obj);
> >- if (ret)
> >- goto out;
> >-
> > *offset = drm_vma_node_offset_addr(&obj->vma_node);
> > DRM_DEBUG_KMS("offset = 0x%llx\n", *offset);
> >-out:
> > drm_gem_object_unreference_unlocked(obj);
> > return 0;
>
>
> --
> ?ark Yao
>
>
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 18+ messages in thread