> On Mar 12, 2015, at 5:42 PM, Sorin Vinturis > <svintu...@cloudbasesolutions.com> wrote: > > Modified 'nl_sock_transact_multiple__' function to send asynchronous > I/O requests to the kernel driver. > > Signed-off-by: Sorin Vinturis <svintu...@cloudbasesolutions.com> > Reported-by: Alin Gabriel Serdean <aserd...@cloudbasesolutions.com> > Reported-at: > https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_openvswitch_ovs-2Dissues_issues_64&d=AwIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=pNHQcdr7B40b4h6Yb7FIedI1dnBsxdDuTLBYD3JqV80&m=ptyT-4_KX9K220F3nw2WYFTDxbFegQddmHarVmB6PKc&s=__1QDaT8DB13yENZfteePG67ZSDCflB3F6hDW29RCjY&e= > > --- > lib/netlink-socket.c | 26 ++++++++++++++++++++++---- > 1 file changed, 22 insertions(+), 4 deletions(-) > > diff --git a/lib/netlink-socket.c b/lib/netlink-socket.c > index e4f153f..64f93ff 100644 > --- a/lib/netlink-socket.c > +++ b/lib/netlink-socket.c > @@ -832,10 +832,28 @@ nl_sock_transact_multiple__(struct nl_sock *sock, > ofpbuf_data(txn->request), > ofpbuf_size(txn->request), > reply_buf, sizeof reply_buf, > - &reply_len, NULL)) {
Sorin, I understand that there was discussion that this patch is not needed. Without this patch, what is the expectation when STATUS_PENDING is returned from the kernel for a OVS_VPORT_CMD_NEW/OVS_VPORT_CMD_DEL request? From what I can see, the return value would be 0, and we’d end up returning EINVAL to the caller. Is that the desired behavior? -- Nithin _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
