Re: [2.6.6 PATCH] Exposing EFI memory map

[Greg KH <greg@kroah.com>]

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