On Fri, Sep 28, 2012 at 6:25 AM, ravi kerur <rke...@gmail.com> wrote: > On Thu, Sep 27, 2012 at 5:44 PM, Simon Horman <ho...@verge.net.au> wrote: >> On Thu, Sep 27, 2012 at 04:45:22PM -0700, Jesse Gross wrote: >> > On Thu, Sep 27, 2012 at 12:05 PM, ravi kerur <rke...@gmail.com> wrote: >> > > On Thu, Sep 27, 2012 at 11:16 AM, Jesse Gross <je...@nicira.com> >> > >> * If it is truly userspace only, the handling of multiple levels of >> > >> tags >> > >> seems a little incomplete since we actually have the full packet. >> > > >> > > >> > > Can you elaborate please? >> > >> > Multiple levels of tags aren't handled, for example, when you pop a tag. >> >> Do you think that adding support for multiple levels of tags to the kernel >> is appropriate. Or should such cases be bounced to user-space? > > > <rk> multiple tags do work in both userspace and kernel i.e. any combination > of push/pop actions. I am not sure what the confusion is? > > Simon: I have not looked into your patch, has anything changed here?
This patch is not the same as yours. Please actually read it before making assertions. >> > >> If this is supposed to be a quick stepping stone to kernel support >> > >> that seems less important since we will no longer have complete >> > >> packet >> > >> access. >> > >> >> > >> So it basically comes down to what the short term plans are. There's >> > >> also Leo's patch (which I haven't looked at) that I can post if there >> > >> are plans to do kernel support. >> > > >> > > >> > > <rk> so will there be separate/different kernel interface? >> > >> > Separate from what? > > > <rk> Our earlier disagreement was on packet recirculation for multiple > actions. You mention about Leo's patch and I assumed it contains packet > recirculation support and you want mpls changes for that new piece of kernel > datapath and wanted clarification on that. All I'm looking for is something either implements recirculation or at the very least doesn't preclude it from being done in the future. The patch that I was referring to takes the latter approach and implements just the basic operations in the kernel. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
