Em Mon, 10 Mar 2014 11:31:29 +0100 Borislav Petkov <bp@xxxxxxxxx> escreveu: > On Mon, Mar 10, 2014 at 07:04:35AM -0300, Mauro Carvalho Chehab wrote: > > Changing the format breaks any userspace application that relies on > > parsing them. That's an API breakage. Adding more data could be > > fine, if we take enough care when doing it, and properly document > > how userspace is supposed to parse it. > > Yes, we don't want code duplication if it can be helped. Besides, having > one function doing the error format issual keeps us from the case when > having two or more diverge from one another. > > > Well, actually, EDAC drivers use 0 to indicate an unknown physical address. > > The better is to use the same standard used there. > > However, physical address 0 is a valid address... all FFF...Fs is hardly valid. True, but if we decide to go on that direction, a change like that should then be done on all EDAC drivers, and that's an API change. We also need to be sure that userspace will handle this change properly. Regards, Mauro -- Regards, Mauro -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
