From owner-freebsd-current@FreeBSD.ORG Wed Jul 14 10:05:01 2004 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 9992016A4CE; Wed, 14 Jul 2004 10:05:01 +0000 (GMT) Received: from critter.freebsd.dk (critter.freebsd.dk [212.242.86.163]) by mx1.FreeBSD.org (Postfix) with ESMTP id B423243D46; Wed, 14 Jul 2004 10:05:00 +0000 (GMT) (envelope-from phk@critter.freebsd.dk) Received: from critter.freebsd.dk (localhost [127.0.0.1]) by critter.freebsd.dk (8.12.11/8.12.11) with ESMTP id i6EA4wO8038609; Wed, 14 Jul 2004 12:04:58 +0200 (CEST) (envelope-from phk@critter.freebsd.dk) To: Marcel Moolenaar From: "Poul-Henning Kamp" In-Reply-To: Your message of "Tue, 13 Jul 2004 19:46:23 PDT." <20040714024623.GB623@dhcp50.pn.xcllnt.net> Date: Wed, 14 Jul 2004 12:04:57 +0200 Message-ID: <38608.1089799497@critter.freebsd.dk> Sender: phk@critter.freebsd.dk cc: stable@freebsd.org cc: current@freebsd.org Subject: Re: [TEST/REVIEW/HEADSUP] tty drivers mega-patch X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 14 Jul 2004 10:05:01 -0000 In message <20040714024623.GB623@dhcp50.pn.xcllnt.net>, Marcel Moolenaar writes : >On Wed, Jul 14, 2004 at 12:09:47AM +0200, Poul-Henning Kamp wrote: >> >> http://phk.freebsd.dk/patch/tty.patch >> >> This patch removes 2500 lines of copy&paste insanity in tty drivers >> and generally tries to get things to be less confused & confusing. >> >> I need testers for all the different kinds of serial hardware we support. >> >> Please help test! > >boot -s is generally broken. No characters can be typed. This is >with syscons(4) and uart(4) and probably any other driver. Might >be related to the lack of NL->CRNL I have this fixed in a new patch I just uploaded. marcel: I don't think the kassert in uart which tripped you should be there any more. Can you try to remove it and see that nothing bad happens ? With respect to the "init"/"lock" devices: Drivers are not forced to them, they get created if ttymakeslaves() is called, and the driver is free not to do that. I added it to uart because I thought it was a sufficiently high-profile driver that we should have the cua/init/lock ability on it. (as opposed to ptys or dcons for instance). If you don't want it, I'll remove it again (but read on first). The traditional naming has been the inserted "i" and "l" which I agree is arcane. I'm not against changing it to ".init" and ".lock" if we can get concensus. I would prefer to stick to the "tty" and "cua" prefixes however. The plan is to make them on-demand-devices (clonable) so that they do not clutter up /dev but appear if you access them. That would leave only the ttyu0 and cuau0 devices. -- Poul-Henning Kamp | UNIX since Zilog Zeus 3.20 phk@FreeBSD.ORG | TCP/IP since RFC 956 FreeBSD committer | BSD since 4.3-tahoe Never attribute to malice what can adequately be explained by incompetence.