Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 31 Dec 1998 21:34:36 -0500
From:      Garance A Drosihn <drosih@rpi.edu>
To:        adrian@FreeBSD.ORG, freebsd-hackers@FreeBSD.ORG
Subject:   Re: auto-allocation of ptys
Message-ID:  <v04011700b2b1e326c60d@[128.113.24.47]>
In-Reply-To: <199812262138.WAA04817@ywing.creative.net.au>

next in thread | previous in thread | raw e-mail | index | archive | help
At 10:38 PM +0100 12/26/98, adrian@FreeBSD.ORG wrote:
> I got sick of running out of ptys while running X, and having to
> result in rebuilding a kernel with more ptys enabled.
>
> So, I modified sys/kern/tty_pty.c to dynamically allocate pt_ioctl
> and tty structs when a pty access is made.

> This patch breaks pstat -t (strangely enough :) ...

> It also will break devfs - I'm sure a devfs solution could be nutted
> out, but I'm still looking at it atm, ...

> The upshoot of all this of course is that you don't have to rebuild
> your kernel whenever you need more ptys. :-)

> (patch is at  http://www.freebsd.org/~adrian/tty_pty.c.diff  )

> Does anyone want to take a look at the idea and comment on it ?

I haven't looked at it, and wouldn't know enough about the internals
to comment on it at that level anyway.  Still, I'm always willing to
comment on interesting ideas, even if I don't know the details   :-)

I like the idea of not having to rebuild the kernel, but somehow I'm
not too comfortable with them being dynamic either.  Would it be
easier (on things like 'pstat' and devfs) if it were a fixed number
that was settable at bootup?  That way it would only be a system
reboot, instead of a kernel rebuild.

Just my 2 cents...

---
Garance Alistair Drosehn           =   gad@eclipse.its.rpi.edu
Senior Systems Programmer          or  drosih@rpi.edu
Rensselaer Polytechnic Institute

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-hackers" in the body of the message



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