I would also say something like: "Adds additional input parameter validation" > -----Original Message----- > From: Brown, Len > Sent: Tuesday, September 17, 2013 12:46 PM > To: Moore, Robert; Zheng, Lv > Cc: Wysocki, Rafael J; linux acpi > Subject: RE: [PATCH 06/14] ACPICA: Validate start object for > acpi_walk_namespace. > > > Just adds a sanity check. > > Ideal if commit messages answer the question: > > "Is this patch important?" > "If yes, what bad things happen if I don't have it in my kernel?" > > Remember, the commit messages have customers. > Nice to have a BZ referenced, but this BZ entry had no such information, > and "prevent problems later" didn't answer that key question either. > > thanks, > -Len -- 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