Bug #4175
closed
BNX driver vlan/mtu fix
100%
Description
The bnx closed source driver doesn't allocate enough space for packets with vlan headers. The amount of space allocated depends on the mtu setting passed to the driver, plus normal ethernet headers. This causes the last 4 bytes to be cut off the packets if you attempt to use vlans, or you need to lower your mtu by 4 bytes, or switch to jumbo frames, and use normal 1500mtu.
This adjusts it to add 4 extra bytes to handle the vlan tag, without any of the above workarounds.
So far running this in production has not caused any issues.
edit closed/root_i386-nd/kernel/drv/bnx, exchange byte 0x1278 from 0x0e to 0x12
and, closed/root_i386-nd/kernel/drv/amd64/bnx, exchange byte 0x40b5 from 0x0e to 0x12Related issues
Updated by Alexandru Pirvulescu almost 8 years ago
Patrick Domack wrote:
edit closed/root_i386-nd/kernel/drv/bnx, exchange byte 0x1278 from 0x0e to 0x12
and, closed/root_i386-nd/kernel/drv/amd64/bnx, exchange byte 0x40b5 from 0x0e to 0x12
The current illumos-gate 64-bit driver version (6.0.3) has the offset to be patched at 0x4185 (thanks, Patrick!). i386 seems to be the same
Updated by Robert Mustacchi almost 5 years ago
This has been fixed with the integration of 8928. It no longer has the same problem and thankfully, editing hex will no longer be required.
Updated by Robert Mustacchi almost 5 years ago
- Status changed from New to Resolved
- Assignee set to Robert Mustacchi
- % Done changed from 0 to 100
- Tags deleted (
needs-triage)
Updated by Robert Mustacchi almost 5 years ago
- Has duplicate Feature #8928: Want open source bnx added