Re: [ovs-dev] [PATCH] upcall: Remove redundant xlate_actions_for_side_effects().

2014-05-22 Thread Murphy McCauley
ns() called on it at least once. Therefore, there's no >> need to re-xlate slow path flows just to shove their packets through >> the system. >> >> This also may fix a bug discussed here: >> http://openvswitch.org/pipermail/discuss/2014-April/013670.html >>

Re: [ovs-dev] [PATCH] lib/netdev-linux.c: Prevent receiving of sent packets

2013-06-13 Thread Murphy McCauley
On Jun 13, 2013, at 2:42 PM, Ben Pfaff wrote: > On Thu, Jun 13, 2013 at 04:41:37PM +0900, Simon Horman wrote: >> On Wed, Jun 12, 2013 at 07:30:44PM -0700, Murphy McCauley wrote: >>> Commit 796223f5 (netdev: Add new "struct netdev_rx" for capturing packets >>&g

Re: [ovs-dev] Possible Regression in 'netdev: Add new "struct netdev_rx" for capturing packets from a netdev.'

2013-06-12 Thread Murphy McCauley
On Jun 12, 2013, at 5:34 PM, Simon Horman wrote: > On Wed, Jun 12, 2013 at 08:08:50AM -0700, Ben Pfaff wrote: >> On Wed, Jun 12, 2013 at 07:11:50AM -0700, Murphy McCauley wrote: >>> >>> On Jun 12, 2013, at 6:28 AM, Ed Maste wrote: >>> >>>&g

[ovs-dev] [PATCH] lib/netdev-linux.c: Prevent receiving of sent packets

2013-06-12 Thread Murphy McCauley
mon Horman co-discovered and initially reported this issue. Signed-off-by: Murphy McCauley --- Discussed on ovs-dev under the subject "Possible Regression in 'netdev: Add new "struct netdev_rx" for capturing packets from a netdev.'" lib/netdev-linux.c | 18 +

Re: [ovs-dev] Possible Regression in 'netdev: Add new "struct netdev_rx" for capturing packets from a netdev.'

2013-06-12 Thread Murphy McCauley
On Jun 12, 2013, at 6:28 AM, Ed Maste wrote: > On 12 June 2013 07:04, Murphy McCauley wrote: >> (Sorry this isn't an actual reply and is missing context -- I wasn't on the >> list when it was originally posted.) >> >> Simon and I have been in touch a

Re: [ovs-dev] Possible Regression in 'netdev: Add new "struct netdev_rx" for capturing packets from a netdev.'

2013-06-12 Thread Murphy McCauley
(Sorry this isn't an actual reply and is missing context -- I wasn't on the list when it was originally posted.) Simon and I have been in touch about this, and I thought I'd share my findings for what they're worth. The problem is from the commit Simon mentioned (796223f5bc3a4896e6398733c79839