The NetBSD Project

CVS log for src/sys/kern/uipc_socket.c

[BACK] Up to [cvs.NetBSD.org] / src / sys / kern

Request diff between arbitrary revisions


Default branch: MAIN
Current tag: netbsd-5-1-2-RELEASE


Revision 1.177.4.3.2.1 / (download) - annotate - [select for diffs], Mon Aug 8 19:47:01 2011 UTC (2 years, 8 months ago) by riz
Branch: netbsd-5-1
CVS Tags: netbsd-5-1-4-RELEASE, netbsd-5-1-3-RELEASE, netbsd-5-1-2-RELEASE, netbsd-5-1-1-RELEASE
Changes since 1.177.4.3: +48 -68 lines
Diff to previous 1.177.4.3 (colored) next main 1.177.4.4 (colored)

Pull up following revision(s) (requested by bouyer in ticket #1644):
	sys/sys/socketvar.h: revision 1.126
	sys/kern/init_main.c: revision 1.433
	sys/kern/uipc_socket.c: revision 1.205
Fix kern/45093 as discussed on tech-kern@:
http://mail-index.netbsd.org/tech-kern/2011/06/17/msg010734.html
The cause of the problem is that the so_pendfree is processed with
the softnet_lock held at one point, and processing the list
calls sodoloanfree() which may kpause(). As the thread sleeps with
softnet_lock held, it ultimately cause a deadlock (see the PR or tech-kern
thread for details).
Although it should be possible to call sodopendfree() after releasing
the socket lock, it's not so easy to know where he socket lock is held and
where it's not, so we may hit the issue again later.
Add a kernel thread to handle the so_pendfree list, and wake up this
thread when adding mbufs to this list. Get rid of the various sodopendfree()
calls, hopefully fixing definitively the problem.

This form allows you to request diff's between any two revisions of a file. You may select a symbolic revision name using the selection box or you may type in a numeric name using the type-in text box.




CVSweb <webmaster@jp.NetBSD.org>