Page MenuHomeFreeBSD
Feed Advanced Search

Dec 6 2016

gnn retitled D8718: Fix a kernel panic in DTrace's rw_iswriter subroutine. On FreeBSD the sense of rw_write_held() and rw_iswriter() were reversed probably due to a cut and paste error. Using rw_iswriter() would cause the kernel to panic. from to Fix a kernel panic in DTrace's rw_iswriter subroutine. On FreeBSD the sense of rw_write_held() and rw_iswriter() were reversed probably due to a cut and paste error. Using rw_iswriter() would cause the kernel to panic..
Dec 6 2016, 2:00 PM

Nov 28 2016

gnn requested changes to D8021: Correctness fixes to Cubic .

Unless I'm mistaken this is several fixes, not all of which are related to CUBIC, but contained in a single patch. Please split these out into separate changes, in particular the mathematical equation change in cc_cubic.h. While I believe that the math change there is correct (from reading the latest draft) I'd like it to be possible to revert that one change simply if it proves to be an issue.

Nov 28 2016, 5:43 PM
gnn accepted D7986: Simplify cubic_ack_received.

LGTM

Nov 28 2016, 5:32 PM

Nov 23 2016

gnn committed rS309069: Add tunable to disable destructive dtrace.
Add tunable to disable destructive dtrace
Nov 23 2016, 10:50 PM
gnn closed D8624: Add tunable to disable destructive dtrace Security by committing rS309069: Add tunable to disable destructive dtrace.
Nov 23 2016, 10:50 PM
gnn retitled D8624: Add tunable to disable destructive dtrace Security from to Add tunable to disable destructive dtrace Security.
Nov 23 2016, 4:47 PM
gnn committed rS309057: MFC: 307541.
MFC: 307541
Nov 23 2016, 1:14 PM

Nov 19 2016

gnn committed rP426538: Update distinfo to match the Makefile version.
Update distinfo to match the Makefile version
Nov 19 2016, 9:18 PM

Nov 18 2016

gnn accepted D8443: Improve TCP state change reporting via dtrace.
Nov 18 2016, 10:52 PM
gnn accepted D8552: Ignore ACKs with tsecr 0 during 3whs in syncache code..
Nov 18 2016, 10:50 PM
gnn committed rP426304: Improve timing and fix a race condition.
Improve timing and fix a race condition
Nov 18 2016, 2:14 AM

Nov 5 2016

gnn committed rP425374: Update the port to the latest version moved to the freebsd-net github….
Update the port to the latest version moved to the freebsd-net github…
Nov 5 2016, 4:50 AM

Nov 3 2016

gnn added a comment to D3687: Implement kernel support for hardware rate limited sockets.

A few of us discussed this today in the transport conference call. hps@ will have an update based on that discussion. The gist is that np@, hps@ and other NIC facing folks should make the decision on how to use the fields.

Nov 3 2016, 5:46 PM

Oct 28 2016

gnn committed rS308053: Corrected non-portable reuse of va_list in dt_printf().
Corrected non-portable reuse of va_list in dt_printf()
Oct 28 2016, 4:28 PM
gnn accepted D8279: Provide individual RTT measurements to CC algos..

I think this is a good addition.

Oct 28 2016, 3:25 PM

Oct 22 2016

gnn added a comment to D5213: convert ixgbe to iflib.

I am confused by the question about the manual page. That can be put under a separate review if you like.

Oct 22 2016, 3:27 PM

Oct 19 2016

gnn added a comment to D3687: Implement kernel support for hardware rate limited sockets.

We have 100G hardware at Sentex on the Mercats. I can free those hosts up for you.

Oct 19 2016, 4:20 PM
gnn added a comment to D3687: Implement kernel support for hardware rate limited sockets.

A quick note that function calls are definitely the preferred solution.

Oct 19 2016, 12:15 AM

Oct 17 2016

gnn closed D5157: Patch for bug 100219 by committing rS307541: Limit the number of mbufs that can be allocated for IPV6_2292PKTOPTIONS.
Oct 17 2016, 11:25 PM
gnn committed rS307541: Limit the number of mbufs that can be allocated for IPV6_2292PKTOPTIONS.
Limit the number of mbufs that can be allocated for IPV6_2292PKTOPTIONS
Oct 17 2016, 11:25 PM
gnn committed rP424122: Note that this port is broken on FreeBSD 9..
Note that this port is broken on FreeBSD 9.
Oct 17 2016, 3:00 PM
gnn accepted D8211: Fix a double-free when an inp transitions to INP_TIMEWAIT state after having been dropped..
Oct 17 2016, 2:01 PM
gnn accepted D5157: Patch for bug 100219.

