All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kefeng Wang <wangkefeng.wang@huawei.com>
To: Mike Rapoport <rppt@kernel.org>
Cc: <linux-arm-kernel@lists.infradead.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	Anshuman Khandual <anshuman.khandual@arm.com>,
	Ard Biesheuvel <ardb@kernel.org>,
	Catalin Marinas <catalin.marinas@arm.com>,
	David Hildenbrand <david@redhat.com>,
	Marc Zyngier <maz@kernel.org>,
	"Mark Rutland" <mark.rutland@arm.com>,
	Mike Rapoport <rppt@linux.ibm.com>,
	"Will Deacon" <will@kernel.org>, <kvmarm@lists.cs.columbia.edu>,
	<linux-kernel@vger.kernel.org>, <linux-mm@kvack.org>
Subject: Re: arm32: panic in move_freepages (Was [PATCH v2 0/4] arm64: drop pfn_valid_within() and simplify pfn_valid())
Date: Thu, 29 Apr 2021 08:48:26 +0800	[thread overview]
Message-ID: <6ad2956c-70ae-c423-ed7d-88e94c88060f@huawei.com> (raw)
In-Reply-To: <YIj5zcbHBHt7CC8B@kernel.org>


On 2021/4/28 13:59, Mike Rapoport wrote:
> On Tue, Apr 27, 2021 at 07:08:59PM +0800, Kefeng Wang wrote:
>> On 2021/4/27 14:23, Mike Rapoport wrote:
>>> On Mon, Apr 26, 2021 at 11:26:38PM +0800, Kefeng Wang wrote:
>>>> On 2021/4/26 13:20, Mike Rapoport wrote:
>>>>> On Sun, Apr 25, 2021 at 03:51:56PM +0800, Kefeng Wang wrote:
>>>>>> On 2021/4/25 15:19, Mike Rapoport wrote:
>>>>>>
>>>>>>        On Fri, Apr 23, 2021 at 04:11:16PM +0800, Kefeng Wang wrote:
>>>>>>
>>>>>>            I tested this patchset(plus arm32 change, like arm64 does)
>>>>>>            based on lts 5.10,add some debug log, the useful info shows
>>>>>>            below, if we enable HOLES_IN_ZONE, no panic, any idea,
>>>>>>            thanks.
>>>>>>
>>>>>>        Are there any changes on top of 5.10 except for pfn_valid() patch?
>>>>>>        Do you see this panic on 5.10 without the changes?
>>>>>>
>>>>>> Yes, there are some BSP support for arm board based on 5.10,
>>> Is it possible to test 5.12?
> Do you use SPARSMEM? If yes, what is your section size?
> What is the value if CONFIG_FORCE_MAX_ZONEORDER in your configuration?

Yes,

CONFIG_SPARSEMEM=y

CONFIG_SPARSEMEM_STATIC=y

CONFIG_FORCE_MAX_ZONEORDER = 11

CONFIG_PAGE_OFFSET=0xC0000000
CONFIG_HAVE_ARCH_PFN_VALID=y
CONFIG_HIGHMEM=y
#define SECTION_SIZE_BITS    26
#define MAX_PHYSADDR_BITS    32
#define MAX_PHYSMEM_BITS     32


>

WARNING: multiple messages have this Message-ID (diff)
From: Kefeng Wang <wangkefeng.wang@huawei.com>
To: Mike Rapoport <rppt@kernel.org>
Cc: David Hildenbrand <david@redhat.com>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Anshuman Khandual <anshuman.khandual@arm.com>,
	linux-kernel@vger.kernel.org, Mike Rapoport <rppt@linux.ibm.com>,
	linux-mm@kvack.org, kvmarm@lists.cs.columbia.edu,
	Marc Zyngier <maz@kernel.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	Will Deacon <will@kernel.org>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: arm32: panic in move_freepages (Was [PATCH v2 0/4] arm64: drop pfn_valid_within() and simplify pfn_valid())
Date: Thu, 29 Apr 2021 08:48:26 +0800	[thread overview]
Message-ID: <6ad2956c-70ae-c423-ed7d-88e94c88060f@huawei.com> (raw)
In-Reply-To: <YIj5zcbHBHt7CC8B@kernel.org>


