Page MenuHomeFreeBSD

Fix regression introduced on 272446r
ClosedPublic

Authored by araujo on Jan 26 2016, 8:20 AM.
Tags
None
Referenced Files
F132089680: D5076.id13485.diff
Mon, Oct 13, 2:09 PM
Unknown Object (File)
Sun, Oct 12, 5:24 AM
Unknown Object (File)
Tue, Sep 23, 6:35 AM
Unknown Object (File)
Sun, Sep 14, 11:34 PM
Unknown Object (File)
Aug 18 2025, 1:44 AM
Unknown Object (File)
Jul 17 2025, 6:32 PM
Unknown Object (File)
Jun 30 2025, 9:25 PM
Unknown Object (File)
Jun 28 2025, 8:49 AM
Subscribers

Details

Summary

(based on) PR: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=206478

lagg(4) supports the protocol none, where it disables any traffic without
disabling the lagg(4) interface itself.

Test Plan

ifconfig lagg0 create
ifconfig lagg0 laggproto none

Diff Detail

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

Event Timeline

araujo retitled this revision from to Fix regression introduced on 272446r.
araujo updated this object.
araujo edited the test plan for this revision. (Show Details)
araujo added reviewers: rodrigc, bapt.

Rework the patch, now the statement looks more clean and
easy to understand.

sys/net/if_lagg.c
1268 ↗(On Diff #12932)

why not a single level of indentation?

sys/net/if_lagg.c
1268 ↗(On Diff #12932)

Do you mean a single level of statement? If it is about the statement.

For two reasons:

  1. I would prefer have the statement for the LAGG_PROTO explicit, much clear to understand, what is the intention there.
  1. Theoretically only one statement might be faster, but in the piece of code, the only way to know that is checking what the compiler will build with this statement. However, as this statement is not implemented in lagg_ioctl(), and it is not a fast path. It is OK to choose have more clarity in the code.
sys/net/if_lagg.c
1268 ↗(On Diff #12932)

Just rephrase here:

  1. Theoretically only one statement might be faster, but in the piece of code, the only way to know that is checking what the compiler will build with this statement. However, as this statement IS implemented in lagg_ioctl(), and it is not a fast path. It is OK to choose have more clarity in the code.

I just fix a misspelling that might make my affirmation confused.

bapt edited edge metadata.
This revision is now accepted and ready to land.Feb 4 2016, 12:18 PM

Abandon this revision in favor of: https://reviews.freebsd.org/D5226
That patch seemed much more reasonable and I made the tests it gets close with the same fix for 10-Stable.

Best,

This revision was automatically updated to reflect the committed changes.