Looks good to me. I can commit this if you like.

Oct 17 2016, 12:16 PM

Oct 11 2016

gnn closed D8157: Corrected non-portable reuse of va_list in dt_printf() by committing rS307044: Corrected non-portable reuse of va_list in dt_printf().
Oct 11 2016, 4:12 PM
gnn committed rS307044: Corrected non-portable reuse of va_list in dt_printf().
Corrected non-portable reuse of va_list in dt_printf()
Oct 11 2016, 4:12 PM

Oct 10 2016

gnn accepted D8203: sfxge(4): sync tlv_layout.h with firmwaresrc and update port-mode definition use.
Oct 10 2016, 2:57 AM

Oct 7 2016

gnn accepted D8002: Clean up memory management in pmcstat.

LGTM

Oct 7 2016, 7:39 PM
gnn committed rP423467: Undo accidental inclusion of packetdrill update in tcptestsuite commit.
Undo accidental inclusion of packetdrill update in tcptestsuite commit
Oct 7 2016, 3:10 PM
gnn committed rP423466: Add a port for a set of scripts to test TCP using packetdrill.
Add a port for a set of scripts to test TCP using packetdrill
Oct 7 2016, 3:01 PM

Oct 5 2016

gnn updated the diff for D8157: Corrected non-portable reuse of va_list in dt_printf().

Address comments from markj@

Oct 5 2016, 10:34 PM
gnn retitled D8157: Corrected non-portable reuse of va_list in dt_printf() from to Corrected non-portable reuse of va_list in dt_printf().
Oct 5 2016, 8:59 PM
gnn accepted D5334: Move netmap selinfo.h in to sensible location.
Oct 5 2016, 8:51 PM

Oct 3 2016

gnn accepted D8072: Remove an extraneous call to soisconnected() in syncache_socket(), introduced with r261242..

LGTM

Oct 3 2016, 9:59 AM

Oct 1 2016

gnn added a comment to D8097: Create pkgbase packages for CDDL libraries and binaries.

Do we have a document on how the packaging for base works? I'd like to understand the process a bit better before approving this review.

Oct 1 2016, 1:53 PM

Sep 30 2016

gnn requested changes to D8097: Create pkgbase packages for CDDL libraries and binaries.

I'd like a reference to what these will actually do. It seems that each library is now a package but that may not make sense based on the structure of those libraries.

Sep 30 2016, 9:27 PM

Sep 28 2016

gnn added a comment to D8053: Properly preserve ip_tos bits for IPv4 packets.

I'd prefer a broad, cleanup commit.

Sep 28 2016, 12:41 PM
gnn accepted D8053: Properly preserve ip_tos bits for IPv4 packets.

This looks correct to me.

Sep 28 2016, 12:04 PM

Sep 24 2016

gnn committed rS306296: MFC: 305066,305304,305312.
MFC: 305066,305304,305312
Sep 24 2016, 10:38 PM

Sep 20 2016

gnn accepted D7980: fix sooptcopyin on big-endian platforms (e.g. MIPS64EB).
Sep 20 2016, 3:04 PM

Sep 15 2016

gnn added a comment to D6448: Allow bigger number of packets to generate.

I will test and commit this one.

Sep 15 2016, 3:30 PM

Sep 14 2016

gnn accepted D7878: Remove the 4.3BSD compatible macro m_copy(), use m_copym() instead.
Sep 14 2016, 3:41 PM

Sep 12 2016

gnn committed rS305748: MFC: 304825.
MFC: 304825
Sep 12 2016, 5:05 PM
gnn committed rS305747: MFC: 304825.
MFC: 304825
Sep 12 2016, 5:02 PM

Sep 3 2016

gnn closed D7766: Add cpuset(2) support to sepearate forked processes. by committing rS305312: Add cpuset support to separate forked processes..
Sep 3 2016, 12:23 AM
gnn committed rS305312: Add cpuset support to separate forked processes..
Add cpuset support to separate forked processes.
Sep 3 2016, 12:23 AM
gnn added a comment to D7766: Add cpuset(2) support to sepearate forked processes..

I've address the main issues.

Sep 3 2016, 12:20 AM

Sep 2 2016

gnn updated the diff for D7766: Add cpuset(2) support to sepearate forked processes..

Address issue of too many threads.