On 2021/4/28 13:59, Mike Rapoport wrote:
> On Tue, Apr 27, 2021 at 07:08:59PM +0800, Kefeng Wang wrote:
>> On 2021/4/27 14:23, Mike Rapoport wrote:
>>> On Mon, Apr 26, 2021 at 11:26:38PM +0800, Kefeng Wang wrote:
>>>> On 2021/4/26 13:20, Mike Rapoport wrote:
>>>>> On Sun, Apr 25, 2021 at 03:51:56PM +0800, Kefeng Wang wrote:
>>>>>> On 2021/4/25 15:19, Mike Rapoport wrote:
>>>>>>
>>>>>>        On Fri, Apr 23, 2021 at 04:11:16PM +0800, Kefeng Wang wrote:
>>>>>>
>>>>>>            I tested this patchset(plus arm32 change, like arm64 does)
>>>>>>            based on lts 5.10,add some debug log, the useful info shows
>>>>>>            below, if we enable HOLES_IN_ZONE, no panic, any idea,
>>>>>>            thanks.
>>>>>>
>>>>>>        Are there any changes on top of 5.10 except for pfn_valid() patch?
>>>>>>        Do you see this panic on 5.10 without the changes?
>>>>>>
>>>>>> Yes, there are some BSP support for arm board based on 5.10,
>>> Is it possible to test 5.12?
> Do you use SPARSMEM? If yes, what is your section size?
> What is the value if CONFIG_FORCE_MAX_ZONEORDER in your configuration?

Yes,

CONFIG_SPARSEMEM=y

CONFIG_SPARSEMEM_STATIC=y

CONFIG_FORCE_MAX_ZONEORDER = 11

CONFIG_PAGE_OFFSET=0xC0000000
CONFIG_HAVE_ARCH_PFN_VALID=y
CONFIG_HIGHMEM=y
#define SECTION_SIZE_BITS    26
#define MAX_PHYSADDR_BITS    32
#define MAX_PHYSMEM_BITS     32


>
_______________________________________________
kvmarm mailing list
kvmarm@lists.cs.columbia.edu
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm

WARNING: multiple messages have this Message-ID (diff)
From: Kefeng Wang <wangkefeng.wang@huawei.com>
To: Mike Rapoport <rppt@kernel.org>
Cc: <linux-arm-kernel@lists.infradead.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	Anshuman Khandual <anshuman.khandual@arm.com>,
	Ard Biesheuvel <ardb@kernel.org>,
	Catalin Marinas <catalin.marinas@arm.com>,
	David Hildenbrand <david@redhat.com>,
	Marc Zyngier <maz@kernel.org>,
	"Mark Rutland" <mark.rutland@arm.com>,
	Mike Rapoport <rppt@linux.ibm.com>,
	"Will Deacon" <will@kernel.org>, <kvmarm@lists.cs.columbia.edu>,
	<linux-kernel@vger.kernel.org>, <linux-mm@kvack.org>
Subject: Re: arm32: panic in move_freepages (Was [PATCH v2 0/4] arm64: drop pfn_valid_within() and simplify pfn_valid())
Date: Thu, 29 Apr 2021 08:48:26 +0800	[thread overview]
Message-ID: <6ad2956c-70ae-c423-ed7d-88e94c88060f@huawei.com> (raw)
In-Reply-To: <YIj5zcbHBHt7CC8B@kernel.org>


On 2021/4/28 13:59, Mike Rapoport wrote:
> On Tue, Apr 27, 2021 at 07:08:59PM +0800, Kefeng Wang wrote:
>> On 2021/4/27 14:23, Mike Rapoport wrote:
>>> On Mon, Apr 26, 2021 at 11:26:38PM +0800, Kefeng Wang wrote:
>>>> On 2021/4/26 13:20, Mike Rapoport wrote:
>>>>> On Sun, Apr 25, 2021 at 03:51:56PM +0800, Kefeng Wang wrote:
>>>>>> On 2021/4/25 15:19, Mike Rapoport wrote:
>>>>>>
>>>>>>        On Fri, Apr 23, 2021 at 04:11:16PM +0800, Kefeng Wang wrote:
>>>>>>
>>>>>>            I tested this patchset(plus arm32 change, like arm64 does)
>>>>>>            based on lts 5.10,add some debug log, the useful info shows
>>>>>>            below, if we enable HOLES_IN_ZONE, no panic, any idea,
>>>>>>            thanks.
>>>>>>
>>>>>>        Are there any changes on top of 5.10 except for pfn_valid() patch?
>>>>>>        Do you see this panic on 5.10 without the changes?
>>>>>>
>>>>>> Yes, there are some BSP support for arm board based on 5.10,
>>> Is it possible to test 5.12?
> Do you use SPARSMEM? If yes, what is your section size?
> What is the value if CONFIG_FORCE_MAX_ZONEORDER in your configuration?

Yes,

CONFIG_SPARSEMEM=y

CONFIG_SPARSEMEM_STATIC=y

CONFIG_FORCE_MAX_ZONEORDER = 11

CONFIG_PAGE_OFFSET=0xC0000000
CONFIG_HAVE_ARCH_PFN_VALID=y
CONFIG_HIGHMEM=y
#define SECTION_SIZE_BITS    26
#define MAX_PHYSADDR_BITS    32
#define MAX_PHYSMEM_BITS     32


>

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2021-04-29  0:48 UTC|newest]

