From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id DDF48C6FD1D for ; Wed, 15 Mar 2023 10:09:19 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 7FB626B0078; Wed, 15 Mar 2023 06:09:19 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 7AB916B007B; Wed, 15 Mar 2023 06:09:19 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 673056B007D; Wed, 15 Mar 2023 06:09:19 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id 58CAA6B0078 for ; Wed, 15 Mar 2023 06:09:19 -0400 (EDT) Received: from smtpin28.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id 2AA22A0EA7 for ; Wed, 15 Mar 2023 10:09:19 +0000 (UTC) X-FDA: 80570710038.28.A2DBB64 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf28.hostedemail.com (Postfix) with ESMTP id 84CBCC0004 for ; Wed, 15 Mar 2023 10:09:16 +0000 (UTC) Authentication-Results: imf28.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=jjtPd61r; dmarc=pass (policy=none) header.from=kernel.org; spf=pass (imf28.hostedemail.com: domain of rppt@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=rppt@kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1678874956; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=4eS9h8lEodt6vlt5EM9P8ASes5pRWaMrEDfMmrRTh5A=; b=xcGCBpdRiugGjEeHxtefSQhlk0E2pSyjI8RqGsJBM1lTh+ZFnnESTQ4W2cBQgeoYjjW5Kk cU1UAL1Y1MWuxm1YFsbUQ87yW5I/GKlv3X6c+htR8soHmDQSXMncKOU2zX6YpOwO6wvlri PvrgUoaCVm9QyUxz8zrjmYuZFO5feoE= ARC-Authentication-Results: i=1; imf28.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=jjtPd61r; dmarc=pass (policy=none) header.from=kernel.org; spf=pass (imf28.hostedemail.com: domain of rppt@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=rppt@kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1678874956; a=rsa-sha256; cv=none; b=1w/lg0HZ0SYSn+LP6Y4f0blRFssmuCPzrWVM8FRlth3jkJvJuZCZoIZ8m+QPq441yvQEUq MqdHVxNBxKcwdvrc8Y8HnPZjvSnIQo2MPudbTdmFFgvPgMbf1HPFZPUS+m9MUzQ9VqQN8B asZLVlDze7RAUTVp3YgEa1z5EXIg43Q= Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id C01F061CBB; Wed, 15 Mar 2023 10:09:15 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 70B5EC433EF; Wed, 15 Mar 2023 10:09:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1678874955; bh=XKqVr5hICGOBlR+cx6c6sjb7Ir7DaL2tVvV4MlSqRP8=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=jjtPd61r7kNsD8GPQd8Ts0XVGWPrBIjz1KFFHvNKDZo8mbpQGpYLsC/rvSACKN0U/ EnFzRuLtdl/2KZ4jO7r+1IzGT7gyylb6DcCfrRgNd7piPJHf2Y/qDihiY6E6HxT+33 ot91x0o4Rkwb27i7YzaY3hrEOqE9CZuJpDzWhUvK30JLsn5BLyNM9e6r3wQPVE/PUr 1RTlne3apaibEOAsBhlHUEteUlshoZOdpnixYNvBWmLCiqvGdAa1zfUEiuLKJSjgeg YiVXhP34fav+cakYGfKejOL1xmPvnk0IaT5+YqKS7S+73mEIyu/UN8X75RXZ/3PN0f JjEiLzfdAyqvQ== Date: Wed, 15 Mar 2023 12:09:02 +0200 From: Mike Rapoport To: "Matthew Wilcox (Oracle)" Cc: linux-arch@vger.kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org, Jonas Bonn , Stefan Kristiansson , Stafford Horne , linux-openrisc@vger.kernel.org Subject: Re: [PATCH v4 18/36] openrisc: Implement the new page table range API Message-ID: References: <20230315051444.3229621-1-willy@infradead.org> <20230315051444.3229621-19-willy@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230315051444.3229621-19-willy@infradead.org> X-Rspamd-Queue-Id: 84CBCC0004 X-Rspamd-Server: rspam09 X-Rspam-User: X-Stat-Signature: nxpg8aoeqccacgq9uwh1rj8t8wpb6sie X-HE-Tag: 1678874956-767586 X-HE-Meta: U2FsdGVkX1/56orez/zl2WWw6HJ+/q8g13SOhQ8DFAphB5FM0twQt+bkc+UhMoB4M+m+/rCn2Hx2UMW441jzC+1+RtegTK/fIuHAusHlabHvpz8uUsyQw1rCovJKdP3juZNd+aP8udLIY8TCHS3Wp+BkNoUVBLY8HylyTeSV0ZxsUwSNpL3ADVmUqw8S8d2lDGqpsxTfJaUArjDsXPf0DWMjxX0gh2WskkkUjA8gUrclgCJf61fyVlTSEnWlWCRugO/OYCTEZONWDFoFfbomE9utAn1jFHes5VURNd+4/WlOrZYR0fGX8vJuSH4fWZug+zz6Vge0xMKHy6DHawJdiwHYGALGd2FMB+IhqG9wezgNdAyLQDTNTvcrRlIsdpeNUyodPwduHRFltvwG0YJyTm3qeuTtQbxOxX9f8dKWiB5Xw2QgbJIaIQ2lzZqkk6MAUlblhZTdMEWSoTvm4cSo8XZvBLvjuxbOSMVohf/ztqx1YHuMiMzSFW6KnThQEH3glRVEa32UuuziH8j+ZsnlAeAm9drGzBXHMBGYnpkupX7fwDBhw6ZOid0pbmZIeHhUx51Lrs0gW53vtav6tSI3EW2a3ei2DhhfiolNHVy2t6RNC/qVsN0rXND/uAofxMzkpyHdT/GvVK6mle6xz6x69Y8gNBnltxyTCj2pkKIjJPXBUc+YS6HLptvmoF4lZ4LtlJbdWcc/1jauonFuwKvHnSf7MSWGg4Fej5nbeiVfKtLYQMtiqXOXVty320FyvTEDRS9kbFHwS3qwvaQJguzY8qIM9YXifWW5ETlbhKePB1/kVnuZF9lQqrh7cBvxwHwN+gTb6hkcFZlROz+vJ/j0HzqbpReS2aG/DVFscbkx/PlERA4HxKlVNJQChx2ZozMU1g2MtBtbIl7EnPUwodmWsQ/eaWQL/YFhdwviURuyY1+ylkyWpbxU+npaDKAihI9iLp+UlWr2EQ1sk8EB0pF qLykIEGx JGJP01Dhk8fe5Iq4W92QduGC2cgjgbP9mgl2hhNtZqSlIvygqZpcoi2yj4goMzN37ow4OFPeBViopppDfz9UiAXkpDzNp+LfPQAsJq/CD5ffAkWHaofAOtVZQj9opUxSenK4Ec8N3IVOAjVOIQ6jaxEgLPnGTzLfPCcNjt95Zm2wOSzAObQfPk+Q5YH3s4qnLGjPuxJDYXBEbpSSVmmmDetK1pvvaoP7mBss5lkeCbUfgAVW36eblt/7LLqau0B8E+80LXsC+JJMWXxIglhwCfbIBqgFlhGeB+2G8+w7+OZXRyO4SbSX3LoREfIASfeEMv2oXkKUxJ4ulOexzAgG7ogodNms+50kNZfdzd12niOJf2ilGLkIG4V3Fjxh9B0UhCZZNOXE0WDFRcyg79jpvcrpz8kGXzIWI5cewknnX2ilmcntrYHJkd8vXp6CwUNOO7+Yt X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Wed, Mar 15, 2023 at 05:14:26AM +0000, Matthew Wilcox (Oracle) wrote: > Add PFN_PTE_SHIFT, update_mmu_cache_range() and flush_dcache_folio(). > Change the PG_arch_1 (aka PG_dcache_dirty) flag from being per-page > to per-folio. > > Signed-off-by: Matthew Wilcox (Oracle) > Cc: Jonas Bonn > Cc: Stefan Kristiansson > Cc: Stafford Horne > Cc: linux-openrisc@vger.kernel.org Acked-by: Mike Rapoport (IBM) > --- > arch/openrisc/include/asm/cacheflush.h | 8 +++++++- > arch/openrisc/include/asm/pgtable.h | 14 +++++++++----- > arch/openrisc/mm/cache.c | 12 ++++++++---- > 3 files changed, 24 insertions(+), 10 deletions(-) > > diff --git a/arch/openrisc/include/asm/cacheflush.h b/arch/openrisc/include/asm/cacheflush.h > index eeac40d4a854..984c331ff5f4 100644 > --- a/arch/openrisc/include/asm/cacheflush.h > +++ b/arch/openrisc/include/asm/cacheflush.h > @@ -56,10 +56,16 @@ static inline void sync_icache_dcache(struct page *page) > */ > #define PG_dc_clean PG_arch_1 > > +static inline void flush_dcache_folio(struct folio *folio) > +{ > + clear_bit(PG_dc_clean, &folio->flags); > +} > +#define flush_dcache_folio flush_dcache_folio > + > #define ARCH_IMPLEMENTS_FLUSH_DCACHE_PAGE 1 > static inline void flush_dcache_page(struct page *page) > { > - clear_bit(PG_dc_clean, &page->flags); > + flush_dcache_folio(page_folio(page)); > } > > #define flush_icache_user_page(vma, page, addr, len) \ > diff --git a/arch/openrisc/include/asm/pgtable.h b/arch/openrisc/include/asm/pgtable.h > index 3eb9b9555d0d..2f42a12c40ab 100644 > --- a/arch/openrisc/include/asm/pgtable.h > +++ b/arch/openrisc/include/asm/pgtable.h > @@ -46,7 +46,7 @@ extern void paging_init(void); > * hook is made available. > */ > #define set_pte(pteptr, pteval) ((*(pteptr)) = (pteval)) > -#define set_pte_at(mm, addr, ptep, pteval) set_pte(ptep, pteval) > + > /* > * (pmds are folded into pgds so this doesn't get actually called, > * but the define is needed for a generic inline function.) > @@ -357,6 +357,7 @@ static inline unsigned long pmd_page_vaddr(pmd_t pmd) > #define __pmd_offset(address) \ > (((address) >> PMD_SHIFT) & (PTRS_PER_PMD-1)) > > +#define PFN_PTE_SHIFT PAGE_SHIFT > #define pte_pfn(x) ((unsigned long)(((x).pte)) >> PAGE_SHIFT) > #define pfn_pte(pfn, prot) __pte((((pfn) << PAGE_SHIFT)) | pgprot_val(prot)) > > @@ -379,13 +380,16 @@ static inline void update_tlb(struct vm_area_struct *vma, > extern void update_cache(struct vm_area_struct *vma, > unsigned long address, pte_t *pte); > > -static inline void update_mmu_cache(struct vm_area_struct *vma, > - unsigned long address, pte_t *pte) > +static inline void update_mmu_cache_range(struct vm_area_struct *vma, > + unsigned long address, pte_t *ptep, unsigned int nr) > { > - update_tlb(vma, address, pte); > - update_cache(vma, address, pte); > + update_tlb(vma, address, ptep); > + update_cache(vma, address, ptep); > } > > +#define update_mmu_cache(vma, addr, ptep) \ > + update_mmu_cache_range(vma, addr, ptep, 1) > + > /* __PHX__ FIXME, SWAP, this probably doesn't work */ > > /* > diff --git a/arch/openrisc/mm/cache.c b/arch/openrisc/mm/cache.c > index 534a52ec5e66..eb43b73f3855 100644 > --- a/arch/openrisc/mm/cache.c > +++ b/arch/openrisc/mm/cache.c > @@ -43,15 +43,19 @@ void update_cache(struct vm_area_struct *vma, unsigned long address, > pte_t *pte) > { > unsigned long pfn = pte_val(*pte) >> PAGE_SHIFT; > - struct page *page = pfn_to_page(pfn); > - int dirty = !test_and_set_bit(PG_dc_clean, &page->flags); > + struct folio *folio = page_folio(pfn_to_page(pfn)); > + int dirty = !test_and_set_bit(PG_dc_clean, &folio->flags); > > /* > * Since icaches do not snoop for updated data on OpenRISC, we > * must write back and invalidate any dirty pages manually. We > * can skip data pages, since they will not end up in icaches. > */ > - if ((vma->vm_flags & VM_EXEC) && dirty) > - sync_icache_dcache(page); > + if ((vma->vm_flags & VM_EXEC) && dirty) { > + unsigned int nr = folio_nr_pages(folio); > + > + while (nr--) > + sync_icache_dcache(folio_page(folio, nr)); > + } > } > > -- > 2.39.2 > > -- Sincerely yours, Mike.