xref: /freebsd/share/man/man4/siftr.4 (revision 1d386b48)
1.\"
2.\" Copyright (c) 2010 The FreeBSD Foundation
3.\"
4.\" Portions of this software were developed at the Centre for Advanced
5.\" Internet Architectures, Swinburne University of Technology, Melbourne,
6.\" Australia by Lawrence Stewart under sponsorship from the FreeBSD
7.\" Foundation.
8.\"
9.\" Redistribution and use in source and binary forms, with or without
10.\" modification, are permitted provided that the following conditions
11.\" are met:
12.\" 1. Redistributions of source code must retain the above copyright
13.\"    notice, this list of conditions, and the following disclaimer,
14.\"    without modification, immediately at the beginning of the file.
15.\" 2. The name of the author may not be used to endorse or promote products
16.\"    derived from this software without specific prior written permission.
17.\"
18.\" THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND
19.\" ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
20.\" IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
21.\" ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE FOR
22.\" ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
23.\" DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
24.\" OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
25.\" HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
26.\" LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
27.\" OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
28.\" SUCH DAMAGE.
29.\"
30.\" $FreeBSD$
31.\"
32.Dd May 29, 2023
33.Dt SIFTR 4
34.Os
35.Sh NAME
36.Nm SIFTR
37.Nd Statistical Information For TCP Research
38.Sh SYNOPSIS
39To load
40the driver
41as a module at run-time, run the following command as root:
42.Bd -literal -offset indent
43kldload siftr
44.Ed
45.Pp
46Alternatively, to load
47the driver
48as a module at boot time, add the following line into the
49.Xr loader.conf 5
50file:
51.Bd -literal -offset indent
52siftr_load="YES"
53.Ed
54.Sh DESCRIPTION
55The
56.Nm
57.Po
58.Em S Ns tatistical
59.Em I Ns nformation
60.Em F Ns or
61.Em T Ns CP
62.Em R Ns esearch
63.Pc
64kernel module logs a range of statistics on active TCP connections to
65a log file.
66It provides the ability to make highly granular measurements of TCP connection
67state, aimed at system administrators, developers and researchers.
68.Ss Compile-time Configuration
69The default operation of
70.Nm
71is to capture IPv4 TCP/IP packets.
72.Nm
73can be configured to support IPv4 and IPv6 by uncommenting:
74.Bd -literal -offset indent
75CFLAGS+=-DSIFTR_IPV6
76.Ed
77.Pp
78in
79.Aq sys/modules/siftr/Makefile
80and recompiling.
81.Pp
82In the IPv4-only (default) mode, standard dotted decimal notation (e.g.
83"136.186.229.95") is used to format IPv4 addresses for logging.
84In IPv6 mode, standard dotted decimal notation is used to format IPv4 addresses,
85and standard colon-separated hex notation (see RFC 4291) is used to format IPv6
86addresses (e.g. "fd00::2") for logging.
87.Ss Run-time Configuration
88.Nm
89utilises the
90.Xr sysctl 8
91interface to export its configuration variables to user-space.
92The following variables are available:
93.Bl -tag -offset indent -width Va
94.It Va net.inet.siftr.enabled
95controls whether the module performs its
96measurements or not.
97By default, the value is set to 0, which means the module
98will not be taking any measurements.
99Having the module loaded with
100.Va net.inet.siftr.enabled
101set to 0 will have no impact on the performance of the network stack, as the
102packet filtering hooks are only inserted when
103.Va net.inet.siftr.enabled
104is set to 1.
105.El
106.Bl -tag -offset indent -width Va
107.It Va net.inet.siftr.ppl
108controls how many inbound/outbound packets for a given TCP connection will cause
109a log message to be generated for the connection.
110By default, the value is set to 1, which means the module will log a message for
111every packet of every TCP connection.
112The value can be set to any integer in the range [1,2^32], and can be changed at
113any time, even while the module is enabled.
114.El
115.Bl -tag -offset indent -width Va
116.It Va net.inet.siftr.logfile
117controls the path to the file that the module writes its log messages to.
118By default, the file /var/log/siftr.log is used.
119The path can be changed at any time, even while the module is enabled.
120.El
121.Bl -tag -offset indent -width Va
122.It Va net.inet.siftr.port_filter
123controls on which source or destination port
124.Nm
125should capture.
126By default, the value is set to 0, which means all ports are eligible for logging.
127Set to any other value, only packets where either the source or destination
128port is equal to this number are logged.
129.El
130.Ss Log Format
131A typical
132.Nm
133log file will contain 3 different types of log message.
134All messages are written in plain ASCII text.
135.Pp
136Note: The
137.Qq \e
138present in the example log messages in this section indicates a
139line continuation and is not part of the actual log message.
140.Pp
141The first type of log message is written to the file when the module is
142enabled and starts collecting data from the running kernel.
143The text below shows an example module enable log.
144The fields are tab delimited key-value
145pairs which describe some basic information about the system.
146.Bd -literal -offset indent
147enable_time_secs=1685191807    enable_time_usecs=160752 \\
148siftrver=1.3.0    sysname=FreeBSD    sysver=1400089    ipmode=4
149.Ed
150.Pp
151Field descriptions are as follows:
152.Bl -tag -offset indent -width Va
153.It Va enable_time_secs
154time at which the module was enabled, in seconds since the UNIX epoch.
155.El
156.Bl -tag -offset indent -width Va
157.It Va enable_time_usecs
158time at which the module was enabled, in microseconds since enable_time_secs.
159.El
160.Bl -tag -offset indent -width Va
161.It Va siftrver
162version of
163.Nm .
164.El
165.Bl -tag -offset indent -width Va
166.It Va sysname
167operating system name.
168.El
169.Bl -tag -offset indent -width Va
170.It Va sysver
171operating system version.
172.El
173.Bl -tag -offset indent -width Va
174.It Va ipmode
175IP mode as defined at compile time.
176An ipmode of "4" means IPv6 is not supported and IP addresses are logged in
177regular dotted quad format.
178An ipmode of "6" means IPv6 is supported, and IP addresses are logged in dotted
179quad or hex format, as described in the
180.Qq Compile-time Configuration
181subsection.
182.El
183.Pp
184The second type of log message is written to the file when a data log message
185is generated.
186The text below shows an example data log triggered by an IPv4
187TCP/IP packet.
188The data is CSV formatted.
189.Bd -literal -offset indent
190o,1685191814.185109,10.1.1.2,32291,10.1.1.3,5001,1073725440, \\
19114480,2,65160,65700,7,9,4,1460,1000,1,16778209,230000,33580,0, \\
19265700,0,0,0,86707916,130
193.Ed
194.Pp
195Field descriptions are as follows:
196.Bl -tag -offset indent -width Va
197.It Va 1
198Direction of packet that triggered the log message.
199Either
200.Qq i
201for in, or
202.Qq o
203for out.
204.El
205.Bl -tag -offset indent -width Va
206.It Va 2
207Time at which the packet that triggered the log message was processed by
208the
209.Xr pfil 9
210hook function, in seconds and microseconds since the UNIX epoch.
211.El
212.Bl -tag -offset indent -width Va
213.It Va 3
214The IPv4 or IPv6 address of the local host, in dotted quad (IPv4 packet)
215or colon-separated hex (IPv6 packet) notation.
216.El
217.Bl -tag -offset indent -width Va
218.It Va 4
219The TCP port that the local host is communicating via.
220.El
221.Bl -tag -offset indent -width Va
222.It Va 5
223The IPv4 or IPv6 address of the foreign host, in dotted quad (IPv4 packet)
224or colon-separated hex (IPv6 packet) notation.
225.El
226.Bl -tag -offset indent -width Va
227.It Va 6
228The TCP port that the foreign host is communicating via.
229.El
230.Bl -tag -offset indent -width Va
231.It Va 7
232The slow start threshold for the flow, in bytes.
233.El
234.Bl -tag -offset indent -width Va
235.It Va 8
236The current congestion window for the flow, in bytes.
237.El
238.Bl -tag -offset indent -width Va
239.It Va 9
240The current state of the t_flags2 field for the flow.
241.El
242.Bl -tag -offset indent -width Va
243.It Va 10
244The current sending window for the flow, in bytes.
245The post scaled value is reported.
246.El
247.Bl -tag -offset indent -width Va
248.It Va 11
249The current receive window for the flow, in bytes.
250The post scaled value is always reported.
251.El
252.Bl -tag -offset indent -width Va
253.It Va 12
254The current window scaling factor for the sending window.
255.El
256.Bl -tag -offset indent -width Va
257.It Va 13
258The current window scaling factor for the receiving window.
259.El
260.Bl -tag -offset indent -width Va
261.It Va 14
262The current state of the TCP finite state machine, as defined
263in
264.Aq Pa netinet/tcp_fsm.h .
265.El
266.Bl -tag -offset indent -width Va
267.It Va 15
268The maximum segment size for the flow, in bytes.
269.El
270.Bl -tag -offset indent -width Va
271.It Va 16
272The current smoothed RTT estimate for the flow, in units of microsecond.
273.El
274.Bl -tag -offset indent -width Va
275.It Va 17
276SACK enabled indicator. 1 if SACK enabled, 0 otherwise.
277.El
278.Bl -tag -offset indent -width Va
279.It Va 18
280The current state of the TCP flags for the flow.
281See
282.Aq Pa netinet/tcp_var.h
283for information about the various flags.
284.El
285.Bl -tag -offset indent -width Va
286.It Va 19
287The current retransmission timeout length for the flow, in units microsecond.
288.El
289.Bl -tag -offset indent -width Va
290.It Va 20
291The current size of the socket send buffer in bytes.
292.El
293.Bl -tag -offset indent -width Va
294.It Va 21
295The current number of bytes in the socket send buffer.
296.El
297.Bl -tag -offset indent -width Va
298.It Va 22
299The current size of the socket receive buffer in bytes.
300.El
301.Bl -tag -offset indent -width Va
302.It Va 23
303The current number of bytes in the socket receive buffer.
304.El
305.Bl -tag -offset indent -width Va
306.It Va 24
307The current number of unacknowledged bytes in-flight.
308Bytes acknowledged via SACK are not excluded from this count.
309.El
310.Bl -tag -offset indent -width Va
311.It Va 25
312The current number of segments in the reassembly queue.
313.El
314.Bl -tag -offset indent -width Va
315.It Va 26
316Flowid for the connection.
317A caveat: Zero '0' either represents a valid flowid or a default value when it's
318not being set.
319There is no easy way to differentiate without looking at actual
320network interface card and drivers being used.
321.El
322.Bl -tag -offset indent -width Va
323.It Va 27
324Flow type for the connection.
325Flowtype defines which protocol fields are hashed to produce the flowid.
326A complete listing is available in
327.Pa sys/mbuf.h
328under
329.Dv M_HASHTYPE_* .
330.El
331.Pp
332The third type of log message is written to the file when the module is disabled
333and ceases collecting data from the running kernel.
334The text below shows an example module disable log.
335The fields are tab delimited key-value pairs which provide statistics about
336operations since the module was most recently enabled.
337.Bd -literal -offset indent
338disable_time_secs=1685191816    disable_time_usecs=629397 \\
339num_inbound_tcp_pkts=10    num_outbound_tcp_pkts=10 \\
340total_tcp_pkts=20    num_inbound_skipped_pkts_malloc=0 \\
341num_outbound_skipped_pkts_malloc=0    num_inbound_skipped_pkts_tcpcb=2 \\
342num_outbound_skipped_pkts_tcpcb=2    num_inbound_skipped_pkts_inpcb=0 \\
343num_outbound_skipped_pkts_inpcb=0    total_skipped_tcp_pkts=4 \\
344flow_list=10.1.1.2;32291-10.1.1.3;5001,10.1.1.2;58544-10.1.1.3;5001,
345.Ed
346.Pp
347Field descriptions are as follows:
348.Bl -tag -offset indent -width Va
349.It Va disable_time_secs
350Time at which the module was disabled, in seconds since the UNIX epoch.
351.El
352.Bl -tag -offset indent -width Va
353.It Va disable_time_usecs
354Time at which the module was disabled, in microseconds since disable_time_secs.
355.El
356.Bl -tag -offset indent -width Va
357.It Va num_inbound_tcp_pkts
358Number of TCP packets that traversed up the network stack.
359This only includes inbound TCP packets during the periods when
360.Nm
361was enabled.
362.El
363.Bl -tag -offset indent -width Va
364.It Va num_outbound_tcp_pkts
365Number of TCP packets that traversed down the network stack.
366This only includes outbound TCP packets during the periods when
367.Nm
368was enabled.
369.El
370.Bl -tag -offset indent -width Va
371.It Va total_tcp_pkts
372The summation of num_inbound_tcp_pkts and num_outbound_tcp_pkts.
373.El
374.Bl -tag -offset indent -width Va
375.It Va num_inbound_skipped_pkts_malloc
376Number of inbound packets that were not processed because of failed
377.Fn malloc
378calls.
379.El
380.Bl -tag -offset indent -width Va
381.It Va num_outbound_skipped_pkts_malloc
382Number of outbound packets that were not processed because of failed
383.Fn malloc
384calls.
385.El
386.Bl -tag -offset indent -width Va
387.It Va num_inbound_skipped_pkts_tcpcb
388Number of inbound packets that were not processed because of failure to find the
389TCP control block associated with the packet.
390.El
391.Bl -tag -offset indent -width Va
392.It Va num_outbound_skipped_pkts_tcpcb
393Number of outbound packets that were not processed because of failure to find
394the TCP control block associated with the packet.
395.El
396.Bl -tag -offset indent -width Va
397.It Va num_inbound_skipped_pkts_inpcb
398Number of inbound packets that were not processed because of failure to find the
399IP control block associated with the packet.
400.El
401.Bl -tag -offset indent -width Va
402.It Va num_outbound_skipped_pkts_inpcb
403Number of outbound packets that were not processed because of failure to find
404the IP control block associated with the packet.
405.El
406.Bl -tag -offset indent -width Va
407.It Va total_skipped_tcp_pkts
408The summation of all skipped packet counters.
409.El
410.Bl -tag -offset indent -width Va
411.It Va flow_list
412A CSV list of TCP flows that triggered data log messages to be generated since
413the module was loaded.
414Each flow entry in the CSV list is
415formatted as
416.Qq local_ip;local_port-foreign_ip;foreign_port .
417If there are no entries in the list (i.e., no data log messages were generated),
418the value will be blank.
419If there is at least one entry in the list, a trailing comma will always be
420present.
421.El
422.Pp
423The total number of data log messages found in the log file for a module
424enable/disable cycle should equate to total_tcp_pkts - total_skipped_tcp_pkts.
425.Sh IMPLEMENTATION NOTES
426.Nm
427hooks into the network stack using the
428.Xr pfil 9
429interface.
430In its current incarnation, it hooks into the AF_INET/AF_INET6 (IPv4/IPv6)
431.Xr pfil 9
432filtering points, which means it sees packets at the IP layer of the network
433stack.
434This means that TCP packets inbound to the stack are intercepted before
435they have been processed by the TCP layer.
436Packets outbound from the stack are intercepted after they have been processed
437by the TCP layer.
438.Pp
439The diagram below illustrates how
440.Nm
441inserts itself into the stack.
442.Bd -literal -offset indent
443----------------------------------
444           Upper Layers
445----------------------------------
446    ^                       |
447    |                       |
448    |                       |
449    |                       v
450 TCP in                  TCP out
451----------------------------------
452    ^                      |
453    |________     _________|
454            |     |
455            |     v
456           ---------
457           | SIFTR |
458           ---------
459            ^     |
460    ________|     |__________
461    |                       |
462    |                       v
463IPv{4/6} in            IPv{4/6} out
464----------------------------------
465    ^                       |
466    |                       |
467    |                       v
468Layer 2 in             Layer 2 out
469----------------------------------
470          Physical Layer
471----------------------------------
472.Ed
473.Pp
474.Nm
475uses the
476.Xr alq 9
477interface to manage writing data to disk.
478.Pp
479At first glance, you might mistakenly think that
480.Nm
481extracts information from
482individual TCP packets.
483This is not the case.
484.Nm
485uses TCP packet events (inbound and outbound) for each TCP flow originating from
486the system to trigger a dump of the state of the TCP control block for that
487flow.
488With the PPL set to 1, we are in effect sampling each TCP flow's control block
489state as frequently as flow packets enter/leave the system.
490For example, setting PPL to 2 halves the sampling rate i.e., every second flow
491packet (inbound OR outbound) causes a dump of the control block state.
492.Pp
493The distinction between interrogating individual packets versus interrogating the
494control block is important, because
495.Nm
496does not remove the need for packet capturing tools like
497.Xr tcpdump 1 .
498.Nm
499allows you to correlate and observe the cause-and-affect relationship between
500what you see on the wire (captured using a tool like
501.Xr tcpdump 1 Ns )
502and changes in the TCP control block corresponding to the flow of interest.
503It is therefore useful to use
504.Nm
505and a tool like
506.Xr tcpdump 1
507to gather the necessary data to piece together the complete picture.
508Use of either tool on its own will not be able to provide all of the necessary
509data.
510.Pp
511As a result of needing to interrogate the TCP control block, certain packets
512during the lifecycle of a connection are unable to trigger a
513.Nm
514log message.
515The initial handshake takes place without the existence of a control block or
516the complete initialization of the control block, and the final ACK is
517exchanged when the connection is in the TIMEWAIT state.
518.Pp
519.Nm
520was designed to minimise the delay introduced to packets traversing the network
521stack.
522This design called for a highly optimised and minimal hook function that
523extracted the minimal details necessary whilst holding the packet up, and
524passing these details to another thread for actual processing and logging.
525.Pp
526This multithreaded design does introduce some contention issues when accessing
527the data structure shared between the threads of operation.
528When the hook function tries to place details in the structure, it must first
529acquire an exclusive lock.
530Likewise, when the processing thread tries to read details from the structure,
531it must also acquire an exclusive lock to do so.
532If one thread holds the lock, the other must wait before it can obtain it.
533This does introduce some additional bounded delay into the kernel's packet
534processing code path.
535.Pp
536In some cases (e.g., low memory, connection termination), TCP packets that enter
537the
538.Nm
539.Xr pfil 9
540hook function will not trigger a log message to be generated.
541.Nm
542refers to this outcome as a
543.Qq skipped packet .
544Note that
545.Nm
546always ensures that packets are allowed to continue through the stack, even if
547they could not successfully trigger a data log message.
548.Nm
549will therefore not introduce any packet loss for TCP/IP packets traversing the
550network stack.
551.Ss Important Behaviours
552The behaviour of a log file path change whilst the module is enabled is as
553follows:
554.Bl -enum
555.It
556Attempt to open the new file path for writing.
557If this fails, the path change will fail and the existing path will continue to
558be used.
559.It
560Assuming the new path is valid and opened successfully:
561.Bl -dash
562.It
563Flush all pending log messages to the old file path.
564.It
565Close the old file path.
566.It
567Switch the active log file pointer to point at the new file path.
568.It
569Commence logging to the new file.
570.El
571.El
572.Pp
573During the time between the flush of pending log messages to the old file and
574commencing logging to the new file, new log messages will still be generated and
575buffered.
576As soon as the new file path is ready for writing, the accumulated log messages
577will be written out to the file.
578.Sh EXAMPLES
579To enable the module's operations, run the following command as root:
580sysctl net.inet.siftr.enabled=1
581.Pp
582To change the granularity of log messages such that 1 log message is
583generated for every 10 TCP packets per connection, run the following
584command as root:
585sysctl net.inet.siftr.ppl=10
586.Pp
587To change the log file location to /tmp/siftr.log, run the following
588command as root:
589sysctl net.inet.siftr.logfile=/tmp/siftr.log
590.Sh SEE ALSO
591.Xr tcpdump 1 ,
592.Xr tcp 4 ,
593.Xr sysctl 8 ,
594.Xr alq 9 ,
595.Xr pfil 9
596.Sh ACKNOWLEDGEMENTS
597Development of this software was made possible in part by grants from the
598Cisco University Research Program Fund at Community Foundation Silicon Valley,
599and the FreeBSD Foundation.
600.Sh HISTORY
601.Nm
602first appeared in
603.Fx 7.4
604and
605.Fx 8.2 .
606.Pp
607.Nm
608was first released in 2007 by Lawrence Stewart and James Healy whilst working on
609the NewTCP research project at Swinburne University of Technology's Centre for
610Advanced Internet Architectures, Melbourne, Australia, which was made possible
611in part by a grant from the Cisco University Research Program Fund at Community
612Foundation Silicon Valley.
613More details are available at:
614.Pp
615http://caia.swin.edu.au/urp/newtcp/
616.Pp
617Work on
618.Nm
619v1.2.x was sponsored by the FreeBSD Foundation as part of
620the
621.Qq Enhancing the FreeBSD TCP Implementation
622project 2008-2009.
623More details are available at:
624.Pp
625https://www.freebsdfoundation.org/
626.Pp
627http://caia.swin.edu.au/freebsd/etcp09/
628.Sh AUTHORS
629.An -nosplit
630.Nm
631was written by
632.An Lawrence Stewart Aq Mt lstewart@FreeBSD.org
633and
634.An James Healy Aq Mt jimmy@deefa.com .
635.Pp
636This manual page was written by
637.An Lawrence Stewart Aq Mt lstewart@FreeBSD.org .
638.Sh BUGS
639Current known limitations and any relevant workarounds are outlined below:
640.Bl -dash
641.It
642The internal queue used to pass information between the threads of operation is
643currently unbounded.
644This allows
645.Nm
646to cope with bursty network traffic, but sustained high packet-per-second
647traffic can cause exhaustion of kernel memory if the processing thread cannot
648keep up with the packet rate.
649.It
650If using
651.Nm
652on a machine that is also running other modules utilising the
653.Xr pfil 9
654framework e.g.
655.Xr dummynet 4 ,
656.Xr ipfw 8 ,
657.Xr pf 4 Ns ,
658the order in which you load the modules is important.
659You should kldload the other modules first, as this will ensure TCP packets
660undergo any necessary manipulations before
661.Nm
662.Qq sees
663and processes them.
664.It
665There is a known, harmless lock order reversal warning between the
666.Xr pfil 9
667mutex and tcbinfo TCP lock reported by
668.Xr witness 4
669when
670.Nm
671is enabled in a kernel compiled with
672.Xr witness 4
673support.
674.It
675There is no way to filter which TCP flows you wish to capture data for.
676Post processing is required to separate out data belonging to particular flows
677of interest.
678.It
679The module does not detect deletion of the log file path.
680New log messages will simply be lost if the log file being used by
681.Nm
682is deleted whilst the module is set to use the file.
683Switching to a new log file using the
684.Em net.inet.siftr.logfile
685variable will create the new file and allow log messages to begin being written
686to disk again.
687The new log file path must differ from the path to the deleted file.
688.It
689The hash table used within the code is sized to hold 65536 flows.
690This is not a
691hard limit, because chaining is used to handle collisions within the hash table
692structure.
693However, we suspect (based on analogies with other hash table performance data)
694that the hash table look up performance (and therefore the module's packet
695processing performance) will degrade in an exponential manner as the number of
696unique flows handled in a module enable/disable cycle approaches and surpasses
69765536.
698.It
699There is no garbage collection performed on the flow hash table.
700The only way currently to flush it is to disable
701.Nm .
702.It
703The PPL variable applies to packets that make it into the processing thread,
704not total packets received in the hook function.
705Packets are skipped before the PPL variable is applied, which means there may be
706a slight discrepancy in the triggering of log messages.
707For example, if PPL was set to 10, and the 8th packet since the last log message
708is skipped, the 11th packet will actually trigger the log message to be
709generated.
710This is discussed in greater depth in CAIA technical report 070824A.
711.It
712At the time of writing, there was no simple way to hook into the TCP layer
713to intercept packets.
714.Nm Ap s
715use of IP layer hook points means all IP
716traffic will be processed by the
717.Nm
718.Xr pfil 9
719hook function, which introduces minor, but nonetheless unnecessary packet delay
720and processing overhead on the system for non-TCP packets as well.
721Hooking in at the IP layer is also not ideal from the data gathering point of
722view.
723Packets traversing up the stack will be intercepted and cause a log message
724generation BEFORE they have been processed by the TCP layer, which means we
725cannot observe the cause-and-affect relationship between inbound events and the
726corresponding TCP control block as precisely as could be.
727Ideally,
728.Nm
729should intercept packets after they have been processed by the TCP layer i.e.
730intercept packets coming up the stack after they have been processed by
731.Fn tcp_input ,
732and intercept packets coming down the stack after they have been
733processed by
734.Fn tcp_output .
735The current code still gives satisfactory granularity though, as inbound events
736tend to trigger outbound events, allowing the cause-and-effect to be observed
737indirectly by capturing the state on outbound events as well.
738.It
739The
740.Qq inflight bytes
741value logged by
742.Nm
743does not take into account bytes that have been
744.No SACK Ap ed
745by the receiving host.
746.El
747