# fd patch

- Thread: https://externals.io/message/1777
- Original archive: https://news-web.php.net/php.internals/1777
- Started by Uwe Schindler on 2003-05-20T11:15:51+00:00
- Messages: 1
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #1777 — Uwe Schindler — 2003-05-20T11:15:51+00:00

URL: https://externals.io/message/1777#1777

I agree +1

At 00:00 20.05.2003 -0700, you wrote:
>
>
> > […]
> exactly
> > […]
> release,
> > […]
>
>My only reservation is that it went in somewhat late in the RC cycle.
>This change has been needed for quite a while.  I know the issue itself
>has been around for quite a while, but threaded web servers have become
>much more common on Solaris in recent months and problems in our nsapi api
>have been worked out so PHP is now a viable iPlanet plugin.  This was
>never really the case before.  However, the last hurdle for decent iPlanet
>support is this patch.
>
>The second reason is performance.  By moving away from stdio streams we
>save a `stat()` call for every single file open.  Believe me, disk-touching
>syscalls is a killer.
>
>And finally, there has been wide support for testing this thing to death.
>We are definitely putting it through the ringer at Yahoo (hence the patch
>from Brian).  It is running on php.net and Sascha has been pounding it as
>well.  There may very well still be issues, but with a bit of dedicated
>testing, the performance benefits plus finally having real iPlanet support
>is worth a little extra pain as far as I am concerned.
>
>-Rasmus