Re: [PATCH v2] radeon: Deinline indirect register accessor functions

2015-05-20 Thread Denys Vlasenko
List >> Subject: Re: [PATCH v2] radeon: Deinline indirect register accessor functions >> >> On Mon, May 18, 2015 at 9:09 PM, Christian König >> wrote: >>>> r600_uvd_ctx_rreg: 111 bytes, 4 callsites >>>> r600_uvd_ctx_wreg: 113 bytes, 5 callsites >&

Re: [PATCH v2] radeon: Deinline indirect register accessor functions

2015-05-20 Thread Denys Vlasenko
: Deinline indirect register accessor functions On Mon, May 18, 2015 at 9:09 PM, Christian König christian.koe...@amd.com wrote: r600_uvd_ctx_rreg: 111 bytes, 4 callsites r600_uvd_ctx_wreg: 113 bytes, 5 callsites eg_pif_phy0_rreg: 106 bytes, 13 callsites eg_pif_phy0_wreg: 108 bytes, 13 callsites

RE: [PATCH v2] radeon: Deinline indirect register accessor functions

2015-05-18 Thread Deucher, Alexander
> -Original Message- > From: Denys Vlasenko [mailto:vda.li...@googlemail.com] > Sent: Monday, May 18, 2015 6:50 PM > To: Koenig, Christian > Cc: Denys Vlasenko; Deucher, Alexander; Linux Kernel Mailing List > Subject: Re: [PATCH v2] radeon: Deinline indirect register

Re: [PATCH v2] radeon: Deinline indirect register accessor functions

2015-05-18 Thread Denys Vlasenko
On Mon, May 18, 2015 at 9:09 PM, Christian König wrote: >> r600_uvd_ctx_rreg: 111 bytes, 4 callsites >> r600_uvd_ctx_wreg: 113 bytes, 5 callsites >> eg_pif_phy0_rreg: 106 bytes, 13 callsites >> eg_pif_phy0_wreg: 108 bytes, 13 callsites >> eg_pif_phy1_rreg: 107 bytes, 13 callsites >>

Re: [PATCH v2] radeon: Deinline indirect register accessor functions

2015-05-18 Thread Christian König
On 18.05.2015 21:02, Denys Vlasenko wrote: This patch deinlines indirect register accessor functions. These functions perform two mmio accesses, framed by spin lock/unlock. Spin lock/unlock by itself takes more than 50 cycles in ideal case (if lock is exclusively cached on current CPU). With

Re: [PATCH v2] radeon: Deinline indirect register accessor functions

2015-05-18 Thread Christian König
On 18.05.2015 21:02, Denys Vlasenko wrote: This patch deinlines indirect register accessor functions. These functions perform two mmio accesses, framed by spin lock/unlock. Spin lock/unlock by itself takes more than 50 cycles in ideal case (if lock is exclusively cached on current CPU). With

Re: [PATCH v2] radeon: Deinline indirect register accessor functions

2015-05-18 Thread Denys Vlasenko
On Mon, May 18, 2015 at 9:09 PM, Christian König christian.koe...@amd.com wrote: r600_uvd_ctx_rreg: 111 bytes, 4 callsites r600_uvd_ctx_wreg: 113 bytes, 5 callsites eg_pif_phy0_rreg: 106 bytes, 13 callsites eg_pif_phy0_wreg: 108 bytes, 13 callsites eg_pif_phy1_rreg: 107 bytes, 13 callsites

RE: [PATCH v2] radeon: Deinline indirect register accessor functions

2015-05-18 Thread Deucher, Alexander
-Original Message- From: Denys Vlasenko [mailto:vda.li...@googlemail.com] Sent: Monday, May 18, 2015 6:50 PM To: Koenig, Christian Cc: Denys Vlasenko; Deucher, Alexander; Linux Kernel Mailing List Subject: Re: [PATCH v2] radeon: Deinline indirect register accessor functions On Mon