On Thu, 2007-08-16 at 09:26 -0700, Luck, Tony wrote: > > -#define ACPI_ID_LEN 9 > > +#define ACPI_ID_LEN 16 /* only 9 bytes needed here, 16 bytes are used */ > > What will happen if someone uses more than 9 bytes? With the old > limit there would be a compile time error it someone initialized > with: > > {"PNP0C0ABCDEFGH", 0}, > > But if we change ACPI_ID_LEN the error will move to run-time. That should not harm. >From spec point of view only 8 bytes are needed. Here 9 bytes are used as then string functions can be used. The whole rest of the kernel does not use ACPI_ID_LEN. If someone defines such an id, his driver won't ever get loaded and he gets bug reports very soon anyway. Hmm, I wonder whether this even may come in handy later: Some BIOSes have some spec violating strange hids like: "*PNP0C0A" or "_PNP0C0A" While I thought the "_" or "*" should be cut away by ACPI parser, it might be a hint to the OS to do something special. AFAIK only very specific devices (WMI and some graphics?) have such strange hids defined, those drivers could explicitly match for the spec violating string then. Thomas - To unsubscribe from this list: send the line "unsubscribe linux-ia64" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
![]() |