Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) client-ip=23.128.96.33;
From:   Catalin Marinas <catalin.marinas@arm.com>
To:     Will Deacon <will@kernel.org>, Steven Price <steven.price@arm.com>,
        Peter Collingbourne <pcc@google.com>,
        Ryan Roberts <ryan.roberts@arm.com>
Cc:     linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2] arm64/mm: Hoist synchronization out of set_ptes() loop
Date:   Wed, 18 Oct 2023 11:15:59 +0100
Message-Id: <169762410085.3624197.15162390059059851386.b4-ty@arm.com>
In-Reply-To: <20231005140730.2191134-1-ryan.roberts@arm.com>
References: <20231005140730.2191134-1-ryan.roberts@arm.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
Precedence: bulk

On Thu, 05 Oct 2023 15:07:30 +0100, Ryan Roberts wrote:
> set_ptes() sets a physically contiguous block of memory (which all
> belongs to the same folio) to a contiguous block of ptes. The arm64
> implementation of this previously just looped, operating on each
> individual pte. But the __sync_icache_dcache() and mte_sync_tags()
> operations can both be hoisted out of the loop so that they are
> performed once for the contiguous set of pages (which may be less than
> the whole folio). This should result in minor performance gains.
> 
> [...]

Applied to arm64 (for-next/misc), thanks!

[1/1] arm64/mm: Hoist synchronization out of set_ptes() loop
      https://git.kernel.org/arm64/c/3425cec42c3c

Also pushed the fix to mark 'addr' __always_unused.

-- 
Catalin