On Wed, 18 Jun, at 06:48:35PM, Daniel Kiper wrote: > > > > Why don't you want to export efi.fw_vendor, etc? Rationale please. > > I am exporting real addresses (machine addresses) of things which > I am able to get. Stuff which was created artificially and lives > in dom0 address space or does not exist are not exported. So, wouldn't it be easier to just leave those fields as EFI_INVALID_TABLE_ADDR? If they're not usable why fill out an address? > Hmmm... I do not know what is wrong with this minimal shuffling. We are > playing here with internal stuff which is not visible outside of any > given kernel. Additionally, as I saw in a few places arch bits are > defined in following way: > > #define ARCH_1 10 > > #define A_ARCH_CONST ARCH_1 > #define B_ARCH_CONST (ARCH_1 + 1) > #define C_ARCH_CONST (ARCH_1 + 2) > ... > > So I think addition is more natural here than subtraction. No, because we hit the same problem if we need more non-arch bits after bit 9, it moves the problem, it doesn't fix it. Though admittedly, using this level of indirection (going through ARCH_1) instead of using constants does reduce the problem. Yes, these bits are internal, and yes the shuffling is minimal, but we can do better. I'm not suggesting you need to modify your patch. I'm really just thinking out loud. I'll take care of fixing this up later. -- Matt Fleming, Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-efi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
