| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20231211135052.4fb016a6@canb.auug.org.au>
Date: Mon, 11 Dec 2023 13:50:52 +1100
From: Stephen Rothwell <sfr@...b.auug.org.au>
To: Paolo Bonzini <pbonzini@...hat.com>,
Andrew Morton <akpm@...ux-foundation.org>
Cc: KVM <kvm@...r.kernel.org>, Ackerley Tng <ackerleytng@...gle.com>,
Chao Peng <chao.p.peng@...ux.intel.com>,
Isaku Yamahata <isaku.yamahata@...el.com>,
"Kirill A. Shutemov" <kirill.shutemov@...ux.intel.com>,
Michael Roth <michael.roth@....com>,
Sean Christopherson <seanjc@...gle.com>,
Yu Zhang <yu.c.zhang@...ux.intel.com>,
"Matthew Wilcox (Oracle)" <willy@...radead.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Linux Next Mailing List <linux-next@...r.kernel.org>
Subject: Re: linux-next: build failure after merge of the kvm tree
Hi all,
On Mon, 20 Nov 2023 15:22:27 +1100 Stephen Rothwell <sfr@...b.auug.org.au> wrote:
>
> After merging the kvm tree, today's linux-next build (x86_64 allmodconfig)
> failed like this:
>
> arch/x86/kvm/../../../virt/kvm/guest_memfd.c:306:10: error: 'const struct address_space_operations' has no member named 'error_remove_page'; did you mean 'error_remove_folio'?
> 306 | .error_remove_page = kvm_gmem_error_page,
> | ^~~~~~~~~~~~~~~~~
> | error_remove_folio
> arch/x86/kvm/../../../virt/kvm/guest_memfd.c:306:30: error: initialization of 'int (*)(struct folio *)' from incompatible pointer type 'int (*)(struct address_space *, struct page *)' [-Werror=incompatible-pointer-types]
> 306 | .error_remove_page = kvm_gmem_error_page,
> | ^~~~~~~~~~~~~~~~~~~
> arch/x86/kvm/../../../virt/kvm/guest_memfd.c:306:30: note: (near initialization for 'kvm_gmem_aops.launder_folio')
>
> Caused by commit
>
> 640be5bc564f ("fs: convert error_remove_page to error_remove_folio")
>
> from the mm tree intercting with commit
>
> a7800aa80ea4 ("KVM: Add KVM_CREATE_GUEST_MEMFD ioctl() for guest-specific backing memory")
>
> I have applied the following supplied merge fix patch (thanks Andrew).
>
> From: Andrew Morton <akpm@...ux-foundation.org>
> Date: Fri, 17 Nov 2023 09:28:33 -0800
> Subject: [PATCH] fs: Convert error_remove_page to error_remove_folio
>
> On Fri, 17 Nov 2023 16:14:47 +0000 "Matthew Wilcox (Oracle)" <willy@...radead.org> wrote:
>
> > There were already assertions that we were not passing a tail page
> > to error_remove_page(), so make the compiler enforce that by converting
> > everything to pass and use a folio.
> >
> > Signed-off-by: Matthew Wilcox (Oracle) <willy@...radead.org>
> > ---
> > Documentation/filesystems/locking.rst | 4 ++--
> > Documentation/filesystems/vfs.rst | 6 +++---
> > block/fops.c | 2 +-
> > fs/afs/write.c | 2 +-
> > fs/bcachefs/fs.c | 2 +-
> > fs/btrfs/inode.c | 2 +-
> > fs/ceph/addr.c | 4 ++--
> > fs/ext2/inode.c | 2 +-
> > fs/ext4/inode.c | 6 +++---
> > fs/f2fs/compress.c | 2 +-
> > fs/f2fs/inode.c | 2 +-
> > fs/gfs2/aops.c | 4 ++--
> > fs/hugetlbfs/inode.c | 6 +++---
> > fs/nfs/file.c | 2 +-
> > fs/ntfs/aops.c | 6 +++---
> > fs/ocfs2/aops.c | 2 +-
> > fs/xfs/xfs_aops.c | 2 +-
> > fs/zonefs/file.c | 2 +-
> > include/linux/fs.h | 2 +-
> > include/linux/mm.h | 3 ++-
> > mm/memory-failure.c | 10 +++++-----
> > mm/shmem.c | 6 +++---
> > mm/truncate.c | 9 ++++-----
> > virt/kvm/guest_memfd.c | 9 +++++----
>
> virt/kvm/guest_memfd.c exists only in the KVM tree (and hence
> linux-next). So I assume Stephen will use the change from this patch
> when doing his resolution.
>
> This:
> ---
Now this:
virt/kvm/guest_memfd.c | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/virt/kvm/guest_memfd.c b/virt/kvm/guest_memfd.c
index c2e2371720a9..c23ce219e21c 100644
--- a/virt/kvm/guest_memfd.c
+++ b/virt/kvm/guest_memfd.c
@@ -267,7 +267,8 @@ static int kvm_gmem_migrate_folio(struct address_space *mapping,
return -EINVAL;
}
-static int kvm_gmem_error_page(struct address_space *mapping, struct page *page)
+static int kvm_gmem_error_folio(struct address_space *mapping,
+ struct folio *folio)
{
struct list_head *gmem_list = &mapping->private_list;
struct kvm_gmem *gmem;
@@ -275,8 +276,8 @@ static int kvm_gmem_error_page(struct address_space *mapping, struct page *page)
filemap_invalidate_lock_shared(mapping);
- start = page->index;
- end = start + thp_nr_pages(page);
+ start = folio->index;
+ end = start + folio_nr_pages(folio);
list_for_each_entry(gmem, gmem_list, entry)
kvm_gmem_invalidate_begin(gmem, start, end);
@@ -301,7 +302,7 @@ static int kvm_gmem_error_page(struct address_space *mapping, struct page *page)
static const struct address_space_operations kvm_gmem_aops = {
.dirty_folio = noop_dirty_folio,
.migrate_folio = kvm_gmem_migrate_folio,
- .error_remove_page = kvm_gmem_error_page,
+ .error_remove_folio = kvm_gmem_error_folio,
};
static int kvm_gmem_getattr(struct mnt_idmap *idmap, const struct path *path,
--
Cheers,
Stephen Rothwell
Content of type "application/pgp-signature" skipped
Powered by blists - more mailing lists