From: Barry Song <[email protected]>

dcache_by_myline_op ensures completion of the data cache operations for a
region, while dcache_by_myline_op_nosync only issues them without waiting.
This enables deferred synchronization so completion for multiple regions
can be handled together later.

Cc: Leon Romanovsky <[email protected]>
Cc: Catalin Marinas <[email protected]>
Cc: Will Deacon <[email protected]>
Cc: Marek Szyprowski <[email protected]>
Cc: Robin Murphy <[email protected]>
Cc: Ada Couprie Diaz <[email protected]>
Cc: Ard Biesheuvel <[email protected]>
Cc: Marc Zyngier <[email protected]>
Cc: Anshuman Khandual <[email protected]>
Cc: Ryan Roberts <[email protected]>
Cc: Suren Baghdasaryan <[email protected]>
Cc: Tangquan Zheng <[email protected]>
Signed-off-by: Barry Song <[email protected]>
---
 arch/arm64/include/asm/assembler.h  | 24 +++++++++++++++++++-----
 arch/arm64/kernel/relocate_kernel.S |  3 ++-
 2 files changed, 21 insertions(+), 6 deletions(-)

diff --git a/arch/arm64/include/asm/assembler.h 
b/arch/arm64/include/asm/assembler.h
index f0ca7196f6fa..b408ed61866f 100644
--- a/arch/arm64/include/asm/assembler.h
+++ b/arch/arm64/include/asm/assembler.h
@@ -371,14 +371,13 @@ alternative_endif
  * [start, end) with dcache line size explicitly provided.
  *
  *     op:             operation passed to dc instruction
- *     domain:         domain used in dsb instruction
  *     start:          starting virtual address of the region
  *     end:            end virtual address of the region
  *     linesz:         dcache line size
  *     fixup:          optional label to branch to on user fault
  *     Corrupts:       start, end, tmp
  */
-       .macro dcache_by_myline_op op, domain, start, end, linesz, tmp, fixup
+       .macro raw_dcache_by_myline_op op, start, end, linesz, tmp, fixup
        sub     \tmp, \linesz, #1
        bic     \start, \start, \tmp
 .Ldcache_op\@:
@@ -402,14 +401,13 @@ alternative_endif
        add     \start, \start, \linesz
        cmp     \start, \end
        b.lo    .Ldcache_op\@
-       dsb     \domain
 
        _cond_uaccess_extable .Ldcache_op\@, \fixup
        .endm
 
 /*
  * Macro to perform a data cache maintenance for the interval
- * [start, end)
+ * [start, end) and wait for completion
  *
  *     op:             operation passed to dc instruction
  *     domain:         domain used in dsb instruction
@@ -420,7 +418,23 @@ alternative_endif
  */
        .macro dcache_by_line_op op, domain, start, end, tmp1, tmp2, fixup
        dcache_line_size \tmp1, \tmp2
-       dcache_by_myline_op \op, \domain, \start, \end, \tmp1, \tmp2, \fixup
+       raw_dcache_by_myline_op \op, \start, \end, \tmp1, \tmp2, \fixup
+       dsb \domain
+       .endm
+
+/*
+ * Macro to perform a data cache maintenance for the interval
+ * [start, end) without waiting for completion
+ *
+ *     op:             operation passed to dc instruction
+ *     start:          starting virtual address of the region
+ *     end:            end virtual address of the region
+ *     fixup:          optional label to branch to on user fault
+ *     Corrupts:       start, end, tmp1, tmp2
+ */
+       .macro dcache_by_line_op_nosync op, start, end, tmp1, tmp2, fixup
+       dcache_line_size \tmp1, \tmp2
+       raw_dcache_by_myline_op \op, \start, \end, \tmp1, \tmp2, \fixup
        .endm
 
 /*
diff --git a/arch/arm64/kernel/relocate_kernel.S 
b/arch/arm64/kernel/relocate_kernel.S
index 413f899e4ac6..71938eb3a3a3 100644
--- a/arch/arm64/kernel/relocate_kernel.S
+++ b/arch/arm64/kernel/relocate_kernel.S
@@ -64,7 +64,8 @@ SYM_CODE_START(arm64_relocate_new_kernel)
        mov     x19, x13
        copy_page x13, x12, x1, x2, x3, x4, x5, x6, x7, x8
        add     x1, x19, #PAGE_SIZE
-       dcache_by_myline_op civac, sy, x19, x1, x15, x20
+       raw_dcache_by_myline_op civac, x19, x1, x15, x20
+       dsb     sy
        b       .Lnext
 .Ltest_indirection:
        tbz     x16, IND_INDIRECTION_BIT, .Ltest_destination
-- 
2.43.0


Reply via email to