drm/radeon: drop register readback in cayman_cp_int_cntl_setup
authorLucas Stach <dev@lynxeye.de>
Mon, 24 Oct 2016 21:32:04 +0000 (23:32 +0200)
committerSasha Levin <alexander.levin@verizon.com>
Sat, 26 Nov 2016 03:57:02 +0000 (22:57 -0500)
commit90a107c0b85b4b5b2ecfb9fa6b7a714d8bb349e8
treee20c3a9d90215742ad8bb004d9deb01c3a908556
parent9a9a2373142ac2c6fd9df9d038eb929b919e30d7
drm/radeon: drop register readback in cayman_cp_int_cntl_setup

[ Upstream commit 537b4b462caa8bfb9726d9695b8e56e2d5e6b41e ]

The read is taking a considerable amount of time (about 50us on this
machine). The register does not ever hold anything other than the ring
ID that is updated in this exact function, so there is no need for
the read modify write cycle.

This chops off a big chunk of the time spent in hardirq disabled
context, as this function is called multiple times in the interrupt
handler. With this change applied radeon won't show up in the list
of the worst IRQ latency offenders anymore, where it was a regular
before.

Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Lucas Stach <dev@lynxeye.de>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Cc: stable@vger.kernel.org
Signed-off-by: Sasha Levin <alexander.levin@verizon.com>
drivers/gpu/drm/radeon/ni.c