Sep 2 2016, 10:31 PM
gnn committed rS305307: Add a runner script for cryptotest..
Add a runner script for cryptotest.
Sep 2 2016, 9:35 PM
gnn added a reviewer for D7766: Add cpuset(2) support to sepearate forked processes.: cem.
Sep 2 2016, 9:14 PM
gnn retitled D7766: Add cpuset(2) support to sepearate forked processes. from to Add cpuset(2) support to sepearate forked processes..
Sep 2 2016, 9:14 PM
gnn closed D7765: Update help and remove support for arc4 by committing rS305304: Clean up the usage message and remove dead code..
Sep 2 2016, 9:11 PM
gnn committed rS305304: Clean up the usage message and remove dead code..
Clean up the usage message and remove dead code.
Sep 2 2016, 9:11 PM
gnn updated the diff for D7765: Update help and remove support for arc4.

Add support for using cpuset to spearate forked processes.

Sep 2 2016, 8:53 PM
gnn updated the diff for D7765: Update help and remove support for arc4.

Remove dead code.

Sep 2 2016, 8:41 PM
gnn added a reviewer for D7765: Update help and remove support for arc4: cem.
Sep 2 2016, 8:36 PM
gnn retitled D7765: Update help and remove support for arc4 from to Update help and remove support for arc4.
Sep 2 2016, 8:36 PM

Aug 30 2016

gnn closed D7598: Update cryptotest for modern algorithms. by committing rS305066: Update cryptotest for modern algorithms.
Aug 30 2016, 2:28 PM
gnn committed rS305066: Update cryptotest for modern algorithms.
Update cryptotest for modern algorithms
Aug 30 2016, 2:28 PM

Aug 29 2016

gnn added a comment to D7598: Update cryptotest for modern algorithms..

Address more comments.

Aug 29 2016, 2:51 PM

Aug 25 2016

gnn closed D7648: Unlike Solaris, in FreeBSD p_args can be NULL so check for that before checking ar_args..

Committed revision 304825.

Aug 25 2016, 11:25 PM
gnn committed rS304825: Unlike Solaris, in FreeBSD p_args can be 0 so check for that.
Unlike Solaris, in FreeBSD p_args can be 0 so check for that
Aug 25 2016, 11:25 PM
gnn updated the diff for D7648: Unlike Solaris, in FreeBSD p_args can be NULL so check for that before checking ar_args..

jhb@ reports that if p_args is non 0 then ar_args will be valid.

Aug 25 2016, 11:09 PM
gnn updated the diff for D7648: Unlike Solaris, in FreeBSD p_args can be NULL so check for that before checking ar_args..

Tighten up the logic.

Aug 25 2016, 11:06 PM
gnn added reviewers for D7648: Unlike Solaris, in FreeBSD p_args can be NULL so check for that before checking ar_args.: markj, DTrace.
Aug 25 2016, 10:58 PM
gnn retitled D7648: Unlike Solaris, in FreeBSD p_args can be NULL so check for that before checking ar_args. from to Unlike Solaris, in FreeBSD p_args can be NULL so check for that before checking ar_args..
Aug 25 2016, 10:57 PM

Aug 23 2016

gnn added a comment to D7615: Adding randomized MAC address option on ifconfig..

I guess my other fear is that the address might overlap an existing allocation. What is the use case for random MACs?

Aug 23 2016, 7:44 PM
gnn requested changes to D7615: Adding randomized MAC address option on ifconfig..

I am somewhat confused. I don't see anything about random addresses in the RFC mentioned in the followup.

Aug 23 2016, 7:24 PM
gnn added a comment to D7598: Update cryptotest for modern algorithms..

There is no reason for the functions to be static at all, so removing them just cleans up the code and makes it trivially debuggable.

Aug 23 2016, 12:53 PM

Aug 22 2016

gnn added reviewers for D7598: Update cryptotest for modern algorithms.: ngie, ae.
Aug 22 2016, 5:30 PM
gnn retitled D7598: Update cryptotest for modern algorithms. from to Update cryptotest for modern algorithms..
Aug 22 2016, 5:27 PM
gnn accepted D7591: Fix L2 caching for UDP over IPv6.

Approved by mentor.

Aug 22 2016, 8:49 AM

Aug 18 2016

