RE: upsteam find bo api

[Deucher, Alexander]

I actually have a branch of amd-staging based on drm-next.  See the 
amd-staging-drm-next branch.

Alex

From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of 
Christian König
Sent: Monday, June 19, 2017 4:50 AM
To: Zhou, David(ChunMing); Alex Deucher
Cc: amd-gfx@lists.freedesktop.org
Subject: Re: upsteam find bo api

Am 19.06.2017 um 04:23 schrieb zhoucm1:

On 2017年06月17日 18:09, Christian König wrote:
The patch should still work on amd-staging-4.11, but drm-next has seen some 
changes on how the handles are stored for dma-buf.

This can potentially break the implementation and I think we need to adapt 
before pushing it upstream, but I'm not 100% sure.
Do you mean the patch needs to wait for next kernel promotion? 4.12? 4.13?

No, well maybe. The problem is that I'm not 100% sure if there isn't anything 
in drm-next which breaks this.

The easiest way to figure that out is to try to apply the patch to Dave's 
current drm-next branch and see if it still works as expected.

I think Alex plan was to setup an amd-staging-next branch to solve such 
problems, but that hasn't got very far I think.

Regards,
Christian.



Regards,
David Zhou


Christian.

Am 16.06.2017 um 22:35 schrieb Alex Deucher:
The patches look good to me.  Christian what did you have in mind?
Acked-by: Alex Deucher 
<alexander.deuc...@amd.com<mailto:alexander.deuc...@amd.com>>

Alex

On Fri, Jun 16, 2017 at 4:08 AM, zhoucm1 
<david1.z...@amd.com<mailto:david1.z...@amd.com>> wrote:
ping...?



 Forwarded Message 
Subject:

upsteam find bo api

Date:

Wed, 14 Jun 2017 18:16:24 +0800

From:

zhoucm1 <david1.z...@amd.com><mailto:david1.z...@amd.com>

To:

amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org>



Hi all,



Since patches are one feature, and contain kernel and libdrm, I attached

them not by send-mail. Hope not inconvenience.



0001-drm-amdgpu-return-bo-itself-if-userptr-is-cpu-addr-o.patch is

kernel patch.

Other three is libdrm patches including unit test.



please review.



Regards,

David Zhou





___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org>
https://lists.freedesktop.org/mailman/listinfo/amd-gfx





___

amd-gfx mailing list

amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org>

https://lists.freedesktop.org/mailman/listinfo/amd-gfx





___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Re: upsteam find bo api

[Christian König]

Am 19.06.2017 um 04:23 schrieb zhoucm1:



On 2017年06月17日 18:09, Christian König wrote:
The patch should still work on amd-staging-4.11, but drm-next has 
seen some changes on how the handles are stored for dma-buf.


This can potentially break the implementation and I think we need to 
adapt before pushing it upstream, but I'm not 100% sure.

Do you mean the patch needs to wait for next kernel promotion? 4.12? 4.13?


No, well maybe. The problem is that I'm not 100% sure if there isn't 
anything in drm-next which breaks this.


The easiest way to figure that out is to try to apply the patch to 
Dave's current drm-next branch and see if it still works as expected.


I think Alex plan was to setup an amd-staging-next branch to solve such 
problems, but that hasn't got very far I think.


Regards,
Christian.



Regards,
David Zhou


Christian.

Am 16.06.2017 um 22:35 schrieb Alex Deucher:

The patches look good to me.  Christian what did you have in mind?
Acked-by: Alex Deucher >


Alex

On Fri, Jun 16, 2017 at 4:08 AM, zhoucm1 > wrote:


ping...?



 Forwarded Message 
Subject:upsteam find bo api
Date:   Wed, 14 Jun 2017 18:16:24 +0800
From:   zhoucm1  
To: amd-gfx@lists.freedesktop.org




Hi all,

Since patches are one feature, and contain kernel and libdrm, I attached
them not by send-mail. Hope not inconvenience.

0001-drm-amdgpu-return-bo-itself-if-userptr-is-cpu-addr-o.patch is
kernel patch.
Other three is libdrm patches including unit test.

please review.

Regards,
David Zhou




___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org 
https://lists.freedesktop.org/mailman/listinfo/amd-gfx





___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx







___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

RE: upsteam find bo api

[He, Roger]

Patches Look good to me in itself. Acked-by: Roger.He 
<hongbo...@amd.com<mailto:hongbo...@amd.com>>
But of course  you should double confirm if there is Christian’s concern.

Thanks
Roger(Hongbo.He)
From: Zhou, David(ChunMing)
Sent: Friday, June 16, 2017 7:18 PM
To: He, Roger <hongbo...@amd.com>; amd-gfx@lists.freedesktop.org
Subject: 回复: upsteam find bo api

find bo API is used by vulkan， which is being in hybrid only previous， but 
now，they want to upstream.

Sent from my Huawei Mobile

 原始邮件 ----
主题：RE: upsteam find bo api
发件人："He, Roger"
收件人："Zhou, David(ChunMing)" ,amd-gfx@lists.freedesktop.org
抄送：
What is the background or what is it for?

Thanks
Roger(Hongbo.He)
From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of 
zhoucm1
Sent: Friday, June 16, 2017 4:09 PM
To: amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org>
Subject: Fwd: upsteam find bo api

ping...?


 Forwarded Message 
Subject:

upsteam find bo api

Date:

Wed, 14 Jun 2017 18:16:24 +0800

From:

zhoucm1 <david1.z...@amd.com><mailto:david1.z...@amd.com>

To:

amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org>



