On Wed, Dec 22, 2010 at 08:42:39AM -0800, Narendra_K@xxxxxxxx wrote: > Hello, > > This patch exports ACPI _DSM provided firmware instance number and > string name to sysfs. > > Please review - There are now two different meanings for the 'index' file: 1) SMBIOS-provided "type instance" value, which I've only seen in range [1..N] for N devices, monotonically stepwise increasing. 2) ACPI-provided "index" value, which per spec only needs to be a "sort key", not starting at 0 or 1, and while monotonically increasing, not necessarily stepwise. It's perfectly valid for the values to be (12, 16, 27, 29) if that's convenient for BIOS to generate. Therefore, a consumer of this value (such as biosdevname) must know which of the two it's dealing with, and either accept the value as-is, or sort the value list. While I suppose it could sort the value list in either case, I'd prefer the ACPI value to be exposed in its own file, perhaps 'acpi_index', to make this explicit rather than implicit. 'label' is fine for either case, with ACPI taking priority over SMBIOS if both happen to be present. Thanks, Matt -- Matt Domsch Technology Strategist Dell | Office of the CTO -- To unsubscribe from this list: send the line "unsubscribe linux-hotplug" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
