Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 27 Dec 2010 19:38:26 +0000 (UTC)
From:      Robert Watson <rwatson@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r216742 - head/sys/netinet
Message-ID:  <201012271938.oBRJcQM6034698@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: rwatson
Date: Mon Dec 27 19:38:25 2010
New Revision: 216742
URL: http://svn.freebsd.org/changeset/base/216742

Log:
  Remove comment bemoaning the lack of an INP_INHASHLIST above in_pcbdrop();
  I fixed this in r189657 in early 2009, so the comment is OBE.
  
  Reviewed by:	bz
  MFC after:	3 days

Modified:
  head/sys/netinet/in_pcb.c

Modified: head/sys/netinet/in_pcb.c
==============================================================================
--- head/sys/netinet/in_pcb.c	Mon Dec 27 18:55:16 2010	(r216741)
+++ head/sys/netinet/in_pcb.c	Mon Dec 27 19:38:25 2010	(r216742)
@@ -1077,12 +1077,6 @@ in_pcbfree(struct inpcb *inp)
  * maintaining the invariant that so_pcb always points to a valid inpcb until
  * in_pcbdetach().
  *
- * XXXRW: An inp_lport of 0 is used to indicate that the inpcb is not on hash
- * lists, but can lead to confusing netstat output, as open sockets with
- * closed TCP connections will no longer appear to have their bound port
- * number.  An explicit flag would be better, as it would allow us to leave
- * the port number intact after the connection is dropped.
- *
  * XXXRW: Possibly in_pcbdrop() should also prevent future notifications by
  * in_pcbnotifyall() and in_pcbpurgeif0()?
  */



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201012271938.oBRJcQM6034698>