On Wed, 19 Sep 2018 11:25:17 +0200, Johannes Berg wrote: > Now, with this patch, all I'm doing is changing the internal behaviour > of nla_parse/nla_validate - externally, it still overwrites any existing > message if an error occurs, but internally it keeps the inner-most > error.
Ah, okay, that answers my question about putting the flag into the ext_ack struct, too. It may still be worthwhile to have a mechanism for prioritizing certain extack messages over another ones but it's clearly out of scope of this patchset. The patchset looks good to me. Just include the description you just wrote in the commit message :-) Thanks! Jiri