[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: How to detect the amount of RAM on non-EFI platforms?
From: |
Daniel Kiper |
Subject: |
Re: How to detect the amount of RAM on non-EFI platforms? |
Date: |
Thu, 13 Jul 2023 20:38:45 +0200 |
User-agent: |
NeoMutt/20170113 (1.7.2) |
On Tue, Jul 11, 2023 at 09:37:15PM +0800, Cyan Yang via Grub-devel wrote:
> Hi again,
>
> On 2023/7/11 10:08, Cyan via Grub-devel wrote:
>
> > static int
> > traverse_mmap_hook (grub_uint64_t addr, grub_uint64_t size,
> > grub_memory_type_t type,
> > void *data __attribute__ ((unused)))
> > {
> > total_mem_bytes += size;
> > }
>
> I discovered a huge problem in my hook function: missing a return at the end
> of the hook function.
> Without this return the hook function will execute only once.
> And, regions with type GRUB_MEMORY_RESERVED should be skipped. So the hook
> function body should look like this:
>
> if (type != GRUB_MEMORY_RESERVED) {
> total_mem_bytes += size;
> }
> return GRUB_ERR_NONE;
You are mixing enum and int types. The function should return 0 here.
Anyway, I think it would be nice if you could upstream this module.
Potentially other people may find it useful.
Daniel