Re: [ovs-dev] [PATCH 1/3] datapath: Add genl_exec().

2011-12-28 Thread Jesse Gross
On Tue, Dec 27, 2011 at 8:17 PM, Pravin Shelar wrote: > On Thu, Dec 22, 2011 at 5:01 PM, Jesse Gross wrote: >> On Dec 21, 2011, at 8:28 PM, Pravin B Shelar wrote: >>> +     skb_get(genlmsg_skb); >>> + >>> +     nlh = genlmsg_put(genlmsg_skb, 0, 0, &genl_exec_family, >>> NLM_F_REQUEST, GENL_EXEC_

Re: [ovs-dev] [PATCH 1/3] datapath: Add genl_exec().

2011-12-27 Thread Pravin Shelar
On Thu, Dec 22, 2011 at 5:01 PM, Jesse Gross wrote: > On Dec 21, 2011, at 8:28 PM, Pravin B Shelar wrote: >> diff --git a/datapath/linux/compat/genetlink.inc >> b/datapath/linux/compat/genetlink.inc >> index bf96980..1301006 100644 >> --- a/datapath/linux/compat/genetlink.inc >> +++ b/datapath/li

Re: [ovs-dev] [PATCH 1/3] datapath: Add genl_exec().

2011-12-22 Thread Jesse Gross
On Dec 21, 2011, at 8:28 PM, Pravin B Shelar wrote: > diff --git a/datapath/linux/compat/genetlink.inc > b/datapath/linux/compat/genetlink.inc > index bf96980..1301006 100644 > --- a/datapath/linux/compat/genetlink.inc > +++ b/datapath/linux/compat/genetlink.inc > +static struct sk_buff *genlmsg_s

[ovs-dev] [PATCH 1/3] datapath: Add genl_exec().

2011-12-21 Thread Pravin B Shelar
Resending this patch as net-namespace patches depends on it. --8<--cut here-->8- genl_lock is not exported from older kernel. Following patch add genl_exec() which can run any function (passed as arg) with genl_lock held. Signed-off-by: Pravin B Sh

Re: [ovs-dev] [PATCH 1/3] datapath: Add genl_exec().

2011-12-16 Thread Pravin Shelar
On Thu, Dec 15, 2011 at 6:03 PM, Jesse Gross wrote: > On Wed, Dec 14, 2011 at 6:09 PM, Pravin B Shelar wrote: >> diff --git a/datapath/linux/compat/genetlink.inc >> b/datapath/linux/compat/genetlink.inc >> index bf96980..43c3227 100644 >> --- a/datapath/linux/compat/genetlink.inc >> +++ b/datapa

Re: [ovs-dev] [PATCH 1/3] datapath: Add genl_exec().

2011-12-15 Thread Jesse Gross
On Wed, Dec 14, 2011 at 6:09 PM, Pravin B Shelar wrote: > diff --git a/datapath/linux/compat/genetlink.inc > b/datapath/linux/compat/genetlink.inc > index bf96980..43c3227 100644 > --- a/datapath/linux/compat/genetlink.inc > +++ b/datapath/linux/compat/genetlink.inc > +static int genl_exec_cmd(st

[ovs-dev] [PATCH 1/3] datapath: Add genl_exec().

2011-12-14 Thread Pravin B Shelar
Fixed according to comments from Ansis. --8<--cut here-->8-- genl_lock is not exported from older kernel. Following patch add genl_exec() which can run any function (passed as arg) with genl_mutex held. Signed-off-by: Pravin B Shelar --- datapath

Re: [ovs-dev] [PATCH 1/3] datapath: Add genl_exec().

2011-12-14 Thread Pravin Shelar
On Wed, Dec 14, 2011 at 5:16 PM, Jesse Gross wrote: > On Wed, Dec 14, 2011 at 4:48 PM, Pravin Shelar wrote: >> On Wed, Dec 14, 2011 at 4:37 PM, Jesse Gross wrote: >>> On Wed, Dec 14, 2011 at 3:47 PM, Pravin Shelar wrote: On Wed, Dec 14, 2011 at 3:34 PM, Jesse Gross wrote: > On Wed, De

Re: [ovs-dev] [PATCH 1/3] datapath: Add genl_exec().

2011-12-14 Thread Jesse Gross
On Wed, Dec 14, 2011 at 4:48 PM, Pravin Shelar wrote: > On Wed, Dec 14, 2011 at 4:37 PM, Jesse Gross wrote: >> On Wed, Dec 14, 2011 at 3:47 PM, Pravin Shelar wrote: >>> On Wed, Dec 14, 2011 at 3:34 PM, Jesse Gross wrote: On Wed, Dec 14, 2011 at 3:07 PM, Pravin Shelar wrote: > On Wed,

Re: [ovs-dev] [PATCH 1/3] datapath: Add genl_exec().

2011-12-14 Thread Pravin Shelar
On Wed, Dec 14, 2011 at 4:37 PM, Jesse Gross wrote: > On Wed, Dec 14, 2011 at 3:47 PM, Pravin Shelar wrote: >> On Wed, Dec 14, 2011 at 3:34 PM, Jesse Gross wrote: >>> On Wed, Dec 14, 2011 at 3:07 PM, Pravin Shelar wrote: On Wed, Dec 14, 2011 at 2:47 PM, Ansis Atteka wrote: > On Wed, D

Re: [ovs-dev] [PATCH 1/3] datapath: Add genl_exec().

2011-12-14 Thread Jesse Gross
On Wed, Dec 14, 2011 at 3:47 PM, Pravin Shelar wrote: > On Wed, Dec 14, 2011 at 3:34 PM, Jesse Gross wrote: >> On Wed, Dec 14, 2011 at 3:07 PM, Pravin Shelar wrote: >>> On Wed, Dec 14, 2011 at 2:47 PM, Ansis Atteka wrote: On Wed, Dec 14, 2011 at 2:08 PM, Pravin B Shelar wrote: >

Re: [ovs-dev] [PATCH 1/3] datapath: Add genl_exec().

2011-12-14 Thread Pravin Shelar
On Wed, Dec 14, 2011 at 3:34 PM, Jesse Gross wrote: > On Wed, Dec 14, 2011 at 3:07 PM, Pravin Shelar wrote: >> On Wed, Dec 14, 2011 at 2:47 PM, Ansis Atteka wrote: >>> On Wed, Dec 14, 2011 at 2:08 PM, Pravin B Shelar wrote: +int genl_exec(genl_exec_func_t func, void *data) +{ +  

Re: [ovs-dev] [PATCH 1/3] datapath: Add genl_exec().

2011-12-14 Thread Jesse Gross
On Wed, Dec 14, 2011 at 3:07 PM, Pravin Shelar wrote: > On Wed, Dec 14, 2011 at 2:47 PM, Ansis Atteka wrote: >> On Wed, Dec 14, 2011 at 2:08 PM, Pravin B Shelar wrote: >>> +int genl_exec(genl_exec_func_t func, void *data) >>> +{ >>> +       struct sk_buff *skb; >>> + >>> +       skb = genlmsg_ne

Re: [ovs-dev] [PATCH 1/3] datapath: Add genl_exec().

2011-12-14 Thread Pravin Shelar
On Wed, Dec 14, 2011 at 2:47 PM, Ansis Atteka wrote: > > On Wed, Dec 14, 2011 at 2:08 PM, Pravin B Shelar wrote: >> >> genl_lock is not exported from older kernel. Following patch add >> genl_exec() which can run any function (passed as arg) with >> genl_lock held. >> >> Signed-off-by: Pravin B S

[ovs-dev] [PATCH 1/3] datapath: Add genl_exec().

2011-12-14 Thread Pravin B Shelar
genl_lock is not exported from older kernel. Following patch add genl_exec() which can run any function (passed as arg) with genl_lock held. Signed-off-by: Pravin B Shelar --- datapath/datapath.c |9 ++- datapath/linux/compat/genetlink.inc | 102 +