[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <91a621ea-d18a-4428-b1a5-943b4cf2c79e@redhat.com>
Date: Thu, 21 Nov 2024 09:11:23 +0100
From: David Hildenbrand <david@...hat.com>
To: John Hubbard <jhubbard@...dia.com>,
Andrew Morton <akpm@...ux-foundation.org>
Cc: LKML <linux-kernel@...r.kernel.org>, linux-mm@...ck.org,
Oscar Salvador <osalvador@...e.de>,
Vivek Kasireddy <vivek.kasireddy@...el.com>, Dave Airlie
<airlied@...hat.com>, Gerd Hoffmann <kraxel@...hat.com>,
Matthew Wilcox <willy@...radead.org>, Christoph Hellwig <hch@...radead.org>,
Jason Gunthorpe <jgg@...dia.com>, Peter Xu <peterx@...hat.com>,
Arnd Bergmann <arnd@...db.de>, Daniel Vetter <daniel.vetter@...ll.ch>,
Dongwon Kim <dongwon.kim@...el.com>, Hugh Dickins <hughd@...gle.com>,
Junxiao Chang <junxiao.chang@...el.com>, stable@...r.kernel.org
Subject: Re: [PATCH v2] mm/gup: handle NULL pages in unpin_user_pages()
On 21.11.24 04:49, John Hubbard wrote:
> The recent addition of "pofs" (pages or folios) handling to gup has a
> flaw: it assumes that unpin_user_pages() handles NULL pages in the
> pages** array. That's not the case, as I discovered when I ran on a new
> configuration on my test machine.
>
> Fix this by skipping NULL pages in unpin_user_pages(), just like
> unpin_folios() already does.
>
> Details: when booting on x86 with "numa=fake=2 movablecore=4G" on Linux
> 6.12, and running this:
>
> tools/testing/selftests/mm/gup_longterm
>
> ...I get the following crash:
>
> BUG: kernel NULL pointer dereference, address: 0000000000000008
> RIP: 0010:sanity_check_pinned_pages+0x3a/0x2d0
> ...
> Call Trace:
> <TASK>
> ? __die_body+0x66/0xb0
> ? page_fault_oops+0x30c/0x3b0
> ? do_user_addr_fault+0x6c3/0x720
> ? irqentry_enter+0x34/0x60
> ? exc_page_fault+0x68/0x100
> ? asm_exc_page_fault+0x22/0x30
> ? sanity_check_pinned_pages+0x3a/0x2d0
> unpin_user_pages+0x24/0xe0
> check_and_migrate_movable_pages_or_folios+0x455/0x4b0
> __gup_longterm_locked+0x3bf/0x820
> ? mmap_read_lock_killable+0x12/0x50
> ? __pfx_mmap_read_lock_killable+0x10/0x10
> pin_user_pages+0x66/0xa0
> gup_test_ioctl+0x358/0xb20
> __se_sys_ioctl+0x6b/0xc0
> do_syscall_64+0x7b/0x150
> entry_SYSCALL_64_after_hwframe+0x76/0x7e
>
> Fixes: 94efde1d1539 ("mm/gup: avoid an unnecessary allocation call for FOLL_LONGTERM cases")
> Cc: David Hildenbrand <david@...hat.com>
> Cc: Oscar Salvador <osalvador@...e.de>
> Cc: Vivek Kasireddy <vivek.kasireddy@...el.com>
> Cc: Dave Airlie <airlied@...hat.com>
> Cc: Gerd Hoffmann <kraxel@...hat.com>
> Cc: Matthew Wilcox <willy@...radead.org>
> Cc: Christoph Hellwig <hch@...radead.org>
> Cc: Jason Gunthorpe <jgg@...dia.com>
> Cc: Peter Xu <peterx@...hat.com>
> Cc: Arnd Bergmann <arnd@...db.de>
> Cc: Daniel Vetter <daniel.vetter@...ll.ch>
> Cc: Dongwon Kim <dongwon.kim@...el.com>
> Cc: Hugh Dickins <hughd@...gle.com>
> Cc: Junxiao Chang <junxiao.chang@...el.com>
> Cc: <stable@...r.kernel.org>
> Signed-off-by: John Hubbard <jhubbard@...dia.com>
> ---
Thanks!
Acked-by: David Hildenbrand <david@...hat.com>
--
Cheers,
David / dhildenb
Powered by blists - more mailing lists