On Thu, Sep 8, 2011 at 4:37 PM, Ben Pfaff <b...@nicira.com> wrote:
> On Thu, Sep 08, 2011 at 03:26:04PM -0700, Jesse Gross wrote:
>> On Thu, Sep 8, 2011 at 11:36 AM, Ben Pfaff <b...@nicira.com> wrote:
>> > When ovs-vswitchd executes actions on a synthesized packet, that is, on a
>> > packet that is not being forwarded from any particular port but is being
>> > generated by ovs-vswitchd itself or by an OpenFlow controller (using a
>> > OFPT_PACKET_OUT message with an in_port of OFPP_NONE), there is no good
>> > choice for the in_port to pass to the kernel in the flow in the
>> > OVS_PACKET_CMD_EXECUTE message. ??This commit allows ovs-vswitchd to omit
>> > the in_port entirely in this case.
>> >
>> > This fixes a bug in OFPT_PACKET_OUT: using an in_port of OFPP_NONE would
>> > cause the packet to be dropped by the kernel, since that's an invalid
>> > input port.
>> >
>> > Signed-off-by: Ben Pfaff <b...@nicira.com>
>> > Reported-by: Aaron Rosen <aro...@clemson.edu>
>>
>> Acked-by: Jesse Gross <je...@nicira.com>
>
> Thanks.  I pushed this to master.  Here's the crossport to branch-1.2,
> I assume that it looks OK?  The only change was to s/OVS_/ODP_/ in
> random places.

This one looks good too.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to