Sorry, that was an honest mistake. Let's say I want to tweak push_vlan to
add a new tag, even if theres an existing one-
I saw that there's a check to see if there's a vlan header on the frame and
if so, it 'pushes down' the current header, creating space for a new one.
If I drop it, will a new header get created?

I'm trying to figure out which code I need to tweak in order to support,
even partially, two vlan headers. (I only want to access the outer one,
never the inner).

After that I can start working on a full implementation. Any thoughts?

Thanks!

On Wednesday, January 1, 2014, Jesse Gross wrote:

> Please don't drop the mailing list.
>
> You are correct that's it's not currently possible to do this. All
> that I'm saying is that there is no existing working patch.
>
> On Wed, Jan 1, 2014 at 10:58 AM, Oded Lazar <oded...@gmail.com<javascript:;>>
> wrote:
> > Oh. Umm... The thing is that I have a frame that has a vlan header and I
> > need to add another one on top. Current ovs implementations don't allow
> such
> > thing, right? (ovs-ofctl push_vlan keeps changing my vlan header instead
> of
> > adding a new one)
> >
> > That's why I wanted to get into implementing QinQ.
> >
> >
> > On Wednesday, January 1, 2014, Jesse Gross wrote:
> >>
> >> On Wed, Jan 1, 2014 at 2:30 AM, Oded Lazar 
> >> <oded...@gmail.com<javascript:;>>
> wrote:
> >> > Hey guys,
> >> > I'm Looking into implementing QinQ in OVS and I read in the mailing
> >> > lists
> >> > that a patch has already been created and Ravi/Jesse stopped working
> on
> >> > it
> >> > about a year ago. I want to port it to OVS 2.x and hopefully commit
> it.
> >>
> >> I doubt there is any benefit in starting from that code. It was never
> >> reviewed at all, it depends on code that was never committed, and many
> >> other things have changed in the mean time.
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to