On Sep 13, 2012, at 2:11 PM, Ian Campbell wrote: > On Thu, 2012-09-13 at 18:28 +0100, Andres Lagar-Cavilla wrote: >> >> * Add placeholder in array of grant table error descriptions for >> unrelated error code we jump over. > > Why not just define it, it's listed here: > http://xenbits.xen.org/docs/unstable/hypercall/include,public,grant_table.h.html#Enum_grant_status Well, a) we'd be defining something no one will be using (for the moment) b) I would be signing-off on something unrelated. > > I notice the specific wording of the error msg is different here too. That should not be, I'll re-spin the patch.
Thanks Andres > It'd probably be best to use the same wording as the Xen definition, so > all OSes end up with the same name for the same condition (else bug > reports will be confusing). > > Ian. > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
