Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: Handle oversized actions more gracefully.

2013-11-02 Thread Ben Pfaff
On Fri, Nov 01, 2013 at 04:23:12PM -0700, Joe Stringer wrote: > This looks good in principle. > > Test 706 (ofproto-dpif - too many output actions) fails on this, as it > expects the old VLOG_ERR in the logs. Oops. Thanks for pointing that out. I fixed it. > There is also an error from ofproto

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: Handle oversized actions more gracefully.

2013-11-01 Thread Joe Stringer
This looks good in principle. Test 706 (ofproto-dpif - too many output actions) fails on this, as it expects the old VLOG_ERR in the logs. There is also an error from ofproto_dpif_xlate in this case, complaining about resubmits yielding >64kB of actions. I assume this is intended to remain as a h

[ovs-dev] [PATCH] ofproto-dpif-xlate: Handle oversized actions more gracefully.

2013-10-09 Thread Ben Pfaff
If the datapath actions exceed the maximum size of a Netlink attribute (about 64 kB), then previously we would assert-fail (before commit 542024c4c3d36 "ofproto-dpif-xlate: Suppress oversize datapath actions.") or just drop all of them (after that commit). This commit makes OVS cope by slow-pathin