| Summary: | Mangled IPv6 NAs with VLAN + bonding + non-VLAN-offload hardware | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Red Hat Enterprise Linux 5 | Reporter: | Ben Hutchings <bhutchings> | ||||
| Component: | kernel | Assignee: | Veaceslav Falico <vfalico> | ||||
| Status: | CLOSED WONTFIX | QA Contact: | Network QE <network-qe> | ||||
| Severity: | medium | Docs Contact: | |||||
| Priority: | high | ||||||
| Version: | 5.4 | CC: | ccui, haliu, hjia, jarod, jfeeney, jpirko, kzhang, peterm, qcai, sghosh, vfalico | ||||
| Target Milestone: | rc | ||||||
| Target Release: | --- | ||||||
| Hardware: | All | ||||||
| OS: | Linux | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | Doc Type: | Bug Fix | |||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2013-08-22 10:47:21 UTC | Type: | --- | ||||
| Regression: | --- | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Bug Depends On: | 809207 | ||||||
| Bug Blocks: | |||||||
| Attachments: |
|
||||||
|
Description
Ben Hutchings
2011-02-11 22:08:24 UTC
Created attachment 478321 [details]
bonding/vlan: Avoid mangled NAs on slaves without VLAN tag insertion
Thanks, Ben. I will get this queued up for the next update. This request was evaluated by Red Hat Product Management for inclusion in a Red Hat Enterprise Linux maintenance release. Product Management has requested further review of this request by Red Hat Engineering, for potential inclusion in a Red Hat Enterprise Linux Update release for currently deployed products. This request is not yet committed for inclusion in an Update release. I've been looking at Ben's patch more closely and I think it looks good. Thanks again, Ben.
The first hunk took me a bit longer to understand, but I realized that RHEL5 did not have the 'software emulation for vlan acceleration' patch[0] from 2.6.37, so it is necessary to drop the all vlan-tagged tagged NA when using hardware without support for TX VLAN acceleration.
The only change I would consider making to this patch is a warning that the hardware is not recommended with bonding since it doesn't support vlan TX acceleration and ipv6. It might be nice for users to know why the NA are not coming out of the box.
0. commit 7b9c60903714bf0a19d746b228864bad3497284e
Author: Jesse Gross <jesse>
Date: Wed Oct 20 13:56:04 2010 +0000
vlan: Enable software emulation for vlan accleration.
This also appears to affect RHEL 6 as of kernel version 2.6.32-122.el6 (the version in 6.1 beta). (In reply to comment #6) > This also appears to affect RHEL 6 as of kernel version 2.6.32-122.el6 (the > version in 6.1 beta). I suspect that is the case based on the vintage of the patches that appear to resolve this. This request was evaluated by Red Hat Product Management for inclusion in a Red Hat Enterprise Linux maintenance release. Product Management has requested further review of this request by Red Hat Engineering, for potential inclusion in a Red Hat Enterprise Linux Update release for currently deployed products. This request is not yet committed for inclusion in an Update release. patch in comment#12 has been reviewed and deem to cause feature regression hence defer this issue to 5.9: http://post-office.corp.redhat.com/archives/rhkernel-list/2011-September/msg00544.html I can't see comment 12. If you're concerned about the fact that I proposed to disable unsolicited NAs for devices without VLAN tag acceleration, this is not a feature regression since the feature never worked. (In reply to comment #15) > I can't see comment 12. If you're concerned about the fact that I proposed to > disable unsolicited NAs for devices without VLAN tag acceleration, this is not > a feature regression since the feature never worked. Although I agree with you this is not regression, there was a pushback against your patch on our mailing list. DaveM's comment: "Handling the packet properly, in software, is the only proper way to resolve this." Not sure how exactly to do that nicely. Hi Ben, I'm really sorry to say this, but I'm closing this bug without getting your patch included :(. It's based on comment c#16 and the fact that 5.11 is a release for small fixes, and probably won't get any big patches sucked in, especially with these kind of workarounds :(. If you *really* need it, I can try to push it, however the chances are really low... Upgrading (and fixing, if needed) RHEL6 would be the best approach for this... Thanks for understanding and sorry again. :-/ |