On Tue, Feb 4, 2014 at 9:19 AM, Jarno Rajahalme <[email protected]> wrote: > > On Feb 3, 2014, at 9:56 AM, Jesse Gross <[email protected]> wrote: > >> On Fri, Jan 24, 2014 at 2:58 PM, Jarno Rajahalme <[email protected]> >> wrote: >>> Userspace already sets NLM_F_ECHO when it wants a reply, make use of >>> it. >>> >>> For set and del, OVS userspace only sets the NLM_F_ECHO if stats are >>> needed back. So, we minimize the data sent back by only responding >>> with key and stats for ovs_flow_cmd_new, ovs_flow_cmd_set, and >>> ovs_flow_cmd_del. >>> >>> When not returning actions, we can allocate the reply (if requested) >>> before finding the flow, i.e., before locking anything. We make use >>> of this in following patches. >>> >>> Signed-off-by: Jarno Rajahalme <[email protected]> >> >> I think this is too tailored to what userspace is currently expecting >> today. Currently, we send notifications of flow changes to a multicast >> group and any other subscribers should get notifications even if >> NLM_F_ECHO is not set. In addition, dropping certain fields depending >> on what we currently need is definitely API breakage and seems error >> prone in the future. >> > > Thanks, I wasn’t sure of any current use of the NLM_F_ECHO feature outside > of the OVS userspace (or in past OVS versions), and hence marked the series > as RFC. > > Is there a way to find out if there are any other subscribers around? I still > think > it would be better not allocate and fill the response if we could know no-one > is > interested in it.
I agree that it is wasteful to generate a response that no one will receive. In theory it should be possible to do this without changing any semantics although it would probably require adding some support to the netlink code to expose this information. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
