On Thu, Jun 4, 2015 at 1:13 AM, David Miller <[email protected]> wrote: > From: Neil McKee <[email protected]> > Date: Wed, 3 Jun 2015 15:31:07 -0700 > >> Back out the OVS_USERSPACE_ATTR_ACTIONS attribute that was >> gating the inclusion of datapath actions in the upcall. >> Instead include the actions every time, provided there is no >> problem with the atomic allocation. If the atomic allocation >> fails then try it again with the actions omitted. >> >> Signed-off-by: Neil McKee <[email protected]> > > Conditionally providing the attributes to the user is completely > terrible semantics. > > If the user really needs these, then he won't be happy to get the > upcall without them, he'd rather the upcall fail completely > instead. > > I'm not applying this patch, this change was not well thought out > at all.
OK. We can use the OVS_USERSPACE_ATTR_ACTIONS mechanism that you checked in already. Thanks! -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
