From cebka@mailsupport.rambler.ru  Tue Dec 18 15:08:15 2007
Return-Path: <cebka@mailsupport.rambler.ru>
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 8AC3116A419
	for <FreeBSD-gnats-submit@freebsd.org>; Tue, 18 Dec 2007 15:08:15 +0000 (UTC)
	(envelope-from cebka@mailsupport.rambler.ru)
Received: from mailsupport.rambler.ru (mailsupport.rambler.ru [81.19.66.129])
	by mx1.freebsd.org (Postfix) with ESMTP id 3487913C4CE
	for <FreeBSD-gnats-submit@freebsd.org>; Tue, 18 Dec 2007 15:08:15 +0000 (UTC)
	(envelope-from cebka@mailsupport.rambler.ru)
Received: by mailsupport.rambler.ru (Postfix, from userid 6006)
	id 3FC21294; Tue, 18 Dec 2007 17:52:54 +0300 (MSK)
Message-Id: <20071218145254.3FC21294@mailsupport.rambler.ru>
Date: Tue, 18 Dec 2007 17:52:54 +0300 (MSK)
From: Vsevolod Stakhov <vsevolod@FreeBSD.org>
Reply-To: Vsevolod Stakhov <vsevolod@FreeBSD.org>
To: FreeBSD-gnats-submit@freebsd.org
Cc:
Subject: libmilter in FreeBSD is using select (3) by default that has FD_SETSIZE restriction on descriptor's number
X-Send-Pr-Version: 3.113
X-GNATS-Notify:

>Number:         118824
>Category:       kern
>Synopsis:       [libmilter] [patch] libmilter in FreeBSD is using select (3) by default that has FD_SETSIZE restriction on descriptor's number
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    gshapiro
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Tue Dec 18 15:10:00 UTC 2007
>Closed-Date:    Sat Mar 15 20:56:42 UTC 2008
>Last-Modified:  Sat Mar 15 21:00:04 UTC 2008
>Originator:     Vsevolod Stakhov
>Release:        FreeBSD 7.0-BETA1 amd64
>Organization:
>Environment:
System: FreeBSD mailsupport 7.0-BETA1 FreeBSD 7.0-BETA1 #0: Thu Oct 25 17:57:49 MSD 2007 citrin@mailsupport:/usr/obj/usr/src/sys/MAIL amd64

>Description:
Libmilter in FreeBSD is using select (3) by default that causes ERANGE
faults on loaded system, when number of descriptors that are used in
milter are too high to fit in FD_SETSIZE limit.
>How-To-Repeat:
>Fix:
Build libmilter with poll (3) support by default.

--- lib/libmilter/Makefile.orig	2007-11-23 17:38:51.000000000 +0300
+++ lib/libmilter/Makefile	2007-11-23 17:39:30.000000000 +0300
@@ -14,7 +14,7 @@
 .endif
 
 # User customizations to the sendmail build environment
-CFLAGS+=${SENDMAIL_CFLAGS}
+CFLAGS+=${SENDMAIL_CFLAGS} -DSM_CONF_POLL
 
 INCSDIR=${INCLUDEDIR}/libmilter
 INCS=	${SENDMAIL_DIR}/include/libmilter/mfapi.h \
>Release-Note:
>Audit-Trail:
State-Changed-From-To: open->feedback 
State-Changed-By: linimon 
State-Changed-When: Tue Dec 18 15:17:13 UTC 2007 
State-Changed-Why:  
To which port does this PR apply? 


Responsible-Changed-From-To: freebsd-bugs->linimon 
Responsible-Changed-By: linimon 
Responsible-Changed-When: Tue Dec 18 15:17:13 UTC 2007 
Responsible-Changed-Why:  

http://www.freebsd.org/cgi/query-pr.cgi?pr=118824 
State-Changed-From-To: feedback->open 
State-Changed-By: linimon 
State-Changed-When: Wed Dec 19 12:34:22 UTC 2007 
State-Changed-Why:  
Assign this to kern@. 


