From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with archive (Exim 4.43) id 1KU2sS-0002cM-VP for mharc-grub-devel@gnu.org; Fri, 15 Aug 2008 13:14:17 -0400 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1KU2sS-0002cH-3t for grub-devel@gnu.org; Fri, 15 Aug 2008 13:14:16 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1KU2sR-0002c4-Nl for grub-devel@gnu.org; Fri, 15 Aug 2008 13:14:15 -0400 Received: from [199.232.76.173] (port=40990 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1KU2sR-0002c1-BJ for grub-devel@gnu.org; Fri, 15 Aug 2008 13:14:15 -0400 Received: from mta-out.inet.fi ([195.156.147.13]:42071 helo=kirsi2.inet.fi) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1KU2sQ-00079C-Kx for grub-devel@gnu.org; Fri, 15 Aug 2008 13:14:15 -0400 Received: from [127.0.0.1] (88.193.32.97) by kirsi2.inet.fi (8.5.014) id 488DC54E00BFC4D5 for grub-devel@gnu.org; Fri, 15 Aug 2008 20:14:13 +0300 Message-ID: <48A5B96D.7040209@nic.fi> Date: Fri, 15 Aug 2008 20:14:21 +0300 From: =?ISO-8859-1?Q?Vesa_J=E4=E4skel=E4inen?= User-Agent: Thunderbird 2.0.0.16 (Windows/20080708) MIME-Version: 1.0 To: The development of GRUB 2 References: <1218684975.8757.139.camel@localhost> <48A4589D.3040902@nic.fi> <20080814180005.GB5614@thorin> <1218749362.19647.20.camel@localhost> <48A5AF50.2040906@nic.fi> <1218819798.2510.13.camel@localhost> In-Reply-To: <1218819798.2510.13.camel@localhost> X-Enigmail-Version: 0.95.7 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: Quoted-Printable X-detected-kernel: by monty-python.gnu.org: Linux 2.6 (newer, 3) Subject: Re: [RFC] Platform information services X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: The development of GRUB 2 List-Id: The development of GRUB 2 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 15 Aug 2008 17:14:16 -0000 Javier Mart=EDn wrote: > El vie, 15-08-2008 a las 19:31 +0300, Vesa J=E4=E4skel=E4inen escribi=F3= : >> Javier Mart=EDn wrote: >>> WRT "kernel and modules going hand by hand", think about external >>> modules: if the drivemap module is finally rejected for introduction = in >>> GRUB, I will not scrap it, but keep it as a module external to the >>> official GNU sources and possibly offer it in a web in the form of >>> patches to the official GRUB2. In this case, changes made to the kern= el >>> would not take into account that module, which would break if I weren= 't >>> monitoring this list daily. >> Then it is really your problem ;) > Indeed, but bitrot is not just the real of external modules: it's > happening right now even within the GRUB trunk as you admit in the > "Build problems on powerpc" thread... And? If Power PC maintainer is nowhere to update to newest additions then it is rightly in in-compilable state and if it rots too long its support will be removed. That's life. >>> Additionally, the cost of this function in platforms which don't have >>> any structs registered yet, as the function could be a stub like this= : >>> >>> void* grub_machine_get_platform_structure (int stidx) >>> { >>> grub_error (GRUB_ERR_BAD_ARGUMENT, "Struct %d not supported", stidx= ); >>> return 0; >>> } >>> >>> The kernel space taken would most likely be less than 50 bytes. For >>> i386-pc, it could be like this (also lightweight) function: >>> >>> void* grub_machine_get_platform_structure (int stidx) >>> { >>> grub_errno =3D GRUB_ERR_NONE; >>> >>> switch (stidx) >>> { >>> case GRUB_MACHINE_I386_IVT: >>> return /* Call to asm function that runs SIDT in real mode */ ; >>> case GRUB_MACHINE_I386_BDA: >>> return (void*)0x400; >>> default: >>> grub_error (GRUB_ERR_BAD_ARGUMENT, "Struct %d not supported", >>> stidx); >>> return 0; >>> } >>> } >> And what lets assume couple of extra platforms... how about >> x86-32bit-efi and ppc. What do they do? >> >> Implement their own enum entries for those indexes and only use their >> own indices...? > At first, they would just have the stub which does not recognize any > index, but yes, i386-efi devs could decide that certain > firmware-provided structure (like a video modes info table or such, I > don't know the internals of EFI) might be interesting to a module > they're creating, so they create an index for it and add it to the > version of the function in their platform. >=20 > If I had not mentioned it before, the function would be declared in a > cross-platform file, but _implemented_ in platform-specific files, and > the indices would be declared in the platform-specific machine.h. Thus= , > there would not be a "single" indices namespace: structure #1 might be > the IVT in i386-pc, but some devices info table in powerpc-ieee1275. >=20 >> Where here we are sharing any code? (if we do not count the name of th= e >> fuction.) Interface is kinda useless if there is no possibility that >> no-one is sharing its functionality... > The idea is a single function to retrieve the addresses of > firmware-provided/used structures. This includes the IVT and BDA in > i386-pc, but as I said before it could also be used by other platforms > for their own structures. The alternative would be just creating such > "get struct X" functions on each platform as they are needed, but I > imagined that a single interface (with such a low cost in space) would > be a more elegant solution. I still do not see the need to create cross platform function that cannot be used for cross platform purposes. It is much more reasonable to write such needs as in kernel for the platform or in platform specific modules. I do not have a problem with function to retrieve pointer to some platform specific function on platfrom specific code. That is normal life of the platform. And if we think about code safety, casting is bad. And if you keep indices colliding on different platforms then you are just calling for problems. It completely removes any advantage what this kinda wrapper could have had.