[ovs-dev] [PATCH 6/7] ofproto-dpif-upcall: Create ukey only for miss upcalls.
Ben Pfaff
blp at nicira.com
Tue Mar 3 18:22:49 UTC 2015
On Tue, Feb 24, 2015 at 03:42:05PM -0800, Jarno Rajahalme wrote:
> There is no point trying to create ukeys for non-miss upcalls, such as
> upcalls due to an explicit userspace actions.
>
> Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
At first glance this was a no-brainer. When I looked closer, it became
more subtle.
First, upcall_xlate() is only called for miss upcalls, as classified by
classify_upcall() into the MISS_UPCALL category. Most of those are in
fact DPIF_UC_MISS, but slow-pathed packets also get classified as
MISS_UPCALL. I believe that those are the only packets that this patch
affects. That's reasonable, since slow-pathed packets don't need ukeys,
but the commit message doesn't reflect the subtlety, and maybe a comment
on the new "if" statement should explain the situation also.
Acked-by: Ben Pfaff <blp at nicira.com>
More information about the dev
mailing list