gnn committed rS304402: Remove the obsolete and unused openbsd_poll system call. (Phase 2).
Remove the obsolete and unused openbsd_poll system call. (Phase 2)
Aug 18 2016, 10:55 AM
gnn closed D7548: Remove the obsolete openbsd_poll system call. by committing rS304395: Remove unusedd and obsolete openbsd_poll system call. (Phase 1).
Aug 18 2016, 10:50 AM
gnn committed rS304395: Remove unusedd and obsolete openbsd_poll system call. (Phase 1).
Remove unusedd and obsolete openbsd_poll system call. (Phase 1)
Aug 18 2016, 10:50 AM
gnn updated the diff for D7548: Remove the obsolete openbsd_poll system call..

Add the version update

Aug 18 2016, 10:33 AM

Aug 17 2016

gnn accepted D6448: Allow bigger number of packets to generate.
Aug 17 2016, 2:51 PM
gnn updated the diff for D7548: Remove the obsolete openbsd_poll system call..

Address issues pointed out by brooks@

Aug 17 2016, 1:15 PM
gnn added reviewers for D7548: Remove the obsolete openbsd_poll system call.: brooks, jhb.
Aug 17 2016, 12:29 PM
gnn retitled D7548: Remove the obsolete openbsd_poll system call. from to Remove the obsolete openbsd_poll system call..
Aug 17 2016, 12:29 PM
gnn abandoned D7144: Corrected non-portable reuse of va_list in dt_printf().

Chuck this one because arc.

Aug 17 2016, 12:26 PM
gnn added a reviewer for D7144: Corrected non-portable reuse of va_list in dt_printf(): jhb.
Aug 17 2016, 11:31 AM
gnn updated the diff for D7144: Corrected non-portable reuse of va_list in dt_printf().

Remove the obsolete opebsd_poll system call.

Aug 17 2016, 11:30 AM

Aug 16 2016

gnn added a comment to D7529: libifconfig: First baby steps towards libification of ifconfig.

A few points about how we put code into FreeBSD.

Aug 16 2016, 1:10 PM

Aug 11 2016

gnn accepted D7309: Don't check for broadcast IPs on non-bcast pkts.
Aug 11 2016, 4:08 PM

Aug 2 2016

gnn updated the diff for D7144: Corrected non-portable reuse of va_list in dt_printf().

Address issue pointed out by markj

Aug 2 2016, 3:19 PM

Jul 29 2016

gnn accepted D7350: PMC: Collect user call chains while in kernel space.
Jul 29 2016, 10:26 PM

Jul 27 2016

gnn added a comment to D2901: Prepare for network stack as a module..

Accepted. Go for it.

Jul 27 2016, 1:46 PM
gnn accepted D2901: Prepare for network stack as a module..
Jul 27 2016, 1:46 PM

Jul 25 2016

gnn accepted D7309: Don't check for broadcast IPs on non-bcast pkts.
Jul 25 2016, 10:08 PM

Jul 11 2016

gnn accepted D5431: Reduce thread count in driver to reduce out of order transmit processing.
Jul 11 2016, 8:23 PM

Jul 8 2016

gnn committed rS302474: On FreeBSD there is a setsockopt option SO_USER_COOKIE which allows.
On FreeBSD there is a setsockopt option SO_USER_COOKIE which allows
Jul 8 2016, 11:44 PM
gnn closed D7152: Add support for the user cookie to DTrace and TCP by committing rS302474: On FreeBSD there is a setsockopt option SO_USER_COOKIE which allows.
Jul 8 2016, 11:44 PM
gnn added a comment to D7152: Add support for the user cookie to DTrace and TCP.

I'll make that change in the commit.

Jul 8 2016, 11:35 PM
gnn updated D7152: Add support for the user cookie to DTrace and TCP.
Jul 8 2016, 2:02 AM
gnn added reviewers for D7152: Add support for the user cookie to DTrace and TCP: hiren, markj.
Jul 8 2016, 12:19 AM
gnn retitled D7152: Add support for the user cookie to DTrace and TCP from to Add support for the user cookie to DTrace and TCP.
Jul 8 2016, 12:18 AM

Jul 6 2016

gnn added reviewers for D7144: Corrected non-portable reuse of va_list in dt_printf(): markj, rwatson.
Jul 6 2016, 10:11 PM
gnn retitled D7144: Corrected non-portable reuse of va_list in dt_printf() from to Corrected non-portable reuse of va_list in dt_printf().
Jul 6 2016, 10:10 PM
gnn abandoned D7143: corrected non-portable reuse of va_list in dt_printf().
Jul 6 2016, 10:08 PM
gnn retitled D7143: corrected non-portable reuse of va_list in dt_printf() from to corrected non-portable reuse of va_list in dt_printf().
Jul 6 2016, 10:07 PM