Responsible-Changed-From-To: linimon->freebsd-bugs 
Responsible-Changed-By: linimon 
Responsible-Changed-When: Wed Dec 19 12:34:22 UTC 2007 
Responsible-Changed-Why:  

http://www.freebsd.org/cgi/query-pr.cgi?pr=118824 

From: Vsevolod Stakhov <vsevolod@highsecure.ru>
To: linimon@FreeBSD.org
Cc: vsevolod@FreeBSD.org, freebsd-bugs@FreeBSD.org, 
 FreeBSD-gnats-submit@FreeBSD.org
Subject: Re: misc/118824: libmilter in FreeBSD is using select (3) by default
 that has FD_SETSIZE restriction on descriptor's number
Date: Wed, 19 Dec 2007 15:26:05 +0300

 linimon@FreeBSD.org wrote:
 > To which port does this PR apply?
 
 This issue is not related to FreeBSD ports, but to libmilter that is in 
 base system.
Responsible-Changed-From-To: freebsd-bugs->gshapiro 
Responsible-Changed-By: remko 
Responsible-Changed-When: Thu Dec 20 08:30:14 UTC 2007 
Responsible-Changed-Why:  
over to sendmail maintainer 

http://www.freebsd.org/cgi/query-pr.cgi?pr=118824 
State-Changed-From-To: open->patched 
State-Changed-By: gshapiro 
State-Changed-When: Sun Feb 17 05:15:01 UTC 2008 
State-Changed-Why:  
I've switched HEAD to using poll(2) instead of select(2).  I'll give it 
some time to settle before MFC'ing to RELENG_{7,6}. 

Thanks! 

http://www.freebsd.org/cgi/query-pr.cgi?pr=118824 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/118824: commit references a PR
Date: Sun, 17 Feb 2008 05:14:54 +0000 (UTC)

 gshapiro    2008-02-17 05:14:47 UTC
 
   FreeBSD src repository
 
   Modified files:
     lib/libmilter        Makefile 
   Log:
   Switch libmilter from select(2) to poll(2) so milters are not limited
   by the size of FD_SETSIZE.
   
   PR:             118824
   Submitted by:   vsevolod
   MFC after:      3 weeks
   
   Revision  Changes    Path
   1.8       +1 -0      src/lib/libmilter/Makefile
 _______________________________________________
 cvs-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/cvs-all
 To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
 
State-Changed-From-To: patched->closed 
State-Changed-By: gshapiro 
State-Changed-When: Sat Mar 15 20:56:17 UTC 2008 
State-Changed-Why:  
The change to use poll() instead of select() in libmilter has been 
MFC'ed to RELENG_7 and RELENG_6. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=118824 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/118824: commit references a PR
Date: Sat, 15 Mar 2008 20:54:14 +0000 (UTC)

 gshapiro    2008-03-15 20:54:08 UTC
 
   FreeBSD src repository
 
   Modified files:        (Branch: RELENG_7)
     lib/libmilter        Makefile 
   Log:
   MFC: Switch libmilter from select(2) to poll(2) so milters are not limited
        by the size of FD_SETSIZE.
   
        PR:             118824
        Submitted by:   vsevolod
   
   Revision  Changes    Path
   1.7.2.1   +1 -0      src/lib/libmilter/Makefile
 _______________________________________________
 cvs-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/cvs-all
 To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/118824: commit references a PR
Date: Sat, 15 Mar 2008 20:55:42 +0000 (UTC)

 gshapiro    2008-03-15 20:55:37 UTC
 
   FreeBSD src repository
 
   Modified files:        (Branch: RELENG_6)
     lib/libmilter        Makefile 
   Log:
   MFC: Switch libmilter from select(2) to poll(2) so milters are not limited
        by the size of FD_SETSIZE.
   
        PR:             118824
        Submitted by:   vsevolod
   
   Revision  Changes    Path
   1.5.2.2   +1 -0      src/lib/libmilter/Makefile
 _______________________________________________
 cvs-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/cvs-all
 To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
 
>Unformatted:
