[PATCH] drm/radeon: Inline r100_mm_rreg, v2

2014-04-20 Thread Christian König
Am 19.04.2014 19:33, schrieb Lauri Kasanen: > On Sat, 19 Apr 2014 11:15:53 -0400 > Alex Deucher wrote: > >> On Sat, Apr 19, 2014 at 6:06 AM, Christian K?nig This was originally un-inlined by Andi Kleen in 2011 citing size concerns. Indeed, a first attempt at inlining it grew radeon.ko by

[PATCH] drm/radeon: Inline r100_mm_rreg, v2

2014-04-19 Thread Lauri Kasanen
On Sat, 19 Apr 2014 11:15:53 -0400 Alex Deucher wrote: > On Sat, Apr 19, 2014 at 6:06 AM, Christian K?nig > >> This was originally un-inlined by Andi Kleen in 2011 citing size concerns. > >> Indeed, a first attempt at inlining it grew radeon.ko by 7%. > >> > >> However, 2% of cpu is spent in this

[PATCH] drm/radeon: Inline r100_mm_rreg, v2

2014-04-19 Thread Christian König
Am 18.04.2014 23:11, schrieb Lauri Kasanen: > This was originally un-inlined by Andi Kleen in 2011 citing size concerns. > Indeed, a first attempt at inlining it grew radeon.ko by 7%. > > However, 2% of cpu is spent in this function. Simply inlining it gave 1% more > fps > in Urban Terror. > > v2:

[PATCH] drm/radeon: Inline r100_mm_rreg, v2

2014-04-19 Thread Alex Deucher
On Sat, Apr 19, 2014 at 6:06 AM, Christian K?nig wrote: > Am 18.04.2014 23:11, schrieb Lauri Kasanen: > >> This was originally un-inlined by Andi Kleen in 2011 citing size concerns. >> Indeed, a first attempt at inlining it grew radeon.ko by 7%. >> >> However, 2% of cpu is spent in this function.

[PATCH] drm/radeon: Inline r100_mm_rreg, v2

2014-04-19 Thread Lauri Kasanen
This was originally un-inlined by Andi Kleen in 2011 citing size concerns. Indeed, a first attempt at inlining it grew radeon.ko by 7%. However, 2% of cpu is spent in this function. Simply inlining it gave 1% more fps in Urban Terror. v2: We know the minimum MMIO size. Adding it to the if allows