Page MenuHomeFreeBSD

In m_clrprotoflags() cleanup the entire mbuf chain.
ClosedPublic

Authored by glebius on Dec 22 2014, 3:50 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 28 2024, 8:19 AM
Unknown Object (File)
Dec 19 2023, 9:41 PM
Unknown Object (File)
Apr 7 2017, 8:13 PM
Unknown Object (File)
Apr 1 2017, 6:18 PM
Unknown Object (File)
Dec 31 2016, 6:28 PM
Unknown Object (File)
Sep 2 2016, 2:17 PM
Unknown Object (File)
Jul 19 2016, 10:48 PM
Unknown Object (File)
Jun 28 2016, 12:02 AM
Subscribers

Details

Summary

The function is designed to cleanup M_PROTO flags when mbufs cross
protocol and facility boundaries. However, now it cleans up only the
first mbuf, while intent clearly means all mbufs in a possible chain.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

glebius retitled this revision from to In m_clrprotoflags() cleanup the entire mbuf chain..
glebius updated this object.
glebius added a reviewer: network.
rpaulo added a reviewer: rpaulo.
rpaulo added a subscriber: rpaulo.

In the commit log, I wouldn't use the pronoun "the" but instead you could make it clear you're talking about m_clrprotoflags().

This revision is now accepted and ready to land.Dec 22 2014, 6:11 PM
gnn added a reviewer: gnn.
gnn added a subscriber: gnn.

Looks good to me.

adrian added a reviewer: adrian.
adrian added a subscriber: adrian.

Yup, looks good to me too.

glebius updated this revision to Diff 2827.

Closed by commit rS276136 (authored by @glebius).