Thread overview: 143+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-21  6:51 [PATCH v2 0/4] arm64: drop pfn_valid_within() and simplify pfn_valid() Mike Rapoport
2021-04-21  6:51 ` Mike Rapoport
2021-04-21  6:51 ` Mike Rapoport
2021-04-21  6:51 ` [PATCH v2 1/4] include/linux/mmzone.h: add documentation for pfn_valid() Mike Rapoport
2021-04-21  6:51   ` Mike Rapoport
2021-04-21  6:51   ` Mike Rapoport
2021-04-21 10:49   ` Anshuman Khandual
2021-04-21 10:49     ` Anshuman Khandual
2021-04-21 10:49     ` Anshuman Khandual
2021-04-21  6:51 ` [PATCH v2 2/4] memblock: update initialization of reserved pages Mike Rapoport
2021-04-21  6:51   ` Mike Rapoport
2021-04-21  6:51   ` Mike Rapoport
2021-04-21  7:49   ` David Hildenbrand
2021-04-21  7:49     ` David Hildenbrand
2021-04-21  7:49     ` David Hildenbrand
2021-04-21 10:51   ` Anshuman Khandual
2021-04-21 10:51     ` Anshuman Khandual
2021-04-21 10:51     ` Anshuman Khandual
2021-04-21  6:51 ` [PATCH v2 3/4] arm64: decouple check whether pfn is in linear map from pfn_valid() Mike Rapoport
2021-04-21  6:51   ` Mike Rapoport
2021-04-21  6:51   ` Mike Rapoport
2021-04-21 10:59   ` Anshuman Khandual
2021-04-21 10:59     ` Anshuman Khandual
2021-04-21 10:59     ` Anshuman Khandual
2021-04-21 12:19     ` Mike Rapoport
2021-04-21 12:19       ` Mike Rapoport
2021-04-21 12:19       ` Mike Rapoport
2021-04-21 13:13       ` Anshuman Khandual
2021-04-21 13:13         ` Anshuman Khandual
2021-04-21 13:13         ` Anshuman Khandual
2021-04-21  6:51 ` [PATCH v2 4/4] arm64: drop pfn_valid_within() and simplify pfn_valid() Mike Rapoport
2021-04-21  6:51   ` Mike Rapoport
2021-04-21  6:51   ` Mike Rapoport
2021-04-21  7:49   ` David Hildenbrand
2021-04-21  7:49     ` David Hildenbrand
2021-04-21  7:49     ` David Hildenbrand
2021-04-21 11:06   ` Anshuman Khandual
2021-04-21 11:06     ` Anshuman Khandual
2021-04-21 11:06     ` Anshuman Khandual
2021-04-21 12:24     ` Mike Rapoport
2021-04-21 12:24       ` Mike Rapoport
2021-04-21 12:24       ` Mike Rapoport
2021-04-21 13:15       ` Anshuman Khandual
2021-04-21 13:15         ` Anshuman Khandual
2021-04-21 13:15         ` Anshuman Khandual
2021-04-22  7:00 ` [PATCH v2 0/4] " Kefeng Wang
2021-04-22  7:00   ` Kefeng Wang
2021-04-22  7:00   ` Kefeng Wang
2021-04-22  7:29   ` Mike Rapoport
2021-04-22  7:29     ` Mike Rapoport
2021-04-22  7:29     ` Mike Rapoport
2021-04-22 15:28     ` Kefeng Wang
2021-04-22 15:28       ` Kefeng Wang
2021-04-22 15:28       ` Kefeng Wang
2021-04-23  8:11       ` Kefeng Wang
2021-04-23  8:11         ` Kefeng Wang
2021-04-23  8:11         ` Kefeng Wang
2021-04-25  7:19         ` arm32: panic in move_freepages (Was [PATCH v2 0/4] arm64: drop pfn_valid_within() and simplify pfn_valid()) Mike Rapoport
2021-04-25  7:19           ` Mike Rapoport
2021-04-25  7:19           ` Mike Rapoport
2021-04-25  7:51           ` Kefeng Wang
2021-04-25  7:51             ` Kefeng Wang
2021-04-26  5:20             ` Mike Rapoport
2021-04-26  5:20               ` Mike Rapoport
2021-04-26  5:20               ` Mike Rapoport
2021-04-26 15:26               ` Kefeng Wang
2021-04-26 15:26                 ` Kefeng Wang
2021-04-26 15:26                 ` Kefeng Wang
2021-04-27  6:23                 ` Mike Rapoport
2021-04-27  6:23                   ` Mike Rapoport
2021-04-27  6:23                   ` Mike Rapoport
2021-04-27 11:08                   ` Kefeng Wang
2021-04-27 11:08                     ` Kefeng Wang
2021-04-27 11:08                     ` Kefeng Wang
2021-04-28  5:59                     ` Mike Rapoport
2021-04-28  5:59                       ` Mike Rapoport
2021-04-28  5:59                       ` Mike Rapoport
2021-04-29  0:48                       ` Kefeng Wang [this message]
2021-04-29  0:48                         ` Kefeng Wang
2021-04-29  0:48                         ` Kefeng Wang
2021-04-29  6:57                         ` Mike Rapoport
2021-04-29  6:57                           ` Mike Rapoport
2021-04-29  6:57                           ` Mike Rapoport
2021-04-29 10:22                           ` Kefeng Wang
2021-04-29 10:22                             ` Kefeng Wang
2021-04-29 10:22                             ` Kefeng Wang
2021-04-30  9:51                             ` Mike Rapoport
2021-04-30  9:51                               ` Mike Rapoport
2021-04-30  9:51                               ` Mike Rapoport
2021-04-30 11:24                               ` Kefeng Wang
2021-04-30 11:24                                 ` Kefeng Wang
2021-04-30 11:24                                 ` Kefeng Wang
2021-05-03  6:26                                 ` Mike Rapoport
2021-05-03  6:26                                   ` Mike Rapoport
2021-05-03  6:26                                   ` Mike Rapoport
2021-05-03  8:07                                   ` David Hildenbrand
2021-05-03  8:07                                     ` David Hildenbrand
2021-05-03  8:07                                     ` David Hildenbrand
2021-05-03  8:44                                     ` Mike Rapoport
2021-05-03  8:44                                       ` Mike Rapoport
2021-05-03  8:44                                       ` Mike Rapoport
2021-05-06 12:47                                       ` Kefeng Wang
2021-05-06 12:47                                         ` Kefeng Wang
2021-05-06 12:47                                         ` Kefeng Wang
2021-05-07  7:17                                         ` Kefeng Wang
2021-05-07  7:17                                           ` Kefeng Wang
2021-05-07  7:17                                           ` Kefeng Wang
2021-05-07 10:30                                           ` Mike Rapoport
2021-05-07 10:30                                             ` Mike Rapoport
2021-05-07 10:30                                             ` Mike Rapoport
2021-05-07 12:34                                             ` Kefeng Wang
2021-05-07 12:34                                               ` Kefeng Wang
2021-05-07 12:34                                               ` Kefeng Wang
2021-05-09  5:59                                               ` Mike Rapoport
2021-05-09  5:59                                                 ` Mike Rapoport
2021-05-09  5:59                                                 ` Mike Rapoport
2021-05-10  3:10                                                 ` Kefeng Wang
2021-05-10  3:10                                                   ` Kefeng Wang
2021-05-10  3:10                                                   ` Kefeng Wang
2021-05-11  8:48                                                   ` Mike Rapoport
2021-05-11  8:48                                                     ` Mike Rapoport
2021-05-11  8:48                                                     ` Mike Rapoport
2021-05-12  3:08                                                     ` Kefeng Wang
2021-05-12  3:08                                                       ` Kefeng Wang
2021-05-12  3:08                                                       ` Kefeng Wang
2021-05-12  8:26                                                       ` Mike Rapoport
2021-05-12  8:26                                                         ` Mike Rapoport
2021-05-12  8:26                                                         ` Mike Rapoport
2021-05-13  3:44                                                         ` Kefeng Wang
2021-05-13  3:44                                                           ` Kefeng Wang
2021-05-13  3:44                                                           ` Kefeng Wang
2021-05-13 10:55                                                           ` Mike Rapoport
2021-05-13 10:55                                                             ` Mike Rapoport
2021-05-13 10:55                                                             ` Mike Rapoport
2021-05-14  2:18                                                             ` Kefeng Wang
2021-05-14  2:18                                                               ` Kefeng Wang
2021-05-14  2:18                                                               ` Kefeng Wang
2021-05-12  3:50             ` Matthew Wilcox
2021-05-12  3:50               ` Matthew Wilcox
2021-05-12  3:50               ` Matthew Wilcox
2021-04-25  6:59       ` [PATCH v2 0/4] arm64: drop pfn_valid_within() and simplify pfn_valid() Mike Rapoport
2021-04-25  6:59         ` Mike Rapoport
2021-04-25  6:59         ` Mike Rapoport

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=6ad2956c-70ae-c423-ed7d-88e94c88060f@huawei.com \
    --to=wangkefeng.wang@huawei.com \
    --cc=akpm@linux-foundation.org \
    --cc=anshuman.khandual@arm.com \
    --cc=ardb@kernel.org \
    --cc=catalin.marinas@arm.com \
    --cc=david@redhat.com \
    --cc=kvmarm@lists.cs.columbia.edu \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mark.rutland@arm.com \
    --cc=maz@kernel.org \
    --cc=rppt@kernel.org \
    --cc=rppt@linux.ibm.com \
    --cc=will@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.