Re: [2.6.6 PATCH] Exposing EFI memory map

Greg KH
2004-05-07T21:49:03Z

On Fri, May 07, 2004 at 03:15:30PM +0530, Sourav Sen wrote:
> Here we have "Array of values of same types". And it does not do much
> nifty formatting either. Is that not acceptable?

But they are not of the same time, as you admitted :)

Also, they could be bigger than a single page, right? That is not
possible right now in sysfs.

> If that is not, how about the following.
>
> 1. Create a directory "memmap" under firmware/efi/
> 2. Create files "map_start", "map_size" and "mapdesc_size" under
> that exposing ia64_boot_param->efi_memmap,
> ia64_boot_param->efi_memmap_size
> and ia64_boot_param->efi_memdesc_size respectively.
>
> Userland can make meaning out of them by knowing the values and knowing
> about "efi_memory_desc_t" which is already there in
> /usr/include/asm/efi.h

I think you should address the other issues in this thread before
worrying about the sysfs interface (why have this at all, incorrect
data for hotplug mem, etc.)

> PS: BTW, Your slides on "Dealing with the Linux Kernel Community" at:
> http://www.kroah.com/linux/talks/cgl_talk_2002_10_16/mgp00001.html
> is kind of useful to me. This is my first time :-)

If only more people/companies would heed those rules...

thanks,

greg k-h