On Fri, May 22, 2015 at 10:53 AM, Pravin Shelar <pshe...@nicira.com> wrote:
> On Wed, May 20, 2015 at 12:32 PM, Neil McKee <neil.mc...@inmon.com> wrote:
>> diff --git a/net/openvswitch/actions.c b/net/openvswitch/actions.c
>> index b491c1c..ee5760d 100644
>> --- a/net/openvswitch/actions.c
>> +++ b/net/openvswitch/actions.c
>> @@ -608,7 +608,8 @@ static void do_output(struct datapath *dp, struct
>> sk_buff *skb, int out_port)
>> }
>>
>> static int output_userspace(struct datapath *dp, struct sk_buff *skb,
>> - struct sw_flow_key *key, const struct nlattr
>> *attr)
>> + struct sw_flow_key *key, const struct nlattr
>> *attr,
>> + const struct nlattr *actions, int actions_len)
>> {
>> struct ovs_tunnel_info info;
>> struct dp_upcall_info upcall;
>> @@ -619,6 +620,8 @@ static int output_userspace(struct datapath *dp, struct
>> sk_buff *skb,
>> upcall.userdata = NULL;
>> upcall.portid = 0;
>> upcall.egress_tun_info = NULL;
>> + upcall.actions = actions;
>> + upcall.actions_len = actions_len;
>>
> Rather than unconditionally passing actions to the upcall, there
> should be attribute in ovs_userspace_attr to request the actions list.
Why? It seems simpler to just always pass the actions and I'm not sure
that this is really performance critical (which is the only reason
that comes to mind to not always pass this).
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
