Thread (4 messages) flat view 4 messages, 3 authors, 2017-08-04
DORMANTno replies

[PATCH][RESEND] arm64: kvm: reuse existing cache type/info related macros

From: Sudeep Holla <hidden>
Date: 2017-08-04 13:18:01
Also in: kvmarm

Hi Christoffer,

On 04/08/17 14:08, Christoffer Dall wrote:
Hi Sudeep,

On Fri, Aug 04, 2017 at 01:53:57PM +0100, Sudeep Holla wrote:
quoted
We already have various macros related to cache type and bitfields in
CLIDR system register. We can replace some of the hardcoded values
here using those existing macros.

This patch reuses those existing cache type/info related macros and
replaces the hardcorded values. It also removes some of the comments
that become trivial with the macro names.

Cc: Catalin Marinas <catalin.marinas@arm.com>
Cc: Will Deacon <redacted>
Cc: Christoffer Dall <redacted>
Cc: Marc Zyngier <redacted>
Signed-off-by: Sudeep Holla <redacted>
---
 arch/arm64/include/asm/cache.h |  7 +++++++
 arch/arm64/kernel/cacheinfo.c  |  7 -------
 arch/arm64/kvm/sys_regs.c      | 29 +++++++++++++++--------------
 3 files changed, 22 insertions(+), 21 deletions(-)

Hi,

I dropped the support for 64bit format CCSIDR after Will's commit a8d4636f96ad
("arm64: cacheinfo: Remove CCSIDR-based cache information probing"). However
I forgot to follow up on this patch which can be still applied. So just
reposting again rebasing on v4.13-rc3 as mentioned by Will as it was too
late for last cycle. Christoffer was fine with the changes but has not
given an official ACK.
Reviewed-by: Christoffer Dall <redacted>
Thanks for the quick response and review tag.

-- 
Regards,
Sudeep
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help