Hi all,



Since patches are one feature, and contain kernel and libdrm, I attached

them not by send-mail. Hope not inconvenience.



0001-drm-amdgpu-return-bo-itself-if-userptr-is-cpu-addr-o.patch is

kernel patch.

Other three is libdrm patches including unit test.



please review.



Regards,

David Zhou




___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Re: upsteam find bo api

[zhoucm1]

On 2017年06月17日 18:09, Christian König wrote:
The patch should still work on amd-staging-4.11, but drm-next has seen 
some changes on how the handles are stored for dma-buf.


This can potentially break the implementation and I think we need to 
adapt before pushing it upstream, but I'm not 100% sure.

Do you mean the patch needs to wait for next kernel promotion? 4.12? 4.13?

Regards,
David Zhou


Christian.

Am 16.06.2017 um 22:35 schrieb Alex Deucher:

The patches look good to me.  Christian what did you have in mind?
Acked-by: Alex Deucher >


Alex

On Fri, Jun 16, 2017 at 4:08 AM, zhoucm1 > wrote:


ping...?



 Forwarded Message 
Subject:upsteam find bo api
Date:   Wed, 14 Jun 2017 18:16:24 +0800
From:   zhoucm1  
To: amd-gfx@lists.freedesktop.org




Hi all,

Since patches are one feature, and contain kernel and libdrm, I attached
them not by send-mail. Hope not inconvenience.

0001-drm-amdgpu-return-bo-itself-if-userptr-is-cpu-addr-o.patch is
kernel patch.
Other three is libdrm patches including unit test.

please review.

Regards,
David Zhou




___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org 
https://lists.freedesktop.org/mailman/listinfo/amd-gfx





___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx





___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Re: upsteam find bo api

[Christian König]

The patch should still work on amd-staging-4.11, but drm-next has seen 
some changes on how the handles are stored for dma-buf.


This can potentially break the implementation and I think we need to 
adapt before pushing it upstream, but I'm not 100% sure.


Christian.

Am 16.06.2017 um 22:35 schrieb Alex Deucher:

The patches look good to me.  Christian what did you have in mind?
Acked-by: Alex Deucher >


Alex

On Fri, Jun 16, 2017 at 4:08 AM, zhoucm1 > wrote:


ping...?



 Forwarded Message 
Subject:upsteam find bo api
Date:   Wed, 14 Jun 2017 18:16:24 +0800
From:   zhoucm1  
To: amd-gfx@lists.freedesktop.org




Hi all,

Since patches are one feature, and contain kernel and libdrm, I attached
them not by send-mail. Hope not inconvenience.

0001-drm-amdgpu-return-bo-itself-if-userptr-is-cpu-addr-o.patch is
kernel patch.
Other three is libdrm patches including unit test.

please review.

Regards,
David Zhou




___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org 
https://lists.freedesktop.org/mailman/listinfo/amd-gfx





___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx



___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Re: upsteam find bo api

[Alex Deucher]

We are getting the patches out in preparation for open source vulkan.
Additionally Christian has plans to use them in mesa as well.

Alex

On Fri, Jun 16, 2017 at 4:47 AM, He, Roger  wrote:

> What is the background or what is it for?
>
>
>
> Thanks
>
> Roger(Hongbo.He)
>
> *From:* amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] *On Behalf
> Of *zhoucm1
> *Sent:* Friday, June 16, 2017 4:09 PM
> *To:* amd-gfx@lists.freedesktop.org
> *Subject:* Fwd: upsteam find bo api
>
>
>
> ping...?
>
>
>
>  Forwarded Message 
>
> *Subject: *
>
> upsteam find bo api
>
> *Date: *
>
> Wed, 14 Jun 2017 18:16:24 +0800
>
> *From: *
>
> zhoucm1  
>
> *To: *
>
> amd-gfx@lists.freedesktop.org
>
>
>
> Hi all,
>
>
>
> Since patches are one feature, and contain kernel and libdrm, I attached
>
> them not by send-mail. Hope not inconvenience.
>
>
>
> 0001-drm-amdgpu-return-bo-itself-if-userptr-is-cpu-addr-o.patch is
>
> kernel patch.
>
> Other three is libdrm patches including unit test.
>
>
>
> please review.
>
>
>
> Regards,
>
> David Zhou
>
>
>
>
>
>
>
> ___
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
>
>
___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Re: upsteam find bo api

[Emil Velikov]

On 16 June 2017 at 09:08, zhoucm1  wrote:
>
> ping...?
>
Normally you want to send patches as plain text (ideally git
send-email), so that people can reply/comment inline.
But above all a summary of the "requirements" and "how it's achieved"
helps your colleagues, amongst others ;-)

-Emil
___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

RE: upsteam find bo api

[He, Roger]

What is the background or what is it for?

Thanks
Roger(Hongbo.He)
From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of 
zhoucm1
Sent: Friday, June 16, 2017 4:09 PM
To: amd-gfx@lists.freedesktop.org
Subject: Fwd: upsteam find bo api

ping...?


 Forwarded Message 
Subject:

upsteam find bo api

Date:

Wed, 14 Jun 2017 18:16:24 +0800

From:

zhoucm1 

To:

amd-gfx@lists.freedesktop.org



Hi all,



Since patches are one feature, and contain kernel and libdrm, I attached

them not by send-mail. Hope not inconvenience.



0001-drm-amdgpu-return-bo-itself-if-userptr-is-cpu-addr-o.patch is

kernel patch.

Other three is libdrm patches including unit test.



please review.



Regards,

David Zhou




___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx