Page MenuHomeFreeBSD
Differential D6440

Update de_DE/books/handbook/linuxemu/chapter.xml to r48090
ClosedPublic
Actions

Authored by bhd on May 18 2016, 5:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 8, 5:07 AM
Unknown Object (File)
Wed, Mar 5, 9:36 PM
Unknown Object (File)
Feb 25 2025, 10:07 PM
Unknown Object (File)
Feb 24 2025, 1:18 AM
Unknown Object (File)
Feb 20 2025, 12:32 PM
Unknown Object (File)
Feb 7 2025, 12:04 PM
Unknown Object (File)
Feb 7 2025, 7:19 AM
Unknown Object (File)
Nov 13 2024, 11:50 AM
View All 51 Files
Subscribers
None

Details

Reviewers
bcr
Commits
rD48830: Update to r48090:
Summary

Dieser Patch basiert nicht auf einzelnen Revisionen, sondern er bringt das Kapiel in einem Rutsch auf den aktuellen Stand. Ich konnte nicht widerstehen... :)

Diff Detail

Repository
rD FreeBSD doc repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 3813
Build 3856: Jenkins Build DocJenkins

Event Timeline

bhd updated this revision to Diff 16537.May 18 2016, 5:32 PM
bhd retitled this revision from to Update de_DE/books/handbook/linuxemu/chapter.xml to r48090.
bhd updated this object.
bhd edited the test plan for this revision. (Show Details)
bhd added a reviewer: bcr.
bhd set the repository for this revision to rD FreeBSD doc repository - subversion.
bcr accepted this revision.May 18 2016, 6:17 PM
bcr edited edge metadata.

Kein Problem, kannst du gerne so machen, weil es in diesem Fall wirklich sinnvoller ist.

linuxemu/chapter.xml
246

s/benutzten/benutzen/

This revision is now accepted and ready to land.May 18 2016, 6:17 PM
Closed by commit rD48830: Update to r48090: (authored by bhd). · Explain WhyMay 18 2016, 7:57 PM
This revision was automatically updated to reflect the committed changes.
bhd marked an inline comment as done.

Revision Contents
Changeset List

PathSize
linuxemu/
605 lines

Diff 16537

View Options

linuxemu/chapter.xml

Show First 20 Lines • Show All 1,629 Lines • ▼ Show 20 Lines#endif
* fall back to non RFC1323 RTT calculation. Normalize * fall back to non RFC1323 RTT calculation. Normalize
* timestamp if syncookies were used when this connection * timestamp if syncookies were used when this connection
* was established. * was established.
*/ */
if ((to.to_flags & TOF_TS) && (to.to_tsecr != 0)) { if ((to.to_flags & TOF_TS) && (to.to_tsecr != 0)) {
to.to_tsecr -= tp->ts_offset; to.to_tsecr -= tp->ts_offset;
if (TSTMP_GT(to.to_tsecr, tcp_ts_getticks())) { if (TSTMP_GT(to.to_tsecr, tcp_ts_getticks())) {
to.to_tsecr = 0; to.to_tsecr = 0;
} else if (tp->t_rxtshift == 1 &&
tp->t_flags & TF_PREVVALID &&
tp->t_badrxtwin != 0 &&
TSTMP_LT(to.to_tsecr, tp->t_badrxtwin)) {
cc_cong_signal(tp, th, CC_RTO_ERR);
} }
} }
/* /*
* Process options only when we get SYN/ACK back. The SYN case * Process options only when we get SYN/ACK back. The SYN case
* for incoming connections is handled in tcp_syncache. * for incoming connections is handled in tcp_syncache.
* According to RFC1323 the window field in a SYN (i.e., a <SYN> * According to RFC1323 the window field in a SYN (i.e., a <SYN>
* or <SYN,ACK>) segment itself is never scaled. * or <SYN,ACK>) segment itself is never scaled.
* XXX this is traditional behavior, may need to be cleaned up. * XXX this is traditional behavior, may need to be cleaned up.
▲ Show 20 Lines • Show All 136 Lines • ▼ Show 20 Lines if (tlen == 0) {
(to.to_flags & TOF_SACK) == 0 && (to.to_flags & TOF_SACK) == 0 &&
TAILQ_EMPTY(&tp->snd_holes)) { TAILQ_EMPTY(&tp->snd_holes)) {
/* /*
* This is a pure ack for outstanding data. * This is a pure ack for outstanding data.
*/ */
TCPSTAT_INC(tcps_predack); TCPSTAT_INC(tcps_predack);
/* /*
* "bad retransmit" recovery without timestamps. * "bad retransmit" recovery.
*/ */
if ((to.to_flags & TOF_TS) == 0 && if (tp->t_rxtshift == 1 &&
tp->t_rxtshift == 1 &&
tp->t_flags & TF_PREVVALID && tp->t_flags & TF_PREVVALID &&
tp->t_badrxtwin != 0 && tp->t_badrxtwin != 0 &&
TSTMP_LT(ticks, tp->t_badrxtwin)) { (((to.to_flags & TOF_TS) != 0 &&
to.to_tsecr != 0 &&
TSTMP_LT(to.to_tsecr, tp->t_badrxtwin)) ||
((to.to_flags & TOF_TS) == 0 &&
TSTMP_LT(ticks, tp->t_badrxtwin))))
cc_cong_signal(tp, th, CC_RTO_ERR); cc_cong_signal(tp, th, CC_RTO_ERR);
}
/* /*
* Recalculate the transmit timer / rtt. * Recalculate the transmit timer / rtt.
* *
* Some boxes send broken timestamp replies * Some boxes send broken timestamp replies
rscheffUnsubmitted
Done
Inline Actions

Any specific reason to write the or statement as two separate if clauses?

I believe in other parts we do it ((true & x & y & z) | (false & a & b &c )) - as multiline checks;

I don't like them both get executed "always", so at the very least, an "else" just before the 2nd if would be good.

The logic looks ok to me.

rscheff: Any specific reason to write the or statement as two separate if clauses? I believe in other…
glebiusUnsubmitted
Done
Inline Actions

I agree with Richard, better one complex if clause. The two clauses are mutually exclusive as one wants TOF_TS and other !TOF_TS, so cc_cong_signal() can be executed only once. My first reading was that it is intentionally written this way cause we may want two calls into cc_cong_singal(), so it was a misleading reading.

Oh, the upper level embracing if can also be merged into one big clause:

if ((tp->t_rxtshift == 1 &&
    tp->t_flags & TF_PREVVALID &&
    tp->t_badrxtwin != 0) &&
    (((to.to_flags & TOF_TS) != 0 &&
    (to.to_tsecr != 0) &&
    TSTMP_LT(to.to_tsecr, tp->t_badrxtwin)) ||
    ((to.to_flags & TOF_TS) == 0 &&
    TSTMP_LT(ticks, tp->t_badrxtwin))))
        cc_cong_signal(tp, th, CC_RTO_ERR);

This will also fix lines longer than 79 chars. IMHO, we may slightly violate style(9) and add single space indentation to highlight our three main large blocks (a && (b || c)).

glebius: I agree with Richard, better one complex if clause. The two clauses are mutually exclusive as…
tuexenAuthorUnsubmitted
Done
Inline Actions

Now using one and two spaces of indentation to make the expression more readable.

tuexen: Now using one and two spaces of indentation to make the expression more readable.
tuexenAuthorUnsubmitted
Done
Inline Actions

It was a try to break things up for improved readability by humans.

tuexen: It was a try to break things up for improved readability by humans.
* during the SYN+ACK phase, ignore * during the SYN+ACK phase, ignore
* timestamps of 0 or we could calculate a * timestamps of 0 or we could calculate a
* huge RTT and blow up the retransmit timer. * huge RTT and blow up the retransmit timer.
*/ */
if ((to.to_flags & TOF_TS) != 0 && if ((to.to_flags & TOF_TS) != 0 &&
to.to_tsecr) { to.to_tsecr) {
uint32_t t; uint32_t t;
▲ Show 20 Lines • Show All 2,424 Lines • Show Last 20 Lines