Page MenuHomeFreeBSD

Fix mlx5en(4) to properly call m_defrag.
ClosedPublic

Authored by mjoras on Aug 16 2017, 7:01 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 7, 11:32 PM
Unknown Object (File)
Jan 13 2024, 3:39 AM
Unknown Object (File)
Jan 6 2024, 3:54 PM
Unknown Object (File)
Dec 20 2023, 4:53 AM
Unknown Object (File)
Nov 25 2023, 2:55 PM
Unknown Object (File)
Nov 22 2023, 8:37 PM
Unknown Object (File)
Nov 15 2023, 9:46 PM
Unknown Object (File)
Nov 13 2023, 11:48 AM
Subscribers

Details

Summary

This is essentially the same change as was previously made to mlx4en(4).
By stripping the chain of "empty" mbufs, it was removing M_PKTHDR from
the chain. This stripping is not needed since BUSDMA(9) does not map
empty mbufs.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

sys/dev/mlx5/mlx5_en/mlx5_en_tx.c
334 ↗(On Diff #32134)

Can you add braces {} around this multi line statement?

sys/dev/mlx5/mlx5_en/mlx5_en_tx.c
334 ↗(On Diff #32134)

Sure, that's my preference as well but I wasn't so sure about the prevailing style for the driver.

This patch looks good to me. @gallatin do you want to test it aswell before it reaches -head? I'll give it a spin tomorrow.

Hans, did you ever get a chance to test this?

I'll have a look. I think we have a patch in the upstreaming list which covers this.

This revision was not accepted when it landed; it landed in state Needs Review.Mar 8 2018, 3:53 PM
This revision was automatically updated to reflect the committed changes.