Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1376193

Re: [PATCH 1/2] arm64: mem-model: add flatmem model for arm64

Show all headers | View raw

On 04/11/2016 03:57 AM, Chen Feng wrote:
> Hi Will,
>
> On 2016/4/11 18:40, Will Deacon wrote:
>> On Mon, Apr 11, 2016 at 12:31:53PM +0200, Ard Biesheuvel wrote:
>>> On 11 April 2016 at 11:59, Chen Feng <puck.chen@hisilicon.com> wrote:
>>>> Please see the pg-tables below.
>>>>
>>>>
>>>> With sparse and vmemmap enable.
>>>>
>>>> ---[ vmemmap start ]---
>>>> 0xffffffbdc0200000-0xffffffbdc4800000          70M     RW NX SHD AF    UXN MEM/NORMAL
>>>> ---[ vmemmap end ]---
>>>>
>>>
>>> OK, I see what you mean now. Sorry for taking so long to catch up.
>>>
>>>> The board is 4GB, and the memap is 70MB
>>>> 1G memory --- 14MB mem_map array.
>>>
>>> No, this is incorrect. 1 GB corresponds with 16 MB worth of struct
>>> pages assuming sizeof(struct page) == 64
>>>
>>> So you are losing 6 MB to rounding here, which I agree is significant.
>>> I wonder if it makes sense to use a lower value for SECTION_SIZE_BITS
>>> on 4k pages kernels, but perhaps we're better off asking the opinion
>>> of the other cc'ees.
>>
>> You need to be really careful making SECTION_SIZE_BITS smaller because
>> it has a direct correlation on the use of page->flags and you can end up
>> running out of bits fairly easily.
>
> Yes, making SECTION_SIZE_BITS smaller can solve the current situation.
>
> But if the phys-addr is 64GB, but only 4GB ddr is the valid address. And the
>
> holes are not always 512MB.
>
> But, can you tell us why *smaller SIZE makes running out of bits fairly easily*?
>

Think about page tables and TLB pressure. A larger page size can cover the
same memory area with fewer page table entries. The same type of logic applies
to memory sections here as well. If the section size is smaller, you need
more bits to represent the number of sections used. page->flags is a long

In include/linux/mm.h

/* Page flags: | [SECTION] | [NODE] | ZONE | [LAST_CPUPID] | ... | FLAGS | */

and

#if SECTIONS_WIDTH+NODES_WIDTH+ZONES_WIDTH > BITS_PER_LONG - NR_PAGEFLAGS
#error SECTIONS_WIDTH+NODES_WIDTH+ZONES_WIDTH > BITS_PER_LONG - NR_PAGEFLAGS
#endif

So it's a trade off of what can be encoded in an unsigned long.

We're hitting the upper bound on zones as well (see 033fbae988fc 'mm:
ZONE_DEVICE for "device memory"')


> And how about the flat-mem model?
>
>>
>> Will
>>
>> .
>>
>

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 1/2] arm64: mem-model: add flatmem model for arm64 Chen Feng <puck.chen@hisilicon.com> - 2016-04-05 10:30 +0200
  Re: [PATCH 1/2] arm64: mem-model: add flatmem model for arm64 Chen Feng <puck.chen@hisilicon.com> - 2016-04-07 09:50 +0200
  Re: [PATCH 1/2] arm64: mem-model: add flatmem model for arm64 Will Deacon <will.deacon@arm.com> - 2016-04-07 16:30 +0200
    Re: [PATCH 1/2] arm64: mem-model: add flatmem model for arm64 Chen Feng <puck.chen@hisilicon.com> - 2016-04-11 05:00 +0200
      Re: [PATCH 1/2] arm64: mem-model: add flatmem model for arm64 Ard Biesheuvel <ard.biesheuvel@linaro.org> - 2016-04-11 09:40 +0200
        Re: [PATCH 1/2] arm64: mem-model: add flatmem model for arm64 Chen Feng <puck.chen@hisilicon.com> - 2016-04-11 10:00 +0200
          Re: [PATCH 1/2] arm64: mem-model: add flatmem model for arm64 Ard Biesheuvel <ard.biesheuvel@linaro.org> - 2016-04-11 10:10 +0200
            Re: [PATCH 1/2] arm64: mem-model: add flatmem model for arm64 Chen Feng <puck.chen@hisilicon.com> - 2016-04-11 12:10 +0200
              Re: [PATCH 1/2] arm64: mem-model: add flatmem model for arm64 Ard Biesheuvel <ard.biesheuvel@linaro.org> - 2016-04-11 12:40 +0200
                Re: [PATCH 1/2] arm64: mem-model: add flatmem model for arm64 Will Deacon <will.deacon@arm.com> - 2016-04-11 12:50 +0200
                Re: [PATCH 1/2] arm64: mem-model: add flatmem model for arm64 Chen Feng <puck.chen@hisilicon.com> - 2016-04-11 13:00 +0200
                Re: [PATCH 1/2] arm64: mem-model: add flatmem model for arm64 Laura Abbott <labbott@redhat.com> - 2016-04-11 20:20 +0200
                Re: [PATCH 1/2] arm64: mem-model: add flatmem model for arm64 Catalin Marinas <catalin.marinas@arm.com> - 2016-04-12 16:50 +0200
                Re: [PATCH 1/2] arm64: mem-model: add flatmem model for arm64 Catalin Marinas <catalin.marinas@arm.com> - 2016-04-12 17:00 +0200
              Re: [PATCH 1/2] arm64: mem-model: add flatmem model for arm64 Chen Feng <puck.chen@hisilicon.com> - 2016-04-11 12:50 +0200
                Re: [PATCH 1/2] arm64: mem-model: add flatmem model for arm64 Ard Biesheuvel <ard.biesheuvel@linaro.org> - 2016-04-11 13:10 +0200
      Re: [PATCH 1/2] arm64: mem-model: add flatmem model for arm64 Jungseok Lee <jungseoklee85@gmail.com> - 2016-04-12 16:10 +0200

csiph-web