unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: master 16bb108: Add a regression test for Bug#24325.
       [not found] ` <20201229172040.85BB72094D@vcs0.savannah.gnu.org>
@ 2020-12-29 17:55   ` Michael Albinus
  2020-12-29 18:10     ` Philipp Stephani
  0 siblings, 1 reply; 3+ messages in thread
From: Michael Albinus @ 2020-12-29 17:55 UTC (permalink / raw)
  To: emacs-devel; +Cc: Philipp Stephani

p.stephani2@gmail.com (Philipp Stephani) writes:

Hi Philipp,

> +(ert-deftest process-tests/fd-setsize-no-crash ()
> +  "Check that Emacs doesn't crash when trying to use more than
> +FD_SETSIZE file descriptors (Bug#24325)."
> +  (let ((sleep (executable-find "sleep"))
> +        ;; FD_SETSIZE is typically 1024 on Unix-like systems.
> +        (fd-setsize 1024)

This could be determined by "ulimit -n" or "getconf OPEN_MAX".

Best regards, Michael.



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: master 16bb108: Add a regression test for Bug#24325.
  2020-12-29 17:55   ` master 16bb108: Add a regression test for Bug#24325 Michael Albinus
@ 2020-12-29 18:10     ` Philipp Stephani
  2021-01-04 17:24       ` Robert Pluim
  0 siblings, 1 reply; 3+ messages in thread
From: Philipp Stephani @ 2020-12-29 18:10 UTC (permalink / raw)
  To: Michael Albinus; +Cc: Philipp Stephani, Emacs developers

Am Di., 29. Dez. 2020 um 18:56 Uhr schrieb Michael Albinus
<michael.albinus@gmx.de>:
>
> p.stephani2@gmail.com (Philipp Stephani) writes:
>
> Hi Philipp,
>
> > +(ert-deftest process-tests/fd-setsize-no-crash ()
> > +  "Check that Emacs doesn't crash when trying to use more than
> > +FD_SETSIZE file descriptors (Bug#24325)."
> > +  (let ((sleep (executable-find "sleep"))
> > +        ;; FD_SETSIZE is typically 1024 on Unix-like systems.
> > +        (fd-setsize 1024)
>
> This could be determined by "ulimit -n" or "getconf OPEN_MAX".
>

That's a different limit (resource limit vs. size of a file descriptor set).



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: master 16bb108: Add a regression test for Bug#24325.
  2020-12-29 18:10     ` Philipp Stephani
@ 2021-01-04 17:24       ` Robert Pluim
  0 siblings, 0 replies; 3+ messages in thread
From: Robert Pluim @ 2021-01-04 17:24 UTC (permalink / raw)
  To: Philipp Stephani; +Cc: Philipp Stephani, Michael Albinus, Emacs developers

Philipp Stephani <p.stephani2@gmail.com> writes:

> Am Di., 29. Dez. 2020 um 18:56 Uhr schrieb Michael Albinus
> <michael.albinus@gmx.de>:
>>
>> p.stephani2@gmail.com (Philipp Stephani) writes:
>>
>> Hi Philipp,
>>
>> > +(ert-deftest process-tests/fd-setsize-no-crash ()
>> > +  "Check that Emacs doesn't crash when trying to use more than
>> > +FD_SETSIZE file descriptors (Bug#24325)."
>> > +  (let ((sleep (executable-find "sleep"))
>> > +        ;; FD_SETSIZE is typically 1024 on Unix-like systems.
>> > +        (fd-setsize 1024)
>>
>> This could be determined by "ulimit -n" or "getconf OPEN_MAX".
>>
>
> That's a different limit (resource limit vs. size of a file descriptor set).

Itʼs possible to raise FD_SETSIZE at compile time on macOS (to 4096 if
memory serves), so the only reliable way to test this is to expose
FD_SETSIZE to lisp. poll(2) anyone? ;-)

Robert



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-01-04 17:24 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20201229172038.13382.38744@vcs0.savannah.gnu.org>
     [not found] ` <20201229172040.85BB72094D@vcs0.savannah.gnu.org>
2020-12-29 17:55   ` master 16bb108: Add a regression test for Bug#24325 Michael Albinus
2020-12-29 18:10     ` Philipp Stephani
2021-01-04 17:24       ` Robert Pluim

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).