Bug 1182107

Summary: assertion failure with very high MTU
Product: [Fedora] Fedora Reporter: Flavio Leitner <fleitner>
Component: openvswitchAssignee: Flavio Leitner <fleitner>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: high Docs Contact:
Priority: high    
Version: 22CC: chrisw, fleitner, tgraf
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 1182191 1182193 (view as bug list) Environment:
Last Closed: 2015-10-29 23:26:40 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Flavio Leitner 2015-01-14 13:01:07 UTC
Update OVS package to include the following two fixes:

commit 3282e5118707efc3ba07f7bd17f5f82fe931a732                                
Author: Ben Pfaff <blp>
Date:   Sun Jan 11 13:45:36 2015 -0800

    dpif-linux: Drop oversized packets instead of assert-failing.
    
    A packet sent to a Netlink datapath has to fit within a Netlink attribute.
    Until now, this was only checked in an assertion inside the Netlink code,
    which meant that trying to send a too-large packet (approximate 64 kB or
    larger) would assert-fail.  It's better to just drop those packets, which
    this commit does.
    
    Reported-by: Shuping Cui <scui>
    Reported-by: Jiri Benc <jbenc>
    Signed-off-by: Ben Pfaff <blp>
    Acked-by: Jesse Gross <jesse>

commit f3c7ec6a2a19bffdfa5218f984bda53582ecb8dc
Author: Ben Pfaff <blp>
Date:   Wed Jan 7 13:19:41 2015 -0800

    netlink: Refine calculation of maximum-length attributes.
    
    Until now the Netlink code has considered an attribute to exceed the
    maximum length if the *padded* size of the attribute exceeds 65535 bytes.
    For example, an attribute with a 65529-byte payload, together with 4-byte
    header and 3 bytes of padding, takes up 65536 bytes and therefore the
    existing code rejected it.
    
    However, the restriction on Netlink attribute sizes is to ensure that the
    length fits in the 16-bit nla_len field.  This field includes the 4-byte
    header but not the padding, so a 65529-byte payload is acceptable because,
    with the header but not the padding, it comes to only 65533 bytes.
    
    Thus, this commit relaxes the restriction on Netlink attribute sizes by
    omitting padding from size checks.  It also changes one piece of code that
    inlined a size check to use the central function nl_attr_oversized().
    
    This change should fix an assertion failure when OVS userspace passes a
    maximum-size (65529+ byte) packet back to the kernel.
    
    Reported-by: Shuping Cui <scui>
    Reported-by: Jiri Benc <jbenc>
    Signed-off-by: Ben Pfaff <blp>
    Acked-by: Jesse Gross <jesse>

Comment 1 Jaroslav Reznik 2015-03-03 17:06:54 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 22 development cycle.
Changing version to '22'.

More information and reason for this action is here:
https://fedoraproject.org/wiki/Fedora_Program_Management/HouseKeeping/Fedora22