On Wed, Mar 20, 2013 at 9:22 AM, David Miller <[email protected]> wrote: > From: Jesse Gross <[email protected]> > Date: Wed, 20 Mar 2013 08:57:53 -0700 > >> On Wed, Mar 20, 2013 at 5:14 AM, Wei Yongjun <[email protected]> wrote: >>> From: Wei Yongjun <[email protected]> >>> >>> Fix to return a negative error code from the error handling >>> case instead of 0, as returned elsewhere in this function. >>> >>> Signed-off-by: Wei Yongjun <[email protected]> >> >> This isn't right because at this point in the function the change has >> already been made, only the notification allocation failed. Therefore >> we don't want to return an error since the caller will assume that >> nothing has changed; instead an error is set on the Netlink socket. > > Then the way to fix this is to allocate the netlink SKB first, before > any config changes occur, then pass that SKB into ovs_vport_cmd_build_info. > > We can still get errors, for -EMSGSIZE situations, but I'd say that'd > be an implementation bug that should be logged. If we aren't allocating > large enough SKBs for the netlink reply, that's really an internal > error.
That solves the problem here. I'll take care of it, thanks. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
