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, Oct 16, 6:36 PM
Unknown Object (File)
Thu, Oct 16, 8:17 AM
Unknown Object (File)
Thu, Oct 16, 8:17 AM
Unknown Object (File)
Thu, Oct 16, 8:17 AM
Unknown Object (File)
Wed, Oct 15, 8:42 PM
Unknown Object (File)
Mon, Oct 13, 11:51 PM
Unknown Object (File)
Sat, Oct 11, 9:00 PM
Unknown Object (File)
Sat, Oct 11, 4:26 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.